kernel/sound: Port MIDI support.

This was dropped without a clear rationale. Our uaudio(4) for example
supports a whole bunch of MIDI devices.

Tested-by: Ben Woolley <tautolog@gmail.com>

Revert "sound: Drop midi support"

This reverts commit 5bea67f41d941ba13d0b25782f6a3b9a9460e98e.

rpcbind - Fix crash

Bring in 88cb9496ec48 from the FreeBSD repo.

The Sun RPC framework uses a netbuf structure to represent the
transport specific form of a universal transport address.  The
structure is expected to be opaque to consumers.  In the current
implementation, the structure contains a pointer to a buffer
that holds the actual address.

In rpcbind(8), netbuf structures are copied directly, which would
result in two netbuf structures that reference to one shared
address buffer.  When one of the two netbuf structures is freed,
access to the other netbuf structure would result in an undefined
result that may crash the rpcbind(8) daemon.

Fix this by making a copy of the buffer that is going to be freed
instead of doing a shallow copy.

Author:     delphij <delphij@FreeBSD.org>
Security:   FreeBSD-SA-15:24.rpcbind
Security:   CVE-2015-7236

i386 removal, part 44/x: Remove a no longer used header.

kernel/drm: Use #ifdef __DragonFly__ to mark DragonFly specific stuff.

This is better findable across the tree than some home-rolled
"XXX begin/end" markers. Also prettier.

drm/i915: Use gtt.mappable_base

Greatly reducing dependencies on the agp driver.

drm: Add io_mapping_map_wc()

drm: Improve the Linux IO mapping API implementation

* Track address space mappings for real this time

agp: Fix a bug in intel_gtt_get()

mappable_base was not correctly initialized, leading to problems on
some GPU models down the line

revoke(2): Use unix socket externalize code to handle revoke.

Use revoke token in shared mode in unix socket code and in exclusive
mode in fdrevoke(); mainly to make sure that all fps externalized and
marked FREVOKED will be picked up by later allproc_scan() called by
fdrevoke().

This one greatly simplies the code on unix socket side.  The original
unix socket revoke handling was also kinda broken: it tried to hold
socket reception buffer token w/ all filedesc spin lock being held.

Go-Ahead-by: dillon@

kernel: Allow fhold() in allfiles_scan_exclusive()

Before this commit following race could happen, if fhold() is used
during allfiles_scan_exclusive():

      Thread1                      Thread2
         :                            :
         :                      fdrop(fp)
         :                      {
    Scan all fps                  if (f_count-- == 1) {
    (*) fhold(fp) [f_count++]         :
         :                          fo_close(fp);
    Access fp                       Unlink fp from filehead;
    fdrop(fp)                       free(fp);
         :                        }
         :                      }

This could cause use-after-free or double free (since f_count == 1
when fdrop() was called in Thread1, fp will be freed again).

We now handle f_count 1->0 transition specially: hold the filehead
spin lock then do the f_count 1->0 transition, if the f_count 1->0
transition succeeds, the fp is unlinked from the filehead.  This
prevents the above race from happening.  This solution is mainly
based on dillon@'s input.

sys/vfs/hammer: Fix cppcheck warning

Spotted-by: swildner

sbin/hammer: Cleanups

data->inode.obj_type (ot=%02x) should equal that of elm base.

sbin/hammer: Change hammer show's printf format for elm base

This commit changes printf format for struct hammer_base_elm
fields. It doesn't change information to be printed, but it
does change the order.

This commit better explains how B-Tree nodes are located in
the B-Tree by having all 5 keys lo:objid:rectype:key:tid in
the same line. Since now that hammer show supports these keys
as search directives, it's better to directly reflect those
of each elm to hammer show's output.

ot (obj_type) isn't one of the 5 keys to compare node elms,
so this could(should) go to the next line.

===== (A) before this commit
...
G------ ELM  8 L lo=00000001 obj=0000000101da3ef7 rt=11 key=4c94cade3f178000 ot=00
         tids=0000000104018590:0000000000000000 suboff=8000000020c78000 mirror=0000000104018590 *
...
G------ ELM  1 R lo=00000002 obj=0000000000000001 rt=15 key=0000000000020000 ot=01
       d tids=00000001007e0ff0:000000010403b9b0

