[v2] locking/mutex: Mark devm_mutex_init() as __must_check

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

Thomas Weißschuh posted 2 patches 10 months, 1 week ago

Diff against v1 v3 v4 v5 v6 v7
Download series mbox

There is a newer version of this series

drivers/leds/leds-st1202.c |  4 +++-
include/linux/mutex.h      | 17 ++++++++---------
2 files changed, 11 insertions(+), 10 deletions(-)

Expand all Fold all

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

Posted by Thomas Weißschuh 10 months, 1 week ago

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@weissschuh.net>
---
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@weissschuh.net

---
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      | 17 ++++++++---------
 2 files changed, 11 insertions(+), 10 deletions(-)
---
base-commit: 2014c95afecee3e76ca4a56956a936e23283f05b
change-id: 20241031-must_check-devm_mutex_init-cac583bda8fe

Best regards,
-- 
Thomas Weißschuh <linux@weissschuh.net>