Message-ID: <a5376c47-156d-4841-85ab-615f2c942960@cachyos.org>
Date: Wed, 21 May 2025 22:13:45 +0800
From: Eric Naim <dnaim@...hyos.org>
To: Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
 Mario Limonciello <mario.limonciello@....com>
Cc: stable@...r.kernel.org, patches@...ts.linux.dev,
 linux-kernel@...r.kernel.org, torvalds@...ux-foundation.org,
 akpm@...ux-foundation.org, linux@...ck-us.net, shuah@...nel.org,
 patches@...nelci.org, lkft-triage@...ts.linaro.org, pavel@...x.de,
 jonathanh@...dia.com, f.fainelli@...il.com, sudipm.mukherjee@...il.com,
 srw@...dewatkins.net, rwarsow@....de, conor@...nel.org,
 hargar@...rosoft.com, broonie@...nel.org, David.Wu3@....com,
 alexander.deucher@....com
Subject: Re: [PATCH 6.14 000/145] 6.14.8-rc1 review

On 5/21/25 13:35, Greg Kroah-Hartman wrote:
> On Tue, May 20, 2025 at 09:42:13PM -0500, Mario Limonciello wrote:
>> On 5/20/2025 4:34 PM, Eric Naim wrote:
>>> Hi Greg,
>>>
>>> On 5/20/25 21:49, Greg Kroah-Hartman wrote:
>>>> This is the start of the stable review cycle for the 6.14.8 release.
>>>> There are 145 patches in this series, all will be posted as a response
>>>> to this one.  If anyone has any issues with these being applied, please
>>>> let me know.
>>>>
>>>> Responses should be made by Thu, 22 May 2025 12:57:37 +0000.
>>>> Anything received after that time might be too late.
>>>>
>>>> The whole patch series can be found in one patch at:
>>>> 	https://www.kernel.org/pub/linux/kernel/v6.x/stable-review/patch-6.14.8-rc1.gz
>>>> or in the git tree and branch at:
>>>> 	git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git linux-6.14.y
>>>> and the diffstat can be found below.
>>>>
>>>> thanks,
>>>>
>>>> greg k-h
>>>>
>>>> -------------
>>>> Pseudo-Shortlog of commits:
>>>>
>>>> Greg Kroah-Hartman <gregkh@...uxfoundation.org>
>>>>      Linux 6.14.8-rc1
>>>>
>>>> Dan Carpenter <dan.carpenter@...aro.org>
>>>>      phy: tegra: xusb: remove a stray unlock
>>>>
>>>> Tiezhu Yang <yangtiezhu@...ngson.cn>
>>>>      perf tools: Fix build error for LoongArch
>>>>
>>>> Kirill A. Shutemov <kirill.shutemov@...ux.intel.com>
>>>>      mm/page_alloc: fix race condition in unaccepted memory handling
>>>>
>>>> Daniele Ceraolo Spurio <daniele.ceraolospurio@...el.com>
>>>>      drm/xe/gsc: do not flush the GSC worker from the reset path
>>>>
>>>> Maciej Falkowski <maciej.falkowski@...ux.intel.com>
>>>>      accel/ivpu: Flush pending jobs of device's workqueues
>>>>
>>>> Karol Wachowski <karol.wachowski@...el.com>
>>>>      accel/ivpu: Fix missing MMU events if file_priv is unbound
>>>>
>>>> Karol Wachowski <karol.wachowski@...el.com>
>>>>      accel/ivpu: Fix missing MMU events from reserved SSID
>>>>
>>>> Karol Wachowski <karol.wachowski@...el.com>
>>>>      accel/ivpu: Move parts of MMU event IRQ handling to thread handler
>>>>
>>>> Karol Wachowski <karol.wachowski@...el.com>
>>>>      accel/ivpu: Dump only first MMU fault from single context
>>>>
>>>> Maciej Falkowski <maciej.falkowski@...ux.intel.com>
>>>>      accel/ivpu: Use workqueue for IRQ handling
>>>>
>>>> Shuai Xue <xueshuai@...ux.alibaba.com>
>>>>      dmaengine: idxd: Refactor remove call with idxd_cleanup() helper
>>>>
>>>> Shuai Xue <xueshuai@...ux.alibaba.com>
>>>>      dmaengine: idxd: fix memory leak in error handling path of idxd_pci_probe
>>>>
>>>> Shuai Xue <xueshuai@...ux.alibaba.com>
>>>>      dmaengine: idxd: fix memory leak in error handling path of idxd_alloc
>>>>
>>>> Shuai Xue <xueshuai@...ux.alibaba.com>
>>>>      dmaengine: idxd: Add missing idxd cleanup to fix memory leak in remove call
>>>>
>>>> Shuai Xue <xueshuai@...ux.alibaba.com>
>>>>      dmaengine: idxd: Add missing cleanups in cleanup internals
>>>>
>>>> Shuai Xue <xueshuai@...ux.alibaba.com>
>>>>      dmaengine: idxd: Add missing cleanup for early error out in idxd_setup_internals
>>>>
>>>> Shuai Xue <xueshuai@...ux.alibaba.com>
>>>>      dmaengine: idxd: fix memory leak in error handling path of idxd_setup_groups
>>>>
>>>> Shuai Xue <xueshuai@...ux.alibaba.com>
>>>>      dmaengine: idxd: fix memory leak in error handling path of idxd_setup_engines
>>>>
>>>> Shuai Xue <xueshuai@...ux.alibaba.com>
>>>>      dmaengine: idxd: fix memory leak in error handling path of idxd_setup_wqs
>>>>
>>>> Yemike Abhilash Chandra <y-abhilashchandra@...com>
>>>>      dmaengine: ti: k3-udma: Use cap_mask directly from dma_device structure instead of a local copy
>>>>
>>>> Ronald Wahl <ronald.wahl@...rand.com>
>>>>      dmaengine: ti: k3-udma: Add missing locking
>>>>
>>>> Barry Song <baohua@...nel.org>
>>>>      mm: userfaultfd: correct dirty flags set for both present and swap pte
>>>>
>>>> Wupeng Ma <mawupeng1@...wei.com>
>>>>      mm: hugetlb: fix incorrect fallback for subpool
>>>>
>>>> hexue <xue01.he@...sung.com>
>>>>      io_uring/uring_cmd: fix hybrid polling initialization issue
>>>>
>>>> Jens Axboe <axboe@...nel.dk>
>>>>      io_uring/memmap: don't use page_address() on a highmem page
>>>>
>>>> Nathan Chancellor <nathan@...nel.org>
>>>>      net: qede: Initialize qede_ll_ops with designated initializer
>>>>
>>>> Steven Rostedt <rostedt@...dmis.org>
>>>>      ring-buffer: Fix persistent buffer when commit page is the reader page
>>>>
>>>> Ming Yen Hsieh <mingyen.hsieh@...iatek.com>
>>>>      wifi: mt76: mt7925: fix missing hdr_trans_tlv command for broadcast wtbl
>>>>
>>>> Fedor Pchelkin <pchelkin@...ras.ru>
>>>>      wifi: mt76: disable napi on driver removal
>>>>
>>>> Jarkko Sakkinen <jarkko@...nel.org>
>>>>      tpm: Mask TPM RC in tpm2_start_auth_session()
>>>>
>>>> Aaron Kling <webgeek1234@...il.com>
>>>>      spi: tegra114: Use value to check for invalid delays
>>>>
>>>> Jethro Donaldson <devel@....nz>
>>>>      smb: client: fix memory leak during error handling for POSIX mkdir
>>>>
>>>> Steve Siwinski <ssiwinski@...o.com>
>>>>      scsi: sd_zbc: block: Respect bio vector limits for REPORT ZONES buffer
>>>>
>>>> Claudiu Beznea <claudiu.beznea.uj@...renesas.com>
>>>>      phy: renesas: rcar-gen3-usb2: Set timing registers only once
>>>>
>>>> Claudiu Beznea <claudiu.beznea.uj@...renesas.com>
>>>>      phy: renesas: rcar-gen3-usb2: Fix role detection on unbind/bind
>>>>
>>>> Oleksij Rempel <o.rempel@...gutronix.de>
>>>>      net: phy: micrel: remove KSZ9477 EEE quirks now handled by phylink
>>>>
>>>> Oleksij Rempel <o.rempel@...gutronix.de>
>>>>      net: dsa: microchip: let phylink manage PHY EEE configuration on KSZ switches
>>>>
>>>> Ma Ke <make24@...as.ac.cn>
>>>>      phy: Fix error handling in tegra_xusb_port_init
>>>>
>>>> Wayne Chang <waynec@...dia.com>
>>>>      phy: tegra: xusb: Use a bitmask for UTMI pad power state tracking
>>>>
>>>> Steven Rostedt <rostedt@...dmis.org>
>>>>      tracing: samples: Initialize trace_array_printk() with the correct function
>>>>
>>>> Ashish Kalra <ashish.kalra@....com>
>>>>      x86/sev: Make sure pages are not skipped during kdump
>>>>
>>>> Ashish Kalra <ashish.kalra@....com>
>>>>      x86/sev: Do not touch VMSA pages during SNP guest memory kdump
>>>>
>>>> pengdonglin <pengdonglin@...omi.com>
>>>>      ftrace: Fix preemption accounting for stacktrace filter command
>>>>
>>>> pengdonglin <pengdonglin@...omi.com>
>>>>      ftrace: Fix preemption accounting for stacktrace trigger command
>>>>
>>>> Christophe JAILLET <christophe.jaillet@...adoo.fr>
>>>>      i2c: designware: Fix an error handling path in i2c_dw_pci_probe()
>>>>
>>>> Nathan Chancellor <nathan@...nel.org>
>>>>      kbuild: Disable -Wdefault-const-init-unsafe
>>>>
>>>> Michael Kelley <mhklinux@...look.com>
>>>>      Drivers: hv: vmbus: Remove vmbus_sendpacket_pagebuffer()
>>>>
>>>> Michael Kelley <mhklinux@...look.com>
>>>>      Drivers: hv: Allow vmbus_sendpacket_mpb_desc() to create multiple ranges
>>>>
>>>> Michael Kelley <mhklinux@...look.com>
>>>>      hv_netvsc: Remove rmsg_pgcnt
>>>>
>>>> Michael Kelley <mhklinux@...look.com>
>>>>      hv_netvsc: Preserve contiguous PFN grouping in the page buffer array
>>>>
>>>> Michael Kelley <mhklinux@...look.com>
>>>>      hv_netvsc: Use vmbus_sendpacket_mpb_desc() to send VMBus messages
>>>>
>>>> Dragan Simic <dsimic@...jaro.org>
>>>>      arm64: dts: rockchip: Remove overdrive-mode OPPs from RK3588J SoC dtsi
>>>>
>>>> Sam Edwards <cfsworks@...il.com>
>>>>      arm64: dts: rockchip: Allow Turing RK1 cooling fan to spin down
>>>>
>>>> Christian Hewitt <christianshewitt@...il.com>
>>>>      arm64: dts: amlogic: dreambox: fix missing clkc_audio node
>>>>
>>>> Hyejeong Choi <hjeong.choi@...sung.com>
>>>>      dma-buf: insert memory barrier before updating num_fences
>>>>
>>>> Nicolas Chauvet <kwizart@...il.com>
>>>>      ALSA: usb-audio: Add sample rate quirk for Microdia JP001 USB Camera
>>>>
>>>> Christian Heusel <christian@...sel.eu>
>>>>      ALSA: usb-audio: Add sample rate quirk for Audioengine D1
>>>>
>>>> Wentao Liang <vulab@...as.ac.cn>
>>>>      ALSA: es1968: Add error handling for snd_pcm_hw_constraint_pow2()
>>>>
>>>> Jeremy Linton <jeremy.linton@....com>
>>>>      ACPI: PPTT: Fix processor subtable walk
>>>>
>>>> Emanuele Ghidoli <emanuele.ghidoli@...adex.com>
>>>>      gpio: pca953x: fix IRQ storm on system wake up
>>>>
>>>> Alexey Makhalov <alexey.makhalov@...adcom.com>
>>>>      MAINTAINERS: Update Alexey Makhalov's email address
>>>>
>>>> Wayne Lin <Wayne.Lin@....com>
>>>>      drm/amd/display: Avoid flooding unnecessary info messages
>>>>
>>>> Wayne Lin <Wayne.Lin@....com>
>>>>      drm/amd/display: Correct the reply value when AUX write incomplete
>>>>
>>>> Philip Yang <Philip.Yang@....com>
>>>>      drm/amdgpu: csa unmap use uninterruptible lock
>>>>
>>>> Tim Huang <tim.huang@....com>
>>>>      drm/amdgpu: fix incorrect MALL size for GFX1151
>>>>
>>>> David (Ming Qiang) Wu <David.Wu3@....com>
>>>>      drm/amdgpu: read back register after written for VCN v4.0.5
>>>>
>>>
>>> This commit seems to breaking a couple of devices with the Phoenix APU, most notably the Ryzen AI chips. Note that this commit in mainline seems to work as intended, and after doing a little bit of digging, [1] landed in 6.15 and so this cherrypick may not be so trivial after all. Attached is a kernel trace highlighting the breakage caused by this commit, along with [2] for the full log.
>>>
>>> Also adding Alex, David and Mario to Ccs.
>>>
>>
>> Just a minor correction - VCN 4.0.5 is on Strix.  So this report is not
>> likely from a Phoenix APU.
>>
>> Nonetheless I agree; I suspect backporting
>> ecc9ab4e924b7eb9e2c4a668162aaa1d9d60d08c will help the issue.
> 
> If it is required, someone is going to need to provide a working
> version, as that does not apply cleanly as-is.
> 
> thanks,
> 
> greg k-h

Hi Greg,

I finally got someone to test the mentioned commit, and I (or they) can confirm that ecc9ab4e924b7eb9e2c4a668162aaa1d9d60d08c is indeed the missing commit needed for "drm/amdgpu: read back register after written for VCN v4.0.5".

I don't know if this will help, but I got everything to apply cleanly by first applying ecc9ab4e924b7eb9e2c4a668162aaa1d9d60d08c, then cherry-picking the mainline version of "drm/amdgpu: read back register after written for VCN v4.0.5" [1] since the one in stable-rc seems to have been adjusted for the stable tree and made it conflict.

-- 
Regards,
  Eric

[1] ee7360fc27d6045510f8fe459b5649b2af27811a

Powered by blists - more mailing lists