[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL 7/7] vfio: fix mapping of MSIX bar
From: |
Kai Huang |
Subject: |
Re: [Qemu-devel] [PULL 7/7] vfio: fix mapping of MSIX bar |
Date: |
Sun, 19 Jan 2014 22:03:58 +0800 |
On Sat, Jan 18, 2014 at 3:25 AM, Alex Williamson
<address@hidden> wrote:
> From: Alexey Kardashevskiy <address@hidden>
>
> VFIO virtualizes MSIX table for the guest but not mapping the part of
> a BAR which contains an MSIX table. Since vfio_mmap_bar() mmaps chunks
> before and after the MSIX table, they have to be aligned to the host
> page size which may be TARGET_PAGE_MASK (4K) or 64K in case of PPC64.
>
> This fixes boundaries calculations to use the real host page size.
>
> Without the patch, the chunk before MSIX table may overlap with the MSIX
> table and mmap will fail in the host kernel. The result will be serious
> slowdown as the whole BAR will be emulated by QEMU.
>
> Signed-off-by: Alexey Kardashevskiy <address@hidden>
> Signed-off-by: Alex Williamson <address@hidden>
> ---
> hw/misc/vfio.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/hw/misc/vfio.c b/hw/misc/vfio.c
> index 432547c..8a1f1a1 100644
> --- a/hw/misc/vfio.c
> +++ b/hw/misc/vfio.c
> @@ -2544,7 +2544,7 @@ static void vfio_map_bar(VFIODevice *vdev, int nr)
> * potentially insert a direct-mapped subregion before and after it.
> */
> if (vdev->msix && vdev->msix->table_bar == nr) {
> - size = vdev->msix->table_offset & TARGET_PAGE_MASK;
> + size = vdev->msix->table_offset & qemu_host_page_mask;
> }
>
> strncat(name, " mmap", sizeof(name) - strlen(name) - 1);
> @@ -2556,8 +2556,8 @@ static void vfio_map_bar(VFIODevice *vdev, int nr)
> if (vdev->msix && vdev->msix->table_bar == nr) {
> unsigned start;
>
> - start = TARGET_PAGE_ALIGN(vdev->msix->table_offset +
> - (vdev->msix->entries *
> PCI_MSIX_ENTRY_SIZE));
> + start = HOST_PAGE_ALIGN(vdev->msix->table_offset +
> + (vdev->msix->entries * PCI_MSIX_ENTRY_SIZE));
>
Hi Alex,
I am new to vfio and qemu, and have some questions. Does MSIX have one
dedicated bar when qemu emulating the device? Looks your code maps
both the content before and after the MSIX table? If MSIX has
dedicated bar, I think we can just skip the MSIX bar, why do we need
to map the context before and after the MSIX table?
Thanks,
-Kai
> size = start < bar->size ? bar->size - start : 0;
> strncat(name, " msix-hi", sizeof(name) - strlen(name) - 1);
>
> --
> To unsubscribe from this list: send the line "unsubscribe kvm" in
> the body of a message to address@hidden
> More majordomo info at http://vger.kernel.org/majordomo-info.html
- [Qemu-devel] [PULL 0/7] vfio pull request, Alex Williamson, 2014/01/17
- [Qemu-devel] [PULL 1/7] vfio: Destroy memory regions, Alex Williamson, 2014/01/17
- [Qemu-devel] [PULL 2/7] vfio: warn if host device rom can't be read, Alex Williamson, 2014/01/17
- [Qemu-devel] [PULL 3/7] vfio: Do not reattempt a failed rom read, Alex Williamson, 2014/01/17
- [Qemu-devel] [PULL 4/7] vfio: Filter out bogus mappings, Alex Williamson, 2014/01/17
- [Qemu-devel] [PULL 5/7] vfio-pci: Fail initfn on DMA mapping errors, Alex Williamson, 2014/01/17
- [Qemu-devel] [PULL 6/7] kvm: initialize qemu_host_page_size, Alex Williamson, 2014/01/17
- [Qemu-devel] [PULL 7/7] vfio: fix mapping of MSIX bar, Alex Williamson, 2014/01/17
- Re: [Qemu-devel] [PULL 7/7] vfio: fix mapping of MSIX bar,
Kai Huang <=