Re: [patch] Add i8253 spinlocks where needed.

Alan Cox (alan@lxorguk.ukuu.org.uk)
27 May 2002 12:56:20 +0100


On Mon, 2002-05-27 at 11:10, Vojtech Pavlik wrote:
> Well, probably yes, but still hd.c is a glacial relict, a driver nobody
> (almost - it's for non-IDE "two ribbon" AT harddrives) uses. So this
> driver is probably not enough justification for a global (as in all
> archs) spinlock being added.

It only uses the timer in the case that HD_DELAY > 0. This code is
ultimately used for timing functions. A better approach would be to
remove the use of the timer chip from the file entirely and use the
perfectly adequate udelay() function instead.

That would also conveniently make it do cpu_relax properly improving the
performance of your ancient IDE controller when plugged into
hyperthreading pentium IV 8)

Alan
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/