remove unneeded includes

constify struct cfattach

Cleanup <sys/kthread.h> and <sys/proc.h> includes.

Do not include <sys/kthread.h> where it is not needed and stop including
<sys/proc.h> in it.

ok visa@, anton@

all pools have their ipl set via pool_setipl, so fold it into pool_init.

the ioff argument to pool_init() is unused and has been for many
years, so this replaces it with an ipl argument. because the

all pools have their ipl set via pool_setipl, so fold it into pool_init.

the ioff argument to pool_init() is unused and has been for many
years, so this replaces it with an ipl argument. because the ipl
will be set on init we no longer need pool_setipl.

most of these changes have been done with coccinelle using the spatch
below. cocci sucks at formatting code though, so i fixed that by hand.

the manpage and subr_pool.c bits i did myself.

ok tedu@ jmatthew@

@ipl@
expression pp;
expression ipl;
expression s, a, o, f, m, p;
@@
-pool_init(pp, s, a, o, f, m, p);
-pool_setipl(pp, ipl);
+pool_init(pp, s, a, ipl, f, m, p);

show more ...

pool_setipl for cardbus and drm pools.

ok kettenis@

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@

remove the second void * argument on tasks.

when workqs were introduced, we provided a second argument so you
could pass a thing and some context to work on it in. there were
very few things that to

remove the second void * argument on tasks.

when workqs were introduced, we provided a second argument so you
could pass a thing and some context to work on it in. there were
very few things that took advantage of the second argument, so when
i introduced pools i suggested removing it. since tasks were meant
to replace workqs, it was requested that we keep the second argument
to make porting from workqs to tasks easier.

now that workqs are gone, i had a look at the use of the second
argument again and found only one good use of it (vdsp(4) on sparc64
if you're interested) and a tiny handful of questionable uses. the
vast majority of tasks only used a single argument. i have since
modified all tasks that used two args to only use one, so now we
can remove the second argument.

so this is a mechanical change. all tasks only passed NULL as their
second argument, so we can just remove it.

ok krw@

show more ...

if you really must look inside the pool to decide if you've called
pool_init already, the pr_size field is the least worst field to peek at.

Replace workqs with tasks.

ok dlg@

Enforce ca_activate tree-walks over the entire heirarchy for all events,
cleaning up some shutdown-hook related code on the way.
(A few drivers related to sparc are still skipped at kettenis' request

Enforce ca_activate tree-walks over the entire heirarchy for all events,
cleaning up some shutdown-hook related code on the way.
(A few drivers related to sparc are still skipped at kettenis' request)
ok kettenis mlarkin, tested by many others too

show more ...

Make CardBus ral(4) suspend and resume properly on machines that keep the
CardBus bridge powered up during suspend. Step zero on the long road
towards fully suspending CardBus on all machines.

ok d

Make CardBus ral(4) suspend and resume properly on machines that keep the
CardBus bridge powered up during suspend. Step zero on the long road
towards fully suspending CardBus on all machines.

ok deraadt@, damien@

show more ...

As per NetBSD move HAYAKAWA Koichi's licenses to two clause.

Get size of the struct cardslot_event, not the function cardslot_event.

Found by pcc.

OK deraadt.

use workq instead of crafting our own kthread, since the tasks being done
are fairly small. tested by a few, and kettenis says he will test it sooner
if it is commited...

Don't panic if someone inserts a CardBus slot for which we disabled CardBus
support; print a properly formatted message instead.

ok jsing@

Don't crash if we only try to attach pcmcia(4).

ok jsing@, fgsch@

Un-staticize if option DDB.

knf.

typo, enevt -> event

- The card_attach_now flag hasn't been necessary or useful for some time.
Garbage collect the code and corresponding documentation.
- some minor grammar/spelling fixes.

From NetBSD, the documentatio

- The card_attach_now flag hasn't been necessary or useful for some time.
Garbage collect the code and corresponding documentation.
- some minor grammar/spelling fixes.

From NetBSD, the documentation change changed slightly
after a suggestion from jmc@

ok deraadt@

show more ...

Remove useless ``elm'' argument from the SIMPLEQ_REMOVE_HEAD macro.
This matches our SLIST behaviour and NetBSD's SIMPLEQ as well.

ok millert krw deraadt

typos from Tom Cosgrove;

First round of __P removal in sys

Initial check-in for support of 32-bit CardBus PC Cards; from NetBSD. On many
machines, this code needs the new PCIBIOS* options enabled in the kernel config
file to work, but your mileage may vary.

Initial check-in for support of 32-bit CardBus PC Cards; from NetBSD. On many
machines, this code needs the new PCIBIOS* options enabled in the kernel config
file to work, but your mileage may vary. Included is a working 3c575 driver for
3Com 10/100 CardBus PC Card NICs (tested only with the 'C' revision). The 3c575
is the pccard version of the PCI EtherLink XL cards, and thus the xl driver has
been split into /sys/dev/ic.

show more ...