Fix leaks and test for getpagesize() returning == -1

- close file descriptors after use.
- Always munmap memory regions after mmap'ing them.
- Make sure getpagesize() returns a value greater than 0 and use a
  cached value instead of always calling getpagesize(3).

CID: 1331374-1331377, 1331653-1331662
Differential Revision: https://reviews.freebsd.org/D6011
MFC after: 2 weeks
Reported by: Coverity
Reviewed by: cem
Sponsored by: EMC / Isilon Storage Division

dev/usb: use our nitems() macro when param.h is available.

Reviewed by: hselasky

Link bcache into userboot.so, was not added in r298230

This should help speed up bhyve boots too

Reviewed by: olivier

restore: use our roundup2/rounddown2() macros when param.h is available.

While here cleanup a little a malloc call.

Always emit an error message on passthru configuration errors.

Previously, many errors (such as the PCI device not being attached
to the ppt(4) driver) resulted in bhyve silently exiting without
starting the virtual machine.  Now any errors encountered when
configuring a virtual slot for a PCI passthru device should be noted
on stderr.

Reviewed by: neel
Differential Revision: https://reviews.freebsd.org/D5990

1. modify fwdump (a.k.a grcdump) so that grcdump memory is allocated
   and freed on as needed basis.
2. grcdump can be taken at failure points by invoking bxe_grc_dump()
   when trigger_grcdump sysctl flag is set. When grcdump is taken
   grcdump_done sysctl flag is set.
3. grcdump_done can be monitored by the user to retrieve the grcdump.

Submitted by:vaishali.kulkarni@qlogic.com

net80211: do not reschedule scan_curchan_task() if the scan was canceled.

This should fix possible use-after-free in the scheduled task.

PR: 208605

rtld-elf: use our roundup2() macro when param.h is available.

dump: use NULL instead of zero for pointers.

Clean out the casts from calloc(3) while here.

fsck_msdosfs: use NULL instead of zero for pointers.

rtld-elf: use NULL instead of zero for pointers.

aacraid(4): Sanely copyin userland pointers and ensure that we don't get
anything janky from a user. (cturt)

aac(4): landergriffith+freebsdbugzilla@gmail.com pointed out that aacraid(4)
had the same issue and handling of pointers, so let's change that too.

PR: 206573
Submitted by: cturt@hardenedbsd.org
Obtained from: HardenedBSD
MFC after: 1 week

Plug memory leak in ctl(4) when ctl_copyin_args() is called with a non-
null terminated ASCII string.

PR: 207626
Submitted by: cturt@hardenedbsd.org
MFC after: 2 days

[bhnd] Standardize bhnd device tables and quirk matching.

This add a bhnd device table mechanism that standardizes matching of
devices on the bhnd(4) bus, discovery of device quirk flags, and should
be pluggable into the new PNPINFO machinery.

Submitted by: Landon Fuller <landonf@landonf.org>
Differential Revision: https://reviews.freebsd.org/D5759

[bhnd] Clean up bhnd resource handling and inherited bus methods

To facilitate use by SoC implementors working with bhnd-inheriting fdt/nexus
drivers:

* Splits bhnd_bus method implementations into generic bus implementations
  (bhnd_bus_generic) and generic bhnd(4) driver implementations (bhnd_generic)
* Simplifies bhnd resource handling, allowing bhnd bus implementations to
  support bhnd resource activation by implementing the standard BUS_*
  resource APIs and BHND_BUS_ACTIVATE_RESOURCE().

Submitted by: Landon Fuller <landonf@landonf.org>
Differential Revision: https://reviews.freebsd.org/D5758

[bhnd] Add support for specifying the address space used by bhndb children

This adds support for specifying the address space used by a bridge child;
this will either be the bridged SoC address space, or the host address space
required by children that map non SoC-address ranges from the PCI BAR.

This is necessary to support SROM/OTP child devices that live directly
beneath the bhndb device and require access to host resources, instead
of the standard behavior of delegating access to the bridged SoC address
space.

Submitted by: Landon Fuller <landonf@landonf.org>
Differential Revision: https://reviews.freebsd.org/D5757

Fix pc98 build error introduced in r298230

Submitted by: Toomas Soome <tsoome@me.com>
Spotted by: bz
Differential Revision: https://reviews.freebsd.org/D6002

Revert r298268 (Add optional chip_select/deselect methods).
None of supported hardware do require that.

