drm/sched: Avoid data corruptions

From Andrey Grodzovsky
0687411e2a8858262de2fc4a1d576016fd77292e in linux 5.10.y/5.10.51
0b10ab80695d61422337ede6ff496552d8ace99d in mainline linux

drm/scheduler: Fix hang when sched_entity released

From Andrey Grodzovsky
5ed8c298b2e140c640af8813a490fb4d77165e97 in linux 5.10.y/5.10.51
c61cdbdbffc169dc7f1e6fe94dfffaf574fe672a in mainline linux

drm/amd/display: fix use_max_lb flag for 420 pixel formats

From Dmytro Laktyushkin
3ca86d44b9023cd96c893d6dd90aacbca90e4d74 in linux 5.10.y/5.10.51
8809a7a4afe90ad9ffb42f72154d27e7c47551ae in mainline linux

drm/amd/amdgpu/sriov disable all ip hw status by default

From Jack Zhang
b025bc07c94770ab5ca68a8b2ead12628c2a0698 in linux 5.10.y/5.10.51
95ea3dbc4e9548d35ab6fbf67675cef8c293e2f5 in mainline linux

drm/amd/display: fix HDCP reset sequence on reinitialize

From Brandon Syu
fb7479d64d77a3bb0ee992f9e450cf52d56c6b2c in linux 5.10.y/5.10.51
99c248c41c2199bd34232ce8e729d18c4b343b64 in mainline linux

sync kfd_ioctl.h with linux 5.10.y

Use SUDO when setting up hostkey.

Mop up dtls1_get_ccs_header() and struct ccs_header_st.

All this code does is read one byte from memory with an unknown length,
potentially being a one byte overread... and then nothing is actually done
with the value.

ok tb@

Inline DTLS1_CCS_HEADER_LENGTH rather than having a single use variable.

ok tb@

Increase time margin for rekey tests.  Should help reliability on very
heavily loaded hosts.

Ensure that all returned SSHFP records for the specified host name and
hostkey type match instead of only one.  While there, simplify the code
somewhat and add some debugging.  Based on discussion in bz#3322, ok djm@.

Add test for host key verification via SSHFP records.  This requires
some external setup to operate so is disabled by default (see
comments in sshfp-connect.sh).

Add ed25519 key and test SSHFP export of it.  Only test RSA SSHFP export
if we have RSA functionality compiled in.

Put dh_set_moduli_file call inside ifdef WITH_OPENSSL.  Fixes build with
OPENSSL=no.

Group keygen tests together.

Add test for ssh-keygen printing of SSHFP records.

Markup promises with Cm not Va

These are not variable names but rather keywords (as port of one big
string).

I originally changed this such that `.Bl -tag' would automatically tag them
and thus allow ":t id" in the manual pager;  since schwarze's recent
mandoc(1) commit this is no longer needed as `.Va' gained tagging with it,
but its use is incorrect here nonetheless, so still change it to the more
appropiate `.Cm'.

Input jmc schwarze

Don't save the prt_scyl/prt_ecyl values, change them, use them
and then restore them. Just change/use the saved values and skip
the restoring.

Allows PRT_make() to add 'const' to its struct mbr parameter, and
thus allows MBR_make() to add 'const' to its struct mbr
parameter.

No intentional functional change.

Introduce and use garbage collector for 'ipsec_ids' struct entities
destruction instead of using per-entity timeout. This fixes the races
between ipsp_ids_insert(), ipsp_ids_free() and ipsp_ids_timeout().

ipsp_ids_insert() can't stop ipsp_ids_timeout() timeout handler which is
already running and awaiting netlock to be released, so reused `ids' will
be silently removed in this case.

ipsp_ids_free() can't determine is ipsp_ids_timeout() timeout handler
running because timeout_del(9) called by ipsp_ids_insert() clears it's
triggered state. So ipsp_ids_timeout() could be scheduled to run twice in
this case.

