regtest: add a fdleak filter for write on write on linux arm64

Helgrind: fix unused result of write warnings

Thanks to GCC deciding that we can't ignore wur annotated
functions by casting to void we need to do otherwise. It was
either pragmas or adding an annotated unused local. Pragmas
seem to be the least awful.

Bug 469782 - Valgrind does not support zstd-compressed debug sections

drd: Split handle_client_request()

Make handle_client_request() easier to read by splitting it into two
functions: one for Valgrind core client requests and one for thread-
related client requests.

drd: Reorder functions

Reorder two functions such that a forward declaration can be removed.

drd: Remove a superfluous if-statement

Check the client request ID once instead of twice.

Solaris - fix a couple of warnings for message formatting

Illumos regtest: turn off memcheck/tests/x86/bug487993

For some reason GCC 13.3 on Illumos does not use aligned operator
new when compiling this testcase. So turn it off.

arm: hook kcmp syscall

coregrind/m_gdbserver/remote-utils.c (prepare_resume_reply): Use memcpy

GCC8 (but apparently not later versions) complain about the use of
strncpy when not actually copying a string:

remote-utils.c:1140:14: warning: 'char* strncpy(char*, const char*, size_t)' output truncated before terminating nul copying 6 bytes from a string of the same length [-Wstringop-truncation]
          strncpy (buf, "watch:", 6);
          ~~~~~~~~^~~~~~~~~~~~~~~~~~

This is "harmless" because buf is large enough and we will add more
chars (including a zero terminator) later. But using strncpy here is a
bit odd because we don't really want to copy a string, but an array of
6 chars. So use memcpy here to do so, simplyfing the code.

Illumos regtest: x86 mdb diffs wrt Solaris

vgdb.c (fork_and_exec_valgrind): Fix another off-by-one error write

commit 646978d9adc5 ("vgdb: Handle EINTR and EAGAIN more
consistently") introduced another off-by-one issue trying to write
back the error from child to parent.

Instead of +1 it should have been +written (which initially is zero).

This is when the child needs to do a chdir and that chdir fails. If
that happens the parent would have gotten the wrong error code.

vgdb.c (fork_and_exec_valgrind): Fix off-by-one error write

commit 646978d9adc5 ("vgdb: Handle EINTR and EAGAIN more
consistently") introduced an off-by-one issue trying to write back the
error from child to parent.

Instead of +1 it should have been +written (which initially is zero).

This is in an "should never happen" path, so hopefully didn't really
cause issues. But if it did happen the parent would have gotten the
wrong error code.

Illumos regtest: updates in none/tests/amd64-solaris

Add a filter for an extra level in a callstack.
Add an expected for changed mdb output.

Add 494218 to NEWS

This went in 3.24.

Illumos regtest: Add suppressions to none fdleak_cmsg_supp.supp

Illumos regtest: filter more signal messages from bash

Illumos regtest: a couple of fixes to none fd tests

Illumos regtest: add filters to gdbserver tests

Illumos: Don't call ML_(fd_allowed) from PRE(sys_fstat)

This was causing many false positives with --track-fds=yes

s390x regtest: Add missing memory clobbers in mvc.c

The s390x test case `mvc' was seen to fail when using gcc -O3 with GCC
11.4.1 on Ubuntu:

   mvc: mvc.c:48: main: Assertion `full[i] == 'x'' failed.

In this case the compiler optimized the assertion check away, because it
doesn't consider `full' to be affected by the inline assembly.

Add memory clobbers to fix this.

s390x regtest: Add missing register clobber in misc3.c

Depending on compiler options, the translation of test_mvcrl() in
`misc3.c' can misbehave because the inline assembly writes to r0 and
doesn't declare that.

Add a register clobber to fix this.  Also, tell the compiler the actual
length of `from' and `to' instead of the decremented value.

Compiler warning: unused label on non-FreeBSD platforms

Used by a goto to jump over the POST mutex lock userreq,
but only on FreeBSD.

Helgrind regtest: fix wrong updates to tc04_free_lock expecteds

Bug 494327 - Crash when running Helgrind built with #define TRACE_PTH_FNS 1

Use write() rather than 'fprintf()' for the TRACE_PTH_FNS blocks for
pthread_mutex_lock and pthread_mutex_lock. Mixing FILE and fd isn't
great, but this is to stderr which gets flushed on every line, and
it is only for developer builds that modify that TRACE_PTH_FNS macro.

Helgrind FreeBSD: make previous hack conditonal on version

Don't want to ignore any pthread_mutex_lock calls on FreeBSD
14.1 and earlier.

Bug 494337 - All threaded applications cause still holding lock errors

Don't count the first call to pthread_mutex_lock() from exit().

Needed to update a couple of expecteds as exit() is now wrapped
so it turns up in the callstacks.

This is only for FreeBSD.

FreeBSD regtest: update x86 scalar expected

FreeBSD regtest: spaces not quite right in scalar fakes

clangd fixes

Mainly useless casts and consistent function prototype and
definition arg names.

Correct typos and spelling mistakes

Bug 495488 - Add FreeBSD getrlimitusage syscall wrapper

Fix compiler warnings produced by clang 19.

Add /none/tests/use_after_close to .gitignore

FreeBSD syscall: fix syscall name used in kcmp traces.

FreeBSD regtest: add _write to none filter_fdleak

Also bump up news and configure to 3.25 git.

-> 3.24.0 final

Make const OpenFd *ML_(find_OpenFd) static

DRM_IOCTLs SYNCOBJ_HANDLE_TO_FD, PRIME_HANDLE_TO_FD and MODE_CREATE_LEASE

These three DRM_IOCTLs create new file descriptors, so track them using
ML_(record_fd_open_nameless).

https://bugs.kde.org/show_bug.cgi?id=492422

gsl19test does not work properly when /bin/sh is /bin/dash

There were 2 issues:

1) The output redirect operators seem to be bash specific. All output
   went to the terminal.....

2) When invoking valgrind the 'eval' needs to precede
  GSL_TEST_VERBOSE=1. Otherwise that environment variable is not seen
  by valgrind.

