Re: [Qemu-devel] [PATCH v6 1/8] atomic: introduce atomic operations

2012-11-14 Thread liu ping fan
On Wed, Nov 14, 2012 at 5:47 PM, Paolo Bonzini wrote: > Il 14/11/2012 10:38, liu ping fan ha scritto: >> On Tue, Nov 13, 2012 at 6:11 PM, Paolo Bonzini wrote: > Il 05/11/2012 06:38, Liu Ping Fan ha scritto: >> From: Liu Ping Fan >> >> If out of global lock, we will be challenged

[Qemu-devel] [PATCH] Fix the inconsistency in x509-dh-key-file parameter

2012-11-14 Thread Lei Li
Fix the inconsistency between the declaration of the option 'x509-dh-key-file' to -spice command line and its parameter parsing. https://bugs.launchpad.net/qemu/+bug/1035042 Reported-by: Alon Bar-Lev Signed-off-by: Lei Li --- ui/spice-core.c |2 +- 1 files changed, 1 insertions(+), 1 delet

[Qemu-devel] [PATCH] sd: Send debug printfery to stderr not stdout

2012-11-14 Thread Peter Crosthwaite
Some debug printfs for SD are coming up in stdout. Redirected them to stderr instead. Signed-off-by: Peter Crosthwaite --- hw/sd.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/hw/sd.c b/hw/sd.c index 3c34d43..607edba 100644 --- a/hw/sd.c +++ b/hw/sd.c @@ -1439,8 +1

Re: [Qemu-devel] [PATCH] target-mips: Clean up microMIPS32 major opcode

2012-11-14 Thread Wei-Ren Chen
On Thu, Nov 15, 2012 at 04:01:32AM +, Johnson, Eric wrote: > Hi Chen, > > Sorry I must have made a copy paste error. I access the documents > internally. I'll double check the link tomorrow. > > The document I referenced is the MIPS64 not the microMIPS64. > > Do not change the names. The L

Re: [Qemu-devel] [RFC PATCH v4 0/3] reload kernel/initrd/elf images when reset

2012-11-14 Thread Yin Olivia-R63875
I test loading the same images on X86/PowerPC platforms. Size of image files: 3432KuImage 80948K vmlinux 25180K ramdisk Test 1) X86 $ qemu-system-ppc -M mpc8544ds -cpu MPC8544 -kernel vmlinux -initrd ramdisk -m 256M -nographic $ qemu-system-ppc -M mpc8544ds -cpu MPC8544 -kernel uImage -

Re: [Qemu-devel] [RFC] libqblock OOM issue

2012-11-14 Thread Wenchao Xia
于 2012-11-14 18:32, Paolo Bonzini 写道: Il 14/11/2012 10:55, Wenchao Xia ha scritto: In order to resolve OOM issue, I am trying wrap all APIs using sunrpc, need some suggestion before coding. Is the client/server approach really necessary or can you write a library that invokes qemu-nbd/qemu

Re: [Qemu-devel] [PATCH] target-mips: Clean up microMIPS32 major opcode

2012-11-14 Thread Johnson, Eric
Hi Chen, Sorry I must have made a copy paste error. I access the documents internally. I'll double check the link tomorrow. The document I referenced is the MIPS64 not the microMIPS64. Do not change the names. The LD32 and SD32 are microMIPS specific. The assembler LD and SD opcodes work for

[Qemu-devel] [PATCH 02/11] Move qemu_irq typedef out of qemu-common.h

2012-11-14 Thread Andreas Färber
From: Igor Mammedov It's necessary for making CPU child of DEVICE without causing circular header deps. Signed-off-by: Igor Mammedov [ehabkost: re-added the typedef to hw/irq.h after rebasing] Signed-off-by: Eduardo Habkost Signed-off-by: Andreas Färber --- hw/arm-misc.h |1 + hw/bt.h

[Qemu-devel] [PATCH 11/11] target-i386: Add Haswell CPU model

2012-11-14 Thread Andreas Färber
From: Eduardo Habkost Features added to the model, in relation to SandyBridge: fma CPUID[1].ECX[12] pcid CPUID[1].ECX[17] movbe CPUID[1].ECX[22] fsgsbase CPUID[EAX=7,ECX=0].EBX[0] bmi1 CPUID[EAX=7,ECX=0].EBX[3] hle CPUID[EAX=7,ECX=0].EBX[4] avx2 CPUID[EAX=7,ECX=0]

[Qemu-devel] [PATCH 01/11] qemu-common.h: Comment about usage rules

2012-11-14 Thread Andreas Färber
From: Eduardo Habkost Every time we make a tiny change on a header file, we often find circular header dependency problems. To avoid this nightmare, we need to stop including qemu-common.h from other headers, and we should gradually move the declarations from the catch-all qemu-common.h header to

[Qemu-devel] [PATCH 07/11] osdep: Move qemu_{open, close}() prototypes

2012-11-14 Thread Andreas Färber
They are implemented in osdep.c, so keep the prototypes in osdep.h. Suggested-by: Igor Mammedov Signed-off-by: Andreas Färber Acked-by: Eduardo Habkost --- osdep.h |3 +++ qemu-common.h |2 -- 2 Dateien geändert, 3 Zeilen hinzugefügt(+), 2 Zeilen entfernt(-) diff --git a/osdep.h

Re: [Qemu-devel] [PATCH 0/3] Opteron_G5 and Haswell CPU models

2012-11-14 Thread Andreas Färber
Am 14.11.2012 21:29, schrieb Eduardo Habkost: > Andreas, I just noticed that I didn't CC you in this series, and I don't > know if this is something that would go through your tree. Should I CC > you on x86-specific series unrelated to QOM, like this one? If I get cc'ed or notice and they're trivi

Re: [Qemu-devel] [PATCH] target-mips: Clean up microMIPS32 major opcode

2012-11-14 Thread Wei-Ren Chen
On Thu, Nov 15, 2012 at 02:34:31AM +, Johnson, Eric wrote: > Hi Chen, > > Please only remove the POOL48A opcode. > > The others are documented in the microMIPS64 Instruction Set manual ( > http://www.mips.com/secure-download/index.dot?product_name=/auth/MD00087-2B-MIPS64BIS-AFP-03.51.pdf >

[Qemu-devel] [PATCH 10/11] target-i386/cpu: Add new Opteron CPU model

2012-11-14 Thread Andreas Färber
From: Andre Przywara Add a new base CPU model called Opteron_G5 to model the latest Opteron CPUs. This increases the model value and model numbers and adds TBM, F16C and FMA over the latest G4 model. Signed-off-by: Andre Przywara Signed-off-by: Boris Ostrovsky [ehabkost: edited commit message]

[Qemu-devel] [PATCH 09/11] target-i386/cpu: Name new CPUID bits

2012-11-14 Thread Andreas Färber
From: Andre Przywara Update QEMU's knowledge of CPUID bit names. This allows to enable/disable those new features on QEMU's command line when using KVM and prepares future feature enablement in QEMU. This adds F16C, RDRAND, LWP, TBM, TopoExt, PerfCtr_Core, PerfCtr_NB, FSGSBASE, BMI1, AVX2, BMI2,

[Qemu-devel] [PATCH 08/11] qapi-types.h: Don't include qemu-common.h

2012-11-14 Thread Andreas Färber
From: Igor Mammedov Needed to prevent build breakage when CPUState becomes a child of DeviceState. Signed-off-by: Igor Mammedov [ehabkost: include too] Signed-off-by: Eduardo Habkost Signed-off-by: Andreas Färber --- scripts/qapi-types.py |3 ++- 1 Datei geändert, 2 Zeilen hinzugefügt(+

[Qemu-devel] [PATCH 05/11] vnc-palette.h: Include

2012-11-14 Thread Andreas Färber
From: Eduardo Habkost is needed for the 'bool' type, used in the header. The header is probably being included by accident because some other headers are including qemu-common.h, but those headers should eventually stop including qemu-common.h. Signed-off-by: Eduardo Habkost Signed-off-by: An

[Qemu-devel] [PATCH 03/11] qdev: Split up header so it can be used in cpu.h

2012-11-14 Thread Andreas Färber
From: Anthony Liguori Header file dependency is a frickin' nightmare right now. cpu.h tends to get included in our 'include everything' header files but qdev also needs to include those headers mainly for qdev-properties since it knows about CharDriverState and friends. We can solve this for no

[Qemu-devel] [PATCH 06/11] qemu-config.h: Include headers it needs

2012-11-14 Thread Andreas Färber
From: Eduardo Habkost Include: - for FILE - qemu-option.h for QemuOptsList Some of those headers were probably being included by accident because some other headers were including qemu-common.h, but those headers should eventually stop including qemu-common.h. Signed-off-by: Eduardo Habkost S

[Qemu-devel] [PATCH 04/11] qemu-fsdev-dummy.c: Include module.h

2012-11-14 Thread Andreas Färber
From: Eduardo Habkost module.h is where machine_init() is defined, but qemu-fsdev-dummy.c doesn't include it. The header is probably being included by accident because some other headers are including qemu-common.h, but those headers should eventually stop including qemu-common.h. Signed-off-by

[Qemu-devel] [PULL for-1.3] QOM CPUState patch queue 2012-11-15

2012-11-14 Thread Andreas Färber
Hello Anthony, Here's some steps towards CPU-as-a-device and two new x86 CPU models. Please pull. I decided to take only a conservative patch selection at this late hour: Getting the qdev split in avoids having to rebase and review this over and over. While the CPU as a device would have been he

Re: [Qemu-devel] [PATCH] target-mips: Clean up microMIPS32 major opcode

2012-11-14 Thread Johnson, Eric
Hi Chen, Please only remove the POOL48A opcode. The others are documented in the microMIPS64 Instruction Set manual ( http://www.mips.com/secure-download/index.dot?product_name=/auth/MD00087-2B-MIPS64BIS-AFP-03.51.pdf ). See http://www.mips.com/products/architectures/mips64/ for other relavent

[Qemu-devel] [PATCH] target-mips: Clean up microMIPS32 major opcode

2012-11-14 Thread Wei-Ren Chen
Hi all, I check MIPS microMIPS manual [1], and found the major opcode might be wrong. I add a comment to explicitly indicate what manual I am refering to, and according that manual I remove some microMIPS32 major opcodes. Major opcode 0x1f is reserved, so I just remove it. As for others, like 0x

Re: [Qemu-devel] [PATCH 3/3] target-i386: add Haswell CPU model

2012-11-14 Thread Igor Mammedov
On Wed, 14 Nov 2012 16:28:54 -0200 Eduardo Habkost wrote: > Features added to the model, in relation to SandyBridge: > > fma CPUID[1].ECX[12] > pcid CPUID[1].ECX[17] > movbe CPUID[1].ECX[22] > fsgsbase CPUID[EAX=7,ECX=0].EBX[0] > bmi1 CPUID[EAX=7,ECX=0].EBX[3] > hle CP

Re: [Qemu-devel] [PATCH 2/3] target-i386: cpu: add new Opteron CPU model

2012-11-14 Thread Igor Mammedov
On Wed, 14 Nov 2012 16:28:53 -0200 Eduardo Habkost wrote: > From: Andre Przywara > > Add a new base CPU model called Opteron_G5 to model the latest > Opteron CPUs. This increases the model value and model numbers and > adds TBM, F16C and FMA over the latest G4 model. > > Signed-off-by: Andre P

Re: [Qemu-devel] [PATCH 1/3] target-i386: cpu: name new CPUID bits

2012-11-14 Thread Igor Mammedov
On Wed, 14 Nov 2012 16:28:52 -0200 Eduardo Habkost wrote: > From: Andre Przywara > > Update QEMU's knowledge of CPUID bit names. This allows to > enable/disable those new features on QEMU's command line when > using KVM and prepares future feature enablement in QEMU. > > This adds F16C, RDRAND

[Qemu-devel] [PATCH] virtio-blk: Remove duplicate property definition

2012-11-14 Thread David Gibson
For the virtio-blk device (via virtio-pci) the property "config-wce" is defined in two places. First, it's defined from the DEFINE_VIRTIO_BLK_FEATURES macro, second it's defined directly in virtio-pci, just two lines above the call to that macro. The direct definition in virtio-pci.c is broken, s

Re: [Qemu-devel] Adding another debug protocol

2012-11-14 Thread Peter Cheung
> Date: Wed, 14 Nov 2012 17:43:12 +0100 > From: jan.kis...@siemens.com > To: mcheun...@hotmail.com > CC: peter.mayd...@linaro.org; qemu-devel@nongnu.org > Subject: Re: [Qemu-devel] Adding another debug protocol > > On 2012-11-14 17:28, Peter Cheung wrote: > > hi Jan, you are the maintainer of the

Re: [Qemu-devel] [PATCH 19/24] qdev: move reset handler list from vl.c to qdev.c

2012-11-14 Thread Andreas Färber
Am 09.11.2012 15:56, schrieb Eduardo Habkost: > The core qdev code uses the reset handler list from vl.c, so move > qemu_register_reset(), qemu_unregister_reset() and qemu_devices_reset() > to qdev.c. > > The function declarations were moved to a new qdev-reset.h file, that is > included by hw.h t

[Qemu-devel] [PATCH 1/2] acpi_piix4: fix migration of gpe fields

2012-11-14 Thread Marcelo Tosatti
Migrate 16 bytes for en/sts fields (which is the correct size), increase version to 3, and document how to support incoming migration from qemu-kvm 1.2. Acked-by: Paolo Bonzini Signed-off-by: Marcelo Tosatti --- hw/acpi_piix4.c | 50 ++ 1 files

[Qemu-devel] [PATCH 2/2] kvm: Actually remove software breakpoints from list on cleanup

2012-11-14 Thread Marcelo Tosatti
From: Jan Kiszka So far we only removed them from the guest, leaving its states in the list. This made it impossible for gdb to re-enable breakpoints on the same address after re-attaching. Signed-off-by: Jan Kiszka Signed-off-by: Marcelo Tosatti --- kvm-all.c |2 ++ 1 files changed, 2 in

[Qemu-devel] [PATCH v2] qga/channel-posix.c: include headers it needs

2012-11-14 Thread Igor Mammedov
From: Eduardo Habkost Include: - for errno - & for fcntl() - "qemu-stdio.h" for qemu_open() Some of those headers were probably being included by accident because some other headers were including qemu-common.h, but those headers should eventually stop including qemu-common.h. Signed-off-

[Qemu-devel] [PATCH 0/2] [PULL] qemu-kvm.git uq/master queue

2012-11-14 Thread Marcelo Tosatti
The following changes since commit ce34cf72fe508b27a78f83c184142e8d1e6a048a: Merge remote-tracking branch 'awilliam/tags/vfio-pci-for-qemu-1.3.0-rc0' into staging (2012-11-14 08:53:40 -0600) are available in the git repository at: git://git.kernel.org/pub/scm/virt/kvm/qemu-kvm.git uq/master

Re: [Qemu-devel] [PATCH v5 3/3] convert pci-host to QOM

2012-11-14 Thread Andreas Färber
Am 08.11.2012 06:36, schrieb Wanpeng Li: > Take advantage of Andreas's pci-host patchset, add instance_init function > to fully implement convert pci-host to QOM. > > Signed-off-by: Anthony Liguori > Signed-off-by: Wanpeng Li > --- > hw/pci_host.c |9 + > 1 files changed, 9 insertio

Re: [Qemu-devel] [PATCH v5 0/3] refactor PC machine, i440fx and piix3 to take advantage of QOM

2012-11-14 Thread Wanpeng Li
On Thu, Nov 08, 2012 at 01:36:09PM +0800, Wanpeng Li wrote: >This series aggressively refactors the PC machine initialization to be more >modelled and less ad-hoc. The highlights of this series are: > >1) Things like -m and -bios-name are now device model properties > >2) The i440fx and piix3 are

Re: [Qemu-devel] [PATCH] KVM: MMU: lazily drop large spte

2012-11-14 Thread Xiao Guangrong
On 11/14/2012 10:44 PM, Marcelo Tosatti wrote: > On Wed, Nov 14, 2012 at 12:33:50AM +0900, Takuya Yoshikawa wrote: >> Ccing live migration developers who should be interested in this work, >> >> On Mon, 12 Nov 2012 21:10:32 -0200 >> Marcelo Tosatti wrote: >> >>> On Mon, Nov 05, 2012 at 05:59:26PM

[Qemu-devel] [PATCH] tests: Add tests for fdsets

2012-11-14 Thread Corey Bryant
Signed-off-by: Corey Bryant --- tests/qemu-iotests/044| 129 ++ tests/qemu-iotests/044.out| 5 ++ tests/qemu-iotests/group | 1 + tests/qemu-iotests/iotests.py | 12 4 files changed, 147 insertions(+) create mode 100755 tests/qe

Re: [Qemu-devel] [PATCH 15/24] qlist.h: do not include qemu-common.h

2012-11-14 Thread Andreas Färber
Am 09.11.2012 15:56, schrieb Eduardo Habkost: > I don't know why it was including it, as I don't see any code that > depends on anything from qemu-common.h. > > Signed-off-by: Eduardo Habkost > --- > qlist.h | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/qlist.h b/qlist.h > index ae776

[Qemu-devel] [Bug 1078892] [NEW] qemu doesn't general protection fault if there are reserved bits set in page-directory-pointer table entries

2012-11-14 Thread Alexander Chernyakhovsky
Public bug reported: While working on implementing 32-bit PAE mode in a custom operating system, which I was testing in QEMU, I noticed that my OS worked correctly, but resulted in a general protection fault when booted on VMware, VirtualBox, or bochs. According to the Intel Architecture Manual,

[Qemu-devel] Question on Xen management of qemu ram blocks and memory regions

2012-11-14 Thread Andres Lagar-Cavilla
Stefano, and Xen-qemu team, I have a question. The standard Xen-qemu workflow has Xen manage the physmap for a VM, and allocate all the backing memory for valid pfns, regardless of whether they are MMIO, RAM, etc. On save/migrate, when using upstream qemu, a special monitor command is used to s

Re: [Qemu-devel] [PATCH 14/24] sysemu.h: include qemu-types.h instead of qemu-common.h

2012-11-14 Thread Andreas Färber
Am 14.11.2012 22:56, schrieb Andreas Färber: > Am 09.11.2012 15:56, schrieb Eduardo Habkost: >> It just needs the Monitor and DeviceState typedefs, so it doesn't need >> all of qemu-common.h. >> >> Signed-off-by: Eduardo Habkost >> --- >> sysemu.h | 2 +- >> 1 file changed, 1 insertion(+), 1 dele

Re: [Qemu-devel] [PATCH 16/24] qapi-types.h: don't include qemu-common.h

2012-11-14 Thread Andreas Färber
Am 09.11.2012 15:56, schrieb Eduardo Habkost: > From: Igor Mammedov > > needed to prevent build breakage when CPU becomes a child of DeviceState > > Signed-off-by: Igor Mammedov > [ehabkost: include too] > Signed-off-by: Eduardo Habkost Acked-by: Andreas Färber Andreas -- SUSE LINUX Prod

[Qemu-devel] [PATCH v4 12/14] q35: Add kvmclock support

2012-11-14 Thread Jason Baron
From: Jan Kiszka Reviewed-by: Paolo Bonzini Signed-off-by: Jan Kiszka Signed-off-by: Jason Baron --- hw/pc_q35.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/hw/pc_q35.c b/hw/pc_q35.c index 142bf8a..3429a9a 100644 --- a/hw/pc_q35.c +++ b/hw/pc_q35.c @@ -34,6 +34,

[Qemu-devel] [PATCH v4 04/14] pc/piix_pci: factor out smram/pam logic

2012-11-14 Thread Jason Baron
From: Isaku Yamahata Factor out smram/pam logic for use by other chipsets, namely q35 at this point. Note: Should be factored out into a generic North Bridge Class. [jba...@redhat.com: changes for updated memory API] Signed-off-by: Isaku Yamahata Signed-off-by: Jason Baron --- hw/Makefile.ob

Re: [Qemu-devel] [PATCH 14/24] sysemu.h: include qemu-types.h instead of qemu-common.h

2012-11-14 Thread Andreas Färber
Am 09.11.2012 15:56, schrieb Eduardo Habkost: > It just needs the Monitor and DeviceState typedefs, so it doesn't need > all of qemu-common.h. > > Signed-off-by: Eduardo Habkost > --- > sysemu.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/sysemu.h b/sysemu.h > index

Re: [Qemu-devel] [PATCH 13/24] create qemu-types.h for struct typedefs

2012-11-14 Thread Andreas Färber
Am 09.11.2012 15:56, schrieb Eduardo Habkost: > Instead of keeping all those struct typedefs on qemu-common.h, move it > to a header that can be safely included by other headers, containing > only the struct typedefs and not pulling other dependencies. > > Also, move some of the qdev-core.h typede

[Qemu-devel] [PATCH v4 09/14] ich9: Add i82801b11 dmi-to-pci bridge

2012-11-14 Thread Jason Baron
From: Jason Baron Add the dmi-to-pci i82801b11 bridge chip. This is the pci bridge chip that q35 uses on its host bus for PCI bus arbitration. Signed-off-by: Isaku Yamahata Signed-off-by: Jason Baron --- hw/Makefile.objs |1 + hw/i82801b11.c | 125 ++

[Qemu-devel] [PATCH v4 10/14] q35: Suppress SMM BIOS initialization under KVM

2012-11-14 Thread Jason Baron
From: Jan Kiszka Same as for i44fx: KVM does not support SMM yet. Signal it initialized to Seabios to avoid failures. Reviewed-by: Paolo Bonzini Signed-off-by: Jan Kiszka Signed-off-by: Jason Baron --- hw/acpi_ich9.c |7 +++ 1 files changed, 7 insertions(+), 0 deletions(-) diff --gi

[Qemu-devel] [PATCH v4 05/14] ich9: Add acpi support and definitions

2012-11-14 Thread Jason Baron
From: Jason Baron Lay the groundwork for subsequent ich9 support. Signed-off-by: Isaku Yamahata Signed-off-by: Jason Baron --- hw/Makefile.objs |1 + hw/acpi_ich9.c | 315 ++ hw/acpi_ich9.h | 47 hw/ich9.h| 207 +

[Qemu-devel] [PATCH v4 08/14] q35: Introduce q35 pc based chipset emulator

2012-11-14 Thread Jason Baron
From: Isaku Yamahata pc q35 based chipset emulator to support pci express natively. Based on Anthony Liguori's suggestion, the machine name is 'q35-next', with an alias of 'q35'. At this point, there are no compatibility guarantees. When the chipset stabilizes more, we will begin to version the m

[Qemu-devel] [PATCH v4 13/14] Add a fallback bios file search, if -L fails.

2012-11-14 Thread Jason Baron
From: Jason Baron If -L is specified, and qemu does not find the bios file in , then the search fails. Add infrastructure such that the search will continue in the default paths, if not found in the -L path. Reviewed-by: Paolo Bonzini Signed-off-by: Jason Baron --- vl.c | 36 ++

[Qemu-devel] [PATCH v4 14/14] q35: automatically load the q35 dsdt table

2012-11-14 Thread Jason Baron
From: Jason Baron Automatically, locate the required q35 dsdt table on load. Otherwise we error out. This could be done in the bios, but its harder to produce a good error message. Signed-off-by: Jason Baron --- hw/pc.c | 19 +++ hw/pc.h |2 ++ hw/pc_q35.c |7

[Qemu-devel] [PATCH v4 02/14] pc: Move ioapic_init() from pc_piix.c to pc.c

2012-11-14 Thread Jason Baron
From: Jason Baron Move ioapic_init() from pc_piix.c to pc.c, to make it a common function. Rename ioapic_init() -> ioapic_init_gsi(). Move to pc.h so q35 can use them as well. Reviewed-by: Paolo Bonzini Signed-off-by: Jason Baron --- hw/pc.c | 24 hw/pc.h

[Qemu-devel] [PATCH v4 07/14] ich9: Add smbus

2012-11-14 Thread Jason Baron
From: Jason Baron Add support for the ich9 smbus chip. Signed-off-by: Isaku Yamahata Signed-off-by: Jason Baron --- hw/Makefile.objs |2 +- hw/smbus_ich9.c | 159 ++ 2 files changed, 160 insertions(+), 1 deletions(-) create mode 10064

[Qemu-devel] [PATCH v4 03/14] pc_piix: Move kvm irq routing functions out of pc_piix.c

2012-11-14 Thread Jason Baron
From: Jason Baron Rename: kvm_piix3_gsi_handlei() -> kvm_pc_gsi_handler() kvm_piix3_setup_irq_routing() -> kvm_pc_setup_irq_routing() This is in preparation for other users, namely q35 at this time. Signed-off-by: Jason Baron --- hw/kvm/ioapic.c | 40

[Qemu-devel] [PATCH v4 01/14] pc, pc_piix: split out pc nic initialization

2012-11-14 Thread Jason Baron
From: Isaku Yamahata Factor out pc nic initialization. This simplifies the pc initialization and will reduce the code duplication of q35 pc initialization. Reviewed-by: Paolo Bonzini Reviewed-by: Anthony Liguori Signed-off-by: Isaku Yamahata Signed-off-by: Jason Baron --- hw/pc.c | 1

[Qemu-devel] [PATCH v4 11/14] q35: Fix non-PCI IRQ processing in ich9_lpc_update_apic

2012-11-14 Thread Jason Baron
From: Jan Kiszka Avoid passing a non-PCI IRQ to ich9_gsi_to_pirq. It's wrong and triggers an assertion. Reviewed-by: Paolo Bonzini Signed-off-by: Jan Kiszka Signed-off-by: Jason Baron --- hw/lpc_ich9.c |6 -- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/hw/lpc_ich9.

[Qemu-devel] [PATCH v4 06/14] ich9: Add the lpc chip

2012-11-14 Thread Jason Baron
From: Jason Baron Add support for the ICH9 LPC chip. Signed-off-by: Isaku Yamahata Signed-off-by: Jason Baron --- hw/i386/Makefile.objs |1 + hw/lpc_ich9.c | 523 + 2 files changed, 524 insertions(+), 0 deletions(-) create mode 100

[Qemu-devel] [PATCH v4 00/14] Add Q35 base support

2012-11-14 Thread Jason Baron
Hi, These patches are intened to give us a base set of patches for Q35 upon which to build. The major change in this series is to add the memory controller hub, or 'mch' as proper member of the q35 host structure. This change refactors the code a bit, and moves more intialization out of pc_q35.c a

Re: [Qemu-devel] [PATCH 0/3] Opteron_G5 and Haswell CPU models

2012-11-14 Thread Eduardo Habkost
Andreas, I just noticed that I didn't CC you in this series, and I don't know if this is something that would go through your tree. Should I CC you on x86-specific series unrelated to QOM, like this one? On Wed, Nov 14, 2012 at 04:28:51PM -0200, Eduardo Habkost wrote: > I'm resending the patches s

[Qemu-devel] [Bug 1077838] Re: qemu-nbd -r -c taints device for subsequent usage, even after -d

2012-11-14 Thread Launchpad Bug Tracker
This bug was fixed in the package qemu-kvm - 1.2.0+noroms-0ubuntu4 --- qemu-kvm (1.2.0+noroms-0ubuntu4) raring; urgency=low [ Serge Hallyn ] * debian/qemu-kvm.postinst: use udevadm trigger to change /dev/kvm perms as recommended by Steve Langasek (LP: #1057024) * apply debia

[Qemu-devel] [PATCH 01/26] qemu: Document GlusterFS block driver usage

2012-11-14 Thread Kevin Wolf
From: Bharata B Rao Signed-off-by: Bharata B Rao Signed-off-by: Kevin Wolf --- qemu-doc.texi | 49 + qemu-options.hx | 17 + 2 files changed, 66 insertions(+), 0 deletions(-) diff --git a/qemu-doc.texi b/qemu-doc.texi index

[Qemu-devel] [PULL 00/26] Block patches

2012-11-14 Thread Kevin Wolf
The following changes since commit ce34cf72fe508b27a78f83c184142e8d1e6a048a: Merge remote-tracking branch 'awilliam/tags/vfio-pci-for-qemu-1.3.0-rc0' into staging (2012-11-14 08:53:40 -0600) are available in the git repository at: git://repo.or.cz/qemu/kevin.git for-anthony Bharata B Rao (

[Qemu-devel] [PATCH 17/26] fdc: fix false FD_SR0_SEEK

2012-11-14 Thread Kevin Wolf
From: Hervé Poussineau Do not always set FD_SR0_SEEK, as callers already set it if needed. Signed-off-by: Hervé Poussineau Signed-off-by: Kevin Wolf --- hw/fdc.c | 10 +++--- tests/fdc-test.c |6 +++--- 2 files changed, 10 insertions(+), 6 deletions(-) diff --git a/hw/fdc.c

[Qemu-devel] [PATCH 13/26] fdc-test: add tests for non-DMA READ command

2012-11-14 Thread Kevin Wolf
From: Hervé Poussineau Signed-off-by: Hervé Poussineau Signed-off-by: Kevin Wolf --- tests/fdc-test.c | 98 ++ 1 files changed, 98 insertions(+), 0 deletions(-) diff --git a/tests/fdc-test.c b/tests/fdc-test.c index 67bfb22..4649e3f 100644

[Qemu-devel] [PATCH 03/26] qemu-iotests: qcow2: Test growing large refcount table

2012-11-14 Thread Kevin Wolf
Actually writing all the content with 512 byte sector size would take forever, therefore build the image file with a Python script and use qemu-io for the last write that actually triggers the refcount table growth. Signed-off-by: Kevin Wolf --- tests/qemu-iotests/044| 117 +

[Qemu-devel] [PATCH 12/26] fdc-test: insert media before fuzzing registers

2012-11-14 Thread Kevin Wolf
From: Hervé Poussineau A media will be required for future fdc tests. Signed-off-by: Hervé Poussineau Signed-off-by: Kevin Wolf --- tests/fdc-test.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/tests/fdc-test.c b/tests/fdc-test.c index a4303d1..67bfb22 100644 --- a

[Qemu-devel] [PATCH 06/26] tests: allow qemu-iotests to be run against nbd backend

2012-11-14 Thread Kevin Wolf
From: Nick Thomas To do this, we start a qemu-nbd process at _make_test_img and kill it in _cleanup_test_img. $TEST_IMG is changed to point at the TCP server. We also remove the checks for existence of binaries from common.config - they're duplicated in common, and we can make the qemu-nbd check

[Qemu-devel] [PATCH 19/26] fdc: implement VERIFY command

2012-11-14 Thread Kevin Wolf
From: Hervé Poussineau VERIFY command is like a READ command, except that read data is not transfered by DMA. As DMA engine is not used, so we have to start data transfer ourselves. Signed-off-by: Hervé Poussineau Signed-off-by: Kevin Wolf --- hw/fdc.c | 25 ++--- 1 file

[Qemu-devel] [PATCH 16/26] fdc: fix FD_SR0_SEEK for initial seek on DMA transfers

2012-11-14 Thread Kevin Wolf
From: Hervé Poussineau fdctrl_start_transfer() used to set FD_SR0_SEEK no matter if there actually was a seek or not. This is obviously wrong. fdctrl_start_transfer() has this information because it performs the initial seek itself. Signed-off-by: Hervé Poussineau Signed-off-by: Kevin Wolf --

[Qemu-devel] [PATCH 20/26] fdc-tests: add tests for VERIFY command

2012-11-14 Thread Kevin Wolf
From: Hervé Poussineau Signed-off-by: Hervé Poussineau Signed-off-by: Kevin Wolf --- tests/fdc-test.c | 16 +--- 1 files changed, 13 insertions(+), 3 deletions(-) diff --git a/tests/fdc-test.c b/tests/fdc-test.c index e8ce686..4b0301d 100644 --- a/tests/fdc-test.c +++ b/tests/fd

Re: [Qemu-devel] [PATCH] mips/malta: fix CBUS UART interrupt pin

2012-11-14 Thread Johnson, Eric
> -Original Message- > From: qemu-devel-bounces+ericj=mips@nongnu.org [mailto:qemu-devel- > bounces+ericj=mips@nongnu.org] On Behalf Of Aurelien Jarno > Sent: Wednesday, November 14, 2012 6:38 AM > To: qemu-devel@nongnu.org > Cc: Aurelien Jarno > Subject: [Qemu-devel] [PATCH] mips/m

[Qemu-devel] [PATCH 08/26] aio: use g_slice_alloc() for AIOCB pooling

2012-11-14 Thread Kevin Wolf
From: Stefan Hajnoczi AIO control blocks are frequently acquired and released because each aio request involves at least one AIOCB. Therefore, we pool them to avoid heap allocation overhead. The problem with the freelist approach in AIOPool is thread-safety. If we want BlockDriverStates to ass

[Qemu-devel] [PATCH 05/26] block: Workaround for older versions of MinGW gcc

2012-11-14 Thread Kevin Wolf
From: Stefan Weil Versions before gcc-4.6 don't support unnamed fields in initializers (see http://gcc.gnu.org/bugzilla/show_bug.cgi?id=10676). Offset and OffsetHigh belong to an unnamed struct which is part of an unnamed union. Therefore the original code does not work with older versions of gc

[Qemu-devel] [PATCH 1/3] target-i386: cpu: name new CPUID bits

2012-11-14 Thread Eduardo Habkost
From: Andre Przywara Update QEMU's knowledge of CPUID bit names. This allows to enable/disable those new features on QEMU's command line when using KVM and prepares future feature enablement in QEMU. This adds F16C, RDRAND, LWP, TBM, TopoExt, PerfCtr_Core, PerfCtr_NB, FSGSBASE, BMI1, AVX2, BMI2,

[Qemu-devel] [PATCH 26/26] qemu-io: Use bdrv_drain_all instead of qemu_aio_flush

2012-11-14 Thread Kevin Wolf
This is harmless as of today because I/O throttling is not used in qemu-io, however as soon as .bdrv_drain handlers will be introduced, qemu-io must be sure to call bdrv_drain_all(). Signed-off-by: Kevin Wolf --- qemu-io.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a

[Qemu-devel] [PATCH 02/26] qcow2: Fix refcount table size calculation

2012-11-14 Thread Kevin Wolf
A missing factor for the refcount table entry size in the calculation could mean that too little memory was allocated for the in-memory representation of the table, resulting in a buffer overflow. Signed-off-by: Kevin Wolf Reviewed-by: Michael Tokarev Tested-by: Michael Tokarev --- block/qcow2

Re: [Qemu-devel] [PATCH] block: vpc support for ~2 TB disks

2012-11-14 Thread Thanos Makatos
We don't use qemu's VHD driver in XenServer. Instead, we use blktap2 to create a block device in dom0 serving the VHD file in question, and have qemu open that block device instead of the VHD file itself. > -Original Message- > From: Stefano Stabellini [mailto:stefano.stabell...@eu.citri

[Qemu-devel] [PATCH 18/26] fdc-test: Check READ ID

2012-11-14 Thread Kevin Wolf
ST0 shouldn't include 0x20 (FD_SR0_SEEK) after READ ID. Signed-off-by: Kevin Wolf Tested-by: Hervé Poussineau --- tests/fdc-test.c | 66 ++ 1 files changed, 66 insertions(+), 0 deletions(-) diff --git a/tests/fdc-test.c b/tests/fdc-test.c i

[Qemu-devel] [PATCH 21/26] fdc: remove double affectation of FD_MSR_CMDBUSY flag

2012-11-14 Thread Kevin Wolf
From: Hervé Poussineau FD_MSR_CMDBUSY flag is already set in fdctrl_write_data(), just before calling the command handler (fdctrl_start_transfer() here). Signed-off-by: Hervé Poussineau Signed-off-by: Kevin Wolf --- hw/fdc.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --

[Qemu-devel] [PATCH 24/26] vmdk: Fix data corruption bug in WRITE and READ handling

2012-11-14 Thread Kevin Wolf
From: Gerhard Wiesinger Fixed a MAJOR BUG in VMDK files on file boundaries on reads and ALSO ON WRITES WHICH MIGHT CORRUPT THE IMAGE AND DATA!! Triggered for example with the following VMDK file (partly listed): RW 4193792 FLAT "XP-W1-f001.vmdk" 0 RW 2097664 FLAT "XP-W1-f002.vmdk" 0 RW 41937

[Qemu-devel] [PATCH 25/26] megasas: Use bdrv_drain_all instead of qemu_aio_flush

2012-11-14 Thread Kevin Wolf
Calling qemu_aio_flush() directly can hang when combined with I/O throttling. Signed-off-by: Kevin Wolf Acked-by: Paolo Bonzini --- hw/megasas.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/hw/megasas.c b/hw/megasas.c index 291ff40..61b6527 100644 --- a/hw/megasas.c

[Qemu-devel] [PATCH 11/26] fdc-test: split test_media_change() test, so insert part can be reused

2012-11-14 Thread Kevin Wolf
From: Hervé Poussineau Signed-off-by: Hervé Poussineau Signed-off-by: Kevin Wolf --- tests/fdc-test.c |9 - 1 files changed, 8 insertions(+), 1 deletions(-) diff --git a/tests/fdc-test.c b/tests/fdc-test.c index fa74411..a4303d1 100644 --- a/tests/fdc-test.c +++ b/tests/fdc-test.c

[Qemu-devel] [PATCH 15/26] fdc: fix FD_SR0_SEEK for non-DMA transfers and multi sectors transfers

2012-11-14 Thread Kevin Wolf
From: Hervé Poussineau On non-DMA transfers, fdctrl_stop_transfer() used to set FD_SR0_SEEK no matter if there actually was a seek or not. This is obviously wrong. fdctrl_seek_to_next_sect() has this information because it performs the seek itself. Signed-off-by: Hervé Poussineau Signed-off-by

[Qemu-devel] [PATCH 07/26] aio: switch aiocb_size type int -> size_t

2012-11-14 Thread Kevin Wolf
From: Stefan Hajnoczi Using appropriate types for variables is a good thing :). All users simply do sizeof(MyType) and the value is passed to a memory allocator, it should be size_t. Signed-off-by: Stefan Hajnoczi Reviewed-by: Paolo Bonzini Signed-off-by: Kevin Wolf --- qemu-aio.h |2 +-

[Qemu-devel] [PATCH 23/26] fdc: remove last usage of FD_STATE_SEEK

2012-11-14 Thread Kevin Wolf
From: Hervé Poussineau Replace it by directly setting FD_SR0_SEEK if required Signed-off-by: Hervé Poussineau Signed-off-by: Kevin Wolf --- hw/fdc.c | 12 ++-- 1 files changed, 2 insertions(+), 10 deletions(-) diff --git a/hw/fdc.c b/hw/fdc.c index de8778b..29b5449 100644 --- a/hw/

[Qemu-devel] [PATCH 09/26] aio: rename AIOPool to AIOCBInfo

2012-11-14 Thread Kevin Wolf
From: Stefan Hajnoczi Now that AIOPool no longer keeps a freelist, it isn't really a "pool" anymore. Rename it to AIOCBInfo and make it const since it no longer needs to be modified. Signed-off-by: Stefan Hajnoczi Signed-off-by: Kevin Wolf --- block.c | 22 +++---

[Qemu-devel] [PATCH 10/26] fdc: Remove status0 parameter from fdctrl_set_fifo()

2012-11-14 Thread Kevin Wolf
It decided whether an interrupt is triggered. Only one caller made use of this functionality, so move the code there. In this one caller, the interrupt must actually be triggered unconditionally, like it was before commit 2fee0088. For example, a successful read without an implied seek can result

[Qemu-devel] [PATCH 14/26] fdc: use status0 field instead of a local variable

2012-11-14 Thread Kevin Wolf
From: Hervé Poussineau Signed-off-by: Hervé Poussineau Signed-off-by: Kevin Wolf --- hw/fdc.c | 27 --- 1 files changed, 16 insertions(+), 11 deletions(-) diff --git a/hw/fdc.c b/hw/fdc.c index 3585220..7e4b0ce 100644 --- a/hw/fdc.c +++ b/hw/fdc.c @@ -327,7 +327,7 @@

[Qemu-devel] [PATCH 04/26] MAINTAINERS: add Stefan Hajnoczi as block and virtio-blk co-maintainer

2012-11-14 Thread Kevin Wolf
From: Stefan Hajnoczi Kevin has requested co-maintainership to give him more time to write code. We will alternate patch review duties on a weekly basis. Signed-off-by: Stefan Hajnoczi Signed-off-by: Kevin Wolf --- MAINTAINERS |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff

[Qemu-devel] [PATCH 22/26] fdc: fix typo in zero constant

2012-11-14 Thread Kevin Wolf
From: Hervé Poussineau Signed-off-by: Hervé Poussineau Signed-off-by: Kevin Wolf --- hw/fdc.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/hw/fdc.c b/hw/fdc.c index a197c48..de8778b 100644 --- a/hw/fdc.c +++ b/hw/fdc.c @@ -1244,7 +1244,7 @@ static void fdctrl_start

[Qemu-devel] [PATCH 3/3] target-i386: add Haswell CPU model

2012-11-14 Thread Eduardo Habkost
Features added to the model, in relation to SandyBridge: fma CPUID[1].ECX[12] pcid CPUID[1].ECX[17] movbe CPUID[1].ECX[22] fsgsbase CPUID[EAX=7,ECX=0].EBX[0] bmi1 CPUID[EAX=7,ECX=0].EBX[3] hle CPUID[EAX=7,ECX=0].EBX[4] avx2 CPUID[EAX=7,ECX=0].EBX[5] smep CPUID[

[Qemu-devel] [PATCH 0/3] Opteron_G5 and Haswell CPU models

2012-11-14 Thread Eduardo Habkost
I'm resending the patches sent by Boris Ostrovsky previously, as it has to be rebased against latest qemu.git, and to add a new Haswell CPU model too. Andre Przywara (2): target-i386: cpu: name new CPUID bits target-i386: cpu: add new Opteron CPU model Eduardo Habkost (1): target-i386: add

Re: [Qemu-devel] [PATCH 5/6] pixman: build internal version early

2012-11-14 Thread Johnson, Eric
> -Original Message- > From: Gerd Hoffmann [mailto:kra...@redhat.com] > Sent: Wednesday, November 14, 2012 4:42 AM [...] > On 11/13/12 19:41, Johnson, Eric wrote: [...] > > I wasn't sure how to submit a patch to an unsubmitted patch. > > As usual: "git commit -s" + "git format-patch" + "gi

[Qemu-devel] [PATCH 2/3] target-i386: cpu: add new Opteron CPU model

2012-11-14 Thread Eduardo Habkost
From: Andre Przywara Add a new base CPU model called Opteron_G5 to model the latest Opteron CPUs. This increases the model value and model numbers and adds TBM, F16C and FMA over the latest G4 model. Signed-off-by: Andre Przywara Signed-off-by: Boris Ostrovsky [ehabkost: edited commit message]

Re: [Qemu-devel] [PATCH 2/2] x86/cpu: add new Opteron CPU model

2012-11-14 Thread Eduardo Habkost
On Fri, Nov 02, 2012 at 09:21:38AM -0400, Boris Ostrovsky wrote: > From: Andre Przywara > > Add a new base CPU model called Opteron_G5 to model the latest > Opteron CPUs. This increases the model value and model numbers and > adds TBM, F16C and FMA over the latest G4 model. > > Signed-off-by: An

Re: [Qemu-devel] [PATCH 1/2] x86/cpu: name new CPUID bits

2012-11-14 Thread Eduardo Habkost
On Fri, Nov 02, 2012 at 09:21:26AM -0400, Boris Ostrovsky wrote: > From: Andre Przywara > > Update QEMU's knowledge of CPUID bit names. This allows to > enable/disable those new features on QEMU's command line when > using KVM and prepares future feature enablement in QEMU. > > This adds F16C, R

[Qemu-devel] [Bug 1077838] Re: qemu-nbd -r -c taints device for subsequent usage, even after -d

2012-11-14 Thread Serge Hallyn
** Changed in: qemu-kvm (Ubuntu) Assignee: (unassigned) => Serge Hallyn (serge-hallyn) ** Changed in: qemu-kvm (Ubuntu) Status: Triaged => In Progress -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad

[Qemu-devel] [Bug 1077838] Re: qemu-nbd -r -c taints device for subsequent usage, even after -d

2012-11-14 Thread Serge Hallyn
Thanks, Paul, I'll cherrypick commit c8969eded252058e90e91f12f75f32aceae46ec9 into the ubuntu packages -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1077838 Title: qemu-nbd -r -c taints device for

[Qemu-devel] [PATCH 04/10] ehci: Add a ehci_writeback_async_complete_packet helper function

2012-11-14 Thread Hans de Goede
Also drop the warning printf, which was there mainly because this was an untested code path (as the previous bug fixes to it show), but that no longer is the case now :) Signed-off-by: Hans de Goede --- hw/usb/hcd-ehci.c | 28 +--- 1 file changed, 17 insertions(+), 11 del

  1   2   >