ACK Re: [SRU][PULL][linux-firmware][Jammy] System hangs at S3 for amdgpu Navi 31/Navi33

[Timo Aaltonen]

You-Sheng Yang kirjoitti 6.7.2023 klo 11.26:
> BugLink: https://bugs.launchpad.net/bugs/2024427
> 
> [Impact]
> 
> Under stress testing it was reported that the amdgpu Navi31/Navi33 platforms
> will sometimes fail to wake from S3.
> 
> [Fix]
> 
> kernel patches:
> ac2f5739fdca drm/amdgpu/mes11: enable reg active poll
> a2fe4534bb38 drm/amd/amdgpu: update mes11 api def
> da9a8dc33da2 drm/amdgpu: reserve the old gc_11_0_*_mes.bin
> 616843d5a11b drm/amd/amdgpu: introduce gc_*_mes_2.bin v2
> 09bf14907d86 drm/amdgpu: declare firmware for new MES 11.0.4
> 
> firmware patches:
> * Navi31: ffe1a41e2ddb amdgpu: update GC 11.0.0 firmware for amd.5.5 release
> * Navi33: a5d7b4df1a76 amdgpu: update GC 11.0.2 firmware for amd.5.5 release
> 
> [Test Case]
> 
> $ checkbox-cli run com.canonical.certification::stress-suspend-30-cycles-with-reboots-automated
> 
> [Where problems could occur]
> 
> Little we know about the firmware fixes. However, while with these commits have
> been pulled via stable kernel fixes, the driver begins to request new firmware
> blobs of a different filename.
> 
> [Other Info]
> 
> The kernel driver commits are in v6.4-rc1, backported to v6.3.4, v6.1.31, and
> partially (missing da9a8dc33da2, 616843d5a11b) v6.2.16. Only linux/lunar has to
> be fixed.
> 
> For the firmware parts, they have been included in linux-firmware/mantic,
> leaving linux-firmware/lunar and linux-firmware/jammy to be fixed.
> 
> ----------------------------------------------------------------
> The following changes since commit b813e8a771c0078c56618fcb446d8a0d70151945:
> 
>    iwlwifi: update core69 and core72 firmwares for So device (2023-06-21 12:21:20 +0200)
> 
> are available in the Git repository at:
> 
>    https://git.launchpad.net/~vicamo/ubuntu/+source/linux-firmware bug-2024427/amdgpu-navi31-navi33-s3-hang/jammy-pull
> 
> for you to fetch changes up to c52c59b207625ca056e6d391a3c4a13f040d36a9:
> 
>    amdgpu: update GC 11.0.2 firmware for amd.5.5 release (2023-07-06 16:02:26 +0800)
> 
> ----------------------------------------------------------------
> Alex Deucher (2):
>        amdgpu: update GC 11.0.0 firmware for amd.5.5 release
>        amdgpu: update GC 11.0.2 firmware for amd.5.5 release
> 
>   WHENCE                     |   2 ++
>   amdgpu/gc_11_0_0_imu.bin   | Bin 132352 -> 132352 bytes
>   amdgpu/gc_11_0_0_me.bin    | Bin 189392 -> 189520 bytes
>   amdgpu/gc_11_0_0_mec.bin   | Bin 407072 -> 407072 bytes
>   amdgpu/gc_11_0_0_mes1.bin  | Bin 230624 -> 211696 bytes
>   amdgpu/gc_11_0_0_mes_2.bin | Bin 0 -> 289280 bytes
>   amdgpu/gc_11_0_0_pfp.bin   | Bin 207712 -> 207984 bytes
>   amdgpu/gc_11_0_0_rlc.bin   | Bin 184992 -> 184992 bytes
>   amdgpu/gc_11_0_2_imu.bin   | Bin 132352 -> 132352 bytes
>   amdgpu/gc_11_0_2_me.bin    | Bin 189408 -> 189552 bytes
>   amdgpu/gc_11_0_2_mec.bin   | Bin 407072 -> 407072 bytes
>   amdgpu/gc_11_0_2_mes1.bin  | Bin 230912 -> 212048 bytes
>   amdgpu/gc_11_0_2_mes_2.bin | Bin 0 -> 289408 bytes
>   amdgpu/gc_11_0_2_pfp.bin   | Bin 207712 -> 207984 bytes
>   amdgpu/gc_11_0_2_rlc.bin   | Bin 178448 -> 178448 bytes
>   15 files changed, 2 insertions(+)
>   create mode 100644 amdgpu/gc_11_0_0_mes_2.bin
>   create mode 100644 amdgpu/gc_11_0_2_mes_2.bin
> 
> diff --git a/WHENCE b/WHENCE
> index fca29f9d0..e62ac85f8 100644
> --- a/WHENCE
> +++ b/WHENCE
> @@ -2900,6 +2900,7 @@ File: amdgpu/gc_11_0_0_me.bin
>   File: amdgpu/gc_11_0_0_mec.bin
>   File: amdgpu/gc_11_0_0_mes1.bin
>   File: amdgpu/gc_11_0_0_mes.bin
> +File: amdgpu/gc_11_0_0_mes_2.bin
>   File: amdgpu/gc_11_0_0_pfp.bin
>   File: amdgpu/gc_11_0_0_rlc.bin
>   File: amdgpu/gc_11_0_1_imu.bin
> @@ -2914,6 +2915,7 @@ File: amdgpu/gc_11_0_2_me.bin
>   File: amdgpu/gc_11_0_2_mec.bin
>   File: amdgpu/gc_11_0_2_mes1.bin
>   File: amdgpu/gc_11_0_2_mes.bin
> +File: amdgpu/gc_11_0_2_mes_2.bin
>   File: amdgpu/gc_11_0_2_pfp.bin
>   File: amdgpu/gc_11_0_2_rlc.bin
>   File: amdgpu/gc_11_0_4_imu.bin
> diff --git a/amdgpu/gc_11_0_0_imu.bin b/amdgpu/gc_11_0_0_imu.bin
> index d166a5cc9..ff93420f7 100644
> Binary files a/amdgpu/gc_11_0_0_imu.bin and b/amdgpu/gc_11_0_0_imu.bin differ
> diff --git a/amdgpu/gc_11_0_0_me.bin b/amdgpu/gc_11_0_0_me.bin
> index ce7a22ca3..e681e41f3 100644
> Binary files a/amdgpu/gc_11_0_0_me.bin and b/amdgpu/gc_11_0_0_me.bin differ
> diff --git a/amdgpu/gc_11_0_0_mec.bin b/amdgpu/gc_11_0_0_mec.bin
> index c4610600d..e12206fbe 100644
> Binary files a/amdgpu/gc_11_0_0_mec.bin and b/amdgpu/gc_11_0_0_mec.bin differ
> diff --git a/amdgpu/gc_11_0_0_mes1.bin b/amdgpu/gc_11_0_0_mes1.bin
> index 50d71dfd3..5682ae355 100644
> Binary files a/amdgpu/gc_11_0_0_mes1.bin and b/amdgpu/gc_11_0_0_mes1.bin differ
> diff --git a/amdgpu/gc_11_0_0_mes_2.bin b/amdgpu/gc_11_0_0_mes_2.bin
> new file mode 100644
> index 000000000..7bb688aa8
> Binary files /dev/null and b/amdgpu/gc_11_0_0_mes_2.bin differ
> diff --git a/amdgpu/gc_11_0_0_pfp.bin b/amdgpu/gc_11_0_0_pfp.bin
> index 1b3d493eb..f6d10bad4 100644
> Binary files a/amdgpu/gc_11_0_0_pfp.bin and b/amdgpu/gc_11_0_0_pfp.bin differ
> diff --git a/amdgpu/gc_11_0_0_rlc.bin b/amdgpu/gc_11_0_0_rlc.bin
> index 64754229c..a8bbc9417 100644
> Binary files a/amdgpu/gc_11_0_0_rlc.bin and b/amdgpu/gc_11_0_0_rlc.bin differ
> diff --git a/amdgpu/gc_11_0_2_imu.bin b/amdgpu/gc_11_0_2_imu.bin
> index 3fa2f53ef..dcdadecf7 100644
> Binary files a/amdgpu/gc_11_0_2_imu.bin and b/amdgpu/gc_11_0_2_imu.bin differ
> diff --git a/amdgpu/gc_11_0_2_me.bin b/amdgpu/gc_11_0_2_me.bin
> index 4a40c325c..99c732dfa 100644
> Binary files a/amdgpu/gc_11_0_2_me.bin and b/amdgpu/gc_11_0_2_me.bin differ
> diff --git a/amdgpu/gc_11_0_2_mec.bin b/amdgpu/gc_11_0_2_mec.bin
> index f12f10df8..70053544b 100644
> Binary files a/amdgpu/gc_11_0_2_mec.bin and b/amdgpu/gc_11_0_2_mec.bin differ
> diff --git a/amdgpu/gc_11_0_2_mes1.bin b/amdgpu/gc_11_0_2_mes1.bin
> index 9acac7a57..63338510a 100644
> Binary files a/amdgpu/gc_11_0_2_mes1.bin and b/amdgpu/gc_11_0_2_mes1.bin differ
> diff --git a/amdgpu/gc_11_0_2_mes_2.bin b/amdgpu/gc_11_0_2_mes_2.bin
> new file mode 100644
> index 000000000..d93ccfc8f
> Binary files /dev/null and b/amdgpu/gc_11_0_2_mes_2.bin differ
> diff --git a/amdgpu/gc_11_0_2_pfp.bin b/amdgpu/gc_11_0_2_pfp.bin
> index 6c19a57ec..5cd5df055 100644
> Binary files a/amdgpu/gc_11_0_2_pfp.bin and b/amdgpu/gc_11_0_2_pfp.bin differ
> diff --git a/amdgpu/gc_11_0_2_rlc.bin b/amdgpu/gc_11_0_2_rlc.bin
> index fea27d75e..a2b65d6eb 100644
> Binary files a/amdgpu/gc_11_0_2_rlc.bin and b/amdgpu/gc_11_0_2_rlc.bin differ
> 

Acked-by: Timo Aaltonen <timo.aaltonen at canonical.com>

-- 
t