Make atm WARNS=6 clean. The changes are mostly:

 - remove some instances of __P()
 - use real prototypes and un-K&R function headers
 - constify where necessary (mostly strings and structures containing
   strings)
 - make functions and variables static that need not to be global
 - tag unused function parameters as __unused

Testing: a fresh universe

Add a PATH_INQ flag, PIM_NO_6_BYTE, which indicates the SIM never wishes to
receive 6 byte commands.  Add a check for this flag to da(4) and cd(4) so
that they honor it.  This is a quick workaround for many devices (especially
USB) that require da(4) quirks to operate.  The more complete approach is
to finish the new transport code which will be aware of the SCSI version a
transport implements.

MFC after: 1 day

None of the "alloc" functions used by UMA assume that Giant is held any
longer.  (If they still need it, e.g., contigmalloc(), they acquire it
themselves.)  Therefore, we need not acquire Giant in slab_zalloc().

auth.conf seems to be being gradually subsumed by user classes
and PAM configuration.  Remove the line concerning "auth_list"
from the template, since it's referenced only in the tinyware
password command, and only #ifdef KERBEROS, which isn't defined
in tinyware.  Add a comment about auth.conf being on the way
out the door.  The one remaining consumer of auth.conf is
crypt(3).

The LP_ETH_10_100_CF entry needs to be tagged as a DL100019.

Submitted by: Scott Renfro

Reset the per-CPU unique value at boot and clear it in the PCB of the
child when forking. This provides a consistent initial state.
Note that cpu_set_upcall() does not clear the per-CPU unique value as
it is followed by a call to set_mcontext(), which sets it accordingly.

Make pmap_pvo_allocf() callable without Giant.

Add -dynamic to natd if dhcp is used for the natd interface.
Kill natd in stop().

Reviewed by: mtm

Pass the file descriptor index down to vn_open.

If the method vector was replaced and we got the "special return code"
smile and trust that whatever happened below DTRT.

Pass the fdidx argument from vn_open{_cred}() onto VOP_OPEN()

Remove GIANT_REQUIRED from kmem_alloc().

Some Typos fix

Add fdidx argument to vn_open() and vn_open_cred() and pass -1 throughout.

Call the new argument "fdidx" that is more precise than "fd".

Try a lot harder to get dependancies right. This involves some ugly
looking ${.OBJDIR} work that has the up-side of actually working
in upgrade and make -jN cases.

This needs to be revisited further, and it is conceivable that
the ${.OBJDIR} stuff can be simplified, but the sheer number of
edge cases and other causes make this Hard(tm). For now, this works.

ip6fw does not handle ESP correctly

PR: kern/54874
Submitted by: JINMEI Tatuya <jinmei@shuttle.wide.toshiba.co.jp>
MFC after: 1 week

Respect BUS_DMA_ZERO in iommu_dvmamem_alloc().

Use the BUS_DMA_ZERO flag.

Now that we can call kmem_malloc without Giant it should be safe
to do mbuf allocation without Giant, so remove the GIANT_REQUIRED
from mb_alloc in the M_TRYWAIT case.

Enable the safe, harp, hatm and patm driver documentations for pc98.

Document the new BUS_DMA_ZERO flag.

