From: Reinette Chatre <reinette.chatre@intel.com>
To: Shaopeng Tan <tan.shaopeng@jp.fujitsu.com>,
Fenghua Yu <fenghua.yu@intel.com>, Shuah Khan <shuah@kernel.org>
Cc: <linux-kernel@vger.kernel.org>, <linux-kselftest@vger.kernel.org>
Subject: Re: [PATCH v6 0/6] selftests/resctrl: Add resctrl_tests into kselftest set
Date: Fri, 18 Mar 2022 14:00:49 -0700 [thread overview]
Message-ID: <a7b28139-c097-385d-be22-c61f82409626@intel.com> (raw)
In-Reply-To: <20220318075807.2921063-1-tan.shaopeng@jp.fujitsu.com>
Hi Shaopeng Tan,
On 3/18/2022 12:58 AM, Shaopeng Tan wrote:
> Hello,
>
> The aim of this series is to make resctrl_tests run by using
> kselftest framework.
> - I modify resctrl_test Makefile and kselftest Makefile,
> to enable build/run resctrl_tests by using kselftest framework.
> Of course, users can also build/run resctrl_tests without
> using framework as before.
> - I change the default limited time for resctrl_tests to 120 seconds, to
> ensure the resctrl_tests finish in limited time on different environments.
> - When resctrl file system is not supported by environment or
> resctrl_tests is not run as root, return skip code of kselftest framework.
> - If resctrl_tests does not finish in limited time, terminate it as
> same as executing ctrl+c that kills parent process and child process.
>
> Difference from v5:
> - Changed skip message when resctrlfs is not supported. [PATCH v6 3/6]
> - Changed resctrl's Makefile to always build with latest kernel headers and
> keep enabling gcc checks to detect buffer overflows. [PATCH v6 4/6]
> - Made README easier to understand. [PATCH v6 5/6]
> https://lore.kernel.org/lkml/20220311072147.3301525-1-tan.shaopeng@jp.fujitsu.com/ [PATCH V5]
>
> This patch series is based on v5.17-rc8.
>
I do not think this is accurate because the KHDR_INCLUDES you use in patch 4/6
does not yet exist in v5.17.rc8. It seems to be available on the "next" branch
of git://git.kernel.org/pub/scm/linux/kernel/git/shuah/linux-kselftest.git
Reinette
prev parent reply other threads:[~2022-03-18 21:00 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-03-18 7:58 [PATCH v6 0/6] selftests/resctrl: Add resctrl_tests into kselftest set Shaopeng Tan
2022-03-18 7:58 ` [PATCH v6 1/6] selftests/resctrl: Kill child process before parent process terminates if SIGTERM is received Shaopeng Tan
2022-03-18 22:47 ` Fenghua Yu
2022-03-18 7:58 ` [PATCH v6 2/6] selftests/resctrl: Change the default limited time to 120 seconds Shaopeng Tan
2022-03-18 22:48 ` Fenghua Yu
2022-03-18 7:58 ` [PATCH v6 3/6] selftests/resctrl: Fix resctrl_tests' return code to work with selftest framework Shaopeng Tan
2022-03-18 20:58 ` Reinette Chatre
2022-03-18 22:51 ` Fenghua Yu
2022-03-18 7:58 ` [PATCH v6 4/6] selftests/resctrl: Make resctrl_tests run using kselftest framework Shaopeng Tan
2022-03-18 20:59 ` Reinette Chatre
2022-03-18 22:55 ` Fenghua Yu
2022-03-18 7:58 ` [PATCH v6 5/6] selftests/resctrl: Update README about using kselftest framework to build/run resctrl_tests Shaopeng Tan
2022-03-18 20:59 ` Reinette Chatre
2022-03-18 22:57 ` Fenghua Yu
2022-03-18 7:58 ` [PATCH v6 6/6] selftests/resctrl: Add missing SPDX license to Makefile Shaopeng Tan
2022-03-18 21:00 ` Reinette Chatre
2022-03-18 22:58 ` Fenghua Yu
2022-03-18 21:00 ` Reinette Chatre [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=a7b28139-c097-385d-be22-c61f82409626@intel.com \
--to=reinette.chatre@intel.com \
--cc=fenghua.yu@intel.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-kselftest@vger.kernel.org \
--cc=shuah@kernel.org \
--cc=tan.shaopeng@jp.fujitsu.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox