Hi,
I also think that we can close this thread.
It seems like the recent kernel won't display the warning with the commit.
Thanks, Sungbae ________________________________ From: Olivier Deprez Olivier.Deprez@arm.com Sent: Monday, May 5, 2025 5:30 PM To: Sudeep Holla Sudeep.Holla@arm.com Cc: hafnium@lists.trustedfirmware.org hafnium@lists.trustedfirmware.org; Sungbae Yoo sungbaey@nvidia.com; Jon Hunter jonathanh@nvidia.com; Byungkuk Seo bseo@nvidia.com; Bertrand Marquis Bertrand.Marquis@arm.com Subject: Re: Notification setup failed log in the latest Linux
You don't often get email from olivier.deprez@arm.com. Learn why this is importanthttps://aka.ms/LearnAboutSenderIdentification External email: Use caution opening links or attachments
Hi,
To follow up and hopefully close on this thread,
* I thought this was fixed in Hafnium with Commit 17c9ad327bfa ("fix(ff-a): report NPI and MEI unsupported to normal world")
Confirming using a recent tip of tree hafnium incl. patch above rejects reporting the NPI virtual id to normal world. In which case a recent kernel won't display the mentioned warning.
Regards, Olivier.
________________________________ From: Sudeep Holla sudeep.holla@arm.com Sent: 17 April 2025 15:19 To: Olivier Deprez Olivier.Deprez@arm.com Cc: hafnium@lists.trustedfirmware.org hafnium@lists.trustedfirmware.org; Sudeep Holla Sudeep.Holla@arm.com; Sungbae Yoo sungbaey@nvidia.com; Jon Hunter jonathanh@nvidia.com; Byungkuk Seo bseo@nvidia.com; Bertrand Marquis Bertrand.Marquis@arm.com Subject: Re: Notification setup failed log in the latest Linux
On Thu, Apr 17, 2025 at 12:06:16PM +0100, Olivier Deprez wrote:
Hi, On this actually after checking the driver * Meanwhile the error message seems to suggest the returned value is INTID 6 ? Which doesn't correlate with hafnium HF_NOTIFICATION_PENDING_INTID = 5? Value 6 may be the return value from irq_create_of_mapping [1], not the value returned by FFA_FEATURES(FFA_FEAT_NOTIFICATION_PENDING_INT) [2]
Yes that's correct.
----------------------------------------------------------------------From: Olivier Deprez via Hafnium hafnium@lists.trustedfirmware.org Sent: 17 April 2025 13:01 To: hafnium@lists.trustedfirmware.org hafnium@lists.trustedfirmware.org; Sungbae Yoo sungbaey@nvidia.com Cc: Jon Hunter jonathanh@nvidia.com; Byungkuk Seo bseo@nvidia.com; Sudeep Holla Sudeep.Holla@arm.com; Bertrand Marquis Bertrand.Marquis@arm.com Subject: [Hafnium] Re: Notification setup failed log in the latest Linux
Hi,
Thanks for your observations.
My own perspective is that normal world shall not try to enquire about the NPI virtual interrupt id used by hafnium to inject into secure partitions. NPI virtual interrupt id shall be defined by an Hypervisor and SPMC independently and don't necessarily need to match. An hypervisor uses whichever value it defines to inject into VMs. Similarly for an SPMC and secure partitions.
Indeed.
There may have been other reasons to design the driver this way because of presence of an hypervisor (and the driver running in a VM) vs a system without an hypervisor.
I thought this was fixed in Hafnium with Commit 17c9ad327bfa ("fix(ff-a): report NPI and MEI unsupported to normal world")
I don't see any issue with upstream hafnium and hence wanted to confirm if the above commit is present or not on the setup where the issue is observed.
We're discussing this internally and will come back to you with a more precise answer.
We can do SRI XOR NPI in the driver, but I don't recall the details as why it wasn't done that way. I do remember at the time we added this, we were not clear about it.
Not sure if Bertrand remembers anything more especially if this was needed by Xen ?
-- Regards, Sudeep