games: Fix -Wint-in-bool-context warnings.

These cases were the last two in base.

binutils227: Fix comparison of pointer against char.

Backport from upstream.

binutils225: Fix comparison of pointer against char.

Backport from upstream.

While there, deal with -Wshift-overflow in aarch64.cc (unused).

gdb: Fix wrong comparison of pointer against char.

Backport from upstream.

Oops, fix up my previous commit properly.

Fix gcc80 -Wmultistatement-macros.

gcc50/Makefile.inc: Add newline at EOF.

Revert "terminfo: Add xterm-termite to Makefile.entries"

This reverts commit 5f4713d27c03fbf254d1d7e9939f9864261337b6.

gnu/sbin: Raise WARNS generally to 1 via Makefile.inc.

gcc50/libgcov: Fix whitespace in Makefile.

Revert "kernel: Disable attributes that take no effect."

This reverts commit 191430ddf0486f992e75ff14effbb5a69e1da2bd.

Needs more checking.

sbin/mount_autofs: Add description

TL;DR users won't be running this command to mount autofs.

Having autofs_enable=YES won't run this command either.
There is a high level command automount(8) and that does everything
based on /etc/auto_master contents.

mount_autofs(8) is a low level version where users can manually
specify things automount(8) internally passes to mount(2).
Thus it's not easy or needed to understand what -fOp options exactly
do without having clear understanding of autofs internal.
But I'll write it here in case interested.

 -f is a string which eventually gets copied to statfs::f_mntfromname.
  Unlike most of the filesystems with backing store, this is just
  a label to identify the autofs mount.
 -O is mount options of autofs(5).
 -p is a prefix string to distinct direct and indirect mapping.

Fix some -Wunused-const-variable warnings and a -Wmisleading-indentation.

dconschat(8): Put the fflush() under 'verbose' too.

dconschat(8): Fix -Wmisleading-indentation.

Fix one last -Wnonnull-compare in m4(1).

Fix gcc80 -Wnonnull-compare.

For arguments that are checked against NULL in the function, we don't
want a nonnull attribute, because that will cause the NULL check to be
optimized away in higher -O levels and it will also trigger a
-Wnonnull-compare warning.

See also 0f037c78525bfb7df49de8338826e15137859b1d commit message.

lib/i18n_module: Add a missing break.

It is intended and correct, but has no correctness implications due
to is94 => is96.

Confirmed-by: cem@freebsd, eadler@freebsd

rcs: Fix gcc80 warnings (-Wdangling-else and -Wmisleading-indentation).

hammer2: Remove duplicate initialialization of fields.

Avoids -Woverride-init diagnostics.

While there, minor whitespace cleanup.

kernel: Disable few nonnull attributes.

To avoid -Wnonnull-compare, similarly as in:
0f037c78525bfb7df49de8338826e15137859b1d

devfs_core.c:   devfs_find_device_by_name()
1st arg checked (NULL == return)
kern_slaballoc.c: kfree()
1st arg checked (NULL == panic)
2nd arg not checked
kern_synch.c:   tsleep()
1st arg checked (NULL == KASSERT)
subr_prf.c: kvcprintf()
1st arg checked (NULL == "(fmt null)")
subr_sbuf.c: sbuf_vprintf()
2nd arg checked (NULL == KASSERT)

kernel/bus: Avoid panic on NULL passed to kfree().

Our kfree() will issue panic() on NULL pointer.

Caught-by: gcc80 -Wnonnull

kernel/ipsec: Avoid -Warray-bounds warning.

Just panic if something else than 0, 1 or 2 is passed.

kernel/ipsec: Fix useless m_pkthdr.len check.

Compare it against sadb_msg.

Caught-by: gcc80 -Wtautological-compare

ig_hal: Reindent e1000_ich8lan.c a bit.

No functional change.

ipfw3_nat: Reindent switch cases.

No functional change.

While there, make add_redir_spool_cfg() and del_redir_spool_cfg() static.

kernel: Disable attributes that take no effect.

Both gcc50 and gcc80 ignore these attributes:
warning: ignoring attribute 'aligned' because it conflicts with attribute 'packed' [-Wattributes]
No differences for final objects.

netgraph: Disable unused structs.

This needs recheck.

While there, document that musycc and if_mn requires netgraph and
are not compatible with netgraph7.

usr.sbin/autofs: Drop unneeded header include

common.c has no dependency on autofs api itself.

Sync zoneinfo database with tzdata2018b from ftp://ftp.iana.org/tz/releases

* Sao Tome and Principe switched from +00 to +01.

* Brazil's DST will now start on November's first Sunday.

* A discrepancy of 4 s in timestamps before 1931 in South Sudan has
    been corrected.

* The abbreviation invented for Bolivia Summer Time (1931-2) is now
    BST instead of BOST.

* Various smaller adjustments.

For a detailed list of changes, see share/zoneinfo/NEWS.

limit(1): fix always true condition

drm/i915: Backport fix for a bad bitmask.

No-op, we explictly disable stolen memory in i915_gem_init_stolen().