===== (B) with this commit
...
G------ ELM  8 L lo=00000001 obj=0000000101da3ef7 rt=11 key=4c94cade3f178000 tid=0000000104018590
         del=0000000000000000 ot=00 suboff=8000000020c78000 mirror=0000000104018590 *
...
G------ ELM  1 R lo=00000002 obj=0000000000000001 rt=15 key=0000000000020000 tid=00000001007e0ff0
       d del=000000010403b9b0 ot=01

sbin/hammer: Refactor get_elm_flags()/print_btree_elm() [2/2]

Call get_elm_flags() from print_btree_elm(). The elm flag
is used and only used when printing elm. This makes things
clear without passing the flag from one function to another.

No functional changes.

sbin/hammer: Refactor get_elm_flags()/print_btree_elm() [1/2]

These two functions have way too many arguments and
some of them are redundant that they could be get rid of.

No functional changes.

sys/vfs/hammer: Adjust raw kprintfs using hkprintf variants

This is part3 of hkprintf related after the following two.
sys/vfs/hammer: Change hkprintf() to macro and add variants [2/2]
sys/vfs/hammer: Change hkprintf() to macro and add variants [1/2]

Above two commits have replaced the existing kprintf calls
using "HAMMER:" or "HAMMER(label)" or function name prefix
with hkprintf and newly added variants, which basically didn't
change actual output other than fixing wrong function names
to the right ones, etc.

This commit continues replacing remaining kprintfs to make
output more understandable than raw kprintf calls with no clue
that they're hammer related.

For example, an error message like
"BIGBLOCK UNDERFLOW\n"
or a debug message like
"rt %3u, xt %3u, tt %3u\n"
become more understanbale with "HAMMER:" prefix or the name
of the function.

This commit is based on the followings.
1. Use hdkprintf which is hkprintf variant with __func__ prefix
   if that kprintf call is used when vfs.hammer.debug_xxx is
   enabled. This implies the messages are only for debugging
   and those are usually better and more understandable with
   a function name prefix as mentioned above. Also this is
   what's been done mostly in the existing hammer code.
2. Use hkprintf which has "HAMMER:" prefix if that kprintf
   call is a regular hammer message that appears in regular
   filesystem operations such as
   "Formatting of valid HAMMER volume %s denied. Erase with dd!\n".
3. Use h[vm]kprintf which are hkprintf variants with hammer
   label prefix "HAMMER(label)" if that kprintf can safely
   access the label via vol or hmp pointer. Some kprintfs in
   hammer does this rather than just "HAMMER:" and this seems
   to be better, however this commit doesn't go far as to
   aggressively replace the existing ones with this because
   a caller safely dereferencing hmp or vol is a different
   topic from merely replacing.

sbin/hammer: Fix and adjust hammer show's matched character '*'

Remove unnecessary space before '*'.
Adjust position of '*' for matched record.

sbin/hammer: Fix and change hammer show behavior when only lo is used

913b6663 has expanded hammer show's B-Tree directive from
lo:objid
to
lo:objid:rectype:key:tid
which allowed hammer show to filter B-Tree iteration in the
same way that the actual filesystem does on B-Tree lookup.

There was an exception in 913b6663 where one could not just
use lo directive, but was forced to use at least the first
two directives lo:objid. If objid wasn't specified then
HAMMER_MIN_OBJID was used for objid by default.

This was to keep compatibility with old behavior before
913b6663 added the rest of the directives (rectype:key:tid),
but this old behavior wasn't really useful for anything and
rather confusing given that all unspecified fields (after
the last specified field) except for objid were wildcard.
See sbin/hammer/hammer.c and sbin/hammer/cmd_show.c before
913b6663 (or really old one like v4.0 era) for details.

This commit normalizes this exceptional behavior and make
all option directives behave the same way. For example, if
*only* lo field is specified, then the rest of the fields
are wildcard. If the first n fields are specified, then the
remaining (5-n) fields are wildcard.

This does change the behavior when *only* lo is specified,
but the change should make the behavior less confusing.

Also update hammer(8) manpage since it hasn't been updated.

sbin/hammer: Cleanup search directive structure

Remove btree elm fields from hammer show's search directive
struct btree_search, and embed struct hammer_base_elm.

