Re: [PATCH v7 4/6] selftests/resctrl: Cleanup properly when an error occurs in CAT test

From: Reinette Chatre
Date: Mon Feb 13 2023 - 14:20:20 EST




On 2/13/2023 2:00 AM, Ilpo Järvinen wrote:
> On Mon, 13 Feb 2023, Shaopeng Tan wrote:
>
>> After creating a child process with fork() in CAT test, if an error
>> occurs when parent process runs cat_val() or check_results(), the child
>
> I'd replace the rest of the paragraph with this:
>
> "returns early. The parent will wait pipe message from child which will
> never be sent by the child and the parent cannot proceeed to unmount
> resctrlfs."

Note that the description is about an error within the parent process.
In the case snipped above it is the parent that exits early, not the child.
This first paragraph describes two scenarios, (a) error in parent, and (b)
error in child. I think it is good information to keep descriptions of
both scenarios.

The proposed addition could be used to expand the description of the scenario
when an error occurs in the child. In this case please consider changing "wait
pipe message from child" to "wait for the pipe message from the child", and
"proceeed" to "proceed".

Reinette