Two tweaks:

1) Determine the number of available processors and use that in make
   -j when building gsl-1.9

2) Replace -v with -q in the valgrind invocation. The file out-V is
   already quite large (approx 600 MB). It would be enormous with -v.

https://bugs.kde.org/show_bug.cgi?id=494960

Report track-fd errors for fd used which was not opened or already closed

Add (optional) pathname, description, where_closed and where_opened
fields to struct FdBadUse. Print those fields when set in fd_pp_Error.

Add a new function ML_(find_OpenFd) that provides a recorded OpenFd
given an fd (or NULL when the fd was never recorded).

In ML_(fd_allowed) when using a file descriptor use ML_(find_OpenFd)
to see if the fd was ever created, if not create an "was never
created" FdBadUse error. If it was created, but already closed create
an "was closed already", filling in as much details as we can.

Add none/tests/use_after_close.vgtest to test, already closed, never
created, invalid, double (double) close and invalid close issues.

Adjust error message in none/tests/fdbaduse.stderr.exp.

https://bugs.kde.org/show_bug.cgi?id=493418

Illumos: fix wartnings

Macro redefinition, calculating the size of an array and printf
format specifiers.

macOS: compiler warnings

Debug traces in vgdb.

Bug 495470 - s390x: Add s390-opcodes.csv to distribution

Include s390-opcodes.csv in the distribution, since it is referenced in
README.s390.

Bug 493959 - s390x: Fix regtest failure for op00 with /bin/dash

On different machines /bin/sh may be impersonated by different shells, and
those behave differently as to whether they write "Illegal instruction ..."
to stderr.  While newer versions of bash do not, dash does.

For the op00 test case this means that an additional line may be written
to `op00.stderr.out', depending on which shell is being used.  Hence
adding "Illegal instruction ..." as an expected line to `op00.stderr.exp'
wouldn't work on all systems.

