Commit 7f464fc7 authored Jul 19, 2022 by Edgar Arriaga Committed by Edgar Arriaga García Jul 19, 2022

Fix for compaction vma batch not consuming after first send

This wrong condition causes it to only send the first batch in full
however, if the batch was not processed in full by process madvise
then it would bail out instead of resending at starting failure point.

Bug: 237600211
Test: am compact full <package> <uid>
Change-Id: Ibe8e8ecd7ff97a856e533363f6b7ded318968889

parent 84ce5aff

services/core/jni/com_android_server_am_CachedAppOptimizer.cpp

+1 −1

Original line number	Diff line number	Diff line
		@@ -113,7 +113,7 @@ uint64_t consumeBytes(VmaBatch& batch, uint64_t bytesToConsume) {

		uint64_t bytesConsumed = 0;
		while (bytesConsumed < bytesToConsume) {
		if (CC_UNLIKELY(batch.totalVmas > 0)) {
		if (CC_UNLIKELY(batch.totalVmas == 0)) {
		// No more vmas to consume
		break;
		}