summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMarkus Elfring <elfring@users.sourceforge.net>2017-05-06 08:55:43 +0200
committerDavid Teigland <teigland@redhat.com>2017-08-07 18:23:09 +0200
commit0d37eca752e0bd90b26bf6e7be6c68ed5d1178f2 (patch)
tree3387d15b63c56ebaa90c15941becc88f4ba21cf4
parentdlm: Improve a size determination in dlm_recover_waiters_pre() (diff)
downloadlinux-0d37eca752e0bd90b26bf6e7be6c68ed5d1178f2.tar.xz
linux-0d37eca752e0bd90b26bf6e7be6c68ed5d1178f2.zip
dlm: Delete an error message for a failed memory allocation in dlm_recover_waiters_pre()
Omit an extra message for a memory allocation failure in this function. Link: http://events.linuxfoundation.org/sites/events/files/slides/LCJ16-Refactor_Strings-WSang_0.pdf Signed-off-by: Markus Elfring <elfring@users.sourceforge.net> Signed-off-by: David Teigland <teigland@redhat.com>
-rw-r--r--fs/dlm/lock.c4
1 files changed, 1 insertions, 3 deletions
diff --git a/fs/dlm/lock.c b/fs/dlm/lock.c
index ffadb817ad39..d4aaddec1b16 100644
--- a/fs/dlm/lock.c
+++ b/fs/dlm/lock.c
@@ -5120,10 +5120,8 @@ void dlm_recover_waiters_pre(struct dlm_ls *ls)
int dir_nodeid;
ms_stub = kmalloc(sizeof(*ms_stub), GFP_KERNEL);
- if (!ms_stub) {
- log_error(ls, "dlm_recover_waiters_pre no mem");
+ if (!ms_stub)
return;
- }
mutex_lock(&ls->ls_waiters_mutex);