On 11/06/2013 06:04 AM, Juan Quintela wrote:
> Signed-off-by: Juan Quintela <quint...@redhat.com>
> ---
>  exec.c                         | 3 ++-
>  include/exec/memory-internal.h | 6 ++++++
>  2 files changed, 8 insertions(+), 1 deletion(-)
> 
> diff --git a/exec.c b/exec.c
> index 7cf5634..36b0a66 100644
> --- a/exec.c
> +++ b/exec.c
> @@ -1374,9 +1374,10 @@ found:
>  static void notdirty_mem_write(void *opaque, hwaddr ram_addr,
>                                 uint64_t val, unsigned size)
>  {
> +
>      int dirty_flags;

Is this whitespace addition spurious?

...
> 
> +static inline bool cpu_physical_memory_get_dirty_flag(ram_addr_t addr,
> +                                                     int dirty_flag)

Indentation off by one.

But whitespace doesn't impact the patch, so you can mark 1 through 6 as:

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

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

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to