Code cleanup.  Refactor some functions.  Push some globals into local scope.

- Bring in patch to make.1 related to previous patch.

taken-from: FreeBSD
author: harti

Explicitly mention in RETURN VALUES that socket() sets
errno upon failure.

Noticed-by: Liam J. Foy <liamfoy@sepulcrum.org>

Don't build a new mbuf and bpf_mtap the old one.

Fix the quoting of MAKEFLAGS as noted in rev. 1.47.
This still has a slight problem with trailing backslashes
on the comment line, but that will be documented in the man page.

Written-by: Harti Brandt, harti AT freebsd DOT org

Add some spaces for typographical correctness.

- Add support for attaching alternative DLTs to an interface.
  The function is named bpfattach_dlt, FreeBSD used bpfattach2.
- Change bpf_mtap to take the bpf_if directly.
- Add support for BIOCSDLT and BIOCGDLT to recieve a list of DLTs
  available on an interface.

Obtained-from: FreeBSD

- Add bpf_ptap, which takes an mbuf and a header, builds a temporary
  mbuf chain and calls bpf_mtap on that.

Inspired-by: FreeBSD's bpf_mtap2
- Use the new interface for all drivers doing this by hand before.
- Include the formerly commented out support for radio headers in wi(4)
  and wlan(4).
- Change if_loop, i4b_ipr, disc and ic to prepend a uint32_t, not an int.

Remove the _KERNEL parts.

Remove !_KERNEL parts.

Fix a type in rev. 1.16

Disable hardware checksum support by default, it produces packet corruption.
It is unclear why the corruption occurs, but certain fragmented packets
consistently reproduce it so there's a good chance that there may be alignment
or length requirements that we don't know about, or just pure hardware
brokedness with certain packets.

The checksumming can be turned on again with ifconfig for testing purposes.

 EXTRACTION from Aggelos's packet dumps ]

CLIENT:  (RE0)

11:43:27.261710 192.168.2.2.183764104 > 192.168.2.4.nfs: 1472 write [|nfs] (frag 8031:1480@0+)
11:43:27.261718 192.168.2.2 > 192.168.2.4: udp (frag 8031:1480@1480+)
11:43:27.261729 192.168.2.2 > 192.168.2.4: udp (frag 8031:1480@2960+)
11:43:27.261743 192.168.2.2 > 192.168.2.4: udp (frag 8031:1480@4440+)
11:43:27.261756 192.168.2.2 > 192.168.2.4: udp (frag 8031:1480@5920+)
11:43:27.261767 192.168.2.2 > 192.168.2.4: udp (frag 8031:1480@7400+)
11:43:27.261781 192.168.2.2 > 192.168.2.4: udp (frag 8031:1480@8880+)
11:43:27.261793 192.168.2.2 > 192.168.2.4: udp (frag 8031:1480@10360+)
11:43:27.261807 192.168.2.2 > 192.168.2.4: udp (frag 8031:4@11840)

SERVER:  (RL0)

13:56:59.783671 192.168.2.2.183764104 > 192.168.2.4.nfs: 1472 write [|nfs] (frag 8031:1480@0+)
13:56:59.783785 192.168.2.2 > 192.168.2.4: udp (frag 8031:1480@1480+)
13:56:59.783915 192.168.2.2 > 192.168.2.4: udp (frag 8031:1480@2960+)
13:56:59.784037 192.168.2.2 > 192.168.2.4: udp (frag 8031:1480@4440+)
13:56:59.784159 192.168.2.2 > 192.168.2.4: udp (frag 8031:1480@5920+)
13:56:59.784283 192.168.2.2 > 192.168.2.4: udp (frag 8031:1480@7400+)
13:56:59.784407 192.168.2.2 > 192.168.2.4: udp (frag 8031:1480@8880+)
13:56:59.784527 192.168.2.2 > 192.168.2.4: udp (frag 8031:1480@10360+)
13:56:59.784532 0.0.0.0 > 0.0.2.4: udp (frag 8031:4@11840)

Reported-by: aoiko@cc.ece.ntua.gr (Aggelos)
Note-Also: also turned off in FreeBSD /usr/src/sys/dev/re/if_re.c:1.37

Mostly switch to the patch framework. For cvs.1.patch, it's easier to do by
hand.

Add a patch to let CVS ignore the environment variable CVSREADONLYFS for
remote repositories.  This allows a user to make lock-operation the
default behaviour for local repositories without cludges for remote
support.

- Add -v option (verbose). Taken from FreeBSD with my modifications.
- Remove unnecessary header (errno.h)
- Static functions
- Add new -v option to man page

Repo-copy vinumparser.c and vinumutil.c from /usr/src/sys/dev/raid/vinum
and clean up for userland compilation.

Requested-by: Joerg Sonnenberger <joerg@britannica.bec.de>

re-fix typo.  Maybe its time to start drinking coffee in the morning. :-)

Fix typo.

Sent in by: Richard Bradshaw <theglassishalfempty@gmail.com>

Reverting all of today's changes.

- Cleanup field names in Buffer object, and add better comments.

- New function Buf_AppendRange(), which is given a pointer to a string and
  a pointer to the end of the range to append to the Buffer object.

- New function Buf_Append(), which is given a pointer to a string to
  append to the Buffer object.

- Inline silly #define MACRO.

- Removed unnessisary append of '\0' to Buffer object.  Buffer
  already maintains a terminating null byte.

- Rewrite Buffer object to maintain only write pointer.  There is no
  need to maintain both a write pointer and a length.

Use BPF_TAP and BPF_MTAP instead of the expanded versions where possible.

General clean up of lptcontrol. Similar to FreeBSD 1.15
with my modifications:

- Remove many unnecessary headers and sort them
- Remove unused #define (PATH_LPCTL)
- Don't bother calling set_interrupt_status. Just call
  both open(2) and ioctl(2) in main.
- Remove modes in open(2)

Use memchr(3) instead of a hand-rolled loop.

Requested-by: joerg

Fix kerberos5 build by adding some headers files. These should be generated on
the fly, but the make depend step is acting up.

Reported by: joerg

ether_input already handles BPF and dropping of
non-local packages, so stop doing this. Use BPF_MTAP instead of the
expanded version.

Replace IPv6 related manual pages that may have violated
the IETF's copyright.

Obtained from: KAME and OpenBSD (via FreeBSD)

- Remove a duplicate unnecessary check on errno. strtol(3) will only ever set
  errno to ERANGE which is already checked.

Remove the support for lib/csu/${MACHINE_ARCH}-elf.
This should also prevent some people from shooting themselves with
partly cleaned sources.

Consolidate ifqueue macros for the upcoming ALTQ work.
We don't use ifqueue locking and don't intend to add it,
so the whole IF_LOCK / IF_UNLOCK compatibility with
FreeBSD 5 can just be removed. Similiar the "unlocked"
versions of IF_PREPEND, IF_DEQUEUE and IF_ENQUEUE
(with leading _) are replaced with the normal versions.
IF_QEMPTY from the i4b sources is added to iv_var.h for
symmetry with the ALTQ macros, but not yet used elsewhere.

Use __DEQUALIFY, not only __DECONST to get rid of the volatile too.
This is safe for fuword (any interesting architecture has atomic
fetch instructions for integers) and the other uses doesn't touch
the memory at all.

Add forward declaration of struct vnode to fix compiler warning.

Adjust calling convention for idp_input, idp_output and spp_input to match
what the rest of the kernel uses in protosw.

Add missing 'by' in the license text of the man page.
Hook seq(1) into the build process.

Now that we generate the ethernet header in place in the mbuf instead
of in a secondary buffer, we have to remove space for the old ethernet
header before we send a packet waiting for ARP resolution back to
ether_output() again.

Add a simple tool to generate a sequence of numbers, without all the
difficulty of jot. A very similiar tool is part of plan9 and GNU's
core-utilities.

Submitted-by: Brian Ginsbach <ginsbach@pressenter.com>

No longer used.

- Complete re-write of sasc.

Ok'ed by: Joerg, and others.

- put #ifdef around PrintAddr() proto type.

