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

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

  • Next message: David Weinehall: "Re: v2.4.0test9 NFSv3 server woes Linux-->Solaris"

    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 : Fri Oct 06 2000 - 11:29:16 EDT