> -----Original Message----- > From: Kai Heng Feng [mailto:kai.heng.feng@xxxxxxxxxxxxx] > Sent: Thursday, January 18, 2018 10:57 AM > To: David Miller <davem@xxxxxxxxxxxxx> > Cc: Hayes Wang <hayeswang@xxxxxxxxxxx>; gregkh@xxxxxxxxxxxxxxxxxxx; linux- > usb@xxxxxxxxxxxxxxx; netdev@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; > Limonciello, Mario <Mario_Limonciello@xxxxxxxx>; nic_swsd@xxxxxxxxxxx > Subject: Re: [PATCH v2] r8152: disable RX aggregation on Dell TB16 dock > > > > > On 18 Jan 2018, at 10:50 PM, David Miller <davem@xxxxxxxxxxxxx> wrote: > > > > From: Hayes Wang <hayeswang@xxxxxxxxxxx> > > Date: Thu, 18 Jan 2018 03:04:08 +0000 > > > >> [...] > >>>> r8153 on Dell TB15/16 dock corrupts rx packets. > >>>> > >>>> This change is suggested by Realtek. They guess that the XHCI > >>>> controller doesn't have enough buffer, and their guesswork is correct, > >>>> once the RX aggregation gets disabled, the issue is gone. > >>>> > >>>> ASMedia is currently working on a real sulotion for this issue. > >>>> > >>>> Dell and ODM confirm the bcdDevice and iSerialNumber is unique for TB16. > >>>> > >>>> Note that TB15 has different bcdDevice and iSerialNumber, which are > >>>> not unique values. If you still have TB15, please contact Dell to > >>>> replace it with TB16. > >> > >> Excuse me. I don't understand why this patch is for specific USB nic rather than > xHCI. > >> It seems to make the specific USB nic working and the other ones keeping error. > > > > Well, are we sure that the device being in the TB16 dock doesn't > > contribute to the issue as well? Previous version of this patch checked the parent device to ensure it was in TB16. I believe there was negative feedback to that approach, which prompted the discussion to check bcdDevice and iSerialNumber with all vendors involved. If it's still desirable to analyze parentage tree, I suppose bcdDevice, iSerialNumber and parent's USB device VID/PID can be analyzed all at the same time. > > This is what vendors concluded for now. The very same NIC on WD15 doesn’t > have the issue. And just so it's extra clear to everyone on this list - WD15 has different bcdDevice iSerialNumber, and doesn't connect to ASMedia host controller. ��.n��������+%������w��{.n�����{���)�� jg��� �����ݢj����G���� ���j:+v���w�m������w�������h�����٥
