| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
| |
before usbd_close_pipe(), since usbd_close_pipe() already takes care
about aborting non-empty pipes.
As investigated by gerhard@ usbdi.c rev. 1.57 did add usbd_abort_pipe()
to usbd_close_pipe(), but the drivers didn't get cleaned up afterwards.
ok gerhard@
|
|
|
|
|
|
|
|
|
|
| |
when no IO endpoints were found. Coverity CID 1453263, 1453156
Check return value of usbd_interface2endpoint_descriptor() in
alloc_all_endpoints_yamaha() as it may return NULL which ends up being
dereferenced by the UE_GET_XFERTYPE() macro. Coverity CID 271356
ok ratchov@
|
| |
|
|
|
|
| |
ok ratchov@
|
| |
|
|
|
|
|
|
| |
USBD_INVAL.
ok mpi@
|
|
|
|
|
|
| |
in free_all_jacks().
ok stsp@ ratchov@
|
|
|
|
|
|
|
| |
and attach_all_mididevs(). this ensures unbind_all_jacks() and
usbd_deactivate() happen for both cases.
ok ratchov@
|
|
|
|
| |
ok deraadt@ krw@
|
|
|
|
|
| |
Errors will result in nice clean panic messages so we know what's wrong.
Reviewed by dhill visa natano jsg.
|
|
|
|
| |
code. From Michael W. Bombardieri" <mb at ii.net>. Thanks.
|
|
|
|
| |
From Michael W. Bombardieri. Thanks.
|
|
|
|
| |
routine. From Michael W. Bombardieri <mb at ii.net>. Thanks!
|
|
|
|
|
|
|
| |
have any direct symbols used. Tested for indirect use by compiling
amd64/i386/sparc64 kernels.
ok tedu@ deraadt@
|
|
|
|
| |
ok deraadt@ tedu@
|
|
|
|
| |
after discussions with beck deraadt kettenis.
|
|
|
|
| |
Okay mpi@
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
structures (including sound-card registers) from concurent
access by syscall and interrupt code-paths. Since critical
sections remain the same, calls to splraise/spllower can be
safely replaced by calls to mtx_enter/mtx_leave with two
exceptions: (1) mutexes are not reentrant (the inner splraise
is thus removed), and (2) we're not allowed to sleep with a
mutex (either msleep is used or the mutex is released before
sleeping).
ok and help from kettenis, a lot of work from armani
|
|
|
|
|
|
|
|
|
| |
definitions instead. We don't change usb.h for now to stay compatible
with userland.
Tested by mpi@ on macppc and myself on i386.
ok mpi@
|
| |
|
|
|
|
|
| |
absolutely do not need to include vnode.h because it includes uvm_extern.h
and you want the idiotic TRUE FALSE defines from uvm.
|
|
|
|
|
|
|
| |
just return and make the midi(4) driver retry later. UART buffers
are large enough for this, except eap(4) which uses interrupts for
output.
help from and ok jsg@
|
|
|
|
|
|
|
|
| |
that's ever used it, and it's long since been changed to use
DVACT_{QUIESCE,SUSPEND,RESUME} instead.
ok deraadt@, dlg@; miod@ also agreed with this idea when I brought it
up a few weeks ago
|
|
|
|
| |
This one wanted some more comments before commit.
|
|
|
|
|
|
|
| |
If one of the usb calls we did here failed we'd return immediately and
not free our buffer.
ok miod@
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
1) Allocating with M_WAITOK, checking for NULL, and calling panic() is
pointless (malloc() will panic if it can't allocate) so remove the check
and the call.
2) Allocating with M_WAITOK, checking for NULL, and then gracefully
handling failure to allocate is pointless. Instead also pass M_CANFAIL
so malloc() doesn't panic so we can actually handle it gracefully.
1) was done using Coccinelle.
Input from oga.
ok miod.
|
|
|
|
|
|
| |
them from userland, and nothing in the kernel uses them.
ok krw@, miod@
|
|
|
|
|
|
| |
The dying flag will be set in activate()/DEACTIVATE.
ok deraadt@
|
|
|
|
|
|
| |
This is needed for the addition of further suspend/resume actions.
Okay deraadt@, marco@.
|
|
|
|
|
|
|
|
|
| |
Not sure what's more surprising: how long it took for NetBSD to
catch up to the rest of the BSDs (including UCB), or the amount of
code that NetBSD has claimed for itself without attributing to the
actual authors.
OK deraadt@
|
|
|
|
|
|
| |
so that it shows up before the :. as a result, all the usb devices do not
need to have name printing code anymore. all this now works and prints
nicely because usbd_probe_and_attach() is serialized. ok kettenis
|
|
|
|
|
| |
otherwise just adding M_ZERO to malloc() and removing the immediately
adjacent memset(,0,).
|
|
|
|
|
|
|
|
| |
USB_DECLARE_DRIVER macros.
No binary change.
ok dlg.
|
|
|
|
|
|
| |
No binary change.
ok mk.
|
|
|
|
|
|
|
| |
(This might look easy, but it was a big diff. Thanks to dlg and especially jsg
for looking over it; we found at least four mistakes in the initial diff.)
ok jsg.
|
|
|
|
|
|
|
|
| |
and the address of it's argument: USBDEV(sc->sc_dev) yields &sc->sc_dev.
No binary changes.
ok jsg.
|
|
|
|
| |
ok deraadt@ krw@ mbalmer@
|
| |
|
|
|
|
|
|
| |
something, then it MUST return zero on DVACT_ACTIVATE, not EOPNOTSUPP;
this very popular bug has been cut and pasted a lot of times...
ok deraadt@ mickey@
|
|
|
|
|
|
|
| |
Allow umidi(4) to send multiple events in a single USB transfer.
This greatly improves the number of interrupts umidi is able to generate.
From Alexandre Ratchov.
|
|
|
|
|
| |
userland-visible sys/select.h. Consistent with what Net and Free do.
OK deraadt@, tested with full ports build by naddy@.
|
|
|
|
|
|
|
|
|
|
| |
when a single real time byte is inserted.
From Alexandre Ratchov. While I don't have a umidi device to
test here it looks correct to me and alex has a history of sending
good MIDI diffs.
ok deraadt@
|
|
|
|
|
|
|
|
| |
This should cure some rare stack overflows.
From augustss NetBSD
ok dlg@ pascoe@
|
|
|
|
|
| |
if we are building a new packet, from Alexandre Ratchov.
ok deraadt@
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|