[PATCH v2 0/5] Minor cleanups in common/

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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.

[v1] --> v[2]
 1. Added R.B from Darrick in patch 1 of [v1]
 2. Kept the init_rc call that was deleted in the v1.
 3. Introduced _exit wrapper around exit command. This will help us get correct
    exit codes ("$?") on failures.

[1] https://lore.kernel.org/all/20250206155251.GA21787@frogsfrogsfrogs/

[2] https://lore.kernel.org/all/20250210142322.tptpphdntglsz4eq@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx/

[v1] https://lore.kernel.org/all/cover.1741248214.git.nirjhar.roy.lists@xxxxxxxxx/

Nirjhar Roy (IBM) (5):
  generic/749: Remove redundant sourcing of common/rc
  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             |   8 +---
 common/btrfs      |   6 +--
 common/ceph       |   2 +-
 common/config     |  15 +++++--
 common/ext4       |   2 +-
 common/populate   |   2 +-
 common/preamble   |   3 +-
 common/punch      |  12 +++---
 common/rc         | 105 ++++++++++++++++++++++------------------------
 common/xfs        |   8 ++--
 tests/generic/749 |   1 -
 11 files changed, 81 insertions(+), 83 deletions(-)

--
2.34.1





[Index of Archives]     [Reiser Filesystem Development]     [Ceph FS]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux FS]     [Yosemite National Park]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Device Mapper]     [Linux Media]

  Powered by Linux