]> sourceware.org Git - valgrind.git/log
valgrind.git
15 months agodebuginfo.c: Add more hooks for lazy loading debuginfo
Aaron Merey [Wed, 20 Sep 2023 17:00:02 +0000 (13:00 -0400)]
debuginfo.c: Add more hooks for lazy loading debuginfo

Add calls to di_load_di in search_one_{symtab,loctab,cfitab} to ensure
that debuginfo is always loaded before searching these tables.

This fixes a bug introduced in commit 60f7e89ba32 that results in
callgrind failing to read source filenames from debuginfo.

Also add a test which checks that callgrind successfully finds a
filename from debuginfo.

Co-Authored-By: Philippe Waroquiers <philippe.waroquiers@skynet.be>
15 months agoTry to find section .opd in read_elf_object
Aaron Merey [Mon, 18 Sep 2023 21:42:42 +0000 (17:42 -0400)]
Try to find section .opd in read_elf_object

read_elf_debug skips checking for .opd ELF section, causing
assert failures on PPC BE.

This patch adds the missing check for .opd.

15 months agoFreeBSD: update README.freebsd
Paul Floyd [Wed, 20 Sep 2023 19:03:25 +0000 (21:03 +0200)]
FreeBSD: update README.freebsd

Remove specific version numbers for availability.
Add more details about releases of devel/valgrind and
devel/valgrind-devel

15 months agoDarwin: fix build with lazy debuginfo reading
Paul Floyd [Wed, 20 Sep 2023 06:47:14 +0000 (08:47 +0200)]
Darwin: fix build with lazy debuginfo reading

Needs the macho reader, not ELF.

15 months agoFreeBSD: add intercepts to timingsafe variants of memcmp and bcmp
Paul Floyd [Wed, 20 Sep 2023 06:16:43 +0000 (08:16 +0200)]
FreeBSD: add intercepts to timingsafe variants of memcmp and bcmp

15 months agoFreeBSD: refactor VG_(load_all_debuginfo)
Paul Floyd [Tue, 19 Sep 2023 17:53:06 +0000 (19:53 +0200)]
FreeBSD: refactor VG_(load_all_debuginfo)

Thanks to Philippe Waroquiers for the suggestion

15 months agoBug 432801 - Valgrind 3.16.1 reports a jump based on uninitialized memory somehow...
Eyal Soha [Thu, 10 Feb 2022 16:52:54 +0000 (09:52 -0700)]
Bug 432801 - Valgrind 3.16.1 reports a jump based on uninitialized memory somehow related to clang and signals

Add support for precise computation of SIMD greater-than on
amd64 and x86.

This adds support for 64bit, 16bit, and 8bit to the existing 32bit
support.

15 months agoHelgrind s390: add a suppression for __printf_buffer_write
Paul Floyd [Tue, 19 Sep 2023 05:48:26 +0000 (07:48 +0200)]
Helgrind s390: add a suppression for __printf_buffer_write

Again Fedora 38

15 months agoDRD s390: add a suppression for _dl_exception_create_format
Paul Floyd [Tue, 19 Sep 2023 05:31:17 +0000 (07:31 +0200)]
DRD s390: add a suppression for _dl_exception_create_format

Seen on Fedora 38

15 months agoregtest: copy and paste error in new_delete_mismatch_size vgtest files
Paul Floyd [Mon, 18 Sep 2023 18:19:56 +0000 (20:19 +0200)]
regtest: copy and paste error in new_delete_mismatch_size vgtest files

15 months agotest scripts: add cachegrind to list of tools with export headers
Paul Floyd [Sun, 17 Sep 2023 08:12:48 +0000 (10:12 +0200)]
test scripts: add cachegrind to list of tools with export headers

15 months agoFreeBSD: add syscall wrapper for membarrier (added in FreeBSD 15)
Paul Floyd [Sun, 17 Sep 2023 09:50:54 +0000 (11:50 +0200)]
FreeBSD: add syscall wrapper for membarrier (added in FreeBSD 15)

I also noticed that the Linux membarrier syscall wrapper
is out of date. It only takes one argument. Depending on the kernel
version it can also take 2 or 3 arguments (3 being the latest).

15 months agoFreeBSD: Add syscall wrapper for kqueuex (FreeBSD 15)
Paul Floyd [Sun, 17 Sep 2023 09:06:05 +0000 (11:06 +0200)]
FreeBSD: Add syscall wrapper for kqueuex (FreeBSD 15)

15 months agoUpdate .gitignore
Paul Floyd [Sat, 16 Sep 2023 20:44:21 +0000 (22:44 +0200)]
Update .gitignore

15 months agoLinux: change timerfd wrapper names used for arguments
Paul Floyd [Sat, 16 Sep 2023 13:00:39 +0000 (15:00 +0200)]
Linux: change timerfd wrapper names used for arguments

The  now match the manpage names:

SYNOPSIS
       #include <sys/timerfd.h>

       int timerfd_create(int clockid, int flags);

       int timerfd_settime(int fd, int flags,
                           const struct itimerspec *new_value,
                           struct itimerspec *old_value);
       int timerfd_gettime(int fd, struct itimerspec *curr_value);

15 months agoFreeBSD: add wrapperd for timerfd_create timerfd_gettime and timerfd_settime for...
Paul Floyd [Sat, 16 Sep 2023 14:06:12 +0000 (16:06 +0200)]
FreeBSD: add wrapperd for timerfd_create timerfd_gettime and timerfd_settime for FreeBSD 15

Mostly copied from the Linux implementation.
Add a testcase and a scalar for FreeBSD 15

15 months agoAdd some client requests to Cachegrind.
Nicholas Nethercote [Thu, 10 Aug 2023 04:10:36 +0000 (14:10 +1000)]
Add some client requests to Cachegrind.

15 months agoLinux regtest: update MUSL version of aligned_alloc test
Paul Floyd [Sun, 10 Sep 2023 16:15:38 +0000 (18:15 +0200)]
Linux regtest: update MUSL version of aligned_alloc test

15 months agoLinux regtest: Update aligned_alloc test for GNU libc 2.38
Paul Floyd [Sun, 10 Sep 2023 15:53:05 +0000 (15:53 +0000)]
Linux regtest: Update aligned_alloc test for GNU libc 2.38

15 months agoBug 474332 - aligned_alloc under Valgrind returns nullptr when alignment is not a...
Paul Floyd [Sun, 10 Sep 2023 13:05:57 +0000 (15:05 +0200)]
Bug 474332 - aligned_alloc under Valgrind returns nullptr when alignment is not a multiple of sizeof(void *)

At configure time use glibc version to set a HAVE flag for C17 aligned_alloc.
The use the HAVE flag to select which redir macro to use.
Also make the (normally unused) glibc ALIGNED_ALLOC macro
the same as MEMALIGN, just in case.

15 months agoFreeBSD: close_range versioning issues
Paul Floyd [Sun, 10 Sep 2023 08:39:34 +0000 (10:39 +0200)]
FreeBSD: close_range versioning issues

I mixed up a bit the syscall wrapper versioning.
Also for some reason the syscall has been backported to FreeBSD 12.2
but not the CLOEXEC flag. So I had to modify the testcase for that.

15 months agoIllumos regtest: update expected for aligned_alloc
Paul Floyd [Sat, 9 Sep 2023 10:26:40 +0000 (12:26 +0200)]
Illumos regtest: update expected for aligned_alloc

15 months agocoverity: various minor fixes
Paul Floyd [Sat, 9 Sep 2023 06:53:43 +0000 (08:53 +0200)]
coverity: various minor fixes

mostly asserts that unsigned values are 0 or positive

15 months agoFreeBSD: add syscall wrapper for close_range
Paul Floyd [Fri, 8 Sep 2023 21:40:12 +0000 (23:40 +0200)]
FreeBSD: add syscall wrapper for close_range

15 months agos390x regtest: Reduce test output for vector test cases
Andreas Arnez [Thu, 7 Sep 2023 15:00:42 +0000 (17:00 +0200)]
s390x regtest: Reduce test output for vector test cases

The s390x test cases `vector', `vector_integer', `vector_float' and
`vector_string' generate a lot of unnecessary output, because for each
tested instruction they usually dump more registers than the instruction
modifies.

Without changing the test approach itself and without losing information,
the output can still be reduced significantly by comparing the register
states before and after the tested instruction and then only dumping the
changed parts.

15 months agos390x regtest: Fix typo in previous commit
Andreas Arnez [Thu, 7 Sep 2023 15:00:42 +0000 (17:00 +0200)]
s390x regtest: Fix typo in previous commit

This amends the commit "Build DFP test cases unconditionally", where
"pfpo" was accidentally replaced by "pfpfo" in Makefile.am.

15 months agos390x regtest: Build DFP test cases unconditionally
Andreas Arnez [Thu, 7 Sep 2023 14:43:25 +0000 (16:43 +0200)]
s390x regtest: Build DFP test cases unconditionally

Now that the DFP test cases no longer depend on the compiler's support for
`_Decimal*' types, they can be enabled unconditionally.  Do this in the
"Makefile.am".

15 months agos390x regtest: Rework pfpo
Andreas Arnez [Thu, 7 Sep 2023 14:43:25 +0000 (16:43 +0200)]
s390x regtest: Rework pfpo

The pfpo test case relies on `_Decimal*' type support, which currently
excludes Clang.  Rework the test to avoid this.  Also, fix a typo that
causes conversions to 64-bit decimal types to be shown as "-> d6" instead
of "-> d64" in the output.  Adjust the output file accordingly.

15 months agos390x regtest: Rework dfptest
Andreas Arnez [Thu, 7 Sep 2023 14:43:25 +0000 (16:43 +0200)]
s390x regtest: Rework dfptest

The dfptest test case uses `_Decimal*' types, which is currently not
supported by Clang.  Rework the test to avoid this.

