Fix (a kind of) violation of strict aliasing rule.

Due to the rule, "sin" and "sin6" can be treated as restrict pointers.
Compilers seem to be confused by structure copy for those pointed by
them before assignments.

For aarch64eb, GCC 9 and 10 compile t_tcp.c rev 1.11 into a code, where
fetch for "sin6->sin6_port" is preceding the structure copy "ss = bs".
This results in failure of connect(2) with EADDRNOOTAVAIL.

(rin)

acpi_print_gicc_flags: print {en,dis}abled

(skrll)

remove unused ioctl command named SIOCGLAGGPORT
to get status of l2tp(4) added to lagg

NOTE:
SIOCGLAGGPORT is based on FreeBSD implementation.
And, currently, it is not used in NetBSD kernel/userland.

(yamaguchi)

Added tests for lagg(4) about MAC addresses

(yamaguchi)

lagg: renew MAC addresses to change the value of interface type

The interface type(ifnet::if_type) is changed on adding to lagg(4)
and deleting from it.

(yamaguchi)

Update the MAC address of all child interface
when that of lagg is changed.

(yamaguchi)

fix various typos, mainly s/prefered/preferred/

(andvar)

tests/indent: document quick-and-dirty approach of lex_number

(rillig)

tests/indent: test identifiers containing '$'

(rillig)

Handle RGMII variants.

(jmcneill)

indent: fix handling of C99 comments after 'if (expr)'

(rillig)

indent: demonstrate controlled buffer overflow in C99 comment

(rillig)

indent: demonstrate disappearing form feed

(rillig)

tests/indent: fix test for form feed

Previously, I had written a vertical tab instead of a form feed, which
had a different effect.

(rillig)

indent: various cleanups

Make several comments more precise.

Rename process_end_of_file to process_eof to match the token name.

Change the order of assignments in analyze_comment to keep the com_ind
computations closer together.

In copy_comment_wrap, use pointer difference instead of pointer addition
to stay away from undefined behavior.

No functional change.

(rillig)

indent: parse special options strictly

(rillig)

Fix linking for multi-source file tests, the second part of the tuple
needs to be the C source file (not the base name)

(martin)

dts: Import dts files from Linux 5.15.

(jmcneill)

dts updated to 5.15

(jmcneill)

cwfg: update for dts-5.15 binding changes

(jmcneill)

Look for child node by compat string snps,dwc3 instead of by name.

(jmcneill)

pwm_backlight: pick defaults if none are provided

(jmcneill)

fdt: add helper for finding intr parent phandle

(jmcneill)

sunxi: nmi: add support for #interrupt-cells 3 bindings

(jmcneill)

sunxi: sun50i-a64: add support for A64_CLK_CPUX clock

(jmcneill)

sunxi: ccu: add support for basic "mux" clocks

(jmcneill)

ti: cpsw: adapt to dts-5.15 bindings

(jmcneill)

arm: ti: adapt to dts-5.15 bindings

(jmcneill)

pinctrl-single: support #pinctrl-cells 2

(jmcneill)

fdtbus: match simple-pm-bus

(jmcneill)

dts: adapt to dts-5.15 bindings

(jmcneill)

arm: dts: adapt for dts-5.15

(jmcneill)

dtb: Re-generate set lists for 5.15

(jmcneill)

Extra dts files no longer required.

(jmcneill)

CONFIG_ARCH_SOCFPGA has been renamed CONFIG_ARCH_INTEL_SOCFPGA

(jmcneill)

Merge conflicts.

(jmcneill)

tests/indent: test lenient option parsing

(rillig)

new tests for posix_spawn_chdir from Piyush Sachdeva

(christos)

tests/indent: test cast, line counting, comment in struct

(rillig)

indent: rename ps.decl_nest to decl_level

This better matches the comment.

No functional change.

(rillig)

tests/indent: test options without '-' in profile files

(rillig)

Commit the userland portion of the posix_spawn_chdir project by Piyush Sachdeva

(christos)

