Re: [PATCH] QLogic ISP2x00: remove needless busyloop

From: Bjorn Helgaas
Date: Mon Aug 09 2004 - 15:27:02 EST


On Monday 09 August 2004 2:03 pm, Christoph Hellwig wrote:
> On Mon, Aug 09, 2004 at 12:52:58PM -0600, Bjorn Helgaas wrote:
> > There's no need to wait for an isp2x00 to recognize a fabric if
> > there's no isp2x00. Probably nobody will notice the unnecessary
> > slowdown on real hardware, but it's a significant delay on a
> > simulator.
>
> Don't use that driver. qla2xxx is the right driver, and qlogicfc is only
> still there and confuses people because davem is lazy and can shout louder
> than others.

I don't use it (note the "there's no isp2x00" bit above). But it's
still part of ia64 defconfig (which I don't maintain), and it's easier
to tell people "use generic_defconfig" than to tell them to remove
CONFIG_SCSI_QLOGIC_FC by hand.

In general, I think if a driver is in the tree, it should be fair
game for bugfixes. In fact, I see you did the most recent one to
qlogicfc :-)
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/