From mboxrd@z Thu Jan 1 00:00:00 1970 From: Artur Paszkiewicz Subject: [PATCH] imsm: don't call abort_reshape() in imsm_manage_reshape() Date: Fri, 4 Sep 2015 11:33:41 +0200 Message-ID: <1441359221-18920-1-git-send-email-artur.paszkiewicz@intel.com> Return-path: Sender: linux-raid-owner@vger.kernel.org To: neilb@suse.de Cc: linux-raid@vger.kernel.org, Artur Paszkiewicz , Konrad Dabrowski List-Id: linux-raid.ids Calling abort_reshape() in imsm_manage_reshape() is unnecessary in case of an error because it is handled by reshape_array(). Calling it when reshape completes successfully is also unnecessary and leads to a race condition: - reshape ends - mdadm calls abort_reshape() -> sets sync_action to idle - MD_RECOVERY_INTR is set and md_reap_sync_thread() does not finish the reshape Signed-off-by: Artur Paszkiewicz Signed-off-by: Konrad Dabrowski --- super-intel.c | 1 - 1 file changed, 1 deletion(-) diff --git a/super-intel.c b/super-intel.c index 95a72b6..e609e0c 100644 --- a/super-intel.c +++ b/super-intel.c @@ -10601,7 +10601,6 @@ static int imsm_manage_reshape( ret_val = 1; abort: free(buf); - abort_reshape(sra); return ret_val; } -- 2.1.4