On Tue, Feb 13, 2001 at 10:24:14AM +0100, Oliver Neukum
<[EMAIL PROTECTED]> wrote:
> > > The attached patch does two things: it creates a new header file called
> > > scsi_dataout.h, which has a single copy of the switch statement (as a
> > > static function -- is that all right?) and is included by the relevant
> > > drivers. I checked that all the drivers had exactly equivalent switch
>
> This means that it would be compiled into the kernel several times.
> You should add it to the core scsi code and _not_ declare it static.
I considered doing it this way. As I saw it, it's a tradeoff between
including the code even if none of the drivers in question are
configured in, or compiling it multiple times if several of them are. I
guess there's also the possibility of an ifdef so that it will be
compiled if at least one of drivers is configured in.
However, in light of other email in this thread, it looks like maybe the
table can go altogether, so hopefully this question is now moot.
-Daniel
--
Daniel E. Eisenbud
[EMAIL PROTECTED]
-
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to [EMAIL PROTECTED]
