From joel@automatvapen.se  Wed Dec  8 18:39:51 2004
Return-Path: <joel@automatvapen.se>
Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP id A17F716A4CE
	for <FreeBSD-gnats-submit@freebsd.org>; Wed,  8 Dec 2004 18:39:51 +0000 (GMT)
Received: from av8-1-sn3.vrr.skanova.net (av8-1-sn3.vrr.skanova.net [81.228.9.183])
	by mx1.FreeBSD.org (Postfix) with ESMTP id E7A8043D5A
	for <FreeBSD-gnats-submit@freebsd.org>; Wed,  8 Dec 2004 18:39:48 +0000 (GMT)
	(envelope-from joel@automatvapen.se)
Received: by av8-1-sn3.vrr.skanova.net (Postfix, from userid 502)
	id BAEDC38024; Wed,  8 Dec 2004 19:39:47 +0100 (CET)
Received: from smtp3-1-sn3.vrr.skanova.net (smtp3-1-sn3.vrr.skanova.net [81.228.9.101])
	by av8-1-sn3.vrr.skanova.net (Postfix) with ESMTP id A7BCE37EA8
	for <FreeBSD-gnats-submit@freebsd.org>; Wed,  8 Dec 2004 19:39:47 +0100 (CET)
Received: from dude.automatvapen.se (t9o55p109.telia.com [81.225.221.109])
	by smtp3-1-sn3.vrr.skanova.net (Postfix) with SMTP id B52E237E45
	for <FreeBSD-gnats-submit@freebsd.org>; Wed,  8 Dec 2004 19:39:46 +0100 (CET)
Received: by dude.automatvapen.se (sSMTP sendmail emulation); Wed,  8 Dec 2004 19:39:57 +0100
Message-Id: <20041208183946.B52E237E45@smtp3-1-sn3.vrr.skanova.net>
Date: Wed,  8 Dec 2004 19:39:57 +0100
From: "Joel Dahl" <joel@automatvapen.se>
Reply-To: Joel Dahl <joel@automatvapen.se>
To: FreeBSD-gnats-submit@freebsd.org
Cc:
Subject: [patch] update tuning(7) man page
X-Send-Pr-Version: 3.113
X-GNATS-Notify:

>Number:         74862
>Category:       docs
>Synopsis:       [patch] update tuning(7) man page
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    keramida
>State:          closed
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          doc-bug
>Submitter-Id:   current-users
>Arrival-Date:   Wed Dec 08 18:40:24 GMT 2004
>Closed-Date:    Thu Dec 23 00:34:28 GMT 2004
>Last-Modified:  Thu Dec 23 00:34:28 GMT 2004
>Originator:     Joel Dahl
>Release:        FreeBSD 5.3-STABLE i386
>Organization:
>Environment:
System: FreeBSD dude.automatvapen.se 5.3-STABLE FreeBSD 5.3-STABLE #1: Sat Nov 13 19:50:36 CET 2004 joel@dude.automatvapen.se:/usr/obj/usr/src/sys/WRK i386

	
>Description:
(1) Remove reference to IDE_DELAY.  I'm not sure
    that my memory is correct, but I think that
    it was removed a long time ago, so I don't
    see any point in keeping it here.

(2) Reword things a bit since SCSI_DELAY defaults
    to 5000 milliseconds in CURRENT, not 15000.

	
>How-To-Repeat:
	
>Fix:

	
--- tuning.diff begins here ---
Index: tuning.7
===================================================================
RCS file: /home/ncvs/src/share/man/man7/tuning.7,v
retrieving revision 1.69
diff -u -r1.69 tuning.7
--- tuning.7	16 Nov 2004 21:22:09 -0000	1.69
+++ tuning.7	8 Dec 2004 18:03:44 -0000
@@ -814,17 +814,12 @@
 by a megabyte or more, leaving more memory available for applications.
 .Pp
 .Dv SCSI_DELAY
-and
-.Dv IDE_DELAY
 may be used to reduce system boot times.
 The defaults are fairly high and
-can be responsible for 15+ seconds of delay in the boot process.
+can be responsible for 5+ seconds of delay in the boot process.
 Reducing
 .Dv SCSI_DELAY
-to 5 seconds usually works (especially with modern drives).
-Reducing
-.Dv IDE_DELAY
-also works but you have to be a little more careful.
+to something below 5 seconds could work (especially with modern drives).
 .Pp
 There are a number of
 .Dv *_CPU
--- tuning.diff ends here ---


>Release-Note:
>Audit-Trail:

From: Giorgos Keramidas <keramida@freebsd.org>
To: Joel Dahl <joel@automatvapen.se>
Cc: bug-followup@freebsd.org
Subject: Re: docs/74862: [patch] update tuning(7) man page
Date: Thu, 9 Dec 2004 03:54:43 +0200

 On 2004-12-08 19:39, Joel Dahl <joel@automatvapen.se> wrote:
 > (1) Remove reference to IDE_DELAY.  I'm not sure that my memory is
 >     correct, but I think that it was removed a long time ago, so I
 >     don't see any point in keeping it here.
 
 IDE_DELAY is still used by the wd(4) driver of PC98.  Conditional
 inclusion of text in manpages is not possible AFAIK, so we could
 probably reword that part a bit to make it more obvious that it only
 applies to PC98.
 
 > (2) Reword things a bit since SCSI_DELAY defaults to 5000 milliseconds
 >     in CURRENT, not 15000.
 
 This part looks good.
 

From: Joel Dahl <joel@automatvapen.se>
To: Giorgos Keramidas <keramida@freebsd.org>
Cc: bug-followup@freebsd.org
Subject: Re: docs/74862: [patch] update tuning(7) man page
Date: Thu, 09 Dec 2004 16:36:34 +0100

 --=-FpG7KYPhzL6Yn4BKolKp
 Content-Type: text/plain
 Content-Transfer-Encoding: 7bit
 
 On Thu, 2004-12-09 at 03:54 +0200, Giorgos Keramidas wrote:
 > On 2004-12-08 19:39, Joel Dahl <joel@automatvapen.se> wrote:
 > > (1) Remove reference to IDE_DELAY.  I'm not sure that my memory is
 > >     correct, but I think that it was removed a long time ago, so I
 > >     don't see any point in keeping it here.
 > 
 > IDE_DELAY is still used by the wd(4) driver of PC98.  Conditional
 > inclusion of text in manpages is not possible AFAIK, so we could
 > probably reword that part a bit to make it more obvious that it only
 > applies to PC98.
 
 New patch attached. Any better?
 
 (I had to check src/sys/pc98/pc98/wd.c for a reference to IDE_DELAY. If
 this is an option available for the kernel configuration, why doesn't
 NOTES for PC98 mention it?)
 
 > 
 > > (2) Reword things a bit since SCSI_DELAY defaults to 5000 milliseconds
 > >     in CURRENT, not 15000.
 > 
 > This part looks good.
 > 
 
 Thanks. :-)
 
 --=-FpG7KYPhzL6Yn4BKolKp
 Content-Disposition: attachment; filename=tuning2.diff
 Content-Type: text/x-patch; name=tuning2.diff; charset=us-ascii
 Content-Transfer-Encoding: 7bit
 
 Index: tuning.7
 ===================================================================
 RCS file: /home/ncvs/src/share/man/man7/tuning.7,v
 retrieving revision 1.69
 diff -u -r1.69 tuning.7
 --- tuning.7	16 Nov 2004 21:22:09 -0000	1.69
 +++ tuning.7	9 Dec 2004 15:17:45 -0000
 @@ -814,17 +814,15 @@
  by a megabyte or more, leaving more memory available for applications.
  .Pp
  .Dv SCSI_DELAY
 -and
 -.Dv IDE_DELAY
  may be used to reduce system boot times.
  The defaults are fairly high and
 -can be responsible for 15+ seconds of delay in the boot process.
 +can be responsible for 5+ seconds of delay in the boot process.
  Reducing
  .Dv SCSI_DELAY
 -to 5 seconds usually works (especially with modern drives).
 +to something below 5 seconds could work (especially with modern drives).
  Reducing
  .Dv IDE_DELAY
 -also works but you have to be a little more careful.
 +(only available for PC98) also works but you have to be a little more careful.
  .Pp
  There are a number of
  .Dv *_CPU
 
 --=-FpG7KYPhzL6Yn4BKolKp--
 
State-Changed-From-To: open->patched 
State-Changed-By: keramida 
State-Changed-When: Thu Dec 9 15:53:49 GMT 2004 
State-Changed-Why:  
I committed a slightly modified version of your last patch 
(one that minimizes the lines affected by the change). 

http://www.freebsd.org/cgi/query-pr.cgi?pr=74862 
State-Changed-From-To: patched->closed 
State-Changed-By: keramida 
State-Changed-When: Thu Dec 23 00:33:08 GMT 2004 
State-Changed-Why:  
I've now merged this to RELENG_5.  Thanks! 


Responsible-Changed-From-To: freebsd-doc->keramida 
Responsible-Changed-By: keramida 
Responsible-Changed-When: Thu Dec 23 00:33:08 GMT 2004 
Responsible-Changed-Why:  

http://www.freebsd.org/cgi/query-pr.cgi?pr=74862 
>Unformatted:
