grdc.6: Fix .Dd typo.

cgram(6): Don't move over the last char and fix an assertion

The maximum value of cursor_x was the text length and thus also
*equaled* to extent_x, which broke the assertion in substitute()
and caused crashes.

In addition, the cursor could move just beyond the last character
on a line, which didn't make much sense.

So only allow the cursor move to the last char but not over it.
This also properly fix the above assertion issue.

cgram(6): Sync the rewritten version from NetBSD

Fixes:

* Do not consider the puzzle solved if all letters in the visible area
  are substituted correctly.  To be properly solved, the whole puzzle
  must be solved, even those parts that are currently off-screen.

* Never place the cursor at the very right edge of the screen since that
  does not work well with some terminals.  The maximum valid x
  coordinate is COLS - 1.

* Add horizontal scrolling.  Make all coordinate handling symmetric in
  regard to the horizontal and vertical axes.  Previously, lines longer
  than 80 characters could not be seen on the screen.

* Fix undefined behavior when pressing function keys, which was
  resulting parts of the screen being cleaned.

Improvements:

* Remove the arbitrary limit of 128 characters per line.  Even if
  fortune(6) may never generate such long lines, the code is easy
  enough to adapt to other sources.

* Properly clean up the allocated memory.  Previously, only the string
  arrays were freed but not the strings themselves.

* Allow navigation with page up/down keys.

* Consistently use char for characters, avoiding converting back and
  forth between char, unsigned char and int.

bmake: Remove unneeded .PATH from the Makefile.

libaura: Improve fspred test a bit

grdc(6): Add '-t' option to show time in 12-hour format

Partially based on FreeBSD's implementation (revision 112749).  But I
moved the AM/PM mark to the upper-left corner instead of the
bottom-right corner as in FreeBSD, because the mark in the bottom-right
corner would be flushed by scrolling.

In addition, improve the determination of AM/PM mark, including:
00:mm -> 12:mm AM
12:mm -> 12:mm PM

grdc(6): Improve scrolling timestamp precisions

Use clock_gettime() to obtain the current time with higher precision,
which is used to improve the scrolling timestamp, trying to make the
scrolling end on the integral seconds.

Inspired by OpenBSD's implementation.

grdc(6): Allow 'q' to quit immediately

Add support to allow pressing 'q' to quit immediately.

Partially based on OpenBSD's implementation:
https://github.com/openbsd/src/commit/8ca82b000914eb1e6f45d22092c9358df4233247

grdc(6): Improve handling of long scroll delays

Add the scroll delay to the current time, so that the finally showed
time matches the real current time.

In addition, skip nanosleep() if the sleep interval is negative.

grdc(6): Various code cleanups

* Staticize various variables and functions.
* Reduce file-global variables.
* Minor style updates.
* Add a copyright header (as did by OpenBSD).

grdc(6): Fix 100% CPU usage and make timeout work again

grdc(6) was passing a timespec with tv_nsec = 1000000*1000, which is
invalid.  So every nanosleep() call just returns with EINVAL error.
This causes grdc(6) to loop hard and uses 100% CPU.  It also brreaks
the optional timeout argument; grdc(6) just exits immediately.

Fix the above issue by correctly setting the timespec argument.

This fix also makes the optional timeout argument work again.

ching(6): More minor code cleanups

ching(6): Fix HOME key to move to the beginning of line

cgram.6: Mention that cursor keys can also be used

cgram(6): Reformat code styles

Apply indent(1) and then manually adjust a bit.

games: Bring in cgram(6) from NetBSD

cgram is a curses-based widget for solving Sunday-paper-type cryptograms
based on substitution ciphers.  A random cleartext is chosen using
fortune(6) and a random substitution key is generated.

games: Bring in monop(6) from NetBSD

monop is reminiscent of the Parker Brother's game Monopoly, and monitors
a game between 2 to 9 users.  It is assumed that the rules of Monopoly
are known.  The game follows the standard rules, with the exception that,
if a property goes up for auction and there are only two solvent players,
no auction is held and the property remains unowned.

games: Bring in gomoku(6) from NetBSD

gomoku is a two player game where the object is to get 5 in a row
horizontally, vertically or diagonally on a 19 by 19 grid.

ching.6: Mention mandoc(1) together with nroff(1)

ching(6): Simplify the sed expression

No functional/style changes.

<sys/sysent.h>: Remove the old SCARG() macro, de-SCARG() the mqueue code.

nrelease: Fix typo.

libaura: Do not initialize va_list args twice

  - Fixes a segfault in the installer right after
    "Begin Installing files".

Reported-by: @swildner

make(1): Bring in some overlooked local modifications.

  - Bring back the following global variables and document them
    a bit. Also bring back the manpage information about them.