hammer show originally only had localization:obj_id fields for
search directives, so struct btree_search had these two fields,
but since 913b6663 has added (quite long time ago) all the
remaining fields rec_type:key:create_tid, struct btree_search
could simply embed struct hammer_base_elm instead of having 5
fields separately, which makes code clear.

sbin/hammer: Fix wrong next element boundary test

When elm is the current node element to be tested by
test_btree_out_of_range() by hammer show,

1. it's out of range if ((elm + 1) == search),
   if all optional search directives are used.
2. it may not be out or range if ((elm + 1) == search),
   if not all optional search directives are used.

Unused search directives are wildcard, so elm could still
have a subtree with records that meet the search condition
in above case 2.

This existed before test_btree_out_of_range() was added by
the previous commit, and because it's a different topic,
this commit is kept apart from the previous one without
squashed into one.

sbin/hammer: Add B-Tree helper functions and cleanups

and some cleanups that make no functional changes.

This makes B-Tree search/filtering code using optional search
directives much more readable. Also this makes it easier to
do fix in the next commit (fix wrong next elm boundary test).

sbin/hammer: Make B-Tree walk a bit more effective

hammer show recursively calls print_btree_node() when node
type is internal. If the node is not internal (leaf), it just
prints records with no further B-Tree walk.

Having
node->type == HAMMER_BTREE_TYPE_INTERNAL
check outside the
for (i = 0; i < node->count; ++i) {
is more effective since it doesn't go through unnecessary
for loop (with max count=63) if the node isn't internal.

Also note this looks more sane in the sense that node
contains elms[] and node->type determines union type of elms,
instead of node->type depends on each elms[i].

sbin/hammer: Fix minor memory leak

sbin/newfs_hammer: Adjust output of -E

This commit adjusts and aligns output of -E option of
newfs_hammer with the previous line.

It should use vol->type instead of typing "Device" in
lowercase, since this is what the vol->type is really
made for.

===== before this commit
  # newfs_hammer -E -L TEST /dev/da1 /dev/da2 /dev/da3
  Volume 0 DEVICE /dev/da1        size 111.79GB
  Trimming Device:/dev/da1, sectors (0 -234441648)
  Volume 1 DEVICE /dev/da2        size 111.79GB
  Trimming Device:/dev/da2, sectors (0 -234441648)
  Volume 2 DEVICE /dev/da3        size 111.79GB
  Trimming Device:/dev/da3, sectors (0 -234441648)
  ...

===== with this commit, looks more sane
  # newfs_hammer -E -L TEST /dev/da1 /dev/da2 /dev/da3
  Volume 0 DEVICE /dev/da1        size 111.79GB
  Trimming DEVICE /dev/da1, sectors (0 -234441648)
  Volume 1 DEVICE /dev/da2        size 111.79GB
  Trimming DEVICE /dev/da2, sectors (0 -234441648)
  Volume 2 DEVICE /dev/da3        size 111.79GB
  Trimming DEVICE /dev/da3, sectors (0 -234441648)
  ...

sbin/newfs_hammer: Remove global variable Eflag

Eflag could be a local variable after ff8644cb merged
two check_volume() into one.

sys/vfs/hammer: Remove ambiguous comment

Upper 16 bits of val[1] is always 0 (which equals pfs id of pfs0)
regardless of how many pfs there are.

sys/vfs/hammer: Fix wrong comment on ambiguous variable name

hammer_volume_ondisk::vol_name[] is a filesystem label string
specified by newfs_hammer via -L option (see below), but not
the name of the volume like "/dev/da1".

  # grep "ondisk->vol_name" sbin/newfs_hammer -rI
  sbin/newfs_hammer/newfs_hammer.c:       \
  snprintf(ondisk->vol_name, sizeof(ondisk->vol_name), "%s", label);

sys/vfs/hammer: Fix wrong comment on B-Tree node count

cursor->node->base->count
was right in the early stage of hammer devel,
but it has changed to
cursor->node->ondisk->count

hammer: Print "B-Tree" (conform to hammer specification)

Hammer specification, manpage, comments, etc mostly use
"B-Tree" instead of "btree" or "b-tree".

This commit replaces "btree" and "b-tree" strings that appear
in printf() and hammer(8) with "B-Tree", assuming no user or
userspace triggers anything by these strings.

hammer: Cleanup hammer(8) manpage

hammer(8) uses "PFS" but not "pfs" unless "pfs" appears in a path.

hammer: Update hammer(5) manpage

Update links to publication by
dillon@ in 2008
mneumann@ in 2010

sbin/mount_hammer: Minor fixes and cleanups

sbin/mount_hammer: Add CFLAGS+= -I${.CURDIR}/../../sys

just like sbin/hammer and sbin/newfs_hammer.

sbin/hammer: Remove -DALIST_NO_DEBUG

-DALIST_NO_DEBUG is no longer necessary.

sys/kern/subr_alist.c was only used in the early stage
of hammer devel, and no longer used.

  # grep ALIST_NO_DEBUG . -rI
  ./sbin/newfs_hammer/Makefile:CFLAGS+= -I${.CURDIR}/../../sys -I${.CURDIR}/../hammer -DALIST_NO_DEBUG
  ./sbin/hammer/Makefile:CFLAGS+= -I${.CURDIR}/../../sys -DALIST_NO_DEBUG
  ./sys/kern/subr_alist.c:#ifndef ALIST_NO_DEBUG

sbin/newfs_hammer: Remove a local header newfs_hammer.h

newfs_hammer.c is the only file that needs newfs_hammer.h,
and all it does is include other headers, so we really
don't need to have this as a separate header.

Also note that mount_hammer, newfs_hammer2, mount_hammer2
don't have .h as well.

sys/vfs/hammer: Fix incomplete mountctl(2) vop behavior

mount(8) is unable to print nomirror mount option while other
two (nohistory, master=) are properly printed. This is because
hammer's mountctl and mount flags aren't implemented the way
vfs expects them to be. mount(8) generates option strings by
mountctl(2).

===== missing nomirror string
  # mount_hammer -o nohistory -o master=5 -o nomirror /dev/da1:/dev/da2:/dev/da3 /HAMMER
  # mount | grep /HAMMER
  TEST on /HAMMER (hammer, local, nohistory, master)

===== using this commit
  # mount_hammer -o nohistory -o master=5 -o nomirror /dev/da1:/dev/da2:/dev/da3 /HAMMER
  # mount | grep /HAMMER
  TEST on /HAMMER (hammer, local, nohistory, master, nomirror)

This commit adds HMNT_NOMIRROR using a reserved HMNT_RESERVED
and assigns HMNT_NOMIRROR to nomirror. This is necessary since
two options (master=, nomirror) using the same HMNT_MASTERID
is the root cause of above behavior.

This change would affect userspace in theory, however the only
userspace that would actually use these mount option flags is
mount command itself (for inbox userspace programs at least).
It's more important that hammer properly handles mount(8).

This commit also adds a missing nomirror mountctl element in
hammer_vop_mountctl() using HMNT_NOMIRROR so mountctl becomes
aware of nomirror option.

The reason nomirror needs to use HMNT_NOMIRROR instead of
HMNT_MASTERID is because vfs_flagstostr() assumes each option
has an unique flag (see vfs_flagstostr() where it does the
following).
  flags &= ~optp->o_opt;

This could have been fixed by changing above vfs code, but
fixing hammer code is probably the right approach. Also note
that master= and nomirror options aren't alias of each other.
These two are similar but they do have different purpose, which
should make sense to have independent flags, and that's what
vfs expects.

sys/vfs/hammer: Rename HMNT_EXPORTREQ to HMNT_RESERVED

This was added in 513ca7d7 for NFS support but never used.
Change it (0x00000004) to HMNT_RESERVED.

sys/vfs/hammer: Conform to style(9)

Remove unnecessary {} for a switch case element.

sys/vfs/hammer: Add hpanic() [2/2]

Apply hpanic() to remaining several raw panic() calls.

sys/vfs/hammer: Add hpanic() [1/2]

This commit does the following.

1. Add a macro hpanic() which is a wrapper for panic()
   that embed the following prefix.
   "function_name: ..."
2. Replace raw panic() calls that have above prefix with
   hpanic().
3. Fix some wrong function name literals using __func__.

sys/vfs/hammer: Change hkprintf() to macro and add variants [2/2]

This commit does the following.

1. Add macros hdkprintf() and hdkrateprintf() that embed
   the following prefix.
   "function_name: ..."
2. Replace raw kprintf() calls that have above prefix with
   newly added macros.
3. Fix some wrong function name literals using __func__.

sys/vfs/hammer: Change hkprintf() to macro and add variants [1/2]

This commit does the following.

1. Remove an inline function hkprintf().
2. Add a macro version of hkprintf() and its variants that
   embed the following prefix.
   "HAMMER: ..."
   "HAMMER(label) ..."
3. Replace raw kprintf() calls that have above prefix with
   newly added macros.

hkprintf() macro doesn't use hammer_debug_debug flag that was
used by the inline function version of hkprintf(), but this
doesn't make much difference from the way hkprintf() and
hammer_debug_debug have been used.

sbin/mount_hammer: Remove typedef ary_ptr_t

It makes code more clear and readable without this typedef
which is only used for the first arg of extract_volumes().

info.volumes is char**, and an array internally used within
the function is char**, then using char*** for the first arg
is better than using typedef for char**.

sbin/hammer: Make hammer history support per-path @offset,length

This commit allows each arg of the hammer history to take
@offset,length attribute. It was strange behavior that the
command takes a global @offset,length attribute while the
command supports multiple paths to lookup history for.

This commit doesn't break the existing behavior where one
specifies @offset,length attribute as a part of the command
name as "history@offset,length".

If the old style attribute (history@offset,length) is used,
then the command doesn't parse per-path attribute, but sees
the whole string as a path. Otherwise the command parses
per-path attribute and uses offset,length unique to each,
unless the whole string including the attribute part exists
as a path (can stat(2)).

=====
  # touch /HAMMER/out1
  # touch /HAMMER/out2
  # for x in {1..5}; do
  > cat dragonfly/sys/vfs/hammer/hammer.h >> /HAMMER/out1
  > sync; sync; sync; sync; sleep 1
  > done
  # for x in {1..5}; do
  > cat dragonfly/sys/vfs/hammer/hammer.h >> /HAMMER/out2
  > sync; sync; sync; sync; sleep 1
  > done
  # hammer history /HAMMER/out1 /HAMMER/out2
  /HAMMER/out1    0000000184acac5e clean {
      0000000184b59b00 15-Sep-2015 03:56:46
      0000000184b59b20 15-Sep-2015 03:56:47
      0000000184b59c80 15-Sep-2015 03:56:48
      0000000184b59dc0 15-Sep-2015 03:56:49
      0000000184b59f20 15-Sep-2015 03:56:50
      0000000184b5a060 15-Sep-2015 03:56:51
  }
  /HAMMER/out2    0000000184acaca6 clean {
      0000000184b59b00 15-Sep-2015 03:56:46
      0000000184b5a1c0 15-Sep-2015 03:57:02
      0000000184b5a320 15-Sep-2015 03:57:03
      0000000184b5a460 15-Sep-2015 03:57:04
      0000000184b5a5c0 15-Sep-2015 03:57:05
      0000000184b5a700 15-Sep-2015 03:57:06
  }
  # hammer history@50000,1000 /HAMMER/out1 /HAMMER/out2
  /HAMMER/out1    0000000184acac5e clean {
      0000000184b59b20 15-Sep-2015 03:56:47
      0000000184b59c80 15-Sep-2015 03:56:48
  }
  /HAMMER/out2    0000000184acaca6 clean {
      0000000184b5a1c0 15-Sep-2015 03:57:02
      0000000184b5a320 15-Sep-2015 03:57:03
  }
  # hammer history /HAMMER/out1@50000,1000 /HAMMER/out2@50000,1000
  /HAMMER/out1    0000000184acac5e clean {
      0000000184b59b20 15-Sep-2015 03:56:47
      0000000184b59c80 15-Sep-2015 03:56:48
  }
  /HAMMER/out2    0000000184acaca6 clean {
      0000000184b5a1c0 15-Sep-2015 03:57:02
      0000000184b5a320 15-Sep-2015 03:57:03
  }
  # hammer history /HAMMER/out1@50000,1000 /HAMMER/out2@200000,1000
  /HAMMER/out1    0000000184acac5e clean {
      0000000184b59b20 15-Sep-2015 03:56:47
      0000000184b59c80 15-Sep-2015 03:56:48
  }
  /HAMMER/out2    0000000184acaca6 clean {
      0000000184b5a5c0 15-Sep-2015 03:57:05
  }
  # hammer history@50000,1000 /HAMMER/out1@50000,1000 /HAMMER/out2@200000,1000
  /HAMMER/out1@50000,1000 No such file or directory
  /HAMMER/out2@200000,1000        No such file or directory
  # touch /HAMMER/out1@50000,1000
  # sync; sync; sync; sync; sleep 1
  # touch /HAMMER/out2@200000,1000
  # sync; sync; sync; sync; sleep 1
  # hammer history /HAMMER/out1@50000,1000 /HAMMER/out2@200000,1000
  /HAMMER/out1@50000,1000 0000000184acaca7 clean {
      0000000184b6ae20 15-Sep-2015 04:05:09
  }
  /HAMMER/out2@200000,1000        0000000184acae20 clean {
      0000000184b6b180 15-Sep-2015 04:06:11
  }

sbin/hammer: Remove obsolete option -s

-s was added in da804f11 in the early stage of hammer devel
but no longer used. Using -s does nothing after b5aaba7f.

(b5aaba7f did remove -s from hammer(8) manpage)

sbin/hammer: Remove prototype of not existing function

sbin/hammer: Sync hammer usage() with hammer(8) manpage

A function usage() in sbin/hammer/hammer.c is missing several
options that exist in hammer(8) manpage. Add missing ones and
sync with manpage.

sbin/hammer: Update hammer usage()

Add one line that was missing in 3f760d89.

sbin/hammer: Make a global variable static

sbin/hammer: Separate read/write inode ops on hammer *stats

It should show read and write inode ops separately since
the original hammer sysctl accounts have them separately
(just like all other fields printed by *stats command),
and it's trivial to add these two to get total inode ops.

sbin/hammer: Cleanup hammer *stats commands

sys/vfs/hammer: Fix error message when volumes are already mounted

It should show volumes are mounted instead of "No root volume"
when the volumes are hammer volumes and failed with EBUSY.

  # mount_hammer /dev/da1:/dev/da2:/dev/da3 /HAMMER
  # mount_hammer /dev/da1:/dev/da2:/dev/da3 /HAMMER
  mount_hammer: Unknown error: Device busy
  # dmesg | tail -1
  hammer_mount: The volumes are probably mounted

sys/vfs/hammer: Add HAMMER_DIR_INODE_LOCALIZATION()

Move ondisk definition part of hammer_dir_localization()
to sys/vfs/hammer/hammer_disk.h.

It's better to separate kernel fs code from ondisk spec
(and that's what hammer.h and hammer_disk.h are made for).

sys/vfs/hammer: Remove #if 1 that no longer has any meaning

sys/vfs/hammer: Remove obsolete macro HAMMER_HEAD|TAIL_ONDISK_SIZE

These macros are no longer used and not necessary.

These are only used in the early stage of hammer devel and
macro values do not reflect hammer implementation today.

sbin/hammer: Fix comment on how to compile sbin/hammer/test_dupkey.c

This comment was never updated after icrc32.c was added.

sbin/hammer: Cleanup header includes regarding hammer_util.h

sbin/hammer: Cleanup header includes regarding hammer.h

sys/vfs/hammer: Fix possible race on volume-del

hmp->volume_to_remove should get reset to -1 only after the
volume is successfully removed. Otherwise there is a possible
race between this and blockmap allocation code who checks if
hmp->volume_to_remove is the volume being removed.

Spotted-by: dillon

sys/vfs/hammer: Remove unnecessary header includes

sys/vfs/hammer: Remove header includes from hammer.h

Remove
 #include <sys/buf2.h>
 #include <sys/mountctl.h>
 #include <sys/globaldata.h>
 #include <vm/vm_page2.h>
from sys/vfs/hammer/hammer.h

sys/vfs/hammer: Add header includes to hammer.h

Add
 #include <sys/fcntl.h>
 #include <sys/dirent.h>
 #include <sys/sysctl.h>
 #include <sys/event.h>
 #include <sys/file.h>
 #include <vm/swap_pager.h>
to sys/vfs/hammer/hammer.h

sys/vfs/hammer: Add hammer_data_zone(_index) macros

since now that all hammer code selects data zone to
allocate blockmap space based on the following basic
rule without an exception (that is actually the same).

zone = (data_len >= 16KB ? LARGE_DATA : SMALL_DATA);

sys/vfs/hammer: Remove exceptional zone selection case

hammer_alloc_data() has an unusual and exceptional way to
determine data zone (i.e. LARGE_DATA or SMALL_DATA) as shown
in below (B). This commit changes that to the normal way of
selecting the data zone to use as shown in (A). It's simply
better not to use an exceptional rule when that doesn't
make any difference as explained below. Also see 901e04c7.

=====(A)
Hammer is designed to use LARGE_DATA zone for >=16KB
length data, and SMALL_DATA zone for <16KB length data.
zone = (data_len >= 16KB ? ZONE_LARGE_DATA_INDEX : ZONE_SMALL_DATA_INDEX);
=====

=====(B)
An exceptional case that doesn't follow an above rule.
zone = (data_len > 8KB ? ZONE_LARGE_DATA_INDEX : ZONE_SMALL_DATA_INDEX);
=====

hammer_alloc_data() using (B) actually doesn't make any
difference because strategy write has already rounded up the
data size to hammer's buf size if data_len is >8KB as shown
below, by the time hammer mirror write code (which is the only
caller with a record type of HAMMER_RECTYPE_DATA) allocates
blockmap space to write the original-rounded-up src data to
the dst fs.

hammer_vop_strategy_write()
> if (bio->bio_offset || ip->ino_data.size > HAMMER_HBUFSIZE)
> bytes = bp->b_bufsize;

In other words, data_len is never going to be
(data_len > 8KB && data_len < 16KB)
when hammer_alloc_data() is called with the record type
of HAMMER_RECTYPE_DATA. If that's the case (A) just works.

sys/vfs/hammer: Remove prototype of not existing function

sys/vfs/hammer: Remove HAMMER_BUFFER_BITS

This macro is only used by compile time sanity check,
and the check is way too obvious that it can be get rid of.

Hammer code used to use this macro to right-shift offset
(>>14 had meaning in the early stage of hammer devel),
and it made sense to have the compile time sanity check,
but not really when having a way too obvious check is
the only use case.

sys/vfs/hammer: Add HAMMER_HBUFSIZE

Add HAMMER_HBUFSIZE which is 1/2 of HAMMER_BUFSIZE.

8KB (HAMMER_BUFSIZE / 2) is somewhat important number as
vfs opses use this value to round up a size to allocate
within blockmap address space.

sys/vfs/hammer: Adjust and cleanup _KERNEL

Remove no longer necessary _KERNEL that was originally
added for usr.bin/fstat/hammer.c to include
<vfs/hammer/hammer.h> by 872a7eee.

<vfs/hammer/hammer.h> isn't supposed to be necessary by
userspace except for a few (actually only above file)
unusual userspace programs that want to access in-kernel
data structures (like incore inode), so it's better not
to pollute <vfs/hammer/hammer.h> with bunch of _KERNEL
ifdefs in the first place if possible.

Also see 23e66b3b and 1caa2035.

sys/vm: Adjust position to define _KERNEL

Do the same as what sys/sys/buf2.h does.

Userspace can't include <sys/spinlock2.h> anyway, so this
commit doesn't make much difference. It's either including
<sys/vm/vm_page2.h> does nothing or results compile time
error.

sbin/hammer: Add ifndef/define/endif for headers

Some headers are missing this, so add it to those.

sys/vfs/hammer: Add ifndef/define/endif for headers

Some headers are missing this, so add it to those.

sys/vfs/hammer: Erase btype of a new boundary on internal split

It's more sane to erase btype of RBN of the original node
on internal split than only erasing subtree_offset.
(subtree_offset==0 means the elm has no child, but btype
not being 0 means its child is either leaf or internal node
which is insane if not a bug)

This doesn't affect comparison of nodes because btype is
not a part of keys to determine > or = or < of nodes.

This also makes possible to write non-ambiguous test code
in userspace as it eliminates multiple possible ways that
test code needs to deal with and be more explicit, which is
considered a good thing if the kernel change doesn't degrade
performance, etc. Also see 6580f642.

libc/sysctl: Fix the user.* branch.

This is a combination of FreeBSD's r285188 and more work by myself.
After r285188, FreeBSD still has issues with most user.posix2_* values
(due to a leading _ missing in the define names; they also need to be
checked for being > 0).

Partially-taken-from: FreeBSD

getconf(1): Add _POSIX2_PBS*.

<unistd.h>: Clean up the _POSIX2_* defines (Shells and Utilities options).

* Bump the _POSIX2_C_BIND, _POSIX2_FORT_RUN and _POSIX2_UPE defines to
  200809.

* Define _POSIX2_C_DEV, _POSIX2_LOCALEDEF and _POSIX2_SW_DEV to 200809.

* Improve comments.

Things I know of that we still have to do:

* For 100% _POSIX2_UPE conformance we still need to implement at least
  sh(1)'s HISTFILE and MAILCHECK environment variables.

* For 100% _POSIX2_SW_DEV conformance we still need to implement at
  least make(1)'s -p option.

revoke(2): Replace 'count' w/ a boolean 'found'

rtadvctl.8: Use Mt for email addresses.

kernel: Handle zero-length ELF sections better

FreeBSD simply skips all zero-length ELF sections when scanning an
REL type file.  The first time we tried it, DF panicked with a lost
bigs message.  It turns out this was because the skip should have been
applied in 3 places and we missed the first one.

The same approach is needed by the boot loader as well.

Bad commits always come in threes (copied lined instead of moving it)

Fix mis-paste (move section down 1 line)

kernel: Fix loading of gold-linked modules

Kernel modules linked by gold differed from those modules linked by ld.bfd in that
the .note.GNU-stack section on ld.bfd were set as size "1" whereas on gold this section
was size "0".  The object loading code was not set handle sizes less than 1, causing
module loading failure.

The fix of ensuring the smallest alignment to be zero by checking for -1 is required
over FreeBSD's technique of continuing if the section size is zero because an internal
check causes a "lost progbits" panic when FreeBSD's method is used.

This adjustment should enable a fully functional world and kernel to be built by gold
linker using WORLD_LDVER=ld.gold in make.conf.  Before this, only the world worked;
the kernel could not be booted.

Approved by: dillon

rtadvd - Import newer rtadvd, add rtadvctl from FreeBSD-10

* This patch updates the rtvdvd daemon and adds the rtadvctl
  utility to the base system. Both of these are from
  FreeBSD-10.2-RELEASE.  The intent is to add RFC 6106 functionality
  (DNS information distributed via router advertisements) to the
  system

Submitted-by: Charles Musser <cmusser@sonic.net>

kernel/vga: Remove a no longer needed variable.

It was used until 4e193d20af and I oversaw that it could be removed.

i386 removal, part 43/x: Remove/change i386 specific info in manual pages.

iwm - DragonFly cleanup (no functional changes)

* Cleanup and document iwm work.

* Remove test fences that aren't needed.

uipc: Rework unp_pcblist() to make it MPSAFE

uipc: Make unp_discard MPSAFE by deferring closef to a dedicated taskqueue

Add iwm.4 and iwmfw.4 manual pages (taken from FreeBSD).

i915kms.4: Add missing .El

kernel: Make iwm compilable into the kernel and add it to LINT64.

iwm - DragonFly modifications and a bug fix

* Modifications to make iwm build for DragonFly.

* Fix a serious bug, iwm_stop_device() must clear the rxq stat
  data primarily to reset to hw rx ring index to 0.  Otherwise
  all packets already handled from 0 to the current hw rx ring index
  will be reprocessed on restart.

* Implement iwmsleep() instead of using msleep().  iwmsleep() will
  handle exiting and re-entering the wlan_serializer.  Without this
  we can easily deadlock on the double lock.

* Implement Makefile infrastructure.

* Revert the FreeBSD patch to remove the per-debvice netif, since our
  802.11 infrastructure has not yet been updated for that.

* Purge the ifq on start if the interface is not running.

* Misc other DragonFly adjustments.

wlan - Import if_iwm from FreeBSD (base)

wlan - Minor updates for if_iwm import.

* Add PCIM_STATUS_INTxSTATE

* Expose ieee80211broadcastaddr[]

i386 removal, part 42/x: Remove i386 info from the syscall(9) manpage.

Remove libcom_err.so{,.3} via 'make upgrade' too.

kernel/dup: Correct dup{,2}()'s fd params to int in the syscall glue.

uipc: Allow fdalloc() to fail in unp_externalize()

While I'm here, add comment about never reached code.

uipc: White space

kernel: Use fhold() instead of increasing f_count manually