Re: [PATCH v3 3/6] hw/usb/hcd-xhci-pci: Use event ring 0 if mapping unsupported

[Philippe Mathieu-Daudé]

On 27/12/24 13:13, Phil Dennis-Jordan wrote:
> The XHCI specification, section 4.17.1 specifies that "If the
> Number of Interrupters (MaxIntrs) field is greater than 1, then
> Interrupter Mapping shall be supported." and "If Interrupter
> Mapping is not supported, the Interrupter Target field shall be
> ignored by the xHC and all Events targeted at Interrupter 0."
>
> QEMU's XHCI device has so far not specially addressed this case,
> so we add a check to xhci_event() to redirect to event ring and
> interrupt 0 if mapping is disabled.
>
> Signed-off-by: Phil Dennis-Jordan <phil@philjordan.eu>
> ---
>   hw/usb/hcd-xhci.c | 4 ++++
>   1 file changed, 4 insertions(+)

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>