[Qemu-devel] [PATCH 0/2 v3] kvm: notify host when guest panicked

2012-03-07 Thread Wen Congyang
We can know the guest is paniced when the guest runs on xen. But we do not have such feature on kvm. Another purpose of this feature is: management app(for example: libvirt) can do auto dump when the guest is crashed. If management app does not do auto dump, the guest's user can do dump by hand if

Re: [Qemu-devel] XP install cores with SCSI LSI 53C895A disks

2012-03-07 Thread Gerd Hoffmann
On 03/07/12 20:58, Gerhard Wiesinger wrote: > On Wed, 7 Mar 2012, Brian Jackson wrote: >> I think most people trying to use qemu for anything useful have given >> up on if=scsi. Some distros even disable support because they don't >> want to QA it. That should be a decent sign that you may want to

Re: [Qemu-devel] [PATCH 13/21] libcacard: fix reported ATR length

2012-03-07 Thread Wen Congyang
At 02/28/2012 06:20 PM, Gerd Hoffmann Wrote: > From: Alon Levy > > Signed-off-by: Alon Levy > Signed-off-by: Gerd Hoffmann > --- > libcacard/vcardt.h |4 ++-- > 1 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/libcacard/vcardt.h b/libcacard/vcardt.h > index 538bdde..d4d8e

Re: [Qemu-devel] PC BIOS Flash emulation

2012-03-07 Thread Dmitry Zamaruev
On Wed, Mar 7, 2012 at 7:39 PM, Jordan Justen wrote: > On Wed, Mar 7, 2012 at 08:51, Dmitry Zamaruev > wrote: >> On Wed, Mar 7, 2012 at 6:32 PM, Avi Kivity wrote: >>> On 03/07/2012 03:59 PM, Dmitry Zamaruev wrote: I'm trying to emulate BIOS Flash chip (something Intel FWH compatible)

[Qemu-devel] [patch] make trace_thread_create() use its function arg

2012-03-07 Thread Jun Koi
this patch makes trace_thread_create() to use its function arg to initialize thread. the other choice is to make this a function to use void arg, but i prefer this way. Signed-off-by: Jun Koi diff --git a/trace/simple.c b/trace/simple.c index bbc9930..33ae486 100644 --- a/trace/simple.c +++ b/t

[Qemu-devel] Virtual FAT disk images error, can somebody help me , thank you !

2012-03-07 Thread Zhi Hui Li
when I use the command : 1: mkdir /my_directory 2: qemu linux.img -fda fat:floppy:/my_directory when I write to the floppy(just "mkdir test.dir" in the floppy), the error occur: "FLOPPY ERROR: fdctrl_transfer_handler: writing sector 1" and nothing is written to the /my_directory. I don'

Re: [Qemu-devel] [PATCH 2/2] cache-utils.h needs stdint.h for uintptr_t

2012-03-07 Thread Stefan Weil
Am 08.03.2012 02:09, schrieb David Gibson: cache-utils.h uses the C99 standard uintptr_t type. However, that type comes from stdint.h which is not #included before cache-utils.h in all configurations. This patch adds the necessary include to fix this. Signed-off-by: David Gibson --- cache-ut

Re: [Qemu-devel] Interpretation of key symbols in QEMU's VNC server

2012-03-07 Thread Gerhard Wiesinger
Hello Fabian, I'm having also isssues with german keymappings. E.g. Under DOS when pressing shift keys will always be uppercase. Also ALT-GR doesn't work. Testcase in this order: a => a a => a Shift-a => a (should be A) a => A (should be a) a => A (should be a) a => A (should be a) Shift-a =>

[Qemu-devel] [RFC] Future goals for autotest and virtualization tests

2012-03-07 Thread Lucas Meneghel Rodrigues
Hi guys. For a while we have been discussing ways to make the virtualization tests written on top of autotest useful for development level testing. One of our main goals is to provide useful tools for the qemu community, since we have a good number of tests and libraries written to perform in

Re: [Qemu-devel] [PATCH 1/2 v7] block: add-cow file format

