Re: [PATCH] make vfio and DAX cache work together by skipping virtio fs cache section during vfio memory region add

[Laurent Vivier <lvivier@redhat.com>]

On 26/04/2021 07:45, Edge NFV wrote:
> Signed-off-by: Edge NFV <edgenfv@gmail.com>

You must use your real name for a patch submission:

https://wiki.qemu.org/Contribute/SubmitAPatch#Patch_emails_must_include_a_Signed-off-by:_line

https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/SubmittingPatches?id=f6f94e2ab1b33f0082ac22d71f66385a60d8157f#n297

And CC: maintainer of the file:

$ ./scripts/get_maintainer.pl -f hw/vfio/common.c
Alex Williamson <alex.williamson@redhat.com> (supporter:VFIO)
qemu-devel@nongnu.org (open list:All patches CC here)

If you want to submit it to qemu-trivial, don't send a separate mail, but cc: also
qemu-trivial.

Thanks,
Laurent

> ---
>  hw/vfio/common.c | 9 +++++++++
>  1 file changed, 9 insertions(+)
> 
> diff --git a/hw/vfio/common.c b/hw/vfio/common.c
> index ae5654fcdb..83e15bf7a3 100644
> --- a/hw/vfio/common.c
> +++ b/hw/vfio/common.c
> @@ -668,6 +668,15 @@ static void vfio_listener_region_add(MemoryListener *listener,
>                  int128_get64(int128_sub(section->size, int128_one())));
>          return;
>      }
> +    
> +    /* Do not add virtio fs cache section */                  
> +    if (!strcmp(memory_region_name(section->mr), "virtio-fs-cache")) {
> +        trace_vfio_listener_region_add_skip(
> +                section->offset_within_address_space,
> +                section->offset_within_address_space +
> +                int128_get64(int128_sub(section->size, int128_one())));
> +        return;
> +    }  
>  
>      if (unlikely((section->offset_within_address_space &
>                    ~qemu_real_host_page_mask) !=
>