From: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> Date: Thu, 17 Aug 2023 12:52:25 +0300 > On Wed, Aug 16, 2023 at 12:39:06PM -0500, Bjorn Helgaas wrote: >> On Wed, Aug 16, 2023 at 05:21:15PM +0000, Bartosz Pawlowski wrote: >>> There is a HW issue in A and B steppings of Intel IPU E2000 that it >>> expects wrong endianness in ATS invalidation message body. This problem >>> can lead to outdated translations being returned as valid and finally >>> cause system instability. >>> >>> In order to prevent such issues introduce quirk_intel_e2000_no_ats() >>> function to disable ATS for vulnerable IPU E2000 devices. >>> >>> Signed-off-by: Bartosz Pawlowski <bartosz.pawlowski@xxxxxxxxx> >>> Reviewed-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> >>> Reviewed-by: Alexander Lobakin <aleksander.lobakin@xxxxxxxxx> >> >> Andy, Alexander, would you please reiterate your reviewed-by on the >> mailing list? I try to avoid relying on internal reviews collected by >> the author. Those are great and I'm glad they happen, but it's good >> if they also appear as part of the public conversation on the mailing >> list. > > They are legit for my name, I dunno what exactly you want to hear from me. > The internal process of review requires these tags by default, it's hard > to deviate from maintainer to maintainer. +1 under everything, it works the same way for me. [...] Thanks, Olek
