Re: [PATCH] eal/vfio: fix sPAPR IOMMU mapping
From: Takeshi T Yoshimura <hidden>
Date: 2018-08-07 02:28:42
06/08/2018 10:43, Takeshi Yoshimura:quoted
Commit 73a639085938 ("vfio: allow to map other memory regions") introduced a bug in sPAPR IOMMU mapping. The commit removednecessaryquoted
ioctl with VFIO_IOMMU_SPAPR_REGISTER_MEMORY. Also,vfio_spapr_map_walkquoted
should call vfio_spapr_dma_do_map instead ofvfio_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=quoted
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: stable@dpdk.orgquoted
Signed-off-by: Takeshi Yoshimura <redacted>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: stable@dpdk.org.