- We use vfork() so if execv fails we should use _exit().

- Move PrintAddr() into only file that uses it.

- Put #ifdef around PrintAddr()

Move libutil up above KERBEROS5 libs and remove the duplicate entry when
WITH_KERBEROS5 is defined during build.

From FreeBSD:
MFC 1.98:
    Plug a mem leak in vinum_scandisk().

The vnode reclamation code contains a race whereby a blocking condition may
occur between the clearing of an inode's bitmap bit, the removal of the
inode from the hash table, and the reclamation of the related vnode.

This can cause a file create to block on the vnode undergoing reclamation
if it happens to select the same inode (a case which is more likely these
days due to dirpref).  To avoid the race, adjust UFS to *NOT* allocate
inode numbers which are still present in the inode hash table.

This race is NOT a bug, or at least it is not a known cause for existing bugs,
but both the standard vnode code and especially the softupdates code is
so complex that I am not taking any chances.  As of this commit both FreeBSD-4,
FreeBSD-5, and DragonFly are known to have rare 'dup alloc' and other
related filesystem/inode panics and this commit removes one variable from
that equation.

Replace the cache-point linear search algorithm for VM map entries with
a red-black tree.  This makes VM map lookups O(log N) in all cases.

Note that FreeBSD seems to have gone the splay-tree route, but I really
dislike the fact that splay trees are constantly writing to memory even
for simple lookups.  This would also limit our ability to implement a
separate hinting/caching mechanism.  A red-black tree is basically a
binary tree with internal nodes containing real data in addition to the leafs,
simlar to a B+Tree.  A red-black tree is very similar to a splay tree but it
does not attempt to modify the data structure for pure lookups.

Caveat: we tried to revive the map->hint mechanism but there is currently
a serious crash/lockup bug related to it so it is disabled in this commit.

Submitted-by: Eirik Nygaard <eirikn@kerneled.com>
Using-Red-Black-Macros-From: NetBSD (sys/tree.h)

Properly vget() vnodes that the syncer intends to VOP_FSYNC(), rather then
just locking the vnode.  The original FreeBSD code allowed a vnode to be
locked without being refd, but DragonFly requires that a vnode be ref'd
if it is going to be locked.  This fixes a rare panic that can occur when
the syncer is flushing a vnode that mainline code tries to access at the
same time.

This also brings the syncer inline with the filesystem sync code which has
been using normal vget()'s for several months now.

Reported-by: David Rhodus

Sort sections.

Taken-from: FreeBSD
Author: ru

Add a man page for this program.

Whitespace cleanup.

Change lockcount->lockcountnb in an assertion.

Noticed-by: David Rhodus <sdrhodus@gmail.com>

Kerberos no longer comes with it's own telnet. Always build the one in
src/usr.bin/

Kerberos no longer comes with it's own telnet, so always build the one in
src/usr.bin/

ANSI and whitespace cleanup.  No operational changes.

Add some descriptive comments.

When a PCMCIA networking card is removed the IF code may free() the network
interface before processing has completed on pending packets, leaving a
dangling pointer in the mbuf and causing a crash.

The two solutions are to either ref-count the network interface on a
per-packet basis or to synchronize against consumers of the packet.
ref-counting is very expensive in an MP system so we have chosen to
synchronize against consumers by sending a NOP message to all protocol
processing threads and waiting for it to be replied.  This only occurs when
an interface is being brought down and is not expected to introduce any
performance issues.

Crash-Reported-by: Jonathon McKitrick <jcm@FreeBSD-uk.eu.org>

Remove left-over.

Test commit.

Ignore zero length mbuf in bus_dmamap_load_mbuf.

Submitted-by: Andrew Atrens <atrens@nortelnetworks.com>

Unlike printf(3), fwrite(3) doesn't stop at the first NUL character
it finds in the string; and since the full length of the string was
being passed to fwrite(3), sysctl(8) commands were mistakenly
outputting a NUL character after their text output.

Fix by only fwrite(3)ing the string up to the first NUL character.

Discovered-by: y0netan1

