Messages in this thread | | | Subject | Re: [PATCH] Work around breakage introduced in pata_sil680 by switching it to MMIO | From | Benjamin Herrenschmidt <> | Date | Wed, 26 Mar 2008 19:20:22 +1100 |
| |
On Tue, 2008-03-25 at 23:36 +0000, Alan Cox wrote: > On Wed, 26 Mar 2008 00:31:19 +0100 (CET) > Guennadi Liakhovetski <g.liakhovetski@gmx.de> wrote: > > > pata_sil680 is broken on Linkstation amd Kurobox HG machines since 2.6.24. > > Work around the breakage until a real fix is found. > > > > Signed-off-by: Guennadi Liakhovetski <g.liakhovetski@gmx.de> > > Just disable the mmio patch on all systems - we know it doesn't work, we > know what work needs to be done, it should remain off for everyone until > that work is done. It should never have been merged in the first place > and I think my statement to that effect has been proven nicely.
Agreed. We did some patches to work on the possible write posting issues, and they didn't fix the problem, though they still should go in ultimately (after more testing, I need to dig them back out of my pile).
So I think the MMIO is on standby now until we really corner what's going on. I hope to be able to work a bit on Guennadi's setup one of these days though I can't promise, I may instead get that SiL controller out of a cell blade and stick it in every HW I have around see if I can reproduce the problem. I think I have a couple different revisions even.
We need to keep it enabled for the QS20 cell blades as it's the only way those will boot (and they happen to apparently not suffer from any adverse effect, we've tortured them pretty deeply, could be luck due to how the bridge works on them, or due to other workarounds we do for horrid breakage in the PCI host bridge that end up making everything else work too) but everybody else can stick with PIO.
Jeff, can you ask Linus to pull my patch (the one Alan just acked) that does that in .25 ?
Cheers, Ben.
| |