All of lore.kernel.org
 help / color / mirror / Atom feed
From: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
To: ltp@lists.linux.it
Subject: [LTP] [PATCH] device-drivers/cpufreq_boost: skip test on virtual machines
Date: Wed, 23 Jun 2021 10:11:19 +0200	[thread overview]
Message-ID: <2a08d5c4-c002-7284-03d1-ff4441f8c9c5@canonical.com> (raw)
In-Reply-To: <20210623080157.26424-1-krzysztof.kozlowski@canonical.com>

On 23/06/2021 10:01, Krzysztof Kozlowski wrote:
> Testing CPU frequency boost on a virtual machines is not reliable.  Even
> if cpufreq driver reports boost frequencies, the underlying hypervisor
> might decide otherwise.
> 
> For example on AWS x1e.xlarge instance (claimed to be running on Intel
> Xeon E7 8880) under Xen hypervisor the test always fails:
> 
>     cpufreq_boost    0  TINFO  :  found 'intel_pstate' driver, sysfs knob '/sys/devices/system/cpu/intel_pstate/no_turbo'
>     cpufreq_boost    0  TINFO  :  maximum speed is 3100000 KHz
>     cpufreq_boost    0  TINFO  :  load CPU0 with boost enabled
>     cpufreq_boost    0  TINFO  :  elapsed time is 1155 ms
>     cpufreq_boost    0  TINFO  :  load CPU0 with boost disabled
>     cpufreq_boost    0  TINFO  :  elapsed time is 1155 ms
>     cpufreq_boost    1  TFAIL  :  cpufreq_boost.c:186: compare time spent with and without boost (-2%)
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
> ---
>  testcases/kernel/device-drivers/cpufreq/cpufreq_boost.c | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/testcases/kernel/device-drivers/cpufreq/cpufreq_boost.c b/testcases/kernel/device-drivers/cpufreq/cpufreq_boost.c
> index b9739db37cb7..67917b3fea25 100644
> --- a/testcases/kernel/device-drivers/cpufreq/cpufreq_boost.c
> +++ b/testcases/kernel/device-drivers/cpufreq/cpufreq_boost.c
> @@ -90,6 +90,9 @@ static void setup(void)
>  	unsigned int i;
>  	tst_require_root();
>  
> +	if (tst_is_virt(VIRT_ANY))
> +		tst_brkm(TCONF, NULL, "running in a virtual machine, overclock not reliably measureable");
> +
>  	for (i = 0; i < ARRAY_SIZE(cdrv); ++i) {
>  		fd = open(cdrv[i].file, O_RDWR);
>  		if (fd == -1)
> 

Optionally, under virtual machine the test failure could be converted to
accepted pass. This would still allow to test CPUfreq boosting
interface. Any preferences?

Best regards,
Krzysztof

  reply	other threads:[~2021-06-23  8:11 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-06-23  8:01 [LTP] [PATCH] device-drivers/cpufreq_boost: skip test on virtual machines Krzysztof Kozlowski
2021-06-23  8:11 ` Krzysztof Kozlowski [this message]
2021-06-23 11:38   ` Cyril Hrubis
2021-06-23 15:33     ` Krzysztof Kozlowski
2021-08-11 10:12       ` Krzysztof Kozlowski
2021-09-08 13:03         ` Cyril Hrubis
2021-09-08 13:03           ` Cyril Hrubis

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=2a08d5c4-c002-7284-03d1-ff4441f8c9c5@canonical.com \
    --to=krzysztof.kozlowski@canonical.com \
    --cc=ltp@lists.linux.it \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.