[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
RE: [PATCH v5 1/2] arm/p2m: Rework p2m_init()
- To: "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>
- From: Henry Wang <Henry.Wang@xxxxxxx>
- Date: Tue, 18 Oct 2022 14:48:26 +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=yonvIVVAI+uAHvGelOtgqlT/jbY6Fp77ncSqyDsN0/M=; b=F05p4Ua5xKiQVhxPMEatOH/F4yjcUIm4Cn9nX7kTdzsrfVLuSY4nL790HVGvvCErSaWJlmqVww3FO8D/NQj93mrriZ2fMVH7g6lw6TjSuTsF0E+jGpWjO+AvyXG5r3+fnwVYi+3OdY6Cn8Nk18vOKPk36KoxdvQwcoRyFWefTNR2z4H7Yu0N1xZahmvoe+oA9IhG9t9GjlHWYkNRWEM+ZZVBpZGXLtJV1JoqRweaEntWa5Lcapg2GND5DiU4iSFjykXUbDACBuvaOMqfhPZ8x0S/6+U62LC3ftcy//WYV1Lj9edciFNEBPSktag7aa48M+dh8mQ1Ut0avpxLcuXWzg==
- 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=yonvIVVAI+uAHvGelOtgqlT/jbY6Fp77ncSqyDsN0/M=; b=JRaKLMKwq/roIwKrv+WOuYcL9M4tmjPC3h9n2GAIJ/DzOvlooNcAmevT5aUpSr7Een3A7apxZbAu4rdwTY2C6jCqI93XTUvZOE+rm9CpWHEO9YN/a91F8r9UXWocWpnyaly8YPkiEleBzFCQMGPDTQhcH5viRSw9z82t6DM2TfnXwSYHI/bElakzcCchGYEidoK4wdBwNg/IZWXjKGdxMDwfwqlJPtYBQzBo4DZJ9tMaVnqi4f8DRCAwIGaKGRnzQK5SIzuTjagkRhphMD/blA4yl+qTJa076L+0upEu0aLzMlWGcDvh1M9jgrXrwX/2Q8tL9Hup1rwpALt1wb4wng==
- Arc-seal: i=2; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=pass; b=XMSzDbz50vWvUnViWt3WBqouhATQ3nXn+S4UiWQuSNd+EnPGgWjVnX3MrLWXoLERkwLLPzxt9tSiStWYWM3TcwFS6LfbodPmwuOG7cYzFLANSV7xkoKRF4ACRTtu7jdOY4J1qgHMUc8TejWQtYjN95KweT+KOWg4DniNrnPo9ejL/YAa4IunaVsarwZx2q2W25uolcN9xsvkm8YsrBjrfuoVVGYTK/2dbd1+pzGT7OM1venesMNmp3UGHqE1xOh6PjLcLfty0MXHGiyVTqJsr8ulHrOBSpsHeMgEwkJ9PrEg9gZ1HIotmIvcb1fW8DPReYPWy+ZrJazRuL7qGGHV8Q==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=KTu6qNB6aLoM+CIwDbXY1iK/PZuWbhCObjOW9/5dD5hpRbueARtmq+k1kQszJs6Itz2jg7SqBFMOQVPwo8gux35liLGTcgKxZNLSPwzdDcSSof4XCaGzcPw5lBXFoWjqT6UPcsFtJGaRN6U/ZyMLKBgsoZfF0s27/xls/4/I5AUYuJPO7FYHt3Jrku/t55U22JhcR9I1HsumgVj2p6cEFfLe881Os0o6TqlSrb2T5QNQ3bu8xkIRHoUqiDs/RRi3TLucR7g+OyMzgCmdUHpA8fkmDUvwgHzpQA7LzxxpV6F2MEOXicms8mAuWlEm7biEhBfyBlMSk1GtqurxLBM7dQ==
- Authentication-results-original: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com;
- Cc: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Julien Grall <julien@xxxxxxx>, Bertrand Marquis <Bertrand.Marquis@xxxxxxx>, Henry Wang <Henry.Wang@xxxxxxx>, Volodymyr Babchuk <Volodymyr_Babchuk@xxxxxxxx>
- Delivery-date: Tue, 18 Oct 2022 14:48:57 +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: AQHY4v1ZlXAD3ap4t02LRXIwxIJsRK4UOW7Q
- Thread-topic: [PATCH v5 1/2] arm/p2m: Rework p2m_init()
> -----Original Message-----
> Subject: [PATCH v5 1/2] arm/p2m: Rework p2m_init()
>
> From: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
>
> p2m_init() is mostly trivial initialisation, but has two fallible operations
> which are on either side of the backpointer trigger for teardown to take
> actions.
>
> p2m_free_vmid() is idempotent with a failed p2m_alloc_vmid(), so rearrange
> p2m_init() to perform all trivial setup, then set the backpointer, then
> perform all fallible setup.
>
> This will simplify a future bugfix which needs to add a third fallible
> operation.
>
> No practical change.
>
> Signed-off-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
> Reviewed-by: Julien Grall <jgrall@xxxxxxxxxx>
> Reviewed-by: Bertrand Marquis <bertrand.marquis@xxxxxxx>
Release-acked-by: Henry Wang <Henry.Wang@xxxxxxx>
Kind regards,
Henry
|
