[dpdk-dev] [PATCH v2] vhost: avoid iotlb mempool allocation while IOMMU disabled

[Xia, Chenbo]

> -----Original Message-----
> From: David Marchand <david.marchand at redhat.com>
> Sent: Thursday, May 13, 2021 4:39 PM
> To: Xia, Chenbo <chenbo.xia at intel.com>; Maxime Coquelin
> <maxime.coquelin at redhat.com>
> Cc: Wan Junjie <wanjunjie at bytedance.com>; dev at dpdk.org; Peng He
> <hepeng.0320 at bytedance.com>; Zhihong Wang <wangzhihong.wzh at bytedance.com>
> Subject: Re: [dpdk-dev] [PATCH v2] vhost: avoid iotlb mempool allocation while
> IOMMU disabled
> 
> On Wed, Apr 7, 2021 at 9:10 AM Xia, Chenbo <chenbo.xia at intel.com> wrote:
> >
> > > -----Original Message-----
> > > From: Wan Junjie <wanjunjie at bytedance.com>
> > > Sent: Tuesday, February 2, 2021 4:15 PM
> > > To: Xia, Chenbo <chenbo.xia at intel.com>; Maxime Coquelin
> > > <maxime.coquelin at redhat.com>
> > > Cc: dev at dpdk.org; Wan Junjie <wanjunjie at bytedance.com>; Peng He
> > > <hepeng.0320 at bytedance.com>; Zhihong Wang <wangzhihong.wzh at bytedance.com>
> > > Subject: [PATCH v2] vhost: avoid iotlb mempool allocation while IOMMU
> disabled
> > >
> > > If vhost device's IOMMU feature is disabled, iotlb mempool allocation
> > > is unnecessary.
> > >
> > > Reported-by: Peng He <hepeng.0320 at bytedance.com>
> > > Signed-off-by: Wan Junjie <wanjunjie at bytedance.com>
> > > Reviewed-by: Zhihong Wang <wangzhihong.wzh at bytedance.com>
> > > ---
> > > 2.11.0
> >
> > Patch applied to next-virtio/main with conflict resolved, Thanks
> 
> This patch triggered a regression reported by Red Hat QE.
> https://bugs.dpdk.org/show_bug.cgi?id=703
> 
> Reverting the patch is enough to fix the regression.
> 
> I'll let you guys decide on the next step for 21.05.
> 
> 
> --
> David Marchand

I cannot reproduce this issue on my machine.

@Wan Junjie Could you reproduce this? I prefer to revert the patch if things
are not very clear as it is only a minor optimization.

Thanks,
Chenbo