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

[Wan Junjie]

On Thu, May 13, 2021 at 6:24 PM Xia, Chenbo <chenbo.xia at intel.com> wrote:
>
> > -----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
>
>

Hi Chenbo,

I am not available for this now. Please revert it.
Will dig it later.

Thanks,
Junjie