Paths

Table of Contentst

bhyve: usb: marginally improve debugging output
AcceptedPublic
Actions

Authored by kevans on Fri, Sep 18, 1:58 AM.

Tags

None

Referenced Files

	F175340867: D59783.id187022.diff
	Sat, Oct 10, 3:17 AM

	Unknown Object (File)
	Thu, Oct 8, 3:26 AM

	Unknown Object (File)
	Tue, Oct 6, 3:46 PM

	Unknown Object (File)
	Sun, Oct 4, 6:36 AM

	Unknown Object (File)
	Fri, Oct 2, 12:26 PM

	Unknown Object (File)
	Thu, Oct 1, 1:11 PM

	Unknown Object (File)
	Thu, Oct 1, 7:11 AM

	Unknown Object (File)
	Wed, Sep 30, 10:43 PM

Subscribers

Details

Reviewers

markj
bnovkov

Group Reviewers

bhyve

Summary

Note when we pci_xhci_assert_interrupt, and add the trb type to
pci_xhci_cmd_reset_ep. These are both relatively straightforward.

Sponsored by: The FreeBSD Foundation (aokblast)

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 77049
Build 73932: arc lint + arc unit

Event Timeline

kevans created this revision.Fri, Sep 18, 1:58 AM

Herald added subscribers: bcran, rgrimes, imp. · View Herald TranscriptFri, Sep 18, 1:58 AM

kevans requested review of this revision.Fri, Sep 18, 1:58 AM

Harbormaster completed remote builds in B77049: Diff 187022.Fri, Sep 18, 1:58 AM

kevans added a parent revision: D59782: bhyve: usb: plug some leaks in bus init failure path.Fri, Sep 18, 1:58 AM

kevans added a child revision: D59785: bhyve: usb: properly handle port init/deinit/power-on.Fri, Sep 18, 2:28 AM

markj accepted this revision as: markj.Fri, Sep 18, 8:12 PM

This revision is now accepted and ready to land.Fri, Sep 18, 8:12 PM

One minor comment, LGTM otherwise.

usr.sbin/bhyve/pci_xhci.c
1208	Maybe it would make sense to add an `%s` modifier to the `DPRINTF` macro.

Revision Contents
Changeset List

Path

Size

usr.sbin/

bhyve/

pci_xhci.c

6 lines

Diff 187022

View Options

bhyve: usb: marginally improve debugging outputAcceptedPublicActions