2012-03-07 Thread Dong Xu Wang
On Wed, Mar 7, 2012 at 00:59, Stefan Hajnoczi wrote: > On Thu, Mar 1, 2012 at 2:56 AM, Dong Xu Wang > wrote: >> diff --git a/block/add-cow-cache.c b/block/add-cow-cache.c >> new file mode 100644 >> index 000..6be02ff >> --- /dev/null >> +++ b/block/add-cow-cache.c >> @@ -0,0 +1,171 @@ >> +/*

[Qemu-devel] [PATCH] ioapic: fix build with DEBUG_IOAPIC

2012-03-07 Thread Jason Wang
ioapic.c:198: error: format ‘%08x’ expects type ‘unsigned int’, but argument 3 has type ‘uint64_t’ Signed-off-by: Jason Wang --- hw/ioapic.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/hw/ioapic.c b/hw/ioapic.c index 3fee011..2b0e6e9 100644 --- a/hw/ioapic.c +++ b/

Re: [Qemu-devel] [PATCH 2/2] block: add-cow support snapshot_blkdev

2012-03-07 Thread Dong Xu Wang
image file will be created automaticly, file name is formated as: sprintf(image_file, "%s.raw", snapshot_file); I just add ".raw" to the file name of #1 in you comments as file name of #2. On Wed, Mar 7, 2012 at 21:10, Stefan Hajnoczi wrote: > On Thu, Mar 1, 2012 at 2:49 AM, Dong Xu Wang > wro

Re: [Qemu-devel] [PATCH 0/2] core dump: re-purpose VM_ALWAYSDUMP to user controlled VM_DONTDUMP

2012-03-07 Thread Jason Baron
On Wed, Mar 07, 2012 at 02:30:28PM -0800, Andrew Morton wrote: > On Wed, 7 Mar 2012 12:00:46 -0500 > Jason Baron wrote: > > > Hi, > > > > The motivation for this change was that I was looking at a way for a > > qemu-kvm > > process, to exclude the guest memory from its core dump, which can be q

Re: [Qemu-devel] [PATCH] PPC: Fix large page support in TCG

2012-03-07 Thread David Gibson
On Sat, Mar 03, 2012 at 10:39:34AM -0600, Nathan Whitehorn wrote: > Fix large page support in TCG. The old code would overwrite the > large page table entry with the fake 4 KB > one generated here whenever the ref/change bits were updated, > causing it to point to the wrong area of memory. Instead

[Qemu-devel] [0/2] Updated pseries patches

2012-03-07 Thread David Gibson
Hi Alex, Here are the two patches from my last pseries batch which didn't go in, updated to incorporate comments from the list.

[Qemu-devel] [PATCH 1/2] pseries: Add support for level interrupts to XICS

2012-03-07 Thread David Gibson
The pseries "xics" interrupt controller, like most interrupt controllers can support both message (i.e. edge sensitive) interrupts and level sensitive interrupts, but it needs to know which are which. When I implemented the xics emulation for qemu, the only devices we supported were the PAPR virtu

[Qemu-devel] [PATCH 2/2] pseries: Configure PCI bridge using properties

2012-03-07 Thread David Gibson
Currently, the function spapr_create_phb() uses its parameters to initialize the correct memory windows for the new PCI Host Bridge (PHB). This is not the way things are supposed to be done with qdevs, and means you can't create extra PHBs easily using -device. Since pSeries machines can and do h

[Qemu-devel] [PATCH] usb-ohci: DMA writeback bug fixes

2012-03-07 Thread David Gibson
From: Wei Yang This patch fixes two bugs in the OHCI device where the device writes back data to system memory that should be exclusively under the control of the guest side driver. In OHCI specification Section 5.2.7, it mentioned "In all cases, Host Controller Driver is responsible for the ins

[Qemu-devel] [PATCH 2/2] cache-utils.h needs stdint.h for uintptr_t

2012-03-07 Thread David Gibson
cache-utils.h uses the C99 standard uintptr_t type. However, that type comes from stdint.h which is not #included before cache-utils.h in all configurations. This patch adds the necessary include to fix this. Signed-off-by: David Gibson --- cache-utils.h |2 ++ 1 files changed, 2 insertion

[Qemu-devel] [PATCH 1/2] .gitignore update

2012-03-07 Thread David Gibson
This adds a few previously missing generated files to .gitignore: the qemu-bridge-helper binary, and the linuxboot and multiboot images from pc-bios/optionrom. Signed-off-by: David Gibson --- .gitignore |1 + pc-bios/optionrom/.gitignore |5 + 2 files changed, 6 ins

[Qemu-devel] [PATCH 1/2] core dump: drop VM_ALWAYSDUMP flag

2012-03-07 Thread Jason Baron
The VM_ALWAYSDUMP flag is currently used by the coredump code to indicate that a vma is part of a vsyscall or vdso section, and then always dump it. However, we can determine if a vma is in in one of these sections by using 'arch_vma_name()', and thus re-purpose a valuable vma bit. Signed-off-by

[Qemu-devel] [PATCH 0/2] core dump: re-purpose VM_ALWAYSDUMP to user controlled VM_DONTDUMP

2012-03-07 Thread Jason Baron
Hi, The motivation for this change was that I was looking at a way for a qemu-kvm process, to exclude the guest memory from its core dump, which can be quite large. There are already a number of filter flags in /proc//coredump_filter, however, these allow one to specify 'types' of kernel memory, n

[Qemu-devel] How to get gpr register contents in translate.c

2012-03-07 Thread Elta
Hello everyone, I'm a newcomer here, nice to see you! Now I get trouble in translate.c. I really want to get gpr value in translate.c, but cpu_gpr[rs]'s type is TCGv, can not get the real value of rs. Is the have any method or macro can made it? Thanks for your attention, have a nice day!

[Qemu-devel] [PATCH 4/4] pci: Factor out bounds checking on config space accesses

2012-03-07 Thread David Gibson
There are several paths into the code to emulate PCI config space accesses: one for MMIO to a plain old PCI bridge one for MMIO to a PCIe bridge and one for the pseries machine which provides para-virtualized access to PCI config space. Each of these functions does their own bounds checking agains

[Qemu-devel] [PATCH 2/4] Endian fix an assertion in usb-msd

2012-03-07 Thread David Gibson
From: Benjamin Herrenschmidt This fixes a broken endian assumption in an assertion in usb-msd. Cc: Gerd Hoffman Signed-off-by: Benjamin Herrenschmidt Signed-off-by: David Gibson --- hw/usb-msd.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/hw/usb-msd.c b/hw/usb-

Re: [Qemu-devel] [PATCH 1/2] core dump: drop VM_ALWAYSDUMP flag

2012-03-07 Thread Jason Baron
On Wed, Mar 07, 2012 at 04:26:46PM -0500, Chris Metcalf wrote: > On 3/7/2012 4:19 PM, Jason Baron wrote: > > On Wed, Mar 07, 2012 at 11:43:02AM -0800, Roland McGrath wrote: > >>> well, sure, we can provide an arch interface, for this check. I'm more > >>> concerned with the general idea. If it seem

[Qemu-devel] [PATCH 1/4] kvm: Comparison with ioctl number macros needs to be unsigned

2012-03-07 Thread David Gibson
In kvm-all.c we store an ioctl cmd number in the irqchip_inject_ioctl field of KVMState, which has type 'int'. This seems to make sense since the ioctl() man page says that the cmd parameter has type int. However, the kernel treats ioctl numbers as unsigned - sys_ioctl() takes an unsigned int, an

[Qemu-devel] (no subject)

2012-03-07 Thread David Gibson
These are patches that I've sent in before but don't seem to have been picked up by anybody yet. Resending so they don't fall off the radar. 1, 2, and 3 of 4 in particular fix real, observed bugs on powerpc.

[Qemu-devel] [PATCH 3/4] kvm: Fix dirty tracking with large kernel page size

2012-03-07 Thread David Gibson
If the kernel page size is larger than TARGET_PAGE_SIZE, which happens for example on ppc64 with kernels compiled for 64K pages, the dirty tracking doesn't work. Cc: Avi Kivity Cc: Marcelo Tossatti Signed-off-by: Benjamin Herrenschmidt Signed-off-by: David Gibson --- kvm-all.c |8 +--

[Qemu-devel] Interpretation of key symbols in QEMU's VNC server

2012-03-07 Thread Fabian Holler
Hello, I'm not sure if I found a bug in QEMU's VNC keysymbol to scancode mapping or if it is a general Problem with the implemented VNC server: Scenario: QEMU started with: "-k de" Keyboard layout in VM: de Keyboard layout from Client OS: us What i expect: I type the '/' character on the Client

[Qemu-devel] [PATCH 2/2] core dump: add a new VM_DONTDUMP flag

2012-03-07 Thread Jason Baron
Since we no longer need the VM_ALWAYSDUMP flag, let's use the freed bit for 'VM_DONTDUMP' flag. The idea is to add a new madvise() flag: MADV_DONTDUMP, which can be set by applications to specifically request memory regions which should not dump core. The specific application I have in mind is qemu

Re: [Qemu-devel] [PATCH 1/2] core dump: drop VM_ALWAYSDUMP flag

2012-03-07 Thread Jason Baron
On Wed, Mar 07, 2012 at 11:43:02AM -0800, Roland McGrath wrote: > > well, sure, we can provide an arch interface, for this check. I'm more > > concerned with the general idea. If it seems ok, I can re-do this bit > > with an arch interface. > > Anything based on strcmp is dismal. VM_ALWAYSDUMP wa

Re: [Qemu-devel] [PATCH 1/2] core dump: drop VM_ALWAYSDUMP flag

2012-03-07 Thread Jason Baron
On Wed, Mar 07, 2012 at 09:59:58AM -0800, Roland McGrath wrote: > On Wed, Mar 7, 2012 at 9:00 AM, Jason Baron wrote: > > +       if (vma_name) { > > +               if ((strcmp(vma_name, "[vdso]") == 0) || > > +                   (strcmp(vma_name, "[vectors]") == 0) || > > +                   (str

[Qemu-devel] Keysymbol interpretation missing in QEMU's VNC server?

2012-03-07 Thread Fabian Holler
Hello, I'm not sure if I found a bug in QEMU's VNC keyboard layout mapping or if it's a general problem in the implemented VNC server: Scenario: QEMU started with: "-k de" Keyboard layout in VM: de Keyboard layout from Client OS: us What i expect: I type the '/' character on the Client OS (key l

[Qemu-devel] Keysymbol interpretation missing in QEMU's VNC server?

2012-03-07 Thread Fabian Holler
Hello, I'm not sure if I found a bug in QEMU's VNC keyboard layout mapping or if it's a general problem in the implemented VNC server: Scenario: QEMU started with: "-k de" Keyboard layout in VM: de Keyboard layout from Client OS: us What i expect: I type the '/' character on the Client OS (key l

Re: [Qemu-devel] [libvirt] Qemu, libvirt, and CPU models

2012-03-07 Thread Eric Blake
On 03/07/2012 03:26 PM, Eduardo Habkost wrote: > Thanks a lot for the explanations, Daniel. > > Comments about specific items inline. > >>> - How can we make sure there is no confusion between libvirt and Qemu >>> about the CPU models? For example, what if cpu_map.xml says model >>> 'M

Re: [Qemu-devel] [PATCH v2 0/3] New sigaltstack backend for coroutine

2012-03-07 Thread Peter Maydell
On 7 March 2012 22:44, Alex Barcelo wrote: > On Wed, Mar 7, 2012 at 23:17, Peter Maydell wrote: >> Did you see the comment I added to an earlier thread regarding >> FORTIFY_SOURCE? > > Sorry about that! Yes, I got the comment mixed between some other > threads, and now I was checking and didn't r

Re: [Qemu-devel] [PATCH 0/6] PCI hotplug improvements

2012-03-07 Thread Alex Williamson
On Thu, 2012-03-08 at 00:17 +0200, Gleb Natapov wrote: > On Wed, Mar 07, 2012 at 02:44:13PM -0700, Alex Williamson wrote: > > On Wed, 2012-03-07 at 23:00 +0200, Gleb Natapov wrote: > > > On Wed, Mar 07, 2012 at 12:51:48PM -0700, Alex Williamson wrote: > > > > On Wed, 2012-03-07 at 20:59 +0200, Gleb

Re: [Qemu-devel] [PATCH v2 0/3] New sigaltstack backend for coroutine

2012-03-07 Thread Alex Barcelo
On Wed, Mar 7, 2012 at 23:17, Peter Maydell wrote: > On 7 March 2012 22:01, Alex Barcelo wrote: >> Is this patch okay? The first version had some comments, and now the >> v2 has been a bit too silent, not sure if that's a good sign or a bad >> sign. > > Did you see the comment I added to an earli

Re: [Qemu-devel] [PATCH 0/2] core dump: re-purpose VM_ALWAYSDUMP to user controlled VM_DONTDUMP

2012-03-07 Thread Andrew Morton
On Wed, 7 Mar 2012 12:00:46 -0500 Jason Baron wrote: > Hi, > > The motivation for this change was that I was looking at a way for a qemu-kvm > process, to exclude the guest memory from its core dump, which can be quite > large. There are already a number of filter flags in > /proc//coredump_filt

Re: [Qemu-devel] [libvirt] Qemu, libvirt, and CPU models

2012-03-07 Thread Eduardo Habkost
Thanks a lot for the explanations, Daniel. Comments about specific items inline. On Wed, Mar 07, 2012 at 02:18:28PM +, Daniel P. Berrange wrote: > > I have two main points I would like to understand/discuss: > > > > 1) The relationship between libvirt's cpu_map.xml and the Qemu CPU model > >

Re: [Qemu-devel] [PATCH 0/6] PCI hotplug improvements

2012-03-07 Thread Gleb Natapov
On Wed, Mar 07, 2012 at 02:44:13PM -0700, Alex Williamson wrote: > On Wed, 2012-03-07 at 23:00 +0200, Gleb Natapov wrote: > > On Wed, Mar 07, 2012 at 12:51:48PM -0700, Alex Williamson wrote: > > > On Wed, 2012-03-07 at 20:59 +0200, Gleb Natapov wrote: > > > > On Wed, Mar 07, 2012 at 10:20:49AM -070

Re: [Qemu-devel] [PATCH v2 0/3] New sigaltstack backend for coroutine

2012-03-07 Thread Peter Maydell
On 7 March 2012 22:01, Alex Barcelo wrote: > Is this patch okay? The first version had some comments, and now the > v2 has been a bit too silent, not sure if that's a good sign or a bad > sign. Did you see the comment I added to an earlier thread regarding FORTIFY_SOURCE? I think in general my o

Re: [Qemu-devel] [PATCH v2 0/3] New sigaltstack backend for coroutine

2012-03-07 Thread Alex Barcelo
Is this patch okay? The first version had some comments, and now the v2 has been a bit too silent, not sure if that's a good sign or a bad sign. Thanks & sorry! On Tue, Feb 28, 2012 at 12:25, Alex Barcelo wrote: > This series of patches implements coroutines method with > sigaltstack. > > The fl

Re: [Qemu-devel] [PATCH 1/2] core dump: drop VM_ALWAYSDUMP flag

2012-03-07 Thread Chris Metcalf
On 3/7/2012 4:19 PM, Jason Baron wrote: > On Wed, Mar 07, 2012 at 11:43:02AM -0800, Roland McGrath wrote: >>> well, sure, we can provide an arch interface, for this check. I'm more >>> concerned with the general idea. If it seems ok, I can re-do this bit >>> with an arch interface. >> Anything base

Re: [Qemu-devel] [PATCH 0/6] PCI hotplug improvements

2012-03-07 Thread Alex Williamson
On Wed, 2012-03-07 at 23:00 +0200, Gleb Natapov wrote: > On Wed, Mar 07, 2012 at 12:51:48PM -0700, Alex Williamson wrote: > > On Wed, 2012-03-07 at 20:59 +0200, Gleb Natapov wrote: > > > On Wed, Mar 07, 2012 at 10:20:49AM -0700, Alex Williamson wrote: > > > > On Wed, 2012-03-07 at 14:43 +0200, Gleb

Re: [Qemu-devel] [PATCH v2 0/4] slirp: Fix for requeuing crash, cleanups

2012-03-07 Thread Stefan Weil
Am 06.03.2012 08:41, schrieb Jan Kiszka: On 2012-03-05 10:06, Jan Kiszka wrote: Grmbl. Was very hard to reproduce here (triggered once every few hours with lots of interaction beforehand), but now I think I got the point (recursion of if_start due to if_encap). Will rework the code to address th

Re: [Qemu-devel] [PATCH 1/2] core dump: drop VM_ALWAYSDUMP flag

2012-03-07 Thread Roland McGrath
A few of the vDSO VM_ALWAYSDUMP cases have a comment about the core dump. (It looks like they were all copied from my original comment in arch/x86/vdso/vdso32-setup.c.) So those should be removed where they are. (You removed some of them, but not all.) You should put that comment into always_dump

Re: [Qemu-devel] [PATCH 0/6] PCI hotplug improvements

2012-03-07 Thread Gleb Natapov
On Wed, Mar 07, 2012 at 12:51:48PM -0700, Alex Williamson wrote: > On Wed, 2012-03-07 at 20:59 +0200, Gleb Natapov wrote: > > On Wed, Mar 07, 2012 at 10:20:49AM -0700, Alex Williamson wrote: > > > On Wed, 2012-03-07 at 14:43 +0200, Gleb Natapov wrote: > > > > On Tue, Mar 06, 2012 at 05:13:36PM -070

Re: [Qemu-devel] [PATCH 3/6] pci: Add notifier for device probing

2012-03-07 Thread Alex Williamson
On Wed, 2012-03-07 at 10:19 +0100, Paolo Bonzini wrote: > Il 07/03/2012 01:14, Alex Williamson ha scritto: > > +void pci_host_remove_dev_probe_notifier(Notifier *notify) > > +{ > > +notifier_list_remove(&pci_host_dev_probe_notifiers, notify); > > +} > > + > > Looks like qemu-kvm needs a merge.

Re: [Qemu-devel] QAPI conversion status and async commands support

2012-03-07 Thread Alon Levy
On Wed, Mar 07, 2012 at 11:36:06AM -0600, Anthony Liguori wrote: > On 03/07/2012 11:29 AM, Paolo Bonzini wrote: > >Il 07/03/2012 17:36, Luiz Capitulino ha scritto: > >>Hi there, > >> > >>In the last few weeks we've had some proposals for new QMP commands that > >>need > >>to be asynchronous. As we

Re: [Qemu-devel] XP install cores with SCSI LSI 53C895A disks

2012-03-07 Thread Gerhard Wiesinger
On Wed, 7 Mar 2012, Brian Jackson wrote: I think most people trying to use qemu for anything useful have given up on if=scsi. Some distros even disable support because they don't want to QA it. That should be a decent sign that you may want to avoid it. OK, but SAS (Serial attached SCSI) is tec

Re: [Qemu-devel] [PATCH 0/6] PCI hotplug improvements

2012-03-07 Thread Alex Williamson
On Wed, 2012-03-07 at 20:59 +0200, Gleb Natapov wrote: > On Wed, Mar 07, 2012 at 10:20:49AM -0700, Alex Williamson wrote: > > On Wed, 2012-03-07 at 14:43 +0200, Gleb Natapov wrote: > > > On Tue, Mar 06, 2012 at 05:13:36PM -0700, Alex Williamson wrote: > > > > Here's a re-work of the patch that adde

Re: [Qemu-devel] [PATCH 1/2] core dump: drop VM_ALWAYSDUMP flag

2012-03-07 Thread Roland McGrath
> well, sure, we can provide an arch interface, for this check. I'm more > concerned with the general idea. If it seems ok, I can re-do this bit > with an arch interface. Anything based on strcmp is dismal. VM_ALWAYSDUMP was nice and clean. A hook along the lines of arch_vma_name would be clean e

Re: [Qemu-devel] [PATCH 08/20] memory: store MemoryRegionSection pointers in phys_map

2012-03-07 Thread Peter Maydell
On 7 March 2012 17:49, Peter Maydell wrote: > git bisect blames this commit (5312bd8b3) for causing a Linux kernel > on spitz to produce a bunch of pxa2xx_i2c warnings that weren't > being emitted before: What seems to happen here is that we register a memory region (this is for the second i2c de

Re: [Qemu-devel] [PATCH 0/6] PCI hotplug improvements

2012-03-07 Thread Gleb Natapov
On Wed, Mar 07, 2012 at 10:20:49AM -0700, Alex Williamson wrote: > On Wed, 2012-03-07 at 14:43 +0200, Gleb Natapov wrote: > > On Tue, Mar 06, 2012 at 05:13:36PM -0700, Alex Williamson wrote: > > > Here's a re-work of the patch that added _STA for the purpose of > > > using it as an ack from the gue

Re: [Qemu-devel] QAPI conversion status and async commands support

2012-03-07 Thread Luiz Capitulino
On Wed, 07 Mar 2012 11:36:06 -0600 Anthony Liguori wrote: > On 03/07/2012 11:29 AM, Paolo Bonzini wrote: > > Il 07/03/2012 17:36, Luiz Capitulino ha scritto: > >> Hi there, > >> > >> In the last few weeks we've had some proposals for new QMP commands that > >> need > >> to be asynchronous. As we

[Qemu-devel] [PATCH 3/3] virtio-blk: always enable VIRTIO_BLK_F_SCSI

2012-03-07 Thread Paolo Bonzini
VIRTIO_BLK_F_SCSI is supposed to mean whether the host can *parse* SCSI requests, not *execute* them. We need to change this to fix a migration compatibility problem related to the version of *management* (hence unrelated to the QEMU machine type), so we must do this unconditionally even on older

[Qemu-devel] [PATCH 2/3] virtio-blk: define VirtIOBlkConf

2012-03-07 Thread Paolo Bonzini
Signed-off-by: Paolo Bonzini --- hw/s390-virtio-bus.c |7 +++ hw/s390-virtio-bus.h |4 ++-- hw/virtio-blk.c | 28 ++-- hw/virtio-blk.h |7 +++ hw/virtio-pci.c |8 +++- hw/virtio-pci.h |4 ++-- hw/virtio.h |

[Qemu-devel] [PATCH 0/3] decouple VIRTIO_BLK_F_SCSI from SG_IO support

2012-03-07 Thread Paolo Bonzini
VIRTIO_BLK_F_SCSI is supposed to mean whether the host can *parse* SCSI requests, not *execute* them. So it should always be enabled, and the scsi=on/off property tied to a separate configuration variable that is not guest visible. With this change, Linux has problems understanding failed request

[Qemu-devel] [PATCH 1/3] virtio-blk: report non-zero status when failing SG_IO requests

2012-03-07 Thread Paolo Bonzini
Linux really looks only at scsi->errors. Arguably it is their bug, but we can make it safe for older guests now. Signed-off-by: Paolo Bonzini --- hw/virtio-blk.c | 48 +++- 1 files changed, 23 insertions(+), 25 deletions(-) diff --git a/hw/virtio-b

Re: [Qemu-devel] [PATCH 1/2] core dump: drop VM_ALWAYSDUMP flag

2012-03-07 Thread Roland McGrath
On Wed, Mar 7, 2012 at 9:00 AM, Jason Baron wrote: > +       if (vma_name) { > +               if ((strcmp(vma_name, "[vdso]") == 0) || > +                   (strcmp(vma_name, "[vectors]") == 0) || > +                   (strcmp(vma_name, "[vsyscall]") == 0)) That's just disgusting.

Re: [Qemu-devel] [PATCH 08/20] memory: store MemoryRegionSection pointers in phys_map

2012-03-07 Thread Peter Maydell
On 14 February 2012 09:27, Avi Kivity wrote: > Instead of storing PhysPageDesc, store pointers to MemoryRegionSections. > The various offsets (phys_offset & ~TARGET_PAGE_MASK, > PHYS_OFFSET & TARGET_PAGE_MASK, region_offset) can all be synthesized > from the information in a MemoryRegionSection.  

Re: [Qemu-devel] [PATCH v2 1/2] qapi-schema: fix typos and explain 'spice' auth

2012-03-07 Thread Luiz Capitulino
On Mon, 5 Mar 2012 15:13:27 +0200 Alon Levy wrote: > Signed-off-by: Alon Levy Applied both patches to the qmp branch, thanks. > --- > qapi-schema.json | 14 -- > 1 files changed, 8 insertions(+), 6 deletions(-) > > v1->v2: > - Fixes wrong indentation for second spice channel-

Re: [Qemu-devel] [PATCH RESEND v5 3/6] Set runstate to INMIGRATE earlier

2012-03-07 Thread Luiz Capitulino
On Tue, 28 Feb 2012 15:51:32 + Stefano Stabellini wrote: > Set runstate to RUN_STATE_INMIGRATE as soon as we can on resume. > > Signed-off-by: Stefano Stabellini Would be nice to explain why, but I suggested this myself so: Acked-by: Luiz Capitulino > --- > vl.c |2 +- > 1 files c

Re: [Qemu-devel] PC BIOS Flash emulation

2012-03-07 Thread Jordan Justen
On Wed, Mar 7, 2012 at 08:51, Dmitry Zamaruev wrote: > On Wed, Mar 7, 2012 at 6:32 PM, Avi Kivity wrote: >> On 03/07/2012 03:59 PM, Dmitry Zamaruev wrote: >>> I'm trying to emulate BIOS Flash chip (something Intel FWH compatible) >>> on PC target (i386-softmmu) and to verify possibility of such >

Re: [Qemu-devel] [RFC][PATCH 00/16 v8] introducing a new, dedicated memory dump mechanism

2012-03-07 Thread Luiz Capitulino
On Mon, 5 Mar 2012 21:41:02 -0300 Luiz Capitulino wrote: > On Mon, 05 Mar 2012 17:12:00 +0800 > Wen Congyang wrote: > > > At 03/02/2012 05:59 PM, Wen Congyang Wrote: > > > Hi, all > > > > > > 'virsh dump' can not work when host pci device is used by guest. We have > > > discussed this issue he

Re: [Qemu-devel] QAPI conversion status and async commands support

2012-03-07 Thread Anthony Liguori
On 03/07/2012 11:29 AM, Paolo Bonzini wrote: Il 07/03/2012 17:36, Luiz Capitulino ha scritto: Hi there, In the last few weeks we've had some proposals for new QMP commands that need to be asynchronous. As we lack a standard asynchronous API today, each command ends up adding its own way to exec

Re: [Qemu-devel] QAPI conversion status and async commands support

2012-03-07 Thread Paolo Bonzini
Il 07/03/2012 17:36, Luiz Capitulino ha scritto: > Hi there, > > In the last few weeks we've had some proposals for new QMP commands that need > to be asynchronous. As we lack a standard asynchronous API today, each command > ends up adding its own way to execute in the background. > > This multi

Re: [Qemu-devel] PC BIOS Flash emulation

2012-03-07 Thread Avi Kivity
On 03/07/2012 03:59 PM, Dmitry Zamaruev wrote: > Hi, > > I'm trying to emulate BIOS Flash chip (something Intel FWH compatible) > on PC target (i386-softmmu) and to verify possibility of such > emulation I first tried to use pflash_cfi02 example from PPC platform. So is Jordan (copied). > Trying

Re: [Qemu-devel] [PATCH] configure: Quote the configure args printed in config.log

2012-03-07 Thread Peter Maydell
On 7 March 2012 17:14, Avi Kivity wrote: > This still breaks if an argument has a single quote.  bash's printf's %q > deals with this correctly, but not sure how portable it is. True, but we've lived with it for the code that writes to config-host.mak since 2008 (commit fd69fe2b)... -- PMM

Re: [Qemu-devel] [PATCH 0/6] PCI hotplug improvements

2012-03-07 Thread Alex Williamson
On Wed, 2012-03-07 at 14:43 +0200, Gleb Natapov wrote: > On Tue, Mar 06, 2012 at 05:13:36PM -0700, Alex Williamson wrote: > > Here's a re-work of the patch that added _STA for the purpose of > > using it as an ack from the guest. Instead of that, add a notifier > > for device access. Once the gue

Re: [Qemu-devel] [PATCH] configure: Quote the configure args printed in config.log

2012-03-07 Thread Eric Blake
On 03/07/2012 10:08 AM, Peter Maydell wrote: > If you look at the whole change, rather than just the > fragment Stefan quoted to comment on: > > # Print a helpful header at the top of config.log > echo "# QEMU configure log $(date)" >> config.log > -echo "# produced by $0 $*" >> config.log > +pr

Re: [Qemu-devel] [PATCH] configure: Quote the configure args printed in config.log

2012-03-07 Thread Avi Kivity
On 03/07/2012 02:16 PM, Peter Maydell wrote: > Use the same mechanism we use for printing the configure command > line to config-host.mak to print it to config.log. This fixes a > bug where the config.log version didn't quote arguments with spaces. > > # Print a helpful header at the top of conf

[Qemu-devel] QAPI conversion status and async commands support

2012-03-07 Thread Luiz Capitulino
Hi there, In the last few weeks we've had some proposals for new QMP commands that need to be asynchronous. As we lack a standard asynchronous API today, each command ends up adding its own way to execute in the background. This multiplies the API complexity as each command has to be implemented

Re: [Qemu-devel] [PATCH] configure: Quote the configure args printed in config.log

2012-03-07 Thread Peter Maydell
On 7 March 2012 12:40, Stefan Hajnoczi wrote: >> +printf "# Configured with:" >> config.log >> +printf " '%s'" "$0" "$@" >> config.log > > You use an interesting feature of printf(1)- it applies the (short) > format string as long as there are arguments.  I checked that this is > in the specificat

Re: [Qemu-devel] [PATCH] configure: Quote the configure args printed in config.log

2012-03-07 Thread Peter Maydell
On 7 March 2012 17:06, Eric Blake wrote: >>>  # Print a helpful header at the top of config.log >>>  echo "# QEMU configure log $(date)" >> config.log >>> -echo "# produced by $0 $*" >> config.log > > This left a trailing newline, > >>> +printf "# Configured with:" >> config.log >>> +printf " '%s'

Re: [Qemu-devel] [PATCH] configure: Quote the configure args printed in config.log

2012-03-07 Thread Eric Blake
On 03/07/2012 05:40 AM, Stefan Hajnoczi wrote: > On Wed, Mar 7, 2012 at 12:16 PM, Peter Maydell > wrote: >> Use the same mechanism we use for printing the configure command >> line to config-host.mak to print it to config.log. This fixes a >> bug where the config.log version didn't quote argument

Re: [Qemu-devel] regarding qcow2metadata

2012-03-07 Thread Mulyadi Santosa
Hi... On Wed, Mar 7, 2012 at 14:11, PANKAJ RAWAT wrote: > [root@t06 p]# ls -lh > total 1.4M > -rw-r--r-- 1 root root 8.1G Mar  7 12:43 guest > > is there is any way to allocate metadata to half size or predefined size ? > i.e 4GB will be allocated and rest 4GB can be allocated dynamically have y

[Qemu-devel] QEMU RX offload issues with net device frontend that does not support virtio extensions

2012-03-07 Thread Dmitry Fleytman
Hello, It looks like there is a problem in QEMU related to usage of frontends without virtio extensions. Normally on RX path QEMU receives packet metadata with offloads information in virtio header. In particular there is an information about checksumming state. For example, when packet is being

Re: [Qemu-devel] VirtIO 9p mount_tag (bogus?) limit of 32 bytes

2012-03-07 Thread M. Mohan Kumar
Hi Anthony, When I tried with ldconfig version 2.14.90, ldconfig successfully completed QEMU version: 1.0.50 Kernel version: 3.3.0-rc6+ Could you please try with recent ldconfig? On 02/22/2012 09:28 AM, C Anthony Risinger wrote: On Sat, Feb 18, 2012 at 11:38 AM, Aneesh Kumar K.V wrote: On

Re: [Qemu-devel] [PATCH] Remove type field in ModuleEntry as it's not used.

2012-03-07 Thread Andreas Färber
Am 07.03.2012 16:36, schrieb Chen Yufei: > Signed-off-by: Chen Yufei There's separate init_type_list[]s per type and the type field is not even initialized, so: Acked-by: Andreas Färber Thanks, Andreas > --- > module.c |1 - > 1 files changed, 0 insertions(+), 1 deletions(-) > > diff --

Re: [Qemu-devel] PC BIOS Flash emulation

2012-03-07 Thread Dmitry Zamaruev
On Wed, Mar 7, 2012 at 6:32 PM, Avi Kivity wrote: > On 03/07/2012 03:59 PM, Dmitry Zamaruev wrote: >> Hi, >> >> I'm trying to emulate BIOS Flash chip (something Intel FWH compatible) >> on PC target (i386-softmmu) and to verify possibility of such >> emulation I first tried to use pflash_cfi02 exa

[Qemu-devel] [PATCH v3] VMXNET3 paravirtual NIC device implementation

2012-03-07 Thread Dmitry Fleytman
Changes in V3: Fixed crash when net device that is used as network fronted has no virtio HDR support. Task offloads emulation for cases when net device that is used as network fronted has no virtio HDR support. Changes in V2: License text changed accoring to community suggestions

Re: [Qemu-devel] [PATCH] PPC: Fix large page support in TCG

2012-03-07 Thread Alexander Graf
On 03/03/2012 07:07 PM, Andreas Färber wrote: Am 03.03.2012 17:39, schrieb Nathan Whitehorn: Fix large page support in TCG. The old code would overwrite the large page table entry with the fake 4 KB one generated here whenever the ref/change bits were updated, causing it to point to the wrong ar

Re: [Qemu-devel] [PATCH] PPC: Add PIR register to POWER7 CPU

2012-03-07 Thread Alexander Graf
On 03/03/2012 07:06 PM, Andreas Färber wrote: Am 03.03.2012 17:36, schrieb Nathan Whitehorn: The POWER7 emulation is missing the Processor Identification Register, mandatory in recent POWER CPUs, that is required for SMP on at least some operating systems (e.g. FreeBSD) to function properly. Thi

[Qemu-devel] [PATCH] Remove type field in ModuleEntry as it's not used.

2012-03-07 Thread Chen Yufei
Signed-off-by: Chen Yufei --- module.c |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/module.c b/module.c index 106a969..c3a6da7 100644 --- a/module.c +++ b/module.c @@ -19,7 +19,6 @@ typedef struct ModuleEntry { -module_init_type type; void (*init)(void);

Re: [Qemu-devel] [RFC][PATCH 05/16 v8] Add API to get memory mapping

2012-03-07 Thread HATAYAMA Daisuke
From: Wen Congyang Subject: [RFC][PATCH 05/16 v8] Add API to get memory mapping Date: Fri, 02 Mar 2012 18:18:23 +0800 > Add API to get all virtual address and physical address mapping. > If there is no virtual address for some physical address, the virtual > address is 0. > > Signed-off-by: Wen

Re: [Qemu-devel] XP install cores with SCSI LSI 53C895A disks

2012-03-07 Thread Brian Jackson
On Wed, 07 Mar 2012 00:41:39 -0600, Gerhard Wiesinger wrote: Ping. Any comments? Thnx. Ciao, Gerhard -- http://www.wiesinger.com/ On Sun, 4 Mar 2012, Gerhard Wiesinger wrote: Hello, Clean XP install cores with SCSI LSI 53C89A disk when copying files. Reproduceable. Driver used is s

[Qemu-devel] [PATCH 13/20] usb: add USB_RET_IOERROR

2012-03-07 Thread Gerd Hoffmann
From: Hans de Goede We already have USB_RET_NAK, but that means that a device does not want to send/receive right now. But with host / network redirection we can actually have a transaction fail due to some io error, rather then ie the device just not having any data atm. This patch adds a new e

[Qemu-devel] [PATCH 02/13] sh7750: Remove redundant 'struct' from MemoryRegionOps

2012-03-07 Thread Stefan Hajnoczi
From: Stefan Weil The 'struct' is not needed, and all other MemoryRegionOps don't use it. Signed-off-by: Stefan Weil Signed-off-by: Stefan Hajnoczi --- hw/sh7750.c |2 +- hw/sh_intc.c |2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/hw/sh7750.c b/hw/sh7750.c inde

[Qemu-devel] [PATCH 08/20] usb-ehci: Fix cerr tracking

2012-03-07 Thread Gerd Hoffmann
From: Hans de Goede cerr should only be decremented on errors which cause XactErr to be set, and when that happens the failing transaction should be retried until cerr reaches 0 and only then should USBSTS_ERRINT be set (and inactive cleared and USBSTS_INT set if requested). Since we don't have

[Qemu-devel] [PATCH 16/20] usb: add pipelining option to usb endpoints

2012-03-07 Thread Gerd Hoffmann
With this patch applied USB drivers can enable pipelining per endpoint. With pipelining enabled the usb core will continue submitting packets even when there are still async transfers in flight instead of passing them on one by one. Signed-off-by: Gerd Hoffmann --- hw/usb.c | 11 ++- h

[Qemu-devel] [PATCH 14/20] uhci_fill_queue: zap debug printf

2012-03-07 Thread Gerd Hoffmann
--- hw/usb-uhci.c |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/hw/usb-uhci.c b/hw/usb-uhci.c index 2c6ed38..304b84b 100644 --- a/hw/usb-uhci.c +++ b/hw/usb-uhci.c @@ -951,7 +951,6 @@ static void uhci_fill_queue(UHCIState *s, UHCI_TD *td) UHCI_TD ptd; int ret;

[Qemu-devel] [PATCH 06/13] Fix sign of sscanf format specifiers

2012-03-07 Thread Stefan Hajnoczi
From: Stefan Weil All values read by sscanf are unsigned, so replace %d by %u. This signed / unsigned mismatch was detected by splint. Signed-off-by: Stefan Weil Signed-off-by: Stefan Hajnoczi --- cursor.c |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/cursor.c b/

[Qemu-devel] [PATCH 19/20] xhci: fix control xfers

2012-03-07 Thread Gerd Hoffmann
Use the new, direct control transfer submission method instead of bypassing the usb core by calling usb_device_handle_control directly. The later fails for async control transfers. This patch gets xhci + usb-host combo going. --- hw/usb-xhci.c | 13 + 1 files changed, 5 insertions(+

Re: [Qemu-devel] virtio-blk performance regression and qemu-kvm

2012-03-07 Thread Stefan Hajnoczi
On Wed, Mar 7, 2012 at 2:21 PM, Reeted wrote: > On 03/07/12 09:04, Stefan Hajnoczi wrote: >> >> On Tue, Mar 6, 2012 at 10:07 PM, Reeted  wrote: >>> >>> On 03/06/12 13:59, Stefan Hajnoczi wrote: BTW, I'll take the opportunity to say that 15.8 or 20.3 k IOPS are very low figures

[Qemu-devel] Re : Regression: more 0.12 regression (SeaBIOS related?)

2012-03-07 Thread Alain Ribière
I ran qemu 1.0.1 and the latest SeaBIOS (from the git) with the following options : qemu-system-i386 -L git/bios -fda disk.img -no-fd-bootchk -boot a -m 16 Here is the log : https://docs.google.com/open?id=0B7mz0vq6Rpb7UE1ibjJDcEhTRWlNV050QnMyMWwtZw Here is the floppy disk image I used : https

[Qemu-devel] [PATCH 10/13] Spelling fixes in comments (it's -> its)

2012-03-07 Thread Stefan Hajnoczi
From: Stefan Weil * it's -> its (fixed for all files) * dont -> don't (only fixed in a line which was touched by the previous fix) * distrub -> disturb (fixed in the same line) Reviewed-by: Andreas Färber Signed-off-by: Stefan Weil Signed-off-by: Stefan Hajnoczi --- configure |

Re: [Qemu-devel] Configuring QEMU to translate x86 instructions to native ARM instructions

2012-03-07 Thread Roger
Thank you both. The --CPU option worked like a charm. Roger On Mar 6, 2012, at 11:14 AM, Peter Maydell wrote: > On 6 March 2012 16:09, Roger wrote: >> I will try the --cross-prefix option. I have a feeling this >> will do it. I know that the prologue is being initialized with >> the i386

  1   2   >