Minor changes to the documentation and commends related to demangling.

regtest: callgrind bug497723 again, order of output can vary

The filter was picking up two mangled names but the order
of thw two can vary between systems. So just filter one of them.

regtest: add check for -Wl,--no-warn-execstack

Add missing zstd.h to coregrind Makefile.am noinst_HEADERS

The header was missing from dist tarballs

Bug 497723 - tweak awk filter in regtest

Bug 497723 - forgot to restore callgrind output cleanup

Command line help - demangling isn't just for C++

Make the first line generic and add a list of languages
that are suppoted.

Bug 497723 - improve filter

Just print out the function names. Previously it was unstable.

Bug 497723 - Enabling Ada demangling breaks callgrind differentiation between overloaded functions and procedures

regtest: turn off exec stack warning

Rather than turning off exec stack.

Implement the new GDB 16.0 'x addr,len' packet.

GDB can read memory faster with this packet than with the 'm addr,len' packet.

VEX/priv/ir_opt.c: Also fold Iop_And1 expressions when possible

Treat Iop_And1 just like And16/And32/And64. Fold fully constant And1
expressions and handle And1(x,True), And1(True,x), And1(x,False),
And1(False,x) and And1(x,x).

Make sure isOnesU handles Ico_U1 (isZerosU and sameIRExprs already
did).

https://bugs.kde.org/show_bug.cgi?id=468575#c52

Use Ints for fds in PRE and POST sys_close_range

The double_close_range test failed on riscv64-linux because the
close_range wrapper is using unsigned int and the ARG regwords
directly. Which causes the ARG2 == ~0U check to fail. Explicitly
using Int for the fd arguments fixes this. I am not clear on why
this was only an issue for the riscv port. It seems this patch is
OK for other arches (tested on amd64 and i386).

Add 469782 to NEWS

Bug 497455 - Update drd/scripts/download-and-build-gcc

The other script also are likely to need some attention.
This one is probably the most useful for users that want to test OpenMP
aplications with DRD or Helgrind.

FreeBSD: add a suppression for reachable from __cxa_exit

FreeBSD regtest: update getrlimit for FreeBSD 14.2

More output and memory counts are quite variable.

Recognize new DWARF5 DW_LANG constants

When using --read-var-info=yes readdwarf3 will try to read and
interpret the CU DW_AT_langauge attribute. Since DWARF5 was released a
number if new language constants have been introduced. See
https://dwarfstd.org/languages.html

GCC15 might start emitting some of these when switching to C23 by
default.

When valgrind --read-var-info=yes encounters an unknown DW_LANG
constant it will produce an error and stop processing any further
DWARF.

Recognize all currently known language constants. In particular
recognize DW_LANG_C17, DW_LANG_C23, DW_LANG_C_plus_plus_17,
DW_LANG_C_plus_plus_20, DW_LANG_C_plus_plus_23, DW_LANG_Fortran18,
DW_LANG_Fortran23, DW_LANG_Ada2005, DW_LANG_Ada2012 and DW_LANG_Rust.

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

regtest: fix a warning building none/tests/nestedfns on some platforms

s390x: Update instruction list with arch15 insns

Recent commits in Binutils added arch15 instructions to s390-opc.txt.
Since then, the script s390-check-opcodes.pl complains about many insns
that are not known to Valgrind.

Update Valgrind's instruction list to be in synch with Binutils again.
Add the new instructions to s390-opcodes.csv and declare them as "not
implemented".

Bug 495817 - s390x: Fix disassembly for SEL[G]R and SELFHR

The disassemblies of selr, selgr, and selfhr have their register operands
mixed up, and no extended mnemonics are used.  Fix this.

Bug 495817 - s390x: Fix disassembly for compare-and-branch/trap insns

The insns C[L][G]R[BJ], C[L][G]I[BJ], CL[G]T, and CL[FG]IT are
disassembled incorrectly.  Fix this.

Replace s390_format_RIEv1 with s390_format_R0UU and s390_format_R0IU.
Handling both a signed and unsigned immediate constant field with the same
s390_format_... function does not work.

Add function s390_format_RSY_R0RD for CLT and CLGT.  Those opcodes have
extended mnemonics.  So adjusting the formerly used s390_format_RSY_RURD
wasn't an option as that function is also used for CLM[HY], STCM[HY], and
ICM[HY] which don't have extended mnemonics.

Bug 495817 - s390x: Fix disassembly for BC[R], BR[C]L, and BIC

The disassembly of the conditional branch insns bc, bcr, brl, brcl, and
bic differs from objdump's output.  Some examples:

*** mismatch VEX: |bic 20,0|               vs objdump: |bic      0,20|
*** mismatch VEX: |bic 20(%r6),0|          vs objdump: |bic      0,20(%r6)|
*** mismatch VEX: |bic 20(%r4),0|          vs objdump: |bic      0,20(%r4,%r0)|
*** mismatch VEX: |bih 20(%r12)|           vs objdump: |bih      20(%r12,%r0)|
*** mismatch VEX: |nopr|                   vs objdump: |nopr     %r6|
*** mismatch VEX: |b         12(%r11)|     vs objdump: |b        12(%r11,%r0)|
*** mismatch VEX: |blh       12(%r11)|     vs objdump: |blh      12(%r11,%r0)|
*** mismatch VEX: |brc       0,.+0|        vs objdump: |jnop     c|
*** mismatch VEX: |brcl      0,.+0|        vs objdump: |jgnop    c|

