From: Pavel Tatashin <pasha.tatashin@soleen.com>
To: Wei Liu <wei.liu@kernel.org>
Cc: Linux on Hyper-V List <linux-hyperv@vger.kernel.org>,
Joerg Roedel <jroedel@suse.de>,
Stephen Hemminger <sthemmin@microsoft.com>,
Will Deacon <will@kernel.org>,
Haiyang Zhang <haiyangz@microsoft.com>,
Linux Kernel List <linux-kernel@vger.kernel.org>,
Michael Kelley <mikelley@microsoft.com>,
"open list:IOMMU DRIVERS" <iommu@lists.linux-foundation.org>,
Nuno Das Neves <nunodasneves@linux.microsoft.com>,
Sunil Muthuswamy <sunilmut@microsoft.com>,
virtualization@lists.linux-foundation.org,
Vineeth Pillai <viremana@linux.microsoft.com>,
Joerg Roedel <joro@8bytes.org>
Subject: Re: [PATCH v5 04/16] iommu/hyperv: don't setup IRQ remapping when running as root
Date: Wed, 20 Jan 2021 11:08:34 -0500 [thread overview]
Message-ID: <CA+CK2bByGvCr_H3_wv_3-vAOONhRenonFNeHff5UdeFLDxSoUw@mail.gmail.com> (raw)
In-Reply-To: <20210120120058.29138-5-wei.liu@kernel.org>
On Wed, Jan 20, 2021 at 7:01 AM Wei Liu <wei.liu@kernel.org> wrote:
>
> The IOMMU code needs more work. We're sure for now the IRQ remapping
> hooks are not applicable when Linux is the root partition.
>
> Signed-off-by: Wei Liu <wei.liu@kernel.org>
> Acked-by: Joerg Roedel <jroedel@suse.de>
> Reviewed-by: Vitaly Kuznetsov <vkuznets@redhat.com>
> ---
> drivers/iommu/hyperv-iommu.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/iommu/hyperv-iommu.c b/drivers/iommu/hyperv-iommu.c
> index 1d21a0b5f724..b7db6024e65c 100644
> --- a/drivers/iommu/hyperv-iommu.c
> +++ b/drivers/iommu/hyperv-iommu.c
> @@ -20,6 +20,7 @@
> #include <asm/io_apic.h>
> #include <asm/irq_remapping.h>
> #include <asm/hypervisor.h>
> +#include <asm/mshyperv.h>
>
> #include "irq_remapping.h"
>
> @@ -122,7 +123,7 @@ static int __init hyperv_prepare_irq_remapping(void)
>
> if (!hypervisor_is_type(X86_HYPER_MS_HYPERV) ||
> x86_init.hyper.msi_ext_dest_id() ||
> - !x2apic_supported())
> + !x2apic_supported() || hv_root_partition)
> return -ENODEV;
Reviewed-by: Pavel Tatashin <pasha.tatashin@soleen.com>
_______________________________________________
Virtualization mailing list
Virtualization@lists.linux-foundation.org
https://lists.linuxfoundation.org/mailman/listinfo/virtualization
next prev parent reply other threads:[~2021-01-20 16:10 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20210120120058.29138-1-wei.liu@kernel.org>
[not found] ` <20210120120058.29138-3-wei.liu@kernel.org>
2021-01-20 16:03 ` [PATCH v5 02/16] x86/hyperv: detect if Linux is the root partition Pavel Tatashin
2021-01-26 0:31 ` Michael Kelley via Virtualization
[not found] ` <20210120120058.29138-7-wei.liu@kernel.org>
2021-01-20 15:12 ` [PATCH v5 06/16] x86/hyperv: allocate output arg pages if required kernel test robot
2021-01-20 19:44 ` Pavel Tatashin
2021-01-26 0:41 ` Michael Kelley via Virtualization
[not found] ` <20210120120058.29138-2-wei.liu@kernel.org>
2021-01-20 15:57 ` [PATCH v5 01/16] asm-generic/hyperv: change HV_CPU_POWER_MANAGEMENT to HV_CPU_MANAGEMENT Pavel Tatashin
2021-01-26 0:25 ` Michael Kelley via Virtualization
[not found] ` <20210120120058.29138-4-wei.liu@kernel.org>
2021-01-20 16:06 ` [PATCH v5 03/16] Drivers: hv: vmbus: skip VMBus initialization if Linux is root Pavel Tatashin
2021-01-26 0:32 ` Michael Kelley via Virtualization
[not found] ` <20210120120058.29138-5-wei.liu@kernel.org>
2021-01-20 16:08 ` Pavel Tatashin [this message]
2021-01-26 0:33 ` [PATCH v5 04/16] iommu/hyperv: don't setup IRQ remapping when running as root Michael Kelley via Virtualization
[not found] ` <20210120120058.29138-6-wei.liu@kernel.org>
2021-01-20 16:13 ` [PATCH v5 05/16] clocksource/hyperv: use MSR-based access if " Pavel Tatashin
2021-01-26 0:34 ` Michael Kelley via Virtualization
[not found] ` <20210120120058.29138-8-wei.liu@kernel.org>
2021-01-26 0:48 ` [PATCH v5 07/16] x86/hyperv: extract partition ID from Microsoft Hypervisor if necessary Michael Kelley via Virtualization
[not found] ` <20210202150353.6npksy7tobrvfqlt@liuwe-devbox-debian-v2>
2021-02-04 16:33 ` Michael Kelley via Virtualization
[not found] ` <20210120120058.29138-9-wei.liu@kernel.org>
2021-01-26 0:49 ` [PATCH v5 08/16] x86/hyperv: handling hypercall page setup for root Michael Kelley via Virtualization
[not found] ` <20210120120058.29138-10-wei.liu@kernel.org>
2021-01-26 1:20 ` [PATCH v5 09/16] x86/hyperv: provide a bunch of helper functions Michael Kelley via Virtualization
[not found] ` <20210120120058.29138-11-wei.liu@kernel.org>
2021-01-26 1:21 ` [PATCH v5 10/16] x86/hyperv: implement and use hv_smp_prepare_cpus Michael Kelley via Virtualization
[not found] ` <20210120120058.29138-12-wei.liu@kernel.org>
2021-01-26 1:22 ` [PATCH v5 11/16] asm-generic/hyperv: update hv_msi_entry Michael Kelley via Virtualization
[not found] ` <20210120120058.29138-13-wei.liu@kernel.org>
2021-01-26 1:23 ` [PATCH v5 12/16] asm-generic/hyperv: update hv_interrupt_entry Michael Kelley via Virtualization
[not found] ` <20210120120058.29138-14-wei.liu@kernel.org>
2021-01-26 1:26 ` [PATCH v5 13/16] asm-generic/hyperv: introduce hv_device_id and auxiliary structures Michael Kelley via Virtualization
[not found] ` <20210202170248.4hds554cyxpuayqc@liuwe-devbox-debian-v2>
[not found] ` <20210203132601.ftpwgs57qtok47cg@liuwe-devbox-debian-v2>
[not found] ` <CAK8P3a0m8jEij-RdP1PTcNcJW2+mXQ1dA4=s+JLXhnv+NyFiHw@mail.gmail.com>
[not found] ` <20210203140906.g35zr7366hh7p5f3@liuwe-devbox-debian-v2>
2021-02-04 16:46 ` Michael Kelley via Virtualization
[not found] ` <20210120120058.29138-15-wei.liu@kernel.org>
2021-01-26 1:27 ` [PATCH v5 14/16] asm-generic/hyperv: import data structures for mapping device interrupts Michael Kelley via Virtualization
[not found] ` <20210120120058.29138-16-wei.liu@kernel.org>
2021-01-27 5:47 ` [PATCH v5 15/16] x86/hyperv: implement an MSI domain for root partition Michael Kelley via Virtualization
[not found] ` <20210202173153.jkbvwck2vsjlbjbz@liuwe-devbox-debian-v2>
2021-02-02 18:15 ` Michael Kelley via Virtualization
[not found] ` <20210120120058.29138-17-wei.liu@kernel.org>
2021-01-27 5:47 ` [PATCH v5 16/16] iommu/hyperv: setup an IO-APIC IRQ remapping " Michael Kelley via Virtualization
[not found] ` <20210203124700.ugx5vd526455u7lb@liuwe-devbox-debian-v2>
2021-02-04 16:41 ` Michael Kelley via Virtualization
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=CA+CK2bByGvCr_H3_wv_3-vAOONhRenonFNeHff5UdeFLDxSoUw@mail.gmail.com \
--to=pasha.tatashin@soleen.com \
--cc=haiyangz@microsoft.com \
--cc=iommu@lists.linux-foundation.org \
--cc=joro@8bytes.org \
--cc=jroedel@suse.de \
--cc=linux-hyperv@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mikelley@microsoft.com \
--cc=nunodasneves@linux.microsoft.com \
--cc=sthemmin@microsoft.com \
--cc=sunilmut@microsoft.com \
--cc=viremana@linux.microsoft.com \
--cc=virtualization@lists.linux-foundation.org \
--cc=wei.liu@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).