[PATCH v3 0/2] locking/mutex: Mark devm_mutex_init() as __must_check

From: Thomas Weißschuh
Date: Sat Feb 08 2025 - 11:30:33 EST


Even if it's not critical, the avoidance of checking the error code
from devm_mutex_init() call today diminishes the point of using devm
variant of it. Tomorrow it may even leak something. Enforce all callers
checking the return value through the compiler.

The series should go through the locking tree.

Signed-off-by: Thomas Weißschuh <linux@xxxxxxxxxxxxxx>
---
Changes in v3:
- Introduce and use helper macro __mutex_init_ret()
- Link to v2: https://lore.kernel.org/r/20250204-must_check-devm_mutex_init-v2-0-7b6271c4b7e6@xxxxxxxxxxxxxx

Changes in v2:
- Rebase on 6.14-rc1
- Fix up leds-1202 driver
- Link to v1: https://lore.kernel.org/r/20241202-must_check-devm_mutex_init-v1-1-e60eb97b8c72@xxxxxxxxxxxxxx

---
Thomas Weißschuh (2):
leds: st1202: Check for error code from devm_mutex_init() call
locking/mutex: Mark devm_mutex_init() as __must_check

drivers/leds/leds-st1202.c | 4 +++-
include/linux/mutex.h | 11 +++++++----
2 files changed, 10 insertions(+), 5 deletions(-)
---
base-commit: 2014c95afecee3e76ca4a56956a936e23283f05b
change-id: 20241031-must_check-devm_mutex_init-cac583bda8fe

Best regards,
--
Thomas Weißschuh <linux@xxxxxxxxxxxxxx>