[PATCH v3][for 4.14] xhci: allow TRACE to work with EVENT ring dequeue

Adam Wallis awallis at codeaurora.org
Mon Sep 25 10:03:58 PDT 2017


On 9/25/2017 12:09 PM, David Laight wrote:
> From: Adam Wallis
>> Sent: 25 September 2017 13:26
>> inc_deq() currently bails earlier for EVENT rings than the common return
>> point of the function, due to the fact that EVENT rings do not have
>> link TRBs. The unfortunate side effect of this is that the very useful
>> trace_xhci_inc_deq() function is not called/usable for EVENT ring
>> debug.
> 
> Is it actually worth using different functions for the different
> ring types?

David, that is what this patch attempts to add/fix. Instead of adding another
TRACE function for event rings, it hooks the existing trace functionality
already used in inc_deq()

> From what I remember there are conditionals in a lot of the functions
> but they are fixed for most of the call sites.

That's basically all this patch is...fixing a conditional. Thanks for the
feedback, let me know if I misunderstood your comments!

> 
> 	David
> 
> 
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
> 


-- 
Adam Wallis
Qualcomm Datacenter Technologies as an affiliate of Qualcomm Technologies, Inc.
Qualcomm Technologies, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project.



More information about the linux-arm-kernel mailing list