indent: move ps.p_l_follow closer to lsym_type_outside_parentheses

This makes it easier to see the relation between these two.

No functional change.

(rillig)

indent: reduce negations in process_else, clean up comments

No functional change.

(rillig)

Merge the kernel portion of the posix-spawn-chdir project by Piyush Sachdeva.

(christos)

indent: document the comment buffer more accurately

(rillig)

indent: remove code that accessed out-of-bounds data from buffer

Saving and restoring the exact buffer position had been necessary before
NetBSD pr_comment.c 1.114. The code accessed the buffer data out of the
bounds of [com.s, com.e), which was rather surprising. More
specifically, it accessed com.e[-1] in a case where com.e == com.s,
relying on com.e[-1] being ' ' in most cases and '*' in the case where
the comment delimiter was written to a separate output line.

Make the code easier understandable by only ever accessing the buffer
data in the bounds [buf.s, buf.e).

No functional change.

(rillig)

indent: only access buffer data in the range [buf.s, buf.e)

No functional change.

(rillig)

indent: do not expand comment buffer unnecessarily

This may have been a simple typo or a really tricky optimization that
isn't obvious even after studying the code for several months. Either of
these is bad, so use the standard form of resetting the buffer.

No functional change.

(rillig)

tests/indent: improve code coverage for processing comments

(rillig)

indent: remove dead code in analyze_comment

The case of an otherwise empty line is already handled further above.

(rillig)

indent: remove dead code from copy_comment_wrap

C99 comments are not wrapped, therefore there is no need to check for
them in copy_comment_wrap.

No functional change.

(rillig)

indent: move documentation from process_comment to copy_comment_wrap

(rillig)

indent: clean up process_comment

The assignments to the variables were redundant, they are already done
by analyze_comment.

No functional change.

(rillig)

indent: remove redundant assignment

At that point, ps.next_col_1 is already false.

No functional change.

(rillig)

indent: make end of comment detection of nowrap comments simpler

No functional change.

(rillig)

indent: clean up overcomplicated conditions in copy_comment_nowrap

No functional change.

(rillig)

indent: skip redundant conditions in copy_comment_nowrap

No functional change.

(rillig)

indent: clean up copy_comment_nowrap

The action for '\f' was the same as the default action.

Replacing 'switch' with 'if' makes the code shorter.

No functional change.

(rillig)

indent: make copy_comment_nowrap simpler

Since a nowrap comment is copied unmodified, it need not depend on any
maximum line length.

No functional change.

(rillig)

fix few typos.

(andvar)

Libtsan intercepts the pthread exported functions with
dlsym(RTLD_NEXT, func_name).  This fails for pthread_join() as it
is present both as a stub in libc and as real operation in libpthread
and the search order is "libtsan.so libc.so libpthread.so".

Force the order "libtsan.so libpthread.so libc.so" so symbols
get intercepted from libpthread first.

Ok: Matthew Green <mrg@netbsd.org>

(hannken)

indent: remove dead code from process_comment_nowrap

In comments that are preserved, no additional leading ' * ' is inserted.

No functional change.

(rillig)

indent: remove dead code from copy_comment_wrap

No functional change.

(rillig)

indent: remove dead code from copy_comment_nowrap

No functional change.

(rillig)

indent: split copy_comment into wrapping and non-wrapping

These two cases are processed in an almost entirely different way. In
particular, copy_comment_nowrap should copy the comment verbatim, which
is not obvious from the current code, due to the many conditions and the
complex control flow.

No functional change.

(rillig)

tests/indent: demonstrate buggy combination of '-eei' and '-nlp'

Seen in indent.c.

(rillig)

indent: rename type_at_paren_level_0 to type_outside_parentheses

For symmetry with type_in_parentheses.

No functional change.

(rillig)

indent: distinguish between typename in parentheses and other words

This gets rid of two members of parser_state. No functional change for
well-formed programs. The sequence of '++int' or '--size_t' may be
formatted differently than before, but no program is expected to contain
that sequence.

