On Fri, 7 Aug 2026 16:15:10 +0800 "Rui Qi" <[email protected]> wrote:
> Hi, > > This series fixes two ways add_remove_uprobe can construct an invalid > uprobe offset. Please send the series in the new thread (not reply) next time. Instead, you can make a permalink to the previous discussion from your mail's message-id and write it in the mail body so that we can follow the discussion in the previous version. e.g. https://lore.kernel.org/all/[email protected]/ You can get this using below script. ----- #!/bin/sh # if [ -f "$1" ]; then MID=`grep -i ^"Message-Id:" $1 | cut -f2 -d: | tr -d ' <>'` else MID=$1 fi URL=`curl -Ls -o /dev/null -w %{url_effective} http://lkml.kernel.org/r/$MID` echo ${URL%T/#u} ----- Thank you, > > Patch 1 follows Masami's feedback on v1 and forces the C locale in the > top-level ftracetest runner, rather than setting LC_ALL=C around each > readelf invocation. > > Patch 2 fixes the remaining offset issue. readelf reports the ELF entry > point as a virtual address, while uprobe_events expects a file offset. > Convert the entry point through the LOAD program headers so the test also > works for non-PIE executables. > > Changes in v2: > - Move LC_ALL=C to the top-level ftracetest runner, as suggested by > Masami. > - Keep the entry-point-to-file-offset conversion as the second patch and > rely on the runner-provided locale for readelf output. > > Testing: > - ./scripts/checkpatch.pl --strict -g HEAD~2..HEAD > - sh -n tools/testing/selftests/ftrace/ftracetest > - sh -n tools/testing/selftests/ftrace/test.d/dynevent/add_remove_uprobe.tc > > Rui Qi (2): > selftests/ftrace: Force C locale in ftracetest > selftests/ftrace: Convert ELF entry point to file offset in uprobe > test > > tools/testing/selftests/ftrace/ftracetest | 3 +++ > .../test.d/dynevent/add_remove_uprobe.tc | 27 +++++++++++++++++-- > 2 files changed, 28 insertions(+), 2 deletions(-) > > -- > 2.20.1 > -- Masami Hiramatsu (Google) <[email protected]>
