qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH v3 04/16] exec: ram_block_discard_range


From: Laurent Vivier
Subject: Re: [Qemu-devel] [PATCH v3 04/16] exec: ram_block_discard_range
Date: Sun, 26 Feb 2017 20:23:24 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.7.0

On 24/02/2017 19:28, Dr. David Alan Gilbert (git) wrote:
> From: "Dr. David Alan Gilbert" <address@hidden>
> 
> Create ram_block_discard_range in exec.c to replace
> postcopy_ram_discard_range and most of ram_discard_range.
> 
> Those two routines are a bit of a weird combination, and
> ram_discard_range is about to get more complex for hugepages.
> It's OS dependent code (so shouldn't be in migration/ram.c) but
> it needs quite a bit of the innards of RAMBlock so doesn't belong in
> the os*.c.
> 
> Signed-off-by: Dr. David Alan Gilbert <address@hidden>
> Reviewed-by: Juan Quintela <address@hidden>
> ---
>  exec.c                           | 54 
> ++++++++++++++++++++++++++++++++++++++++
>  include/exec/cpu-common.h        |  1 +
>  include/migration/postcopy-ram.h |  7 ------
>  migration/postcopy-ram.c         | 30 +---------------------
>  migration/ram.c                  | 24 +++---------------
>  migration/trace-events           |  2 +-
>  6 files changed, 60 insertions(+), 58 deletions(-)
> 
> diff --git a/exec.c b/exec.c
> index 865a1e8..ddbc314 100644
> --- a/exec.c
> +++ b/exec.c
> @@ -45,6 +45,7 @@
>  #include "exec/address-spaces.h"
>  #include "sysemu/xen-mapcache.h"
>  #include "trace-root.h"
> +
>  #endif

It seems a blank line has "migrated" here from PATCH v2 11/16. ;)

Reviewed-by: Laurent Vivier <address@hidden>



reply via email to

[Prev in Thread] Current Thread [Next in Thread]