From 8421d161cc0f76ca4d635b8f9e6ebb1b5e7b9b47 Mon Sep 17 00:00:00 2001 From: Jeff Mahoney Date: Mon, 4 Feb 2013 17:05:02 -0500 Subject: [PATCH 1/1] LU-2729 mgs: fix cleanup of nonexistant MGSSELF_NAME proc file The mgs teardown code gets invoked in the failure case but assumes that the MGSSELF_NAME proc file has been created. If it hasn't been created, the proc code issues a WARN_ON. The setup code clears the FSDB_MGS_SELF from &fsdb->fsdb_flags when it creates the proc file, so we can key off of that to decide whether to remove it. Signed-off-by: Jeff Mahoney Change-Id: Idf58d1fa572eeced5dbaa72f03e38ee46a708166 Reviewed-on: http://review.whamcloud.com/5122 Tested-by: Hudson Tested-by: Maloo Reviewed-by: Niu Yawei Reviewed-by: James Simmons Reviewed-by: Andreas Dilger --- lustre/mgs/lproc_mgs.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/lustre/mgs/lproc_mgs.c b/lustre/mgs/lproc_mgs.c index 54f299b..65513fd 100644 --- a/lustre/mgs/lproc_mgs.c +++ b/lustre/mgs/lproc_mgs.c @@ -292,7 +292,9 @@ int lproc_mgs_del_live(struct mgs_device *mgs, struct fs_db *fsdb) if (!mgs->mgs_proc_live) return 0; - lprocfs_remove_proc_entry(fsdb->fsdb_name, mgs->mgs_proc_live); + /* didn't create the proc file for MGSSELF_NAME */ + if (!test_bit(FSDB_MGS_SELF, &fsdb->fsdb_flags)) + lprocfs_remove_proc_entry(fsdb->fsdb_name, mgs->mgs_proc_live); return 0; } -- 1.8.3.1