Do not trust ipfw: check m_len always, not only after m_dup.

Submitted by: ru

- do m_pullup() after m_dup()
- clean style in previous commit

Suggested by: ru

Make sure the last command byte makes it onto the wire.

pullup to sizeof struct ip before sending to ip_output.

Suggested by: ru

Don't read more than the given 'len' bytes from the 'big' string.

PR: misc/77369
Submitted by: Ed Maste <emaste@phaedrus.sandvine.ca>
MFC after: 1 week

Further embellish rev. 1.180: protect against -fno-strict-aliasing
being already in CFLAGS (as is the case with "make buildkernel").

NB: rev. 1.180 is only needed due to a broken setting of CFLAGS in
tinderbox.

Packets from ipfw come with IP header in host byte order. Netgraph works
with net byte order. Change byte order to net in ng_ipfw_input(), change
byte order to host before ip_output(), do not change before ip_input().

In collaboration with: ru

Stylistic fixes: push variable into a local context (this part is going
to be split out into a function soon). Also there is no need to write
back the colon that we have NUL-ed - the string is going to be freed
anyway.

Submitted by: Max Okumoto <okumoto@ucsd.edu>

sysctl node vm.stats can not be static (for ia64 reasons).

Add a bit more caveat text about MUTEX_PROFILING -- try to avoid using it
with other profiling and debugging options, such as INVARIANTS, WITNESS,
kernel profiling, etc.  They all interfere with each other nastily and
will generate fairly useless results.

Add SIGTHR (32) into list of signals permitted to be delivered to the
suid application. The problem is that Linux applications using old Linux
threads (pre-NPTL) use signal 32 (linux SIGRTMIN) for communication between
thread-processes. If such an linux application is installed suid or sgid
and security.bsd.conservative_signals=1 (default), then permission will be
denied to send such a signal and the application will freeze.

I believe the same will be true for native applications that use libthr,
since libthr uses SIGTHR for implementing conditional variables.

PR: 72922
Submitted by: Andriy Gapon <avg@icyb.net.ua>
MFC after: 2 weeks

Semctl with IPC_STAT command should return zero in case of success.

PR: 73778
Submitted by: Andriy Gapon <avg@icyb.net.ua>
MFC after: 2 weeks

Convert several instances of negative logic to positive logic.

Submitted by: Max Okumoto <okumoto@ucsd.edu>

Make non-SOFTUPDATES kernels compile again.

Integrate the stubfile into the main file now that license issues have been
long resolved.

Diff reduction with p4.

Expand BSS.

Improve EOS handling.

Eliminate the ED_VENDOR_{PCCARD,LINKSYS} types.  These cards really
are NOVELL NE2000 with just a tiny quirk that's non vendor specific.
Instead, use the chip_type of DL100XX instead.  This is more inline
with how the AX88190 support was added, and seems a little cleaner.

Skip PNP probes.  If there are isapnp or pnpbios instances of this driver,
then we can support them later.  This keeps the pbio probe from claiming
lots of otherwise unused pnpbios devices on my laptop.

Fix build error (s/wd80x3.c/if_ed_wd80x3.c/).

Merged from sys/dev/sio/sio.c revision 1.458.

Merge in patch to support AP scanning via ifconfig and the new
net80211 API.

Submitted by: Stephane E. Potvin sepotvin at videotron dot ca

Adjust TX threshold on every TX underrun.  Some supported cards (8139-based)
can retransmit on TX underrun and set TOK in addition to TUND.  Also add a
check to prevent overflow of the addressable threshold.

This fixes some reports of rl(4) slowness, believed to be related to ALTQ
before.

PR: kern/61448
Submitted by: Tim Draegen-Gilman <timNOeudaemonSPAMnet> (with changes)
MFC after: 1 week

