Hi Sarah, Sarah Sharp <sarah.a.sharp@xxxxxxxxxxxxxxx> writes: > On Wed, Jan 22, 2014 at 11:43:16PM +0100, Arnaud Ebalard wrote: >> Hi Jason, >> >> Jason Cooper <jason@xxxxxxxxxxxxxx> writes: >> >> > On Wed, Jan 22, 2014 at 11:23:23PM +0100, Arnaud Ebalard wrote: >> >> With the patch applied on top of 3.13.0 kernel recompiled w/ >> >> CONFIG_PCI_MSI enabled, I cannot reproduce the bug. I guess >> >> you can add my: >> >> >> >> Reported-and-tested-By: Arnaud Ebalard <arno@xxxxxxxxxxxx> >> >> >> >> Since you'll have to push the patch to -stable team at least for 3.13, >> >> I wonder if it would not make sense to extend that at least to 3.12. >> >> and possibly 3.10 (3.2 is still widely used but I wonder if it makes >> >> sense to go that far). >> > >> > Can you pinpoint the commit that introduced the regression? >> >> f5182b4155b9d686c5540a6822486400e34ddd98 "xhci: Disable MSI for some Fresco Logic hosts." >> >> Technically, this is not per se the commit which introduced the >> regression but the one that *partially* fixed it by introducing the XHCI >> quirk to skip MSI enabling for Fresco Logic chips. The thing is it >> should have included the FL1009 in the targets. Sarah, can you confirm >> this? > > I don't know if it should have included FL1009, it was just a guess, > based on the fact that the 0x1000 and 0x1400 devices did need MSI > disabled. I can attempt to ask the Fresco Logic folks I know, but I'm > not sure if/when I'll get a response back. > > That still doesn't necessarily rule out MSI issues in the Marvell PCI > host controller code. Can you attach another PCI device with MSI > support under the host and see if it works? Unless you have some objections or some positive feedback from Fresco Logic people, can you queue your quirks for FL1009 for 3.14-rc* and -stable? Note that I am just asking, i.e. if you want to wait a bit more, I am not that in a hurry. Cheers, a+ -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
