Skip to contentSkip to conversation

Remove redundant up() in stop_machine(2.6.18-rc4)

PATCH New Zhou Yingchao 1 participant 1 message Aug 23, 2006

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 Remove redundant up() in stop_machine(2.6.18-rc4)

An up() is called in kernel/stop_machine.c on failure, and also in
the caller unconditionally. I have reported, but now it is still here.

___

stop_machine.c 2006-08-23 14:53:55.000000000 +0800 renamed from stop_machine.c.orig 2006-08-23 14:53:36.000000000 +0800 +0 -1
@@ -111,7 +111,6 @@ static int stop_machine(void)
       /* If some failed, kill them all. */
       if (ret < 0) {
               stopmachine_set_state(STOPMACHINE_EXIT);
-              up(&stopmachine_mutex);
               return ret;
       }