Thread (6 messages) flat view 6 messages, 3 authors, 2017-10-05
STALE3271d

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

From: Adam Wallis <hidden>
Date: 2017-09-25 17:03:58

On 9/25/2017 12:09 PM, David Laight wrote:
From: Adam Wallis
quoted
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.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help