[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 4/5] automation: Add the script for the FVP smoke test
- To: Stefano Stabellini <sstabellini@xxxxxxxxxx>
- From: Henry Wang <Henry.Wang@xxxxxxx>
- Date: Fri, 8 Dec 2023 01:56:18 +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=JVAZ9qZ5G8o9H3m+vA/lVirzaKdcZi3ZZ1l3ZRuToHo=; b=kNte85bSKPlyhtLsTNk5YcNhpeDiN/w4/aOx/OXX1Fdu9LPPdPXQ4KWUDX1nf+yQY/w+yz//1QmQv7LoEaqRxJJKu+hYF1XgDeWpfM/ysYzfe8tviSnE4QmDkktexialTOlSrSpt3808kZRts5own6oPNE3twfU/DwvgenCSTJHmSq+HD80GpgJgf3C71c2g4ueXR0OgZe0YhoHlysyG50USKQQKapDpqtZVrxXmlZQCgwYjbc++wuoKphtprO36w1u45z04JJIMIdXPo6/eWmpEXf/Pod+Tb6bNWtysjGGzHNIh9zWE13GebWkJ8EEikiHzD8qqL4PkdUNCdt2RWg==
- 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=JVAZ9qZ5G8o9H3m+vA/lVirzaKdcZi3ZZ1l3ZRuToHo=; b=NFSFRaR+M+4engdK6scJrzcjhDngyeajEW2lAbBNG2GgJZHaSoFqci6OaB6HVPhdIIQgikKgELUMJwtd5OHhNkh6Ii1WEcOIU0URqefNf8g+9fFTF9U/x0OopRRiIpuoptljd7HgEq4TMFN1CIF0wt/IfEOGnnIkuW3K5m4p6aP1bUSWBn4gfG3uDDPfkM6lx7Fxfrap13TFcSTexYfmV3Hj1Q7e5r4YNZtgCEZ5Xjtu4fq19LmUoiU3dhVYHCfTIhWL7WVE6vycrQSYhocgD3AaNOumJGcVNMaLJaikPn4gWb2Pv3TOkB5xVe/eYn86VlV5licE5gNOgilkrj84yA==
- Arc-seal: i=2; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=pass; b=UvMfUJmKA6UcfCvNM9q0l8l6SkBcht3PlnKXYL7BcsCeRkUc81Xc1dUCuYH/9Jq/7fmixiEa65zZ3HYFCrWltQi7H5Ecsj83T5rsiZBgXJh7fuujZv1n8uq2d3gN0QJwFTNRpTyF49ytuqaC0sEj2BFlh61M+kQR8SC10+vYAZAHec4erEN0XLkj1YULe+CkUL3DSpJ1F6nLU5pOrVHxHSI9n1il0b4obwwKjc+GFxZarTA7MfXzbAfHnBNdmR+ZwWxg9vXskDgIC/wjXvb02fGyDbFc2Rr5lqgKnCQRdyk7ZFwsZUQYOysC2lftZIs09KuYlpDCJ9kUQDzPiYVfpw==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=EY5YGNEh0b8lEU+g5FGWJ+8pG701YvWsDTvpKckGXpOq0fK8jnsFON5lWxxwQNjlrsdj77kAQ5bSH3gY838PnFOCLAMDxl1wPuTlc++Q2ciQMIIaaZixOLYMqlzT2RdUH9yIlObfpZ1PJGQvGivuM/pJGTPSYzEIQIW9rbTxni1ppCat1Tmc6YH65ncmWyJDfVQeo+OCsgnsRI/8yUiNXOxx9HYZr5GUnii6+IL9yv/ZixwQWeQF3wNlpwlCHTHMhTkiD/1rli56Hy4rB/TzCYTBqLviEoyijZdMHV+va8Go4Mtg8KvtuDJChcMuQN1bpdHin6m9mscnbD/cuhyHJg==
- 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>, Michal Orzel <michal.orzel@xxxxxxx>, Julien Grall <julien@xxxxxxx>, Bertrand Marquis <Bertrand.Marquis@xxxxxxx>, Wei Chen <Wei.Chen@xxxxxxx>
- Delivery-date: Fri, 08 Dec 2023 01:57:13 +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: AQHaKRTRqeoumv0W/UiTEosHhxXqcLCenTGAgAAEDgA=
- Thread-topic: [PATCH 4/5] automation: Add the script for the FVP smoke test
Hi Stefano,
> On Dec 8, 2023, at 09:41, Stefano Stabellini <sstabellini@xxxxxxxxxx> wrote:
>
> On Thu, 7 Dec 2023, Henry Wang wrote:
>> This commit adds the shell script for the FVP smoke test. Similarly
>> as the QEMU jobs, the shell script will firstly prepare the DomU
>> BusyBox image, use the ImageBuilder to arrange the binaries in memory
>> and generate the U-Boot script, then start the test. To provide the
>> TFTP service for the FVP, the shell script will also start the TFTP
>> service, and copy the binaries needed by test to the TFTP directory
>> used by the TFTP server.
>>
>> Signed-off-by: Henry Wang <Henry.Wang@xxxxxxx>
>> ---
>> .../scripts/fvp-base-smoke-dom0-arm64.sh | 117 ++++++++++++++++++
>> 1 file changed, 117 insertions(+)
>> create mode 100755 automation/scripts/fvp-base-smoke-dom0-arm64.sh
>
> This script is clear and it is fine:
>
> Reviewed-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>
Thanks!
>
> My only concern is that the expect checks on what booted (Xen, Dom0,
> DomU) are inside the script fvp-base-smoke-dom0-arm64.exp rather than in
> this script. So if in the future we are going to have multiple tests
> with different configurations (for instance see
> qemu-smoke-dom0less-arm64.sh) we'll have to find a way to reuse
> fvp-base-smoke-dom0-arm64.exp somehow.
We do have ways to reuse expect script, for example, we can extract the common
code (variables and test logic) to the common.tcl file and source the
common.tcl file
in the expect script to reuse the code. The reason why I didn’t do that in this
series
is that currently there is only one script so I feel that there is not much
benefit to do
this instantly :) Let’s wait to see if there is more comments from others, and
I am
definitely open to refactor if there is the need to extract the common logic
(for example
when we add dom0less tests in the future).
Kind regards,
Henry
|
