From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pg1-f175.google.com (mail-pg1-f175.google.com [209.85.215.175]) (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 02C5543FD3B for ; Mon, 24 Aug 2026 15:33:18 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.215.175 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1787585600; cv=none; b=uP665RlkYl73XlaKqNGDXH979zuNAjrSF8fWqbPnB5waB9FkGMO0E0lwZmFBSsDmJpHTe7SM/MgYodG9mgqDF/nXntbJ5xO8SuIrCpwTIaV5Xs1yjAvisHiFLLqpeX/r8tPmDTdiWiWRU3NsR7x5QXmn1+LUpP4IyOIN5MsbCCE= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1787585600; c=relaxed/simple; bh=ohjRofdJUcuO4meBoHs4IxeqoaDQ04T5xwdpgwaXgbo=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=hit22ag0PtOBm+ViJTijb1RKWXN0waQGl9otIc3N8JknBRjvy0hu1nOrGxqSxz8gfS6e+7rOfXPHIrs3b+7q6z0mFmBR7mJlj8ZbApBIbmUJROZAUe8pRtVcD+38TE3nw23Ct0/6ePMj5B3dcRyeVudZ+7VOjbZSM1QqQs65FWM= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=ENwteRC9; arc=none smtp.client-ip=209.85.215.175 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="ENwteRC9" Received: by mail-pg1-f175.google.com with SMTP id 41be03b00d2f7-cbe6295f05bso3176949a12.1 for ; Mon, 24 Aug 2026 08:33:18 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1787585598; x=1788190398; darn=vger.kernel.org; h=in-reply-to:content-disposition:content-type:mime-version :references:message-id:subject:cc:to:from:date:from:to:cc:subject :date:message-id:reply-to:content-type; bh=RAedVK5h+CJivT4Yy7dGsRjex7VLZDzbGqNcf02p9qI=; b=ENwteRC9z/m/QTcriDEGChiVIYa8k49pVAs0mICAF4OkxcH5C3oWraUb12jxeImMiZ ardCDN7xeT5s0dn/bV5QVvqLtfSIYNkvvazRLpG3xFa95AYhK+DwFt0mMkfY720TT/q7 s5E5fvLZX5a6VK1DSsbo9Yp44yLm3cfxVURMBA4mWS3hYZ7A+GXt5pME/WJgnkuuAELk D3BWFN98Fepe8B3YS5zIb4RtZtfj3jbHluxpzmyXEjP8HIyNWCeEedxekuqnKu78jFDC BS7MjAHWBwOh0VxO1JstCqT9qpn+m44lRzmVRaCWA/Few731gRQqposs2dljGxDGB9+u I4Jg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1787585598; x=1788190398; h=in-reply-to:content-disposition:content-type:mime-version :references:message-id:subject:cc:to:from:date:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to :content-type; bh=RAedVK5h+CJivT4Yy7dGsRjex7VLZDzbGqNcf02p9qI=; b=X4QN7f7mXbHc+KXLFrLqrs1XGIkMupQnw5nIHjWDAJmwCgwtTPC8BuiAJGIfSVcGWC dQLQEvtHCeD/yPBmsh0ATq1Z+tCfG5PSlrHD48DHBKw5oKtq4RA5X40zb70sYusaDcUI fbJFSiRe8hO6bfFcB1PAYm/J0wlYaERb7nnyuGRUC3mzHKcr2qkJOWaWFtK/CYc+k72J evLGIGw2BlNCDpwzVvvZwx1NPkrNaVSSa/fNUBTgUqUgFqyA30ljmzUwBjHHWyPA7+En XFVsRil1lVN1fqPd64Ihh5R3e1H1Amf9SLAfd/udP79BaWczwltvW+BP4nuqQZn+8XPQ vEBQ== X-Gm-Message-State: AFuF++kfvBsbAdBmX1Gl9gPVcUSBmE5DT5DPon9x/prqT0Iv7s4wVzFu pdHGYvXaeXA4E+7/K0fj/oFyQ27E8ey9SSHV9xUwMj5HIklWcG/+rT0KNzuVF9V0Ycc= X-Gm-Gg: AR+sD12bl3CUkOD0ZRy5kRp2REio7LeaphNAVbIzbIg+m+FJ02ADcrqPOy5vnHJgGt/ ITJDPIfZKmJW4D8K/gyRR8Mk4IrGEHj/jpFcISuRE9NuDxIUxsVbM2/lFSTlXYgVGEts0fjQgim LxmSOw1UgzHNM6Fh219fDvx6hbuoYtkFDfH4sieq48bYeIS8OBQcn3Ea9GYxbnNaNBGCM7Sdakj /gkDRLdK7M+JMEXrPdT6j8Fa6+H5ihceV7AhNbbNnTEuZmpmRlkxs+mOq7P/J6reY72GK2CFBA0 F6AE7ZW+ibfppzvLLtIB3WCaWjgoFaJMAVSEKk3JUSPfB4DqmF9O36xBuOTYSA/2tHB/FFQnXkR zjkHggFXuZtkqVoZCSQZcndYVQD7a5D+1zo273kEEtgeWfJJNZ1rVmFzV1BgvX3eHuPvQ54IQjr sfcDuDifk9z1ueJf0HiwVzm0lwRvna+sT9TuGZe69vMLHK0bx22obIqdcgQWda X-Received: by 2002:a17:90b:4b42:b0:390:84db:888e with SMTP id 98e67ed59e1d1-39645b1c818mr2587a91.7.1787585597901; Mon, 24 Aug 2026 08:33:17 -0700 (PDT) Received: from google.com ([118.150.148.19]) by smtp.gmail.com with ESMTPSA id 98e67ed59e1d1-395e4a15829sm10440341a91.14.2026.08.24.08.33.15 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 24 Aug 2026 08:33:17 -0700 (PDT) Date: Mon, 24 Aug 2026 23:33:13 +0800 From: Kuan-Wei Chiu To: Gabriele Monaco Cc: linux-kernel@vger.kernel.org, Steven Rostedt , Nam Cao , linux-trace-kernel@vger.kernel.org Subject: Re: [PATCH] verification/rvgen: Use .old instead of .bak for kunit backup files Message-ID: References: <20260824081519.81103-2-gmonaco@redhat.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=us-ascii Content-Disposition: inline In-Reply-To: <20260824081519.81103-2-gmonaco@redhat.com> On Mon, Aug 24, 2026 at 10:15:19AM +0200, Gabriele Monaco wrote: > The rvgen kunit command generates .bak backup files and these can be > checked in for selftests (make check). Clean targets like make disclean > remove such files, leaving the tree dirty. > > Switch to .old to preserve a clean tree after make disclean. > > Reported-by: Kuan-Wei Chiu > Closes: https://lore.kernel.org/lkml/aosuwKH5GOEo0xTN@google.com > Fixes: 7b6246294eb0 ("verification/rvgen: Add selftests for rvgen kunit") > Signed-off-by: Gabriele Monaco Thanks for the quick fix! The working tree now remains clean after running make distclean. Tested-by: Kuan-Wei Chiu Regards, Kuan-Wei > --- > tools/verification/rvgen/rvgen/kunit.py | 2 +- > .../{test_bak_kunit_kunit.c.bak => test_bak_kunit_kunit.c.old} | 0 > 2 files changed, 1 insertion(+), 1 deletion(-) > rename tools/verification/rvgen/tests/golden/test_bak_kunit/{test_bak_kunit_kunit.c.bak => test_bak_kunit_kunit.c.old} (100%) > > diff --git a/tools/verification/rvgen/rvgen/kunit.py b/tools/verification/rvgen/rvgen/kunit.py > index ed2082d7d3bc..85973f918c9b 100644 > --- a/tools/verification/rvgen/rvgen/kunit.py > +++ b/tools/verification/rvgen/rvgen/kunit.py > @@ -173,7 +173,7 @@ EXPORT_SYMBOL_IF_KUNIT({struct_name}); > for path in (header_file_path, kunit_c_file_path): > if path.exists(): > try: > - path.rename(path.with_suffix(path.suffix + ".bak")) > + path.rename(path.with_suffix(path.suffix + ".old")) > except OSError as e: > raise KUnitError(f"Error backing up file {path}: {e}") from e > > diff --git a/tools/verification/rvgen/tests/golden/test_bak_kunit/test_bak_kunit_kunit.c.bak b/tools/verification/rvgen/tests/golden/test_bak_kunit/test_bak_kunit_kunit.c.old > similarity index 100% > rename from tools/verification/rvgen/tests/golden/test_bak_kunit/test_bak_kunit_kunit.c.bak > rename to tools/verification/rvgen/tests/golden/test_bak_kunit/test_bak_kunit_kunit.c.old > > base-commit: 0a0d1d55dad570724bf8c7ea83409639cfb4be9b > -- > 2.55.0 >