Re: [dpdk-dev] [PATCH v2] pci_vfio: Support 64KB kernel page_size with vfio-pci driver

2018-11-16 Thread Burakov, Anatoly
On 16-Nov-18 2:34 AM, Tone Zhang (Arm Technology China) wrote: Hi Anatoly, I have some comments. -Original Message- From: Tone Zhang (Arm Technology China) Sent: Thursday, November 15, 2018 8:49 AM To: Burakov, Anatoly ; dev@dpdk.org Cc: Gavin Hu (Arm Technology China) ; Honnappa Nagar

Re: [dpdk-dev] [PATCH v2] pci_vfio: Support 64KB kernel page_size with vfio-pci driver

2018-11-15 Thread Tone Zhang (Arm Technology China)
Hi Anatoly, I have some comments. > -Original Message- > From: Tone Zhang (Arm Technology China) > Sent: Thursday, November 15, 2018 8:49 AM > To: Burakov, Anatoly ; dev@dpdk.org > Cc: Gavin Hu (Arm Technology China) ; Honnappa > Nagarahalli ; Steve Capper > ; nd > Subject: RE: [PATCH v2

Re: [dpdk-dev] [PATCH v2] pci_vfio: Support 64KB kernel page_size with vfio-pci driver

2018-11-14 Thread Tone Zhang (Arm Technology China)
Hi Anatoly, Sorry for the late response. > -Original Message- > From: Burakov, Anatoly > Sent: Friday, November 9, 2018 8:15 PM > To: Tone Zhang (Arm Technology China) ; > dev@dpdk.org > Cc: Gavin Hu (Arm Technology China) ; Honnappa > Nagarahalli ; Steve Capper > ; nd > Subject: Re: [P

Re: [dpdk-dev] [PATCH v2] pci_vfio: Support 64KB kernel page_size with vfio-pci driver

2018-11-09 Thread Burakov, Anatoly
On 09-Nov-18 5:57 AM, tone.zhang wrote: With a larger PAGE_SIZE it is possible for the MSI table to very close to the end of the BAR s.t. when we align the start and end of the MSI table to the PAGE_SIZE, the end offset of the MSI table is out of the PCI BAR boundary. This patch addresses the is

[dpdk-dev] [PATCH v2] pci_vfio: Support 64KB kernel page_size with vfio-pci driver

2018-11-08 Thread tone.zhang
With a larger PAGE_SIZE it is possible for the MSI table to very close to the end of the BAR s.t. when we align the start and end of the MSI table to the PAGE_SIZE, the end offset of the MSI table is out of the PCI BAR boundary. This patch addresses the issue by comparing both the start and the en