mbox series

[SRU,focal:linux-gcp,0/2] Fixes for LP:#2049922

Message ID 20240126210944.33688-1-joseph.salisbury@canonical.com
Headers show
Series Fixes for LP:#2049922 | expand

Message

Joseph Salisbury Jan. 26, 2024, 9:09 p.m. UTC
BugLink: https://bugs.launchpad.net/bugs/2049922

[Impact]

* Request from Google to include Intel patches for IDPF support if used from
out-of-tree module to prevent system instability

Also being requested in bionic:linux-gcp-4.15.  However, bionic:linux-gcp-4.15
required a different backport, so it will be sent separatly.

[Fixes]

f18b1137d38c ("PCI: Extract ATS disabling to a helper function")
a18615b1cfc0 ("PCI: Disable ATS for specific Intel IPU E2000 devices")

[Test Case]

* Compile test
* Boot test
* Test on non-IDPF instance
* Also tested by Google.

[Where things could go wrong]

* Low chance of regression, changes very isolated

[Other Info]

* SF #00376470

Bartosz Pawlowski (2):
  PCI: Extract ATS disabling to a helper function
  PCI: Disable ATS for specific Intel IPU E2000 devices

 drivers/pci/quirks.c | 29 ++++++++++++++++++++++++++---
 1 file changed, 26 insertions(+), 3 deletions(-)

Comments

Andrei Gherzan Feb. 1, 2024, 11:59 a.m. UTC | #1
On 24/01/26 04:09PM, Joseph Salisbury wrote:
> BugLink: https://bugs.launchpad.net/bugs/2049922
> 
> [Impact]
> 
> * Request from Google to include Intel patches for IDPF support if used from
> out-of-tree module to prevent system instability
> 
> Also being requested in bionic:linux-gcp-4.15.  However, bionic:linux-gcp-4.15
> required a different backport, so it will be sent separatly.
> 
> [Fixes]
> 
> f18b1137d38c ("PCI: Extract ATS disabling to a helper function")
> a18615b1cfc0 ("PCI: Disable ATS for specific Intel IPU E2000 devices")
> 
> [Test Case]
> 
> * Compile test
> * Boot test
> * Test on non-IDPF instance
> * Also tested by Google.
> 
> [Where things could go wrong]
> 
> * Low chance of regression, changes very isolated
> 
> [Other Info]
> 
> * SF #00376470
> 
> Bartosz Pawlowski (2):
>   PCI: Extract ATS disabling to a helper function
>   PCI: Disable ATS for specific Intel IPU E2000 devices
> 
>  drivers/pci/quirks.c | 29 ++++++++++++++++++++++++++---
>  1 file changed, 26 insertions(+), 3 deletions(-)

Acked-by: Andrei Gherzan <andrei.gherzan@canonical.com>
Tim Gardner Feb. 6, 2024, 4:27 p.m. UTC | #2
On 1/26/24 2:09 PM, Joseph Salisbury wrote:
> BugLink: https://bugs.launchpad.net/bugs/2049922
> 
> [Impact]
> 
> * Request from Google to include Intel patches for IDPF support if used from
> out-of-tree module to prevent system instability
> 
> Also being requested in bionic:linux-gcp-4.15.  However, bionic:linux-gcp-4.15
> required a different backport, so it will be sent separatly.
> 
> [Fixes]
> 
> f18b1137d38c ("PCI: Extract ATS disabling to a helper function")
> a18615b1cfc0 ("PCI: Disable ATS for specific Intel IPU E2000 devices")
> 
> [Test Case]
> 
> * Compile test
> * Boot test
> * Test on non-IDPF instance
> * Also tested by Google.
> 
> [Where things could go wrong]
> 
> * Low chance of regression, changes very isolated
> 
> [Other Info]
> 
> * SF #00376470
> 
> Bartosz Pawlowski (2):
>    PCI: Extract ATS disabling to a helper function
>    PCI: Disable ATS for specific Intel IPU E2000 devices
> 
>   drivers/pci/quirks.c | 29 ++++++++++++++++++++++++++---
>   1 file changed, 26 insertions(+), 3 deletions(-)
> 
Acked-by: Tim Gardner <tim.gardner@canonical.com>
John Cabaj Feb. 14, 2024, 9:48 p.m. UTC | #3
On 1/26/24 3:09 PM, Joseph Salisbury wrote:
> BugLink: https://bugs.launchpad.net/bugs/2049922
> 
> [Impact]
> 
> * Request from Google to include Intel patches for IDPF support if used from
> out-of-tree module to prevent system instability
> 
> Also being requested in bionic:linux-gcp-4.15.  However, bionic:linux-gcp-4.15
> required a different backport, so it will be sent separatly.
> 
> [Fixes]
> 
> f18b1137d38c ("PCI: Extract ATS disabling to a helper function")
> a18615b1cfc0 ("PCI: Disable ATS for specific Intel IPU E2000 devices")
> 
> [Test Case]
> 
> * Compile test
> * Boot test
> * Test on non-IDPF instance
> * Also tested by Google.
> 
> [Where things could go wrong]
> 
> * Low chance of regression, changes very isolated
> 
> [Other Info]
> 
> * SF #00376470
> 
> Bartosz Pawlowski (2):
>   PCI: Extract ATS disabling to a helper function
>   PCI: Disable ATS for specific Intel IPU E2000 devices
> 
>  drivers/pci/quirks.c | 29 ++++++++++++++++++++++++++---
>  1 file changed, 26 insertions(+), 3 deletions(-)
> 

Applied to focal:linux-gcp master-next branch.


John