Use PCIB_ROUTE_INTERRUPT instead of PCI_ROUTE_INTERRUPT, the latter is
long dead. Remove the declaration in pci_if.m to avoid future problems.

Spotted-by: corecode

Avoid possible copyright problems and add the copyright of the
original code and the third clause.

Raise WARNS to 6.

Submitted-by: Larry Lansing <lansil_at_fuzzynerd_dot_com>

Raise WARNS to 6.

Submitted-by: Larry Lansing <lansil_at_fuzzynerd_dot_com>

Raise WARNS to 6:
- Use parentheses in a logical condition to prevent ambiguous
  interpretation (by humans, who are imperfect C parsers.)  In doing
  so, simplify it:  x == 0 implies (x != 1 and x != 2).

Raise WARNS to 6.

Submitted-by: Larry Lansing <lansil_at_fuzzynerd_dot_com>

WARNS=6. The __DECONST is save, because execv doesn't mess with the strings,
but can't be specified as such.

Support PORTDIRNAME and PKGORIGIN.

Don't print the override warning for package-name.

Instead of trying to guess the PKGNAME, just ask the override port for it.
*Warning*: incorrectly written override ports (using bsd.port.mk) might
produce end-less loops.  The old behaviour didn't correctly handle
PORTSREVISION and PORTSEPOCH.  Also provide PKGFILE.

Update the kerberos5 build framework to work with heimdal 0.6.3.

Based on: Kerberos5 build framework from FreeBSD.

Merge from vendor branch HEIMDAL:
Add heimdal-0.6.3

Add heimdal-0.6.3

Add directory /usr/include/kadm5.

Submitted by: Richard Nyberg <rnyberg@it.su.se>

Fix type. It is pkgs and not pksgs.

Raise WARNS to 6:
- Keep const parameters const while casting.

Raise WARNS to 6.

Submitted-by: Larry Lansing <lansil_at_fuzzynerd_dot_com>

Raise WARNS to 6:
- Constify a string constant;
- Compare a pointer with NULL, not zero.

Submitted-by: Larry Lansing <lansil_at_fuzzynerd_dot_com>

falloc() was not returning an error code on failure.

Reported-by: David Rhodus <sdrhodus@gmail.com>

Back out part of rev 1.24. The intention the quoting backfires and bad
quoting is generated. Running 'make all-depends-list' in x11-fonts/libXft
does something like this:
PARENT_CHECKED=/usr/ports/x11-fonts/fontconfig /usr/ports/devel/pkgconfig
/usr/ports/print/freetype2 /usr/ports/devel/gmake /usr/ports/converters/libiconv
/usr/ports/devel/gettext
/usr/ports/devel/libtool13
/usr/ports/devel/libtool15
/usr/ports/textproc/expat2

After quoting this becomes:
PARENT_CHECKED=/usr/ports/x11-fonts/fontconfig\ /usr/ports/devel/pkgconfi
g\
/usr/ports/print/freetype2\ /usr/ports/devel/gmake\ /usr/ports/converters/libico
nv\
/usr/ports/devel/gettext\
/usr/ports/devel/libtool13\
/usr/ports/devel/libtool15\
/usr/ports/textproc/expat2\

The next make tries to do this, the following happens:
before: PARENT_CHECKED=/usr/ports/x11-fonts/fontconfig /usr/ports/devel/pkgconfi
g\
after: PARENT_CHECKED=/usr/ports/x11-fonts/fontconfig\ /usr/ports/devel/pkgconfi
g\\
and the final \\ confuses the shell.

Thanks to asmodai for suggesting a make problem as source.

Instead of messing around with sprintf, use asprintf to do the allocation
for us. Don't read before the start of the string, if "" is passed in.

strl* conversion.

Original-patch-by: Andre Nathan <andre@digirati.com.br>

Replace temporary allocation from alloca with malloc/free.

Requested-by: dillon

Raise WARNS to 6:
- Use a full array initializer element;
- Fix signedness issues.

Submitted-by: Larry Lansing <lansil_at_fuzzynerd_dot_com>

Raise WARNS to 6.

Submitted-by: Larry Lansing <lansil_at_fuzzynerd_dot_dom>

Fix typo in error message.

