Jeff,
Regarding your June 22nd post:
14) Surely there is a better way to down the adapter than masking the
interrupts and resetting the adapter? If this is ever used in non-MSI
situations (common in Linux today), there is the possibility of
screaming interrupts, in shared-interrupt
Ron Mercer wrote:
Jeff,
Regarding your June 22nd post:
14) Surely there is a better way to down the adapter than masking the
interrupts and resetting the adapter? If this is ever used in non-MSI
situations (common in Linux today), there is the possibility of
screaming interrupts, in
Freeing a kernel interrupt reservation does not imply that
your hardware has stopped delivering them... in a shared
interrupt situation, the code sequence
free_irq()
perform some action
could cause the kernel to call _another_ driver's interrupt
handling routine over
Is the device hotpluggable? If so, this:
qdev-index = cards_found;
in the probe() handler might do odd things - it'll just keep
increasing as the card is removed and re-added.
iirc, that's a common problem with net drivers. AFAICT it'll
cause only cosmetic oddities here.
Ron Mercer [EMAIL PROTECTED] wrote:
Is the device hotpluggable? If so, this:
qdev-index = cards_found;
in the probe() handler might do odd things - it'll just keep
increasing as the card is removed and re-added.
iirc, that's a common problem with net drivers. AFAICT
Ron Mercer wrote:
Is the device hotpluggable? If so, this:
qdev-index = cards_found;
in the probe() handler might do odd things - it'll just keep
increasing as the card is removed and re-added.
iirc, that's a common problem with net drivers. AFAICT it'll
cause only cosmetic
On Wed, 28 Jun 2006 16:07:22 PDT, Ron Mercer wrote:
Is the device hotpluggable? If so, this:
qdev-index = cards_found;
in the probe() handler might do odd things - it'll just keep
increasing as the card is removed and re-added.
iirc, that's a common problem with net
Andrew Morton,
The attached patch fixes the compile bugs you indicated plus some of
Jeff Garzik's concerns.
- Removed non-NAPI code.
- Removed un-needed PCI_POSTING macro.
- Converted msleep() to ssleep() where waiting 1 sec.
- Broke up ql_link_state_machine into two functions for indent