Re: [OS-BUILD PATCH] redhat/configs: Set CONFIG_VIRTIO_IOMMU on x86_64
From: Eric Auger on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1799#note_969390848 Hi Patrick, sure. Thank you for the info! ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email to kernel-le...@lists.fedoraproject.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/kernel@lists.fedoraproject.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[OS-BUILD PATCH] redhat/configs: Set CONFIG_VIRTIO_IOMMU on x86_64
From: Eric Auger redhat/configs: Set CONFIG_VIRTIO_IOMMU on x86_64 Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=2089765 Upstream Status: RHEL only Tested: with 9.1 guest featuring protected virtio-net-pci device We want to support the virtio-iommu on x86_64. So let's compile it. Signed-off-by: Eric Auger diff --git a/redhat/configs/common/generic/x86/x86_64/CONFIG_VIRTIO_IOMMU b/redhat/configs/common/generic/x86/x86_64/CONFIG_VIRTIO_IOMMU new file mode 100644 index blahblah..blahblah 100644 --- /dev/null +++ b/redhat/configs/common/generic/x86/x86_64/CONFIG_VIRTIO_IOMMU @@ -0,0 +1 @@ +CONFIG_VIRTIO_IOMMU=y -- https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1799 ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email to kernel-le...@lists.fedoraproject.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/kernel@lists.fedoraproject.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[OS-BUILD PATCH] redhat/configs: Fix CONFIG_VIRTIO_IOMMU to 'y' on aarch64
From: Eric Auger redhat/configs: Fix CONFIG_VIRTIO_IOMMU to 'y' on aarch64 Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1972795 Upstream Status: RHEL only Tested: on ARM with qemu (Jean-Philippe Brucker's qemu branch as the qemu integration is not yet upstream, with both protected virtio-blk-pci and a virtio-net-pci devices) Commit fe6d9534dbe4 ("redhat/configs: Set CONFIG_VIRTIO_IOMMU on aarch64") enabled the compilation of the virtio-iommu driver on ARM but set the config to 'm'. However if there are devices behind the vIOMMU that are needed during the boot such as a virtio-blk-pci device, the guest fails to boot as the driver is not loaded. Let's fix this bootstrap issue by compiling the driver in-kernel as it is done for other iommu drivers. Signed-off-by: Eric Auger diff --git a/redhat/configs/common/generic/arm/aarch64/CONFIG_VIRTIO_IOMMU b/redhat/configs/common/generic/arm/aarch64/CONFIG_VIRTIO_IOMMU index blahblah..blahblah 100644 --- a/redhat/configs/common/generic/arm/aarch64/CONFIG_VIRTIO_IOMMU +++ b/redhat/configs/common/generic/arm/aarch64/CONFIG_VIRTIO_IOMMU @@ -1 +1 @@ -CONFIG_VIRTIO_IOMMU=m +CONFIG_VIRTIO_IOMMU=y -- https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1289 ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email to kernel-le...@lists.fedoraproject.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/kernel@lists.fedoraproject.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
Re: [OS-BUILD PATCHv2] redhat/configs: Set CONFIG_VIRTIO_IOMMU on aarch64
From: Eric Auger on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1249#note_623588543 Makes sense, thank you. I submitted the updated version. ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email to kernel-le...@lists.fedoraproject.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/kernel@lists.fedoraproject.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[OS-BUILD PATCHv2] redhat/configs: Set CONFIG_VIRTIO_IOMMU on aarch64
From: Eric Auger redhat/configs: Set CONFIG_VIRTIO_IOMMU on aarch64 Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1972795 Upstream Status: RHEL only Tested: on ARM with qemu (Jean-Philippe Brucker's qemu branch as the qemu integration is not yet upstream) We want to support the virtio-iommu on ARM. So let's compile it. Signed-off-by: Eric Auger --- v1 -> v2: - do not set the config in ark/generic/arm/aarch64 but rather in common/generic/arm/aarch64 and just cares about the setting for ARM as the default already is set to false in common. diff --git a/redhat/configs/common/generic/arm/aarch64/CONFIG_VIRTIO_IOMMU b/redhat/configs/common/generic/arm/aarch64/CONFIG_VIRTIO_IOMMU new file mode 100644 index blahblah..blahblah 100644 --- /dev/null +++ b/redhat/configs/common/generic/arm/aarch64/CONFIG_VIRTIO_IOMMU @@ -0,0 +1 @@ +CONFIG_VIRTIO_IOMMU=m -- https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1249 ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email to kernel-le...@lists.fedoraproject.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/kernel@lists.fedoraproject.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[OS-BUILD PATCH] redhat/configs: Set CONFIG_VIRTIO_IOMMU on aarch64
From: Eric Auger redhat/configs: Set CONFIG_VIRTIO_IOMMU on aarch64 Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1972795 Upstream Status: RHEL only Tested: on ARM with qemu (Jean-Philippe Brucker's qemu branch as the qemu integration is not yet upstream) We want to support the virtio-iommu on ARM. So let's compile it. Signed-off-by: Eric Auger diff --git a/redhat/configs/ark/generic/CONFIG_VIRTIO_IOMMU b/redhat/configs/ark/generic/CONFIG_VIRTIO_IOMMU new file mode 100644 index blahblah..blahblah 100644 --- /dev/null +++ b/redhat/configs/ark/generic/CONFIG_VIRTIO_IOMMU @@ -0,0 +1 @@ +# CONFIG_VIRTIO_IOMMU is not set diff --git a/redhat/configs/ark/generic/arm/aarch64/CONFIG_VIRTIO_IOMMU b/redhat/configs/ark/generic/arm/aarch64/CONFIG_VIRTIO_IOMMU new file mode 100644 index blahblah..blahblah 100644 --- /dev/null +++ b/redhat/configs/ark/generic/arm/aarch64/CONFIG_VIRTIO_IOMMU @@ -0,0 +1 @@ +CONFIG_VIRTIO_IOMMU=m -- https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1249 ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email to kernel-le...@lists.fedoraproject.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/kernel@lists.fedoraproject.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
Re: [OS-BUILD PATCHv2] redhat/configs: Set PVPANIC_MMIO for x86 and PVPANIC_PCI for aarch64
From: Eric Auger on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1158#note_590150212 Yes. I meant this would deserve separate enablement and testing at libvirt level AND qemu level. I guess the pvpanic-pci device is not compiled either at qemu level for x86. Anyway I would be in favour of handling this in a separate BZ if we decide to support it. I can raise the point on virt-devel. Drew can you close the thread if you're OK. ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email to kernel-le...@lists.fedoraproject.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/kernel@lists.fedoraproject.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
Re: [OS-BUILD PATCHv2] redhat/configs: Set PVPANIC_MMIO for x86 and PVPANIC_PCI for aarch64
From: Eric Auger on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1158#note_590105917 At this stage my goal was just to fix the regression on x86 PVPANIC/ISA while adding the functionality on aarch64. If we decide to add PVPANIC-PCI on x86 this may handled through another BZ to also test it separately. Also libvirt wouldn't support it. ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email to kernel-le...@lists.fedoraproject.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/kernel@lists.fedoraproject.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
Re: [OS-BUILD PATCHv2] redhat/configs: Set PVPANIC_MMIO for x86 and PVPANIC_PCI for aarch64
From: Eric Auger on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1158#note_586344936 I have updated the MR to set MMIO and PCI configs to "m" ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email to kernel-le...@lists.fedoraproject.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/kernel@lists.fedoraproject.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[OS-BUILD PATCHv2] redhat/configs: Set PVPANIC_MMIO for x86 and PVPANIC_PCI for aarch64
From: Eric Auger redhat/configs: Set PVPANIC_MMIO for x86 and PVPANIC_PCI for aarch64 Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1961178 Upstream Status: RHEL only Tested: on x86 with libvirt, on ARM with qemu as pvpanic-pci is not yet integrated in libvirt With the introduction of pvpanic-pci new configs were introduced for the MMIO and PCI flavour. On x86, currently none is set so the ISA device is not available (this is a regression). The MMIO flavour must be used on x86_64. On ARM, the PCI one must be set. Signed-off-by: Eric Auger --- v1 -> v2: - s/y/m for PCI and MMIO configs diff --git a/redhat/configs/ark/generic/arm/aarch64/CONFIG_PVPANIC b/redhat/configs/ark/generic/arm/aarch64/CONFIG_PVPANIC new file mode 100644 index blahblah..blahblah 100644 --- /dev/null +++ b/redhat/configs/ark/generic/arm/aarch64/CONFIG_PVPANIC @@ -0,0 +1 @@ +CONFIG_PVPANIC=y diff --git a/redhat/configs/ark/generic/arm/aarch64/CONFIG_PVPANIC_MMIO b/redhat/configs/ark/generic/arm/aarch64/CONFIG_PVPANIC_MMIO new file mode 100644 index blahblah..blahblah 100644 --- /dev/null +++ b/redhat/configs/ark/generic/arm/aarch64/CONFIG_PVPANIC_MMIO @@ -0,0 +1 @@ +# CONFIG_PVPANIC_MMIO is not set diff --git a/redhat/configs/ark/generic/arm/aarch64/CONFIG_PVPANIC_PCI b/redhat/configs/ark/generic/arm/aarch64/CONFIG_PVPANIC_PCI new file mode 100644 index blahblah..blahblah 100644 --- /dev/null +++ b/redhat/configs/ark/generic/arm/aarch64/CONFIG_PVPANIC_PCI @@ -0,0 +1 @@ +CONFIG_PVPANIC_PCI=m diff --git a/redhat/configs/ark/generic/x86/x86_64/CONFIG_PVPANIC_MMIO b/redhat/configs/ark/generic/x86/x86_64/CONFIG_PVPANIC_MMIO new file mode 100644 index blahblah..blahblah 100644 --- /dev/null +++ b/redhat/configs/ark/generic/x86/x86_64/CONFIG_PVPANIC_MMIO @@ -0,0 +1 @@ +CONFIG_PVPANIC_MMIO=m diff --git a/redhat/configs/ark/generic/x86/x86_64/CONFIG_PVPANIC_PCI b/redhat/configs/ark/generic/x86/x86_64/CONFIG_PVPANIC_PCI new file mode 100644 index blahblah..blahblah 100644 --- /dev/null +++ b/redhat/configs/ark/generic/x86/x86_64/CONFIG_PVPANIC_PCI @@ -0,0 +1 @@ +# CONFIG_PVPANIC_PCI is not set -- https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1158 ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email to kernel-le...@lists.fedoraproject.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/kernel@lists.fedoraproject.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
Re: [OS-BUILD PATCH] redhat/configs: Set PVPANIC_MMIO for x86 and PVPANIC_PCI for aarch64
From: Eric Auger on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1158#note_585769516 actually I don't see the PVPANIC config being used in the makefile. Maybe it was kept just to avoid removing it? obj-$(CONFIG_PVPANIC_MMIO) += pvpanic.o pvpanic-mmio.o obj-$(CONFIG_PVPANIC_PCI) += pvpanic.o pvpanic-pci.o ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email to kernel-le...@lists.fedoraproject.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/kernel@lists.fedoraproject.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
Re: [OS-BUILD PATCH] redhat/configs: Set PVPANIC_MMIO for x86 and PVPANIC_PCI for aarch64
From: Eric Auger on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1158#note_585765138 As far as I understand, 6861d27cf590 ("misc/pvpanic: split-up generic and platform dependent code") split the legacy tristate PVPANIC_CONFIG into 2 configs: the bool PVPANIC and the tristate PVPANIC_MMIO. To get the same functionality as before you need to set both. Then pvpanic-pci was added by db3a4f0abefd ("misc/pvpanic: add PCI driver") which brought the new PVPANIC_PCI config. To use the ISA device I understand you need to set the MMIO config. By the way I now guess I should set "m" instead of "y" for the MMIO and PCI. ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email to kernel-le...@lists.fedoraproject.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/kernel@lists.fedoraproject.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[OS-BUILD PATCH] redhat/configs: Set PVPANIC_MMIO for x86 and PVPANIC_PCI for aarch64
From: Eric Auger redhat/configs: Set PVPANIC_MMIO for x86 and PVPANIC_PCI for aarch64 Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1961178 Upstream Status: RHEL only Tested: on x86 with libvirt, on ARM with qemu as pvpanic-pci is not yet integrated in libvirt With the introduction of pvpanic-pci new configs were introduced for the MMIO and PCI flavour. On x86, currently none is set so the ISA device is not available (this is a regression). The MMIO flavour must be used on x86_64. On ARM, the PCI one must be set. Signed-off-by: Eric Auger diff --git a/redhat/configs/ark/generic/arm/aarch64/CONFIG_PVPANIC b/redhat/configs/ark/generic/arm/aarch64/CONFIG_PVPANIC new file mode 100644 index blahblah..blahblah 100644 --- /dev/null +++ b/redhat/configs/ark/generic/arm/aarch64/CONFIG_PVPANIC @@ -0,0 +1 @@ +CONFIG_PVPANIC=y diff --git a/redhat/configs/ark/generic/arm/aarch64/CONFIG_PVPANIC_MMIO b/redhat/configs/ark/generic/arm/aarch64/CONFIG_PVPANIC_MMIO new file mode 100644 index blahblah..blahblah 100644 --- /dev/null +++ b/redhat/configs/ark/generic/arm/aarch64/CONFIG_PVPANIC_MMIO @@ -0,0 +1 @@ +# CONFIG_PVPANIC_MMIO is not set diff --git a/redhat/configs/ark/generic/arm/aarch64/CONFIG_PVPANIC_PCI b/redhat/configs/ark/generic/arm/aarch64/CONFIG_PVPANIC_PCI new file mode 100644 index blahblah..blahblah 100644 --- /dev/null +++ b/redhat/configs/ark/generic/arm/aarch64/CONFIG_PVPANIC_PCI @@ -0,0 +1 @@ +CONFIG_PVPANIC_PCI=y diff --git a/redhat/configs/ark/generic/x86/x86_64/CONFIG_PVPANIC_MMIO b/redhat/configs/ark/generic/x86/x86_64/CONFIG_PVPANIC_MMIO new file mode 100644 index blahblah..blahblah 100644 --- /dev/null +++ b/redhat/configs/ark/generic/x86/x86_64/CONFIG_PVPANIC_MMIO @@ -0,0 +1 @@ +CONFIG_PVPANIC_MMIO=y diff --git a/redhat/configs/ark/generic/x86/x86_64/CONFIG_PVPANIC_PCI b/redhat/configs/ark/generic/x86/x86_64/CONFIG_PVPANIC_PCI new file mode 100644 index blahblah..blahblah 100644 --- /dev/null +++ b/redhat/configs/ark/generic/x86/x86_64/CONFIG_PVPANIC_PCI @@ -0,0 +1 @@ +# CONFIG_PVPANIC_PCI is not set -- https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1158 ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email to kernel-le...@lists.fedoraproject.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/kernel@lists.fedoraproject.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure