Linus Torvalds wrote:
On Fri, 26 May 2006, Mark Lord wrote:
Well, this problem has been with us all for a year now,
and at this point it impacts practically *every* new "centrino"
notebook out there.
We have a very simple workaround (previous post) that addresses it
for 2.6.17, and it's about damn time it got fixed.
If there's a better solution for *2.6.17*, then *please* post it.
Otherwise, we have a fix. Maybe Linus or Andrew should just apply it?
I'm definitely in the "at some point, protesting a patch that works
becomes an untenably position to take, no matter _how_ ugly the patch is"
camp.
If the people who complain that it is ugly cannot come up with an
alternate solution that works and isn't ugly, at some point the "ugly"
complaint just becomes totally pointless.
Of course, I'm not on linux-ide, and I didn't see this particular
discussion from the start (or even the alledged simple workaround in the
"previous post"), but can people please fill me in? And if the choice is
not between "ugly" vs "pretty", but between "ugly" vs "nonworking", I
think we know what the answer should be.
Mark is just a slacker, like the rest of us ;-)
The solution, described in [1], is basically "move the delay from <here>
to <there>."
The current code does
resume PCI device
kick the ATA device
when it should do
resume PCI device
bring up the ATA bus
kick the ATA device
Regards,
Jeff
[1] http://marc.theaimsgroup.com/?l=linux-ide&m=114868613527204&w=2
-
: send the line "unsubscribe linux-ide" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
