From nobody Tue Dec 2 01:36:23 2025 Received: from mail-pg1-f202.google.com (mail-pg1-f202.google.com [209.85.215.202]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 979302F99A3 for ; Sat, 22 Nov 2025 08:19:57 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.215.202 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763799599; cv=none; b=bIs4OsuDOIVncQghs/KdaMv9KHCALw0DTXkXt6zampFjj1Pr/1oNhLeKSpjL6tdYFpetK26Xjr1JY3HATQFTEwPvKBrrkwhnTEM/MsPLPMlfe82W0ufIsj02sllwl/z/mvF57HFWtl5vNwJGv92IMU67CqBkW7DxTRnLmYuB3XE= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763799599; c=relaxed/simple; bh=3qBlJKk1kVkHDxdezjAgYYdWUiOjKjzmMQFfmn1yqOY=; h=Date:In-Reply-To:Mime-Version:References:Message-ID:Subject:From: To:Content-Type; b=gfq7oAsqoUhwb62hHmg0wPeIYY9cG/G6+WJVeHWW19gJC5EZ3OrpfDQcb/gI+R3+sQCHU3s/vNKFAWnlo6mNqVgwzvIoyZG6jlY58AogwAxChDiL7A3SzW5IoT9VfBDXe3Hj7x44Ra2oQ5hzrQccU0HVBjmQwrEl0OgvhcMvfcc= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com; spf=pass smtp.mailfrom=flex--irogers.bounces.google.com; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b=OBhg+LWH; arc=none smtp.client-ip=209.85.215.202 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=flex--irogers.bounces.google.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b="OBhg+LWH" Received: by mail-pg1-f202.google.com with SMTP id 41be03b00d2f7-b5edecdf94eso5240171a12.2 for ; Sat, 22 Nov 2025 00:19:57 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20230601; t=1763799597; x=1764404397; darn=vger.kernel.org; h=to:from:subject:message-id:references:mime-version:in-reply-to:date :from:to:cc:subject:date:message-id:reply-to; bh=kTgSld+zWldBlIj9iQhUFIRYK7znK8Muttybqj8QK0I=; b=OBhg+LWHMe5qgXbtiPl84PR5Qm9nLEkw/mEW5zbaJh0cxNDKuZ2D1bIflQmoqweMm+ 5mRD4NiFE9gZgmuljfwGd6dODioAtS+POKWZV1f320dNnfFhAo913IjIfl41ZjjRJXvw s+aramDbabMdXw3HPiV8TNv1RifKdazaI+KozrRNv8To07AjS3Mf/hqn1c8zenpIF3x9 uWq7Nh3YHmcqyfLN5SJrTKBf7GxmshOiQdXArDRBNb3mdGgx7hC67KaIgWb7MoFYNLsj 1XTMHDLPwL4Igm51t5OCuqy5GAKvTg9+3ClTFA7uzR3YxjDPzPgM4/ObxMR/bTOASTQe vYzQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1763799597; x=1764404397; h=to:from:subject:message-id:references:mime-version:in-reply-to:date :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=kTgSld+zWldBlIj9iQhUFIRYK7znK8Muttybqj8QK0I=; b=U9i3kjQoPxNP8rjqefSk0oFk3kUYPpZDMB7duo1j0aV+A9YuWkKhjdbs8HMiAVTFWS 23dtV9mywBxopKVHUH3wHNPgXYtvPbggWx4DK9pb82PevHM3Yw1Qh04wWCaaKP6zJzrq 43gdVlQSZ2XqkAq4eqycxTM3NKsYQF3AD7h9MrLEvDyPdlBq2cxy60IqOxlXeyJdPuiA KsjDuLaGtvYE7T7ACOOZ3MkLDkx5mdabSx+RVHiRNzi5kUM8Jxi+Uz2D1vOZfIOOj/20 PdSboPtBpjvC9QTCGXp999USwtdGA9dkhbAfs4lBg9m4cxvmSvn85UhXeB1x3qbfFzLu 29Fw== X-Forwarded-Encrypted: i=1; AJvYcCX2E3sGEyz9fnA062bc66vDtFWDPRjBhYnfmUp7lJZrt5lmYqEA3qDKLXzev4q7E8V7W68B7BtCF1jKoOI=@vger.kernel.org X-Gm-Message-State: AOJu0YwPkB3wkshUV+yYpLVzQ2N0A8NUrrpPE8AjhA3hn406NQRy7Gug Yj6k+97VKkG9jiV0gBIz6LHisxzmdh7uBeKdsdDmLZHBmLU6tA3PVPefHa/o/0/8+4y1FYeR5vf no8faLyfJaw== X-Google-Smtp-Source: AGHT+IEGZTx82yh6ivSmLyr5eVGFroWnq6Ir7ehD9mrmhm4DljU3Ki7EE9oibo+Jz1VZpx2AIRZHFRGUQN6s X-Received: from dyaa12.prod.google.com ([2002:a05:693c:40cc:b0:2a2:4eb1:3771]) (user=irogers job=prod-delivery.src-stubby-dispatcher) by 2002:a05:7301:5502:b0:2a4:7f22:cc0d with SMTP id 5a478bee46e88-2a7192ce4b1mr2237806eec.32.1763799596840; Sat, 22 Nov 2025 00:19:56 -0800 (PST) Date: Sat, 22 Nov 2025 00:19:23 -0800 In-Reply-To: <20251122081929.7588-1-irogers@google.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 References: <20251122081929.7588-1-irogers@google.com> X-Mailer: git-send-email 2.52.0.rc2.455.g230fcf2819-goog Message-ID: <20251122081929.7588-12-irogers@google.com> Subject: [PATCH v1 11/17] perf tests buildid: Add purge and remove testing From: Ian Rogers To: Peter Zijlstra , Ingo Molnar , Arnaldo Carvalho de Melo , Namhyung Kim , Alexander Shishkin , Jiri Olsa , Ian Rogers , Adrian Hunter , James Clark , Collin Funk , Dmitry Vyukov , Andi Kleen , Thomas Falcon , Leo Yan , Yicong Yang , Greg Kroah-Hartman , "Masami Hiramatsu (Google)" , Stephen Brennan , Haibo Xu , linux-kernel@vger.kernel.org, linux-perf-users@vger.kernel.org Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Add testing for the purge and remove commands. Use the noploop workload rather than just a return to avoid missing samples in the workload in perf record. Tidy up the cleanup code to cleanup when signals happen. Signed-off-by: Ian Rogers --- tools/perf/tests/shell/buildid.sh | 203 ++++++++++++++++++++++++++---- 1 file changed, 177 insertions(+), 26 deletions(-) diff --git a/tools/perf/tests/shell/buildid.sh b/tools/perf/tests/shell/bui= ldid.sh index d2eb213da01d..102808cca9db 100755 --- a/tools/perf/tests/shell/buildid.sh +++ b/tools/perf/tests/shell/buildid.sh @@ -36,16 +36,69 @@ if [ ${run_pe} -eq 1 ]; then unset WAYLAND_DISPLAY fi =20 -ex_md5=3D$(mktemp /tmp/perf.ex.MD5.XXX) -ex_sha1=3D$(mktemp /tmp/perf.ex.SHA1.XXX) +build_id_dir=3D +ex_source=3D$(mktemp /tmp/perf_buildid_test.ex.XXX.c) +ex_md5=3D$(mktemp /tmp/perf_buildid_test.ex.MD5.XXX) +ex_sha1=3D$(mktemp /tmp/perf_buildid_test.ex.SHA1.XXX) ex_pe=3D$(dirname $0)/../pe-file.exe +data=3D$(mktemp /tmp/perf_buildid_test.data.XXX) +log_out=3D$(mktemp /tmp/perf_buildid_test.log.out.XXX) +log_err=3D$(mktemp /tmp/perf_buildid_test.log.err.XXX) =20 -echo 'int main(void) { return 0; }' | cc -Wl,--build-id=3Dsha1 -o ${ex_sha= 1} -x c - -echo 'int main(void) { return 0; }' | cc -Wl,--build-id=3Dmd5 -o ${ex_md5}= -x c - +cleanup() { + rm -f ${ex_source} ${ex_md5} ${ex_sha1} ${data} ${log_out} ${log_err} + if [ ${run_pe} -eq 1 ]; then + rm -r ${wineprefix} + fi + if [ -d ${build_id_dir} ]; then + rm -rf ${build_id_dir} + fi + trap - EXIT TERM INT +} + +trap_cleanup() { + echo "Unexpected signal in ${FUNCNAME[1]}" + cleanup + exit 1 +} +trap trap_cleanup EXIT TERM INT + +# Test program based on the noploop workload. +cat < ${ex_source} +#include +#include +#include =20 +static volatile sig_atomic_t done; + +static void sighandler(int sig) +{ + (void)sig; + done =3D 1; +} + +int main(int argc, const char **argv) +{ + int sec =3D 1; + + if (argc > 1) + sec =3D atoi(argv[1]); + + signal(SIGINT, sighandler); + signal(SIGALRM, sighandler); + alarm(sec); + + while (!done) + continue; + + return 0; +} +EOF +cc -Wl,--build-id=3Dsha1 ${ex_source} -o ${ex_sha1} -x c - +cc -Wl,--build-id=3Dmd5 ${ex_source} -o ${ex_md5} -x c - echo "test binaries: ${ex_sha1} ${ex_md5} ${ex_pe}" =20 -check() +get_build_id() { case $1 in *.exe) @@ -64,6 +117,15 @@ check() id=3D`readelf -n ${1} 2>/dev/null | grep 'Build ID' | awk '{print $3}'` ;; esac + echo ${id} +} + +check() +{ + file=3D$1 + perf_data=3D$2 + + id=3D$(get_build_id $file) echo "build id: ${id}" =20 id_file=3D${id#??} @@ -76,45 +138,53 @@ check() exit 1 fi =20 - file=3D${build_id_dir}/.build-id/$id_dir/`readlink ${link}`/elf - echo "file: ${file}" + cached_file=3D${build_id_dir}/.build-id/$id_dir/`readlink ${link}`/elf + echo "file: ${cached_file}" =20 # Check for file permission of original file # in case of pe-file.exe file echo $1 | grep ".exe" if [ $? -eq 0 ]; then - if [ -x $1 ] && [ ! -x $file ]; then - echo "failed: file ${file} executable does not exist" + if [ -x $1 ] && [ ! -x $cached_file ]; then + echo "failed: file ${cached_file} executable does not exist" exit 1 fi =20 - if [ ! -x $file ] && [ ! -e $file ]; then - echo "failed: file ${file} does not exist" + if [ ! -x $cached_file ] && [ ! -e $cached_file ]; then + echo "failed: file ${cached_file} does not exist" exit 1 fi - elif [ ! -x $file ]; then - echo "failed: file ${file} does not exist" + elif [ ! -x $cached_file ]; then + echo "failed: file ${cached_file} does not exist" exit 1 fi =20 - diff ${file} ${1} + diff ${cached_file} ${1} if [ $? -ne 0 ]; then - echo "failed: ${file} do not match" + echo "failed: ${cached_file} do not match" exit 1 fi =20 - ${perf} buildid-cache -l | grep ${id} + ${perf} buildid-cache -l | grep -q ${id} if [ $? -ne 0 ]; then echo "failed: ${id} is not reported by \"perf buildid-cache -l\"" exit 1 fi =20 + if [ -n "${perf_data}" ]; then + ${perf} buildid-list -i ${perf_data} | grep -q ${id} + if [ $? -ne 0 ]; then + echo "failed: ${id} is not reported by \"perf buildid-list -i ${perf_da= ta}\"" + exit 1 + fi + fi + echo "OK for ${1}" } =20 test_add() { - build_id_dir=3D$(mktemp -d /tmp/perf.debug.XXX) + build_id_dir=3D$(mktemp -d /tmp/perf_buildid_test.debug.XXX) perf=3D"perf --buildid-dir ${build_id_dir}" =20 ${perf} buildid-cache -v -a ${1} @@ -128,12 +198,88 @@ test_add() rm -rf ${build_id_dir} } =20 +test_remove() +{ + build_id_dir=3D$(mktemp -d /tmp/perf_buildid_test.debug.XXX) + perf=3D"perf --buildid-dir ${build_id_dir}" + + ${perf} buildid-cache -v -a ${1} + if [ $? -ne 0 ]; then + echo "failed: add ${1} to build id cache" + exit 1 + fi + + id=3D$(get_build_id ${1}) + if ! ${perf} buildid-cache -l | grep -q ${id}; then + echo "failed: ${id} not in cache" + exit 1 + fi + + ${perf} buildid-cache -v -r ${1} + if [ $? -ne 0 ]; then + echo "failed: remove ${id} from build id cache" + exit 1 + fi + + if ${perf} buildid-cache -l | grep -q ${id}; then + echo "failed: ${id} still in cache after remove" + exit 1 + fi + + echo "remove: OK" + rm -rf ${build_id_dir} +} + +test_purge() +{ + build_id_dir=3D$(mktemp -d /tmp/perf_buildid_test.debug.XXX) + perf=3D"perf --buildid-dir ${build_id_dir}" + + id1=3D$(get_build_id ${ex_sha1}) + ${perf} buildid-cache -v -a ${ex_sha1} + if ! ${perf} buildid-cache -l | grep -q ${id1}; then + echo "failed: ${id1} not in cache" + exit 1 + fi + + id2=3D$(get_build_id ${ex_md5}) + ${perf} buildid-cache -v -a ${ex_md5} + if ! ${perf} buildid-cache -l | grep -q ${id2}; then + echo "failed: ${id2} not in cache" + exit 1 + fi + + # Purge by path + ${perf} buildid-cache -v -p ${ex_sha1} + if [ $? -ne 0 ]; then + echo "failed: purge build id cache of ${ex_sha1}" + exit 1 + fi + + ${perf} buildid-cache -v -p ${ex_md5} + if [ $? -ne 0 ]; then + echo "failed: purge build id cache of ${ex_md5}" + exit 1 + fi + + # Verify both are gone + if ${perf} buildid-cache -l | grep -q ${id1}; then + echo "failed: ${id1} still in cache after purge" + exit 1 + fi + + if ${perf} buildid-cache -l | grep -q ${id2}; then + echo "failed: ${id2} still in cache after purge" + exit 1 + fi + + echo "purge: OK" + rm -rf ${build_id_dir} +} + test_record() { - data=3D$(mktemp /tmp/perf.data.XXX) - build_id_dir=3D$(mktemp -d /tmp/perf.debug.XXX) - log_out=3D$(mktemp /tmp/perf.log.out.XXX) - log_err=3D$(mktemp /tmp/perf.log.err.XXX) + build_id_dir=3D$(mktemp -d /tmp/perf_buildid_test.debug.XXX) perf=3D"perf --buildid-dir ${build_id_dir}" =20 echo "running: perf record $*" @@ -145,7 +291,7 @@ test_record() fi =20 args=3D"$*" - check ${args##* } + check ${args##* } ${data} =20 rm -f ${log_out} ${log_err} rm -rf ${build_id_dir} @@ -166,10 +312,15 @@ if [ ${run_pe} -eq 1 ]; then test_record wine ${ex_pe} fi =20 -# cleanup -rm ${ex_sha1} ${ex_md5} -if [ ${run_pe} -eq 1 ]; then - rm -r ${wineprefix} +# remove binaries manually via perf buildid-cache -r +test_remove ${ex_sha1} +test_remove ${ex_md5} +if [ ${add_pe} -eq 1 ]; then + test_remove ${ex_pe} fi =20 +# purge binaries manually via perf buildid-cache -p +test_purge + +cleanup exit 0 --=20 2.52.0.rc2.455.g230fcf2819-goog