i'm sorry ，it should be memory_region_is_ram_device in new version qemu , memory_region_is_skip_dump is in older version qemu.
芦志朋 luzhipeng IT开发工程师 IT Development Engineer 操作系统产品部/中心研究院/系统产品 OS Product Dept./Central R＆D Institute/System Product 四川省成都市天府大道中段800号 E: [email protected] www.zte.com.cn 原始邮件 发件人：王广10165992 收件人：芦志朋10108272 抄送人：杨斌10080747 日 期 ：2017年04月10日 09:05 主 题 ：转发: Re: [Qemu-devel] [PATCH] vhost: skip RAM device memory sections 发件人： ＜[email protected]＞ 收件人：王广10165992 ＜[email protected]＞ 抄送人：芦志朋10108272 ＜[email protected]＞ 日 期 ：2017年04月08日 01:24 主 题 ：Re: [Qemu-devel] [PATCH] vhost: skip RAM device memory sections On 08/04/2017 09:16, Wang guang wrote: ＞ From: ZhiPeng Lu ＜[email protected]＞ ＞ ＞ A RAM device represents a mapping to a physical device, such as to a PCI ＞ * MMIO BAR of an vfio-pci assigned device. ＞ Vhost listens to this region,and increases the region's reference count ＞ while passthrough?for?network adapters (Physical Function, PF or Virtual Function, VF). ＞ After detaching network adapters with vhost backend dirver or vhost user dirver, ＞ it unregister vhost listen function by memory_listener_unregister. ＞ After detaching the passthrough pf or vf, ＞ the RAM device region's reference by vhost listener increated can not be released, ＞ due to vhost listen function does not exist.So let's just skip RAM device memory. ＞ ＞ Signed-off-by: ZhiPeng Lu ＜[email protected]＞ ＞ --- ＞ hw/virtio/vhost.c | 3 ++- ＞ 1 file changed, 2 insertions(+), 1 deletion(-) ＞ ＞ diff --git a/hw/virtio/vhost.c b/hw/virtio/vhost.c ＞ index 613494d..c1ff98f 100644 ＞ --- a/hw/virtio/vhost.c ＞ +++ b/hw/virtio/vhost.c ＞ @@ -611,7 +611,8 @@ static void vhost_set_memory(MemoryListener *listener, ＞ static bool vhost_section(MemoryRegionSection *section) ＞ { ＞ return memory_region_is_ram(section-＞mr) && ＞ - !memory_region_is_rom(section-＞mr) ＞ + !memory_region_is_rom(section-＞mr) && ＞ + !memory_region_is_skip_dump(section-＞mr) ＞ } Why not memory_region_is_ram_device? Paolo
