Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 7 Dec 2015 02:56:08 +0000 (UTC)
From:      Steven Hartland <smh@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r291911 - head/sys/dev/iscsi
Message-ID:  <201512070256.tB72u8C6057988@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: smh
Date: Mon Dec  7 02:56:08 2015
New Revision: 291911
URL: https://svnweb.freebsd.org/changeset/base/291911

Log:
  Fix panic on shutdown due to iscsi event priority
  
  iscsi's shutdown_pre_sync prio was SHUTDOWN_PRI_FIRST which caused it to
  run before other high priority handlers such as filesystems e.g. ZFS.
  
  This meant the iscsi sessions where removed before the ZFS geom consumer
  was closed, resulting in a panic from g_access calls on debug kernels
  due to negative acr.
  
  Instead use the same as the old iscsi_initiator SHUTDOWN_PRI_DEFAULT-1
  which allows it to run before dashutdown etc but after filesystems.
  
  MFC after:	2 weeks
  Sponsored by:	Multiplay

Modified:
  head/sys/dev/iscsi/iscsi.c

Modified: head/sys/dev/iscsi/iscsi.c
==============================================================================
--- head/sys/dev/iscsi/iscsi.c	Sun Dec  6 21:31:09 2015	(r291910)
+++ head/sys/dev/iscsi/iscsi.c	Mon Dec  7 02:56:08 2015	(r291911)
@@ -2365,7 +2365,7 @@ iscsi_load(void)
 	sc->sc_cdev->si_drv1 = sc;
 
 	sc->sc_shutdown_eh = EVENTHANDLER_REGISTER(shutdown_pre_sync,
-	    iscsi_shutdown, sc, SHUTDOWN_PRI_FIRST);
+	    iscsi_shutdown, sc, SHUTDOWN_PRI_DEFAULT-1);
 
 	return (0);
 }



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201512070256.tB72u8C6057988>