Changed the data types of three index variables, two of them
in the `video_state' structure, to larger ones (from u_char to
u_short).  Each can now hold values at least as large as the
size of the array it is meant to point into.

This eliminates warnings printed by GCC 3.3.1 and hence makes
pcvt compilable using -Werror.

Use the BUS_DMA_ZERO flag.

Use the BUS_DMA_ZERO flag instead of bzero()'ing DMA memory.

- Introduce a new busdma flag BUS_DMA_ZERO to request for zero'ed
  memory in bus_dmamem_alloc().  This is possible now that
  contigmalloc() supports the M_ZERO flag.
- Remove the locking of Giant around calls to contigmalloc() since
  contigmalloc() now grabs Giant itself.

Try a lot harder to get dependancies right. This involves some ugly
looking ${.OBJDIR} work that has the up-side of actually working
in upgrade and make -jN cases.

This needs to be revisited further, and it is conceivable that
the ${.OBJDIR} stuff can be simplified, but the sheer number of
edge cases and other causes make this Hard(tm). For now, this works.

Fix some off-by-one errors dealing with limits of server names,
usernames, workgroup names and passwords. We can now connect to
servers with 15-character NetBIOS names. (Some versions of Windows
use semi-random 15-char names by default.)

PR: 46902

Reserve space for the trailing null byte in the srvname member of
struct smb_vc_info.

PR: 46902

Add the -n maxrec option as an alias for -maxrec for compatibility with
System V and consistency with other utilities. Document the new form
instead of the old form in the manual page.

PR: 54661

Use pmap_zero_page() to zero pages instead of bzero() because
they haven't been vm_map_wire()'d yet.

Simplify sigwait code a bit by using a waitset and removing oldsigmask.

Reviewed by: deischen

Allow vm_object_reference() on kernel_object without Giant.

Changed the type of the variable `qidx' from u_int8_t to int,
mainly to quiet a warning emitted by GCC 3.3 about comparing
a variable to a value which is larger than the former can hold.

The value was checked to make sure the `np->squeue' array is
not accessed behind its boundary.
This worked due to possibly accidental truncation when
(np->squeueput + 1) was larger than or equal to MAX_START (256)
when it was assigned to `qidx'.

`qidx' is used to hold the next position in the start queue
for an insertion.  The new type was chosen because some other
code in the function ncr_freeze_devq() also uses plain integers
to hold those indices.

Wrapped the line after the closing parenthesis of an `if'
condition.

Use M_WAITOK instead of M_WAIT in sppp_attach().

Acquire Giant rather than asserting it is held in contigmalloc().  This is
a prerequisite to removing further uses of Giant from UMA.

Add notice about the local change in r1.25

Fix a couple of bugs in the resume handler.  Don't call the if_init
function unless the device is configured up.  Without this fix, the
device ends up in the RUNNING state even though it is configured down.
Also, check the RUNNING flag before calling the if_start function, in
case the if_init function failed for one reason or another.

Define the stop command to do nothing when only a specific
interface is to be stopped. In this case we don't want to go
beyond releasing the lease.

When stopping a dhcp interface, don't just release the lease on
the address, also kill the dhclient process. Instead of doing the
release in the stop command, move it to the precmd stage and allow
rc.subr(8) to automatically kill the dhclient process by leaving the
stop command undefined.

Noticed by: mbr

Add Socket Communications Low Power 10/100 CF Ethernet card.

Obtained from: NetBSD (information, not code)

Sync to 1.61 of pccarddevs

MFNetBSD:
1.183: gmcgarry; Add Sharp PALDIO 611S
1.184: martin; Socket LP 10/100 CF

Obtained from: NetBSD

o Fix usage(): remove '-l', add missed '-f', sort.

Inline a function that gcc refused to inline. This function was used
only in one place and it just served as semantic sugar.

Silence a gcc-warning. Do this by inlining the macro-call. This is
not very nice - the compiler should just silently optimize away the
unused else clause.

The /rescue system is now enabled by default again, so remove note
saying that it is disabled.

Approved by: ceri (mentor)

Set mc_len to sizeof(mcontext_t), otherwise it is an invalid context.

Remove references to the '-l' option in synopsis.  The rest of the
description of this option was removed in v. 1.22.

PR: docs/54880
Submitted by: Lukas Ertl <l.ertl@univie.ac.at>
Approved by: ceri (mentor)

Minor mdoc(7) cleanup, based on the PR below.

PR: docs/54826
Submitted by: Lukas Ertl <l.ertl@univie.ac.at>
Reviewed by: ru
Approved by: ceri (mentor)

Fix "write from stdin".

Reviewed by: sos
MFC after: 1 week

Fixed wrong function names in the error message that was translated to
C in rev.1.32 so that it doesn't take 2 commits for all branches to fix
this message.

