Re: [Qemu-devel] [PATCHv3 1/9] move vector definitions to qemu-common.h

[Eric Blake <eblake@redhat.com>]

[-- Attachment #1: Type: text/plain, Size: 1066 bytes --]

On 03/21/2013 09:57 AM, Peter Lieven wrote:
> vector optimizations will now be used at various places
> not just in is_dup_page() in arch_init.c
> 
> this patch also adds a zero splat vector.
> 
> Signed-off-by: Peter Lieven <pl@kamp.de>
> ---
>  arch_init.c           |   20 --------------------
>  include/qemu-common.h |   24 ++++++++++++++++++++++++
>  2 files changed, 24 insertions(+), 20 deletions(-)

Appears to be unchanged from v2.  You can reduce a reviewer's time
commitment by adjusting the commit message to add in reviewed-by: lines
from the earlier review where the patch is merely reposted unchanged as
part of a larger series; conversely, if you carry forward reviewed-by
comments, be sure to drop all reviewed-by lines on the commits where you
make substantial fixes.  That way, people who looked at v2 know where to
concentrate efforts when reviewing v3.  At any rate:

Reviewed-by: Eric Blake <eblake@redhat.com>

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 621 bytes --]