Re: rmmod st "hangs" - bad interaction with sg

[Tim Wright <timw@splhi.com>]

Hi Christoph!
On Sat, 2004-07-10 at 08:41, Christoph Hellwig wrote:
> On Sat, Jul 10, 2004 at 08:31:00AM -0700, Tim Wright wrote:
> > Hi,
> > I was working on the qlogicisp/isp1020 driver in 2.6, as I still have
> > one of these antiques and the driver is a bit out of date (a patch is
> > forthcoming). In the process of testing my changes, I came across the
> > following:
> 
> qlogicisp is slowly going away.  If you look at the qla1280 driver in current
> mainline you'll see it has most of the support for the 1020/1040 already,
> I just need to fix a final bug and add firmware/pci ids.  This has come up
> on linux-scsi a few times..
> 

Ah thanks. Of course, as soon as I posted, I realized I should probably
have copied linux-scsi and/or gone and checked the archives there sigh.
Just getting back into the swing of things. If you need a tester, I'm
happy to do so on my setup.

> > This seems bad to me - either the original rmmod should fail with EBUSY,
> > or it should complete. However, for it to do so, it seems that st needs
> > to know that sg has its hooks into the device it controls, and it needs
> > to be able to make it let go. My workaround is impractical if sg is in
> > use on other devices too.
> 
> I don't think we can fix much about this, it's how the driver model code
> works.  Best workarond is to not use sg.

Hmmm... the problem was that it was autoloaded - I haven't gone and
checked the code yet, but it seems that unless you don't actually build
sg at all, it gets pulled in whether you want it or not. Now that the
ATA passthrough stuff works for CDRW etc., I actually have no good
reason to load it. It just seemed a little "sucky" - if I had tried to
e.g. rmmod qlogicisp, it would correctly fail because st depends on it.
It seems weak that an attempt to unload st didn't catch the hidden
dependency. Actually, it seems that a workaround with no side effects is
to 'echo 1 > /sys/class/scsi_device/.../delete' for the tape drive. That
frees things up and unhooks sg without unloading it.

Thanks,

Tim