Re: [PATCH v2 3/5] vfio-pci/zdev: define the vfio_zdev header

2020-10-05 Thread Cornelia Huck
On Mon, 5 Oct 2020 12:16:10 -0400 Matthew Rosato wrote: > On 10/5/20 12:01 PM, Cornelia Huck wrote: > > On Mon, 5 Oct 2020 09:52:25 -0400 > > Matthew Rosato wrote: > > > >> On 10/2/20 5:44 PM, Alex Williamson wrote: > > > >>> Can you discuss why a region with embedded capability chain is

Re: [PATCH v2 3/5] vfio-pci/zdev: define the vfio_zdev header

2020-10-05 Thread Matthew Rosato
On 10/5/20 12:01 PM, Cornelia Huck wrote: On Mon, 5 Oct 2020 09:52:25 -0400 Matthew Rosato wrote: On 10/2/20 5:44 PM, Alex Williamson wrote: Can you discuss why a region with embedded capability chain is a better solution than extending the VFIO_DEVICE_GET_INFO ioctl to support a capability

Re: [PATCH v2 3/5] vfio-pci/zdev: define the vfio_zdev header

2020-10-05 Thread Cornelia Huck
On Mon, 5 Oct 2020 09:52:25 -0400 Matthew Rosato wrote: > On 10/2/20 5:44 PM, Alex Williamson wrote: > > Can you discuss why a region with embedded capability chain is a better > > solution than extending the VFIO_DEVICE_GET_INFO ioctl to support a > > capability chain and providing this info th

Re: [PATCH v2 3/5] vfio-pci/zdev: define the vfio_zdev header

2020-10-05 Thread Matthew Rosato
On 10/2/20 5:44 PM, Alex Williamson wrote: On Fri, 2 Oct 2020 16:00:42 -0400 Matthew Rosato wrote: We define a new device region in vfio.h to be able to get the ZPCI CLP information by reading this region from userspace. We create a new file, vfio_zdev.h to define the structure of the new re

Re: [PATCH v2 3/5] vfio-pci/zdev: define the vfio_zdev header

2020-10-02 Thread Alex Williamson
On Fri, 2 Oct 2020 16:00:42 -0400 Matthew Rosato wrote: > We define a new device region in vfio.h to be able to get the ZPCI CLP > information by reading this region from userspace. > > We create a new file, vfio_zdev.h to define the structure of the new > region defined in vfio.h > > Signed-o

[PATCH v2 3/5] vfio-pci/zdev: define the vfio_zdev header

2020-10-02 Thread Matthew Rosato
We define a new device region in vfio.h to be able to get the ZPCI CLP information by reading this region from userspace. We create a new file, vfio_zdev.h to define the structure of the new region defined in vfio.h Signed-off-by: Matthew Rosato --- include/uapi/linux/vfio.h | 5 ++ incl