From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-dl1-f74.google.com (mail-dl1-f74.google.com [74.125.82.74]) (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 D629C36EAA8 for ; Sat, 21 Mar 2026 06:14:56 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=74.125.82.74 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1774073698; cv=none; b=HZv0Y3hnPzu4J71pEof2WnS3MJjs2Y17azUHEQWlPlZ11modspiJs7ywhnPBphJI5CGGoToWlp2siY7/6frqDDquVDNAaltqlRo79SfGiS9zheNZl8ElCAiysQfMXbJmD9BuKe/ejSb3tBWj3rnTg8y+aAQI1mVmktm78iipeZE= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1774073698; c=relaxed/simple; bh=RaVBhRwD/TXOFi3vjdfMkXMF7yO5ilaxhU/GUdD89vo=; h=Date:In-Reply-To:Mime-Version:References:Message-ID:Subject:From: To:Content-Type; b=g9rwkUSs3EpPoR5bvbQ1YtZyCDicH9K67CLzspNwtuvz0MDT/iv8eqkMC+qOZ35/uVEQUoqcRl8b8M+rHd6+4uXddg4swEQ18PDOagOaIqraOBlNarREmxPLNMdCXhtFex3t155sY0DUXm0RFZ1abyT6evdQXKo8EE8xgYvQKwc= 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=tL6iefNm; arc=none smtp.client-ip=74.125.82.74 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="tL6iefNm" Received: by mail-dl1-f74.google.com with SMTP id a92af1059eb24-12711ec96fbso13711791c88.0 for ; Fri, 20 Mar 2026 23:14:56 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20251104; t=1774073696; x=1774678496; 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=3aUi2AhNT7jHBIGjkzfrKcmaX2J03AlRffrZ+lkaYXk=; b=tL6iefNmIhmMHT9SmT063GLqqmr5h1q1RG9K9/1pXXW851lsc4BVrh5UehNwXndoWp vGyn+6wnjX9r1nFMvKnH1Nv2RXoOQkpqby2n5q7Wd0mV2kIGbXdCyLCcr83N8jarjgfG Fh9w/RWPEpwEDewlsb6XsnK08epr99zSMBhrhWpg4o0FmeFMZjVVjwQq6ltbEzEaGY5W rDSKnuA+D1lsX3lOMzPo7+2fWkpsS8ayTUKvtHW2aiOpIUZOkkZ5voCeeokXxJ66RsP+ NodhroMnCRl24aqOE8/pKs7hHCNTx5dPeCfnMduKkMyRVkvsU168TzZAPzLqjG+396Ub KrSg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1774073696; x=1774678496; 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=3aUi2AhNT7jHBIGjkzfrKcmaX2J03AlRffrZ+lkaYXk=; b=K1P5o2h5uJnRCMfbxZVb0H3YKTmy9XmJFSKRQgF4zhXjZcNQXnI+ujtQIJxIeUtlCU ohnHkhimuSYo1h2lT0IbPvd0AY/nQ0NCjL8MdXcAW9eQiNkYuzjIyNsBa8hMp94y62H8 mv+rx/UegwsESU/MjkKdC3AGx9hKnqGO2LaB6w8+xKcBhz2s5TE4JQ4CNgOuFro5+z/w 4hY19nss3q/4D9JMjDaig8AQB8adboWSAPxPf2nY9w74gq37YvV5Funio4xZIwgCC4k3 WysojkLo892GUyOTXA3xG5Q672kixBsIKoIqYqXtTsrQBAQaLFMZAXYUa4QsoeXmRUZx 5L3A== X-Forwarded-Encrypted: i=1; AJvYcCX5L8Tm9iCWUaNInxe/BnpGYHtg2t/bPnbC/MJ63eSj5XCoo1udF8HLIPcfNteuJQ6LHa9KSINpThHzkBc=@vger.kernel.org X-Gm-Message-State: AOJu0YyAgdeiObdu/ABzUxo732v2q42YMGN3hhPc+/WXHO9R67wTHD0O ktqW+IX2JzK7VQtzkDU/0clecg58Xk6LPEpBvODwmB3PMiOF3Pqf6M6Vcrho+nGFpwnQL5CV2EP wXW2H9jOxkQ== X-Received: from dlai17.prod.google.com ([2002:a05:701b:2711:b0:128:e1c4:18c9]) (user=irogers job=prod-delivery.src-stubby-dispatcher) by 2002:a05:7022:4a1:b0:12a:6a64:8212 with SMTP id a92af1059eb24-12a72648524mr2790183c88.4.1774073695747; Fri, 20 Mar 2026 23:14:55 -0700 (PDT) Date: Fri, 20 Mar 2026 23:14:48 -0700 In-Reply-To: <20260321061448.810525-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: <20260321061448.810525-1-irogers@google.com> X-Mailer: git-send-email 2.53.0.959.g497ff81fa9-goog Message-ID: <20260321061448.810525-2-irogers@google.com> Subject: [PATCH v1 2/2] perf tests sched stats: Write output to temp file From: Ian Rogers To: Peter Zijlstra , Ingo Molnar , Arnaldo Carvalho de Melo , Namhyung Kim , Alexander Shishkin , Jiri Olsa , Ian Rogers , Adrian Hunter , James Clark , Swapnil Sapkal , linux-perf-users@vger.kernel.org, linux-kernel@vger.kernel.org Content-Type: text/plain; charset="UTF-8" Writing to the perf.data file can fail in various contexts such as continual test. Other tests write to a mktemp-ed file, make the "perf sched stats tests" follow this convention. Signed-off-by: Ian Rogers --- tools/perf/tests/shell/perf_sched_stats.sh | 37 ++++++++++++++++------ 1 file changed, 27 insertions(+), 10 deletions(-) diff --git a/tools/perf/tests/shell/perf_sched_stats.sh b/tools/perf/tests/shell/perf_sched_stats.sh index 2b1410b050d0..bef7714ef37a 100755 --- a/tools/perf/tests/shell/perf_sched_stats.sh +++ b/tools/perf/tests/shell/perf_sched_stats.sh @@ -4,10 +4,29 @@ set -e +perfdata=$(mktemp /tmp/__perf_test_sched_stats.perf.data.XXXXX) +perfdata2=$(mktemp /tmp/__perf_test_sched_stats.perf.data.XXXXX) + +cleanup() { + rm -f "${perfdata}" + rm -f "${perfdata}".old + rm -f "${perfdata2}" + rm -f "${perfdata2}".old + + trap - EXIT TERM INT +} + +trap_cleanup() { + echo "Unexpected signal in ${FUNCNAME[1]}" + cleanup + exit 1 +} +trap trap_cleanup EXIT TERM INT + err=0 test_perf_sched_stats_record() { echo "Basic perf sched stats record test" - if ! perf sched stats record true 2>&1 | \ + if ! perf sched stats record -o "${perfdata}" true 2>&1 | \ grep -E -q "[ perf sched stats: Wrote samples to perf.data ]" then echo "Basic perf sched stats record test [Failed]" @@ -19,15 +38,13 @@ test_perf_sched_stats_record() { test_perf_sched_stats_report() { echo "Basic perf sched stats report test" - perf sched stats record true > /dev/null - if ! perf sched stats report 2>&1 | grep -E -q "Description" + perf sched stats record -o "${perfdata}" true > /dev/null + if ! perf sched stats report -i "${perfdata}" 2>&1 | grep -E -q "Description" then echo "Basic perf sched stats report test [Failed]" err=1 - rm perf.data return fi - rm perf.data echo "Basic perf sched stats report test [Success]" } @@ -44,16 +61,14 @@ test_perf_sched_stats_live() { test_perf_sched_stats_diff() { echo "Basic perf sched stats diff test" - perf sched stats record true > /dev/null - perf sched stats record true > /dev/null - if ! perf sched stats diff > /dev/null + perf sched stats record -o "${perfdata}" true > /dev/null + perf sched stats record -o "${perfdata2}" true > /dev/null + if ! perf sched stats diff "${perfdata}" "${perfdata2}" > /dev/null then echo "Basic perf sched stats diff test [Failed]" err=1 - rm perf.data.old perf.data return fi - rm perf.data.old perf.data echo "Basic perf sched stats diff test [Success]" } @@ -61,4 +76,6 @@ test_perf_sched_stats_record test_perf_sched_stats_report test_perf_sched_stats_live test_perf_sched_stats_diff + +cleanup exit $err -- 2.53.0.959.g497ff81fa9-goog