15 months agos390x regtest: Rework dfpext
Andreas Arnez [Thu, 7 Sep 2023 14:43:25 +0000 (16:43 +0200)]
s390x regtest: Rework dfpext

The dfpext test case uses `_Decimal*' types and thus cannot be compiled
with Clang.  Avoid this.  Also, remove a duplicate in the set of tested
values and adjust the output file accordingly.

15 months agos390x regtest: Rework dfpconv
Andreas Arnez [Thu, 7 Sep 2023 14:43:25 +0000 (16:43 +0200)]
s390x regtest: Rework dfpconv

The dfpconv test case has two major issues:

* It truncates certain 64-bit integers to 32 bits when using them as
  inputs or outputs to inline assemblies.  In particular this leads to
  wrong output.

* It uses `_Decimal*' types and thus cannot be compiled with Clang.

Fix both.  Also, remove a duplicate from the set of tested numbers, remove
the dependency on "opcodes.h", and fix indentation.  Since the test output
file dfpconv.stdout.exp is affected by these changes, adjust it
accordingly.

15 months agos390x regtest: Rework dfp-4
Andreas Arnez [Thu, 7 Sep 2023 14:43:25 +0000 (16:43 +0200)]
s390x regtest: Rework dfp-4

Remove dependency on `_Decimal*' types in the dfp-4 test case, to enable
compilation with Clang.  (See Bug 465782.)

15 months agos390x regtest: Rework dfp-3
Andreas Arnez [Thu, 7 Sep 2023 14:43:25 +0000 (16:43 +0200)]
s390x regtest: Rework dfp-3

The dfp-3 test case uses _Decimal types and thus doesn't compile with
Clang.  Fix that.  Also drop the dependency on "opcodes.h" and fix
indentation.

15 months agos390x regtest: Rework dfp-2
Andreas Arnez [Thu, 7 Sep 2023 14:43:25 +0000 (16:43 +0200)]
s390x regtest: Rework dfp-2

The dfp-2 test case has two issues:

* It doesn't test the rounding modes correctly.  Since the inline assembly
  uses a "d" constraint to pass the rounding mode, a random register
  number is placed into the instruction's rounding mode field instead of
  the intended rounding mode.

* It cannot be compiled with Clang.  (See Bug 465782.)

Fix these with a rewrite.  Fixing the first issue also requires adjusting
the test case output `dfp-2.stdout.exp', while fixing the second requires
avoiding `_Decimal*' data types.  Also reduce some code duplication.

15 months agoBug 472875 - s390x regtest: Rework dfp-1
Andreas Arnez [Thu, 7 Sep 2023 14:43:24 +0000 (16:43 +0200)]
Bug 472875 - s390x regtest: Rework dfp-1

The dfp-1 test case has two main issues:

* It doesn't initialize the condition code before executing the DFP
  instructions for "multiply" and "divide", but asserts a certain value
  afterwards.  (See Bug 472875.)

* It doesn't compile with Clang.  (See Bug 465782.)

Fix these with a rework.  The first issue can be fixed by initializing the
condition code before executing the DFP instructions.  The second issue
requires avoiding any `_Decimal*' types, since they are not supported by
Clang.

15 months agos390x regtest: Fix issues with DFP_VAL_PRINT()
Andreas Arnez [Thu, 7 Sep 2023 14:43:24 +0000 (16:43 +0200)]
s390x regtest: Fix issues with DFP_VAL_PRINT()

The DFP_VAL_PRINT macro used by the dfp test cases has some issues:

* It performs illegal pointer type aliasing.

* It suppresses zero digits in 128-bit integers, since it prints their
  second halves with "%08lx" instead of "%016lx".

Fix these.  Due to the latter, this affects the output of all DFP test
cases, so adjust them as well.

15 months agos390x regtest: Reduce pfpo test case output
Andreas Arnez [Thu, 7 Sep 2023 14:43:24 +0000 (16:43 +0200)]
s390x regtest: Reduce pfpo test case output

The `pfpo' test case prints floating-point values with the "%f" and "%Lf"
formats.  Sometimes also "%lf" is used, but that's equivalent to "%f"
since the "l" length modifier doesn't affect floating-point formatting.

These "f"-style formats are not well suited for such a test case:

* They generate excessively long output for numbers with large absolute
  values.  This makes `pfpo.stdout.exp' difficult to handle.

* They can lose precision, in particular for numbers close to zero, where
  they just print zeros.

Use the formats "%a" and "%La" instead and adjust pfpo.stdout.exp
accordingly.

15 months agoSolaris regtest: aligned / sized expecteds and an improved filter
Paul Floyd [Sun, 3 Sep 2023 12:20:42 +0000 (14:20 +0200)]
Solaris regtest: aligned / sized expecteds and an improved filter

A couple of testcases fail due to ever changing output for floating
point infinity.

