Re: Tidying src/release/*/{dokern.sh,drivers.conf}

Ruslan Ermilov wrote:
> On Sun, May 18, 2003 at 10:45:42AM -0600, Scott Long wrote:
> 
>>Ruslan Ermilov wrote:
>>
>>>Perhaps somebody will find this script useful too.
>>>It finds missing GENERIC entries and cross-checks
>>>dokern.sh and drivers.conf for redundant entries.
>>>
>>>It edits out the "SMALL" part of pc98/dokern.sh
>>>so as to not give false matches.
>>>
>>>It should be run with cwd=/usr/src/release.
>>>
>>>The patch that it "generates" is also attached.
>>>
>>
>>Is this an approval request?
>>
> 
> Yes, why not.
> 
> 
>>See below...
>>
>>
>>>Cheers,
>>>
>>>
>>
>>-----------------------------------------------------------------------
> 
> -
> 
>>>[...]
>>
>>-----------------------------------------------------------------------
> 
> -
> 
>>>Index: release/alpha/dokern.sh
>>>===================================================================
>>>RCS file: /home/ncvs/src/release/alpha/dokern.sh,v
>>>retrieving revision 1.72
>>>diff -u -r1.72 dokern.sh
>>>--- release/alpha/dokern.sh	18 May 2003 03:52:22 -0000	1.72
>>>+++ release/alpha/dokern.sh	18 May 2003 08:26:42 -0000
>>>_at__at_ -23,7 +23,6 _at__at_
>>>	-e '/DDB/d' \
>>>	-e '/INVARIANTS/d' \
>>>	-e '/INVARIANT_SUPPORT/d' \
>>>-	-e '/WITNESS/d' \
>>>	-e '/SMP/d' \
>>>	-e '/	atapifd	/d' \
>>>	-e '/	atapist	/d' \
>>>_at__at_ -52,7 +51,6 _at__at_
>>>	-e '/	umass	/d' \
>>>	-e '/	ums	/d' \
>>>	-e '/	aue	/d' \
>>>-	-e '/	axe	/d' \
>>>	-e '/	cue	/d' \
>>>	-e '/	kue	/d' \
>>>	-e '/	firewire	/d' \
>>>Index: release/pc98/dokern.sh
>>>===================================================================
>>>RCS file: /home/ncvs/src/release/pc98/dokern.sh,v
>>>retrieving revision 1.76
>>>diff -u -r1.76 dokern.sh
>>>--- release/pc98/dokern.sh	18 May 2003 03:52:22 -0000	1.76
>>>+++ release/pc98/dokern.sh	18 May 2003 08:27:24 -0000
>>>_at__at_ -33,8 +33,6 _at__at_
>>>	-e '/	pass	/d' \
>>>	-e '/	ses	/d' \
>>>	-e '/	splash	/d' \
>>>-	-e '/	apm$/d' \
>>>-	-e '/	pmtimer$/d' \
>>>	-e '/	ppc$/d' \
>>>	-e '/	ppbus	/d' \
>>>	-e '/	lpt	/d' \
>>>_at__at_ -98,8 +96,6 _at__at_
>>>	-e '/	pass	/d' \
>>>	-e '/	ses	/d' \
>>>	-e '/	splash	/d' \
>>>-	-e '/	apm$/d' \
>>>-	-e '/	pmtimer$/d' \
>>>	-e '/	ppc$/d' \
>>>	-e '/	ppbus	/d' \
>>>	-e '/	lpt	/d' \
>>
>>Can you explain why you are prososing to add these devices back to
>>te BOOTMFS kernel?  I assume that they were trimmed for a reason.
>>
> 
> This does NOT add any devices back, this just removes redundant
> lines from here:
> 
> $ egrep -w 'apm|pmtimer' /sys/pc98/conf/GENERIC
> #device         apm
> #device         pmtimer
> 
> 

This type of thing can probably distilled down to being a matter of
personal style; I can see good reasons on both side of the argument.
Lets hold off on this until after the release, or at least until after
the release scripts have settled and work on all platforms.

The second part of the patch that restored the ispfw, if_ti, and if_sk
drivers to their respective GENERIC homes is approved.

Scott