linux-stable.git, branch v6.9.6

Linux 6.9.6

2024-06-21T12:40:40+00:00

Link: https://lore.kernel.org/r/20240619125609.836313103@linuxfoundation.org
Tested-by: Florian Fainelli 
Tested-by: Ronald Warsow 
Tested-by: SeongJae Park 
Tested-by: Jon Hunter 
Tested-by: Pavel Machek (CIP) 
Tested-by: Allen Pais 
Tested-by: Kelsey Steele 
Tested-by: Bagas Sanjaya 
Tested-by: Peter Schneider 
Tested-by: Salvatore Bonaccorso 
Tested-by: Mark Brown 
Tested-by: Ron Economos 
Tested-by: Shuah Khan 
Signed-off-by: Greg Kroah-Hartman

zap_pid_ns_processes: clear TIF_NOTIFY_SIGNAL along with TIF_SIGPENDING

2024-06-21T12:40:39+00:00

[ Upstream commit 7fea700e04bd3f424c2d836e98425782f97b494e ]

kernel_wait4() doesn't sleep and returns -EINTR if there is no
eligible child and signal_pending() is true.

That is why zap_pid_ns_processes() clears TIF_SIGPENDING but this is not
enough, it should also clear TIF_NOTIFY_SIGNAL to make signal_pending()
return false and avoid a busy-wait loop.

Link: https://lkml.kernel.org/r/20240608120616.GB7947@redhat.com
Fixes: 12db8b690010 ("entry: Add support for TIF_NOTIFY_SIGNAL")
Signed-off-by: Oleg Nesterov 
Reported-by: Rachel Menge 
Closes: https://lore.kernel.org/all/1386cd49-36d0-4a5c-85e9-bc42056a5a38@linux.microsoft.com/
Reviewed-by: Boqun Feng 
Tested-by: Wei Fu 
Reviewed-by: Jens Axboe 
Cc: Allen Pais 
Cc: Christian Brauner 
Cc: Frederic Weisbecker 
Cc: Joel Fernandes (Google) 
Cc: Joel Granados 
Cc: Josh Triplett 
Cc: Lai Jiangshan 
Cc: Mateusz Guzik 
Cc: Mathieu Desnoyers 
Cc: Mike Christie 
Cc: Neeraj Upadhyay 
Cc: Paul E. McKenney 
Cc: Steven Rostedt (Google) 
Cc: Zqiang 
Cc: Thomas Gleixner 
Signed-off-by: Andrew Morton 
Signed-off-by: Sasha Levin

i2c: designware: Fix the functionality flags of the slave-only interface

2024-06-21T12:40:39+00:00

[ Upstream commit cbf3fb5b29e99e3689d63a88c3cddbffa1b8de99 ]

When an I2C adapter acts only as a slave, it should not claim to
support I2C master capabilities.

Fixes: 5b6d721b266a ("i2c: designware: enable SLAVE in platform module")
Signed-off-by: Jean Delvare 
Cc: Luis Oliveira 
Cc: Jarkko Nikula 
Cc: Andy Shevchenko 
Cc: Mika Westerberg 
Cc: Jan Dabros 
Cc: Andi Shyti 
Reviewed-by: Andy Shevchenko 
Acked-by: Jarkko Nikula 
Tested-by: Jarkko Nikula 
Signed-off-by: Andi Shyti 
Signed-off-by: Sasha Levin

i2c: at91: Fix the functionality flags of the slave-only interface

2024-06-21T12:40:39+00:00

[ Upstream commit d6d5645e5fc1233a7ba950de4a72981c394a2557 ]

When an I2C adapter acts only as a slave, it should not claim to
support I2C master capabilities.

Fixes: 9d3ca54b550c ("i2c: at91: added slave mode support")
Signed-off-by: Jean Delvare 
Cc: Juergen Fitschen 
Cc: Ludovic Desroches 
Cc: Codrin Ciubotariu 
Cc: Andi Shyti 
Cc: Nicolas Ferre 
Cc: Alexandre Belloni 
Cc: Claudiu Beznea 
Signed-off-by: Andi Shyti 
Signed-off-by: Sasha Levin

misc: microchip: pci1xxxx: Fix a memory leak in the error handling of gp_aux_bus_probe()

2024-06-21T12:40:39+00:00

[ Upstream commit 77427e3d5c353e3dd98c7c0af322f8d9e3131ace ]

There is a memory leak (forget to free allocated buffers) in a
memory allocation failure path.

Fix it to jump to the correct error handling code.

Fixes: 393fc2f5948f ("misc: microchip: pci1xxxx: load auxiliary bus driver for the PIO function in the multi-function endpoint of pci1xxxx device.")
Signed-off-by: Yongzhi Liu 
Reviewed-by: Kumaravel Thiagarajan 
Link: https://lore.kernel.org/r/20240523121434.21855-4-hyperlyzcs@gmail.com
Signed-off-by: Greg Kroah-Hartman 
Signed-off-by: Sasha Levin

