libusb: emit event when deregister event
ClosedPublic
Actions

Authored by aokblast on May 5 2025, 5:54 PM.

Details

Reviewers

Commits

rG641bb44d8538: libusb: Emit event after deregistering hotplug handler
rGd28526591803: libusb: Emit event after deregistering hotplug handler

Summary

The original implementation emit event when deregister hotplug handler.
Without this, some program follows the best practice from libusb will stuck on waiting an event to trigger after deregister the hotplug handler.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 63913
Build 60797: arc lint + arc unit

Event Timeline

aokblast created this revision.May 5 2025, 5:54 PM

Herald added a subscriber: imp. · View Herald TranscriptMay 5 2025, 5:54 PM

aokblast requested review of this revision.May 5 2025, 5:54 PM

Harbormaster completed remote builds in B63912: Diff 154888.May 5 2025, 5:54 PM

Fix commit message

Harbormaster completed remote builds in B63913: Diff 154889.May 5 2025, 5:55 PM

See https://github.com/libusb/libusb/blob/master/libusb/hotplug.c#L459

emaste added a subscriber: emaste.May 6 2025, 2:18 PM

lwhsu added a subscriber: lwhsu.May 6 2025, 8:02 PM

aokblast edited the summary of this revision. (Show Details)May 7 2025, 1:52 PM

While this looks good, I think the commit log should be way more verbose.

Elaborate on the commit message

Harbormaster completed remote builds in B63994: Diff 155062.May 7 2025, 3:43 PM

aokblast edited the summary of this revision. (Show Details)May 7 2025, 3:44 PM

bapt accepted this revision.May 7 2025, 4:14 PM

This revision is now accepted and ready to land.May 7 2025, 4:14 PM

For somebody want to merge this patch, thanks:
https://github.com/aokblast/freebsd-src/commit/8548727194b05ff97fb48e6406a6f26f7f54e593

Closed by commit rGd28526591803: libusb: Emit event after deregistering hotplug handler (authored by aokblast, committed by emaste). · Explain WhyMay 8 2025, 6:21 PM

This revision was automatically updated to reflect the committed changes.

emaste added a commit: rGd28526591803: libusb: Emit event after deregistering hotplug handler.

emaste added a commit: rG641bb44d8538: libusb: Emit event after deregistering hotplug handler.May 14 2025, 2:24 PM

Revision Contents
Changeset List

Path

Size

lib/

libusb/

libusb10_hotplug.c

1 line

libusb10_io.c

1 line

Diff 154889

View Options

lib/libusb/libusb10_hotplug.c