On Mon, 2006-02-06 at 15:06 -0800, Patrick Mansfield wrote: > On Mon, Feb 06, 2006 at 04:39:00PM -0500, Jeremy Katz wrote: > Okay with all your comments (thanks) except: > > > On Fri, 2006-02-03 at 14:04 -0800, Patrick Mansfield wrote: > > > > + def shutdown(self): > > [snip] > > > + # Note that iscsid has/had code to ignore 2 (SIGINT), hence the > > > + # 9 (SIGKILL). > > > > If we run iscsid with -p and save the pid in /tmp, then we can have a > > much more robust kill. > > There are really two iscsid processes, I think one is the logger and the > other the true daemon, but only one pid in the pid file, so I can't use > that to kill both. > > There was a shutdown patch submitted (add iscsid '-k' option), that would > work really nice here but it is not in iscsi svn (nor I assume FC devel). That's really broken ;) Mike -- any chance of getting this included so that shutdown of iscsid can be much nicer? It would be good for using in the initscript as well. Just killing processes like this is really bad for cases where people are using chroots and, potentially in the future, other types of containers like the openvz patches that were sent to lkml recently. Jeremy
