Re: A patch series improving data quality of perf test for CoreSight

From: Suzuki K Poulose
Date: Wed Jul 13 2022 - 04:17:58 EST

Hi Carsten

On 12/07/2022 14:57, carsten.haitzler@xxxxxxxxxxxx wrote:
This is a prelude to adding more tests to shell tests and in order to
support putting those tests into subdirectories, I need to change the
test code that scans/finds and runs them.

Please could you add versioning to your series and a changelog of what
changed from one version to the other ? That way, it is easier for the
reviewers to understand and spend their time effectively on the newer

If you are using git format-patch, you could use -v <N> for the version
number in all your patches.


nit on Subject: This could be:

perf: test: Add trace data quality tests for CoreSight

where "perf" is the linux subsystem you are targeting and
"test" is the specific area you are contributing. That helps
the reviewers to filter the series in a mailing list with
hundreds of emails.

To support subdirs I have to recurse so it's time to refactor the code to
allow this and centralize the shell script finding into one location and
only one single scan that builds a list of all the found tests in memory
instead of it being duplicated in 3 places.

This code also optimizes things like knowing the max width of desciption
strings (as we can do that while we scan instead of a whole new pass
of opening files). It also more cleanly filters scripts to see only
*.sh files thus skipping random other files in directories like *~
backup files, other random junk/data files that may appear and the
scripts must be executable to make the cut (this ensures the script
lib dir is not seen as scripts to run). This avoids perf test running
previous older versions of test scripts that are editor backup files
as well as skipping files that may appear and so on.

Signed-off-by: Carsten Haitzler <carsten.haitzler@xxxxxxx>

It is also a good idea to have the diffstat of the series in the cover letter to give the people an idea of where the changes are touching.

git format-patch automatically gives you this if you add --cover-letter