Re: [PATCH 3/4] selftests/resctrl: Add support for MBM and MBA tests on AMD

Moger, Babu
2024-02-26T18:02:37Z

Hi Reinette

On 2/23/24 16:39, Reinette Chatre wrote:
> Hi Babu,
>
> On 2/23/2024 11:47 AM, Moger, Babu wrote:
>> On 2/23/24 13:30, Moger, Babu wrote:
>>> On 2/23/24 04:53, Ilpo Järvinen wrote:
>>>> On Thu, 22 Feb 2024, Babu Moger wrote:
>>>>> @@ -131,8 +130,7 @@ static void run_mba_test(const char * const *benchmark_cmd, int cpu_no)
>>>>> }
>>>>>
>>>>> if (!validate_resctrl_feature_request("MB", NULL) ||
>>>>> - !validate_resctrl_feature_request("L3_MON", "mbm_local_bytes") ||
>>>>> - (get_vendor() != ARCH_INTEL)) {
>>>>> + !validate_resctrl_feature_request("L3_MON", "mbm_local_bytes")) {
>>>>> ksft_test_result_skip("Hardware does not support MBA or MBA is disabled\n");
>>>>> goto cleanup;
>>>>> }
>>>>
>>>> These need to be rebased as this code moved into per test files with the
>>>> generic test framework. The .vendor_specific field is the new way to make
>>>> tests limited to particular vendors.
>>>
>>> Hmm. I picked the latest code from tip/master. Where is the latest code
>>> now? Is it yet to be submitted?
>>>
>>> I can wait for your code to merge first.
>>
>> Oh. ok. Here it is
>> https://lore.kernel.org/lkml/cover.1708072203.git.maciej.wieczor-retman@intel.com/
>>
>> I will rebase it on top of this next revision.
>
> The resctrl selftest changes are routed through the kselftest repo. Shuah just picked
> up this series you noticed but it is based on the framework changes from Ilpo that
> is already merged. I recommend that you base your next series on the "next" branch
> of the kselftest repo [1].

Sure. Will do.
>
> Reinette
>
> [1] git://git.kernel.org/pub/scm/linux/kernel/git/shuah/linux-kselftest.git

--
Thanks
Babu Moger