Re: [PATCH v2 00/10] Multi-segment Event Ring support for XHCI

Mathias Nyman <mathias.nyman@xxxxxxxxxxxxxxx> · Fri, 1 Sep 2023 15:40:00 +0300

On 24.8.2023 19.15, Lukas Wunner wrote:
Enlarge the XHCI Event Ring to cope with high load situations
by allowing more than one segment.

This second iteration seeks to address all of Mathias' review feedback on v1:

* Patch [01/10]:
   Use ERST_DESI_MASK instead of ERST_PTR_MASK when constructing the new
   ERDP value to avoid carrying over a set EHB bit.

* Patch [04/10]:
   Only use up to 2 Event Ring segments by default (instead of 8).

* Patch [05/10]:
   New patch to adjust segment numbers after ring expansion.

* Patch [08/10]:
   New patch to clean up ERST_PTR_MASK inversion.

Several other issues caught my eye and I'm including fixes and cleanups
for those as well.

Thanks for working on this.

Tested this series with a forced transfer ring expansion and all worked well.

I'll add this internally first and then later send it forward after the merge window.

Thanks
Mathias