95880592 sevan Dec. 3, 2018, 6 p.m.
Obtained from:	NetBSD
MFC after:	7 days
cgit ViewVC
991699db imp Dec. 3, 2018, 5:51 p.m.
src.opts.mk includes bsd.own.mk. This in turn defines CTFCONVERT_CMD
depending on the MK_CTF value. We then set MK_CTF to no, which has no
real effect. The solution is to set all the MK_foo values before
including src.opts.mk.

This should stop the cdboot binary from exploding in size for releases
built WITH_CTF=yes in src.conf.

Sponsored by: Netflix
cgit ViewVC
668e070f vmaffione Dec. 3, 2018, 5:17 p.m.
Reviewed by:	bcr
Differential Revision:	https://reviews.freebsd.org/D18057
cgit ViewVC
0e450664 emaste Dec. 3, 2018, 3:59 p.m.
An issue remains with BIND_NOW and processes using threads.  For now,
restore libc's BIND_NOW disable, and also disable BIND_NOW in rtld and
libthr.

A patch is in review (D18400) that likely fixes this issue, but just
disable BIND_NOW pending further testing after it is committed.

PR:		233333
Sponsored by:	The FreeBSD Foundation
cgit ViewVC
ecc0f5f6 trasz Dec. 3, 2018, 1:15 p.m.
MFC after:	2 weeks
Sponsored by:	DARPA, AFRL
cgit ViewVC
f74c331a andrew Dec. 3, 2018, 9:40 a.m.
files that haven't been implemented.

Reported by:	sbruno
MFC with:	r339738
Sponsored by:	DARPA, AFRL
cgit ViewVC
f095905c jhibbits Dec. 3, 2018, 4:56 a.m.
It's possible the fdt pointer was passed in via the metadata, as is done in
ubldr.  Check for the fdt here, instead of working with a NULL fdt, and
panicking.
cgit ViewVC
3c0b0819 jhibbits Dec. 3, 2018, 4:47 a.m.
The metadata pointer will almost never be at or above 'btext', as btext is a
relocated symbol, so will be based at VM_MIN_KERNEL_ADDRESS, not at
KERNBASE.  Check the address against kernload, where the kernel is
physically loaded.
cgit ViewVC
df108aaf ian Dec. 3, 2018, 3:58 a.m.
of args data between gptboot/zfsboot and loader(8).

Despite what seems like a lot of changes here, there are no actual
changes in behavior, or in the data layout in the structures involved.
This is just eliminating identical code pasted into multiple locations.

In detail, the changes are...

- Move struct zfs_boot_args definition from libsa/zfs/libzfs.h to
  i386/common/bootargs.h because it is specific to x86 booting and the
  handoff between zfsboot and loader, and has no relation to the zfs
  library code in general.

- The geli_boot_args and zfs_boot_args structs both contain an identical
  set of member variables containing geli information.  Extract this out
  to a new geli_boot_data struct, and embed it in the arg-passing structs.

- Provide new routines geli_import_boot_data() and geli_export_boot_data()
  that can be shared between gptboot, zfsboot, and loader instead of
  pasting identical code into several different .c files.

- Remove some checks for a NULL pointer that can never be true because the
  pointer being tested was set using pointer math (kargs + 1) and that can
  never result in NULL in this code.
cgit ViewVC
e8604394 imp Dec. 2, 2018, 11:13 p.m.
Sponsored by: Netflix
cgit ViewVC
48133c3f imp Dec. 2, 2018, 11:13 p.m.
This was a typo for wdc. Eliminate it since it was in error. People
should use either 'wdc' or 'hgst' for the vendor from now on. 'hgst'
works for all versions this functionality is present for.

Sponsored by: Netflix
Differential Revision: https://reviews.freebsd.org/D18403
cgit ViewVC
2da383a5 imp Dec. 2, 2018, 11:13 p.m.
Move the Intel specific log pages (including the one that samsung
implements) to intel.c. Add comment to the samsung vendor that it will
be going away soon.

Sponsored by: Netflix
Differential Revision: https://reviews.freebsd.org/D18403
cgit ViewVC
d4fdb249 imp Dec. 2, 2018, 11:13 p.m.
Eliminage redundant spaces and nvmecontrol at start of all the usage
strings. Update the usage printing code to add them back when
presenting to the user. Allow multi-line usage messages and print
proper leading spaces for lines starting with a space.

Sponsored by: Netflix
Differential Revision: https://reviews.freebsd.org/D18403
cgit ViewVC
7d923c13 imp Dec. 2, 2018, 11:12 p.m.
Provide a usage() function that takes a struct nvme_function pointer
and produces a usage mssage. Eliminate all now-redundant usage
functions. Propigate the new argument through the program as needed.
Use common routine to print usage.

Sponsored by: Netflix
Differential Revision: https://reviews.freebsd.org/D18403
cgit ViewVC
fbf14fe8 imp Dec. 2, 2018, 11:12 p.m.
If the dispatched function doesn't exit, then we get can get a
spurious function not found message. They all do exit, but this is a
little cleaner.

Sponsored by: Netflix
Differential Revision: https://reviews.freebsd.org/D18403
cgit ViewVC