[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 6/9] cosmetic changes preparing for the followin
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 6/9] cosmetic changes preparing for the following patches |
Date: |
Wed, 04 Feb 2015 13:51:10 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.4.0 |
On 04/02/2015 04:10, Fam Zheng wrote:
>> > - return;
> Other changes are equivalent, but not quite for this one. But I think it is
> still correct, so:
>
> Reviewed-by: Fam Zheng <address@hidden>
True, this is not entirely cosmetic. Still, the loop is guaranteed to
only hit one block, because of the check on offset.
Removing the "return;" matches what we do in the RAM_PREALLOC case.
Paolo
[Qemu-devel] [PATCH 6/9] cosmetic changes preparing for the following patches, Paolo Bonzini, 2015/02/03
[Qemu-devel] [PATCH 8/9] exec: convert ram_list to QLIST, Paolo Bonzini, 2015/02/03
[Qemu-devel] [PATCH 2/9] exec: make iotlb RCU-friendly, Paolo Bonzini, 2015/02/03
[Qemu-devel] [PATCH 7/9] rcu: prod call_rcu thread when calling synchronize_rcu, Paolo Bonzini, 2015/02/03
[Qemu-devel] [PATCH 9/9] Convert ram_list to RCU, Paolo Bonzini, 2015/02/03