Re: [PATCH] Add Etron XHCI quirk to avoid warning spam

[Sarah Sharp <sarah.a.sharp@linux.intel.com>]

On Fri, Jul 27, 2012 at 02:40:56PM +0800, Daniel J Blueman wrote:
> On 27 July 2012 14:14, Sarah Sharp <sarah.a.sharp@linux.intel.com> wrote:
> > On Fri, Jul 27, 2012 at 12:03:44PM +0800, Daniel J Blueman wrote:
> >> When various USB3 devices with Etron XHCI controllers, we see a bunch of
> >> warnings:
> >> xhci_hcd 0000:02:00.0: WARN Successful completion on short TX: needs
> >> XHCI_TRUST_TX_LENGTH quirk?
> >>
> >> Acknowledge the issue by adding the quirk.
> >>
> >> Signed-off-by: Daniel J Blueman <daniel@quora.org>
> []
> > I already have a patch in my queue for this.  However, it keys off the
> > PCI_DEVICE_ID_ASROCK_P67 PCI device ID.  Do you have another Etron
> > device with a different device ID that needs this quirk?
> 
> Yes, the subsystem ID is different [1] (but Zotac program it the same
> as the vendor and device IDs here), however what you say suggests the
> problem is general to this Etron XHCI controller (1b6f:7023), as we'd
> suspect anyway.
> 
> Thus the more general patch I posted makes better sense perhaps?

I'd really like to keep this quirk specific to the particular PCI vendor
and device ID.  It's possible that their next chip version will have the
opposite issue (short TX completion code and bad untransferred length).

Your patch turned it on for all Etron hosts, so I would rather keep my
version:

http://git.kernel.org/?p=linux/kernel/git/sarah/xhci.git;a=commit;h=12751f75720391bb2b607acdb2537f02e313251e

I also added a second patch to rate limit the messages for other hosts
that have this issue but don't have the quirk turned on:

http://git.kernel.org/?p=linux/kernel/git/sarah/xhci.git;a=commit;h=5de3270101d30a4a8023815f8010eef2d2d40613

Sarah Sharp