Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:6 as permitted sender) client-ip=2620:137:e000::3:6;
Message-ID: <213fb592-de7d-405e-88bd-595a1cb33d48@linuxfoundation.org>
Date:   Mon, 16 Oct 2023 13:08:35 -0600
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH v4 0/2] Fix issues observed in selftests/amd-pstate
Content-Language: en-US
To:     Swapnil Sapkal <swapnil.sapkal@amd.com>, ray.huang@amd.com,
        shuah@kernel.org
Cc:     sukrut.bellary@gmail.com, li.meng@amd.com, gautham.shenoy@amd.com,
        wyes.karny@amd.com, Perry.Yuan@amd.com, Mario.Limonciello@amd.com,
        linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org,
        linux-kselftest@vger.kernel.org,
        Shuah Khan <skhan@linuxfoundation.org>
References: <20231012093225.2045109-1-swapnil.sapkal@amd.com>
From:   Shuah Khan <skhan@linuxfoundation.org>
In-Reply-To: <20231012093225.2045109-1-swapnil.sapkal@amd.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk

On 10/12/23 03:32, Swapnil Sapkal wrote:
> This series fixes issues observed with selftests/amd-pstate while
> running performance comparison tests with different governors. First
> patch changes relative paths with absolute paths and also change it
> with correct paths wherever it is broken.
> The second patch adds an option to provide perf binary path to
> handle the case where distro perf does not work.
> 
> Changelog v3->v4:
> 	* Addressed review comments from v3
> 
> Swapnil Sapkal (2):
>    selftests/amd-pstate: Fix broken paths to run workloads in
>      amd-pstate-ut
>    selftests/amd-pstate: Added option to provide perf binary path
> 
>   .../x86/amd_pstate_tracer/amd_pstate_trace.py |  3 +--
>   .../testing/selftests/amd-pstate/gitsource.sh | 17 +++++++++------
>   tools/testing/selftests/amd-pstate/run.sh     | 21 +++++++++++++------
>   tools/testing/selftests/amd-pstate/tbench.sh  |  4 ++--
>   4 files changed, 29 insertions(+), 16 deletions(-)
> 

Both applied to linux-kselftest next for Linux 6.7-rc1

thanks,
-- Shuah