Assert CS for single transfers.

Use GPIOTOGGLE to toggle the pin state instead of read, modify and write.

Mention fsck_ffs -E in tunefs(8). It's non-obvious that one should
use it after enabling TRIM.

Reviewed by: brueffer@
MFC after: 1 month
Sponsored by: The FreeBSD Foundation
Differential Revision: https://reviews.freebsd.org/D5928

Add PCI ID for family 10h model 30h to amdtemp(4).

This adds support to CPU found in PC Engines APU2 series.

MFC after: 3 weeks
Sponsored by: Rubicon Communications (Netgate)

Add driver for Xilinx AXI Quad SPI device. The device was found in
lowRISC hardware.

Sponsored by: DARPA, AFRL
Sponsored by: HEIF5

Add optional chip_select/deselect methods. This is required
when we want to keep CS asserted for multiple transfers.

Fix debugging printf.

MFC after: 1 month
Sponsored by: The FreeBSD Foundation

Fix umtx lock/trylock for compat32.

Sponsored by: The FreeBSD Foundation
MFC after: 3 days

Use nitems() from sys/param.h.

MFC after: 2 weeks.

Use nitems() from sys/param.h.

MFC after: 2 weeks.

hyperv/vmbus: Make device probe/attach synchronous w/ vmbus attach/SYSINIT

Discussed with: Jun Su <junsu microsoft com>, Dexuan Cui <decui microsoft com>
MFC after: 1 week
Sponsored by: Microsoft OSTC

hyperv: Remove two assign-only local variables

Submitted by: Jun Su <junsu microsoft com>
MFC after: 1 week
Sponsored by: Microsoft OSTC

Fix build breakage introduced by r298253.

Use nitems() from sys/param.h.

MFC after: 2 weeks.

Use nitems() from sys/param.h.

MFC after: 2 weeks.

Rename units.lib -> definitions.units
- this matches GNU units 2.12
add ISO country codes from units 2.12

Rename units.lib -> definitions.units
- this matches GNU units 2.12
add ISO country codes from units 2.12

Add VHT power envelope parsing to ifconfig.

Remove pathnames.h
- it only holds a single constant
- it doesn't exist in the GNU variant

Use nitems() from sys/param.h.

MFC after: 2 weeks.

Use nitems() from sys/param.h.

MFC after: 2 weeks.

Use nitems() from sys/param.h.

MFC after: 2 weeks.

Remove redundant parenthesis.

Submitted by: pfg
MFC after: 2 weeks.

Use nitems() from sys/param.h.

MFC after: 2 weeks.

Use nitems() from sys/param.h.

MFC after: 2 weeks.

Use nitems() instead of sizeof(name) / sizeof(*name).

MFC after: 2 weeks.

Add a new installation type to bsdinstall/zfsboot: BIOS+UEFI

Installs both pmbr+gptzfsboot as well as boot1.efifat in separate partitions
The resulting system can be booted with either UEFI or BIOS/CSM.
Preference is controlled by the user's firmware boot settings.

This is now the default for zfsboot installs

PR: 208629
Submitted by: Galael LAPLANCHE <ganael.laplanche@corp.ovh.com> (original version)

Unbreak the build if you enable WITH_NAND

Followup to r298230

Submitted by: Nikolai Lifanov <lifanov@mail.lifanov.com> (original version)
Sponsored by: ScaleEngine Inc.

Use NULL instead of 0 for pointers.

realloc will return NULL in case it cannot allocate memory.

MFC after: 2 weeks.

malloc will return NULL if it cannot allocate memory.

MFC after: 2 weeks.

getfsent(3) will return NULL on EOF or error.

MFC after: 2 weeks.

Use NULL instead of 0 for pointers.

malloc will return NULL if it cannot allocate memory.

MFC after: 2 weeks.

Fix SMP booting for PowerPC Book-E

Summary:
PowerPC Book-E SMP is currently broken for unknown reasons.  Pull in
Semihalf changes made c2012 for e500mc/e5500, which enables booting SMP.

This eliminates the shared software TLB1 table, replacing it with
tlb1_read_entry() function.

This does not yet support ePAPR SMP booting, and doesn't handle resetting CPUs
already released (ePAPR boot releases APs to a spin loop waiting on a specific
address).  This will be addressed in the near future by using the MPIC to reset
the AP into our own alternate boot address.

