inpcb: Don't rely on ncpus2 for local port allocation.

sbin/hammer: Make some readonly pointer args const

sbin/*hammer: Use typedef'd for struct zone_stat*

This is a part of series of commits started from 21e9e7d5ff in 2016.
This is for userspace.

Some ugliness in const pointer in hammer_print_zone_stat()
(but note that HAMMER originally had bunch of pointer typedefs).

sbin/*hammer: Use typedef'd for struct cache_info*

This is a part of series of commits started from 21e9e7d5ff in 2016.
This is for userspace.

sbin/*hammer: Use typedef'd for struct buffer_info*

This is a part of series of commits started from 21e9e7d5ff in 2016.
This is for userspace.

sbin/*hammer: Use typedef'd for struct volume_info*

This is a part of series of commits started from 21e9e7d5ff in 2016.
This is for userspace.

sbin/hammer: Cleanup prototypes

sbin/hammer: Add /* not reached */ (missing ones in 052fd72b)

Some comments for errx(3) were missing in 052fd72b which was intended
to do this for all err(3)/errx(3) in sbin/hammer/*.c.

sound - Delete devfs clone handler for /dev/dsp and /dev/mixer on unload.

* This makes sound work again after unloading and then reloading the
  sound(4) module.

* Only add the devfs clone handler, when the preceeding make_dev() was
  successful.

Revert "kernel - Spread IPIs out to more priority levels"

This commit seems to be causing a performance regression on VMs.
Since it didn't fix a particular bug (was just supposed to make
things more robust), revert it for now.

Reported-by: tkusumi
This reverts commit f240042b9f9bcfdacd70499659fcc5f55d5e49ab.

drm/linux: Add reinit_completion()

kernel - Temporarily set the default machdep.pmap_nx_enable to 0

* Until we work out some issues, temporarily set the default
  machdep.pmap_nx_enable to 0.

* NOTE: This is a loader tunable only.

Revert "libc: Add temporary (get|set)affinity DF404.0 symbols"

This reverts commit 3c2537818c14aaba122a276020143cd4b92d2cbc.

As indicated in the original commit, the (get|set)affinity DF404.0
symbols were temporary and limited to the 4.7 development branch.
Now that 4.8 has been officially released, remove them as they
have served their purpose.  By now, all software built on 4.7
should have been rebuilt and thus nothing depends on the mistake
symbols.

sbin/hammer: Add /* not reached */ for usage() variants

that were originally missing.
Recommended by dillon@.
No diff in sbin/hammer binary when assert(3) is disabled.

sbin/hammer: Add /* not reached */

that were originally missing.
Recommended by dillon@.
No diff in sbin/hammer binary when assert(3) is disabled.

sbin/hammer: Check S_ISREG() if not block device

ioctl(DIOCGPART) failure doesn't mean path is a regfile.

sbin/hammer: Add is_regfile() (avoid directly using a string literal)

ppp - Fix minor overflow in protocol search

* Fix a minor redundant expression in sys/net/libalias/alias_irc.c

* Fix a one-byte overflow in cdcontrol.  Not a security issue.

* Fix an overflow in the protocol search.  This can only overflow a
  static array by one element and is not likely able to cause a security
  issue.

Dragonfly-bugs: 3012 (dcb)

libtelnet - Fix improper statement construction (not a bug in the binary)

* libstand hammer1.c - remove extra format element in debugging printf.
  This code is not compiled by the build anyway.

* Fix statement construction in rsaencpwd.c.  The prior code generated
  the desired result (incrementing the ptr), but there was an extra
  ineffective indirection.  Removed the ineffective indirection.

Dragonfly-bugs: 3010 (dcb)

libdevstat - Limit sscanf field, fix redundant condition

* Limit a sscanf field to MAXHOSTNAMELEN - 1.

* Remove a redundant condition already tested by the first part of a
  conditional.

Dragonfly-bugs: 3009 (dcb)

libalias - Cleanup unused variable

* Cleanup an unused variable.

Dragonfly-bugs: 3008 (dcb)

openssh - Fix a broken assignment

* Fix a broken assignment.  NOTE: OpenSSH master already has this fix.

* Document dangerous sscanf()s, but note that it is not possible for a
  buffer overflow to occur in these cases.

Dragonfly-bugs: 3007 (dcb)

window - Fix Graphics capability enable test

* Fix Graphics capability enable test, code type-o, logical OR has to be AND.

Dragonfly-bugs: 3005 (dcb)

kernel - Fix event preset

* Enable all events was not enabling all events.  Fix the memset().

Dragonfly-bugs: 3004 (dcb)

mfiutil - Fix static buffer overflow

* Fix static buffer that is too small.  Use a dynamic asprintf()
  instead.

Dragonfly-bugs: 3002 (dcb)

pf: Don't assume # of network procecssing CPUs is power-of-2

Reviewed-by: dillon@

toeplitz: Add comment.

sbin/newfs_hammer: Add /* not reached */ for usage()

sbin/mount_hammer: Add /* not reached */

sbin/newfs_hammer: Add /* not reached */

sbin/hammer: Partly bring back removed braces from 52e2f1b5