When processing a timeout() callout and returning it to the free
list, set `curr_callout' to NULL. This ensures that we won't attempt
to cancel the current callout if the original callout structure
gets recycled while we wait to acquire Giant.

This is reported to fix an intermittent syscons problem that was
introduced by revision 1.96.

Move enable/disable 16bit access routines into if_ed.  Slightly
pessmize the error recover path through edintr by calling these
functions, rather than expanding it inline.  This error path already
does a lot in it, so an extra function call will be lost in the noise.
It also happens rarely.

use
while (complicated-expr)
continune;
in preference to
while (complicated-expr);

since the code generated is identical, and the former is easier to read,
especially for complicated-expr that reach to the end of the line...

Remove lots of tab/space errors introduced by massive cut-n-paste action.

Noticed from: diffs of the last commit.
MFC after: 1 week

Implement support for buffers larger than PAGE_SIZE in MemGuard.  Adds
a little bit of complexity but performance requirements lacking (this is
a debugging allocator after all), it's really not too bad (still
only 317 lines).

Also add an additional check to help catch really weird 3-threads-involved
races: make memguard_free() write to the first page handed back, always,
before it does anything else.

Note that there is still a problem in VM+PMAP (specifically with
vm_map_protect) w.r.t. MemGuard uses it, but this will be fixed shortly
and this change stands on its own.

Take into account that Pinnacle screwed up their PCI ID in the beginning..
Older cards have it reversed.
Also, use some already defined values instead of magic numbers.

PR:  73324
Submitted by: arne_woerner@yahoo.com
MFC after: 1 week

Optimize the way reference counting is performed with Mbufs.  We
do not need to perform an extra memory fetch in the Packet (Mbuf+Cluster)
constructor to initialize the reference counter anymore.  The reference
counts are located in a separate memory region (in the slab header,
because this zone is UMA_ZONE_REFCNT), so the memory fetch resulted very
often in a cache miss.  Additionally, and perhaps more significantly,
optimize the free mbuf+cluster (packet) case, which is very common, to
no longer require an atomic operation on free (to verify the reference
counter) if the reference on the cluster has never been increased (also
very common).  Reduces an atomic on mbuf free on average.

Original patch submitted by: Gerrit Nagelhout <gnagelhout@sandvine.com>

bump the Makefile config versions now that config is 600001...

Forgotten by: des

Add strspn() to libkern.

Ok'ed by: rwatson

Declare "cnt" (a number of bytes to read or write) as an "ssize_t", not
as a "long" in dofileread() and dofilewrite().

Discussed with: jhb

read(), pread(), write(), and pwrite() return EINVAL if they are asked
for more than INT_MAX bytes.

1. Revert back to the way the older driver handled XPT_PATH_INQ.  This seems to
   fix the problem with device discovery seen by some people.
2. Change to make 3ware CLI/3DM work on amd64.
3. Fix a potential problem that could cause the driver to do strlen(NULL) when
   using older firmware.

Reviewed by:scottl

Buffers are already NUL-terminated so there is no need to explicitely add
a NULL.

Submitted by: Max Okumoto <okumoto@ucsd.edu>

rev 1.19 fixed wpa supplicant but broke wpa authenticator; when operating
as an authenticator need to always check for the unicast key in the node
(as was the case before)

Submitted by: Divy Le Ray

Mostly stylistic nits in preparation for splitting up Var_Parse:
use a more consistent style with regard to *str and str[0];
simplify code by introducing a temporary variable;
shift a break around and add braces where appropriate.

Submitted by: Max Okumoto <okumoto@ucsd.edu>

propagate state kept in the bss node when re-creating the node
on state transitions; this is a stopgap measure, need to rethink
how we do management of this state

Identified by: Divy Le Ray

Fixed usage().

Require at least one argument.

Bump the version number for the addition of devices / nodevices.

Let the buffer just use the default size by specifying 0 as the size
instead of specifying the default size explicitely.

Submitted by: Max Okumoto <okumoto@ucsd.edu>

Mark the modification of the input string (which should really be const)
with a comment.

Submitted by: Max Okumoto <okumoto@ucsd.edu>

Style nits:
Move some assignments nearer to where they actually used. Convert a loop
from a for() to a while() to make it clearer and add braces to the long
body of it. Split assignment from variable declaration.

Submitted by: Max Okumoto <okumoto@ucsd.edu>

Sync program's usage() with manpage's SYNOPSIS.

Describe a pre-condition of Var_Parse().

Submitted by: Max Okumoto <okumoto@ucsd.edu>

Remove a comment that's not actual anymore.

Submitted by: Max Okumoto <okumoto@ucsd.edu>

Fix spacing by converting mixes of space and tab to tab. Also add a number
of empty lines in appropriate places.

Submitted by: Max Okumoto <okumoto@ucsd.edu>

Initialise the status variable. W_SETTERMSIG uses its value.

Submitted by: stefanf

Fully document (no)?(device|option)s?.

This page should probably be repocopied to src/usr.sbin/config/.

Lst_Append returns void, so the other case of the ? statement
should also be void.

Submitted by: stefanf

Update a little bit.

Full modernize cleanup:
- De-__P()
- constify where appropriate
- ANSI functions instead of K&R

Pointed out by: stefanf

Make various vnode related functions static

Make some file/filedesc related functions static

Make M_NETGRAPH_ETF static

Make various mountpoint related functions static.

Make a SYSCTL_NODE static

MD5Pad() should never have been exposed.

Make a some SYSCTL_NODEs and some of FFS's VFS_ methods static.

Make three SYSCTL_NODEs static

Make npages static and const.

make cluster_callback()  static

Make a SYSCTL_NODE and a mutex static

Make another bunch of SYSCTL_NODEs static

Make a bunch of SYSCTL_NODEs static.

Make various random things static

make M_NTFSMNT and ntfs_calccfree() static

Make fdesc_root static

Statification

Make smbfs_debuglevel private.

don't call vprint with NULL.

Statize malloc types.
Don't call vprint with NULL.

Statize devfs_ops_f

Make a bunch of malloc types static.

Found by: src/tools/tools/kernxref

"device" and "nodevice" lines can actually specify more than one device
(separated by commas), so add "devices" and "nodevices" as aliases.

MFC after:  2 weeks

back out 1.7 changes, unneeded and possibly wrong.

Note that the ICH6 controller is supported.

Reviewed by: sos

Sync program's usage() with manpage's SYNOPSIS.

Sync program's usage() with manpage's SYNOPSIS.

Turn K&R functions into prototypes.

The variable `ROOTKEY' has internal linkage in keyserv.c, don't declare it as
extern here.

