All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jamie Nguyen <jamien@nvidia.com>
To: Sudeep Holla <sudeep.holla@kernel.org>
Cc: Jamie Nguyen <jamien@nvidia.com>,
	Lorenzo Pieralisi <lpieralisi@kernel.org>,
	Hanjun Guo <guohanjun@huawei.com>,
	Catalin Marinas <catalin.marinas@arm.com>,
	Will Deacon <will@kernel.org>,
	"Rafael J. Wysocki" <rafael@kernel.org>,
	Len Brown <lenb@kernel.org>, Dat Mach <dmach@nvidia.com>,
	<linux-acpi@vger.kernel.org>,
	<linux-arm-kernel@lists.infradead.org>,
	<linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v2 0/3] ACPI: arm64: FFH Operation Region support for FF-A (offset 2)
Date: Wed, 23 Sep 2026 05:40:50 -0700	[thread overview]
Message-ID: <20260923124050.3270801-1-jamien@nvidia.com> (raw)
In-Reply-To: <20260922-exuberant-earwig-of-opportunity-4fecff@sudeepholla>

On Tue, Sep 22, 2026 at 07:06:02PM +0100, Sudeep Holla wrote:
> Equally ugly. FF-A spec doesn't talk about ACPI FFH, so it doesn't
> belong in FF-A driver.

Okay -- I'll move the ACPI-specific handling into ffh.c.

> So if on a platform, ASL has FFH with offset 2, we fail if FF-A driver
> fails to initialise ? What if this FFH offset 2 is used in some other
> device methods that need to be executed before FF-A module is inserted ?
> We pull that device down as well ? Is that acceptable ?

You're right. With the current registration, offset 2 may be unavailable
during initial enumeration or before module load. That's not suitable for
AML.

> There is no state management with simple FFA_MSG_SEND_DIRECT_REQ2, it is
> synchronous send request if you look at it along wih offset 0/1 and
> without FF-A driver knowledge.

The request itself is synchronous and register-only. But the whole operation
still shares FF-A state: version negotiation must be coordinated with other
FF-A use, and X1 == 0 needs UUID lookup. It looks like
FFA_PARTITION_INFO_GET_REGS is optional at the Non-secure physical instance
according to DEN0077A Table 13.1. As such, I believe a portable
implementation would need the shared RX/TX buffers. For example, if X1 is
zero and that ABI is unavailable, UUID lookup would need to fall back to
FFA_PARTITION_INFO_GET, whose results are returned through the caller's RX
buffer.

  reply	other threads:[~2026-09-23 12:41 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-09-01 19:29 [PATCH v2 0/3] ACPI: arm64: FFH Operation Region support for FF-A (offset 2) Jamie Nguyen
2026-09-01 19:29 ` [PATCH v2 1/3] firmware: arm_ffa: Split the response out of ffa_msg_send_direct_req2() Jamie Nguyen
2026-09-21  9:41   ` Hanjun Guo
2026-09-21 17:49     ` Jamie Nguyen
2026-09-22  1:49       ` Hanjun Guo
2026-09-01 19:29 ` [PATCH v2 2/3] ACPI: arm64: Add support for the FF-A FFH Operation Region (offset 2) Jamie Nguyen
2026-09-21  9:54   ` Hanjun Guo
2026-09-21 17:49     ` Jamie Nguyen
2026-09-01 19:29 ` [PATCH v2 3/3] firmware: arm_ffa: Back the ACPI FF-A FFH Operation Region Jamie Nguyen
2026-09-18 16:14 ` [PATCH v2 0/3] ACPI: arm64: FFH Operation Region support for FF-A (offset 2) Jamie Nguyen
2026-09-22  8:46 ` Sudeep Holla
2026-09-22 15:52   ` Jamie Nguyen
2026-09-22 18:06     ` Sudeep Holla
2026-09-23 12:40       ` Jamie Nguyen [this message]
2026-09-23 13:38         ` Sudeep Holla
2026-09-23 17:32           ` Jamie Nguyen
2026-09-24  9:39             ` Sudeep Holla
2026-09-24 15:23               ` Jamie Nguyen
2026-09-24 16:07                 ` Sudeep Holla

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20260923124050.3270801-1-jamien@nvidia.com \
    --to=jamien@nvidia.com \
    --cc=catalin.marinas@arm.com \
    --cc=dmach@nvidia.com \
    --cc=guohanjun@huawei.com \
    --cc=lenb@kernel.org \
    --cc=linux-acpi@vger.kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lpieralisi@kernel.org \
    --cc=rafael@kernel.org \
    --cc=sudeep.holla@kernel.org \
    --cc=will@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.