.MAKE.BUILT.BY
.MAKE.DF.VERSION
.MAKE.DF.OSREL

  - Bring back some manpage fixes (partially cf7697c28d)

This should fix any DPorts build problems where those variables are
used.

Reported-by: @swildner

fexecve(2): Return ENOENT if exec a script opened with O_CLOEXEC

If a script (i.e., interpreter file) is opened with the O_CLOEXEC flag,
it would be closed by the time the interpreter is executed, and then the
executation would fail.  So just return ENOENT from fexecve(2).  This
behavior aligns with Linux's.

See Linux's fexecve(2) man page.

See also: https://bugzilla.kernel.org/show_bug.cgi?id=74481

Thank dillon for implementing the holdvnode2() function to obtain the
fileflags together with the fp from fd.

Implement the fexecve(2) system call

The fexecve(2) function is equivalent to execve(2), except that the file
to be executed is determined by the file descriptor fd instead of a
pathname.

The purpose of fexecve(2) is to enable executing a file which has been
verified to be the intended file.  It is possible to actively check the
file by reading from the file descriptor and be sure that the file is
not exchanged for another between the reading and the execution.

See https://pubs.opengroup.org/onlinepubs/9699919799/functions/fexecve.html

This work is partially based on swildner's patch and FreeBSD's
implementation (revisions 177787, 182191, 238220).

XXX: We're missing O_EXEC support in open(2).

Reviewed-by: dillon

kern: Clean error paths in kern_execve()

Merge the original 'exec_fail_dealloc' and 'exec_fail' to a single
'failed' error path.  In addition, introduce the 'nch' variable to
clean some expressions a bit.  These will help the following fexecve()
implementation.

While there, adjust the styles a bit.

Reviewed-by: dillon

kern: Return error from exec_copyin_strings() if fname copy failed

kern: Staticize several functions and variables in kern_exec.c

Staticize exec_copyin_args(), exec_free_args() and print_execve_args()
functions, and move the related prototypes and exec_path_segflg enum
from <sys/imgact.h> here.

In addition, staticize the 'debug_execve_args' variable.

execve(2): Minor markup improvements

execve.2: Document reopening of standard descriptors

Meanwhile, add the STANDARDS section and document the current standard
status.  The reopening of standard descriptors is specified in
IEEE Std 1003.1-2001/Cor 1-2002, item XSH/TC1/D6/15.

Obtained-from: FreeBSD (r101480)

Upgrade make(1). 2/2

  - Update README.DRAGONFLY
  - Adapt Makefile
  - Regenerate config.h

Upgrade make(1). 1/2

Merge branch 'vendor/BMAKE'

dsynth: Oops, parameters were reversed

ching(6): Support to format with mandoc(1)

We don't ship nroff(1) anymore, so adapt ching(6) to support to use
mandoc(1) to format its output.  Mandoc(1) only implements part of
roff(7) language, but it's enough for ching(6), although minor
adjustments to the mandoc(1) output are needed to make it similar to
the nroff(1) output.

Prefer nroff(1) if it's available (provided by groff package).

ching(6): Tweak roff(7) macros to properly format with mandoc(1)

The '\t' sequence (or the literal tab between two arguments) is ignored
by mandoc(1), so change it to spaces, which displays well with mandoc(1).

Quote the last argument of the .LX macro to fix warnings:
"whitespace at end of input line"

Change '.rm LH' to '.ds LH' in the .L macro to fix warnings:
"undefined string, using "": LH"

games: Bring in ching(6) -- The Book of Changes -- from NetBSD

The ching(6) game/utility was nuked from FreeBSD in 1994 ...
(see revision 2563)

It has been added back to NetBSD in June 30, 2005:
-----------------------------------------------------------------------
ching(6) -- The Book of Changes, not seen in BSD since 4.4BSD.

The C code is a complete rewrite done by Guy Harris for 4.4BSD and is
not the original from v7 at all. Unfortunately, the data file itself,
without which the rest isn't useful, was not freed until Caldera
released all of 32v a few years ago, so it was not in 4.4lite.

The data file, the ching nroff macros, and the driver script are under
Caldera or Caldera+Berkeley copyright.

I've partially redone the driver script from 4.4 (there was no point
in using a temporary file for the hexagram), which was already
partially redone from 32v. (As an aside, it is nutty that the script
needs a copyright so many times longer than the code.)

I've renamed "cno" to "castching", "phx" to "printching", (the
traditional names were opaque), and put them in /usr/libexec/ching
instead of the v7 /usr/games/ching.d. The data file and nroff macros
are in /usr/share/games/ching
-----------------------------------------------------------------------
(see https: //github.com/NetBSD/src/commit/5175ead765ed41e9db0f9f59d83ddf8966f6568a)

libthread_xu: Add clock_nanosleep symbols to pthread.map

Although pthread.map is currently unused (commented in the Makefile),
add 'clock_nanosleep' and '_clock_nanosleep' symbols to it anyway.

vendor/bmake: upgrade from 20200710 to 20210206

Summary of notable changes:

  o meta.c: target flagged .META is out-of-date if meta file missing
  o var.c: each flag type gets its own prefix.
  o optimize some buffer operations - avoid strlen
  o parse.c: remove VARE_WANTRES for LINT, we just want to check
    parsing (for now).
  o cond.c: make CondGetString easier to debug
  o var.c: support for read-only variables eg .SHELL being the shell
    used to run scripts.
  o enum.c: distinguish between bitsets containing flags and ordinary enums
  o main.c: do not attempt to read .MAKE.DEPENFILE if set to /dev/null or
    anything starting with "no"
  o parse.c: remove unused, undocumented .NOEXPORT
  o rename VAR_CMD to VAR_CMDLINE
  o .MAKE.{UID,GID} represent uid and gid running make.
  o fix error handling for .BEGIN and .END dependency in -k mode
  o compat.c: fix .ERROR_TARGET in compat -k mode

For detailed list see ChangeLog.

Upgrade libressl. 2/2

Update README.DRAGONFLY.

Upgrade libressl. 1/2

Merge branch 'vendor/LIBRESSL' into master.

vendor/libressl: upgrade from 3.2.3 to 3.2.4

Bug and interoperability fixes:

 * Switch back to certificate verification code from LibreSSL 3.1.x. The
   new verifier is not bug compatible with the old verifier causing issues
   with applications expecting behavior of the old verifier.

 * Unbreak DTLS retransmissions for flights that include a CCS

 * Only check BIO_should_read() on read and BIO_should_write() on write

 * Implement autochain for the TLSv1.3 server

 * Use the legacy verifier for autochain

 * Implement exporter for TLSv1.3

 * Free alert_data and phh_data in tls13_record_layer_free()

 * Plug leak in x509_verify_chain_dup()

 * Free the policy tree in x509_vfy_check_policy()

kernel: Move sys_{get,set}rlimit() to where their kern_* functions are.

libthread_xu: Remove some unused declarations from thr_private.h.

sys/vfs/hammer: modify inode's ctime after successful chmod()

On successful completion of chmod() (ie. the call is gonna return 0),
modify inode's ctime.

Issue: https://bugs.dragonflybsd.org/issues/3251
Link: https://pubs.opengroup.org/onlinepubs/9699919799/functions/chmod.html

sys/vfs/hammer2: modify inode's ctime after successful chmod()

On successful completion of chmod() (ie. the call is gonna return 0),
modify inode's ctime.

Issue: https://bugs.dragonflybsd.org/issues/3251
Link: https://pubs.opengroup.org/onlinepubs/9699919799/functions/chmod.html

dsynth: Make the mount process a bit more portable

<utmpx.h>: Small whitespace fix.

<net/if.h>: Preparation for removing <sys/types.h> from <sys/socket.h>.

* Include <sys/types.h> for the non-standard parts.

<utmpx.h>: Preparation for removing <sys/types.h> from <sys/socket.h>.

* Define pid_t as POSIX specifies.

* Use internal integer types for some non-standard struct utmpx members.

libc: Change microseconds args of ualarm() and usleep() to useconds_t.

It's a standard type and more expressive.

No functional change, unsigned int becomes uint32_t.

usr.sbin/autofs: Fix absolute path when creating a mountpoint

from freebsd 63640b2f552c0476f50484635eb9888eafcd22dc

FreeBSD finally fixed a bug I reported in 2017,
in a bit different way from a workaround in DragonFly and NetBSD.
Apply FreeBSD fix in place of 108ed43a1e5e7ff8f890085e206d970562bf4a7c.

nanosleep.2: Mark up defines with .Dv

Update the pciconf(8) database.

February 12, 2021 snapshot from https://pci-ids.ucw.cz

kernel/nfs: Initialize a sometimes uninitialized variable.

Just for style reasons. In the case where it was passed uninitialized,
it was not used by the called function.

kernel: Comment out some two unused functions.

<sys/stdint.h>: Add missing typedef guards for uint*_t.

kernel/drm: Remove duplicate type definitions.

* <linux/spinlock.h>: A definition of 'spinlock_t' is already present
      from <linux/spinlock_types.h>.

* <linux/types.h>: A definition of 'bool' is already present from
      <sys/types.h>. Also, the defined() check was just wrong because
      it can only check for preprocessor macros, not for typedefs.

rc.d/mounttmpfs: Avoid duplicate tmpfs mount at /tmp

Newer DragonFly installation already mounts a tmpfs at /tmp, as
configured in /etc/fstab.  So check whether /tmp is already a mount
point before mounting a tmpfs there to prevent the duplicate mounts.

Reported-by: swildner

kernel - Remove the #define AF_MAX from sys/mount.h

* Not sure how that managed to wind up in sys/mount.h but
  it doesn't belong there.  Bye bye.

dsynth: Add phase string for the new phases

<sys/iconv.h>: Add <sys/types.h> for u_char and fix a comment.

Fix several typos in calendars, READMEs and other files.

regdomain.xml: Correct various country names.

<sys/iconv.h>: Include <sys/cdefs.h> for __{BEGIN,END}_DECLS.

kernel/vm: Fix a wrong check in vm_fault_page().

It was calling pmam_remove() more often than intended because of
wrong COW detection (the if() was always true).

Approved-by: dillon

tic: Fix wrong path in the Makefile.

It doesn't change the contents of term.h, though.

libkinfo/Makefile: Oops, fix typo in the MLINKS.

Use %U for URIs within reference blocks in various manual pages.

kinfo.3: Mention cputime_pcpu_statistics() and route_pcpu_statistics().

libaura: Limit device/mount point comparison to PATH_MAX

libaura: Restore local variable name 'args'.

  - For consistency's sake across the tree where 'args' or 'ap'
    are widely used.

libaura: Restore fspred CVD Id.

  - It is still important for some people.

libaura: Remove unused variable

Reported-by: daftaupe

libaura: Several fixes to fs predicates

  - Make is_program() a bit more precise.
  - Fix is_device()
  - Simplify the code a bit.
  - Regression tests now pass.

Test case                                                   Result
--------------------------------------------------------------------------------
usr.sbin/installer/libaura/dict1/dict1                      PASS
usr.sbin/installer/libaura/dict2/dict2                      PASS
usr.sbin/installer/libaura/dict3/dict3                      PASS
usr.sbin/installer/libaura/fspred1/fspred1                  PASS

libaura: Fix fspred test

kernel/bpf: Put a struct declaration (netmsg_bpf_output) into the NBPF check.

kernel/rtsock: Remove no longer needed ns_count in struct route_cb.

This is a leftover from the XEROX Network Systems (NS) protocol removal
(62b5ec1d702a90f4c4f06347b3ad258a29ff8929 and later commits).

kernel: Remove two unused declarations.

regression: Initial runlist for the regression tests

libaura: Regression test for fspred

regression: Force rm to avoid dfregress to hang in stdin

dsynth: Add extra information to the build output

  - Sort of bring what marino's synth used to output before
    starting the build of a port. Environment, some make
    variables and /etc/make.conf

dsynth: Warn the user about world/kernel being out of sync

dsynth: Set OPSYS according to the detection or profile config

kernel: Fix two typos in comments.

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

* South Sudan changes from +03 to +02 on 2021-02-01 at 00:00.

* Also update leap second info.

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

sbin/hammer2: Fix compilation on Linux

kernel: Move the flags for kern_dup() to <sys/filedesc.h>.

kernel/acpi: Remove bogus checks for < 0 of unsigned variables.

kernel/acpi: Fix some implementation specific shifting of signed ints.

All these are stored as unsigned.

kernel/acpi: Remove an unused struct definition.

kernel/acpi: Remove unneeded bus number check.

pcib_get_bus() never fails. Parent is pcib, so the bus number will be
valid.

kernel/acpi: Remove two redundant initializations.

kernel/acpi: Remove redundant NULL check.

AcpiGetObjectInfo() returned AE_OK and adinfo has already been
dereferenced at this point.

dsynth: Use semantic versioning-ish numbering for version

kernel - Separate XSAVE support from AVX support

* XSAVE may be supported without AVX, on certain CPUs and also under
  hypervisors that expose XSAVEOPT without AVX.

* Always use XSAVE, when supported, regardless of whether AVX is supported
  or not.

Submitted-by: chicken
Bug-ID: #3264

kernel - Add memory barriers to more __asm functions.

   * Add memory barriers to more __asm functions that have memory
     side-effects.  This should help prevent any unexpected instruction
     reordering around these functions.

Submitted-by: chicken
Bug-ID: #3263

msdosfs: fix #3237 where reading the last sector of a file returned 0s

cluster_readx would load the correct data to cache except for the last
sector of a file, where the cache was filled with zeroes.

On small files (2 clusters) cluster_readx wouldn't even call
msdosfs_bmap, correctly read first cluster and return zeroes for the
second cluster.

Fix this by telling cluster_readx that the file is bigger than it
is. cluster_readx's `nblk` will then include the last sector.

dsynth.1: Several fixes

  - Add some options and directives that were missing.
  - Rearrange the manpage to match the output of 'dsynth help'
  - Fix some new sentence, new line warnings from lint.