22/07/2019 14:56, David Marchand: > Following the issues reported by Jerin and the discussion that emerged > from it, here are fixes to restore and document the behavior of the EAL > and the pci bus driver. > > I pondered all the arguments and tried to have the less changes > possible. > I can't find a need for a flag to just announce support of physical > addresses from the pmd point of view. > So it ended up with something really close to what Jerin had suggested. > > But the problem is that this is still unfinished wrt the documentation. > I will be offline for 10 days and we need this to move forward, so > sending > anyway. > > Changelog since v3: > - fixed typos in patch 2, > - updated patch 3 title, > - moved and reworded comments in the note section in patch 4, > > Changelog since v2 (Jerin): > - Patch 2/4 - Remove personal appeals in log messages(Anatoly) > - Patch 4/4 - Added documentation (Anatoly) > > Changelog since v1 (Jerin): > - Changed RTE_PCI_DRV_IOVA_AS_VA flag name as RTE_PCI_DRV_NEED_IOVA_AS_VA > (patch 3/4) > - Changed IOVA mode as VA for default case(patch 4/4) with documentation > - Tested the patch series on octeontx2 platform
Applied, thanks Jerin, Anatoly and David for converging on a documented solution together.