mei: vsc: Fix wrong invocation of ACPI SID method

2024-06-21T12:40:39+00:00

[ Upstream commit af076156ec6d70332f1555754e99d4a3771ec297 ]

When using an initializer for a union only one of the union members
must be initialized. The initializer for the acpi_object union variable
passed as argument to the SID ACPI method was initializing both
the type and the integer members of the union.

Unfortunately rather then complaining about this gcc simply ignores
the first initializer and only used the second integer.value = 1
initializer. Leaving type set to 0 which leads to the argument being
skipped by acpi acpi_ns_evaluate() resulting in:

ACPI Warning: \_SB.PC00.SPI1.SPFD.CVFD.SID: Insufficient arguments -
Caller passed 0, method requires 1 (20240322/nsarguments-232)

Fix this by initializing only the integer struct part of the union
and initializing both members of the integer struct.

Signed-off-by: Hans de Goede 
Reviewed-by: Sakari Ailus 
Fixes: 566f5ca97680 ("mei: Add transport driver for IVSC device")
Reviewed-by: Wentong Wu 
Link: https://lore.kernel.org/r/20240603205050.505389-1-hdegoede@redhat.com
Signed-off-by: Greg Kroah-Hartman 
Signed-off-by: Sasha Levin

usb-storage: alauda: Check whether the media is initialized

2024-06-21T12:40:39+00:00

[ Upstream commit 16637fea001ab3c8df528a8995b3211906165a30 ]

The member "uzonesize" of struct alauda_info will remain 0
if alauda_init_media() fails, potentially causing divide errors
in alauda_read_data() and alauda_write_lba().
- Add a member "media_initialized" to struct alauda_info.
- Change a condition in alauda_check_media() to ensure the
  first initialization.
- Add an error check for the return value of alauda_init_media().

Fixes: e80b0fade09e ("[PATCH] USB Storage: add alauda support")
Reported-by: xingwei lee 
Reported-by: yue sun 
Reviewed-by: Alan Stern 
Signed-off-by: Shichao Lai 
Link: https://lore.kernel.org/r/20240526012745.2852061-1-shichaorai@gmail.com
Signed-off-by: Greg Kroah-Hartman 
Signed-off-by: Sasha Levin

dt-bindings: usb: realtek,rts5411: Add missing "additionalProperties" on child nodes

2024-06-21T12:40:39+00:00

[ Upstream commit e4228cfd092351c2d9b1a3048b2070287291ccbb ]

All nodes need an explicit additionalProperties or unevaluatedProperties
unless a $ref has one that's false. As that is not the case with
usb-device.yaml, "additionalProperties" is needed here.

Fixes: c44d9dab31d6 ("dt-bindings: usb: Add downstream facing ports to realtek binding")
Signed-off-by: "Rob Herring (Arm)" 
Reviewed-by: Stephen Boyd 
Link: https://lore.kernel.org/r/20240523194500.2958192-1-robh@kernel.org
Signed-off-by: Greg Kroah-Hartman 
Signed-off-by: Sasha Levin

serial: 8250_dw: Don't use struct dw8250_data outside of 8250_dw

2024-06-21T12:40:38+00:00

[ Upstream commit 87d80bfbd577912462061b1a45c0ed9c7fcb872f ]

The container of the struct dw8250_port_data is private to the actual
driver. In particular, 8250_lpss and 8250_dw use different data types
that are assigned to the UART port private_data. Hence, it must not
be used outside the specific driver.

Currently the only cpr_val is required by the common code, make it
be available via struct dw8250_port_data.

This fixes the UART breakage on Intel Galileo boards.

Fixes: 593dea000bc1 ("serial: 8250: dw: Allow to use a fallback CPR value if not synthesized")
Signed-off-by: Andy Shevchenko 
Link: https://lore.kernel.org/r/20240514190730.2787071-2-andriy.shevchenko@linux.intel.com
Signed-off-by: Greg Kroah-Hartman 
Signed-off-by: Sasha Levin

ima: Fix use-after-free on a dentry's dname.name

2024-06-21T12:40:38+00:00

commit be84f32bb2c981ca670922e047cdde1488b233de upstream.

->d_name.name can change on rename and the earlier value can be freed;
there are conditions sufficient to stabilize it (->d_lock on dentry,
->d_lock on its parent, ->i_rwsem exclusive on the parent's inode,
rename_lock), but none of those are met at any of the sites. Take a stable
snapshot of the name instead.

Link: https://lore.kernel.org/all/20240202182732.GE2087318@ZenIV/
Signed-off-by: Al Viro 
Signed-off-by: Stefan Berger 
Signed-off-by: Mimi Zohar 
Signed-off-by: Greg Kroah-Hartman