This does include a change to the dpaa/dtsec(4) driver, to mark the portals as
CPU-private.

Test Plan:
Tested on Amiga X5000/20 (P5020).  Boots, prints the following
messages:

 Adding CPU 0, pir=0, awake=1
 Waking up CPU 1 (dev=1)
 Adding CPU 1, pir=20, awake=1
 SMP: AP CPU #1 launched

top(1) shows CPU1 active.

Obtained from: Semihalf
Relnotes: Yes
Differential Revision: https://reviews.freebsd.org/D5945

Use NULL instead of 0 for pointers.

dbopen(3) will returns a NULL on error.

MFC after: 2 weeks.

Use NULL instead of 0.

malloc will return NULL in case it cannot allocate memory.

MFC after: 2 weeks.

Use NULL for pointers instead of 0.

MFC after: 2 weeks.

Use NULL instead of 0 for pointers.

kvm_open(3) will return NULL when it cannot access kernel virtual memory.

MFC after: 2 weeks.

Use NULL for pointers instead of 0.

MFC after: 2 weeks.

hptmv(4) Fix potential buffer overflow in hpt_set_info.

While here, adjust some whitespace and yeild some useful debug info.

This is untested on this hardware, testing requests to -scsi went
unanswered.

PR: 206585
Submitted by: cturt@hardenedbsd.org
MFC after: 2 weeks

A new implementation of the loader block cache

The block cache implementation in loader has proven to be almost useless, and in worst case even slowing down the disk reads due to insufficient cache size and extra memory copy.
Also the current cache implementation does not cache reads from CDs, or work with zfs built on top of multiple disks.
Instead of an LRU, this code uses a simple hash (O(1) read from cache), and instead of a single global cache, a separate cache per block device.
The cache also implements limited read-ahead to increase performance.
To simplify read ahead management, the read ahead will not wrap over bcache end, so in worst case, single block physical read will be performed to fill the last block in bcache.

Booting from a virtual CD over IPMI:
0ms latency, before: 27 second, after: 7 seconds
60ms latency, before: over 12 minutes, after: under 5 minutes.

Submitted by: Toomas Soome <tsoome@me.com>
Reviewed by: delphij (previous version), emaste (previous version)
Relnotes: yes
Differential Revision: https://reviews.freebsd.org/D4713

Allow -f dirdeps.mk some/dir with no TARGET_MACHINE spec

Use $MACHINE if target does not specify.

Reviewed by: bdrewery

META_MODE: Don't hide the .depend rm -f command.

Otherwise the build command changes every build.  META_MODE will
only remove it if something changes to warrant rebuilding
the file.

Sponsored by: EMC / Isilon Storage Division

Update meta2deps to fix crash when using -X with M2D_EXCLUDES.

libc: do not include <sys/types.h> where <sys/param.h> was already included

According to style(9):
> normally, include <sys/types.h> OR <sys/param.h>, but not both.
(<sys/param.h> already includes <sys/types.h> when LOCORE is not defined).

Update dirdeps.mk et al

dirdeps.mk: move logic to handle -f dirdeps.mk to inside check
for first read of dirdeps.mk

Also fix handling of WITHOUT_DIRDEPS_BELOW

gendirdeps.mk: pass M2D_EXCLUDES to meta2deps

meta.autodep.mk: if we build with nofilemon, leave a cookie to
prevent updating dependencies until cleaned.

Reviewed by: bdrewery

Correct possible underflow conditions when checking for available space
in the tx h/w ring buffer.

Reviewed by: gnn jeb.j.cramer@intel.com
MFC after: 1 week
Sponsored by: Limelight Networks
Differential Revision: https://reviews.freebsd.org/D5918

Address issues found by the XCode code analyzer.

Add MK_AUTO_OBJ=no to ${MAKE} invocation if PKG_VERSION needs to be
created to avoid creating ${.OBJDIR}.

The duplicate REVISION/BRANCH/VERSION evaluation will be addressed
separately.

Sponsored by: The FreeBSD Foundation

Follow-up r298220: Don't pass down META_MODE which will still enable it.

Reported by: Nikolai Lifanov <lifanov@mail.lifanov.com>
Sponsored by: EMC / Isilon Storage Division

META_MODE: Disable during installworld and similar.

