Re: [dpdk-dev] [PATCH v3 0/6] use IOVAs check based on DMA mask

[Alejandro Lucero <alejandro.lucero@netronome.com>]

On Tue, Oct 30, 2018 at 2:14 PM Burakov, Anatoly <anatoly.burakov@intel.com>
wrote:

> On 30-Oct-18 2:04 PM, Alejandro Lucero wrote:
> >
> >
> > On Tue, Oct 30, 2018 at 12:37 PM Alejandro Lucero
> > <alejandro.lucero@netronome.com <mailto:alejandro.lucero@netronome.com>>
>
> > wrote:
> >
> >
> >
> >     On Tue, Oct 30, 2018 at 12:22 PM Lin, Xueqin <xueqin.lin@intel.com
> >     <mailto:xueqin.lin@intel.com>> wrote:
> >
> >         Some found on some our servers:____
> >
> >         If  not add ”intel_iommu=on iommu=pt” in /boot/grub2/grub.cfg
> >         file, then reboot to make it effective.____
> >
> >         18.11 rc1: Success to setup testpmd  and secondary process.____
> >
> >         __ __
> >
> >         If  add  ”intel_iommu=on iommu=pt” in /boot/grub2/grub.cfg file,
> >         then reboot to make it effective.____
> >
> >         18.11 rc1:  Fail to setup testpmd  and secondary process.____
> >
> >         18.11 rc1+ dma_mask_fix patch: success to setup testpmd, but
> >         fail to setup secondary process.____
> >
> >         __ __
> >
> >         Maybe ”intel_iommu=on iommu=pt” enable or not result in our test
> >         gap. ____
> >
> >         Most of our team servers should enable the IOMMU for VT-d and
> >         vfio test. ____
> >
> >         __
> >
> >
> >     It makes sense because the problem is when the IOVA mode is set
> >     inside drivers/bus/pci/linux/pci.c and if there is not IOMMU, not
> >     call to rte_eal_check_dma_mask at all.
> >
> >         __
> >
> >         Best regards,____
> >
> >         Xueqin____
> >
> >         __ __
> >
> >         *From:*Alejandro Lucero [mailto:alejandro.lucero@netronome.com
> >         <mailto:alejandro.lucero@netronome.com>]
> >         *Sent:* Tuesday, October 30, 2018 6:38 PM
> >         *To:* Lin, Xueqin <xueqin.lin@intel.com
> >         <mailto:xueqin.lin@intel.com>>
> >         *Cc:* Yao, Lei A <lei.a.yao@intel.com
> >         <mailto:lei.a.yao@intel.com>>; Thomas Monjalon
> >         <thomas@monjalon.net <mailto:thomas@monjalon.net>>; dev
> >         <dev@dpdk.org <mailto:dev@dpdk.org>>; Xu, Qian Q
> >         <qian.q.xu@intel.com <mailto:qian.q.xu@intel.com>>; Burakov,
> >         Anatoly <anatoly.burakov@intel.com
> >         <mailto:anatoly.burakov@intel.com>>; Yigit, Ferruh
> >         <ferruh.yigit@intel.com <mailto:ferruh.yigit@intel.com>>; Zhang,
> >         Qi Z <qi.z.zhang@intel.com <mailto:qi.z.zhang@intel.com>>
> >         *Subject:* Re: [dpdk-dev] [PATCH v3 0/6] use IOVAs check based
> >         on DMA mask____
> >
> >         __ __
> >
> >         __ __
> >
> >         On Tue, Oct 30, 2018 at 10:34 AM Lin, Xueqin
> >         <xueqin.lin@intel.com <mailto:xueqin.lin@intel.com>> wrote:____
> >
> >             Hi Lucero,____
> >
> >             ____
> >
> >             No, we have reproduced multi-process issues(include
> >             symmetric_mp, simple_mp, hotplug_mp, multi-process unit
> >             test… )on most of our servers. ____
> >
> >             It is also strange that 1~2 servers don’t have the issue.____
> >
> >             ____
> >
> >         __ __
> >
> >         Yes, you are right. I could execute it but it was due to how
> >         this problem triggers. ____
> >
> >         I think I can fix this and at the same time solving properly the
> >         initial issue without any limitation like that potential race
> >         condition I mentioned. ____
> >
> >         I can give you a patch to try in a couple of hours. ____
> >
> >         __
> >
> >
> > Hi Lin,
> >
> > Can you try the patch attached?
> >
> > Thanks
> >
> Hi Alejandro,
>
> Attachments are not supported on the mailing list :)
>

Apologies. I should have sent it just to Lin.


>
> --
> Thanks,
> Anatoly
>