NetBSD Problem Report #35067

From www@NetBSD.org  Fri Nov 17 19:42:03 2006
Return-Path: <www@NetBSD.org>
Received: by narn.NetBSD.org (Postfix, from userid 31301)
	id E832263BAFE; Fri, 17 Nov 2006 19:42:03 +0000 (UTC)
Message-Id: <20061117194203.E832263BAFE@narn.NetBSD.org>
Date: Fri, 17 Nov 2006 19:42:03 +0000 (UTC)
From: blair.sadewitz@gmail.com
Reply-To: blair.sadewitz@gmail.com
To: gnats-bugs@NetBSD.org
Subject: reboot(8) could remove swap devices, preserving RAID parity
X-Send-Pr-Version: www-1.0

>Number:         35067
>Category:       bin
>Synopsis:       reboot(8) could remove swap devices, preserving RAID parity
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    bin-bug-people
>State:          open
>Class:          change-request
>Submitter-Id:   net
>Arrival-Date:   Fri Nov 17 19:45:00 +0000 2006
>Last-Modified:  Sat May 30 06:25:01 +0000 2009
>Originator:     Blair Sadewitz
>Release:        NetBSD-4.0_BETA
>Organization:
>Environment:
amd64, 200911090000Z snapshot
>Description:
If the user uses shutdown(8) to reboot, swapoff=YES in /etc/defaults/rc.conf takes care of this.  However, many people may use reboot(8) out of habit, to save time, from a power management script, etc., and in this case RAID parity is disturbed.
>How-To-Repeat:
Use reboot(8) with RAID.  The array is ``dirtied''.
>Fix:
Have a reboot.conf file (awkward, perhaps), or some other means to have reboot remove swap devices?  I'm not sure if you guys would even want to do this, but it should at least be stated in documentation more frequently that shutdown(8) does this by default, yet reboot(8) does not.

Furthermore, RAID parity is not preserved by default when the system is turned off via an ACPI power button request.  It would be helpful if this were addressed as well.

Thanks!

>Audit-Trail:
From: Martin Husemann <martin@duskware.de>
To: gnats-bugs@NetBSD.org
Cc: 
Subject: Re: bin/35067: reboot(8) could remove swap devices, preserving RAID parity
Date: Fri, 17 Nov 2006 21:16:20 +0100

 This has been discussed before, and the consensus was, IIRC, that it has to
 be solved by pure kernel means - swap has to automagically unconfigured via
 shutdown hooks and close the underlying devices.

 Martin

From: "Blair Sadewitz" <blair.sadewitz@gmail.com>
To: gnats-bugs@netbsd.org
Cc: 
Subject: Re: bin/35067: reboot(8) could remove swap devices, preserving RAID parity
Date: Fri, 17 Nov 2006 16:37:50 -0500

 Yes, that was one of the possibilites that I thought of, but I hate
 telling people that they should do something that I can't do. ;)

 One thing that could still be done in the meantime, however, is
 include a note about the distinction between reboot(8) and shutdown(8)
 in, let's say, the guide, one of the manpages, etc.  Let's say an
 administrator wants to quickly reboot, or is having some problem where
 he doesn't want to go through the ordinary shutdown hooks.  He might
 just type 'reboot', thinking that this is clean enough--and not
 thinking that the parity on his 2TB RAID array would be disturbed.
 I'm sure you've all thought these scenarios before, I just think it
 should be more clearly noted.

 Thanks for your time,

 --Blair

 On 11/17/06, Martin Husemann <martin@duskware.de> wrote:
 > The following reply was made to PR bin/35067; it has been noted by GNATS.
 >
 > From: Martin Husemann <martin@duskware.de>
 > To: gnats-bugs@NetBSD.org
 > Cc:
 > Subject: Re: bin/35067: reboot(8) could remove swap devices, preserving RAID
 > parity
 > Date: Fri, 17 Nov 2006 21:16:20 +0100
 >
 >  This has been discussed before, and the consensus was, IIRC, that it has to
 >  be solved by pure kernel means - swap has to automagically unconfigured via
 >  shutdown hooks and close the underlying devices.
 >
 >  Martin
 >
 >


 -- 
 Support WFMU-FM: free-form radio for the masses!

 <http://www.wfmu.org/>
 91.1 FM Jersey City, NJ
 90.1 FM Mt. Hope, NY

 "The Reggae Schoolroom":
 <http://www.wfmu.org/playlists/RS/>

From: "Blair Sadewitz" <blair.sadewitz@gmail.com>
To: gnats-bugs@netbsd.org
Cc: 
Subject: Re: bin/35067: reboot(8) could remove swap devices, preserving RAID parity
Date: Fri, 17 Nov 2006 16:49:28 -0500

 Sorry for the noise, that makes no sense, that would already be done.

From: John Nemeth <jnemeth@netbsd.org>
To: gnats-bugs@NetBSD.org
Cc: 
Subject: PR/35067 CVS commit: src/sbin/reboot
Date: Thu, 24 May 2007 23:19:43 +0000 (UTC)

 Module Name:	src
 Committed By:	jnemeth
 Date:		Thu May 24 23:19:43 UTC 2007

 Modified Files:
 	src/sbin/reboot: reboot.8

 Log Message:
 Add a caveat about reboot(8) not running shutdown scripts.  This addresses
 (at least partially) PR/35067 by Blair Sadewitz.


 To generate a diff of this commit:
 cvs rdiff -r1.21 -r1.22 src/sbin/reboot/reboot.8

 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.

From: Jeff Rizzo <riz@netbsd.org>
To: gnats-bugs@NetBSD.org
Cc: 
Subject: PR/35067 CVS commit: [netbsd-4] src/sbin/reboot
Date: Wed, 30 May 2007 18:27:20 +0000 (UTC)

 Module Name:	src
 Committed By:	riz
 Date:		Wed May 30 18:27:20 UTC 2007

 Modified Files:
 	src/sbin/reboot [netbsd-4]: reboot.8

 Log Message:
 Pull up following revision(s) (requested by jnemeth in ticket #676):
 	sbin/reboot/reboot.8: revision 1.22
 Add a caveat about reboot(8) not running shutdown scripts.  This addresses
 (at least partially) PR/35067 by Blair Sadewitz.


 To generate a diff of this commit:
 cvs rdiff -r1.21 -r1.21.16.1 src/sbin/reboot/reboot.8

 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.

From: Soren Jacobsen <snj@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc: 
Subject: PR/35067 CVS commit: [netbsd-3] src/sbin/reboot
Date: Sat, 30 May 2009 06:20:44 +0000

 Module Name:	src
 Committed By:	snj
 Date:		Sat May 30 06:20:44 UTC 2009

 Modified Files:
 	src/sbin/reboot [netbsd-3]: reboot.8

 Log Message:
 Pull up following revision(s) (requested by jnemeth in ticket #2016):
 	sbin/reboot/reboot.8: revision 1.22
 Add a caveat about reboot(8) not running shutdown scripts.  This addresses
 (at least partially) PR/35067 by Blair Sadewitz.


 To generate a diff of this commit:
 cvs rdiff -u -r1.21 -r1.21.6.1 src/sbin/reboot/reboot.8

 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.

Home	PR Database Search