#
0f9e9ec2 |
| 13-May-2024 |
jsg <jsg@openbsd.org> |
remove prototypes with no matching function ok mpi@
|
#
cf96265b |
| 10-Nov-2023 |
bluhm <bluhm@openbsd.org> |
Make ifq and ifiq interface MP safe.
Rename ifq_set_maxlen() to ifq_init_maxlen(). This function neither uses WRITE_ONCE() nor a mutex and is called before the ifq mutex is initialized. The new na
Make ifq and ifiq interface MP safe.
Rename ifq_set_maxlen() to ifq_init_maxlen(). This function neither uses WRITE_ONCE() nor a mutex and is called before the ifq mutex is initialized. The new name expresses that it should be used only during interface attach when there is no concurrency.
Protect ifq_len(), ifq_empty(), ifiq_len(), and ifiq_empty() with READ_ONCE(). They can be used without lock as they only read a single integer.
OK dlg@
show more ...
|
#
7eb8d89d |
| 22-Feb-2022 |
guenther <guenther@openbsd.org> |
Delete unnecessary #includes of <sys/domain.h> and/or <sys/protosw.h>
net/if_pppx.c pointed out by jsg@ ok gnezdo@ deraadt@ jsg@ mpi@ millert@
|
#
0cae21bd |
| 10-Jul-2020 |
patrick <patrick@openbsd.org> |
Change users of IFQ_SET_MAXLEN() and IFQ_IS_EMPTY() to use the "new" API.
ok dlg@ tobhe@
|
#
63bcfa73 |
| 10-Jul-2020 |
patrick <patrick@openbsd.org> |
Change users of IFQ_DEQUEUE(), IFQ_ENQUEUE() and IFQ_LEN() to use the "new" API.
ok dlg@ tobhe@
|
#
88a08f2a |
| 22-Jan-2017 |
dlg <dlg@openbsd.org> |
move counting if_opackets next to counting if_obytes in if_enqueue.
this means packets are consistently counted in one place, unlike the many and various ways that drivers thought they should do it.
move counting if_opackets next to counting if_obytes in if_enqueue.
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@
show more ...
|
#
29e604b7 |
| 04-May-2016 |
kettenis <kettenis@openbsd.org> |
Use BUS_DMA_OVERRUN to cope with the broken DMA engine of the Davicom DM9102 found on some Sun sparc64 machines. This fixes the unrecoverable DMA errors people have been seeing ever since dlg@ made
Use BUS_DMA_OVERRUN to cope with the broken DMA engine of the Davicom DM9102 found on some Sun sparc64 machines. This fixes the unrecoverable DMA errors people have been seeing ever since dlg@ made changes to the pool code that changes the memory layout.
show more ...
|
#
1e1858b6 |
| 13-Apr-2016 |
mpi <mpi@openbsd.org> |
G/C IFQ_SET_READY().
|
#
da67112d |
| 28-Nov-2015 |
dlg <dlg@openbsd.org> |
rework dc_start and dc_encap to take advantage of m_defrag.
if the chip needs coalesced packages in tx, set the tx dmamaps up to only use a single dma descriptor. use m_defrag when bus_dmamap_load_m
rework dc_start and dc_encap to take advantage of m_defrag.
if the chip needs coalesced packages in tx, set the tx dmamaps up to only use a single dma descriptor. use m_defrag when bus_dmamap_load_mbuf returns EFBIG rather than copying the packet to a separate mbuf with hand rolled code in dc_coal. that in turn makes the ifq_deq_begin, ifq_deq_commit, and ifq_deq_rollback handling more straightforward.
tested by me on a hppa a180c with a "DEC 21142/3", and fred on bugs@ with a sparc64 netra x1 "Davicom DM9102".
show more ...
|
#
de6cd8fb |
| 25-Nov-2015 |
dlg <dlg@openbsd.org> |
replace IFF_OACTIVE manipulation with mpsafe operations.
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 i
replace IFF_OACTIVE manipulation with mpsafe operations.
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@
show more ...
|
#
f9ad5574 |
| 24-Nov-2015 |
mpi <mpi@openbsd.org> |
The only network driver needing <net/if_types.h> is upl(4) for IFT_OTHER.
|
#
b5d83b91 |
| 20-Nov-2015 |
dlg <dlg@openbsd.org> |
shuffle struct ifqueue so in flight mbufs are protected by a mutex.
the code is refactored so the IFQ macros call newly implemented ifq functions. the ifq code is split so each discipline (priq and
shuffle struct ifqueue so in flight mbufs are protected by a mutex.
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@
show more ...
|
#
d2d9c74c |
| 25-Oct-2015 |
mpi <mpi@openbsd.org> |
arp_ifinit() is no longer needed.
|
#
5b21b8fa |
| 12-Sep-2015 |
miod <miod@openbsd.org> |
ifmedia64 fixes.
|
#
f2a0e423 |
| 11-Sep-2015 |
stsp <stsp@openbsd.org> |
Make room for media types of the future. Extend the ifmedia word to 64 bits. This changes numbers of the SIOCSIFMEDIA and SIOCGIFMEDIA ioctls and grows struct ifmediareq.
Old ifconfig and dhclient b
Make room for media types of the future. Extend the ifmedia word to 64 bits. This changes numbers of the SIOCSIFMEDIA and SIOCGIFMEDIA ioctls and grows struct ifmediareq.
Old ifconfig and dhclient binaries can still assign addresses, however the 'media' subcommand stops working. Recompiling ifconfig and dhclient with new headers before a reboot should not be necessary unless in very special circumstances where non-default media settings must be used to get link and console access is not available.
There may be some MD fallout but that will be cleared up later.
ok deraadt miod with help and suggestions from several sharks attending l2k15
show more ...
|
#
d0c70bf7 |
| 30-Aug-2015 |
deraadt <deraadt@openbsd.org> |
Track rom size, for free()
|
#
db4dc9aa |
| 24-Jun-2015 |
mpi <mpi@openbsd.org> |
Increment if_ipackets in if_input().
Note that pseudo-drivers not using if_input() are not affected by this conversion.
ok mikeb@, kettenis@, claudio@, dlg@
|
#
c35f95b8 |
| 13-Apr-2015 |
mpi <mpi@openbsd.org> |
Now that if_input() set the receiving interface pointer on mbufs for us there's no need to do it in m_devget(9).
Stop passing an ``ifp'' will help for upcoming interface pointer -> index conversion.
Now that if_input() set the receiving interface pointer on mbufs for us 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@
show more ...
|
#
21dab745 |
| 14-Mar-2015 |
jsg <jsg@openbsd.org> |
Remove some includes include-what-you-use claims don't have any direct symbols used. Tested for indirect use by compiling amd64/i386/sparc64 kernels.
ok tedu@ deraadt@
|
#
3efb6d46 |
| 13-Mar-2015 |
jasper <jasper@openbsd.org> |
convert to if_input()
tested by landry@ ok mpi@
|
#
ae81a69a |
| 23-Jan-2015 |
dlg <dlg@openbsd.org> |
break after return is useless.
|
#
f79ee556 |
| 22-Dec-2014 |
tedu <tedu@openbsd.org> |
unifdef INET
|
#
9645d4ae |
| 18-Nov-2014 |
brad <brad@openbsd.org> |
dc_init() calls dc_stop() and dc_reset() so remove some redundant calls to those functions before dc_init() within dc_watchdog() and dc_intr().
ok deraadt@
|
#
9e6eaca1 |
| 22-Jul-2014 |
mpi <mpi@openbsd.org> |
Fewer <netinet/in_systm.h>
|
#
aa3cabd0 |
| 12-Jul-2014 |
tedu <tedu@openbsd.org> |
add a size argument to free. will be used soon, but for now default to 0. after discussions with beck deraadt kettenis.
|