From nobody Wed Nov 27 16:37:14 2024 Received: from mgamail.intel.com (mgamail.intel.com [198.175.65.9]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id EAA1B18BC26; Wed, 9 Oct 2024 09:54:17 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=198.175.65.9 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1728467659; cv=none; b=O4fezxVE1i2m4VpQ/GQ2yLaSrMIEx/Nsfmmq6Chqe9o8+GhEJYjAwspUhIkdvdEuVWzn0iVWr93BCjanHGtTj4U++acC3rVeGfW7DrJEUB00wMg9PiBnVCd6tSnybGFEhQX8C/QpW5vqlkx0uST9BJ6g/QdAgIp3fmCt3ko2a2M= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1728467659; c=relaxed/simple; bh=iju+XNz+de8YRVlWlj9iBGFLV2MGE/LhlLoRYCrP7hg=; h=From:To:Cc:Subject:Date:Message-Id:In-Reply-To:References: MIME-Version:Content-Type; b=Qggn2Bu+mYaVx13wddbyeAyjkqvKeXysoPUBYYlyVk3m1nHDboZ4fZcrWAuXD7px9hJ/Q677T09MTsaj396/y0DCyRIomWdhOjLu6kYXo+97LCYaL7ffUgkcPrI3lu91dtKWSQ9osU2WUFLAqq2xc8fw0yA+tPWUIFCBHPZYmrI= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.intel.com; spf=none smtp.mailfrom=linux.intel.com; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b=bTH8dRPV; arc=none smtp.client-ip=198.175.65.9 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.intel.com Authentication-Results: smtp.subspace.kernel.org; spf=none smtp.mailfrom=linux.intel.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b="bTH8dRPV" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1728467658; x=1760003658; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=iju+XNz+de8YRVlWlj9iBGFLV2MGE/LhlLoRYCrP7hg=; b=bTH8dRPVmu88TQfRRVdTbeAVYJP7ExtmMPq43zGr0c0fXVszt7rFRyNV y/kA8bWYjLAerHuumCl0Pnlb+v9vRPTMA3zxO/Lg94TYLy0ESpWaE1hbM LyMar6eBN1PbE2YIdIbvProWmCeriwkJdW9De4OCZDzQnqJAX9gRq1QQm hlJ4e3vO8jwKKDXHb98Or1zfF9uCHQn2cEtMvd5PerdZaqNEY24od5aj2 gl4a+qidxd6QjGHVYjoFXzDtgg6ra7tzXeRlCe6CRpONubOEs9waVC+9X OD68n5x0JeEMW/NKhw3dP27NZUme9Nrv41uQXOEktOswVb14VTKAWK7V2 w==; X-CSE-ConnectionGUID: zk9/nK2PRmijAiRSow0JDw== X-CSE-MsgGUID: musuy9XnTs+jKZ0pM9BoLA== X-IronPort-AV: E=McAfee;i="6700,10204,11219"; a="50291984" X-IronPort-AV: E=Sophos;i="6.11,189,1725346800"; d="scan'208";a="50291984" Received: from orviesa007.jf.intel.com ([10.64.159.147]) by orvoesa101.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 09 Oct 2024 02:54:08 -0700 X-CSE-ConnectionGUID: 4bf39EobRmidXWlqfHfCmg== X-CSE-MsgGUID: Xbybz74KS8mCDeRHJbuUCQ== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.11,189,1725346800"; d="scan'208";a="76640672" Received: from ijarvine-mobl1.ger.corp.intel.com (HELO localhost) ([10.245.244.41]) by orviesa007-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 09 Oct 2024 02:54:01 -0700 From: =?UTF-8?q?Ilpo=20J=C3=A4rvinen?= To: linux-pci@vger.kernel.org, Bjorn Helgaas , Lorenzo Pieralisi , Rob Herring , =?UTF-8?q?Krzysztof=20Wilczy=C5=84ski?= , "Maciej W. Rozycki" , Jonathan Cameron , Lukas Wunner , Alexandru Gagniuc , Krishna chaitanya chundru , Srinivas Pandruvada , "Rafael J. Wysocki" , linux-pm@vger.kernel.org, Smita Koralahalli , Shuah Khan , =?UTF-8?q?Ilpo=20J=C3=A4rvinen?= , linux-kernel@vger.kernel.org, linux-kselftest@vger.kernel.org Cc: Daniel Lezcano , Amit Kucheria , Zhang Rui , Christophe JAILLET Subject: [PATCH v8 8/8] selftests/pcie_bwctrl: Create selftests Date: Wed, 9 Oct 2024 12:52:23 +0300 Message-Id: <20241009095223.7093-9-ilpo.jarvinen@linux.intel.com> X-Mailer: git-send-email 2.39.5 In-Reply-To: <20241009095223.7093-1-ilpo.jarvinen@linux.intel.com> References: <20241009095223.7093-1-ilpo.jarvinen@linux.intel.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Create selftests for PCIe BW control through the PCIe cooling device sysfs interface. First, the BW control selftest finds the PCIe Port to test with. By default, the PCIe Port with the highest Link Speed is selected but another PCIe Port can be provided with -d parameter. The actual test steps the cur_state of the cooling device one-by-one from max_state to what the cur_state was initially. The speed change is confirmed by observing the current_link_speed for the corresponding PCIe Port. Signed-off-by: Ilpo J=C3=A4rvinen Reviewed-by: Jonathan Cameron --- MAINTAINERS | 1 + tools/testing/selftests/Makefile | 1 + tools/testing/selftests/pcie_bwctrl/Makefile | 2 + .../pcie_bwctrl/set_pcie_cooling_state.sh | 122 ++++++++++++++++++ .../selftests/pcie_bwctrl/set_pcie_speed.sh | 67 ++++++++++ 5 files changed, 193 insertions(+) create mode 100644 tools/testing/selftests/pcie_bwctrl/Makefile create mode 100755 tools/testing/selftests/pcie_bwctrl/set_pcie_cooling_st= ate.sh create mode 100755 tools/testing/selftests/pcie_bwctrl/set_pcie_speed.sh diff --git a/MAINTAINERS b/MAINTAINERS index 393ed7ce5ea1..d7ffef4382df 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -17940,6 +17940,7 @@ S: Supported F: drivers/pci/pcie/bwctrl.c F: drivers/thermal/pcie_cooling.c F: include/linux/pci-bwctrl.h +F: tools/testing/selftests/pcie_bwctrl/ =20 PCIE DRIVER FOR AMAZON ANNAPURNA LABS M: Jonathan Chocron diff --git a/tools/testing/selftests/Makefile b/tools/testing/selftests/Mak= efile index b38199965f99..7181756f47ff 100644 --- a/tools/testing/selftests/Makefile +++ b/tools/testing/selftests/Makefile @@ -72,6 +72,7 @@ TARGETS +=3D net/packetdrill TARGETS +=3D net/rds TARGETS +=3D net/tcp_ao TARGETS +=3D nsfs +TARGETS +=3D pcie_bwctrl TARGETS +=3D perf_events TARGETS +=3D pidfd TARGETS +=3D pid_namespace diff --git a/tools/testing/selftests/pcie_bwctrl/Makefile b/tools/testing/s= elftests/pcie_bwctrl/Makefile new file mode 100644 index 000000000000..3e84e26341d1 --- /dev/null +++ b/tools/testing/selftests/pcie_bwctrl/Makefile @@ -0,0 +1,2 @@ +TEST_PROGS =3D set_pcie_cooling_state.sh +include ../lib.mk diff --git a/tools/testing/selftests/pcie_bwctrl/set_pcie_cooling_state.sh = b/tools/testing/selftests/pcie_bwctrl/set_pcie_cooling_state.sh new file mode 100755 index 000000000000..9df606552af3 --- /dev/null +++ b/tools/testing/selftests/pcie_bwctrl/set_pcie_cooling_state.sh @@ -0,0 +1,122 @@ +#!/bin/bash +# SPDX-License-Identifier: GPL-2.0-or-later + +SYSFS=3D +# Kselftest framework requirement - SKIP code is 4. +ksft_skip=3D4 +retval=3D0 +skipmsg=3D"skip all tests:" + +PCIEPORTTYPE=3D"PCIe_Port_Link_Speed" + +prerequisite() +{ + local ports + + if [ $UID !=3D 0 ]; then + echo $skipmsg must be run as root >&2 + exit $ksft_skip + fi + + SYSFS=3D`mount -t sysfs | head -1 | awk '{ print $3 }'` + + if [ ! -d "$SYSFS" ]; then + echo $skipmsg sysfs is not mounted >&2 + exit $ksft_skip + fi + + if ! ls $SYSFS/class/thermal/cooling_device* > /dev/null 2>&1; then + echo $skipmsg thermal cooling devices missing >&2 + exit $ksft_skip + fi + + ports=3D`grep -e "^$PCIEPORTTYPE" $SYSFS/class/thermal/cooling_device*/ty= pe | wc -l` + if [ $ports -eq 0 ]; then + echo $skipmsg pcie cooling devices missing >&2 + exit $ksft_skip + fi +} + +testport=3D +find_pcie_port() +{ + local patt=3D"$1" + local pcieports + local max + local cur + local delta + local bestdelta=3D-1 + + pcieports=3D`grep -l -F -e "$patt" /sys/class/thermal/cooling_device*/typ= e` + if [ -z "$pcieports" ]; then + return + fi + pcieports=3D${pcieports//\/type/} + # Find the port with the highest PCIe Link Speed + for port in $pcieports; do + max=3D`cat $port/max_state` + cur=3D`cat $port/cur_state` + delta=3D$((max-cur)) + if [ $delta -gt $bestdelta ]; then + testport=3D"$port" + bestdelta=3D$delta + fi + done +} + +sysfspcidev=3D +find_sysfs_pci_dev() +{ + local typefile=3D"$1/type" + local pcidir + + pcidir=3D"$SYSFS/bus/pci/devices/`sed -e "s|^${PCIEPORTTYPE}_||g" $typefi= le`" + + if [ -r "$pcidir/current_link_speed" ]; then + sysfspcidev=3D"$pcidir/current_link_speed" + fi +} + +usage() +{ + echo "Usage $0 [ -d dev ]" + echo -e "\t-d: PCIe port BDF string (e.g., 0000:00:04.0)" +} + +pattern=3D"$PCIEPORTTYPE" +parse_arguments() +{ + while getopts d:h opt; do + case $opt in + h) + usage "$0" + exit 0 + ;; + d) + pattern=3D"$PCIEPORTTYPE_$OPTARG" + ;; + *) + usage "$0" + exit 0 + ;; + esac + done +} + +parse_arguments "$@" +prerequisite +find_pcie_port "$pattern" +if [ -z "$testport" ]; then + echo $skipmsg "pcie cooling device not found from sysfs" >&2 + exit $ksft_skip +fi +find_sysfs_pci_dev "$testport" +if [ -z "$sysfspcidev" ]; then + echo $skipmsg "PCIe port device not found from sysfs" >&2 + exit $ksft_skip +fi + +./set_pcie_speed.sh "$testport" "$sysfspcidev" +retval=3D$? + +exit $retval diff --git a/tools/testing/selftests/pcie_bwctrl/set_pcie_speed.sh b/tools/= testing/selftests/pcie_bwctrl/set_pcie_speed.sh new file mode 100755 index 000000000000..584596949312 --- /dev/null +++ b/tools/testing/selftests/pcie_bwctrl/set_pcie_speed.sh @@ -0,0 +1,67 @@ +#!/bin/bash +# SPDX-License-Identifier: GPL-2.0-or-later + +set -e + +TESTNAME=3Dset_pcie_speed + +declare -a PCIELINKSPEED=3D( + "2.5 GT/s PCIe" + "5.0 GT/s PCIe" + "8.0 GT/s PCIe" + "16.0 GT/s PCIe" + "32.0 GT/s PCIe" + "64.0 GT/s PCIe" +) + +# Kselftest framework requirement - SKIP code is 4. +ksft_skip=3D4 +retval=3D0 + +coolingdev=3D"$1" +statefile=3D"$coolingdev/cur_state" +maxfile=3D"$coolingdev/max_state" +linkspeedfile=3D"$2" + +oldstate=3D`cat $statefile` +maxstate=3D`cat $maxfile` + +set_state() +{ + local state=3D$1 + local linkspeed + local expected_linkspeed + + echo $state > $statefile + + sleep 1 + + linkspeed=3D"`cat $linkspeedfile`" + expected_linkspeed=3D$((maxstate-state)) + expected_str=3D"${PCIELINKSPEED[$expected_linkspeed]}" + if [ ! "${expected_str}" =3D "${linkspeed}" ]; then + echo "$TESTNAME failed: expected: ${expected_str}; got ${linkspeed}" + retval=3D1 + fi +} + +cleanup_skip () +{ + set_state $oldstate + exit $ksft_skip +} + +trap cleanup_skip EXIT + +echo "$TESTNAME: testing states $maxstate .. $oldstate with $coolingdev" +for i in $(seq $maxstate -1 $oldstate); do + set_state "$i" +done + +trap EXIT +if [ $retval -eq 0 ]; then + echo "$TESTNAME [PASS]" +else + echo "$TESTNAME [FAIL]" +fi +exit $retval --=20 2.39.5