[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v2 3/5] automation: Add the expect script with test case for FVP


  • To: Michal Orzel <michal.orzel@xxxxxxx>
  • From: Henry Wang <Henry.Wang@xxxxxxx>
  • Date: Fri, 8 Dec 2023 09:21:45 +0000
  • Accept-language: zh-CN, en-US
  • Arc-authentication-results: i=2; mx.microsoft.com 1; spf=pass (sender ip is 63.35.35.123) smtp.rcpttodomain=lists.xenproject.org smtp.mailfrom=arm.com; dmarc=pass (p=none sp=none pct=100) action=none header.from=arm.com; dkim=pass (signature was verified) header.d=armh.onmicrosoft.com; arc=pass (0 oda=1 ltdi=1 spf=[1,1,smtp.mailfrom=arm.com] dkim=[1,1,header.d=arm.com] dmarc=[1,1,header.from=arm.com])
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=arm.com; dmarc=pass action=none header.from=arm.com; dkim=pass header.d=arm.com; arc=none
  • Arc-message-signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=dOLpXlf3E3D3Vw8JXIqluXz4oNTHkg/1j/2b0/ZaXSI=; b=JXNsu3cls9IfIarZlneagOf7scCBY0esE1zcuLD6CzL2++NiVE6baAALbHhlczwki327mVGyzKmgjAtQaAzL8N6DXP55nGfEccV4s8KT3QxJS+eKGCCC8EAhHaPecoflotJKc8v8p+MQOi90dfiODT1zsoklGqSuEMXv2bEL5ds+cTBdUMF1h4FSmn3MR90rmTtZraME/j6rqr/lqhcPR0/Dy+dEmL7V36UZ4ru/8M9a3umSlY5TJofNPQiYaKRzN+kruvnx2HN44bMqQCcfMw2zPDAhQAW9uPnGxqL2kJ75ziqvLniMPzoi8Ps099UgHo6j1/bdrZdzbUbXcz5/aA==
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=dOLpXlf3E3D3Vw8JXIqluXz4oNTHkg/1j/2b0/ZaXSI=; b=LpkwX1U20y7Ld9ixwlgrITQvw2Jzt18Ov6Cybfs0byfjLVNCrb/cxE8Buex1QVVD+25ofz1OPhNR9/k9lVc/24bDSS7MlqNbeiug2/0Chiq6Ms/R+B79XCR2waiLT/RxUVBe5vtCZbdq+lz/eFNbI0k2EQM2jZhEP97XCweDgYebpFiVeMZVmvEbZNq8iv4vv82/XZbl1JixMs240wpj0knCnEhhGrZLio+imO34L7G+MfNYFZ9hrLDnH2lHHbzrnfTers+IzQY0JmHlHS6YZD6JdPv73hjVHCUDbaYa7CkvS4EItAJXsQgkiZ8lHDB6UyIdtux3imHPKXjYnO/RWg==
  • Arc-seal: i=2; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=pass; b=RLgsvRtEoUmpg4uVGMs3b+HAVkxnxiNLOjMb9XSJAZsqiEYEeerzuVzDfFmv7+E/qCoMjpx4xjNpmT60r6wPQ1Y0nKjBnzf5TnJDpypFElGW3HxTKvTqg/ALwZBJAuY/niPxFEa5sqfzcrw2bObL9od9qV6y85Gr/FTeGwh4xH7YwAcxVkEddGNkXd2YCmiCZirwOagmM8ydmC8i/1ftvAkY3LXkmom6yj16+KTEZu5ioV3/1HIzuW7xPuPyWSzfaDerVOVaovWbUhBPa/kwhQ9TOxgRIZgCBYPlVpq8VlBUIWC4nCbcV4oQlGDFyP07ll8hln2ltTXT4skHtIaU6w==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=ZpZRrFQqIPZGZQW+yce24K4fWguv5MgB35oaLna3Oq1RL4wUsjbt7RQ4x3oRa6S/w9ypLPL+Oe9FOHjkoYahPZE6m7wr6T8Xt9ZQyo0te9GyLwxhEJSwwtwcsT0MCnyX2rXMUFZ+M8Am9IPoCcu39ofIVL539xnh62uXLNscY7QQWHq8dsIHzyR5iK99WhkdGI01+mUfzmBd6hLhB7pPpU4X5gsKlB9FtmyCCDTH+GChl5u7QopeSKclg99NqlgXdEu4uT3BWyHVUlB6ll7GnqqacGSEmTAIRkgsSHS/2oTYVD/CqEzO1ZSrsIOf5xr7l9vyNwhgrWRagxhRU4B53Q==
  • Authentication-results-original: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com;
  • Cc: Xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Doug Goldstein <cardoe@xxxxxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Julien Grall <julien@xxxxxxx>, Bertrand Marquis <Bertrand.Marquis@xxxxxxx>, Wei Chen <Wei.Chen@xxxxxxx>
  • Delivery-date: Fri, 08 Dec 2023 09:22:07 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
  • Nodisclaimer: true
  • Original-authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com;
  • Thread-index: AQHaKZoAVIHO2rCYDECWUL8sUR2sfLCfFe6AgAACFwCAAAHkAIAAAsIA
  • Thread-topic: [PATCH v2 3/5] automation: Add the expect script with test case for FVP

