[edk2-devel] USB: reducing/removing EHCI and XHCI logging when bulk transfer requests timeout

[Rebecca Cran]

On 11/29/22 13:51, Andrew Fish via groups.io wrote:

>> On Nov 29, 2022, at 11:47 AM, Rebecca Cran <quic_rcran at quicinc.com> wrote:
>>
>> On 11/29/22 12:43, Andrew Fish via groups.io wrote:
>>>> On Nov 29, 2022, at 11:12 AM, Rebecca Cran <quic_rcran at quicinc.com> wrote:
>>>>
>>>> I've been working on the UsbNetworkPkg drivers that Richard Ho submitted. One problem I've run into is that since we poll for bulk requests, most of the time they will timeout - and currently both EHCI and XHCI drivers log errors when that happens, which results in log spam and makes interacting with the system very difficult (e.g. using UiApp).
>>>>
>>>> I was wondering if we might consider adding some mechanism to omit those messages if the user desires, perhaps on a per-build basis?
>>>>
>>> Are those DEBUG prints or report status code?
>> They're DEBUG prints.
> If they happen a lot maybe they should be DEBUG_VERBOSE?

I think the UsbNetworkPkg drivers are probably unique in that the network stack polls for packets every 10ms, which will cause them to issue a bulk transfer request with a timeout of 1ms.

So as long as there aren't packets being sent, the timeout messages will occur continuously. Putting them under DEBUG_VERBOSE would certainly work.

-- 

Rebecca Cran



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#96716): https://edk2.groups.io/g/devel/message/96716
Mute This Topic: https://groups.io/mt/95342129/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-