based on comments from dillon@.

(dillon) three types, yah.  (1) multi-line condition in if(), (2) if TRUE or FALSE body spans more than one line, put braces around both
(dillon) (including if a comment is part of the body)

sbin/newfs_hammer: Partly bring back removed braces from 46689eca

based on comments from dillon@.

(dillon) three types, yah.  (1) multi-line condition in if(), (2) if TRUE or FALSE body spans more than one line, put braces around both
(dillon) (including if a comment is part of the body)

sbin/mount_hammer: Partly bring back removed braces from a90e74c5

based on comments from dillon@.

(dillon) three types, yah.  (1) multi-line condition in if(), (2) if TRUE or FALSE body spans more than one line, put braces around both
(dillon) (including if a comment is part of the body)

netisr: Use netisr_ncpus to convert hash to cpuid.

And factor out netisr_hashlsb() for future use.

net: Reorder sysinit a bit.

sbin/hammer: Make info command print a proper error message

Printing a function name of some library makes no sense.

sbin/hammer: Make info command properly handle non HAMMER path

This shouldn't be needed if i/f of libhammer were carefully designed,
but this seems to be the only guaranteed way to detect non HAMMER path,
and print a proper error message that makes sense for UFS, etc.

-- before this commit
 # hammer info /boot
 hammer: libhammer_get_fsinfo     <--- ???

-- with this commit
 # hammer info /boot
 hammer: /boot is probably not a HAMMER filesystem: Inappropriate ioctl for device

sbin/mount_hammer: Cleanup blocks with a single statement

This commit basically does the same as what Linux kernel's coding
style mentions for braces, which is basically the same with BSDs.
https://github.com/torvalds/linux/blob/master/Documentation/process/coding-style.rst#3-placing-braces-and-spaces

No diff in sbin/mount_hammer/mount_hammer binary when assert(3)
is disabled.

sbin/newfs_hammer: Cleanup blocks with a single statement

This commit basically does the same as what Linux kernel's coding
style mentions for braces, which is basically the same with BSDs.
https://github.com/torvalds/linux/blob/master/Documentation/process/coding-style.rst#3-placing-braces-and-spaces

No diff in sbin/newfs_hammer/newfs_hammer binary when assert(3)
is disabled.

sbin/hammer: Cleanup blocks with a single statement

This commit basically does the same as what Linux kernel's coding
style mentions for braces, which is basically the same with BSDs.
https://github.com/torvalds/linux/blob/master/Documentation/process/coding-style.rst#3-placing-braces-and-spaces

No diff in sbin/hammer/hammer binary when assert(3) is disabled.

ifpoll: Use netisr_ncpus.

ifpoll: Use SYSINIT to initialize per-cpu ifpoll context.

sys/vfs/hammer: Add __hammer_crc_get_xxx() for CRC test retry

On retry, we don't need extra conditionals inlined (once or twice)
from calling hammer_datacrc(), since we know retry uses old crc.

Newly added inline functions are renamed versions of original ones
before 4c09d9c4.

sys/vfs/hammer: Cleanups on CRC functions

Just sync style with other part of the same file.

sys/vfs/hammer: Add comments on volume version 7 CRC (4c09d9c4)

There's no official documentation on this, but shouldn't be necessary
as CRC change does nothing directly to users.

inpcb: Use netisr_ncpus for listing inpcbs.

syncache: Use netisr_ncpus instead of ncpus2

syncache: Cache align per-cpu data.

ipdemux: Use netisr_ncpus instead of ncpus2.

inet6: Restore mbuf hash after defragmentation.

Reported-by: zach

ipfrag: Max # of frags per-cpu should use ncpus instead of ncpus2.

inet: Use netisr_ncpus for ifdetach inet processing.

Bump __DragonFly_version

There have been several signficant changes recently, including the
relocation of the signal trampline, a new KERN_PROC_SIGTRAMP sysctl,
the implementation of the NX (no-execute) pmap bit and the
machdep.pmap_nx_enable TUNABLE.

arp: Group llinfo and timer together.

arp: Ignore ARP probes by default.

ARP probes are widely used by Cisco switches.

arp: ARP entry list should be initialized on all available CPUs.

drm: Include opt_drm.h so that 'options VGA_SWITCHEROO' will be seen by vga_switcheroo.h.

net: Make netisr_ncpus sysctl read-only.

net: Define # of cpus for network processing.

Ease ncpus2 -> ncpus transition.

net: netisr_cpu -> netisr_threads; no functional changes.

net: Get rid of ifnet threads.

ipfw3: Reduce dependency on ifnet threads.

Prepare to get rid of them.

ipfw: Reduce dependency on ifnet threads.

Prepare to get rid of them.

bridge: Reduce dependency on ifnet threads.

Prepare to get rid of them.

vlan: Reduce dependency on ifnet threads.

Prepare to get rid of them.

ipflow: Reduce dependency on ifnet threads.

Prepare to get rid of them.

net: Reduce dependency on ifnet threads.

Prepare to get rid of them.

kernel - Implement NX (3) - cleanup

* Cleanup extra PROT_EXEC's.

kernel - Implement machdep.pmap_nx_enable TUNABLE

