summaryrefslogtreecommitdiffstats
path: root/sys/dev/pci/if_nge.c (follow)
Commit message (Collapse)AuthorAgeFilesLines
* Rename the macro MCLGETI to MCLGETL and removes the dead parameter ifp.jan2020-12-121-2/+2
| | | | | | OK dlg@, bluhm@ No Opinion mpi@ Not against it claudio@
* Change users of IFQ_SET_MAXLEN() and IFQ_IS_EMPTY() to use the "new" API.patrick2020-07-101-6/+6
| | | | ok dlg@ tobhe@
* Argument order fix for MCLGETI.kevlo2019-09-251-2/+2
| | | | ok claudio@
* move counting if_opackets next to counting if_obytes in if_enqueue.dlg2017-01-221-2/+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-2/+1
|
* replace IFF_OACTIVE manipulation with mpsafe operations.dlg2015-11-251-6/+7
| | | | | | | | | | | | | | | | | | | | | | | | 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
|
* 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@
* Do not include <net/if_vlan_var.h> when it's not necessary.mpi2015-11-141-6/+1
| | | | | | Because of the VLAN hacks in mpw(4) this file still contains the definition of "struct ifvlan" which depends on <sys/refcnt.h> which in turns pull <sys/atomic.h>...
* arp_ifinit() is no longer needed.mpi2015-10-251-11/+2
|
* More ifmedia64 fallout; that should be the last.miod2015-09-121-2/+3
|
* Increment if_ipackets in if_input().mpi2015-06-241-3/+1
| | | | | | | Note that pseudo-drivers not using if_input() are not affected by this conversion. ok mikeb@, kettenis@, claudio@, dlg@
* Now that if_input() set the receiving interface pointer on mbufs for usmpi2015-04-131-3/+2
| | | | | | | | | | | | there's no need to do it in m_devget(9). Stop passing an ``ifp'' will help for upcoming interface pointer -> index conversion. While here remove unused ``ifp'' argument from m_clget(9) and kill two birds^W layer violations in one commit. ok henning@
* Convert to if_input().mpi2015-04-081-11/+5
| | | | ok dlg@
* unifdef INETtedu2014-12-221-5/+1
|
* replace the custom jumbo allocator with MCLGETI.dlg2014-08-201-183/+3
| | | | putting this in the tree to make it easier for people to test.
* Fewer <netinet/in_systm.h>mpi2014-07-221-3/+1
|
* Instead of comparing the lower and higher addresses of all the multicastmpi2013-11-261-6/+4
| | | | | | | | | | | entries to decide if the IFF_ALLMULTI flag should be set, check if there is at least one real range between them. This should not change the behavior of any driver but if you encounter any problem, feel free to revert the offending chunk and ping me about it. ok naddy@, dlg@
* - ansify some function definitionsbrad2013-11-181-99/+37
| | | | | | | | | - remove return at end of void functions - remove extraneous braces no functional change. ok chris@
* Use %z* for size_tsf2013-10-011-3/+3
| | | | while there, fix a few %d into %u
* get rid of the copy argument in m_devget that let you provide andlg2013-08-211-2/+2
| | | | | | | | | alternative to bcopy since noone uses it. while there use memcpy instead of bcopy because we know the memory cannot overlap. ok henning@ matthew@ mikeb@ deraadt@
* 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@
* Remove setting an initial assumed baudrate upon driver attach which is notbrad2012-11-291-2/+1
| | | | | | necessarily correct, there might not even be a link when attaching. ok mikeb@ reyk@
* Convert a number of old private copies of code which predatesderaadt2012-10-181-31/+2
| | | | | | | | pci_set_powerstate() to using it instead. Many of these chunks of code had bugs in them, especially missing delay() calls. Some of them were doing things our PCI subsystem is now responsible for handling. If you have any of the affected devices, please keep an eye out for regressions. ok kettenis
* kill a few more casts that aren't helpful. ok krw miodtedu2011-06-221-6/+6
|
* Stop doing shutdown hooks in network drivers where possible. We alreadyderaadt2010-09-201-16/+1
| | | | | | | take all interfaces down, via their xxstop routines. Claudio and I have verified that none of the shutdown hooks do much extra beyond what xxstop was already doing; it is largely a pile of junk. ok claudio, some early comments by sthen; also read by matthew, jsg
* BUS_DMA_ZERO instead of alloc, map, bzero.oga2010-05-191-3/+3
| | | | ok krw@
* - consistify cfdriver for the ethernet drivers (0 -> NULL)jasper2009-08-131-2/+2
| | | | ok dlg@
* Eliminate the redundant bits of code for MTU and multicast handlingbrad2008-11-281-19/+10
| | | | | | | | | | | | | | 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@
* Introduce bpf_mtap_ether(), which for the benefit of bpf listenersnaddy2008-11-091-3/+3
| | | | | | | | | creates the VLAN encapsulation from the tag stored in the mbuf header. Idea from FreeBSD, input from claudio@ and canacar@. Switch all hardware VLAN enabled drivers to the new function. ok claudio@
* #if NVLAN > 0 in one more spot.brad2008-10-281-2/+4
|
* Implement a workaround for stupid hw when using VLAN stripping. Framesbrad2008-10-281-4/+18
| | | | | | | that are 64 bytes with a VLAN header appended like ARP frames or ICMP echos are flagged as runts when the tag is stripped. Issue mentioned by yongari@FreeBSD, info gleaned from the Linux driver.
* Don't need NVLAN > 0 checks around this code.brad2008-10-281-5/+1
|
* Re-add support TX VLAN tag insertion and RX VLAN tag stripping.brad2008-10-281-3/+20
|
* Switch the existing TX VLAN hardware support over to having thenaddy2008-10-161-10/+3
| | | | | | tag in the header. Convert TX tagging in the drivers. Help and ok brad@
* Second pass of simple timeout_add -> timeout_add_sec conversionsblambert2008-10-151-3/+3
| | | | | | | This should take care of the simpler ones (i.e., timeout values of integer multiples of hz). ok krw@, art@
* First step towards cleaning up the Ethernet driver ioctl handling.brad2008-10-021-9/+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@
* Simplify the combination use of pci_mapreg_type()/pci_mapreg_map() asbrad2008-05-231-9/+3
| | | | | | suggested by dlg@ awhile ago. ok dlg@
* typos; ok jmc@martynas2007-11-261-2/+2
| | | | | sys/netinet/in_pcb.c and sys/net/bridgestp.c ok henning@ sys/dev/pci/bktr/* ok jakemsr@
* replace a few more instances of hand rolled code with thebrad2006-10-251-4/+3
| | | | LIST_FOREACH macro.
* - remove ETHER_MAX_LEN_JUMBO and ETHERMTU_JUMBO.brad2006-05-281-3/+3
| | | | | | - use if_hardmtu for MTU ioctl handlers. ok reyk@
* unknown ioctl is ENOTTY not EINVALjason2006-05-281-2/+2
|
* remove IFCAP_JUMBO_MTU interface capabilities flag and set if_hardmtu in a fewbrad2006-05-271-2/+3
| | | | | | more drivers. ok reyk@
* set if_jumbo_mtu and the IFCAP_JUMBO_MTU capabilities flag wherebrad2006-05-201-2/+2
| | | | | | appropriate. ok reyk@
* allow bpf(4) to ignore packets based on their direction (inbound ordjm2006-03-251-3/+3
| | | | | outbound), using a new BIOCSDIRFILT ioctl; guidance, feedback and ok canacar@
* remove the colon between "address" and the MAC address.brad2006-03-041-2/+2
|
* Do not bother enabling HW TX checksum offload since the code tobrad2006-02-161-12/+9
| | | | support this feature was removed awhile ago.
* assume vtophys(vaddr_t) just what all the other archs expect; no functional changemickey2005-11-231-9/+10
|
* - splimp -> splnetbrad2005-11-041-50/+46
| | | | | | | | - remove spl from attach - fix dmesg printing - de-allocate resources on failure to attach - remove redundant checks just before pci_mapreg_map() - use pci_mapreg_type()
* fix bus_dmamem_alloc() failure case.brad2005-10-091-3/+2
|