Instead, fix this issue by adding the case of "illegal instruction" to the
general filtering logic in filter_stderr_basic.in, where various other
messages of this kind are already filtered out.

Reviewed-by: Andreas Arnez <arnez@linux.ibm.com>

macOS Helgrind: fix wrong wrapper for sem_trywait.

Bug 495469 - aligned_alloc and posix_memalign missing MALLOC_TRACE with returned pointer

Linux regtest: update arm64 expected with sigaltstack extra details.

Set version to 3.24.0-RC1

Also update vg-lifespan which is used for documentation copyright years.

FreeBSD regtest: final part of scalar rationalisation, x86 expected

FreeBSD regtest: add fakes for older versions in scalar

Syscalls added for FreeBSD 15 or thereabouts.

FreeBSD regtest: remove scalar for version 15+

Now also merged into scalar.c

FreeBSD regtest: remove test for version 13+ syscalls

Now merged into main scalar.c

FreeBSD regtest: add fakes for FreeBSD < 13

Also try again to make mprotect fail. It doesn't fail
with a bogus 'prot' on FreeBSD 12.4.

FreeBSD regtest: start removing versioned scalar variants

FreeBSD: put back stacktrace hack for syscalls in older versions

FreeBSD regtest: x86 scalar expected

FreeBSD regtest: cleanup scalar

Make more tests fail. Add a few more tests. Add some asserts
for tests that don't return -1 on failure.

FreeBSD x86 scalar: update expected after fix for sigwait

FreeBSD sigwait syscall: fixes for unusual return codes

sigwait doesn't follow the usual syscall convention of
returning -1 on error and setting errno. Instead it
returns errno and 0 means success.

Previously the POST was getting called on failure, potentially
resulting in memory getting incorrectly marked as accessible and
initialized.

Also only flag 'may block' if the first sigset argument is
accessible.

FreeBSD Helgrind: turn off check for locks held on exit for FreeBSD 14.2

thread-safe exit included in FreeBSD 14.2-PRERELEASE

Typos in comments about syscall stacktraces.

PowerPC, dcbf instruction

ISA 2.7 and ISA 3.0 "accepts" L = 0 to 3, ISA 3.1 "accepts: L = 0 to 7.
Removed the L field check so valgrind will match the real hardware.  For
the purposes of Valgrind the dcbf instruction is a NOP anyway so it will
not change the behavior of Valgrind.

Helgrind: use static asserts in client requests

Thanks, clangd.

Bug 494218 - Remove FREEBSD_VERS from configure and build

Bug 445235 - Java/Ada/D demangling is probably broken

Ada demangling is now functional.

There maybe some missing functionality like the _GLOBAL_
mentioned in the bugzilla item.

FreeBSD regtest: add missing helgrind expecteds

FreeBSD helgrind: temporary (?) fix for Bug 494337

FreeBSD 15 has added a pthread lock to exit() to ensure that atexit handling
is thread safe. Unfortunately that lock gets leaked which messes up just about
all of the Helgrind tests.

Supression won't work as the callstack is the same for both genuine leaks
and this deliberate leak.

This change simply turns off the check for FreeBSD >= 15.

I see two possible proper fixes. One would be to allow one lock on exit.
The problem with that is that we will need to tell apart a clean exit
(1 lock allowed) and any kind of abort that doesn't call exit (no locks
allowed). That's going to be tricky as the Helgrind check is done before
we get back to core and know whether it is an abort or a clean exit.

The other thing would be to hack the lock counting. If we can detect that
it's a pthread_mutex_lock called from exit() then we could ignore that for
counting purposes. That would mean a possibly significant overhead for
each call to pthread_mutex_lock on FreeBSD.

Add none/tests/fdbaduse to .gitignore

fd regtest: add filter to new fdbaduse testcase

Add BadFdExtra core error.

