Message-ID: <b08348c1-701f-4936-9c82-5d6e91cc701b@cachyos.org>
Date: Thu, 22 May 2025 00:59:10 +0800
From: Eric Naim <dnaim@...hyos.org>
To: Greg Kroah-Hartman <gregkh@...uxfoundation.org>
Cc: Mario Limonciello <mario.limonciello@....com>, 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/22/25 00:25, Greg Kroah-Hartman wrote:
> On Wed, May 21, 2025 at 10:13:45PM +0800, Eric Naim wrote:
>> 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.
> 
> I don't see the conflict that happened here, as I only apply patches
> "cleanly".  But I trust you that things are odd here.
> 
>> -- 
>> Regards,
>>   Eric
>>
>> [1] ee7360fc27d6045510f8fe459b5649b2af27811a
> 
> Ick.  Ok, let me go drop this commit from the tree now, and then can you
> resubmit both of these as a tested series for them to be included
> properly?
> 
> I'll do the same for both 6.12.y and 6.14.y now.
> 
> thanks,
> 
> greg k-h

Hi,

I've sent it over at [1]. I hope I did it correctly as this is the first time I'm formally sending over a patch (even if it's not mine). Please let me know if I did anything wrong and things that I can do to correct them :). I've also opted to add a `Tested-by` tag in response to "a tested series". Reminder that I only built the kernels, and had another user test that build since I don't have an affected device, so feel free to drop it if its wrong.

-- 
Regards,
  Eric

[1] https://lore.kernel.org/stable/20250521165421.293820-1-dnaim@cachyos.org

Powered by blists - more mailing lists