libusb: hotplug, use events instead of a timer when possible

[bapt]

Summary

Try to fetch events from nlsysevent or devd to determine when
to scan the usb bus for devices addition or removal.
if none are available fallback on the regular timer based (4s)
scanner

[bapt]

push the right patch

[bapt]

unexpected stuff added

[bapt]

rebase

[mohammad_fbsdphab_thelightbird.com]

lib/libusb/libusb10_hotplug.c

57

Hi @bapt,

According to devd(8) '/var/run/devd.pipe' is deprecated in favor for '/var/run/devd.seqpacket.pipe', any reason for still using it here ?

[kevans]

Looks fine in principle

lib/libusb/libusb10_hotplug.c

178–179

This doesn't seem like it would work?

194

IMO "ignore any error" should just be a (void) cast for the return vaule, but I don't feel strongly about it.

194

Where is timeout coming from here? With this and the above, kinda seems like something went wrong with the patch.

[emaste]

lib/libusb/libusb10_hotplug.c

194

Yeah not a huge issue but I agree (void)poll(...) is both a programmer-readable and machine-readable way to indicate that errors are intentionally ignored.

[bapt]

fix bad rebase
switch from sock_stream to sock_seqpacket for devd's pipe

[bapt]

lib/libusb/libusb10_hotplug.c

178–179

sorry bad rebase

194

bad rebase as well

[bapt]

actually read the informations bassed in the sockets to avoid poll looping forever
on libusb_exit close the sockets if any to provide immediate shutdown on the hotplug thread.

fallback on the timer based method when something goes wrong