Solaris 11: fix for VG_(lstat)

This function was using SYS_lstat. It works OK with illumos
but not Solaris 11+ which has removed that syscall.

Instead do like Linux and FreeBSD, use SYS_fstatat with the flag
for no follow link.

FreeBSD memalign wrapper: size of 0 now allowed

memalign, the random allocator function amongst allocators

posix_memalign wrapper - remove newline from malloc trace

FreeBSD regtest: arm64 scalar

Right number of underscores in freebsd7 compat syscall fakes

FreeBSD regtest and syscall message

FreeBSD regtest: update fake freebsd7 output for amrm64

First attempt

Bug 509107 - memcheck/tests/duplicate_align_size_errors.cpp fails

FreeBSD syscall: add ML_(fd_allowed) check for fstat

Also PRE(sys_cap_rights_get) was checking the wrong arg
with ML_(fd_allowed). ARG1 is a version number in the
syscall, ARG2 is the fd.

Solaris syscall: add ML_(fd_allowed) check for fstat

FreeBSD regtest: clean up compat syscall messages

Old syscalls never die. They just become freebsdXX_syscall.
And I forget to rename all their arguments.

Fix compilation of tests/arm64/bug484935.c when using fat LTO.

See https://gcc.gnu.org/bugzilla/show_bug.cgi?id=121719

