Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 25 Jun 1998 11:02:52 -0700 (PDT)
From:      Julian Elischer <julian@whistle.com>
To:        Poul-Henning Kamp <phk@FreeBSD.ORG>
Cc:        cvs-committers@FreeBSD.ORG, cvs-all@FreeBSD.ORG, cvs-sys@FreeBSD.ORG
Subject:   Re: cvs commit: src/sys/i386/isa wcd.c src/sys/kern kern_conf.c kern_lkm.c src/sys/sys conf.h src/sys/vm vm_swap.c
Message-ID:  <Pine.BSF.3.95.980625104945.21572A-100000@current1.whistle.com>
In-Reply-To: <Pine.BSF.3.95.980625100054.21522B-100000@current1.whistle.com>

next in thread | previous in thread | raw e-mail | index | archive | help


On Thu, 25 Jun 1998, Julian Elischer wrote:

> hmmmm I should send you my patchces..
> I have removed all the bdevsw in all device drivers that were generating
> the cdevsw from the bdevsw, 
> and reversed bdevsw_add_generic to be cdevsw_add_generic
> which at the moment produces a bdevsw entry from the cdevsw table
> rather than the other way around..
> 
> This may seem like a waste of time as it means I need to go through
> again for a second time to actually remove vestiges of the bdevsw when I
> get further down the track but It looks like we're duplicating work
> here. 
> 
> 
> Poul, look at:
>  no_rawread+cdevsw.diff in my home directory on freefall
> 
Poul:

I have updated this to incorporate these changes of yours.
result of this patch:
No drivers contain filled out bdevsw entries, only cdevsw.
all drivers have their own read/write routine as rawread/rawwrite have
been removed. This becomes important later when you try remove dev_t.


Oops, ignore the patches to devfs, I didn't mean to include them. they
are not yet functional. they will bypass the devsw tables entirely when I
remove the deliberate mistakes. :-)

I have not yet initialised the dump and psize
entries in wd, sd, etc.

I do not include patches I have been working on to 
make all users of bdevs look at the cdevs yet.

that works with devfs as the major Anumbers can become the same.
but it's not generally applicable without it. (I gather you will be
'bouncing' bdevs through the bdevsw to the cdevsw entry to start with?)

> 
> julian
> 
> On Thu, 25 Jun 1998, Poul-Henning Kamp wrote:
> 
> > phk         1998/06/25 04:28:09 PDT
> > 
> >   Modified files:
> >     sys/i386/isa         wcd.c 
> >     sys/kern             kern_conf.c kern_lkm.c 
> >     sys/sys              conf.h 
> >     sys/vm               vm_swap.c 
> >   Log:
> >   Remove bdevsw_add(), change the only two users to use bdevsw_add_generic().
> >   Extend cdevsw to be superset of bdevsw.
> >   Remove non-functional bdev lkm support.
> >   Teach wcd what the open() args mean.
> >   
> >   Revision  Changes    Path
> >   1.55      +27 -53    src/sys/i386/isa/wcd.c
> >   1.25      +7 -2      src/sys/kern/kern_conf.c
> >   1.50      +1 -18     src/sys/kern/kern_lkm.c
> >   1.42      +6 -2      src/sys/sys/conf.h
> >   1.54      +9 -13     src/sys/vm/vm_swap.c
> > 
> 
> 


To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?Pine.BSF.3.95.980625104945.21572A-100000>