Re: [PATCH] export pci_table in aic7xxx for Hotplug

[christophe barbé]

About the Oops, I think it's a hotplug problem but I could be wrong.

I get a Oops when I do the following :

. Insert my card (card detected and driver loaded)
. mount /dev/sdc0 /cdrw
. use /cdrw
. umount /cdrw
. Remove the card (remove event detected but driver not unloaded)
. mount /dev/sdc0 /cdrw (oops and mount segfault)

lspci show the adaptec card after insertion and no more after removing.

Actually the hotplug stuff doesn't unload the driver. This is not simple
because another card can use the driver. This is a new problem compared
to pcmcia stuff where you use a specific module.

My understanding is that the hotplug code should notify to the driver 
that a device managed by it has been removed. And the driver should
rescan for available devices.

I can send you the oops but I'm convinced that the Oops is caused by the
aic7xxx driver accessing a no more there device.

Is there a common way to ask to a driver to rescan it's devices (an
IOCTL). I'm afraid that not but it looks like a generic hotplug problem.

Christophe

On Tue, Oct 16, 2001 at 06:17:26PM +0200, christophe barbe wrote:
> I've patch my kernel with aic7xxx v6.2.4. The pci_table is correctly
> exported.
> I've a little problem (Oops) when I hot-remove the card and try to mount a
> device no more available. But I believe it's a hotplug issue so I will mail
> details to the hotplug ml.
> 
> Thank,
> Christophe
> 
> Le 2001.10.15 22:31:47 +0200, Justin T. Gibbs a écrit :
> > >I have defined __NO_VERSION__ before including module.h because in my
> > >understanding this is required when you include it in a multi-files
> > module.
> > >Only one file must include module.h without defining the __NO_VERSION__.
> > 
> > I can find no reference to "__NO_VERSION__" in module.h or the files
> > it includes.   Perhaps this is a requirement for old kernels?
> > 
> > >I remember to read something about a repository for your new driver.
> > Please
> > >could you point it to me and I will try it ASAP.
> > 
> > http://people.FreeBSD.org/~gibbs/linux/
> > 
> > --
> > Justin
> > 
> -- 
> Christophe Barbé <[e-mail]>
> GnuPG FingerPrint: E0F6 FADF 2A5C F072 6AF8  F67A 8F45 2F1E D72C B41E
> -
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to [e-mail]
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/

-- 
Christophe Barbé <[e-mail]>
GnuPG FingerPrint: E0F6 FADF 2A5C F072 6AF8  F67A 8F45 2F1E D72C B41E