[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL 08/10] memory-mapping: skip non-volatile memory r
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PULL 08/10] memory-mapping: skip non-volatile memory regions in GuestPhysBlockList |
Date: |
Tue, 6 Nov 2018 11:04:43 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.0 |
On 05/11/2018 16:37, Laszlo Ersek wrote:
> On 10/30/18 20:50, Paolo Bonzini wrote:
>> From: Marc-André Lureau <address@hidden>
>>
>> GuestPhysBlockList is currently used to produce dumps. Given the size
>> and the typical usage of NVDIMM for storage, they are not a good idea
>> to have in the dumps. We may want to have an extra dump option to
>> include them. For now, skip non-volatile regions.
>>
>> The TCG memory clear function is going to use the GuestPhysBlockList
>> as well, and will thus skip NVDIMM for similar reasons.
>>
>> Cc: address@hidden
>> Signed-off-by: Marc-André Lureau <address@hidden>
>> Message-Id: <address@hidden>
>> Signed-off-by: Paolo Bonzini <address@hidden>
>> ---
>> memory_mapping.c | 3 ++-
>> 1 file changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/memory_mapping.c b/memory_mapping.c
>> index 775466f..724dd0b 100644
>> --- a/memory_mapping.c
>> +++ b/memory_mapping.c
>> @@ -206,7 +206,8 @@ static void guest_phys_blocks_region_add(MemoryListener
>> *listener,
>>
>> /* we only care about RAM */
>> if (!memory_region_is_ram(section->mr) ||
>> - memory_region_is_ram_device(section->mr)) {
>> + memory_region_is_ram_device(section->mr) ||
>> + memory_region_is_nonvolatile(section->mr)) {
>> return;
>> }
>>
>>
>
> This patch misses my R-b, and (in chronological order) DavidH's, from:
>
> http://mid.mail-archive.com/address@hidden
> http://mid.mail-archive.com/address@hidden
Fixed, thanks.
Paolo