Replace local array size calculations with NELEM().

Convert from __arysize to NELEM

Try to use a single macro across the board

Discussed-with: zrj, swildner

nfs.5: Adjust (c) year.

nfs.5: Remove unneeded .Pp

devfs.5: Document sysctl.

Submitted-by: zrj

ffs.5: Document more sysctls.

Move ffs sysctls to their own section.

Submitted-by: zrj

Add nfs(5) manpage.

Document all sysctls.

Submitted-by: zrj

Add vfs(7) manpage.

Document all sysctls except for sizeof stuct ones.

Submitted-by: zrj

kernel - Fix syscons's visual bell

* The visual bell code was completely broken.

* Refactor the callout, and use the new callout_init_lk facility
  to deal with races.

Reported-by: snurg

drm/linux: Update hlist_for_each_entry_safe() from FreeBSD

sbin/hammer: Fix compile warning on Linux

Warned by gcc 7.3.1 in Fedora.
Extend the size of destination string.
dirent::d_name[] is 256 bytes in Linux (and *BSD).

--
cmd_abort.c: In function 'hammer_cmd_abort_cleanup':
cmd_abort.c:73:44: warning: '%s' directive output may be truncated writing up to 255 bytes into a region of size 55 [-Wformat-truncation=]
    snprintf (pidfile, PIDFILE_BUFSIZE, "%s/%s",
                                            ^~
cmd_abort.c:73:4: note: 'snprintf' output between 10 and 265 bytes into a destination of size 64
    snprintf (pidfile, PIDFILE_BUFSIZE, "%s/%s",
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     pidfile_loc, den->d_name);
     ~~~~~~~~~~~~~~~~~~~~~~~~~

mille(6): minor nits

- adding missing header
- remove unused variable
- static where possible

Reviewed-by: zrj

gcc50: Switch backends to use internal lib framework.

Previously backends where implicitly using bsd.hostlib.mk that have few
unwanted side effects (backends are install targets). Avoid that.
No functional change for current master users.

Tested-with: WORLD_CCVER and _DEFAULT_CCVER combinations.

fish(6): Suppress -Wdiscarded-qualifiers warning.

/usr/Makefile: Fix typo.

correct prior commit...

larn(6): fix open call

Prior to this the ", 0666" was being treated like the comma operator
rather than an argument to open(2).

larn(6): remove HIDEBYLINK

The implementation of HIDEBYLINK is a bit wonky, doesn't really work,
and includes its own reimplementation of bzero.

fish(6): fix several issues

- avoid use of _ in application namespace
- use posix_spawnp instead of system()
- prefer less(1) over more(1)
- mark usage as dead
- staticfy

This world-class programs deserves world-class source code.

Update the pciconf(8) database.

February 15, 2018 snapshot from https://pci-ids.ucw.cz

<pthread.h>: Note that we are missing some functions in #if 0'd protos.

devd: only use correct argument in select(2)

Discussed-with: zrj
Reviewed-by: vangyzen@FreeBSD.org, ian@FreeBSD.org
Objected-to-by: imp@FreeBSD.org

select(2): mark as restrict in man page

This has already been done in code, so change the man page

Discussed-with: swildner

Add missing restrict qualifiers in POSIX function definitions/declarations.

This creates no further -Wrestrict warnings with gcc80 in buildworld.

<sys/cdefs.h>: Add __restrict_arr to declare arrays as non-overlapping.

Based-on: glibc

kernel - Vet b_cmd in swap_pager_strategy().

* swap_pager_strategy() needs to properly vet b_cmd.  This doesn't fix
  any bugs per-say, but is needed for robustness to ensure that the code
  does not accidently corrupt the swapcache in a later evolution.

kernel - Fix rare missed wakeup() in lockmgr

* Fix a rare missed wakeup() case in lockmgr.  lk_count can briefly
  become (SHARED | 0 counts | (EXREQ or UPREQ)).  Three competing cores
  can then cause a situation where undo_shreq() fails to issue a wakeup()
  to threads acquiring a shared lock that are blocked waiting for the
  EXREQ or UPREQ

  Issue the missing wakeup() for this case.

* This race arises because of an optimization we make when dropping a
  shared lock.  atomic_fcmpset*() loops are fairly poor at dealing
  with concurrent increments and decrements, so undo_shreq() (which
  is also used when releasing a standard shared lock) decrements the
  shared lock count first, then deals with EXREQ or UPREQ afterwords.

  Usually a shared lock request bumps the shared lock count before
  blocking, which other lock releases use to determine the need for a
  wakeup().  However, shared lock requests cannot bump the shared lock
  count when the lock is already held SHARED but there is an EXREQ or
  UPREQ pending, because doing do basically grants the shared lock
  immediately.

  This combination leads to the brief situation which allows other cpu
  cores to squeeze in operations of their own without realizing that
  someone might be blocked trying to obtain a shared lock, but with no
  shared count present to indicate so.  If the undo_shreq()'s later
  atomic_fcmpset*() calls then fail, or find that there is no EXREQ or
  UPREQ pending, it fails to issue the needed wakeup().

hammer2 - Add missing bkvasync() (2)

* Add another missing bkvasync().  This fixes a temporary (memory-only)
  corruption issue which crops up in synth tests.

kernel - Document bkvasync() requirements

* Document that bkvasync() needs to be callable from multiple
  cpus concurrently (aka H2 shared buffering).  The lock has to be
  controlled by the caller (owned by the caller in some way), but the
  owner does not have to be the calling cpu.

Bump __DragonFly_version after %b and %r formats removal.

ddb: Some minor adjustments.

kernel: Add security.ptr_restrict sysctl.

Allow to prevent leaking kernel addresses back to the userland
(intentionally or not). We have quite a lot debug prints already and by
default we allow unprivileged processes to read kernel message buffers.

Update porting_drivers.txt

world: Remove references to %r and %b.

While there, remove %r and %b and %y support from libstand(3).
Those format are not used in loaders and only eat space.

kernel: Remove last use cases of "%b" format.

Switch to args safe "%pb%i" internal format.

kernel/disk: Remove use of "%b" format.

Switch to args safe "%pb%i" internal format.

kernel/net: Remove use of "%b" format.

Switch to args safe "%pb%i" internal format.

Note: the if_nfe.c used 0x%4b format, while %b was not handling the width.

kernel: Remove use of "%b" format in main kernel code.

Switch to args safe "%pb%i" internal format.

kernel: Implement "%pb%i" as args safe "%b" replacement.

kernel: Nuke legacy FreeBSD "%r" format support.

Makes a quite a cleanup in subr_prf.c and less hits against things
like "%rip". Only one custom format remains - "%b".

kernel: Remove use of deprecated "%r" radix format.

kernel: Implement makedev_unit_b32()

It would be used to replace deprecated legacy FreeBSD "%r" formats.
Change tty_pty.c to use hex2ascii(), since it has only upto 32 devices.

ddb(4): Remove use of deprecated "%r" radix format.

Use internal helpers like for "%z" to print numbers.

Taken-from: NetBSD (with adaptations by swildner)

kernel - Try to fix delayed mtime updates on SHARED+RW maps again

* Attempt to fix continuing problems with mtime updates related to
  modifications made via SHARED+RW mmap()s.  As evidenced by builds
  under synth sometimes getting confused.

* Don't update vp->v_lastwrite_ts in vop_stdclose().  Instead, update
  the field and set a new flag VLASTWRITETS upon mmap(SHARED+PROT_WRITE)
  or upon mprotect(PROT_WRITE) on a SHARED mmap.

* Clear the flag on any regular write, utimes, or truncation.

* Adjust various filesystems to update mtime from vp->v_lastwrite_ts
  only upon UIO_NOCOPY writes, and only if VLASTWRITETS is set.

  tmpfs, ufs, hammer, hammer2 adjusted.

hammer2 - Add missing bkvasync()

* Add missing bkvasync() when a file read receives a chain from the
  XOP helper thread.

pthread.3: Add missing functions.

<signal.h>: Adjust #if 0'd prototype.

Change __restrict -> restrict in two manual pages.

sbin/hammer: Fix compile error on Linux

Failed with gcc 4.4.7 in CentOS 6.x.
--
cmd_recover.c:78: error: redefinition of typedef 'bigblock_t'
cmd_recover.c:55: note: previous declaration of 'bigblock_t' was here

sys/vfs/autofs: Remove AUTOFS_ASSERT_*LOCKED()

These macros came from FreeBSD, but more readable without these.

sys/vfs/autofs: Change autofs_mount::am_lock to use mutex(9)

sys/vfs/autofs: Change autofs_node::an_vnode_lock to use mutex(9)

hammer(8): Remove include guards in a .c file.

sys/vfs/autofs: Remove #if0'd export related code

Exporting is never going to be implemented.

sys/vfs/autofs: Cleanups

binutils227: Add support for DWARF-4 line number tables.

Backport from upstream trunk the c83344582375e65643c5efe68b3739e87db482f3

Work around the ld.gold issue when '.rela.debug_line' is populated.
We use ld.gold as default system linker that sigbus on DWARF-4 tables.
LDVER=ld.bfd seems to be unaffected, apply 2018-02-03 fix only for 227.
No functional change for default non-debug buildworlds (only for gcc80).

<unistd.h>: Remove commented out ctermid() prototype. It's not needed here.

etc/autofs: Cleanup trailing whitespaces

This generates diff from FreeBSD.

etc/autofs: Sync special_media with FreeBSD

usr.sbin/fstyp: Bring in exFAT support from FreeBSD

exFAT support was added by below FreeBSD@GitHub commit in 2017.
f72e57262fe11f1cc7ffc3b3000bc5747467b8c3

tmpfs - Fix data loss issues

* Fix a data loss issue that can occur when vnodes are cycled.  This
  can be replicated by setting kern.maxvnodes to a fairly low value.
  Vnode recycling does not necessarily issue a vinvalbuf() when clean
  pages are present, which tmpfs needs to retain.

* Fix a data loss issue when swap becomes full or when the system has
  no swap configured.  When tmpfs tries to flush to swap and fails,
  pages can remain associated with the vnode after the vinvalbuf()
  call, and then lost when the vnode is recycled.

* The above fixes are accomplished by call vinvalbuf() on vnode deactivation
  to make sure that all buffers have been flushed, and then moving any
  pages that remain to tn_aobj.

  The pages are moved back when the vnode is reinstantiated and an open(),
  read(), write(), or setattr (i.e.  truncation or extension) is called.

  We try to avoid moving the pages back if the vnode is merely *stat()d
  or deleted.

* Use cluster_read() by default to try to improve read throughput when
  pages are swap-backed.

Reported-by: zrj, marino

kernel - syntax

* Improve a formatting issue.

kernel - Fix two rare namecache bugs

* Fix calculations which use the vfscache_negs global.  This global
  is somewhat heuristical and can values which are a bit off, including
  0.  Copy to a local and limit the range, fixing a divide-by-zero
  bug and a negative-number handling bug.

* Fix a bug in the handling of a race in _cache_cleanneg().  We were
  unlocking the ncp but failing to drop it, leaving it with a ref.
  The accumulating namecache records prevent umount from succeeding.

  This race can only occur regularly when kern.maxvnodes is set to
  a low value.

if: Allow user to override software queue length.

if: Remove ifq maxlen fixup in ifinit.

It will not work, and no drivers in the base require this kind of fixup.

<fcntl.h>: Add #if 0'd prototypes for posix_fa{dvise,llocate}().

syscall.9: Explain better how errno and return value are set in userland.

In-discussion-with: peeterm, zrj

Use 'null' and 'NULL' instead of 'nil' in some manual pages.

kernel - Do not panic on media size of zero in disk_probe()

* Do not panic if the media size is zero in disk_probe().

kernel - Fix CAM peripheral error handling

* cam_periph.c was bcopy()ing the *ENTIRE* saved ccb back to the
  original when working through certain error results.  This completely
  destroys linked list entry fields.

* Refactor by hacking a restore_ccb() function which does not copy
  over the link list entry fields or the callout structure.

* Fixes panics with CDs, particularly audio CDs, and can also fix
  other panics.

  Panics are related to situations where an AHCI error or errors occur
  while multiple CCBs are queued.

* Also always initialize the state tracking field for various scsi
  periphals to ensure that stale data does not result in improper
  processing in scsi/*.c's *done() functions.

Reported-by: htse

ahci - Improve debug output

* TFES errors now also report the slot and ccb pointer

bsd.sys.mk: Don't exclude -Wformat-extra-args in FORMAT_AUDIT builds.

Not sure why this was originally added but we surely want to see them.

Reported-by: zrj

gdb/lvm: Fix two -Wformat-extra-args warnings.

The gdb one was taken from upstram and the lvm one is in __DragonFly__
specific code.

pthread_join.3: Add a missing header in the SYNOPSIS.

cpucontrol(8): Fix -e handling.

We do not have implemented CPUCTL_EVAL_CPU_FEATURES in cpuctl(4) yet.

Fix a few mdoc issues.

newsyslog.8: Comment out another zstd reference (and fix a typo).

callout.9: Remove some obsolete MLINKS.

[fish] fix style

[fish] don't allow users to request cards they have made books for

It is not technically legal to request a card you don't have in your
hand, even if you have the book for it. This really only matters if
you're playing with more than one deck, which fish(6) does not support,
but since this is a critical bug, fix it.

newsyslog(8): Add missing fallthrough.

Normalize libcrypto and libssl DPADD variable names and adjust Makefiles.

drill(1): Use strdup() for known strings.

libarchive: Use memcpy() when constructing buffers.

Avoid common mistakes that strncpy() will terminate the string.
Results here are not expected to be NULL terminated.

ee(1): Fix ispell_op().

Do not pass NULL to sprintf, use tempname returned by mkstemp(3).

truncate(1): Add support for T.

While there, add fallthrough.

tput(1): Constify.

While there, add fallthrough.

rs(1): Staticize.

While there, add few fallthrough.

jot(1): General cleanup.

  * staticize
  * constify
  * add fallthrough

ident(1): Use NULL instead of 0.

While there, add few cases of fallthrough (few need more investigation).

ctags(1): Add missing fallthrough.

The -v option implies the -x.

colcrt(1): Add missing fallthrough.

at(1): Add missing fallthrough.

While there, constify struct.

kgdb(1): Avoid -Wformat-truncation warning.

No functional change.

window(1): Add some format safety.

While there, add some FALLTHROUGH too.

netstat(1): Add some formats safety.

No functional change intended.

sort(1): Use asprintf(3) when fixing legacy options.

While there, make fix_obsolete_keys() static.

tzsetup(8): Suppress -Wformat-overflow warning.

systat(8): Add some formats safety.

Mainly to suppress -Wformat-overflow

sliplogin(8): Add some formats safety.

Use %hu for printing ipv4 address octects and reduce login name lenght a
bit to allow for prepending path.