Submitted-by: Roland Hammerle <rjh_at_who_dot_net>

Testing auth forwarding.  Me am idiot!

Minor correction in umtx_*() calls, the mutex pointer should point to
volatile store.  Also correct the type, we are expecting to point to an
integer.

Minor correction in umtx_*() calls, the mutex pointer should point to
volatile store.

Add umtx.c, a simple utility which implements userland mutexes using
the new umtx_*() system calls.  Compile normally and play around, e.g.:

umtx -v -t 10 /tmp/lock.dat sleep 5
umtx -v -t 10 /tmp/lock.dat ls

The test code includes some simple ^C cleanup code as well, though I
don't block signals during the global variable fixup so there is a race in
the ^C cleanup code.

Add missing kern_umtx.c to sys/conf/files.

Noticed-by: David Rhodus

Don't remove /usr/bin/perl if it's a symlink, which was probably
created by `use.perl port'.

Ok'ed-by: David Rhodus

Initial part of DEVICE_POLLING support for wi(4). Still some rough edges,
e.g. occasional timeouts, but working otherwise.

Initial jail support for varsyms.

Add syscall primitives for generic userland accessible sleep/wakeup
functions.  These functions are capable of sleeping and waking up based on
a generic user VM address.  Programs capable of sharing memory are also
capable of interaction through these functions.

Also regenerate our system calls.

umtx_sleep(ptr, matchvalue, timeout)

    If *(int *)ptr (userland pointer) does not match the matchvalue,
    sleep for timeout microseconds.  Access to the contents of *ptr plus
    entering the sleep is interlocked against calls to umtx_wakeup().
    Various error codes are turned depending on what causes the function
    to return.  Note that the timeout may not exceed 1 second.

utmx_wakeup(ptr, count)

    Wakeup at least count processes waiting on the specified userland
    address.  A count of 0 wakes all waiting processes up.  This function
    interlocks against umtx_sleep().

The typical race case showing resolution between two userland processes is
shown below.  A process releasing a contested mutex may adjust the contents
of the pointer after the kernel has tested *ptr in umtx_sleep(), but this does
not matter because the first process will see that the mutex is set to a
contested state and will call wakeup after changing the contents of the
pointer.  Thus, the kernel itself does not have to execute any
compare-and-exchange operations in order to support userland mutexes.

    PROCESS 1 PROCESS 2 ******** RACE#1 ******

    cmp_exg(ptr, FREE, HELD)
. cmp_exg(ptr, HELD, CONTESTED)
. umtx_sleep(ptr, CONTESTED, 0)
. [kernel tests *ptr]     <<<< COMPARE vs
    cmp_exg(CONTESTED, FREE) . <<<< CHANGE
. tsleep(....)
    umtx_wakeup(ptr, 1) .
. .
. .

    PROCESS 1 PROCESS 2 ******** RACE#2 ******

    cmp_exg(ptr, FREE, HELD)
cmp_exg(ptr, HELD, CONTESTED)
umtx_sleep(ptr, CONTESTED, 0)
    cmp_exg(CONTESTED, FREE) <<<< CHANGE vs
    umtx_wakeup(ptr, 1)
[kernel tests *ptr] <<<< COMPARE
[MISMATCH, DO NOT TSLEEP]

These functions are very loosely based on Jeff Roberson's umtx work in
FreeBSD.  These functions are greatly simplified relative to that work in
order to provide a more generic mechanism.

This is precursor work for a port of David Xu's 1:1 userland threading
library.

Remove the old system perl binary when performing 'make upgrade'.

This fixes several problems such a 'make index' which traversed the PATH
and would use the old system perl.

Followup note last commit: FreeBSD PR/66242, FreeBSD/1.68 originally
submitted by Dmitry Sivachenko.

Do not early terminate if ^C is hit just as a valid job is returned by
wait*(), otherwise the job termination will not be properly recorded.

proc0 is still used by e.g. smbfs to fork off a kernel thread and certain
namecache records may be NULL.  Handle the case.

Oops, undo accidental commit.  The last commit was not related to the
datagram reconnection bug for these particular files.