* Implement machdep.pmap_nx_enable TUNABLE.  You can set this to 0
  in /boot/loader.conf to disable the NX feature.

* Also conditionalize on the cpu feature bit being present.

bsd-family-tree: Sync with FreeBSD (DragonFly 4.8.0, macOS 10.12).

Really ensure that our world/kernel are built under POSIX locale ("C").

dedd9a18e4bbd4f206f9d7f4dba97b92a85e5d49 used LANG but that won't
override any LC_* that are already set.

kernel - Implement NX (2)

* Flesh out NX implementation for main kernel.

* Implement NX support for the vkernel.

kernel - Implement NX

* Implement the NX (no-execute) pmap bit.

* Shift sigtramp down to a page-bound and protect it prot|VM_PROT_EXECUTE.

* Map the rest of the user stack VM_PROT_READ|VM_PROT_WRITE without
  VM_PROT_EXECUTE.

if_vtnet - Add back vtnet_tx_size variable for now, to fix build.

sbin/hammer: Remove local variables only used within assert(3)

These are bothering compile with assert(3) disabled.
Assertions are still there, but local variables are removed.

libc/libpthread: Inject threadsafe locking callbacks for rtld.

* Make rtld thread-safe. Add lib/libthread_xu/thread/thr_rtld.c
  to the build and use it to inject pthread based locking callbacks
  for rtld.

* In rtld_lock.c, do the thread_mask_clear(lock->mask); after releasing
  the actual lock, to avoid deadlocks when rtld is called recursively
  from the pthread locking callbacks.
  XXX An alternative solution for this issue would be, to flag the
      libpthread dynamic library as bind_now to avoid the recursive rtld
      calls.

if_vtnet - Remove vtnet_{rx,tx}_size variables. Change rx_header padding.

* With a padding of 2 bytes, struct vtnet_rx_header will be 12 bytes now,
  which should be correct to ensure the 4 byte alignment of the frame
  payload, when mergeable Rx buffers are not negotiated.

virtio_blk - No need to negotiate unused (and legacy) VIRTIO_BLK_F_BARRIER.

if_vtnet - Default to link up, when VIRTIO_NET_F_STATUS is not negotiated.

virtio_blk - Clean up some unneeded functionalities.

* Get rid of remaining code for handling the B_ORDERED flag.
  B_ORDERED doesn't even exist anymore in DragonFly.

* Get rid of sc->vtblk_req_ready request list. We can just use
  bioqdisksort() to add a request - that we failed to enqueue ino the
  virtqueue - back into the bioq.

* Rename req->vbr_bp field to req->vbr_bio.

* Switch vtblk_req_free list from TAILQ to SLIST. A small optimization,
  none of the additional features of the TAILQ are needed here.

virtio: Replace one cpu_mfence() by cpu_lfence(), and one by cpu_sfence().

* The cpu_mfence() in virtqueue_dequeue() can be replaced by cpu_lfence(),
  since we only need to make sure that we read the vq->vq_ring.used->idx
  value before reading any other entries from the vq->vq_ring.used ring.

* The cpu_mfence() in vq_ring_update_avail() can be replaced by
  cpu_sfence(), since no one else is writing to the vq->vq_ring.avail
  struct.

virtio - Get rid of nop {vtblk/vtnet}_modevent methods.

kernel - Fix broken wakeup in crypto code

* Depending on which crypto driver is being used and the crypto load,
  the crp_sleep variable for the wrong tdinfo is tested to determine
  whether a wakeup is needed or not.  This can result in the crypto thread
  not being woken up when new work is pending.

* Can cause the crypto subsystem to stall indefinitely, in particular
  during a normal shutdown,

Reported-by: jh32

Fix some cases where an index was used before its limits check.

Dragonfly-bugs: 2998, 2999, 3000, 3001

libkvm: Fix char pointer dereference.

usr.sbin/Makefile: Fix sorting.

uefi.8: Leave the .Dd.

Partly revert "uefi.8: Extend SEE ALSO and fix HISTORY."

This partly reverts commit 2bd356f2144fabf57cc538aa4ebb0a70c1baacc9.

UEFI boot support as such was indeed added in 4.5. 4.7 only added
support to the installer.

uefi.8: Extend SEE ALSO and fix HISTORY.

efisetup.8: Some mdoc improvements.

efisetup(8): Fix permissions in the tree.

mdoc: Fix manlint warning in efisetup.8

<sys/libkern.h>: Fix whitespace/indent, remove arg names in prototypes.

<machine/stdint.h>: Check __cplusplus first.

Avoids -Wundef warnings when compiling C++ (which does not define
__STDC_VERSION__).

test/vkernel - Add MAKE_JOBS

nrelease - Show MAKE_JOBS in help

vkernel - Fix build

mixer - Fix sscanf() overflow

* Fix %s buffer sizes in sscanf().

Reported-by: dcb

gcore - fix overflow in sscanf

* Fix an overflow which can occur in the sscanf() used to parse
  /proc/*/map

Reported-by: dcb

camcontrol - non-critical fgetc() cleanup

* fgetc() returns int, check EOF against int return, not char.

Reported-by: dcb