cpdup - Additional fixes for -X absolute_path

* When an absolute path is specified the file is connsidered
  to be local to the machine and is not opened on a remote
  source.

hammer2 - Correct ip->cluster.nchains gap filler

* Fix a bug where gaps in the cluster array were not being
  properly filled.  This is only applicable to clusters
  consisting of more than one element, which we do not
  officially support yet.

Reported-by: zrj (gcc80 found it)

cpdup - Enhance -X operation

* Enhance -X operation.  When the cpignore file is specified as an
  absolute path, it may contain wildcards which match against the
  concatenated source path and directory element (as specified in the
  source), instead of just match against directory elements.

* Thus an absolute cpignore file may contain full paths and wildcards.
  For example, if you:

  cpdup -X $cwd/mycpignore /tmp /fubar

  And $cwd/mycpignore (specified with an absolute path) contains:

      /tmp/xyz/swap*

  Then cpdup will not copy files that match against /tmp/xyz/swap*.

  If you specify a relative path for the cpignore file, or use the
  -x option (instead of -X cpignorefile), operation remains as before
  and only matches against the directory element.

{bs,quiz}(6): Compare pointers with NULL, not '\0'.

drm/radeon: Disable few unused const arrays.

No functional change.

LINT64: Cleanup for -Wunused-const-variable.

sound/hda: Disable hdac_pcie_snoop struct.

It is currently unused.

kernel/mpt: Fix evaluation of args in macro.

kernel/ncr: Remove unused variable.

kernel/isp: Minor cleanup.

Remove unused string and use 'rqo' only if ISP_TARGET_MODE.

<sys/ktr.h>: Use empty macros for non KTR case.

Cleans up the gcc80 -Wunused-const-variable warnings.

kernel/pf: Remove an unused variable.

terminfo: Add xterm-termite to Makefile.entries

termcap: Add xterm-termite

drm/linux: Improve linux/log2.h

With newer GCC compilers it is no longer possible to easily detect bad
cases of ilog2() usage in drm shim, see:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72785

There is just a single use case of this macro in drm/radeon/radeon_device.c
for radeon_vm_size, it is already checked to be positive 2^N small number.
So simplify macro and just return 0 for n < 2.

While there, adjust logic to use proper fls(3) and flsll(3) functions
based on integer size (previously both cases where calling flsl(3)).

Fix few typos.

Spotted-by: swildner

limits(1): Add missing 'break'.

ktrdump(8): Fix wrongly placed 'break' for floating point unpacking.

gzip(1): Add some __attribute__((__noreturn__)).

pam_exec(8): Raise WARNS to 3.

fsdb(8): Fix some -Wformat warnings and raise WARNS to 2.

umount(8): Add missing braces.

mfiutil(8): Remove -fno-builtin-strftime from CFLAGS.

We don't need this.

Provide __{printf,scanf,strfmon,strftime}like() for builtin functions too.

We want our formats checked in -fno-builtin builds as well.

bsd.sys.mk: Set -Werror only for compilers that have the warnings fixed.

This reverts part of b399af1115fa4d2d29e5b5e59eb3a5c3087dccda.

bsd.{sys,kern}.mk: Apply warning restrictions for future GCCs.

Add __attribute__((__noreturn__)) to a number of usage() like functions.

These were not caught in the -Wsuggest-attribute=noreturn cleanup
because they were already static at that point. There are likely
more such cases but I don't know how to find them systematically.

ed(1): Add three __attribute__((__noreturn__)).

I had missed them in a31274954f5c10e5a80969dcfc719d665f36a26c.

pfctl(8): Fix two wrong conditions.

Caught by gcc80's -Wtautological-compare option.

libthread_xu: Fix issues detected by gcc80.

The changes to <pthread.h> all deal with -Wnonnull-compare warnings,
which are issued when a function has a NULL check for a parameter that
at the same time is marked with __attribute__((nonnull(...))), aka
__nonnull(...).

Such __nonnull() decoration only ever catches those cases where NULL is
passed directly to the function, but not any more indirect ways. What
makes matters worse is that in higher optimization levels (-O >= 2),
GCC will happily optimize out any NULL checks within the function for
these parameters.

This means that __nonnull() is generally inferior to a direct NULL check
in the function itself and if we have both, which newer GCCs warn about
with -Wnonnull-compare (that is part of -Wall), we should remove the
__nonnull().

This commit does that for all parameters which our libthread_xu checks
directly in the function. What remains are __nonnull() for parameters
which are _not_ checked in the functions, to at least catch cases where
NULL is passed directly. We should think about adding real checks for
those parameters too.

While here, add a "/* FALLTHROUGH */" comment in thr_printf.c which
fixes a -Wimplicit-fallthrough warning.

mfiutil(8): Use MAN instead of MAN8.

Revert "bsd.sys.mk: support -Wunused-local-typedef"

This reverts commit 14683bdb9c1d41c0d62dc7d11f3fd535973ad258.

It is no longer needed after 6954d52dca31dc7773729c0bf92e14fb1dab5aae.

gcc50: Move out manpage version handling.