Introduce a new FdBadFd type with associated extra info struct.
Which for now just holds the fd number (no path or description).
fd_pp_Error and fd_update_extra have been updated to handle the
new type and produce xml when requested.

Rename showing_core_errors to showing_core_warning
(returns yes when the tools wants to show core errors,
-q isn't given and we aren't producing xml).

In ML_(fd_allowed) we now call VG_(maybe_record_error) to
generate a real error (that can be suppressed and shows up
in the xml output with full execution backtrace). For now
we also produce the legacy warnings when --track-fds=yes
isn't given.

Add none/tests/fdbaduse.vgtest to test the new FdBadUse
core error.

This is the first part of reporting bad fd usage errors.
We are also tracking already closed file descriptors which
should also produce errors like these. The current bad file
descriptors are just those that are negative or above the
current file limit of the process.

https://bugs.kde.org/show_bug.cgi?id=493418

Bug 494252 - s390x: Fix disassembly for LOCHI and friends

The disassembly for LOCHI, LOCGHI, and LOCHHI is bogus.  For instance, the
instruction

    lochi  %r3,7,5

is disassembled as

    lochi  %r5,3,0,.+10

instead of

    lochinhe %r3,7

It seems that s390_format_RIE_RUPIX was copy-pasted from
s390_format_RIS_RUPI without adjusting it appropriately.  Fix this.

Reviewed-by: Andreas Arnez <arnez@linux.ibm.com>

s390x: Add missing early-clobber to GET_STARTREGS

The inline assembly for GET_STARTREGS in m_libcassert.c writes to its
output before using the input argument.  But since the compiler doesn't
know this, it is allowed to allocate the same register for both, causing
problems.  This has been seen when compiling Valgrind with -O0, after
which memcheck/tests/leak-autofreepool-5 fails due to SIGSEGV.

Fix this by declaring the output as early-clobber, so the compiler knows
about the restriction.

Bug 493970 - s390x: Drop saving/restoring FPC upon helper call

Saving the FPC before each helper call and restoring it afterwards creates
unnecessary overhead, and it may also not be desirable.

Drop it.  Also remove the functions in host_s390_defs.c responsible for
emitting LFPC and STFPC instructions.  And since this frees up the FPC
save slot on the stack, adjust the stack layout accordingly.

Fix some issues with GSL for `make auxchecks'

When trying to reproduce Bug 423908, I ran into some trouble performing
`make auxchecks', due to problems in GSL:

Newer compilers complain about incompatible pointer types in argument
passing between

  unsigned int *

and

  size_t *

This affects the fifth argument of function gsl_eigen_jacobi() in
eigen/jacobi.c.  Fix this by passing the right pointer type at invocation.

