This patch series removes some unnecessary sourcing of common/rc and decouples the call to init_rc() from the sourcing of common/rc. This is proposed in [1] and [2]. It also removes direct usage of exit command with a _exit wrapper. The individual patches have the details. [v3] --> v4 1. Added R.Bs from Zorro and Ritesh in patch 2 of [v3] 2. Updated the definition of _exit() - explicitly set the value "status" only if it is passed. 3. Updated the description comment for _exit(). [1] https://lore.kernel.org/all/20250206155251.GA21787@frogsfrogsfrogs/ [2] https://lore.kernel.org/all/20250210142322.tptpphdntglsz4eq@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx/ [3] https://lore.kernel.org/all/Z-xcne3f5Klvuxcq@xxxxxxxxxxxxxxxxxxx/ [v1] https://lore.kernel.org/all/cover.1741248214.git.nirjhar.roy.lists@xxxxxxxxx/ [v2] https://lore.kernel.org/all/cover.1743487913.git.nirjhar.roy.lists@xxxxxxxxx/ [v3] https://lore.kernel.org/all/cover.1744090313.git.nirjhar.roy.lists@xxxxxxxxx/ Nirjhar Roy (IBM) (6): generic/749: Remove redundant sourcing of common/rc generic/367: Remove redundant sourcing of common/config check: Remove redundant _test_mount in check check,common{rc,preamble}: Decouple init_rc() call from sourcing common/rc common/config: Introduce _exit wrapper around exit command common: exit --> _exit check | 10 ++--- common/btrfs | 6 +-- common/ceph | 2 +- common/config | 16 +++++-- common/dump | 11 +++-- common/ext4 | 2 +- common/populate | 2 +- common/preamble | 3 +- common/punch | 13 +++--- common/rc | 107 ++++++++++++++++++++++------------------------ common/repair | 4 +- common/xfs | 8 ++-- tests/generic/367 | 1 - tests/generic/749 | 1 - 14 files changed, 92 insertions(+), 94 deletions(-) -- 2.34.1
