On Sun, Mar 27, 2022 at 4:37 PM Halil Pasic pasic@linux.ibm.com wrote:
For the record, I believe that the partial revert proposed here https://www.spinics.net/lists/linux-wireless/msg222300.html would have been a wiser choice, than a complete revert of commit aa6f8dcbab47 ("swiotlb: rework "fix info leak with DMA_FROM_DEVICE"").
Yeah, the revert is basically my standard "this doesn't work, discussion is still ongoing" thing.
I agree that the revert then brought back that DMA_ATTR_SKIP_CPU_SYNC complexity.
So that part of commit aa6f8dcbab47 was probably all good.
I somehow missed that Oleksandr had a tested-by for that smaller change too.
Linus