From: Karel Zak Date: Fri, 24 Feb 2012 22:03:22 +0000 (+0100) Subject: mount: (new) cleanup mount -a return codes X-Git-Tag: v2.22-rc1~749 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=16b73aae8cb73df2974fd75c2a42ec3b92535851;p=thirdparty%2Futil-linux.git mount: (new) cleanup mount -a return codes New return codes: 0 : all mounted (or all ignored) 64 : some mounted, some failed 32 : all failed Note that already mounted or ignored (filtered out by -t or -O) filesystems don't affect the final return code. The original mount(8) returns 0 instead of 64, so the situation "some mounted, some failed" cannot be detected. Signed-off-by: Karel Zak --- diff --git a/sys-utils/mount.8 b/sys-utils/mount.8 index f1cff7476f..d18881b2ce 100644 --- a/sys-utils/mount.8 +++ b/sys-utils/mount.8 @@ -2759,6 +2759,10 @@ mount failure .TP .BR 64 some mount succeeded +.RE + +The command mount -a returns 0 (all success), 32 (all failed) or 64 (some +failed, some success). .SH NOTES The syntax of external mount helpers is: diff --git a/sys-utils/mount.c b/sys-utils/mount.c index 17991b0fbf..3fbac04a1a 100644 --- a/sys-utils/mount.c +++ b/sys-utils/mount.c @@ -182,6 +182,8 @@ static int mount_all(struct libmnt_context *cxt) struct libmnt_fs *fs; int mntrc, ignored, rc = MOUNT_EX_SUCCESS; + int nsucc = 0, nerrs = 0; + itr = mnt_new_iter(MNT_ITER_FORWARD); if (!itr) { warn(_("failed to initialize libmount iterator")); @@ -197,31 +199,40 @@ static int mount_all(struct libmnt_context *cxt) printf(ignored == 1 ? _("%-25s: ignored\n") : _("%-25s: already mounted\n"), tgt); - } else if (mnt_context_is_fork(cxt)) { if (mnt_context_is_verbose(cxt)) printf("%-25s: mount successfully forked\n", tgt); } else { - rc |= mk_exit_code(cxt, mntrc); + mk_exit_code(cxt, mntrc); /* to print warnings */ if (mnt_context_get_status(cxt)) { - rc |= MOUNT_EX_SOMEOK; + nsucc++; if (mnt_context_is_verbose(cxt)) printf("%-25s: successfully mounted\n", tgt); - } + } else + nerrs++; } } if (mnt_context_is_parent(cxt)) { /* wait for mount --fork children */ - int nerrs = 0, nchildren = 0; + int nchildren = 0; + + nerrs = 0, nsucc = 0; rc = mnt_context_wait_for_children(cxt, &nchildren, &nerrs); if (!rc && nchildren) - rc = nchildren == nerrs ? MOUNT_EX_FAIL : MOUNT_EX_SOMEOK; + nsucc = nchildren - nerrs; } + if (nerrs == 0) + rc = MOUNT_EX_SUCCESS; /* all success */ + else if (nsucc == 0) + rc = MOUNT_EX_FAIL; /* all failed */ + else + rc = MOUNT_EX_SOMEOK; /* some success, some failed */ + mnt_free_iter(itr); return rc; }