[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [XEN PATCH v3 0/5] FF-A notifications
- To: Jens Wiklander <jens.wiklander@xxxxxxxxxx>
- From: Bertrand Marquis <Bertrand.Marquis@xxxxxxx>
- Date: Fri, 26 Apr 2024 09:23:22 +0000
- Accept-language: en-GB, 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=arm.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=FnfeMiupf5ZJtGRQX/qIIqjz8YmKZy6FB5ZRxt3mw1k=; b=VSM8/S/eV+RZUqOvPQWrcXUMzlKNB6x1NY8N61JPPTVW2CovmZFWozU4CLR5UtAWD5pS+QA/YKcH5iA4sMO7sljo5MMG0WsHAzmyPaBf2d8CpIiEo0olVKp+EGsuJ+kc0NP1nwxqHf/c80vZSsEmKD4DHJdqU32GY3ALUwBatio6npD4X7DzKJKM7xlehDSJiKYaRyF0WlZzuEY4Fnr0REAsKHqz9avlL1sZbSyk+rTiIHIO83s7t4rLUEGl8pLnyPxk3qOv97UNszNKP32urz0s++is2QEu13dreOu77iMccBi5hm/bED4f9+NvQSV31r67DHGSL4QihwHMw6TQdg==
- 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=FnfeMiupf5ZJtGRQX/qIIqjz8YmKZy6FB5ZRxt3mw1k=; b=Z3Qj7fdB2+nsZ00InQaRMZq3OsRsSf4iAsUpymgMlguHcKAYFehWXZ88tP58VgAV8WTJ9JuoM3fDLxCcBIg89lIghPeVX/TsusXobaWZEXzCvpovqRk/aiNShCY/WvmsxMu/paLaWhw6YwEnCiJ5LCkQWMx9/pqPUbgzqxr6hSkGweLaKvozL5O+eqmme3OdhfDgDscXu4KTBPoYS73pVDVKw46cBHVsM1vjczwgu6+2Wh8cMywEexMrcbcmeBOGb4GikNd/kAO9Wo8fxtMkC1iBwWID/fb3jIIcMnZnOuNyqGBRsAJopmgrjdiP40mtcjtzUJGtQ5A+AgLqsVt2Dg==
- Arc-seal: i=2; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=pass; b=RoAJhDRMCJ2/bHYO/Frar1eWZDiLcUoRRnlzzUbuTPD1wiGzRIJFpszfUjqbJ9U8ooeN1/WCHZuvWlmdFVkq4hZS5CIiCEngtB4ywZjdr+oB35HEeYMB1unrymNqf/K927wnfI5EmRFEusxvkbADc75YKVlM3LCJbLvKaoPz9sjEQpyp9NRb0VCitJ3smmow1oJwnbAHNZMOXjR29Hq3HH4ike/ZeZiwzpGcC7TohoVoWCZ0D3q2pmfrZ0PRtaWIXnxCP56EarJ0uNn59eh+KfXbRjpatEabeKZ3POkcngl43yxPa1In0gUL7jI6O0gc1erI+v8gHVC+Mb5agRuXZQ==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=doIutRPTrq9vo4eEiMptLIxPuzfecnzmmR61qvO8UMXB7BxhvYd+6ZkrcTMMMoDNE5aDP2bdEEaivNHyEYRA0Z4OXEoEE07YH+FE4u9DAX0RabfDuX5mCaUa8ndJ5oBrzQvjLZv5i6UgsNFcINN63bsxFcTDgslTo844PHawxNUaYRplWhLU9XExi5Hqxa4ff5rnIbZ1W+edemiWz9NjmSIXpaFXjH5qPFRNp7zd3lshe0+5tMEoHaF/7VR3VcvpawRG5VWUNSvxuRSTcZR5ZQTaA7WaO4uQs9mZ7zXQ7tw1RoJ+mtVCYrCeLPC9qXwT8vDSu8D8Ho6FMToh+xmdDA==
- 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>, "patches@xxxxxxxxxx" <patches@xxxxxxxxxx>, Volodymyr Babchuk <volodymyr_babchuk@xxxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Julien Grall <julien@xxxxxxx>, Michal Orzel <michal.orzel@xxxxxxx>
- Delivery-date: Fri, 26 Apr 2024 09:24:06 +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: AQHal7Zpi/+zN0xmAUWEyp+UnnJztrF6R0sA
- Thread-topic: [XEN PATCH v3 0/5] FF-A notifications
Hi Jens
> On 26 Apr 2024, at 10:47, Jens Wiklander <jens.wiklander@xxxxxxxxxx> wrote:
>
> Hi,
>
> This patch set adds support for FF-A notifications. We only support
> global notifications, per vCPU notifications remain unsupported.
>
> The first three patches are further cleanup and can be merged before the
> rest if desired.
>
> A physical SGI is used to make Xen aware of pending FF-A notifications. The
> physical SGI is selected by the SPMC in the secure world. Since it must not
> already be used by Xen the SPMC is in practice forced to donate one of the
> secure SGIs, but that's normally not a problem. The SGI handling in Xen is
> updated to support registration of handlers for SGIs that aren't statically
> assigned, that is, SGI IDs above GIC_SGI_MAX.
>From my point of view:
- patches 1 to 3 are ready to be commited.
- patch 4 will need a R-b from an other maintainer.
- patch 5 has still some stuff to be checked or fixed but could be
handled as a single patch if the rest or the serie is merged.
Regards
Bertrand
>
> Thanks,
> Jens
>
> v2->v3:
> - "xen/arm: ffa: support notification" and
> "xen/arm: allow dynamically assigned SGI handlers" updated as requestsed,
> details in each patch.
>
> v1->v2:
> - "xen/arm: ffa: support notification" and
> "xen/arm: allow dynamically assigned SGI handlers" updated as requestsed,
> details in each patch.
> - Added Bertrands R-B for "xen/arm: ffa: refactor ffa_handle_call()",
> "xen/arm: ffa: use ACCESS_ONCE()", and
> "xen/arm: ffa: simplify ffa_handle_mem_share()"
>
> Jens Wiklander (5):
> xen/arm: ffa: refactor ffa_handle_call()
> xen/arm: ffa: use ACCESS_ONCE()
> xen/arm: ffa: simplify ffa_handle_mem_share()
> xen/arm: allow dynamically assigned SGI handlers
> xen/arm: ffa: support notification
>
> xen/arch/arm/gic.c | 12 +-
> xen/arch/arm/include/asm/gic.h | 2 +-
> xen/arch/arm/irq.c | 18 +-
> xen/arch/arm/tee/Makefile | 1 +
> xen/arch/arm/tee/ffa.c | 83 +++++--
> xen/arch/arm/tee/ffa_notif.c | 378 ++++++++++++++++++++++++++++++++
> xen/arch/arm/tee/ffa_partinfo.c | 9 +-
> xen/arch/arm/tee/ffa_private.h | 56 ++++-
> xen/arch/arm/tee/ffa_shm.c | 33 ++-
> xen/include/public/arch-arm.h | 14 ++
> 10 files changed, 551 insertions(+), 55 deletions(-)
> create mode 100644 xen/arch/arm/tee/ffa_notif.c
>
> --
> 2.34.1
>
|