fix indentation of a comment

fix a typo in a comment

Remove ip_fw_fwd_addr (forgotten in previous commit)
remove some extra whitespace.

plxcard for OLDCARD isn't going to happen.

plxcard for OLDCARD almost certainly isn't going to happen.

As disclosed to arch@, make more interfaces standard.  This allows for
easier loading of modules that might refer to these interfaces.  None
of the code that implements them is standard, just the glue.  This
bloats the kernel a whopping 8k.

Silence on: arch@

Rename the BALLOC flags from B_* to BA_* to avoid confusion with the
struct buf B_ flags.

Approved by: mckusick

What:
o ToPIC is happy with two cards now, even when the two cards are
  modems.
o Fix (all?) hangs on boot when power is applied to the card.  I
  suspect that this will make the Ricoh bridges happier and also
  make a lot of VAIO owners happy (confirm to me in private email
  please :-).
o All Cardbus bridges should now support 3.3V, X.XV and Y.YV cards,
  to the extent that the underlying hardware supports such cards.
  (X.X and Y.Y haven't been assigned values yet :-).
o Better 3.3V support for Ricoh ISA bridges.

How:
o Don't mess with the power register when scanning the cards.  It
  is unnecessary and causes BADVcc conditions on many chipsets.  These
  in turn can cause an interrupt storm.
o Make pcic_disable reset the slot's voltage.
o Move initializing voltage for the slot until after it has been
  disabled.
o Fix a lot of issues with the pcic_cardbus_power routine.  We
  now properly enable the card and take it out of reset after
  a power change.
o When detecting the card's voltage, if we're in a BadVcc state,
  direct the bridge to rescan the card for what it supports.
  (we might need to in the future set the power register to 0
   before doing this).
o Don't preserve CLKSTOP.  need to revisit this.
o Better support for Ricoh ISA bridges for 3.3V cards.
o Don't write to PCIC_POWER directly as offten, but instead go
  through the pcic_power interface.
o All cardbus bridges now default to use cardbus power control.
o Add misc register definitions.
o remove some (now) bogus comments.

Extra Special Thanks To: Scott Lamber for his kind and generous loan
of a Toshiba laptop with a ToPIC 100 in it for my use.

 o Remove the unnecessary acquisition and release of Giant around fdrop()
   in mmap(2).

Correct default location of history file.

Observed by: Chris Bolt <chris-lists@bolt.cx>

Only got one of the Perl usages.

Replace a perl usage with an AWK one.

Submitted by: Bob Willcox <bob@immure.com>

KTR_CT* had one too many trailing zeroes, making KTR_CT5-8 too large for
ktr_mask.

Fix a bug related to marking pages virtually uncacheable due to illegal
dcache aliasing.  A page that already had more than 1 mapping of the
same virtual colour would not be correctly uncached.

Noticed by: Artur Grabowski <art@openbsd.org>

Add additional cpuid feature flags and put into a canonical format.

MFC after: 1 week

Get rid of paranoia that zeros the boot block area as this has
bad effect on existing bootstraps.

Submitted by: Jake Burkholder <jake@locore.ca>
Sponsored by: DARPA & NAI Labs.

Fix several bugs in the i386 asm statements used to speed up Internet
checksumming.  These bugs could possibly cause bad code to be
generated at elevated optimization levels.

First, eliminate the use of preprocessor magic to form the address
fields of asm instructions.  It hid the actual addresses being
referenced from the compiler.  Without knowledge of all the data
dependencies, the compiler might possibly use optimizations which
would result in incorrect code.

Use "__asm __volatile" rather than "__asm" for instruction sequences
that pass information through the condition codes (the carry bit, in
this case).  Without __volatile, the compiler might add unrelated
code between consecutive __asm instructions, modifying the condition
codes.  I have seen GCC insert stack pointer adjustments in this
way, for example.  Unfortunately, GCC doesn't provide a way to
specify dependencies on the condition codes.  You can specify that
they are clobbered, but not that you are going to use them as input.

Finally, simplify the LOAD macro.  This macro is used as a poor
man's prefetch.  The simpler version gives the compiler more leeway
about just how it performs the prefetch.

MFC after: 1 week

Add a reference to the "make world" section of the Handbook.

PR: 39460
Submitted by: nik

This patch fixes a size problem with the stat structure for
64-bit architectures that was introduced in the UFS2 code
merge two days ago. The stat structure change that caused
the problem was the addition of the file create time.

Submitted by: Bruce Evans <bde@zeta.org.au>
Sponsored by: DARPA & NAI Labs.

New release notes:  MCA support on ia64, rp(4) update (+MFC), tx(4)
multicast filtering, ahd driver, calendar(1) -W/-F, nice(1) -n,
pam_echo(8), pam_exec(8), pselect(3), wc(1) -m, USD/PSD documents
restored.

We don't need to check the return value of malloc() against
NULL when the M_WAITOK flag is specified.

This patch fixes a problem whereby filesystems that ran
out of inodes in a cylinder group would fail to check for
free inodes in other cylinder groups. This bug was introduced
in the UFS2 code merge two days ago.

An inode is allocated by calling ffs_valloc which calls
ffs_hashalloc to do the filesystem scan. Ffs_hashalloc
walks around the cylinder groups calling its passed allocator
(ffs_nodealloccg in this case) until the allocator returns a
non-zero result. The bug is that ffs_hashalloc expects the
passed allocator function to return a 64-bit ufs2_daddr_t.
When allocating inodes, it calls ffs_nodealloccg which was
returning a 32-bit ino_t. The ffs_hashalloc code checked
a 64-bit return value and usually found random non-zero bits in
the high 32-bits so decided that the allocation had succeeded
(in this case in the only cylinder group that it checked).
When the result was passed back to ffs_valloc it looked at
only the bottom 32-bits, saw zero and declared the system
out of inodes. But ffs_hashalloc had really only checked
one cylinder group.

The fix is to change ffs_nodealloccg to return 64-bit results.

Sponsored by: DARPA & NAI Labs.
Submitted by: Poul-Henning Kamp <phk@critter.freebsd.dk>
Reviewed by: Maxime Henrion <mux@freebsd.org>

Don't expect NUL terminated data in all netgraph messages received.
Only display message hook values we understand.

NUL terminate the ACNAME passed to userland.

Warning fix.

Reviewed by: peter

Create sendmail required directories if sendmail is enabled.

OLDCARD version of GENERIC.

 o Reduce the scope of Giant in vm_mmap() to just the code that manipulates
   a vnode.  (Thus, MAP_ANON and MAP_STACK never acquire Giant.)

Fix a bug in vfs_bio_clrbuf().  The single-page-clrbuf optimization was
improperly clearing more then just the invalid portions of the page.  (This
bug is not known to have been triggered by anything).

Submitted by: tegge
MFC after: 7 days

 o Replace mtx_assert(&Giant, MA_OWNED) in dev_pager_alloc()
   with the acquisition and release of Giant.  (Annotate as MPSAFE.)
 o Reorder the sanity checks in dev_pager_alloc() to reduce
   the time that Giant is held.

The last bits of the alloca -> mmap fix.  IA64 and SPARC64 (current only).
Untested (testing request went unanswered), but sparc64 is not expected to
cause problems.  IA64 is not expected to cause problems but the patch was
slightly more complex so the possibility exists.

Approved by:    jdp

Use rm -f in the clean target, as seems to be common practice, and also avoids
errors if no LINT exists.

Submitted by: dwcjr

o Remove the initialization of unused fields in the struct
  uio now that we don't use uiomove() anymore.
o Enforce stricter checks on the length of the iov's in
  nmount(2) since we now malloc() them individually and
  corrupted iov's could make the kernel crash in malloc()
  with "kmem_map too small".

Reviewed by: phk

 o In vm_map_insert(), replace GIANT_REQUIRED by the acquisition and
   release of Giant around the direct manipulation of the vm_object and
   the optional call to pmap_object_init_pt().
 o In vm_map_findspace(), remove GIANT_REQUIRED.  Instead, acquire and
   release Giant around the occasional call to pmap_growkernel().
 o In vm_map_find(), remove GIANT_REQUIRED.

For the benefit of those Emacs users amongst us, only cut out gdb.info
rather than *.info.

When linking staticaly libtermcap is a postrequisite of libreadline.
Otherwise `tgoto' (only used by libreadline) isn't resolved.

Submitted by: bde

Take the guaranteed fix -- turn off .info docs for now until we get the
gdb docs sorted out.

Make this compile on Alpha.

Clock frequencies reported by sysctl should be unsigned values. Discovered
when machdep.tsc_freq returned a negative number on a 2.2GHz Xeon.

Submitted by: Brian Harrison <bharrison@ironport.com>
Reviewed by: phk
MFC after: 1 week

Stylistic nit:

main()'s argv argument is char*[], and functions that may inherit that arg
should use char*[] as well.

Check for results of repeated calls to strnsubst(), as well as for the
behaviour with NULL match string, as that has changed over time.

Improve the handling of Encode and Decode operations in MD5.

Use memcpy for all little-endian architectures, sys/kern/md5c.c indicates
this should be safe for all currently supported LE archs.

Change the Encode and Decode functions for other archs to use le32toh()
and htole32() functions instead of explicit byte shuffling.

On sparc64 this gives md5(1) about 8% speed increase.

When giving an example that relies on shell expansion/globbing, don't use a
replstr for -J that will be interpereted by the shell.

MFC after: 1 day

Remove (almost all) global variables that were used to hold
packet forwarding state ("annotations") during ip processing.
The code is considerably cleaner now.

The variables removed by this change are:

        ip_divert_cookie        used by divert sockets
        ip_fw_fwd_addr          used for transparent ip redirection
        last_pkt                used by dynamic pipes in dummynet

Removal of the first two has been done by carrying the annotations
into volatile structs prepended to the mbuf chains, and adding
appropriate code to add/remove annotations in the routines which
make use of them, i.e. ip_input(), ip_output(), tcp_input(),
bdg_forward(), ether_demux(), ether_output_frame(), div_output().

On passing, remove a bug in divert handling of fragmented packet.
Now it is the fragment at offset 0 which sets the divert status of
the whole packet, whereas formerly it was the last incoming fragment
to decide.

Removal of last_pkt required a change in the interface of ip_fw_chk()
and dummynet_io(). On passing, use the same mechanism for dummynet
annotations and for divert/forward annotations.

option IPFIREWALL_FORWARD is effectively useless, the code to
implement it is very small and is now in by default to avoid the
obfuscation of conditionally compiled code.

NOTES:
 * there is at least one global variable left, sro_fwd, in ip_output().
   I am not sure if/how this can be removed.

 * I have deliberately avoided gratuitous style changes in this commit
   to avoid cluttering the diffs. Minor stule cleanup will likely be
   necessary

 * this commit only focused on the IP layer. I am sure there is a
   number of global variables used in the TCP and maybe UDP stack.

 * despite the number of files touched, there are absolutely no API's
   or data structures changed by this commit (except the interfaces of
   ip_fw_chk() and dummynet_io(), which are internal anyways), so
   an MFC is quite safe and unintrusive (and desirable, given the
   improved readability of the code).

MFC after: 10 days

Define an mbuf type, MT_TAG, used for volatile annotations
prepended to mbuf chains in the network stack.
Reuse a previoulsy unused value to avoid changes in other
data structures.

Replace the SWAP(var0,var1) macro with SWAP(type,var0,var1) and use it as
is appropriate to avoid using typeof/__typeof__.  It is worth noting that
SWAP() is only ever used to swap pointer values so 'void *' assumptions would
have been acceptable, but I'd gladly pay you tuesday for a cheeseburger^W
cleaner interface today.

Poked into submission by: bde

Unused macro.

Always drop the p_args reference we held for copyout, even if we're about
to change it. This fixes a leak triggered by setproctitle(3).

Approved by: alfred
Noticed by: Peter Jeremy <peter.jeremy@alcatel.com.au>

Grre-^#$(*+#@$!@% I hate .info files, I really do (thus I have NO_INFO set...)
Don't try to read files from a non-existant gdb.291/.

 o Replace GIANT_REQUIRED in swap_pager_alloc() by the acquisition and
   release of Giant.  (Annotate as MPSAFE.)

 o Remove GIANT_REQUIRED from phys_pager_alloc().  If handle isn't NULL,
   acquire and release Giant.  If handle is NULL, Giant isn't needed.
 o Annotate phys_pager_alloc() and phys_pager_dealloc() as MPSAFE.

 o Replace GIANT_REQUIRED in vnode_pager_alloc() by the acquisition and
   release of Giant.  (Annotate as MPSAFE.)
 o Also, in vnode_pager_alloc(), remove an unnecessary re-initialization
   of struct vm_object::flags and move a statement that is duplicated
   in both branches of an if-else.

Now that the pattern space contains no trailing newline, modify the `l'
command's output so it's the same as what SUSv3 specifies.

Make it clear that this applies only to UFS1 file systems, as UFS2 will
have native extended attributes rather than stacked extended attributes.
While I'm at it, make sure UFS_EXTATTR is not spelt FFS_EXTATTR.

Sponsored by: DARPA, NAI Labs
Obtained from: TrustedBSD Project

Don't store newlines at the end of each line in the hold/pattern spaces,
instead add the newline when the pattern space is printed. Make the `G' and
`H' commands add a newline to the space before the data, remove bogus
addition of newline from `x' command.

PR: 29790, 38195

 o Remove GIANT_REQUIRED from vslock().
 o Annotate kernacc(), useracc(), and vslock() as MPSAFE.

Motivated by: alfred

Tweaks to make these files live beside our hacked config.

Fix logic which resulted in missing a call to INP_UNLOCK().

Submitted by: jlemon, mux

Tweaks for the 64-bit compiler.

TCP notify functions can change the pcb list.

Enable cd9660 support by default.

Enable UFS1_AND_UFS2 support for sparc64 by default.  Booting from ufs1 or
ufs2 filesystems seems to work fine.

Remove CAPABILITIES from NOTES

Hone the rpcbind dependency checking.

Submitted by: Mike Makonnen <makonnen@pacbell.net>

Clarify revision 1.14:
Gcc 3.1's -O0 and -O1 actually optimized alignment for space, but we feel
it should optimize alignment for time like Gcc 2.95 used to.  Optimization
for space should give 1-byte alignment on i386's, but doesn't quite.

Forced commit to keep cvs happy.

Forced commit to keep cvs happy.

Forced commit to keep cvs happy.

Forced commit to keep cvs happy.

Forced commit to note we are pretending contrib/gdb.291 never existed.

This commit was generated by cvs2svn to compensate for changes in r98576,
which included commits to RCS files with non-trunk default branches.

Sync with NetBSD's mainline.

We need to link with libiberty.a before libreadline.so.  Both supply
xmalloc() and xrealloc() and the mixed usage of xmalloc in some .c's from
libiberty.a and other .c's from libreadline.so produces an unusable binary
on the Alpha.

While I am here, preventatively move other libs in the link order.

Submitted by: gallatin

Forced commit after RCS file surgery.

cc -O0 and -O1 didn't do the easy optimization of alignment for space.
Instead it attempted to do the easy optimization of alignment for time,
which should be to 1-byte alignment on i386's.

Submitted by: bde

BDE prefers this organization.

Submitted by: bde

Sync with NetBSD.
* don't whine about non-regular files. It is perfectly normal to keep a
  CVS or RCS directory in /etc/rc.d.
* manpage tweak

This commit was generated by cvs2svn to compensate for changes in r98567,
which included commits to RCS files with non-trunk default branches.

Reduce the -I's as we don't need them all.

New release notes:  ng_device(4), UFS2, newfs(8) -O, newgrp(1),
pam_ksu(8), uuidgen(1).

Updated release notes:  libpcap-0.7.1, tcpdump-3.7.1.

Also do some markup fixes to recently-added items so that they don't
try to use entities for manpages that have been removed, in the spirit
of rev. 1.359.

__FBSDID() strategic insertion.

Kill __P, yuck.

Mark unused variables __unused.

Built standalone, inetd(8) is WARNS=5 clean, WARNS=6 if you ignore %m fits.

Use __typeof__ instead of typeof.

Kill bad whitespace and do some style cleanups as a result of the protoize.

ANSI prototypes via protoize(1).

Better fix for style.

Suggested by: ru (but modified a bit by markm)

Revert the part of Kirks UFS2 commit which added divdi3.c and moddi3.c
to libi386, this issue was resolved already in a cleaner way.

Clean up hex() and octal() to return and work with unsigned integers since
they scan values of unsigned types, and since they do not need otherwise,
have them take const char * arguments.

Fix warnings generated elsewhere.

Style tidy-up.

Modernise this code by rounding up the usual suspects: register keyword,
consts, ANSIfication of functions, proper use of __FBSDID() and so on.
This commit is non-functional from a code-execution perspective, but
pr(1) now compiles WARNS=6 clean.

Don't say that an umounted filesystem is required for -n.  An unmounted
filesystem is no more or less required for -n than for any other option.
The previous commit clarified the actual requirement.

Fix incorrect library ordering.  I thought I'd committed this already...

#include <sys/time.h> before <sys/stat.h> to get timespec.

Make NO_OPENSSL actually imply NO_OPENSSH, as documented in make.conf(5).

Skip fields in the manner required by POSIX, and the way V7 did it.

MFC after: 1 week

Quiet ``make objlink'' when NOOBJ is defined.

PR: bin/21142
Submitted by: Craig Leres <leres@ee.lbl.gov>

Newline characters should not participate in line comparisons. Only apparent
when -s is used or the last line of the file is missing a newline.
Noticed by the textutils test suite.

MFC after: 1 week

A gross patch to tidy up the formatting.

Not objected to by: grog