Remove prototype of ia64_pa_access(). The function has been moved to
mem.c where it's been made static.

Enable wake up GPE before shutdown, not only for sleeping.

Avoid using __aligned(16). Instead define the jmp_buf in terms of
long doubles. This gives us 16-byte alignment. Add a CTASSERT for
the size of the jmp_buf to detect ABI breakages.

Add a "int fd" argument to VOP_OPEN() which in the future will
contain the filedescriptor number on opens from userland.

The index is used rather than a "struct file *" since it conveys a bit
more information, which may be useful to in particular fdescfs and /dev/fd/*

For now pass -1 all over the place.

Unbreak ia64 builds now -Werror is enabled again. Avoid obsolete
memory operand construct.

Guard against MLEN growing larger than a uint8_t due to MSIZE grwoing to a
value of 512 in LINT.  This keeps gcc from complaining.

Bah, I go and get all preachy, and then commit from the wrong branch.  Revert
the use of '-j8'.

Apologies to: ru

Fix the release build with a forgotten '-f'.

Fixes for 64 bit cleanliness.  The length arg to sysctl is a pointer to
size_t not int.

Reviewed by: bp

Use #include <string.h> rather than <strings.h> so that the strdup()
prototype gets brought into scope.  This is a big deal for 64 bit
systems where the default return value of 'int' is != pointer.

Reviewed by: bp

Gulp ... call kmem_malloc() without Giant.

Fix some recently introduced warnings.  'Declaration does not declare
anything' etc.

Turn -Werror back on again.  I've tested with/without
invariants/witness/etc on i386, sparc64, amd64 and alpha for GENERIC.
Lint probably still needs fixing, as do a couple of other drivers
that have broken recently and not been noticed.

Fix typo.

Change the inline limit switch to something that exists in gcc-3.2.
Unfortunately, it has different units.

Revise and improve ntfs_subr.c 1.30: read only a single cluster at a time
in ntfs_writentvattr_plain and ntfs_readntvattr_plain, and purge the boot
block from the buffer cache if isn't exactly one cluster long. These two
changes work around the same buffer cache bug that ntfs_subr.c 1.30 tried
to, but in a different way. This may decrease throughput by reading smaller
amounts of data from the disk at a time, but may increase it by avoiding
bogus writes of clean buffers.
Problem (re)reported by Karel J. Bosschaart on -current.

Revert previous commit. We don't use setjmp()/longjmp() for context
switching anymore, so there's no need to save and restore GP. This
change breaks threaded applications linked against libc_r. Pull the
tier 2 card again: relink. This will link against libthr instead.

revision 1.51 of vm/uma_core.c modified uma_large_malloc() to acquire
Giant when needed.

powerpc isn't tinderbox-ready yet.

Use __FBSDID().

Brought to you by: a boring talk at Ottawa Linux Symposium

Use __FBSDID().

Brought to you by: a boring talk at OLS

Add support for the M_ZERO flag to contigmalloc().

Reviewed by: jeff

Make this 64 bit clean. Use size_t for sysctl() length pointer args.

Remove alpha vtophys() hack from if_bgereg.h and clean up unneeded
#includes formerly required to pull in vtophys support, since we
don't need them anymore.

Convert bge(4) to use busdma. I have not tested this on anything
besides x86 yet since I don't have access to a non-x86 FreeBSD
box at the moment. Volunteers welcome.

Fix for 64 bit platforms.  sysctl's length args are pointers to
size_t, not int.  This could be fatal where size_t is long.

Reviewed by: bp

MFi386 revision 1.416
 Add vm object locking to pmap_prefault().

Note: powerpc and sparc64 do not implement this function.

Discussions on src-committers with alfred and bde pointed out that
the "do-nothing" versions of __RCSID(), __RCSID_SOURCE(), __SCCSID(),
and __COPYRIGHT() were not strictly correct.  They should not expand
into [nothing], because the ';' which follows them would then cause
a syntax error (in a strict C compiler, if not gcc...).

So, change the do-nothing versions of those macros to use the
'struct __hack' tactic, as was already used with __FBSDID().

Approved by: discussions with bde
MFC after: 1 week

Fix amd64 build by deleting trailing whitespace.  Doh!

State that in -exec ... ; the semicolon should be quoted if invoked from
a shell.

PR: docs/54667
Submitted by: Patrick Alken <pa59@cornell.edu>
MFC after: 3 days

Fix output from an error message.
Use sysctl -n instead of sed

Submitted by: Scott Lambert <lambert@lambertfam.org>

Add wrapper for kqueue() to keep track of the allocated fd and allow it to
be closed. This fixes a file descriptor leak when closing a kqueue() fd.

Reviewed by: deischen
MFC after: 1 week

Add an informational debugging printf of the maximum time spent in
EcEventWait().  If you get AE_HARDWARE_NO_RESPONSE errors, please enable
this info by setting hw.acpi.verbose=1 in loader.conf.

Backed out the micro-optimization in 1.4.  It was to help gcc-2.6.3
on i486's (and probably i386's), but it has had very little effect
since gcc-2.7 or gcc-2.95.  With gcc-3.3, it gave a small
pessimization for at least i386's, athlon-xp's and pentium4's, a
small optimization (I think) for pentium1's, and made no difference
for i386's.  (movzbl is best for all the later processors, and the
micro-optimization was to stop it being used on i486's.)

New release notes:  bge(4) 5705 support.

Update release notes:  rescue(7) man page.

MFC:  safe(4).

Make the PC98 small boot floppy usable again.

Spotted by: nyan
Reviewed by: nyan

When deciding whether to download the microcode or not look at the API rather
than at the vendor. We have three different Fore cards and only the PCA200
need the microcode. Look also at the RAM address and load the code only if
it is not zero. A zero RAM address means either a bug in the driver or
this is a interface created by harp(4) in which case fatm(4) handles the
microcode issue.

Print the offending SPANS message only if printing is enabled.

An u_int8_t can never be bigger than 255, so remove a useless check.

Spotted by: GCC

Remove a duplicate entry.

o Do not overwrite saved interrupt priority level by alloc_hash(),
  use a separate variable.
o Restore interrupt priority level before return (no-op in HEAD).

Spotted by: Don Bowman <don@sandvine.com>
MFC after: 5 days

The RELEASE9 knob is now OBE.

OK'ed by: jhb

Moved the boot floppy generation code out from release.9 to release.10.
The release.9 target is now responsible only for generation of MFS root
file systems, that are built for all architectures, even those that do
not provide a floppy installation option.  The release.10 target is now
responsible for creation of a set of boot, MFS root, and fixit floppies,
and the NO_FLOPPIES variable now affects only this target.

Also, replaced the FIXIT_TARGET variable with a check of whether the
*FIXITSIZE variables are present for a given architecture, similar to
how this is done when generating boot floppies.

Discussed with: jhb

Add support for VBR and CBR PVCs for IP over ATM.

Submitted by: Vincent Jardin <vjardin@wanadoo.fr>
MFC after: 2 weeks

Make library WARNS=6 clean. The problems have been: alignment on sparc64
and one of the usual sizeof(in_addr_t) == sizeof(u_long) bugs.

Due to a gcc bug, it doesn't like local variables with names like 'sin'.
Rename this for the moment. Also fix a sparc64 alignment warning.

Make ilmid WARNS=6 clean. The problem were a couple of unused function
arguments and missing consts.

Remove __aligned(16) from the definition of struct _ia64_fpreg. It's
a non-standard construct. Instead, redefine struct _ia64_fpreg as a
union and put a long double in it. On ia64 and for LP64, this is
defined by the ABI to have 16-byte alignment. For ILP32 a long double
has 4-byte alignment, but we don't support ILP32.

Note that the in-memory image of a long double does not match the in-
memory image of spilled FP registers. This means that one cannot use
the fpr_flt field to interpet the bits. For this reason we continue
to use an aggregate type.

Remove INVARIANT* and WITNESS. This makes the simulator much more
pleasant to use.