From nobody Sun Feb 8 23:35:17 2026 Received: from out30-124.freemail.mail.aliyun.com (out30-124.freemail.mail.aliyun.com [115.124.30.124]) (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 461371BC31; Mon, 5 Feb 2024 12:29:33 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=115.124.30.124 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1707136177; cv=none; b=ROZoaiWk0iVoAco5wclFMlaT8SfdI9PhfDoNl6xdw1go1WcnXZXmYqfx3HimkokEMRVfujsaK+HONW871Js+cZxeGpMpq3jQi07+zXmTxfAxKfClIZA1rbHkXgeYfxaYLXms2TPeZHn9mGCsRxedkatT5r0q8YVag9+LxgdxYWU= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1707136177; c=relaxed/simple; bh=70uIplFj6tYmIaM2+/hdbOkFRGVdFZN4V4RRhP1FyCA=; h=From:To:Cc:Subject:Date:Message-Id:In-Reply-To:References: MIME-Version; b=DkdK2KQ5k4L5gYNpWJFOtGRlMo3M3fWp4WrjE3c5KSxRGRB/lpObZM5haUlVXkjHyOhksR0aTWgK7frH3kY5abHNhFvifhkxQSjxP1aiM3pYle6lXbPuFdbhx+FjS14bl6HMEBgQXgzOMIdCHL1JektWQmw5AP+y4xE4j0AbQAE= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.alibaba.com; spf=pass smtp.mailfrom=linux.alibaba.com; dkim=pass (1024-bit key) header.d=linux.alibaba.com header.i=@linux.alibaba.com header.b=PB5BbncE; arc=none smtp.client-ip=115.124.30.124 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.alibaba.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linux.alibaba.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linux.alibaba.com header.i=@linux.alibaba.com header.b="PB5BbncE" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.alibaba.com; s=default; t=1707136172; h=From:To:Subject:Date:Message-Id:MIME-Version; bh=z5XFUY+97aQgYz9qIycNQwP7iD9Nqzay3ltOgamXdOE=; b=PB5BbncEG9zwsbXP5jIu4aLpJRd3nQ2kX9l6d0HoYS2f5k6glOrQjMzKFaFskb7vCaCou+mGfOk+TzjOg/lAjtFc7x4RyULwUSMMs+gTWIEOs5p7ppysi2jsPQUZSeWAtVrISzKAl0jXIhkLFa4A1sX+scarJ5ENOT/chYd1fI8= X-Alimail-AntiSpam: AC=PASS;BC=-1|-1;BR=01201311R911e4;CH=green;DM=||false|;DS=||;FP=0|-1|-1|-1|0|-1|-1|-1;HT=ay29a033018045192;MF=xiangzao@linux.alibaba.com;NM=1;PH=DS;RN=11;SR=0;TI=SMTPD_---0W0AVhQg_1707136170; Received: from localhost.localdomain(mailfrom:xiangzao@linux.alibaba.com fp:SMTPD_---0W0AVhQg_1707136170) by smtp.aliyun-inc.com; Mon, 05 Feb 2024 20:29:31 +0800 From: Yuanhe Shu To: keescook@chromium.org, tony.luck@intel.com, gpiccoli@igalia.com, shuah@kernel.org, corbet@lwn.net Cc: xlpang@linux.alibaba.com, linux-kernel@vger.kernel.org, linux-doc@vger.kernel.org, linux-hardening@vger.kernel.org, linux-kselftest@vger.kernel.org, Yuanhe Shu Subject: [PATCH 3/3] tools/testing: adjust pstore backend related selftest Date: Mon, 5 Feb 2024 20:28:52 +0800 Message-Id: <20240205122852.7069-4-xiangzao@linux.alibaba.com> X-Mailer: git-send-email 2.39.3 In-Reply-To: <20240205122852.7069-1-xiangzao@linux.alibaba.com> References: <20240205122852.7069-1-xiangzao@linux.alibaba.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Pstore now supports multiple backends, the module parameter pstore.backend varies from 'registered backend' to 'backends that are allowed to register'. Adjust selftests to match the change. Signed-off-by: Yuanhe Shu --- tools/testing/selftests/pstore/common_tests | 8 +-- .../selftests/pstore/pstore_post_reboot_tests | 65 ++++++++++--------- tools/testing/selftests/pstore/pstore_tests | 2 +- 3 files changed, 38 insertions(+), 37 deletions(-) diff --git a/tools/testing/selftests/pstore/common_tests b/tools/testing/se= lftests/pstore/common_tests index 4509f0cc9c91..497e6fc3215f 100755 --- a/tools/testing/selftests/pstore/common_tests +++ b/tools/testing/selftests/pstore/common_tests @@ -27,9 +27,9 @@ show_result() { # result_value } =20 check_files_exist() { # type of pstorefs file - if [ -e ${1}-${backend}-0 ]; then + if [ -e ${1}-${2}-0 ]; then prlog "ok" - for f in `ls ${1}-${backend}-*`; do + for f in `ls ${1}-${2}-*`; do prlog -e "\t${f}" done else @@ -74,9 +74,9 @@ prlog "=3D=3D=3D Pstore unit tests (`basename $0`) =3D=3D= =3D" prlog "UUID=3D"$UUID =20 prlog -n "Checking pstore backend is registered ... " -backend=3D`cat /sys/module/pstore/parameters/backend` +backends=3D$(dmesg | sed -n 's/.*pstore: Registered \(.*\) as persistent s= tore backend.*/\1/p') show_result $? -prlog -e "\tbackend=3D${backend}" +prlog -e "\tbackends=3D"$backends prlog -e "\tcmdline=3D`cat /proc/cmdline`" if [ $rc -ne 0 ]; then exit 1 diff --git a/tools/testing/selftests/pstore/pstore_post_reboot_tests b/tool= s/testing/selftests/pstore/pstore_post_reboot_tests index d6da5e86efbf..9e40ccb9c918 100755 --- a/tools/testing/selftests/pstore/pstore_post_reboot_tests +++ b/tools/testing/selftests/pstore/pstore_post_reboot_tests @@ -36,45 +36,46 @@ else fi =20 cd ${mount_point} +for backend in ${backends}; do + prlog -n "Checking ${backend}-dmesg files exist in pstore filesystem .= .. " + check_files_exist dmesg ${backend} =20 -prlog -n "Checking dmesg files exist in pstore filesystem ... " -check_files_exist dmesg + prlog -n "Checking ${backend}-console files exist in pstore filesystem= ... " + check_files_exist console ${backend} =20 -prlog -n "Checking console files exist in pstore filesystem ... " -check_files_exist console + prlog -n "Checking ${backend}-pmsg files exist in pstore filesystem ..= . " + check_files_exist pmsg ${backend} =20 -prlog -n "Checking pmsg files exist in pstore filesystem ... " -check_files_exist pmsg + prlog -n "Checking ${backend}-dmesg files contain oops end marker" + grep_end_trace() { + grep -q "\---\[ end trace" $1 + } + files=3D`ls dmesg-${backend}-*` + operate_files $? "$files" grep_end_trace =20 -prlog -n "Checking dmesg files contain oops end marker" -grep_end_trace() { - grep -q "\---\[ end trace" $1 -} -files=3D`ls dmesg-${backend}-*` -operate_files $? "$files" grep_end_trace + prlog -n "Checking ${backend}-console file contains oops end marker ..= . " + grep -q "\---\[ end trace" console-${backend}-0 + show_result $? =20 -prlog -n "Checking console file contains oops end marker ... " -grep -q "\---\[ end trace" console-${backend}-0 -show_result $? - -prlog -n "Checking pmsg file properly keeps the content written before cra= sh ... " -prev_uuid=3D`cat $TOP_DIR/prev_uuid` -if [ $? -eq 0 ]; then - nr_matched=3D`grep -c "$TEST_STRING_PATTERN" pmsg-${backend}-0` - if [ $nr_matched -eq 1 ]; then - grep -q "$TEST_STRING_PATTERN"$prev_uuid pmsg-${backend}-0 - show_result $? + prlog -n "Checking ${backend}-pmsg file properly keeps the content wri= tten before crash ... " + prev_uuid=3D`cat $TOP_DIR/prev_uuid` + if [ $? -eq 0 ]; then + nr_matched=3D`grep -c "$TEST_STRING_PATTERN" pmsg-${backend}-0` + if [ $nr_matched -eq 1 ]; then + grep -q "$TEST_STRING_PATTERN"$prev_uuid pmsg-${backend}-0 + show_result $? + else + prlog "FAIL" + rc=3D1 + fi else - prlog "FAIL" - rc=3D1 + prlog "FAIL" + rc=3D1 fi -else - prlog "FAIL" - rc=3D1 -fi =20 -prlog -n "Removing all files in pstore filesystem " -files=3D`ls *-${backend}-*` -operate_files $? "$files" rm + prlog -n "Removing all ${backend} files in pstore filesystem " + files=3D`ls *-${backend}-*` + operate_files $? "$files" rm +done =20 exit $rc diff --git a/tools/testing/selftests/pstore/pstore_tests b/tools/testing/se= lftests/pstore/pstore_tests index 2aa9a3852a84..f4665a8c77dc 100755 --- a/tools/testing/selftests/pstore/pstore_tests +++ b/tools/testing/selftests/pstore/pstore_tests @@ -10,7 +10,7 @@ . ./common_tests =20 prlog -n "Checking pstore console is registered ... " -dmesg | grep -Eq "console \[(pstore|${backend})" +dmesg | grep -Eq "console \[(pstore console)" show_result $? =20 prlog -n "Checking /dev/pmsg0 exists ... " --=20 2.39.3