[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
> 




 


Rackspace

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