Differential D26064

Check XHCI endpoint state before issuing XHCI endpoint commands
ClosedPublic
Actions

Authored by • hselasky on Aug 14 2020, 10:07 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Dec 14, 1:17 AM

	Unknown Object (File)
	Sun, Nov 30, 10:03 AM

	Unknown Object (File)
	Sat, Nov 29, 2:05 AM

	Unknown Object (File)
	Wed, Nov 26, 6:42 PM

	Unknown Object (File)
	Sun, Nov 23, 7:48 AM

	Unknown Object (File)
	Sun, Nov 23, 2:48 AM

	Unknown Object (File)
	Nov 20 2025, 9:05 PM

	Unknown Object (File)
	Nov 20 2025, 7:41 PM

Subscribers

Details

Reviewers

kp
bz

Commits

rS365240: MFC r364347:
rS365239: MFC r364347:
rS364347: Check the XHCI endpoint state before issuing XHCI endpoint commands.

Summary

Sponsored by: Mellanox Technologies

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 32990

Event Timeline

• hselasky created this revision.Aug 14 2020, 10:07 AM

Herald added a subscriber: imp. · View Herald TranscriptAug 14 2020, 10:07 AM

• hselasky requested review of this revision.Aug 14 2020, 10:07 AM

Harbormaster completed remote builds in B32987: Diff 75817.Aug 14 2020, 10:07 AM

bz accepted this revision.Aug 14 2020, 10:13 AM

bz added a subscriber: bz.

bz added inline comments.

sys/dev/usb/controller/xhci.c
3826	I assume it's fairly save to assume epno >= 1 here?

This revision is now accepted and ready to land.Aug 14 2020, 10:13 AM

• hselasky added inline comments.Aug 14 2020, 10:32 AM

sys/dev/usb/controller/xhci.c
3868	@bz : A value of zero is checked here.

• hselasky marked an inline comment as done.Aug 14 2020, 10:32 AM

pstef added a subscriber: pstef.Aug 14 2020, 10:41 AM

pstef added inline comments.

sys/dev/usb/controller/xhci.c
3826	Since it's checked somewhere else, maybe it'd be a good idea to put here an assertion?

Assert epno != 0.

This revision now requires review to proceed.Aug 14 2020, 10:47 AM

Harbormaster completed remote builds in B32990: Diff 75820.Aug 14 2020, 10:47 AM

bz accepted this revision.Aug 14 2020, 2:27 PM

This revision is now accepted and ready to land.Aug 14 2020, 2:27 PM

For what it's worth this doesn't fix the issues I'm seeing with a Corigine xHCI controller.
I wonder if it's worth adding explicit cases for ERROR (do nothing) and DISABLED (do nothing).

@kp: Feel free to comander this patch.

While not the fix for my problem I do believe this patch is correct, and aligns us better to the xHCI Endpoint State Diagram (xhci v1.1, 4.8.3 Endpoint Context State).

I believe my problem is that the state transitions for EP0 are different. Other OSs (Linux, NetBSD) simply skip the entire stop/set tr dequeue /... configure dance for EP0. This patch tries to do that for us: https://pastebin.com/hudthnUZ and does fix the issues I see with Corigine xHCI.

Closed by commit rS364347: Check the XHCI endpoint state before issuing XHCI endpoint commands. (authored by • hselasky). · Explain WhyAug 18 2020, 3:44 PM

This revision was automatically updated to reflect the committed changes.

• hselasky added a commit: rS364347: Check the XHCI endpoint state before issuing XHCI endpoint commands..

• hselasky added a commit: rS365239: MFC r364347:.Sep 2 2020, 10:10 AM

• hselasky added a commit: rS365240: MFC r364347:.

Revision Contents
Changeset List

Path

Size

sys/

dev/

usb/

controller/

46 lines

Diff 75820

sys/dev/usb/controller/xhci.c

Loading...