Re: resctrl selftest patches

[Reinette Chatre <reinette.chatre@intel.com>]

Hi Shuah,

On 4/10/2023 11:39 AM, Reinette Chatre wrote:
> On 4/10/2023 11:25 AM, Shuah Khan wrote:
>> On 4/10/23 10:43, Reinette Chatre wrote:
>>> Hi Shuah and kselftest team,
>>>
>>> There are a couple of resctrl selftest patches that are ready for
>>> inclusion. They have been percolating on the list for a while
>>> without expecting more feedback. All have "Reviewed-by" tags from
>>> at least one reviewer. Could you please consider including them
>>> into the kselftest repo? There is one minor merge conflict between
>>> two of the series for which the snippet below shows resolution.
>>> [PATCH v8 0/6] Some improvements of resctrl selftest
>>> https://lore.kernel.org/lkml/20230215083230.3155897-1-tan.shaopeng@jp.fujitsu.com/
>>>
>>> [PATCH v2 0/9] selftests/resctrl: Fixes to error handling logic and cleanups
>>> https://lore.kernel.org/lkml/20230215130605.31583-1-ilpo.jarvinen@linux.intel.com/
>>>
>>
>> I was waiting for another ack from x86, don't see one.
> 
> I see. What are your expectations for resctrl patches to be considered
> ready for inclusion?
> 
>> I just applied them all.
>>
>> v9 patches are now in linux-kselftest next for Linux 6.4
>>
> 
> Thank you very much for picking up Ilpo's series.
> 
> Would you be ok to also pick up Shaopeng and Peter's contributions?
> 
> These are the following (nothing new, just copied from previous email):
> 
> [PATCH v8 0/6] Some improvements of resctrl selftest
> https://lore.kernel.org/lkml/20230215083230.3155897-1-tan.shaopeng@jp.fujitsu.com/

Shaopeng just sent a v9 with the only change being that it is rebased
on top of kselftest's next branch. This leaves you with no merge conflicts to
deal with:
https://lore.kernel.org/lkml/20230413072259.2089348-1-tan.shaopeng@jp.fujitsu.com/

Discussion surrounding this work completed a few months ago (it was ready before the
resctrl selftest series you merged this week and a couple of the patches also carry
your "Reviewed-by:"). I believe that it it ready for inclusion. Could you please
consider including this work in the kselftest repo?

> 
> [PATCH] selftests/resctrl: Use correct exit code when tests fail
> https://lore.kernel.org/lkml/20230309145757.2280518-1-peternewman@google.com/

This patch is also ready for inclusion and continues to apply cleanly.

Thank you very much

Reinette