The variable `arglist' has internal linkage in pw.c, don't declare it as extern
here.

Turn K&R functions into prototypes.

Add __printflike() to vn_printf()

Don't pass NULL to vprint()

Let the generated code include <string.h> instead of <memory.h>.

Code cleanup:
- Prefer modern declaration of functions
- WARNS?= 4 -> 6

Extricate probing of the gwether card (Gateway AT) from the middle of
probing the novell ne[12]000 cards.  It should be its own thing, ala
how we do the dl100xx support doing its own thing at the right time.
For the moment, it is just a function, which makes the mainline of the
generic probe easier to follow.

Also, correct a couple of comments that looked wrong.

# there may be a bug in setting up gwether, in that we set
# sc->rec_page_stop based on memsize, rather than sc->mem_size, so if
# these two are different, then the rec_page_stop will be wrong.  I'm
# hesitant to fix it without real hardware to test with.  Since
# gwether isn't in the hardware list of the man page nor in the commit
# messages, it is hard to know for sure.

Remove an ifdef that's been here for a long time.  So long, in fact,
that it references a function parameter that's not been passed in for
more than 4 years.

This has been if 0'd since 1994.

Fix incorrect comment.

Submitted by: James Bowman <jamesb@acelere.net>

 - Add more information to the getnewbuf() recycling KTR.

Sponsored by: Isilon Systems, Inc.

Implement interrupt routing for DEC_KN20AA.

Tested by: wilko
MFC after: 2 weeks

 - In the softupdates case for ffs_truncate() we use vinvalbuf() to
   invalidate pending io and dependencies.  However, vinvalbuf() rightfully
   does not call vnode_pager_setsize() for us.  We must do this here.  This
   could potentially have caused numerous kinds of bugs, but it was
   specifically causing msync() deadlocks because msync() was writing
   flushing pages that should not have been valid.

Sponsored by: Isilon Systems, Inc.
Reported by: kkenn

Fix pc98 compile: merge in changes to use the local APIC timer.  Also, add
missing initialization of i8254_intsrc while I am here.

- Correctly spell MOD_QUIESCE
- use .Dv for it

MFC after: 3 days

Minor nits noted by ru@

Constify.

Sync program's usage() with manpage's SYNOPSIS.