Subject: Re: CVS commit: src/sys/dev/scsipi
To: None <blymn@baesystems.com.au>
From: YAMAMOTO Takashi <yamt@mwd.biglobe.ne.jp>
List: tech-kern
Date: 08/10/2005 11:07:01
> > if it's generic enough as you say, it shouldn't be in scsipi/.
> >
>
> Yes, I will be moving it - probably to kern/tape_utils.c due to issues
> with some machines not having any scsipi at all. I cannot see any
> neat way of determining whether scsipi is configured in or not so I
> cannot conditionally include "st.h". Unfortunately, this will be dead
> code on non-scsipi machines.
i think you can use config(9) facilities to make any tape devices require
the code. (like wlan does)
i still don't understand why these tape-specific code is necessary, tho.
> > it seems like unnecessary code duplication to me.
> >
>
> There are some error conditions in the disk stats handling that can
> cause a kernel panic, for a disk that makes sense but for a tape these
> errors may not be so critical.
can you give me an example?
can't you solve it by adding some "if (tape)" conditions?
YAMAMOTO Takashi