Also hrvoje@ reported about ipsec(4) throughput increased with this diff
so it seems we caught significant count of ipsp_ids_insert() races.

tests and feedback by hrvoje@
ok bluhm@

Zap dhclient bits, mention resolvd

OK deraadt

Make GPT_read() obtain and validate the on-disk MBR itself, via
MBR_read(), and report success/failure.

Simplifies logic and makes clearer that the protective MBR is a
required part of a GPT.

With the standardization on 0/-1 return values for helper
functions, rename 'valid' to 'error' in GPT_read() to make logic
less mind bending.

No functional change.

Remove hackery around /tmp/i/resolv.conf*.  In particular "lookup file bind"
is required because of TLS servername for contacting ftp.openbsd.org, and
there is no point doing that in resolv.conf.tail because it is no longer used.
ok florian kn

The IPsec authentication before decryption used a different replay
counter than after decryption.  This could result in "esp_input_cb:
authentication failed for packet in SA" errors.  As we run crypto
operations async, thousands of packets are stored in the crypto
task.  During the queueing the replay counter of the tdb can change.
Then the higher 32 bits may increment although the lower 32 bits
did not wrap.
checkreplaywindow() must be called twice per packet with the same
replay counter.  Store the value in struct tdb_crypto while dangling
in the task queue and doing crypto operations.
tested by Hrvoje Popovski; joint work with tobhe@

Do not ask iwx(4) firmware to send probe requests on passive channels.

This seems to fix a problem where firmware becomes unresponsive after
association and eventually raises a fatal error. iwx(4) already has a
workaround in place for this: We always ask firmware to perform a
passive scan, on any channel.
This change is a better fix from iwm(4) CVS commit jJFMeXcos9GOqFQz.
However, the current workaround will stay in place for now, until we
have evidence that active scanning is working on iwx(4) devices.
I will try to organize an out-of-tree round of testing for this.

Put MBR_parse() invocation inside MBR_read() instead of always
following an invocation of MBR_read() with a call to MBR_parse().

No functional change.

Clear the "persistence bit" on iwx(4) devices during hardware init.

According to iwlwifi commit messages this fixes an edge case where
devices fail to resume after system suspend.
See Linux commit 8954e1eb2270fa2effffd031b4839253952c76f2

Same fix was made for iwm(4) in CVS commit x0XTNdEmudy5oBR4

Ignore routers option when a classless static routes option is present
as mandated by RFC3442.
Pointed out by, initial diff, testing & OK bket@

Set MIMO-related flags in the iwx(4) ADD_STA command.

Not sure how we ended up missing these. MIMO apparently works without the
flags set, and setting them doesn't seem to have any immediately visible
effect on firmware. But let's be consistent with iwm(4) and iwlwifi.

Newer iwx(4) firmware doesn't like the DQA command anymore.

Check whether firmware advertises DQA support before sending the DQA
command during device initialization. Newer firmware will raise a
fatal error otherwise. The Tx queue API has been redesigned yet again.

Fix wrong usage of iwx_lookup_cmd_ver() in iwx_send_soc_conf().

Callers are supposed to check whether iwx_lookup_cmd_ver() returns
CMD_VER_UNKNOWN, and this check was missing here. Fortunately, the
buggy check was part of a condition which also requires the
low_latency_xtal constant to be set. We do not yet support devices
where low_latency_xtal is non-zero, so the bug never triggered.

vmd(8): remove invalid errno values from config_setvm

Refactor config_setvm to directly return error code on failure
instead of returning -1 and setting errno. It was setting unsupported
values not defined in <errno.h>.

OK mlarkin@

Support auto-tagging for ".It Va".

This combination is somewhat rare because few libraries expose so many
global variables that they need a list to enumerate them, but when the
idiom does occur, tagging the variable names is generally useful.
For example, this helps awk(1), dc(1), make(1), rc.subr(8), ...

