Re: [PATCH] 2nd go for scsi upper layers + I2O

From: Torben Mathiasen (torben@kernel.dk)
Date: Fri Oct 06 2000 - 10:29:04 EST


On Fri, Oct 06 2000, Douglas Gilbert wrote:
> Torben Mathiasen wrote:
>
> > Ok this patch should be diffed correctly. Same things apply:
> >
> > apply patch
> > copy sd.c st.c sg.c sr.c sr_ioctl.c sr_vendor.c from
> > drivers/scsi to drivers/scsi/upper
> >
> > The EXPORT_SYMBOL has been removed as Jeff suggested.
> >
> > TLAN will hopefully follow soon.....
>
> [snipped most of patch, see lkml]
>
> > +sg_mod.o: $(sg_mod-objs)
> > + $(LD) -r -o $@ $(sg_mod-objs)
>
> Firstly, I just spent 2 months trying to get the sd module
> name reverted to "sd_mod.o" as it is in lk 2.2 and 2.0 .Now
> this patch seems to rename the sg module to "sg_mod.o"!
> Since the vast majority of distributions build sg as a module,
> there could be a lot of irate SANE and cdrecord users out
> there. Also devfsd and other module loading software would
> need to be changed. Hopefully this is an oversight.
>

Sure, no problem.

> Secondly, do we really need the scsi Makefile and directory
> structure changed this close to the lk 2.4 release?
> What does it gain us? Could changes of this dimension be
> sent to Eric Youngdale or at least the linux-scsi list
> rather than just sent to the linux-kernel list?

First of all I _did_ send a patch off to linux-scsi, and this
issue _has_ been discussed with Eric.

And yes its needed. The lowlevel scsi drivers needs to link
in before the high level ones. Alan posted this to l-k.

> You need to get the i2o scsi driver run last afte the low level and before
> the high level drivers despite being in drivers/i2o. There are some other
> drivers which arent in drivers/scsi too
>

Maybe you should take a look at the archives before bitching.

Now I'm tired of fixes this and that. Can we agree on
this patch going in when I rename sg_mod.o to sg.o???

-- 
Torben Mathiasen <tmm@kernel.dk>
Linux ThunderLAN maintainer 
http://tlan.kernel.dk
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Sat Oct 07 2000 - 21:00:19 EST