Also, the configure checks for IEEE comparisons and for IEEE denormalized
values don't work as intended, because they call exit() without declaring
it and thus fail independently from those features being supported.  Fix
this by using `return' instead.

FreeBSD regtest: one more scalar twiddle

Add change to sigaltstack error messages to NEWS.

Solaris regtest: update scalar for sigaltstack changes

macOS regtest: give up trying to build bug492210.c

macOS has to make everything difficult. Global names need
to be decorated with an underscore prefix. And you can't
just read from global variables, you have to do something
like a rip-relative lea.

Linux regtest: update scalar expected for sigaltstack

FreeBSD regtest: update x86 scalar expected

sigaltstack syscall: improve error messages

Previously the same message was generated for the 3 members of
stack_t.

Also on FreeBSD I get a Conditional jump error with this syscall
but not on all platforms, so I've added a suppression.

FreeBSD regtest: fake missing syscalls on arm64 in scalar test

FreeBSD regtest: add FAKE macros for scalar

FreeBSD 15 removed the never-implemented sbrk syscall.
Arm64 also has a few missing syscalls (backward compat ones
that predate the arm64 port). Rather than having an ever
increasing number of expecteds the aim is to use these
FAKE macros. It's a bit fiddly to get the matching text.

FreeBSD regtest: auxv test string map has version dependend size

Implement /proc/self/exe readlink[at] fallback in POST handler

Calling the readlink[at] syscall directly from the PRE handler defeats
the FUSE_COMPATIBLE_MAY_BLOCK (SfMayBlock) flag. Add a POST handler
that only explicitly calls the readlink[at] handler for the
/proc/self/exe fallback (this should be fine unless /proc is also
implemented as fuse in this process).

Adjust readlink[at] GENX_ and LINX_ syswrap macros to GENXY and LINXY.

https://bugs.kde.org/show_bug.cgi?id=493507

Add missing FUSE_COMPATIBLE_MAY_BLOCKs

Various syscalls (in particular "at" variants) PRE handlers were
missing a FUSE_COMPATIBLE_MAY_BLOCK statement.

Add it to the generic PRE handlers of access and statfs64. And the
linux PRE handlers of mknodat, fchownat, futimesat, utimensat,
utimensat_time64, renameat, renameat2, readlinkat, fchmodat,
fchmodat2, faccessat and faccessat2.

https://bugs.kde.org/show_bug.cgi?id=493454

Implement stable variant of sync_file ioctls

We implemented an old staging android variant of the sync_file
ioctls. But the data structures and ioctl numbers changed when these
were upstreamed in the table linux kernel.

This implements the SYNC_IOC_MERGE, SYNC_IOC_FILE_INFO and
SYNC_IOC_SET_DEADLINE ioctls. And makes sure to record the new file
descriptor created by SYNC_IOC_MERGE.

https://bugs.kde.org/show_bug.cgi?id=492422

Update README.s390

The link to the "z/Architecture Principles of Operation" document is out
of date.  Update it.

s390x: Add bug 440180 to NEWS

As reported in Bug 440180, the s390x disassembler could run into an
assertion failure due to the maximum mnemonic length being exceeded.  This
was fixed with commit 67a2bb759a7c9c76fd6aa142bdb6fe342a5998e2.

FreeBSD regtest: remove unneeded expected file

Previously I had #ifdef'd the freebsd7 compatibility syscalls
in the scalar test for all platforms when adding the arm64
port. They are now back for amd64 and x86 so the previous
expected also covers FreeBSD >= 14.0 and the extra expected
file is now superfluous and removed.

FreeBSD syscalls and regtest, cleanup freebsd7 compatible syscalls on arm64

None of the freebsd7 interfaces exist on arm64 as the OS
wasn't ported until well after FreeBSD 7.

Linux regtest: fix for memcheck/tests/amd64/bug492210

On Arch Linux I get an error concerning the use of PIE and
an uninitialized variable warning. Add the no PIE flag
and also turn off the warning.

macOS: fix build after changes for Bug 492663

Missed updating the macOS sections.

FreeBSD procctl syscall: change arg name in error messages

The man page was inconsistent in the use of 'data' or 'arg'
for the fourth argument. I chose to use 'arg'. The manpage
has now been cleaned up and uses 'data'. So I'm switching to
use the same name.

FreeBSD: start reducing uses of FREEBSD_VERS

The goal is to remove version checking from configure.ac,
like Linux. There's a long way to go still.

Compiler warning in ML_(check_elf_and_get_rw_loads) again

This fime for p_vaddr

Compiler warning in ML_(check_elf_and_get_rw_loads)

GCC 12.2 complains that
 previous_rw_a_phdr.p_vaddr + previous_rw_a_phdr.p_filesz
may be using p_filesz uninitialized

That's only possible if ML_(img_get) somehow fails to read all
of a program header such that p_memsz is greater than 0 but
p_filesz remains uninitialized. Hardly likely since p_memsz
comes after p_filesz in the structure.

FreeBSD: remove code for FreeBSD 10

FreeBSD 10 was never really tested - fully working FreeBSD support
arrived around the time of FreeBSD 11.3 and 12.1. FreeBSD had
already been EOL around 2 years by then.

FreeBSD regtest: fix "filter_fdleak is missing in dist_noinst_SCRIPTS"