Re: Please revert upstream commit e4849aff1e16 ("MIPS: SiByte: Enable swiotlb ...") in v4.4.y and v4.14.y

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, 16 Sep 2019, Greg Kroah-Hartman wrote:

> > Upstream commit e4849aff1e16 ("MIPS: SiByte: Enable swiotlb for SWARM,
> > LittleSur and BigSur") results in build failures in v4.4.y and v4.14.y.
> > 
> > make bigsur_defconfig:
> > 
> > warning: (SIBYTE_SWARM && SIBYTE_SENTOSA && SIBYTE_BIGSUR && SWIOTLB_XEN && AMD_IOMMU) selects SWIOTLB which has unmet direct dependencies (CAVIUM_OCTEON_SOC || MACH_LOONGSON64 && CPU_LOONGSON3 || NLM_XLP_BOARD || NLM_XLR_BOARD)
> > warning: (SIBYTE_SWARM && SIBYTE_SENTOSA && SIBYTE_BIGSUR && SWIOTLB_XEN && AMD_IOMMU) selects SWIOTLB which has unmet direct dependencies (CAVIUM_OCTEON_SOC || MACH_LOONGSON64 && CPU_LOONGSON3 || NLM_XLP_BOARD || NLM_XLR_BOARD)
> > 
> > and the actual build:
> > 
> > lib/swiotlb.o: In function `swiotlb_tbl_map_single':
> > (.text+0x1c0): undefined reference to `iommu_is_span_boundary'
> > Makefile:1021: recipe for target 'vmlinux' failed
> > 
> > Please revert.
> 
> Shouldn't I also revert it in 4.9.y?

 NB there's certainly a dependency of some kind between this series and 
Christoph's swiotlb rework, which preceded it just a little, so I'm not at 
all surprised it can't be backported to the beginning of time without also 
pulling a pile of other changes.

 Thank you both for taking care of this revert.

  Maciej



[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux