Re: [libvirt] [PATCH] qemu: add PCI-multibus support for ppc
On 02/28/2014 10:10 AM, hong-hua@freescale.com wrote:
Hi Daniel,
Refer to libvirt/src/qemu/qemu_capabilities.c, only x86_64/i686 support
PCI_MULTIBUS.
/* Currently only x86_64 and i686 support PCI-multibus. */
if (qemuCaps-arch == VIR_ARCH_X86_64 ||
qemuCaps-arch == VIR_ARCH_I686) {
virQEMUCapsSet(qemuCaps, QEMU_CAPS_PCI_MULTIBUS);
libvirt/src/qemu/qemu_capabilities.h:
QEMU_CAPS_PCI_MULTIBUS = 52, /* bus=pci.0 vs bus=pci */
Exactly there're several PowerPC platforms also use pci.0 as PCI bus name.
$ grep -r pci\.0 qemu/hw/
qemu/hw/ppc/prep.c:pci_bus = (PCIBus *)qdev_get_child_bus(dev, pci.0);
qemu/hw/ppc/e500.c:pci_bus = (PCIBus *)qdev_get_child_bus(dev, pci.0);
qemu/hw/ppc/ppc440_bamboo.c:pcibus = (PCIBus *)qdev_get_child_bus(dev,
pci.0);
Looking at 'info qtree' output it seems pseries is the only PPC machine type
using pci (out of those I managed to run):
ppce500
bus: pci.0
mac99
bus: pci.0
g3beige
bus: pci.0
mpc8544ds
bus: pci.0
ref405ep
qemu-system-ppc64: Could not load PowerPC BIOS 'ppc405_rom.bin'
taihu
qemu-system-ppc64: Could not load PowerPC BIOS 'ppc405_rom.bin'
bamboo
bus: pci.0
prep
bus: pci.0
pseries
Warning: Disabling some instructions which are not emulated by TCG (0x0, 0x4)
bus: pci
virtex-ml507
There's no much platform specific code in libvirt.
How can we get the exact PCI bus name and make pci.0 work on the above
PowerPC platforms?
I think we need to check the def-os.machine string on PPC, since we only
probe a particular QEMU binary, not every machine type it supports.
Jan
signature.asc
Description: OpenPGP digital signature
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH] qemu: add PCI-multibus support for ppc
On Fri, Feb 28, 2014 at 09:10:47AM +, hong-hua@freescale.com wrote:
Hi Daniel,
Refer to libvirt/src/qemu/qemu_capabilities.c, only x86_64/i686 support
PCI_MULTIBUS.
/* Currently only x86_64 and i686 support PCI-multibus. */
if (qemuCaps-arch == VIR_ARCH_X86_64 ||
qemuCaps-arch == VIR_ARCH_I686) {
virQEMUCapsSet(qemuCaps, QEMU_CAPS_PCI_MULTIBUS);
libvirt/src/qemu/qemu_capabilities.h:
QEMU_CAPS_PCI_MULTIBUS = 52, /* bus=pci.0 vs bus=pci */
Exactly there're several PowerPC platforms also use pci.0 as PCI bus name.
$ grep -r pci\.0 qemu/hw/
qemu/hw/ppc/prep.c:pci_bus = (PCIBus *)qdev_get_child_bus(dev, pci.0);
qemu/hw/ppc/e500.c:pci_bus = (PCIBus *)qdev_get_child_bus(dev, pci.0);
qemu/hw/ppc/ppc440_bamboo.c:pcibus = (PCIBus *)qdev_get_child_bus(dev,
pci.0);
There's no much platform specific code in libvirt.
How can we get the exact PCI bus name and make pci.0 work on the above
PowerPC platforms?
As I said in my previous comments, if PPC is now using 'pci.0' as the
bus name, this is a change from previous QEMU behaviour. If you just
make libvirt use 'pci.0' for all PPC arches you will be breaking
compatibility with old QEMU versions. You must therefore do proper
version detection for this.
Regards,
Daniel
--
|: http://berrange.com -o-http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org -o- http://virt-manager.org :|
|: http://autobuild.org -o- http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH] qemu: add PCI-multibus support for ppc
Jan, thanks a lot for your comments.
Olivia
-Original Message-
From: Ján Tomko [mailto:jto...@redhat.com]
Sent: Tuesday, March 04, 2014 10:43 PM
To: Yin Olivia-R63875; Daniel P. Berrange
Cc: libvir-list@redhat.com
Subject: Re: [libvirt] [PATCH] qemu: add PCI-multibus support for ppc
On 02/28/2014 10:10 AM, hong-hua@freescale.com wrote:
Hi Daniel,
Refer to libvirt/src/qemu/qemu_capabilities.c, only x86_64/i686 support
PCI_MULTIBUS.
/* Currently only x86_64 and i686 support PCI-multibus. */
if (qemuCaps-arch == VIR_ARCH_X86_64 ||
qemuCaps-arch == VIR_ARCH_I686) {
virQEMUCapsSet(qemuCaps, QEMU_CAPS_PCI_MULTIBUS);
libvirt/src/qemu/qemu_capabilities.h:
QEMU_CAPS_PCI_MULTIBUS = 52, /* bus=pci.0 vs bus=pci */
Exactly there're several PowerPC platforms also use pci.0 as PCI bus name.
$ grep -r pci\.0 qemu/hw/
qemu/hw/ppc/prep.c:pci_bus = (PCIBus *)qdev_get_child_bus(dev,
pci.0);
qemu/hw/ppc/e500.c:pci_bus = (PCIBus *)qdev_get_child_bus(dev,
pci.0);
qemu/hw/ppc/ppc440_bamboo.c:pcibus = (PCIBus *)qdev_get_child_bus(dev,
pci.0);
Looking at 'info qtree' output it seems pseries is the only PPC machine
type using pci (out of those I managed to run):
ppce500
bus: pci.0
mac99
bus: pci.0
g3beige
bus: pci.0
mpc8544ds
bus: pci.0
ref405ep
qemu-system-ppc64: Could not load PowerPC BIOS 'ppc405_rom.bin'
taihu
qemu-system-ppc64: Could not load PowerPC BIOS 'ppc405_rom.bin'
bamboo
bus: pci.0
prep
bus: pci.0
pseries
Warning: Disabling some instructions which are not emulated by TCG (0x0,
0x4)
bus: pci
virtex-ml507
There's no much platform specific code in libvirt.
How can we get the exact PCI bus name and make pci.0 work on the above
PowerPC platforms?
I think we need to check the def-os.machine string on PPC, since we only
probe a particular QEMU binary, not every machine type it supports.
Jan
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH] qemu: add PCI-multibus support for ppc
Daniel,
I got it. We need do machine detection for the special PPC platforms.
Thank you for comments.
Olivia
-Original Message-
From: Daniel P. Berrange [mailto:berra...@redhat.com]
Sent: Tuesday, March 04, 2014 10:53 PM
To: Yin Olivia-R63875
Cc: libvir-list@redhat.com
Subject: Re: [PATCH] qemu: add PCI-multibus support for ppc
On Fri, Feb 28, 2014 at 09:10:47AM +, hong-hua@freescale.com wrote:
Hi Daniel,
Refer to libvirt/src/qemu/qemu_capabilities.c, only x86_64/i686 support
PCI_MULTIBUS.
/* Currently only x86_64 and i686 support PCI-multibus. */
if (qemuCaps-arch == VIR_ARCH_X86_64 ||
qemuCaps-arch == VIR_ARCH_I686) {
virQEMUCapsSet(qemuCaps, QEMU_CAPS_PCI_MULTIBUS);
libvirt/src/qemu/qemu_capabilities.h:
QEMU_CAPS_PCI_MULTIBUS = 52, /* bus=pci.0 vs bus=pci */
Exactly there're several PowerPC platforms also use pci.0 as PCI bus name.
$ grep -r pci\.0 qemu/hw/
qemu/hw/ppc/prep.c:pci_bus = (PCIBus *)qdev_get_child_bus(dev,
pci.0);
qemu/hw/ppc/e500.c:pci_bus = (PCIBus *)qdev_get_child_bus(dev,
pci.0);
qemu/hw/ppc/ppc440_bamboo.c:pcibus = (PCIBus *)qdev_get_child_bus(dev,
pci.0);
There's no much platform specific code in libvirt.
How can we get the exact PCI bus name and make pci.0 work on the above
PowerPC platforms?
As I said in my previous comments, if PPC is now using 'pci.0' as the bus
name, this is a change from previous QEMU behaviour. If you just make
libvirt use 'pci.0' for all PPC arches you will be breaking compatibility
with old QEMU versions. You must therefore do proper version detection for
this.
Regards,
Daniel
--
|: http://berrange.com -o-
http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org -o- http://virt-
manager.org :|
|: http://autobuild.org -o-
http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org -o- http://live.gnome.org/gtk-
vnc :|
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH] qemu: add PCI-multibus support for ppc
Hi Daniel,
Refer to libvirt/src/qemu/qemu_capabilities.c, only x86_64/i686 support
PCI_MULTIBUS.
/* Currently only x86_64 and i686 support PCI-multibus. */
if (qemuCaps-arch == VIR_ARCH_X86_64 ||
qemuCaps-arch == VIR_ARCH_I686) {
virQEMUCapsSet(qemuCaps, QEMU_CAPS_PCI_MULTIBUS);
libvirt/src/qemu/qemu_capabilities.h:
QEMU_CAPS_PCI_MULTIBUS = 52, /* bus=pci.0 vs bus=pci */
Exactly there're several PowerPC platforms also use pci.0 as PCI bus name.
$ grep -r pci\.0 qemu/hw/
qemu/hw/ppc/prep.c:pci_bus = (PCIBus *)qdev_get_child_bus(dev, pci.0);
qemu/hw/ppc/e500.c:pci_bus = (PCIBus *)qdev_get_child_bus(dev, pci.0);
qemu/hw/ppc/ppc440_bamboo.c:pcibus = (PCIBus *)qdev_get_child_bus(dev,
pci.0);
There's no much platform specific code in libvirt.
How can we get the exact PCI bus name and make pci.0 work on the above
PowerPC platforms?
Best Regards,
Olivia
-Original Message-
From: Daniel P. Berrange [mailto:berra...@redhat.com]
Sent: Wednesday, March 20, 2013 7:04 PM
To: Yin Olivia-R63875
Cc: libvir-list@redhat.com
Subject: Re: [PATCH] qemu: add PCI-multibus support for ppc
On Wed, Mar 20, 2013 at 02:11:51AM +, Yin Olivia-R63875 wrote:
Hi Daniel,
PPC also supports PCI multibus. Could you please help review this patch?
IIUC, PPC did not support PCI multibus in the past, so when did this
change ? Libvirt needs to only use this for versions where it is supported,
so unconditionally enabling it for all PPC is not right.
-Original Message-
From: Yin Olivia-R63875
Sent: Wednesday, March 13, 2013 12:44 PM
To: libvir-list@redhat.com
Cc: Yin Olivia-R63875
Subject: [PATCH] qemu: add PCI-multibus support for ppc
Signed-off-by: Olivia Yin hong-hua@freescale.com
---
src/qemu/qemu_capabilities.c | 10 ++
1 files changed, 10 insertions(+), 0 deletions(-)
diff --git a/src/qemu/qemu_capabilities.c
b/src/qemu/qemu_capabilities.c index 7bc1ebc..7d7791d 100644
--- a/src/qemu/qemu_capabilities.c
+++ b/src/qemu/qemu_capabilities.c
@@ -2209,6 +2209,11 @@ virQEMUCapsInitHelp(virQEMUCapsPtr qemuCaps,
uid_t runUid, gid_t runGid)
virQEMUCapsClear(qemuCaps, QEMU_CAPS_NO_ACPI);
}
+/* ppc support PCI-multibus */
+if (qemuCaps-arch == VIR_ARCH_PPC) {
+virQEMUCapsSet(qemuCaps, QEMU_CAPS_PCI_MULTIBUS);
+}
+
/* virQEMUCapsExtractDeviceStr will only set additional caps if
qemu
* understands the 0.13.0+ notion of -device driver,. */
if (virQEMUCapsGet(qemuCaps, QEMU_CAPS_DEVICE) @@ -2450,6
+2455,11 @@ virQEMUCapsInitQMP(virQEMUCapsPtr qemuCaps,
virQEMUCapsSet(qemuCaps, QEMU_CAPS_NO_ACPI);
}
+/* ppc support PCI-multibus */
+if (qemuCaps-arch == VIR_ARCH_PPC) {
+virQEMUCapsSet(qemuCaps, QEMU_CAPS_PCI_MULTIBUS);
+}
+
if (virQEMUCapsProbeQMPCommands(qemuCaps, mon) 0)
goto cleanup;
if (virQEMUCapsProbeQMPEvents(qemuCaps, mon) 0)
--
Daniel
--
|: http://berrange.com -o-
http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org -o- http://virt-
manager.org :|
|: http://autobuild.org -o-
http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org -o- http://live.gnome.org/gtk-
vnc :|
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH] qemu: add PCI-multibus support for ppc
On Wed, Mar 20, 2013 at 02:11:51AM +, Yin Olivia-R63875 wrote:
Hi Daniel,
PPC also supports PCI multibus. Could you please help review this patch?
IIUC, PPC did not support PCI multibus in the past, so when did
this change ? Libvirt needs to only use this for versions where
it is supported, so unconditionally enabling it for all PPC is
not right.
-Original Message-
From: Yin Olivia-R63875
Sent: Wednesday, March 13, 2013 12:44 PM
To: libvir-list@redhat.com
Cc: Yin Olivia-R63875
Subject: [PATCH] qemu: add PCI-multibus support for ppc
Signed-off-by: Olivia Yin hong-hua@freescale.com
---
src/qemu/qemu_capabilities.c | 10 ++
1 files changed, 10 insertions(+), 0 deletions(-)
diff --git a/src/qemu/qemu_capabilities.c b/src/qemu/qemu_capabilities.c
index 7bc1ebc..7d7791d 100644
--- a/src/qemu/qemu_capabilities.c
+++ b/src/qemu/qemu_capabilities.c
@@ -2209,6 +2209,11 @@ virQEMUCapsInitHelp(virQEMUCapsPtr qemuCaps, uid_t
runUid, gid_t runGid)
virQEMUCapsClear(qemuCaps, QEMU_CAPS_NO_ACPI);
}
+/* ppc support PCI-multibus */
+if (qemuCaps-arch == VIR_ARCH_PPC) {
+virQEMUCapsSet(qemuCaps, QEMU_CAPS_PCI_MULTIBUS);
+}
+
/* virQEMUCapsExtractDeviceStr will only set additional caps if qemu
* understands the 0.13.0+ notion of -device driver,. */
if (virQEMUCapsGet(qemuCaps, QEMU_CAPS_DEVICE) @@ -2450,6 +2455,11
@@ virQEMUCapsInitQMP(virQEMUCapsPtr qemuCaps,
virQEMUCapsSet(qemuCaps, QEMU_CAPS_NO_ACPI);
}
+/* ppc support PCI-multibus */
+if (qemuCaps-arch == VIR_ARCH_PPC) {
+virQEMUCapsSet(qemuCaps, QEMU_CAPS_PCI_MULTIBUS);
+}
+
if (virQEMUCapsProbeQMPCommands(qemuCaps, mon) 0)
goto cleanup;
if (virQEMUCapsProbeQMPEvents(qemuCaps, mon) 0)
--
Daniel
--
|: http://berrange.com -o-http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org -o- http://virt-manager.org :|
|: http://autobuild.org -o- http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH] qemu: add PCI-multibus support for ppc
Hi Daniel,
PPC also supports PCI multibus. Could you please help review this patch?
Best Regards,
Olivia
-Original Message-
From: Yin Olivia-R63875
Sent: Wednesday, March 13, 2013 12:44 PM
To: libvir-list@redhat.com
Cc: Yin Olivia-R63875
Subject: [PATCH] qemu: add PCI-multibus support for ppc
Signed-off-by: Olivia Yin hong-hua@freescale.com
---
src/qemu/qemu_capabilities.c | 10 ++
1 files changed, 10 insertions(+), 0 deletions(-)
diff --git a/src/qemu/qemu_capabilities.c b/src/qemu/qemu_capabilities.c
index 7bc1ebc..7d7791d 100644
--- a/src/qemu/qemu_capabilities.c
+++ b/src/qemu/qemu_capabilities.c
@@ -2209,6 +2209,11 @@ virQEMUCapsInitHelp(virQEMUCapsPtr qemuCaps, uid_t
runUid, gid_t runGid)
virQEMUCapsClear(qemuCaps, QEMU_CAPS_NO_ACPI);
}
+/* ppc support PCI-multibus */
+if (qemuCaps-arch == VIR_ARCH_PPC) {
+virQEMUCapsSet(qemuCaps, QEMU_CAPS_PCI_MULTIBUS);
+}
+
/* virQEMUCapsExtractDeviceStr will only set additional caps if qemu
* understands the 0.13.0+ notion of -device driver,. */
if (virQEMUCapsGet(qemuCaps, QEMU_CAPS_DEVICE) @@ -2450,6 +2455,11
@@ virQEMUCapsInitQMP(virQEMUCapsPtr qemuCaps,
virQEMUCapsSet(qemuCaps, QEMU_CAPS_NO_ACPI);
}
+/* ppc support PCI-multibus */
+if (qemuCaps-arch == VIR_ARCH_PPC) {
+virQEMUCapsSet(qemuCaps, QEMU_CAPS_PCI_MULTIBUS);
+}
+
if (virQEMUCapsProbeQMPCommands(qemuCaps, mon) 0)
goto cleanup;
if (virQEMUCapsProbeQMPEvents(qemuCaps, mon) 0)
--
1.6.4
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
[libvirt] [PATCH] qemu: add PCI-multibus support for ppc
Signed-off-by: Olivia Yin hong-hua@freescale.com
---
src/qemu/qemu_capabilities.c | 10 ++
1 files changed, 10 insertions(+), 0 deletions(-)
diff --git a/src/qemu/qemu_capabilities.c b/src/qemu/qemu_capabilities.c
index 7bc1ebc..7d7791d 100644
--- a/src/qemu/qemu_capabilities.c
+++ b/src/qemu/qemu_capabilities.c
@@ -2209,6 +2209,11 @@ virQEMUCapsInitHelp(virQEMUCapsPtr qemuCaps, uid_t
runUid, gid_t runGid)
virQEMUCapsClear(qemuCaps, QEMU_CAPS_NO_ACPI);
}
+/* ppc support PCI-multibus */
+if (qemuCaps-arch == VIR_ARCH_PPC) {
+virQEMUCapsSet(qemuCaps, QEMU_CAPS_PCI_MULTIBUS);
+}
+
/* virQEMUCapsExtractDeviceStr will only set additional caps if qemu
* understands the 0.13.0+ notion of -device driver,. */
if (virQEMUCapsGet(qemuCaps, QEMU_CAPS_DEVICE)
@@ -2450,6 +2455,11 @@ virQEMUCapsInitQMP(virQEMUCapsPtr qemuCaps,
virQEMUCapsSet(qemuCaps, QEMU_CAPS_NO_ACPI);
}
+/* ppc support PCI-multibus */
+if (qemuCaps-arch == VIR_ARCH_PPC) {
+virQEMUCapsSet(qemuCaps, QEMU_CAPS_PCI_MULTIBUS);
+}
+
if (virQEMUCapsProbeQMPCommands(qemuCaps, mon) 0)
goto cleanup;
if (virQEMUCapsProbeQMPEvents(qemuCaps, mon) 0)
--
1.6.4
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