All fixed with this patch.

Note that the issue with the base register sometimes being suppressed
affects various other insns as well.

Bug 495817 - s390x: Disassemble cksm, mvcl, and clcl

There is no output from the disassembler for cksm, mvcl, and clcl.  Fix
this.

Bug 495816 - s390x: Fix disassembler segfault for C[G]RT and CL[G]RT

The function s390_format_RRF_U0RR() is called with one of the following
xmnm_kind parameters:

   S390_XMNM_CAB     when constructing IR for C[G]RT or CL[G]RT
   S390_XMNM_LOCFHR  for LOCFHR
   S390_XMNM_LOCGR   for LOCGR
   S390_XMNM_LOCR    for LOCR

In all cases it invokes the disassembler without providing a mnemonic:

     if (UNLIKELY(vex_traceflags & VEX_TRACE_FE))
        s390_disasm(ENC3(XMNM, GPR, GPR), xmnm_kind, m3, r1, r2);

But in the first case s390_disasm() processes these arguments as if there
was one:

     case S390_XMNM_CAB:
        mnm  = va_arg(args, HChar *);  // <--- m3
        mask = va_arg(args, UInt);     // <--- r1
        p  += vex_sprintf(p, "%s", mnemonic(cab_operand(mnm, mask)));

Thus, m3 is interpreted as a string which then causes the segfault.

Fix this by

(1) replacing all of S390_XMNM_LOC* and S390_XMNM_STOC* by S390_XMNM_CLS,
(2) passing down the mnemonic to s390_disasm, and
(3) changing function cls_operand() to be symmetric with cab_operand() by
    also taking in the base mnemonic.

Apart from s390_format_RRF_U0RR(), further "load/store on condition"
instructions are handled via s390_format_RIE_RUPIX() and
s390_format_RSY_RDRM().  Adjust these functions accordingly as well.

s390x: Add `--check-formats' flag to s390-check-opcodes.pl

Enhance the script s390-check-opcodes.pl: Add the command line option
`--check-formats' to print mismatches in the opcode formats between
guest_s390_toIR.c and s390-opc.txt.

Fix typo in FAQ.

Add bug 494246 to NEWS

Bug 494246 syscall fsopen not wrapped was fixed in commit 4044bcea0.
"Add open_tree, move_mount, fsopen, fsconfig, fsmount, fspick linux
syswraps"

Add open_tree, move_mount, fsopen, fsconfig, fsmount, fspick linux syswraps

Shared linux syscalls implementing various file system mount tasks.
Since linux kernel version 5.2.

Check arguments and track file descriptors.

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

Manual: C23 is now officially released

Remove standard headers from zstddeclib.c

Not needed and safer without them.

Add support for landlock_create_ruleset (444), landlock_add_rule (445) and landlock_restrict_self (446) syscalls

- add support for landlock_create_ruleset (444) syscall
- add support for landlock_add_rule (445) syscall
- add support for landlock_restrict_self (446) syscall

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

Signed-off-by: Peter Seiderer <ps.report@gmx.net>

coregrind/m_gdbserver/server.h: define memcpy and memset to VG_ variants

Then use memset/memcpy in the coregrind/m_gdbserver code instead of
the VG_ variants.

Solaris and Illumos: fix build due to use of memset in zstd code.

Add additional exp-ppc64le files to EXTRA_DIST

none/tests/bug234814.c: sa_handler take an int as argument

GCC15 will turn this warning into an error:

bug234814.c: In function 'main':
bug234814.c:20:18: error: assignment to '__sighandler_t' {aka 'void (*)(int)'} from incompatible pointer type 'void (*)(void)' [-Wincompatible-pointer-types]
   20 |    sa.sa_handler = mysigbus;
      |                  ^

drd/tests/swapcontext.c: Rename typedef struct thread_local to threadlocal

Since C23 thread_local is a keyword (thread storage duration).

swapcontext.c:23:16: error: expected '{' before 'thread_local'
   23 | typedef struct thread_local {
      |                ^~~~~~~~~~~~
swapcontext.c:23:16: warning: 'thread_local' is not at beginning of declaration [-Wold-style-declaration]
swapcontext.c:23:16: error: 'thread_local' used with 'typedef'
swapcontext.c:26:3: warning: data definition has no type or storage class
   26 | } thread_local_t;
      |   ^~~~~~~~~~~~~~

helgrind/tests/tc17_sembar.c: Remove bool typedef

Since C23 bool is a keyword. Also bool wasn't actually used.

tc17_sembar.c:45:14: error: both 'long' and '_Bool' in declaration specifiers
   45 | typedef long bool;
      |              ^~~~
tc17_sembar.c:45:1: warning: useless type name in empty declaration
   45 | typedef long bool;
      | ^~~~~~~

Add exp and supp patterns for missing main frame for ppc64le

In some cases on ppc64le we are missing the main frame.
Add alternative .exp-ppc64le variants for socket_close_xml,
fdleak_cmsg_xml and fdleak_ipv4_xml. And extra suppressions
without a main frame for fdleak_cmsg_supp.

See also commit 04d30049b "Filter away "main" differences in filter_fdleak"

Bug 496571 - False positive for null key passed to bpf_map_get_next_key syscall.

No regtest added because BPF requires privileges. See the bugzilla item
for example usage.

Turn off unused result warnings

FreeBSD: fix a few warnings when building with GCC

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.