On 11/06/2013 06:04 AM, Juan Quintela wrote:

s/allways dirty/always dirties/ in the subject

> So remove the flag argument and do it directly.  After this change, there is 
> nothing else using cpu_physical_memory_set_dirty_flags() so remove it.

Any reason you didn't line wrap the commit message?

> 
> Signed-off-by: Juan Quintela <quint...@redhat.com>
> ---
>  exec.c                         |  2 +-
>  include/exec/memory-internal.h | 11 ++---------
>  memory.c                       |  2 +-
>  3 files changed, 4 insertions(+), 11 deletions(-)

But the code itself is correct.  For 9 through 11:

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