summaryrefslogtreecommitdiffstats
path: root/sys/dev/usb/if_udav.c (follow)
Commit message (Collapse)AuthorAgeFilesLines
* Nuke all occurrences of usbd_abort_pipe() if it gets called rightmglocker2020-07-311-4/+1
| | | | | | | | | | 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@
* Change users of IFQ_SET_MAXLEN() and IFQ_IS_EMPTY() to use the "new" API.patrick2020-07-101-4/+4
| | | | ok dlg@ tobhe@
* use the UE_GET_XFERTYPE macro where applicablejasper2020-02-221-4/+4
| | | | ok mpi@
* Make some USB ioctls return ENXIO, instead of EIO, if usbd_is_dying().stsp2018-10-021-2/+2
| | | | | | | Brings us one step closer towards making this condition's error code consistent across all USB drivers. Patch by Moritz Buhl ok mpi@ bluhm@
* Use LIST_FOREACH construct instead of homebrew loop.kevlo2018-07-031-3/+2
| | | | ok bluhm@, mpi@
* move counting if_opackets next to counting if_obytes in if_enqueue.dlg2017-01-221-3/+1
| | | | | | | this means packets are consistently counted in one place, unlike the many and various ways that drivers thought they should do it. ok mpi@ deraadt@
* G/C IFQ_SET_READY().mpi2016-04-131-3/+1
|
* replace IFF_OACTIVE manipulation with mpsafe operations.dlg2015-11-251-7/+8
| | | | | | | | | | | | | | | | | | | | | | | | there are two things shared between the network stack and drivers in the send path: the send queue and the IFF_OACTIVE flag. the send queue is now protected by a mutex. this diff makes the oactive functionality mpsafe too. IFF_OACTIVE is part of if_flags. there are two problems with that. firstly, if_flags is a short and we dont have any MI atomic operations to manipulate a short. secondly, while we could make the IFF_OACTIVE operates mpsafe, all changes to other flags would have to be made safe at the same time, otherwise a read-modify-write cycle on their updates could clobber the oactive change. instead, this moves the oactive mark into struct ifqueue and provides an API for changing it. there's ifq_set_oactive, ifq_clr_oactive, and ifq_is_oactive. these are modelled on ifsq_set_oactive, ifsq_clr_oactive, and ifsq_is_oactive in dragonflybsd. this diff includes changes to all the drivers manipulating IFF_OACTIVE to now use the ifsq_{set,clr_is}_oactive API too. ok kettenis@ mpi@ jmatthew@ deraadt@
* You only need <net/if_dl.h> if you're using LLADDR() or a sockaddr_dl.mpi2015-11-241-2/+1
|
* No need to include <net/if_arp.h>mpi2015-11-241-2/+1
| | | | | | | | | This header is only needed because <netinet/if_ether.h> declares a structure that needs it. But it turns out that <net/if.h> already includes it as workaround. A proper solution would be to stop declarting "struct ether_arp" there. But no driver should need this header.
* shuffle struct ifqueue so in flight mbufs are protected by a mutex.dlg2015-11-201-3/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | the code is refactored so the IFQ macros call newly implemented ifq functions. the ifq code is split so each discipline (priq and hfsc in our case) is an opaque set of operations that the common ifq code can call. the common code does the locking, accounting (ifq_len manipulation), and freeing of the mbuf if the disciplines enqueue function rejects it. theyre kind of like bufqs in the block layer with their fifo and nscan disciplines. the new api also supports atomic switching of disciplines at runtime. the hfsc setup in pf_ioctl.c has been tweaked to build a complete hfsc_if structure which it attaches to the send queue in a single operation, rather than attaching to the interface up front and building up a list of queues. the send queue is now mutexed, which raises the expectation that packets can be enqueued or purged on one cpu while another cpu is dequeueing them in a driver for transmission. a lot of drivers use IFQ_POLL to peek at an mbuf and attempt to fit it on the ring before committing to it with a later IFQ_DEQUEUE operation. if the mbuf gets freed in between the POLL and DEQUEUE operations, fireworks will ensue. to avoid this, the ifq api introduces ifq_deq_begin, ifq_deq_rollback, and ifq_deq_commit. ifq_deq_begin allows a driver to take the ifq mutex and get a reference to the mbuf they wish to try and tx. if there's space, they can ifq_deq_commit it to remove the mbuf and release the mutex. if there's no space, ifq_deq_rollback simply releases the mutex. this api was developed to make updating the drivers using IFQ_POLL easy, instead of having to do significant semantic changes to avoid POLL that we cannot test on all the hardware. the common code has been tested pretty hard, and all the driver modifications are straightforward except for de(4). if that breaks it can be dealt with later. ok mpi@ jmatthew@
* arp_ifinit() is no longer needed.mpi2015-10-251-5/+1
|
* Increment if_ipackets in if_input().mpi2015-06-241-2/+1
| | | | | | | Note that pseudo-drivers not using if_input() are not affected by this conversion. ok mikeb@, kettenis@, claudio@, dlg@
* Only match devices with a valid configuration.uaa2015-06-191-18/+4
| | | | ok by mpi@
* Convert to if_input(), tested by jsg@, thanks!mpi2015-03-231-16/+6
|
* Remove some includes include-what-you-use claims don'tjsg2015-03-141-2/+1
| | | | | | | have any direct symbols used. Tested for indirect use by compiling amd64/i386/sparc64 kernels. ok tedu@ deraadt@
* unifdef INETtedu2014-12-221-3/+1
|
* No need for <netinet/in_systm.h> here.mpi2014-07-131-5/+1
|
* Kill a bunch of DVACT_DEACTIVATE handlers whose only purpose were tompi2014-07-121-31/+9
| | | | | | | | | | | | | | set the 'dying' flag of a device. Such handlers are useless now that usbd_detach() already set this flag. Even if the purpose of this flag is questionnable on OpenBSD because DVACT_DEACTIVATE acts as a pre-detach event, this is just a first small step towards the simplification of the autoconf(9) device states. This cleaning is now possible thanks to the work of pirofti@ to convert all the USB drivers to properly use usbd_is_dying(). Discussed many times with deraadt@
* udav(4): add CoreChip RD9700 supportsasano2014-01-221-12/+45
| | | | | | RD9700 has no MII-PHY, no serial EEPROM. ok by deraadt@, brad@
* Clean-up usbd_abort_pipe() usage.pirofti2013-11-151-13/+4
| | | | | | | This function never fails. So change it's return type to void and adjust the copy-pasted callers that were checking the return. "If it compiles, ok" mpi@
* Most network drivers include netinet/in_var.h, but apparently theybluhm2013-08-071-2/+1
| | | | | don't have to. Just remove these include lines. Compiled on amd64 i386 sparc64; OK henning@ mikeb@
* Get rid of various 'typedef struct' definitions and use plain structuremglocker2013-04-151-7/+7
| | | | | | | | | 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@
* proc.h is way too much header for usb to handle.tedu2013-03-281-2/+1
|
* Rewrite the receive filter handling code and cleanup the ioctl bits.brad2013-01-291-66/+36
| | | | | | | Fixes a bug when going in and out of promiscuous mode results in the driver not disabling all multicast mode. Tested by kili@ jasper@
* Remove excessive sys/cdefs.h inclusionderaadt2012-12-051-3/+1
| | | | ok guenther millert kettenis
* another udav device from Loganaden Velvindronjsg2011-09-041-2/+3
|
* Remove config_activate() and DVACT_ACTIVATE. PCMCIA's the only thingmatthew2011-07-031-4/+1
| | | | | | | | 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
* a bit more ansi; ok teduderaadt2011-06-231-7/+1
|
* Don't process packets where the USB xfer length is shorter than a packetmk2011-03-311-3/+10
| | | | | | | | | | | | | | | | header, and don't process packets where the packet header length field says the packet is longer than the size of the USB xfer. While there, fix a debug print that was missing the device name from the parameter list, causing it to crash when enabled. Also, use correct MTU from datasheet. From Loganaden Velvindrom with input from me. Tested by kili. ok jasper sthen (with a whitespace nit)
* Update URI to data sheet. New location found on www.davicom.com.tw butmk2011-03-201-2/+2
| | | | | | | | points to some other domain for some reason. Noticed while looking into a udav(4) issue with Loganaden Velvindron. OK jasper
* revert previous as it breaks nfs client as reported by kili@jasper2011-03-161-3/+2
| | | | req'd by jsg@
* If we get a packet larger than if_hardmtu skip it.jsg2011-03-161-2/+3
| | | | | Tweaked version of a diff from Loganaden Velvindron who sees a large garbage frame on his device.
* garbage collect "usb events". without /dev/usb there is no way to accessjakemsr2011-01-251-6/+1
| | | | | | them from userland, and nothing in the kernel uses them. ok krw@, miod@
* * replace per-driver dying and/or other state variables with use ofjakemsr2010-12-061-38/+25
| | | | | | | usbd_deactivete() and usbd_is_dying() * use usbd_deactivate() in activate()/DEACTIVATE * convert a few more direct checks of the associated bus' dying flag with usbd_is_dying()
* don't free network related resources if they were not allocatedjakemsr2010-10-271-3/+5
|
* check that a timeout(9) has been initialized before deleting itjakemsr2010-10-231-2/+3
| | | | ok phessler
* create another kthread to run xfer abort tasks. xfer abort tasks cannotjakemsr2010-10-231-3/+5
| | | | | | | | | | | be run from the generic task kthread, because xfers that need to be aborted block newly queued tasks from running (i.e. the xfer to be aborted blocks the abort of that task). as there are now three types of usb tasks, add an argument to usb_init_task() and another member to struct usb_task to specify the task type. fixes boot hangs that are showing up because we now use usb tasks to attach/detach usb devices.
* Delete comments about these drivers needing powerhook support; they don't.deraadt2010-08-271-2/+1
|
* add a bunch of devices found in linux driversjsg2009-11-221-2/+3
|
* Get rid of devact enum, substitute it with an int and coresponding defines.pirofti2009-10-131-3/+3
| | | | | | This is needed for the addition of further suspend/resume actions. Okay deraadt@, marco@.
* Eliminate the redundant bits of code for MTU and multicast handlingbrad2008-11-281-18/+8
| | | | | | | | | | | | | | from the individual drivers now that ether_ioctl() handles this. Shrinks the i386 kernels by.. RAMDISK - 2176 bytes RAMDISKB - 1504 bytes RAMDISKC - 736 bytes Tested by naddy@/okan@/sthen@/brad@/todd@/jmc@ and lots of users. Build tested on almost all archs by todd@/brad@ ok naddy@
* First step towards cleaning up the Ethernet driver ioctl handling.brad2008-10-021-4/+2
| | | | | | | | | | | | | | | | | | | | Move calling ether_ioctl() from the top of the ioctl function, which at the moment does absolutely nothing, to the default switch case. Thus allowing drivers to define their own ioctl handlers and then falling back on ether_ioctl(). The only functional change this results in at the moment is having all Ethernet drivers returning the proper errno of ENOTTY instead of EINVAL/ENXIO when encountering unknown ioctl's. Shrinks the i386 kernels by.. RAMDISK - 1024 bytes RAMDISKB - 1120 bytes RAMDISKC - 832 bytes Tested by martin@/jsing@/todd@/brad@ Build tested on almost all archs by todd@/brad@ ok jsing@
* Convert timeout_add() calls using multiples of hz to timeout_add_sec()blambert2008-09-101-3/+3
| | | | | | | Really just the low-hanging fruit of (hopefully) forthcoming timeout conversions. ok art@, krw@
* Sanitize the use of timeouts:mbalmer2007-11-231-6/+2
| | | | | | | | | | | | Instead of calling timeout_set(..., NULL, NULL) in attach routines and later timeout_del(...) timeout_set(..., func, arg) timeout_add(..., time) set the function and argument in the initial timeout_set() call and only use timeout_add(..., time) later. ok dlg, fgsch, krw, winiger
* even better attach printing; Nicholas Marriottderaadt2007-11-011-3/+3
|
* fix attach time messages; tested by Nicholas Marriottderaadt2007-11-011-6/+8
|
* Admtek/Shantou ADM8515; nicm__@ntlworld.comderaadt2007-11-011-2/+3
|
* Add ShanTou ZT6688 usb 10/000 nic - A udav sold in a generic package.beck2007-10-201-2/+3
| | | | ok jsg@, krw@, deraadt@
* treat usb vendor/product names as a locator, and have usbd_print handle it,deraadt2007-10-111-6/+1
| | | | | | 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