Missing feature reported and patch reviewed, tested, and OK'ed by kn@.

Fix iwx(4) PHY context updates for newer firmware versions.

Firmware which advertises the BINDING_CDB_SUPPORT capability
needs a remove+add dance when the channel band has changed.

See Linux commit 730a18912bcbde0b94ae7f1b554a9908b3424a22
and Linux commit 91109f42d0ad0c0c282d1fa1257a1548977aa895

The same fix was applied to iwm(4) in CVS commit uQ0WjqRUp03vxHg1

Let the mandoc.db(5) test suite work with parallel make (make -j),
by making sure that different tests use different directory names
for their work such that they do not collide.
As a side benefit, this allows keeping the complete working directories
of the tests until "make cleandir" is run, which may occasionally
help debugging when something breaks.

The failure with make -j was detected and reported by anton@,
who also reviewed, tested, and OK'ed this somewhat lengthy patch.

Document dhcpleased(8) and slaacd(8) besides dhclient(8) as DNS source

Zap the obsolete dhclient.conf(5) supersede quirk while here.

Feedback OK florian

Follow dhclient -> dhcpleased switch, reduce supported DHCP statements

dhcpleased(8) does not support every statement dhclient(8) does, so reflect
the status quo simplify things to the smaller set of statements that still
suffice to use autoinstall(8).

While here, speak of dhcp-options(5) and "DHCP lease" instead of
dhcpd.conf statements and the installer's specific DHCP client/parser.

OK florian

make the printf on skipping reset with gen7/gen8 debug only

With the 5.10 drm on vlv/ivb/hsw/bdw inteldrm has been known to reset
the chip when a gpu hang is falsely detected.  This appears to be
related to the workarounds for cleaning up after switching hardware
contexts.

intel_gt_reset *NOTICE* [drm] Resetting chip for context closure in glsl-uniform-int<99177>
mark_guilty *NOTICE* [drm] glsl-uniform-int[97885] context reset due to GPU hang

A local change skips the reset (which kills Xorg) on gen7/gen8.
The printf is in that path.

Update the list of firmware files used by iwm(4).

regen

add devices from ASUS ROG Strix G513QY and Alienware Aurora R10

The first think MBR_init() does is memset(0) the gpt header and gpt
partition array. So no need to memset(0) the gpt header before each
MBR_init() invocation.

No functional change.

Remove setpassent(3) leftovers

Those should have gone with the switch to uid_from_user(3) in r1.95;

OK millert

Don't document systems which rewrite/change resolv.conf.  Those systems can
document their actions in their own manual pages.
ok kn

Follow up on dhclient(8) to dhcpleased(8)/resolvd(8) switch

hostname.if(5)'s "dhcp" no longer triggers dhclient in any way, so suggest
the simple "inet autoconf" instead while mentioning "dhcp" as an alias for
that.

Add markup to changed sentences while here.
There is more room for improvemen in this manual.

Feedback deraadt, "go ahead"

Make the mandoc.db(5) tests work with BSDOBJDIR set,
by stripping initial parts of paths the do not matter
for the features being tested from man(1) output.

Problem found and reported by, solution developed together with, and
OK anton@

Nuke a bunch of Capt. Obvious or misleading comments.

No functional change.

Change edit/gedit/setpid/gsetpid to return 0/-1 to indicate success or failure
as all other helpers do.

Makes it clear that X* functions are in charge of deciding which CMD_* value
they return to main edting loop. Reduces unnecessary copying/restoring of
original partition information.

No intentional functional change.

Ignore unwind socket with kevent(2) under SMALL

All other unwind related bits are under #ifndef SMALL already,
so complete that.

OK florian

wrap some long lines

fix sftp on ControlPersist connections, broken by recent SessionType
change; spotted by sthen@

sync

drm/amdgpu: add another Renoir DID

