linux-kernel - Re: [PATCH v2] iommu/hyper-v: Allow hyperv irq remapping without x2apic

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: <354a1247-056c-076f-850c-cda78f00c628@gmail.com>
Date:   Fri, 18 Nov 2022 11:01:53 +0800
From:   Tianyu Lan <ltykernel@...il.com>
To:     Nuno Das Neves <nunodasneves@...ux.microsoft.com>,
        linux-hyperv@...r.kernel.org, linux-kernel@...r.kernel.org,
        iommu@...ts.linux.dev
Cc:     mikelley@...rosoft.com, sunilmut@...rosoft.com, wei.liu@...nel.org,
        kys@...rosoft.com, Tianyu.Lan@...rosoft.com,
        haiyangz@...rosoft.com, decui@...rosoft.com, dwmw2@...radead.org,
        joro@...tes.org, will@...nel.org
Subject: Re: [PATCH v2] iommu/hyper-v: Allow hyperv irq remapping without
 x2apic

On 11/18/2022 4:11 AM, Nuno Das Neves wrote:
> If x2apic is not available, hyperv-iommu skips remapping
> irqs. This breaks root partition which always needs irqs
> remapped.
> 
> Fix this by allowing irq remapping regardless of x2apic,
> and change hyperv_enable_irq_remapping() to return
> IRQ_REMAP_XAPIC_MODE in case x2apic is missing.
> 
> Tested with root and non-root hyperv partitions.
> 
> Signed-off-by: Nuno Das Neves<nunodasneves@...ux.microsoft.com>
> ---
>   arch/x86/kernel/cpu/mshyperv.c |  6 ++++++
>   drivers/iommu/Kconfig          |  6 +++---
>   drivers/iommu/hyperv-iommu.c   | 11 ++++++++---
>   3 files changed, 17 insertions(+), 6 deletions(-)


Reviewed-by: Tianyu Lan <Tianyu.Lan@...rosoft.com>