META_MODE may create cookies during staging of files to WORLDTMP that would
also prevent installation of the files to the final DESTDIR, since the cookie
already exists.  This is not yet the case but will be soon.  Prevent other
similar issues by disabling META_MODE for any top-level install targets.

Sponsored by: EMC / Isilon Storage Division

Cause an error during 'make install' if trying to compile with CC.

This is limited to src-tree builds, meaning not extended to ports or other
out-of-tree builds.

This will help ensure that read-only OBJDIRS will be respected at install-time
by causing a more consistent failure for those who don't use a read-only
OBJDIR.  It also will cause Jenkins to yell.  This is a better solution than
trying to see CC=false as has been attempted and discussed before.

Of course this is only relevant for files generated by CC.

Disable this for META_MODE since it will detect the CFLAGS/command
change and force a rebuild.

Sponsored by: EMC / Isilon Storage Division

Follow-up r297842: Rework header generation to fix always rebuilding.

This reworks the handling of common headers to just include the needed
logic rather than invoke MAKE.  This avoids the problem listed in r297842
and avoids other dependency tracking issues.

Pointyhat to: bdrewery
Reported by: Nikolai Lifanov <lifanov@mail.lifanov.com>
Sponsored by: EMC / Isilon Storage Division

META_MODE: Clean .meta files with 'make cleandepend'.

Sponsored by: EMC / Isilon Storage Division

Reuse our roundup2() macro instead of reinventing the wheel.

Obtained from: DragonflyBSD

Re-use our roundup2() macro instead of reinventing the wheel.

Obtained from: DragonflyBSD

libipsec: use NULL instead of zero for pointers.

rbootd: use NULL instead of zero for pointers.

Add the ability to read a SAS device's Target Port NAA designator

sys/cam/scsi/scsi_all.h
sys/cam/scsi/scsi_all.c
Add the scsi_devid_is_port_naa helper function

Reviewed by: ken
MFC after: 4 weeks
Sponsored by: Spectra Logic Corp
Differential Revision: https://reviews.freebsd.org/D5975

ftpd: replace malloc + memset 0 with calloc.

It is faster and usually safer.
Use NULL instead of zero for the pointer.

libstand: use NULL instead of zero for pointers.

Use NULL instead of 0 for pointers.

strchr(3) will return NULL if the character does not appear in the
string.

MFC after: 2 weeks.

libbluetooth: use NULL instead of zero for pointers.

rexec(3): use NULL instead of zero for pointers.

User NULL instead of 0 for pointers.

gethostbyname(3) will return NULL in case of an error.

MFC after: 2 weeks.

Use NULL instead of 0 for pointers.

strchr(3) will return NULL if the character does not appear in the
string.

MFC after: 2 weeks.

Replace ${CURDIR}/../.. with ${CURDIR:H:H} in elftoolchain

This produces a nicer path in debug info and build logs.

MFC after: 1 week
Sponsored by: The FreeBSD Foundation

Fix signed/unsigned warnings.

Pull the MSI/MSI-X handling functions out to help with INTRNG integration.

Obtained from: ABT Systems Ltd
Sponsored by: The FreeBSD Foundation

Fixed indentation, minor style.

Fix a warning about an unused variable.

Put panic() calls under INVARIANTS.

Fix tinderbox LINT build.

Minor cosmetic cleanup

- Remove spurious trailing whitespace in licensing header
- Remove unnecessary semi-colon after comment [*]

MFC after: 3 days
Submitted by: pfg [*]
Sponsored by: EMC / Isilon Storage Division

Use NULL instead of 0 for pointers.

malloc will return NULL if it cannot allocate memory.

MFC after: 2 weeks.

strchr(3) will return NULL if it cannot find the character in the
string.
getfsent(3) will return NULL on EOF or error.

MFC after: 2 weeks.

Use NULL instead of 0 for pointers.

strchr(3) will return NULL if the character does not appear in the
string.

MFC after: 2 weeks.

MFV r298178:

Update file to 5.26.

MFC after: 2 weeks
Relnotes: yes

User NULL instead of 0 for pointers.

getservent(3) returns NULL on EOF or error.

MFC after: 2 weeks.

Use NULL instead of 0 for pointers.

fopen(3) will return NULL in case it cannot open the STREAM.

MFC after: 2 weeks.

Use NULL instead of 0 for pointers.

MFC after: 2 weeks.

Cleanup debug output.

Don't use anonymous unions.