Rename lsym_ident to lsym_word since 'ident' was too specific. This
token type is used for constants and string literals as well. Strictly
speaking, a string literal is not a word, but at least it's better than
before.

(rillig)

indent: rename 'inbuf' functions to 'inp'

The variable 'inp' used to be named 'inbuf'. Make the function names
correspond to the variable name again.

No functional change.

(rillig)

Add a comment describing why we give carte blanche to processes
marked as PK_32.  NFC.

(thorpej)

pslist(9): No need to serialize pserialize_perform any more.

So take it out of the mutex section.

(riastradh)

Set the path unconditionally to what rc sets it to, so that when running
individual scripts like 'env - /etc/rc.d/syslogd restart' programs like
rcorder are found.

(christos)

s/invlaid/invalid/

(andvar)

COMPAT_NETBSD32 is all about running the 32-bit flavor of native
binaries on a 64-bit platform[*], as such:
- Make the logic about which "sendsig" flavor to call MI (as it is in the
  native 64-bit environment) and follow the same rules as the native 32-bit
  environment.
- Make COMPAT_NETBSD32 x COMPAT_16 work the same as it would in the
  native 32-bit environment by providing a netbsd32_sendsig_sigcontext_16_hook,
  rather than overriding the entire sendsig logic with a netbsd32_sendsig_hook.
- In netbsd32___sigaction_sigtramp(), make sure the compat_netbsd32_16
  module is loaded if the trampoline version specifies a sigcontext style
  handler, otherwise return EINVAL so that libc can try again with siginfo
  style.

[*] ...except for arm32, which uses it to mean "run 32-bit OABI binaries
from the 32-bit EABI environment".  Doing it this way was arguably a mistake,
but we are stuck with it for now, so support it by providing a machine-
dependent override for netbsd32_sendsig() that also disables the corresponding
logic in netbsd32___sigaction_sigtramp().

Fixes PR kern/56487.

(thorpej)

Revert part of previous commit that broke DT booting.

(jmcneill)

New sentence, new line; remove trailing whitespace.

(wiz)

Tickets #1371 and #1372

(martin)

