>06/08/2018 10:43, Takeshi Yoshimura: >> Commit 73a639085938 ("vfio: allow to map other memory regions") >> introduced a bug in sPAPR IOMMU mapping. The commit removed >necessary >> ioctl with VFIO_IOMMU_SPAPR_REGISTER_MEMORY. Also, >vfio_spapr_map_walk >> should call vfio_spapr_dma_do_map instead of >vfio_spapr_dma_mem_map. > >This is fixing an old patch: > INVALID URI REMOVED >k_commit_-3Fid-3D73a639085938&d=DwICAg&c=jf_iaSHvJObTbx-siA1ZOg&r=EZR >6Jx10q0q3dTopeH3WIQ&m=JqmiDyEAYS0sG-jH6aSYikeyXYLr12jnaWDUpBN1mgI&s=7 >bvq2dyXbquauFqqhuOTPVOpoqEPKg8MoncacU4OdJU&e= > >> Fixes: 73a639085938 ("vfio: allow to map other memory regions") > >You probably want it to be backported in previous release, >so you need to add Cc: sta...@dpdk.org > >> Signed-off-by: Takeshi Yoshimura <t...@jp.ibm.com> > >It is common to have bugs in sPAPR implementation. >How can we be sure this one is OK? >Should it be added in last minute of 18.08? >Or can it wait 18.11? > >Adding Anatoly and Chao who are maintainers to decide. >
OK. I will resend the patch with Cc: sta...@dpdk.org.