From Jinzhou Su
775da83005cb61d4c213c636df9337da05714ff1 in mainline linux

amdgpu: Add Missing Sienna Cichlid DID

From Ori Messinger
d26bbbcc160f6d9feabed73dca62b9e8b86671b4 in mainline linux

Change PRT_protected_guid() to take native order uuid's instead of little endian
uuid's. This brings it into line with all the other PRT_ functions taking uuid
parameters.

While adapting gsetid(), introduce separate partition type and partition guid
variables to make the logic clearer.

No intentional functional change.

vmd(8): simplify vcpu logic, removing uart & vionet reads

Remove legacy state handling on the ns8250 and virtio network devices
originally put in place before using libevent for async device
events. The vcpu thread doesn't need to process device data as it is
handled by the libevent thread.

This has the benefit of simplifying some of the message passing
between threads introduced to the ns8250 uart since both the vcpu
and libevent threads were processing read events.

No functional change intended. Tested by many, including abieber@,
weerd@, Mischa Peters, and Matthias Schmidt. (Thanks.)

OK mlarkin@

this media can also use dhcpleased now

Switch from dhclient to dhcpleased in the installer.
More cleanup is needed from a real ksh hacker, but looks already
pretty good to kn.

OK deraadt

Create working directories earlier so that resolvd can write
resolv.conf.shadow and start dhcpleased/resolvd.
OK deraadt

Replace dhclient with dhcpleased/resolvd on the ramdisk.
OK deraadt who also helped making some space.

create lease directory for dhcpleased in the ramdisk
OK deraadt

The installer plays weird tricks with /etc/resolv.conf and symlinks it
from /tmp/i. Work around this in resolvd so that it can correctly
update the file until this is fixed in install.sub.

probably grudgingly OK deraadt

switch to dhcpleased/resolvd in base
OK deraadt

Improve comments in IPsec replay window calculation.
OK tobhe@

Update the default block size in the manual, in sndiod.c r1.37 it was
reduced to 480 frames. ok ratchov@

main proc doesn't speak with sockets during the main loop and setpriority plus
chroot (and privdrop) in the child procs at this point are all set and done so
pledge("inet id") promises are not required anymore.

analysis and OK @kn

/etc/ssl/cert.pem is loaded into mem with tls_load_file(3) on local function
priv_constraint_child() which is called before we reach pledge(2), therefore we
we don't need to unveil(2) that file nor having pledge("rpath") since it was
only required to read that same file.

OK kn@

Remove unneeded call to tls_init(3)

As per the manual and lib/libtls/tls.c revision 1.79 from 2018
"Automatically handle library initialisation for libtls." initialisation
is handled automatically by other tls_*(3) functions.

Remove the tls_init() call to not give the impression of it being needed.

Feedback tb
OK Tests mestre

Compare pointer to NULL, not 0.

Be consistent and always return -1 to indicate failure.

DISK_printgeometry() return value was always 0 and never
checked so just make it void.

No functional change.

regen

add some x570 ppbs and another navi 23

recognise more pci capabilities
from PCI Code and ID Assignment Specification

Explicitly check for and start time-based rekeying in the client
and server mainloops.

Previously the rekey timeout could expire but rekeying would not start
until a packet was sent or received. This could cause us to spin in
select() on the rekey timeout if the connection was quiet.

ok markus@

Remove the unveil current directory pointer from struct process. Instead
pass in the vnode to unveil_start_relative() like it is done for *at()
syscalls. This fixes an issue with fchdir() that actually did not correctly
reset this pointer when changing the working directory.
OK beck@

Nuke unused variable.

No need for separate b_sectors, b_offset and b_type variables. We have struct
prt to hold this info in one place.

Construct a struct prt to hold the boot partition information and stash it in
the struct disk for both MBR and GPT to access as needed.

Move the blocks to sectors conversions into DISK_open() with all of its
geometry friends.

No intentional functional change.