Hi Michal,

> On Dec 8, 2023, at 17:11, Michal Orzel <michal.orzel@xxxxxxx> wrote:
> On 08/12/2023 10:05, Henry Wang wrote:
>> 
>> Hi Michal,
>> 
>>> On Dec 8, 2023, at 16:57, Michal Orzel <michal.orzel@xxxxxxx> wrote:
>>> 
>>> Hi Henry,
>>> 
>>> On 08/12/2023 06:46, Henry Wang wrote:
>>>> diff --git a/automation/scripts/expect/fvp-base-smoke-dom0-arm64.exp 
>>>> b/automation/scripts/expect/fvp-base-smoke-dom0-arm64.exp
>>>> new file mode 100755
>>>> index 0000000000..25d9a5f81c
>>>> --- /dev/null
>>>> +++ b/automation/scripts/expect/fvp-base-smoke-dom0-arm64.exp
>>>> @@ -0,0 +1,73 @@
>>>> +#!/usr/bin/expect
>>>> +
>>>> +set timeout 2000
>>> Do we really need such a big timeout (~30 min)?
>>> Looking at your test job, it took 16 mins (quite a lot but I know FVP is 
>>> slow
>>> + send_slow slows things down)
>> 
>> This is a really good question. I did have the same question while working on
>> the negative test today. The timeout 2000 indeed will fail the job at about 
>> 30min,
>> and waiting for it is indeed not really pleasant.
>> 
>> But my second thought would be - from my observation, the overall time now
>> would vary between 15min ~ 20min, and having a 10min margin is not that crazy
>> given that we probably will do more testing from the job in the future, and 
>> if the
>> GitLab Arm worker is high loaded, FVP will probably become slower. And 
>> normally
>> we don’t even trigger the timeout as the job will normally pass. So I decided
>> to keep this.
>> 
>> Mind sharing your thoughts about the better value of the timeout? Probably 
>> 25min?
> From what you said that the average is 15-20, I think we can leave it set to 
> 30.
> But I wonder if we can do something to decrease the average time. ~20 min is 
> a lot
> even for FVP :) Have you tried setting send_slow to something lower than 
> 100ms?
> That said, we don't send too many chars to FVP, so I doubt it would play a 
> major role
> in the overall time.

I agree with the send_slow part. Actually I do have the same concern, here are 
my current
understanding and I think you will definitely help with your knowledge:
If you check the full log of Dom0 booting, for example [1], you will find that 
we wasted so
much time in starting the services of the OS (modloop, udev-settle, etc). All 
of these services
are retried many times but in the end they are still not up, and from my 
understanding they
won’t affect the actual test(?) If we can somehow get rid of these services 
from rootfs, I think
we can save a lot of time.

And honestly, I noticed that qemu-alpine-arm64-gcc suffers from the same 
problem and it also
takes around 15min to finish. So if we managed to tailor the services from the 
filesystem, we
can save a lot of time.

But I found it difficult to do the proper tailoring, any suggestions?

[1] 
https://gitlab.com/xen-project/people/henryw/xen/-/jobs/5708557850/artifacts/file/smoke.serial

Kind regards,
Henry

> I use FVP quite rarely these days, so you should know better if this can be 
> perceived as
> usual/normal behavior.
> 
> ~Michal
> 


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.