linux-kernel - Re: [PATCH v3 1/1] x86/hyperv: Switch to msi_create_parent_irq

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <87o6rqy1yk.ffs@tglx>
Date: Thu, 04 Sep 2025 11:57:39 +0200
From: Thomas Gleixner <tglx@...utronix.de>
To: Nuno Das Neves <nunodasneves@...ux.microsoft.com>, Nam Cao
 <namcao@...utronix.de>, "K . Y . Srinivasan" <kys@...rosoft.com>, Marc
 Zyngier <maz@...nel.org>, Haiyang Zhang <haiyangz@...rosoft.com>, Wei Liu
 <wei.liu@...nel.org>, Dexuan Cui <decui@...rosoft.com>, Ingo Molnar
 <mingo@...hat.com>, Borislav Petkov <bp@...en8.de>, Dave Hansen
 <dave.hansen@...ux.intel.com>, x86@...nel.org, "H . Peter Anvin"
 <hpa@...or.com>, linux-hyperv@...r.kernel.org,
 linux-kernel@...r.kernel.org
Subject: Re: [PATCH v3 1/1] x86/hyperv: Switch to
 msi_create_parent_irq_domain()

On Wed, Sep 03 2025 at 12:40, Nuno Das Neves wrote:
> On 7/18/2025 12:57 PM, Nam Cao wrote:
>> Move away from the legacy MSI domain setup, switch to use
>> msi_create_parent_irq_domain().
>> 
>> While doing the conversion, I noticed that hv_irq_compose_msi_msg() is
>> doing more than it is supposed to (composing message content). The
>> interrupt allocation bits should be moved into hv_msi_domain_alloc().
>> However, I have no hardware to test this change, therefore I leave a TODO
>> note.
>> 
>> Signed-off-by: Nam Cao <namcao@...utronix.de>
>> ---
>>  arch/x86/hyperv/irqdomain.c | 111 ++++++++++++++++++++++++------------
>>  drivers/hv/Kconfig          |   1 +
>>  2 files changed, 77 insertions(+), 35 deletions(-)
>
> Tested on nested root partition.
>
> Looks good, thanks.
>
> Tested-by: Nuno Das Neves <nunodasneves@...ux.microsoft.com>
> Reviewed-by: Nuno Das Neves <nunodasneves@...ux.microsoft.com>

I assume this goes through the hyper-V tree.

Acked-by: Thomas Gleixner <tglx@...utronix.de>