Fix this error:
/tmp/ccfnh7KQ.s: Assembler messages:
/tmp/ccfnh7KQ.s:175: Error: symbol `loop' is already defined

ir_opt.c:  Fix mkZeroOfPrimopResultType

In c93dba5adbd5d I forgot to adjust mkZeroOfPrimopResultType.

This function needs to go. See also 7270e04ab8eb7.

I should be able to query IROp properties easily and for all IRops.
E.g. instead of calling mkZeroOfPrimopResultType I want this:

mkZero(irop->result_type)

Add fd_allowed check to fstat (sys_newfstat)

Makes sure that with --track-fds fstat is also checked.

Remove fdleak.h CLOSE_INHERITED_FDS workaround

This workaround was necessary with very old perl implementations (from
2008) which might execute programs with some non-standard file
descriptors not closed. The macro would close all file descriptors 3
or higher so --track-fds wouldn't report on them.

More recently --track-fds also reports on bad file descriptor
usage. First only double or bad close calls were reported. This would
cause lots of warnings for the close_inherited file descriptor loop
because almost all of those file descriptors were never opened, so
--track-fds would report those. To work around that an fstat call was
added before the close to make sure the file descriptor existed.

This fstat workaround in close_inherited only worked because fstat
didn't have a fd_allowed check. Which is a bug that should be
fixed. On some systems fstat actually calls the fstatat syscall and
that did recently got the fd_allowed check, so on systems that use
fstatat for fstat various fdleak tests started failing.

We could test for and use close_range, which is smart enough to not
warn about never opened file descriptors in the range. But it seems
simpler to just get rid of the CLOSE_INHERITED_FDS macro because the
problematic perl implementation is now so old that nobody uses it
anymore.

ir_opt.c:  Algebraic simplification for Iop_Or1.

Adapt function mkOnesOfPrimopResultType accordingly.

ir_opt.c:  Algebraic simplification for Iop_Shl/Shr/Sar.

The simplification rules for 32-bit and 64-bit operands also apply for
8-bit and 16-bit operands.

ir_opt.c:  Algebraic simplification for Iop_Sub.

The simplification rules for IopSub32/64 also apply for Iop_Sub8/16.

ir_opt.c:  Algebraic simplification for Iop_Add.

The simplification rules for IopAdd32/64 also apply for Iop_Add8/16.

VEX: Fix function mkOnesOfPrimopResultType

This function may be called with Iop_Max32U as argument which isn't
handled.

VEX: Remove functions isZeroU32 and isZeroU64

This was forgotten when function isZeroU was added.

FreeBSD suppression: add a DRD suppression for exterrctl

FreeBSD syscalls: new getgroups and setgroups

Wasn't quite right for FreeBSD 15.

Bug 508958 - FreeBSD: add getgroups and setgroups wrappers

Bug 507867 - perf_event_open group_fd isn't checked

Bug 507866 - fanotify_mark dirfd isn't checked

illumos syscalls: refactor *at directory fd checks

Mainly to reduyce the amount of copied and pasted code.
Slight improvements to messages to syscalls that have two
directory fds.

FreeBSD realpathat syscall: fix length of written memory and add a testcase

There were no tests for /proc/currrproc/file
and the length written in post was out by one

libvex_ir.h: Clarify semantics of division and modulo operation.

iropt-test: Constant folding for DivModU32to32 and DivModS32to32

Needed for Mips. Mips is still disabled in iropt-test because there
are miscompares between the folded the result and the result computed
by the insn sequence that implements the IROp. Several IRops are affected.

Part of fixing https://bugs.kde.org/show_bug.cgi?id=506211

iropt-test: Prepare IROp table for mips.

Part of fixing https://bugs.kde.org/show_bug.cgi?id=506211

Bug 508779 - PRE(sys_prlimit64): reorder check for memory validity

so all errors are displayed and not just the first

Bug 508778 - syscall-wrapper waitid warns about infop=null

This is used by e.g. Qt:
https://github.com/qt/qtbase/blob/e17798560b107b7abd5f2cf472d942e8ac4213ff/src/3rdparty/forkfd/forkfd_linux.c#L128

Bug 508777 - amd64-linux: add minimal scalar test

Bug 508869 - x86-linux: simplify scalar test output

regtest: add vgtest and expected for memcheck duplicate_align_size_errors

The cpp file and .gitignore change were done at the same
time as the aligned size checks, but I missed out the expected
vgtest and Makefile.am changes. There were also a couple of unused
variables in the cpp file.

FreeBSD: fix build error

Deleteted wrong variable when doing *at() syscall refactoring.

Linux FreeBSD and Darwin: refactor *at syscall dirfd checks

I haven't done Solaris. The code there is less messy because
Solaris doesn't use a negative value for AT_FDCWD, meaning
no explicit or implicit cast from unsigned word to signed
int is needed before comparing to the int dirfd parameter.

iropt-test: Constant folding for Iop_DivModU64to32 and Iop_DivModS64to32

Function get_selected_values has been tweaked to return more
"interesting" values for test generation. Namely those that,
when interpreted as a signed integer, are boundary (or near
boundary) values.

Also a bit of reorg because for the DivMod.. IROps we need to be a bit
more careful when generating testcases that do not trap or where the
result does not fit in 32 bit.

Part of fixing https://bugs.kde.org/show_bug.cgi?id=506211

Bug 507869 - Various at syscalls don't check dirfd argument

iropt-test: Enable for x86 and amd64.

Fix Iop_ClzNat32/64 and Iop_CtzNat32/64 on x86 and amd64.

Handle the special case of 0 operand.

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

Typo in comment.

I did correct it, but didn't save before pushing.

Linux syscalls: add checks for remaining *at() directory fd args

Bug 508638 - Self-hosting not working on FreeBSD

Linux syscall: copy and paste error in openat2 fd_allowed check

FreeBSD 64bit: client stack changes

Just use system max stack size, but with a lower limit of 64Mb
Add some -d -d traces

FreeBSD regtest: turn off uninitilised warnings for readlinkat test

FreeBSD: fix guest stack creation for self-hosting

See https://bugs.kde.org/show_bug.cgi?id=508638
Not fully resolved, making progress.

Remove deprecated IROps Iop_Clz32, Iop_Clz64, Iop_Ctz32, and Iop_Ctz64

Part of fixing https://bugs.kde.org/show_bug.cgi?id=507033

PPC specific changes for BC 507033

Replace the deprecated Iop_Clz64 with Iop_ClzNat64. This is safe to
do because insn selection does not distinguish between Iop_Clz64 and
Iop_ClzNat64:

Iop_Clz64 / Iop_ClzNat64 --isel--> Pun_CLZ64 --emit-->cntlzd

The cntlzd insn has "natural" semantics and a 0 operand is not special.

Part of fixing https://bugs.kde.org/show_bug.cgi?id=507033

Add 507720 to NEWS

Darwin regtest: turn off ada mangling test

On Darwin clang adds an underscore to mangle C functions
which interferes with this testcase which tries to fake ada
mangling in C.

Darwin: fix a warning, unused local

Darwin syscalls: fd handling (part 4)

Most already done.

Darwin: add VG_(resolved_exename)

Possibly not needed. On Linux it's used to check that the guest exe
doen't get opend with a write flag. macOS seems happy to allow that.

illumos syscalls: fd handling (part 4)

Almost everything looked OK. Added a couple of POST_newFd_RES
macro uses.

There are a few functions that create fds other than via RES
(like pipe). For these I think that we need to add a
POST_newFd(fd) macro, plus in a few places refactor so that
RES is available to set the status after ML_(get_next_new_fd).

FreeBSD regtest: pdfork_pdfill last fix (hopefully)

Now passes on amd64 arm64 and x86

FreeBSD regtest: updates to scalar for x86

Now that there better checks of fds a couple of syscalls using
rubbish values for fd which were above the max files limit.
That changed the syscall return code causing an assert.

Also some left over updates to the expected from previous changes.

FreeBSD regtest: pdfork_pdkill again

x86 needs some filtering, and I think arm64 will need an expected

FreeeBSD regtest: add the suppression for pdfork_pdkill

FreeBSD regtest: get pdfork_pdkill to pass on arm64

arm64 was giving a few conditional jump errors (suppressed)
and a pdfork was succeeding with uninit flags whilst it was
failing on amd64. Made sure that it has bad flags (-1) but
still uninit.

FreeBSD regtest: cleanup fd handling

Build error on arm64 and update scalar expected.
scalar not looking right for x86

FreeBSD syscalls: fd handling (part 3)

arm64 file

FreeBSD syscalls: fd handling (part 2)

amd64 and x86 files

FreeBSD syscalls: fd handling (part 1)

For
Bug 507720 - Review syscalls returning file descriptors (other platforms)

Some more recording of fd opens and closes
Lots of checks for the directory fd of *at() syscalls
Fixed a few bugs in pdfork and pdkill (and updated the testcase).
Updated a few message strings.
Started improving readlinkat - needs more work.

iropt-test: Run tests with and without constant folding.

That way we can check whether the result of constant folding matches
the value computed by the insn sequence that is generated to implement
a specific IROp. Bugs were found for all architectures. s390 has been
fixed. The other architectures are currently disabled.

- irops.tab modified to enable only those IROps that are actually
  implemented for a given architecture. Architectures considered here
  are amd64, x86, ppc32, ppc64be/le
- IRICB_iropt_payload amended to store both the folded and unfolded
  result
- In valgrind_execute_test call IR injection twice: with and without
  constant folding enabled (--vex-iropt-fold-expr=yes/no)

FreeBSD regtest: update kenv test so that it builds with GCC

Was missing the memory header for std::make_unique,
and GCC doesn't like using smart pointers with cout so use
get().

iropt-test: Fix an unwanted integer wrap-around.

vbit-test: Remove preprocessor atrocity to determine endianess.

Use new function host_is_big_endian instead.

iropt-test: Fix executable name for secondary platform.

FreeeBSD readme: add a section on Capsicum mode

FreeBSD regtest: add an absolute path test to readlinkat test

FreeBSD regtest: add a test for readlinkat

FreeBSD regtest: fix a minor typo

s390: Fix handling of Iop_ExpCmpNE16

Bug was introduced by yours truly in e3f078d847.

FreeBSD: fix build after adding mallinfo support for illumos

s390: Support Iop_Mul8/16/32/64.

Easy as pie and fewer special cases for iropt-test.

s390: Support Iop_ExpCmpNE8 and Iop_ExpCmpNE16.

Easy as pie and fewer special cases for iropt-test.

s390: Fix code generation for Iop_MullS8/16 and Iop_MullU8/16

Multiplication is performed using 4-byte values.
There were two bugs:
(1) Iop_MullS8/16 operands weren't sign-extended
(2) No shifting and OR'ing required to access the result. r11[32:63] has it.

Found by iropt-test.

Bug 507721 - Wire up illumos and Solaris mallinfo

Bug 507970 - -Wcalloc-transposed-args warnings in valgrind-di-server.c

Add ppc64le linux hardwire for ld64.so.2 strcmp

When dlopen is used we might end up in an assembly powerpc/strcmp.S
variant that is optimized in a way memcheck cannot proof correct. We
try to intercept strcmp in ld.so, but might fail when strcmp is called
before our interception code is loaded. Having an hardwire for ld.so
strcmp (earlier intercept) would solve this.

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

s390: Fix code generation for Iop_16Uto32(0x...:I16)

Using LHI is wrong as it sign-extends. Use IILF instead.
This has been wrong since day #1. Found by iropt-test.

PRE(sys_fchownat) not handling VKI_AT_FDCWD

Multiple `make ltpchecks` failures seem tp be caused by
PRE(sys_fchownat) not handling VKI_AT_FDCWD properly.
This specifically impacts aarch64 test results these days.

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

Fix none/tests/cmdline2 breakage introduced in 679e7bced9

New command line flag --vex-iropt-fold-expr

Defaults to "yes".
Add VexControl::iropt_fold_expr and observe it in fold_Expr_WRK.

This flag will be used shortly in the iropt-test program.
Users are not expected to use it which is why I'm not announcing
it in NEWS.

iropt-test tweak

No need to run random tests for binary IROps with 1-bit wide operands.

Fix VALGRIND_CLO_CHANGE behaviour (BZ 508093)

Command line flags that modify variable vex_control did not have the
desired effect when processed dynamically via VALGRIND_CLO_CHANGE.

The fix is to call the new function LibVEX_set_VexControl in
process_dynamic_option.

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

Fix mips32 FTBFS

Define __NR_statmount and __NR_listmount.
This update makes the source buildable on the mips32 arch.

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

faccessat and faccessat2 should do better checks

Do more fine-grained checks within sys_faccessat and sys_faccessat2
syscall wrappers.  Allow passing special value of VKI_AT_FDCWD as a
file descriptor.  Check for valid flags.

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

Add several missing syscall hooks to ppc64-linux

Specifically:

sys_getrlimit
sys_mincore
sys_tkill
sys_unshare
sys_splice
sys_tee
sys_vmsplice
sys_fanotify_init
sys_fanotify_mark
sys_kcmp
sys_bpf

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

Review the vmsplice syscall wrapper

Reuse the vmsplice syscall wrapper in coregrind/m_syswrap/syswrap-linux.c
for mips64 as well. And make sure arm64-linux and riscv64-linux also use
the POST vmsplice wrapper.

Distribute auxprogs scripts and data files

Make sure pybuild.sh, ltp-tester.sh, ltp-apply-patches.sh,
s390-check-opcodes.pl, s390-runone scripts, ltp filters, ltp patches
and pylintrc are added to the dist.

This makes sure users can run the scripts and make ltpchecks not just
from a git checkout, but also from a release or snapshots tar.

Also use the ustar format to package files, the path under
valgrind-x.yy.z/auxprogs/ltp-patches might be larger than 99
chars. automake 1.18 already defaults to the ustar format.

The futimesat syscall wrapper doesn't handle AT_FDCWD

Update futimesat syscall wrapper so that it doesn't consider the
special value AT_FDCWD (-100) an invalid file descriptor.

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

Make fchmodat and fchmodat2 syscall wrappers accept AT_FDCWD

The fchmodat and fchmodat2 syscall wrappers should accept special
value AT_FDCWD as a valid file descriptor.

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

Allow for patching LTP sources

Sometimes there's an upstream LTP patch that helps testing
valgrind, but it's not yet part of the official LTP tarball.
In such cases it's helpful to be able to patch the LTP sources.
Attached patch allows for that.  It comes with a real life
example patch: LTP commit b62b831cf.

Add bug 503241 s390x: Support z17 changes to the NNPA instruction to NEWS

This was added in commit 24b634e8ce04de70d4aa6c61a12149df223f9c68

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

ARM64 specific changes for BZ 507033

Replace the deprecated Iop_Clz64 with Iop_ClzNat64 which have the
same semantics for non-zero inputs.

Part of fixing https://bugs.kde.org/show_bug.cgi?id=507033

ARM specific changes for BZ 507033

Rework code to use Iop_ClzNat32 instead of the deprecated Iop_Clz32.

Iop_Clz32 --isel--> ARMun_CLZ --emit--> CLZ insn  with CLZ(0) == 32
Hence, any workaround for a zero input value can be removed.

Part of fixing https://bugs.kde.org/show_bug.cgi?id=507033

mips specific changes for BZ 507033

Rework code to use Iop_ClzNat32 instead of the deprecated Iop_Clz32.
Likewise for Iop_Clz64.

For Iop_Clz32 a CLZ insn will be emitted which behaves naturally when
its input is 0. That is: CLZ(0) == 32.
Similarly, for Iop_Clz64 a DCLZ will be emitted with DCLZ(0) == 64.
That means we can replace Iop_Clz32/64 with Iop_ClzNat32/64 and remove
any IR that handles the input-is-zero case.
See also commit a5c48217e94.

Part of fixing https://bugs.kde.org/show_bug.cgi?id=507033