Add memalign and aligned_alloc expecteds for Solaris 11.3
(which doesn't have aligned_allloc).

15 months agocoverity: various fixes
Paul Floyd [Sun, 3 Sep 2023 06:28:26 +0000 (08:28 +0200)]
coverity: various fixes

Null check, uninitialized variable and several print format uses of size_t.
clang-tidy also complains about most of them.

15 months agoFreeBSD: clarify which package GNU cp is in (needed for nightly tests)
Paul Floyd [Sun, 3 Sep 2023 06:27:19 +0000 (08:27 +0200)]
FreeBSD: clarify which package GNU cp is in (needed for nightly tests)

15 months agocoverity: fix a couple of forward nulls
Paul Floyd [Sat, 2 Sep 2023 21:43:31 +0000 (23:43 +0200)]
coverity: fix a couple of forward nulls

In the new sized/aligned memcheck code.

15 months agocoverity: a few bits of dead code and one vgdb command line leak
Paul Floyd [Sat, 2 Sep 2023 21:19:17 +0000 (23:19 +0200)]
coverity: a few bits of dead code and one vgdb command line leak

15 months agoregtest: silence some more warnings
Paul Floyd [Sat, 2 Sep 2023 15:33:35 +0000 (17:33 +0200)]
regtest: silence some more warnings

A massif test that checks that --ignore-fn
Sadly neither C nor C++ has a standard way of specifying functons
to _never_ inline, and GCC and LLVM seem to have different extensions.
GCC complains about __attribute__((optnone)).

15 months agoregtest: silence a few warnings
Paul Floyd [Sat, 2 Sep 2023 15:12:23 +0000 (17:12 +0200)]
regtest: silence a few warnings

A couple of testcases that use 'new' that will throw an
uncaught exception. The variable doesn't get used, there
is only an assert to check that nothing is reached after the
new expressions.

15 months agoAdd memcheck errors for aligned and sized allocations and deallocations
Paul Floyd [Sun, 12 Mar 2023 07:26:04 +0000 (08:26 +0100)]
Add memcheck errors for aligned and sized allocations and deallocations

Bug 433857 Add validation to C++17 aligned new/delete alignment size
Bug 433859 Add mismatched detection to C++ 17 aligned new/delete
Bug 466105 aligned_alloc problems, part 2
Bug 467441 Add mismatched detection to C++ 14 sized delete

Memcheck now tests and warns about the values used for
alignment and size. These apply to various functions: memalign,
posix_memalign and aligned_alloc in C and various overloads
of operators new and delete in C++. The kinds of error that can
be detected are
- invalid alignment, for instance the alignment is usually required
  to be a power of 2
- mismatched alignment between aligned allocation and aligned
  deallocation
- mismatched size when sized delete is used
- bad size for functions that have implementation defined behaviour
  when the requested size is zero

An example of the new errors is:

int *pi = memalign(31, 100);

which will generate the following error

==96206== Invalid alignment value: 31 (should be power of 2)
==96206==    at 0x485195E: memalign (vg_replace_malloc.c:1886)
==96206==    by 0x20200E: main (memalign.c:59)

15 months agoregtest: make memcheck sem test quiet
Paul Floyd [Sat, 2 Sep 2023 12:08:07 +0000 (14:08 +0200)]
regtest: make memcheck sem test quiet

On Solaris the summary shows 7 allocations causing a diff.

15 months agoSolaris: fix a few compiler warnings
Paul Floyd [Sat, 2 Sep 2023 11:52:12 +0000 (13:52 +0200)]
Solaris: fix a few compiler warnings

15 months agoFreeBSD: Add support for building on FreeBSD 15.0
Paul Floyd [Sat, 2 Sep 2023 13:20:36 +0000 (15:20 +0200)]
FreeBSD: Add support for building on FreeBSD 15.0

15 months agoExplicitly load libc and any sonames that contain mandatory specs
Mark Wielaard [Fri, 1 Sep 2023 17:10:17 +0000 (19:10 +0200)]
Explicitly load libc and any sonames that contain mandatory specs

We really need symtab for glibc and ld.so libraries early for redir.
Some distros move these into separate debuginfo files, which means
we need to fully load them early.

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

15 months agoFix lazy debuginfo loading on ppc64le
Aaron Merey [Wed, 30 Aug 2023 18:49:09 +0000 (14:49 -0400)]
Fix lazy debuginfo loading on ppc64le

Lazy debuginfo loading introduced in commit 60f7e89ba32 assumed that
either describe_IP or find_DiCfSI will be called before stacktrace
printing.  describe_IP and find_DiCfSI cause debuginfo to be lazily
loaded before symtab lookup occurs during stacktraces.

However this assumption does not hold true on ppc64le, resulting
in debuginfo failing to load in time for stacktraces.  Fix this
by loading debuginfo during get_StackTrace_wrk on ppc arches.

15 months agoFix 473944 Handle mold linker split RW PT_LOAD segments correctly
Philippe Waroquiers [Fri, 1 Sep 2023 18:23:46 +0000 (20:23 +0200)]
Fix 473944  Handle mold linker split RW PT_LOAD segments correctly

Condition to consider segments will be merged has to be more specific
than just having a page rounded file offset p_offset.

Regtested on debian, somewhat poorly due to the amount of tests
failing due to:
473745 must-be-redirected function - strlen - for valgrind 3.22 but not 3.21

15 months agoFreeBSD: add common failure causes to README.freebsd
Paul Floyd [Fri, 1 Sep 2023 06:26:24 +0000 (08:26 +0200)]
FreeBSD: add common failure causes to README.freebsd

Also fix the name of one of the fields of vki_kinfo_vmentry.

15 months agoBug 473870 - FreeBSD 14 applications fail early at startup
Paul Floyd [Thu, 31 Aug 2023 13:32:21 +0000 (15:32 +0200)]
Bug 473870 - FreeBSD 14 applications fail early at startup

FreeBSD recently started adding some functions using @gnu_indirect_function,
specifically strpcmp which was causing this crash.

When running and encountering this ifunc Valgrind looked for the
ifunc_handler. But there wasn't one for FreeBSD so Valgrind asserted.

15 months agoFreeBSD: a bit of cleanup of README.freebsd
Paul Floyd [Wed, 30 Aug 2023 06:06:00 +0000 (08:06 +0200)]
FreeBSD: a bit of cleanup of README.freebsd

15 months agoFreeBSD: add memcheck suppression for libc newlocale still reachable memory
Paul Floyd [Tue, 29 Aug 2023 05:55:31 +0000 (07:55 +0200)]
FreeBSD: add memcheck suppression for libc newlocale still reachable memory

15 months agoLinux: remove a couple of cpmpiler warnings
Paul Floyd [Sun, 27 Aug 2023 16:03:07 +0000 (18:03 +0200)]
Linux: remove a couple of cpmpiler warnings

15 months agoFreeBSD: remove testing code
Paul Floyd [Sun, 27 Aug 2023 15:27:57 +0000 (17:27 +0200)]
FreeBSD: remove testing code

15 months agoFreeBSD: add an extra libc filter for _start in massif deep-D test
Paul Floyd [Sun, 27 Aug 2023 16:45:03 +0000 (18:45 +0200)]
FreeBSD: add an extra libc filter for _start in massif deep-D test

15 months agoRemove merge file added by mistake
Paul Floyd [Sun, 27 Aug 2023 16:33:03 +0000 (18:33 +0200)]
Remove merge file added by mistake

15 months agoFreeBSD: Add a new testcase for FreeBSD 14 _umtx_op set timeout
Paul Floyd [Sun, 27 Aug 2023 16:29:30 +0000 (18:29 +0200)]
FreeBSD: Add a new testcase for FreeBSD 14 _umtx_op set timeout

Uses an existing testcase with pthread APIs that use timeouts
and sets LIBPTHREAD_UMTX_MIN_TIMEOUT to 100 (ns, so almost no
effect other than exercising the syscall).

15 months agoFreeBSD: add 2 new _umtx_op op codes for FreeBSD 14
Paul Floyd [Sun, 27 Aug 2023 14:14:00 +0000 (16:14 +0200)]
FreeBSD: add 2 new _umtx_op op codes for FreeBSD 14

15 months agoregtest: fix compiler warnings with clang 16
Paul Floyd [Sun, 27 Aug 2023 14:57:24 +0000 (16:57 +0200)]
regtest: fix compiler warnings with clang 16

Mostly warnings about deprecated use of K&R function definitions.

15 months agoFreeBSD regtest: add suppression to bug392331_supp for FreeBSD 14.0
Paul Floyd [Sun, 27 Aug 2023 14:41:36 +0000 (16:41 +0200)]
FreeBSD regtest: add suppression to bug392331_supp for FreeBSD 14.0

15 months agoAllow spaces in .valgrindrc files
Paul Floyd [Sun, 27 Aug 2023 07:35:03 +0000 (09:35 +0200)]
Allow spaces in .valgrindrc files

The patch for m_commandline.c comes from the Debian package files.
Also add a regtest and allow --command-line-only=no to override
--command-line-only=yes

16 months agoFreeBSD regtest: use a filter to make eventfd2 consistent
Paul Floyd [Sat, 26 Aug 2023 10:57:34 +0000 (12:57 +0200)]
FreeBSD regtest: use a filter to make eventfd2 consistent

16 months agoSolaris: remove cxx17_aligned_new expected
Paul Floyd [Sat, 26 Aug 2023 05:58:10 +0000 (07:58 +0200)]
Solaris: remove cxx17_aligned_new expected

No longer needed after last cleanup

16 months agoFix various clang warnings
Paul Floyd [Fri, 25 Aug 2023 21:00:58 +0000 (23:00 +0200)]
Fix various clang warnings

Mostly missing 'void' in function prototypes.

16 months agoBug 473677 - make check compile failure with Clang 16 based on GCC 13.x
Paul Floyd [Fri, 25 Aug 2023 20:11:27 +0000 (22:11 +0200)]
Bug 473677 - make check compile failure with Clang 16 based on GCC 13.x

16 months agoSolaris: update cxx17_aligned_new expected
Paul Floyd [Thu, 24 Aug 2023 19:38:46 +0000 (21:38 +0200)]
Solaris: update cxx17_aligned_new expected

16 months agoregtest: fix wrong interfaces in calls to operator delete
Paul Floyd [Thu, 24 Aug 2023 19:06:23 +0000 (21:06 +0200)]
regtest: fix wrong interfaces in calls to operator delete

Will update Solaris later

16 months agoFix an error in `docs/README`.
Nicholas Nethercote [Wed, 23 Aug 2023 06:45:23 +0000 (16:45 +1000)]
Fix an error in `docs/README`.

16 months agoUpdate Cachegrind description in the usage message and header comments.
Nicholas Nethercote [Thu, 10 Aug 2023 04:19:36 +0000 (14:19 +1000)]
Update Cachegrind description in the usage message and header comments.

This matches an earlier change to the description in the user manual.

And fix indentation of the license in a couple of files.

16 months agoCorrect an error in Cachegrind's usage message.
Nicholas Nethercote [Wed, 23 Aug 2023 04:20:42 +0000 (14:20 +1000)]
Correct an error in Cachegrind's usage message.

16 months agoBug 473604 - Fix bug472219.c compile failure with Clang 16
Paul Floyd [Tue, 22 Aug 2023 19:35:00 +0000 (21:35 +0200)]
Bug 473604 - Fix bug472219.c compile failure with Clang 16

16 months agoAdd bug 426751 to NEWS
Paul Floyd [Tue, 22 Aug 2023 19:23:00 +0000 (21:23 +0200)]
Add bug 426751 to NEWS

16 months agogdb --multi mode stdout redirecting to stderr
Mark Wielaard [Thu, 17 Aug 2023 13:40:30 +0000 (15:40 +0200)]
gdb --multi mode stdout redirecting to stderr

When in stdio mode (talking to gdb through stdin/stdout, not
through a socket), redirect stdout to stderr and close stdin
for the inferior. That way at least some output can be seen,
but there will be no input.

This is workaround till we have real terminal handling.

     * coregrind/vgdb.c (main): Pass in_port to do_multi_mode.
     (do_multi_mode): Pass in_port to fork_and_exec_valgrind.
     (fork_and_exec_valgrind): Close stdin, redirect stdout to
     stderr if in_port <= 0.

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

16 months agomusl: enable libstdc++ freeres
Paul Floyd [Sun, 20 Aug 2023 13:24:25 +0000 (15:24 +0200)]
musl: enable libstdc++ freeres

Both libc and libstdc++ freeres were disabled for musl.
That means that libstdc++ was showing still reachable
memory on systems like Alpine.

16 months agoFreeBSD: try to make eventfd2 testcase deterministic
Paul Floyd [Sun, 20 Aug 2023 07:04:17 +0000 (09:04 +0200)]
FreeBSD: try to make eventfd2 testcase deterministic

Add a sleep to give child 1 a chance to run.
Flush stdout every time.

16 months agoFreeBSD: complete loading debuginfo if entering capability mode
Paul Floyd [Sun, 20 Aug 2023 06:52:36 +0000 (08:52 +0200)]
FreeBSD: complete loading debuginfo if entering capability mode

16 months agoAlways cleanup on exit from ML_(read_elf_object)
Paul Floyd [Sat, 19 Aug 2023 19:37:33 +0000 (21:37 +0200)]
Always cleanup on exit from ML_(read_elf_object)

I'm still a but baffled as to why this wasn't seen earlier.
A FreeBSD testcase started failing with kernel 13.2 patch 2,
which is quite a minor change. The testcase gets an fd from
pdfork and the parent does a printf with the fd then zaps the
process with pdkill. Standalone the fd is 3, and that's what
the expected contains. However, when it started failing I saw
with lsof that fds 3 and 4 were associated with the guest exe
and ld-elf.so.1.

16 months agoSolaris: explain configure detection of scf repository door version
Paul Floyd [Fri, 18 Aug 2023 06:12:53 +0000 (08:12 +0200)]
Solaris: explain configure detection of scf repository door version

16 months agoBug 472963 - Broken regular expression in configure.ac
Paul Floyd [Thu, 17 Aug 2023 20:05:47 +0000 (22:05 +0200)]
Bug 472963 - Broken regular expression in configure.ac

Was extracting the last two decimal digits from a hex humber.

I switched to using grep and sed because the proposed solution
didn't work on Solaris 11.3.

16 months agoSupport lazy reading and downloading of DWARF debuginfo
Aaron Merey [Fri, 30 Jun 2023 22:31:42 +0000 (18:31 -0400)]
Support lazy reading and downloading of DWARF debuginfo

Currently valgrind attempts to read DWARF .debug_* sections as well
as separate debuginfo files for ELF binaries as soon as a shared library
is loaded.  This might also result in the downloading of separate debuginfo
files via debuginfod.

This is inefficient when some of this debuginfo never ends up being used
by valgrind while running the client process.

This patch adds support for lazy reading and downloading of DWARF
debuginfo.  When an ELF shared library is loaded, the reading of .debug_*
sections as well as separate or alternate debuginfo is deferred until
valgrind handles an instruction pointer corresponding to a text segment
of the shared library.  At this point the deferred sections and separate
debug files are loaded.

This feature is only supported on ELF platforms.

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

ChangeLog
* debuginfo.c (di_notify_ACHIEVE_ACCEPT_STATE): Replace
read_elf_debug_info with read_elf_object.
(addr_load_di): New function.  Attempts to load deferred debuginfo
associated with a given address.
(load_di): New function.  Attempts to load a given deferred
debuginfo associated with a given address.
(describe_IP): Add calls to load_di and addr_load_di.
(find_DiCfSI): Add call to load_di.

* priv_readelf.h (read_elf_object): New declaration.
(read_elf_debug): Ditto.

* priv_storage.h (struct _DebugInfo): New field 'bool deferred'.

* readelf.c (read_elf_debug_info): Split into read_elf_object and
read_elf_debug.
(read_elf_object): Read non .debug_* section from an ELF binary.
(read_elf_debug): Read .debug_* sections from an ELF binary as
as well any separate/alternate debuginfo files.

* storage.c (canonicaliseSymtab): Remove assert in order to support
canonicalization of deferred _DebugInfo.
(finish_CFSI_arrays): Add early return if _DebugInfo is
deferred in order to avoid freeing memory that will be needed
when reading debuginfo at a later time.
(canonicaliseTables): Ditto.

* pub_core_debuginfo.h (addr_load_di): New declaration.
(load_di): New declaration.

17 months agoBug 472219 - Syscall param ppoll(ufds.events) points to uninitialised byte(s)
Paul Floyd [Mon, 24 Jul 2023 20:06:00 +0000 (22:06 +0200)]
Bug 472219 - Syscall param ppoll(ufds.events) points to uninitialised byte(s)

Add checks that (p)poll fd is not negative. If it is negative, don't check
the events field.

17 months agoSolaris: add a configure test for getaddrinfo
Paul Floyd [Mon, 24 Jul 2023 19:32:45 +0000 (21:32 +0200)]
Solaris: add a configure test for getaddrinfo

Not available on Solaris 11.3

17 months agoFreeBSD: Add a DRD supppression for getaddrinfo
Paul Floyd [Sun, 23 Jul 2023 17:22:51 +0000 (19:22 +0200)]
FreeBSD: Add a DRD supppression for getaddrinfo

On FreeBSD 13.2 x86

17 months agoBug 470132 - s390x: Increase test coverage for VGM
Andreas Arnez [Mon, 22 May 2023 17:49:08 +0000 (19:49 +0200)]
Bug 470132 - s390x: Increase test coverage for VGM

Add more tests for the VGM instruction, to verify the fix for the VGM
wrap-around case.  Also test setting unused bits in the I2 and I3 fields,
to check that Valgrind ignores them as it should.

17 months agoBug 470132 - s390x: Fix the wrap-around case in VGM
Andreas Arnez [Mon, 22 May 2023 16:57:35 +0000 (18:57 +0200)]
Bug 470132 - s390x: Fix the wrap-around case in VGM

Valgrind's implementation of VGM is incomplete:

* It doesn't support generating a wrap-around bit mask.  Such a mask
  should result when the ending bit position is smaller than the starting
  bit position.  Valgrind runs into an assertion failure instead.

* It doesn't ignore unused bits in the I2 and I3 fields of the
  instruction, as it should.

Fix this by re-implementing the main logic in s390_irgen_VGM().

17 months agoFreeBSD: add default to configure.ac FreeBSD 13 versions
Paul Floyd [Sun, 2 Jul 2023 10:59:40 +0000 (12:59 +0200)]
FreeBSD: add default to configure.ac FreeBSD 13 versions

Also add comment to README.freebsd about ensuring that
jails set "uname -r" to be something compatible with the
normal RELEASE/STABLE/CURRENT releases.

18 months agoBug 470978 - s390x: Link the tools with -Wl,--s390-pgste
Andreas Arnez [Thu, 15 Jun 2023 15:24:53 +0000 (17:24 +0200)]
Bug 470978 - s390x: Link the tools with -Wl,--s390-pgste

Programs that require the PGSTE mode to be enabled may currently fail
under Valgrind.  In particular this affects qemu-kvm.

While it is also possible to enable the PGSTE mode globally with

  sysctl vm.allocate_psgte=1

the problem can more easily be prevented by linking the Valgrind tools
with -Wl,--s390-pgste.  Add a configure check if the linker supports this,
and activate the flag if it does.

To verify the intended result, the following shell command can be used to
list the executables having this flag set:

  find . -type f -perm -u+x -execdir \
       /bin/sh -c 'readelf -lW $0 2>/dev/null | grep PGSTE' {} \; -print

18 months agoDon't print action vgdb me ... and continuing ... in vgdb --multi mode
Mark Wielaard [Fri, 9 Jun 2023 13:21:57 +0000 (15:21 +0200)]
Don't print action vgdb me ... and continuing ... in vgdb --multi mode

Guard each (action) vgdb me ... VG_(umsg) printing with
  !(VG_(clo_launched_with_multi))

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

18 months agos390x: Replace absolute jump for a relative one
Tulio Magno Quites Machado Filho [Wed, 14 Jun 2023 14:28:38 +0000 (11:28 -0300)]
s390x: Replace absolute jump for a relative one

The bne instruction expects an absolute target address and it isn't
best-suited for implementing a short range jump, such as the one in
XCHG_M_R().
Replace it with jne which expects a relative address that can be
correctly computed a link time.

Interestingly, the jump is almost never taken. If it would, this would
crash the test.  However, linkers may complain when relacating the
target address used in bne.

18 months ago470121 - Can't run callgrind_control with valgrind 3.21.0 because of perl errors
Paul Floyd [Fri, 9 Jun 2023 11:17:58 +0000 (13:17 +0200)]
470121 - Can't run callgrind_control with valgrind 3.21.0 because of perl errors

18 months agoMerge error, missing continuation in Makefile.am
Paul Floyd [Wed, 7 Jun 2023 20:54:22 +0000 (22:54 +0200)]
Merge error, missing continuation in Makefile.am

18 months agoBug 470713 - Failure on the Yosys project: valgrind: m_libcfile.c:1802 (Bool vgPlain...
Paul Floyd [Wed, 7 Jun 2023 20:27:08 +0000 (22:27 +0200)]
Bug 470713 -  Failure on the Yosys project: valgrind: m_libcfile.c:1802 (Bool vgPlain_realpath(const HChar *, HChar *)): Assertion 'resolved' failed

    When using sysctl kern proc pathname with the pid of the guest or -1
    we need to intercept the call otherwise the syscall will return the path
    of the memcheck tool and not the path of the guest.

    This uses VG_(realpath), which asserts if it doesn't get valid
    input pointers.

    sysctl kern proc pathname can use a NULL pointer in order to
    determine the length of the path (so users can allocate the minumum
    necessary). The NULL pointer was being passed on to VG_(realpath)
    without being checked, resulting in an assert.

18 months agomemcheck: Handle Err_ReallocSizeZero in MC_(eq_Error)
Mark Wielaard [Thu, 1 Jun 2023 14:10:56 +0000 (16:10 +0200)]
memcheck: Handle Err_ReallocSizeZero in MC_(eq_Error)

When an realloc size zero error is emitted MC_(eq_Error) is called to
see if the errors can be deduplicated. This crashed since
Err_ReallocSizeZero wasn't handled. Handle it like Err_Free.

Also add a testcase for this case and test with both
--realloc-zero-bytes-frees=yes and
--realloc-zero-bytes-frees=no.
Which will report a different number of errors.

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

19 months agoAdd --with-gdbscripts-dir=PATH configure option
Mark Wielaard [Sun, 14 May 2023 21:34:05 +0000 (23:34 +0200)]
Add --with-gdbscripts-dir=PATH configure option

Currently the gdb valgrind scripts are installed under VG_LIBDIR
which is normally pkglibexecdir which is likely not in the default
gdb safe-path (a list of directories from which it is safe to
auto-load files). So users will have to add the directory to their
.gdbinit file.

This patch adds a --with-gdbscripts-dir=PATH configure option that
sets VG_GDBSCRIPTS_DIR to the given PATH (${libexecdir}/valgrind if
not given). A user can also configure --without-gdbscripts-dir to
disable adding a .debug_gdb_scripts section to the vgpreload library
and installing the valgrind-monitor python scripts completely.

Use VG_GDBSCRIPTS_DIR as gdbscriptsdir to install the valgrind-monitor
python files and pass it with CPPFLAGS when building vg_preloaded.c
and vgdb.c to use instead of VG_LIBDIR.

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

19 months agoAdd epoll_pwait2
Olli Vanhoja [Mon, 10 Oct 2022 09:44:22 +0000 (11:44 +0200)]
Add epoll_pwait2

The only difference between epoll_pwait and epoll_pwait2 is the
timeout argument. For epoll_pwait2 this is a timespec (which is
always 64bit).

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

19 months agoTypo in massif tests Makefile.am
Paul Floyd [Wed, 17 May 2023 06:09:39 +0000 (08:09 +0200)]
Typo in massif tests Makefile.am

19 months agoLinux regtest: fixes for building on musl x86
Paul Floyd [Tue, 16 May 2023 06:20:59 +0000 (08:20 +0200)]
Linux regtest: fixes for building on musl x86

This page took 0.070108 seconds and 5 git commands to generate.