Consolidate disk geometry fiddling, turning -l/-c/-h/-s values into MBR
partition geometry info, into DISK_open(). Resulting in clearer logic.

No intentional functional change.

more space savings

space savings on install kernels.  (restore and ralink drivers+firmware go
away)

Wrap over 80 long lines in ca.c

Explicitly check pointer variable if it is NULL or not in ca.c

Fix regress for rpki-client

spotted by bluhm

Remove space between '*' and pointer variable in ca.c

Use 'serial' rather than 'ser' in ca.c

input from jsing@

Convert openssl(1) ca option handling

New option handling for openssl(1) ca.
This diff is just replacing with new option handling, no functional change.
I'm using the word DN or RDN in description as manual uses them, rather than
replacing with "Distinguished Name" or "Relative Distinguished Name".

I would like to add another fixes below by follow-up diffs.
- remove space between '*' and pointer variable
- wrap 80+ long lines
- explicitly check pointer variable if it is NULL or not

comments and ok from jsing@

UNVEIL_INSPECT is no longer needed, adjust code accordingly.
OK semarie@

drm/amd/dc: Fix a missing check bug in dm_dp_mst_detect()

From Yingjie Wang
5885fce7b43919406b32d1c942c9b496aac3abbc in linux 5.10.y/5.10.50
655c0ed19772d92c9665ed08bdc5202acc096dda in mainline linux

drm/bridge: Fix the stop condition of drm_bridge_chain_pre_enable()

From Douglas Anderson
0cd39c96574d19e234cee971868c82c3769c2ee6 in linux 5.10.y/5.10.50
bab5cca7e609952b069a550e39fe4893149fb658 in mainline linux

Export SA replay counters via pfkey and print with ipsecctl.
This is useful for debugging replay window issues with 64 bit
sequence numbers in IPsec.

ok bluhm@

After VFS shutdown, init(8) cannot map a missing page that contains
the signal handler code.  Traditionally a process would spin in
such a case, but we changed the logic in revision 1.167 trapsignal()
to receive a fatal signal.  If that happens to init(8), the kernel
panics.  In case of reboot, jump between init signal handler and
page fault trap until the kernel resets the machine.
reported and tested weerd@; OK deraadt@

Resend the TCP packet only if the MTU locked flag appears at the
route and was not there before.  This should prevent a recursion
in path MTU discovery with TCP over IPsec.
reported and tested Matthias Schmidt; tested and OK tobhe@

Shrink cluster size for FAT32 to fullfil the new minimum cluster
limit of newfs_msdos for our small 64 MB vnd disk image.
suggested by jsg@

LibreSSL error message has changed, adapt test.

Remove unneeded calls to tls_init(3)

As per the manual and lib/libtls/tls.c revision 1.79 from 2018
"Automatically handle library initialisation for libtls." initialisation
is handled automatically by other tls_*(3) functions.

Remove explicit tls_init() calls from base to not give the impression of
it being needed.

Feedback tb
OK Tests mestre

Restructure the getopt_long options. Sort them mostly by name with the
exception of no-xyz options that are grouped with the coresponding xyz
option. Fix --no-motd to use the internal flag setting of getopt_long.
Also use some defines instead of numbers for pure long options that
need special handling.
OK benno@

timeout_add() always sets a new expiry time unrelated to the previous
one and requeues as needed so no need to call timeout_del() before
timeout_add()

spotted by kettenis@

Move default value for TERM into tmux.h.

reorder SessionType; ok djm

When mod_timer() is called with a value less than or equal to the
current number of jiffies delete the timeout and reschedule for the next
tick.  Avoids timeout_add() failing the ticks >= 0 assertion when
mod_timer() is called from i915_utils.c set_timer_ms().

Reported and earlier version tested by Tom Murphy on Kaby Lake R.

Adjust expected certificate error message. Caused by recent work to support
partial chains in libcrypto.

ok bluhm@