The Makefile.cco should only affect global ones.

gcc47: Move out manpage version handling.

The Makefile.cco should only affect global ones.

corepower - Add kabylake support

* Add kabylake support (just the id needed)

etc/remote: Fix ucom[234].

kernel/evdev: Whitespace correction in input-event-codes.h

kernel/evdev: Update input-event-codes.h

Taken-from: Linux

ipfw: Add icmpcodes support.

kernel - Fix busdma allocation bug

* Fix a bug where kmalloc() could be used to allocate DMA memory
  that the caller intends to be non-cacheable.

  If a non-cacheable tag is specified, bus_dma now uses contigmalloc.

* Use the bus_dmamap_t pointer to differentiate between kmalloc
  and contigmalloc allocated memory, instead of running tag tests.

  This is a bit of a hack, but its the easiest solution.

kernel - xhci cleanup, lock attach

* Lock xhci during attach to avoid spurious interrupts from
  hitting the interrupt routine too early.

* Install interrupt after controller startup, but before
  the main usb code begins querying it.

* Cleanup syntax

kernel - Add a global priority hack for ACPI

* Allows ACPI to fully probe and attach all acpi devices before
  attaching any non-ACPI devices.

* May improve device attach stability.

* Currently only implemented at the top-level of the acpi
  device tree.

kernel - Handle ig4 controller startup errors

* Fail the attach on controller startup errors.  For some reason the
  dell xps 13 says there's I2C controller, but the controller appears
  to be permanente disabled and will refuse to enable.

cpucontrol - Add amd10h support

* Add support for ryzen and later microcodes.

* Adds general support for AMD microcodes in their
  packed format.  No need to unpack AMD microcode any
  more, cpucontrol will parse the distribution file
  from AMD.

Taken-from: FreeBSD

kernel: Add acpi and a number of related drivers to our default config.

These are: gpio_acpi, gpio_intel, smbus and smbacpi. Leave
ichiic (aka ig4) commented out for now because it seems to have
issues with latest generation Intel CPUs.

Adjust the manual page a bit and remove an obsolete comment in LINT64.

In-discussion-with: dillon, ivadasz

sbin/mount_autofs: Change include path from fs/ to vfs/

/usr/include/fs seems to exist as a symlink to vfs/,
but change include path to vfs.

sbin/mount_autofs: Port mount_autofs(8) from NetBSD

Christos added this to NetBSD, so why not for DragonFly.
One will write something simlar to this when porting autofs,
until autofs(5)/automount(8)/automountd(8) become usable enough.

While I try to minimize diff from FreeBSD for autofs userspace
by not making trivial style fixes, new things from NetBSD are
appreciated.

-o getargs support is dropped since unsupported on DragonFly.

kernel/ath_hal: Add missing header to fix build with -Wundef.

kernel/my: Fix -Wundef.

s/NO_MAN/NOMAN/ in various Makefiles.

Simplify some Makefiles.

If there is just one source file that is named ${PROG}.c, SRCS does
not need to be set.

Remove test/dsched*. This doesn't build anymore after 3573cf7bf66147.

Update the pciconf(8) database.

January 14, 2018 snapshot from http://pciids.sourceforge.net/

sys/vfs/autofs: prevent assert on unmount.

Bring in below commit from NetBSD/autofs. This hasn't existed in
FreeBSD and DragonFly. This at least avoids potential panic via
assertion failure on unmount like PR 3062. Note that the assertion
in autofs_node_delete() is good. There might be mkdir related bug.

The GitHub link below may not be a valid URL in the future.

http://cvsweb.netbsd.org/bsdweb.cgi/src/sys/fs/autofs/autofs_vfsops.c.diff?r1=1.2&r2=1.3
(https://github.com/NetBSD/src/commit/59fdd9b6d94c045e89f22f06c9a6f6d6c0fd60f2)

kernel: Fix some -Wundef warnings in oce(4) and mps(4).

__FreeBSD_version is not defined in DragonFly, so make it clearer
which path we take.

syscons - Skip drawing of the border, when switching to a graphics vty.

* When the target vty in a vty switch is in graphics mode, and we have
  a dumb framebuffer (sc->fbi != NULL), we definitely don't need to
  call sc_set_border().

syscons - Use COL and ROW #define constants.

Add __attribute__((__noreturn__)) to various function prototypes in usr.sbin/.

This general cleanup is useful in various ways. It helps the compiler
optimize better and it also helps later GCCs detect better when to give
-Wimplicit-fallthrough warnings and when not.

Found-with: -Wsuggest-attribute=noreturn

ftp-proxy(8): exit_daemon() never returns, do do not pretend it does.

dntpd(8): client_main() never returns, so do not pretend it does.

autofs: main_auto{,un}mountd() never return, so do not pretend they do.

watchdogd(8): Staticize.

traceroute6(8): Staticize and remove unused variables/functions.

traceroute(8): Staticize.

timed(8): slave() never returns, so don't pretend it does.

timed(8): master() never returns, so don't pretend it does.

sensorsd(8): Staticize.

rwhod(8): Staticize.