Pull up following revision(s) (requested by kre in ticket #1372):

bin/sh/sh.1: revision 1.236 (patch)
bin/sh/cd.c: revision 1.51

PR bin/45390 - fix for folly four

In the pwd builtin, verify that curdir names '.' before
simply printing it.  Never alter PWD or OLDPWD in the
pwd command.

Also while here, implement the (new: coming in POSIX, but has existed
for a while in several other shells) -e option to cd (with -e, cd -P
will exit(1) if the chdir() succeeds, but PWD cannot be discovered).
cd now prints the directory name used (if different from that given,
or cdprint is on) if interactive or (the new bit)in posix mode.

Some additional/changed comments added, and a DEBUG mode trace call
that was accidentally put inside an #if 0 block moved to where it
can do some good.

XXX pullup -9

PR bin/45390

Be explicit about what happens to PWD after a successful cd command.
Also be very clear  that "cd" and "cd -P" are the same thing, and
the only cd variant implemented.
Also, when it is appropriate to print the new directory after a cd
command, note that it happens if interactive (as it always has here)
and also if the posix option is set (for POSIX compat, where "interactive"
is irrelevant).  Mention that "cd -" is a case where the new directory
is printed (along with paths relative to a non-empty CDPATH entry,
and where the "cd old new" (string replacement in curdir) is used.

While here document the new -e option to cd.

XXX pullup -9

(martin)

Mention sht3xtemp(4)

(brad)

Mention sht3xtemp(4)

(brad)

Pull up following revision(s) (requested by kre in ticket #1371):

bin/sh/main.c: revision 1.87
bin/sh/main.c: revision 1.88
bin/sh/memalloc.h: revision 1.20
bin/sh/sh.1: revision 1.235
bin/sh/memalloc.c: revision 1.34
bin/sh/memalloc.c: revision 1.35
bin/sh/memalloc.h: revision 1.19
bin/sh/shell.h: revision 1.31
bin/sh/options.c: revision 1.56

PR bin/56464

After almost 30 years, finally do the right thing and read $HOME/.profile
rather than .profile in the initial directory (it was that way in version
1.1 ...)  All other ash descendants seem to have fixed this long ago.
While here, copy a feature from FreeBSD which allows "set +p" (if a
shell run by a setuid process with the -p flag is privileged) to reset
the privileges.  Once done (the set +p) it cannot be undone (a later
set -p sets the 'p' flag, but that's all it does) - that just becomes a
one bit storage location.

We do this, as (also copying from FreeBSD, and because it is the right
thing to do) we don't run .profile in a privileged shell - FreeBSD run
/etc/suid_profile in that case (not a good name, it also applies to setgid
shells) but I see no real need for that, we run /etc/profile in any case,
anything that would go in /etc/suid_profile can just go in /etc/profile
instead (with suitable guards so the commands only run in priv'd shells).

One or two minor DEBUG mode changes (notably having priv'd shells identify
themselves in the DEBUG trace) and sh.1 changes with doc of the "set +p"
change, the effect that has on $PSc and a few other wording tweaks.

XXX pullup -9  (not -8, this isn't worth it for the short lifetime
that has left - if it took 28+ years for anyone to notice this, it
cannot be having all that much effect).

Use a type-correct end marker for strstrcat() rather than NULL, as
for a function with unknown number & types of args, the compiler isn't
able to automatically convert to the correct type.  Issue pointed out
in off list e-mail by Rolland Illig ... Thanks.

The first arg (pointer to where to put length of result) is of a known
type, so doesn't have the same issue - we can keep using NULL for that
one when the length isn't needed.
Also, make sure to return a correctly null terminated null string in
the (absurd) case that there are no non-null args to strstrcat() (though
there are much better ways to generate "" on the stack).  Since there is
currently just one call in the code, and it has real string args, this
isn't an issue for now, but who knows, some day.

NFCI - if there is any real change, then it is a change that is required.

XXX pullup -9 (together with the previous changes)

(martin)

Driver for the Sensirion SHT30/SHT31/SHT35 temperature and humidity
sensor such as:

https://www.adafruit.com/product/2857

This is a higher priced sensor with a lot of features, including the
ability to do sub-second periodic updates.  The driver supports
everything about the sensor except for the alert pin.

(brad)

s/allign/align/

(msaitoh)

Fix typo in comment. s/phyiscally/physically/

(msaitoh)

USB style. NFC.

(skrll)

Fix typo in comment. s/notifcation/notification/

(msaitoh)

config_pending_incr doesn't need KERNEL_LOCK protection

(skrll)

s/writting/writing/

(msaitoh)

s/maxiumum/maximum/

(msaitoh)

pax.1: minor grammar fixes

(gutteridge)

mcx(4): ensure that RQT size is always a power of two.
Fixes PR kern/56484.
Thanks riastradh for the power of two patch :). OK: mrg, jmcneill, riastradh.

(andvar)

Rework ilog2(3) man page.

- Tweak title.
- Clarify semantics -- round down, not any other way.
- xref ffs(3)
- Fix return type: int.
- Fix language about input type.
  . Any unsigned type is fine.
  . Types smaller than uint32_t will be safely converted to uint32_t.
- Note that it is a constant expression if x is constant.

(riastradh)

indent: consistently use token.e[-1] for the last added character

No functional change.

(rillig)

indent: clean up argument parsing

In struct pro, place the dependent member below its dependency.

In load_profile, consistently use -1 when outside a comment.

No functional change.

(rillig)

indent: the '+ 1' in dump_line_code is not an off-by-one error

(rillig)

indent: add debug output for remaining members of parser_status

(rillig)

indent: rename process_keyword_do to process_do, same for 'else'

Before the symbols from the tokenizer had the prefix 'lsym', the symbols
could not be simply called 'else' and 'do'. The functions for processing
the tokens followed that naming scheme.

When the prefix 'lsym' was introduced, the word 'keyword' was no longer
needed, neither in the constants nor in the function names.

No functional change.

(rillig)

indent: rename ps.curr_newline to next_col_1

For symmetry with ps.curr_col_1.

No functional change.

(rillig)

Normally, to support COMPAT_NETBSD32 we need to define
__HAVE_STRUCT_SIGCONTEXT in order to support the old
"sigcontext" style of handlers for 32-bit binaries.
However, we only support 32-bit EABI binaries on AArch64,
and by happy accident (due to a libc bug introduced in
2006), 32-bit NetBSD EABI binaries never used "sigcontext"
style handlers.  So, we don't need to carry any of this
baggage forward.

This addresses the AArch64 case of PR kern/56487.

(thorpej)

Apply patch from PR port-evbarm/52209

Gumstix: KDASSERT in pxa2x0_gpio if no PCMCIA  bushead given but present

(skrll)

Fix comment. No functional change.

(msaitoh)

Fix a bug that device timeout still happens on ICH/PCH. Fixes PR kern/56478.

- if_wm.c rev. 1.695 for PR kern/40981 still had a problem. The workaround
  flag was unexpectedly cleared when a cable is not connected, SIOCINITIFADDR
  was called, or if_init was called. Fix it.
- Add debug printf()s.

(msaitoh)

Don't power down the PHY when the interface goes down.

- All of other PHY drivers don't power down the PHY. Do the same way.
- At least, keeping the link is required for Intel AMT and WoL.

(msaitoh)

Use macro. No functional change.

(msaitoh)

make compilable at NPCI==0

(jakllsch)

indent: split process_comment_in_code into separate functions

No functional change.

(rillig)

indent: do not discard former error comments anymore

Since io.c 1.20 from 2019-10-19, indent has not placed error comments in
the code anymore. Since these comments are supposed to be cleaned up
immediately, there is no point in having code for handling them.

(rillig)

indent: fix parsing of C99 comments containing '*/'

(rillig)

tests/indent: demonstrate wrong edge case in C99 comments

(rillig)

indent: split process_comments into separate functions

No functional change.

(rillig)

indent: fix joining of adjacent unary '+' operators

(rillig)

indent: extract compute_code_indent_lineup into separate function

Having 9 different paths in a single function made it more complicated
to understand than necessary.

No functional change.

(rillig)

indent: fix off-by-one confusion in paren_indent

The variable was called 'indent' but actually contained a 'column',
which was off by one.

No functional change.

(rillig)

indent: replace column computation with indentation computation

No functional change.

(rillig)

Recognize argon2 passwords as valid in daily security reports.

from RVP in misc/56486

(nia)

Add some sysctl info for debugging.

(msaitoh)

Fix non-ACPI builds.

(skrll)

Remove unnecessary duplicate bit define in mask

PR port-arm/50687

(skrll)

indent: group conditions in compute_code_indent by topic

No functional change.

(rillig)

Provide the ablity to ignore ACPI with an 'acpi' command:

acpi [{on|off}]

(skrll)

indent: inline indentation_after, shorten function name to ind_add

There were only few calls to indentation_after, so inlining it spares
the need to look at yet another function definition. Another effect is
that code.s and code.e appear in the code as a pair now, instead of a
single code.s, making the scope of the function call obvious.

In ind_add, there is no need to check for '\0' anymore since none of the
buffers can ever contain a null character, these are filtered out by
inbuf_read_line.

No functional change.

(rillig)

usbhid(3): use reallocarr

(nia)

Regen.

(msaitoh)

- Rename EHL_TRACE* macros
- Update Jasper Lake's Processor Transaction Routers.

(msaitoh)

futex_release_all_lwp(): No need to pass the "tid" argument separately; that
is a vestige of an older version of the code.  Also, move a KASSERT() that
both futex_release_all_lwp() call sites had inside of futex_release_all_lwp()
itself.

...so make this easier to test this sys_futex.c with trunk.

(thorpej)

fdisk(8): Convert realloc(x * y) to reallocarr. Eliminate temp variables.

(nia)

disklabel(8): convert malloc(x * y) to reallocarr

(nia)

Add MIPSSIM{,64} as appropriate

(skrll)

Catch up with member renaming

(skrll)

CFI: Saved RA needs to point to the PC slot in the ucontext_t.

(thorpej)

Set up the CFI in a slightly more rational way.

(thorpej)

Update glob example from X11R6 to X11R7

(abs)

Update Makefile comment from X11R6 to X11R7

(abs)

Update x11-ssh-askman manpage app-defaults paths to X11R7

Now matches actual code

(abs)

Update some distrib X11R6 references to X11R7

Some of these scripts and notes may no longer be relevant, but now
they are at least not relevant while referring to the current X
installation directory

(abs)

Adjust example path to kde/sounds

Wherever sounds may be on a current NetBSD system, /usr/X11R6 is
unlikely to be in that set...

(abs)

unconditionally define sockaddr_storage; this is what linux and FreeBSD do
and it has been there since Issue 6:
https://pubs.opengroup.org/onlinepubs/009604599/basedefs/sys/socket.h.html

(christos)

Expose sigaltstack, stack_t and friends to Posix Issue 5 (200809L) References:

https://man7.org/linux/man-pages/man2/sigaltstack.2.html
https://pubs.opengroup.org/onlinepubs/9699919799/functions/sigaltstack.html

(christos)

In order to prevent _mcount() from being recursively called when built with COPTS=-O0,
sprinkle `__always_inline' to make _mcount() be generated as a single function.

(ryo)

fix profiling kernel build

(ryo)

fix build with COPTS=-O0

(ryo)

Hopefully fix builds

(skrll)

join(1): convert realloc(x * y) to reallocarr and eliminate temporary
variables

(nia)

fts(3): Use reallocarr instead of realloc(x * y), eliminate a temporary
variable.

(nia)

ed(1): use reallocarr instead of realloc(x * y)

(nia)

Adjust for new __sigaction_siginfo symbol.

(thorpej)

Added tests of combination of lagg(4), vlan(4), and l2tp(4)

(yamaguchi)

Use IPv6 addresses, not IPv4, in combination test of IPv6, lagg and vlan

(yamaguchi)

indent: fix missing blank after 'return' (since 2021-10-31)

In indent.c 1.200 from 2021-10-31, the subtypes of identifier tokens
were removed since they were redundant. An unintended side effect was
that a parenthesized expression after 'return' was no longer separated
by a blank.

Before that change, 'return' was tokenized as an lsym_ident with subtype
kw_other, and want_space_before_lparen handled this case in the last
line. After the change, 'return' was treated as an ordinary identifier,
and unless the option '-pcs' (blank after function call) was given, the
blank was removed.

The other keywords that had kw_other are not affected since they do not
expect a '(' afterwards. These keywords are 'break', 'continue', 'goto',
'inline' and 'restrict'.

Curiously, there was not a single test case that covered 'return(expr)'.

While here, remove the trailing ',' from the enum lexer_symbol, which is
not allowed in standard C, it is a GNU extension. Lint doesn't complain
about this since the default LINTFLAGS include '-g' for GCC mode.

(rillig)

tests/indent: test option '-ci'

There are quite a few tricky special cases, but as they are all listed
in the manual page, they are not surprising.

(rillig)

Force at most partial RELRO for rump_server and related libraries

rump_server with -l uses lazy resolution by default and thus cannot be
used with full RELRO/BIND_NOW.

(nia)

fix typos, mainly in words minimum and maximum, but also few others.

(andvar)

mail.1: fix duplicate word

(rillig)

lint: enter full C90 compatibility mode

The C99 comment in tree.c:3468 has been there since 2017-03-06, without
anyone complaining that their compiler would not handle it.

Strangely, running GCC 10.3.0 in '-std=c90' mode does not complain about
declarations after statements, '-Wdeclaration-after-statement' is needed
separately.

No functional change.

(rillig)