Re: PATCH: clean up the IDE iops, add ones for a dead iface

Alan Cox (alan@lxorguk.ukuu.org.uk)
19 Feb 2003 14:25:39 +0000


On Wed, 2003-02-19 at 09:39, Benjamin Herrenschmidt wrote:
> Hrm... I tend to agree with Russell here... 0x7f is the "safe" value
> for IDE. IDE controllers with nothing wired shall have a pull down
> on D7. The reason is simple: busy loops in the IDE code waiting for
> BSY to go down.
>
> Now, if your point is to keep BSY up and have wait loops timeout,
> then 0xff may actually make some sense ;)

0xFF is what happens with a real hot unplug so we must handle it
irrespective of convenience. Your probe loop might want to check
some kind of 'dead' variable when this is all done I guess. That
way we can have

unplugged_event()
{
change iops
drive->dead = 1;
}

and the probe code can spot ->dead in the poll loop

-
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/