Re: [dpdk-dev] [PATCH v5 1/1] vfio: modify spapr iommu support to use static window sizing

2020-11-07 Thread Thomas Monjalon
06/11/2020 23:16, David Christensen: > On 11/4/20 11:12 PM, Thomas Monjalon wrote: > > 04/11/2020 23:25, David Christensen: > >> On 11/4/20 1:02 PM, Thomas Monjalon wrote: > >>> 04/11/2020 22:00, David Christensen: > On 11/4/20 11:43 AM, Thomas Monjalon wrote: > >> Signed-off-by: David Chr

Re: [dpdk-dev] [PATCH v5 1/1] vfio: modify spapr iommu support to use static window sizing

2020-11-06 Thread David Christensen
On 11/4/20 11:12 PM, Thomas Monjalon wrote: 04/11/2020 23:25, David Christensen: On 11/4/20 1:02 PM, Thomas Monjalon wrote: 04/11/2020 22:00, David Christensen: On 11/4/20 11:43 AM, Thomas Monjalon wrote: Signed-off-by: David Christensen Acked-by: Anatoly Burakov --- -#ifdef VFIO_IOMMU_SPA

Re: [dpdk-dev] [PATCH v5 1/1] vfio: modify spapr iommu support to use static window sizing

2020-11-04 Thread Thomas Monjalon
04/11/2020 23:25, David Christensen: > On 11/4/20 1:02 PM, Thomas Monjalon wrote: > > 04/11/2020 22:00, David Christensen: > >> > >> On 11/4/20 11:43 AM, Thomas Monjalon wrote: > Signed-off-by: David Christensen > Acked-by: Anatoly Burakov > --- > -#ifdef VFIO_IOMMU_SPAPR_INFO

Re: [dpdk-dev] [PATCH v5 1/1] vfio: modify spapr iommu support to use static window sizing

2020-11-04 Thread David Christensen
On 11/4/20 1:02 PM, Thomas Monjalon wrote: 04/11/2020 22:00, David Christensen: On 11/4/20 11:43 AM, Thomas Monjalon wrote: Signed-off-by: David Christensen Acked-by: Anatoly Burakov --- -#ifdef VFIO_IOMMU_SPAPR_INFO_DDW - /* try possible page_shift and levels for workaround

Re: [dpdk-dev] [PATCH v5 1/1] vfio: modify spapr iommu support to use static window sizing

2020-11-04 Thread Thomas Monjalon
04/11/2020 22:00, David Christensen: > > On 11/4/20 11:43 AM, Thomas Monjalon wrote: > >> Signed-off-by: David Christensen > >> Acked-by: Anatoly Burakov > >> --- > >> -#ifdef VFIO_IOMMU_SPAPR_INFO_DDW > >> - /* try possible page_shift and levels for workaround */ > >> + /* if

Re: [dpdk-dev] [PATCH v5 1/1] vfio: modify spapr iommu support to use static window sizing

2020-11-04 Thread David Christensen
On 11/4/20 11:43 AM, Thomas Monjalon wrote: Signed-off-by: David Christensen Acked-by: Anatoly Burakov --- -#ifdef VFIO_IOMMU_SPAPR_INFO_DDW - /* try possible page_shift and levels for workaround */ + /* if at first we don't succeed, try more levels */

Re: [dpdk-dev] [PATCH v5 1/1] vfio: modify spapr iommu support to use static window sizing

2020-11-04 Thread Thomas Monjalon
03/11/2020 23:05, David Christensen: > The SPAPR IOMMU requires that a DMA window size be defined before memory > can be mapped for DMA. Current code dynamically modifies the DMA window > size in response to every new memory allocation which is potentially > dangerous because all existing mappings

[dpdk-dev] [PATCH v5 1/1] vfio: modify spapr iommu support to use static window sizing

2020-11-03 Thread David Christensen
The SPAPR IOMMU requires that a DMA window size be defined before memory can be mapped for DMA. Current code dynamically modifies the DMA window size in response to every new memory allocation which is potentially dangerous because all existing mappings need to be unmapped/remapped in order to resi