[Qemu-devel] [PATCH v2 4/7] target-arm: Convert TCG to using (index, value) list for cp migration

2013-06-03 Thread Peter Maydell
Convert the TCG ARM target to using an (index,value) list for migrating coprocessors. The primary benefit of the (index,value) list is for passing state between KVM and QEMU, but it works for TCG-to-TCG migration as well and is a useful self-contained first step. Signed-off-by: Peter Maydell

[Qemu-devel] [PATCH] blockdev: reset werror/rerror on drive_del

2013-06-03 Thread Stefan Hajnoczi
Paolo Bonzini pbonz...@redhat.com suggested the following test case: 1. Launch a guest and wait at the GRUB boot menu: qemu-system-x86_64 -enable-kvm -m 1024 \ -drive if=none,cache=none,file=test.img,id=foo,werror=stop,rerror=stop -device virtio-blk-pci,drive=foo,id=virtio0,addr=4 2.

Re: [Qemu-devel] Could configure generate QEMU's linker scripts?

2013-06-03 Thread Peter Maydell
On 3 June 2013 15:23, Richard Henderson r...@twiddle.net wrote: On 06/02/2013 10:15 AM, Peter Maydell wrote: So for a new architecture how do we determine whether we need to fiddle with the start address or not? (More specifically, is aarch64 going to need a linker script or just to go in the

Re: [Qemu-devel] Could configure generate QEMU's linker scripts?

2013-06-03 Thread Richard Henderson
On 06/03/2013 07:57 AM, Peter Maydell wrote: Well, the output of ld --verbose says: PROVIDE (__executable_start = SEGMENT_START(text-segment, 0x40)); . = SEGMENT_START(text-segment, 0x40) + SIZEOF_HEADERS; and objdump of a random aarch64 binary suggests the text segment

Re: [Qemu-devel] [PATCH 0/2] linux-user: Drop direct use of openat etc syscalls

2013-06-03 Thread Richard Henderson
On 06/01/2013 03:35 PM, Peter Maydell wrote: Basically, removing the ugly direct syscall access seemed nicer than trying to fix up and render consistent the broken ifdefs :-) [RHEL5 has glibc2.5 and provides these functions. RHEL4 did not but we don't build on RHEL4 anyhow because its glib

Re: [Qemu-devel] [PATCH V2 0/2] Fix memory migration for exynos 4210 SoC

2013-06-03 Thread Peter Maydell
On 10 May 2013 16:32, Igor Mitsyanko i.mitsya...@gmail.com wrote: Fix issues in exynos4210 code which were blocking proper memory migration. Thanks, applied to arm-devs.next. -- PMM

Re: [Qemu-devel] [PATCH v2] i.MX: Improve EPIT timer code.

2013-06-03 Thread Peter Maydell
On 30 May 2013 00:15, pe...@chubb.wattle.id.au wrote: Jean-Christophe wrote: * Unify function and type naming * use dynamic cast whenever possible * simplify Debug printf. * use new style device intialization. Signed-off-by: Jean-Christophe DUBOIS j...@tribudubois.net Reviewed-by: Peter

[Qemu-devel] [PATCH 0/2] gdbstub runstate check follow-ups

2013-06-03 Thread Paolo Bonzini
My patch committed at 87f25c12bfeaaa0c41fb857713bbc7e8a9b757dc was broken. These patches fix the problem in a better way. Paolo Paolo Bonzini (2): gdbstub: fix for commit 87f25c12bfeaaa0c41fb857713bbc7e8a9b757dc gdbstub: let the debugger resume from guest panicked state gdbstub.c | 5

[Qemu-devel] [PATCH 2/2] gdbstub: let the debugger resume from guest panicked state

2013-06-03 Thread Paolo Bonzini
While in general we forbid a continue from the guest panicked state, it makes sense to have an exception for that when continuing in the debugger. Perhaps the guest entered that state due to a bug, for example, and we want to continue no matter what. Signed-off-by: Paolo Bonzini

[Qemu-devel] [PATCH 1/2] gdbstub: fix for commit 87f25c12bfeaaa0c41fb857713bbc7e8a9b757dc

2013-06-03 Thread Paolo Bonzini
This commit used the wrong check to prevent an assertion failure. After this commit, you need to start a guest in the monitor, you cannot use anymore the c command in the debugger. This is undesirable. The commit's aim was to prevent a restart after a KVM internal error or something like that;

[Qemu-devel] [Bug 1186984] Re: large -initrd crashes qemu

2013-06-03 Thread Richard Jones
One way to reproduce this is to just use a large (200 MB) completely random initrd. Note this error seems to happen a long time before even the kernel starts up, so the actual content of the initrd doesn't matter. dd if=/dev/urandom of=/tmp/initrd bs=1M count=200 qemu-system-x86_64 -kernel

Re: [Qemu-devel] [PATCH v5 1/4] Add i.MX FEC Ethernet emulator

2013-06-03 Thread Peter Maydell
On 8 May 2013 09:28, Jean-Christophe DUBOIS j...@tribudubois.net wrote: --- a/include/hw/arm/imx.h +++ b/include/hw/arm/imx.h @@ -23,12 +23,10 @@ typedef enum { uint32_t imx_clock_frequency(DeviceState *s, IMXClk clock); -void imx_timerp_create(const hwaddr addr, -

Re: [Qemu-devel] [PATCH v5 2/4] Add i.MX I2C controller emulator

2013-06-03 Thread Peter Maydell
On 8 May 2013 09:28, Jean-Christophe DUBOIS j...@tribudubois.net wrote: +/* + *if we unset the master mode then it ends the ongoing + * transfer if any + */ nit: missing space between '*' and if'. thanks -- PMM

[Qemu-devel] [Bug 1186984] Re: large -initrd crashes qemu

2013-06-03 Thread Richard Jones
OK I see what's happening. Because I forgot about the -m option, qemu allocates 128 MB of RAM. It's obviously wrapping around in memory and overwriting all the low memory. If you add (eg) -m 1024 it works. -- You received this bug notification because you are a member of qemu- devel-ml, which

Re: [Qemu-devel] [PATCH] linux-user: Allow getdents to be provided by getdents64

2013-06-03 Thread Richard Henderson
On 06/02/2013 10:10 AM, Peter Maydell wrote: +tde-d_ino = tswapal(ino); +tde-d_off = tswapal(off); +tde-d_reclen = tswap16(treclen); +memmove(tde-d_name, de-d_name, namelen + 1); Wouldn't it be better to do the

Re: [Qemu-devel] [PATCH v5 3/4] Add i.MX25 3DS evaluation board support.

2013-06-03 Thread Peter Maydell
On 8 May 2013 09:28, Jean-Christophe DUBOIS j...@tribudubois.net wrote: For now we support: * timers (GPT and EPIT) * serial ports * ethernet (through the newly added FEC emulator) * I2C (through the newly added I2C emulator) +/* add I2C 0 */ +i2c_dev =

[Qemu-devel] [PATCH 0/3] Remove legacy sysfw code

2013-06-03 Thread Paolo Bonzini
The sysfw code to choose between ROM and flash BIOS was a bad idea, because it triggered different behavior between TCG and KVM. We deleted the behavior in 1.5, but we left the code around because it was close to the release. Now it's time to delete it. Paolo Bonzini (3): remove read-only

[Qemu-devel] [PATCH 1/3] sysfw: remove read-only pc_sysfw_flash_vs_rom_bug_compatible

2013-06-03 Thread Paolo Bonzini
The variable is not written anymore. Signed-off-by: Paolo Bonzini pbonz...@redhat.com --- hw/block/pc_sysfw.c | 26 +- 1 file changed, 1 insertion(+), 25 deletions(-) diff --git a/hw/block/pc_sysfw.c b/hw/block/pc_sysfw.c index 412d1b0..c6d4be4 100644 ---

[Qemu-devel] [PATCH 2/3] pc_sysfw: remove the rom_only property

2013-06-03 Thread Paolo Bonzini
With the new semantics of pc_sysfw (no -pflash implies old-style ROM setup, -pflash implies new-style ROM setup), there is no need anymore for a compat property. Old machines simply will never use -pflash, and thus will always use old-style setup. Signed-off-by: Paolo Bonzini pbonz...@redhat.com

[Qemu-devel] [PATCH 3/3] pc_sysfw: do not make it a device anymore

2013-06-03 Thread Paolo Bonzini
Move the code to hw/i386, the sole remaining property is available as !pci_enabled. Signed-off-by: Paolo Bonzini pbonz...@redhat.com --- default-configs/i386-softmmu.mak | 1 - default-configs/x86_64-softmmu.mak | 1 - hw/block/Makefile.objs | 1 - hw/i386/Makefile.objs

Re: [Qemu-devel] [PATCH v5 4/4] Add qtest support for i.MX I2C device emulation.

2013-06-03 Thread Peter Maydell
On 8 May 2013 09:28, Jean-Christophe DUBOIS j...@tribudubois.net wrote: This is using a ds1338 RTC chip on the i2c bus. This RTC chip is not present on the real board. Ideally this should work by the test driver passing a -device ds1338,something,something argument to instantiate the RTC and

[Qemu-devel] [PATCH] e1000: cleanup process_tx_desc

2013-06-03 Thread Andrew Jones
Coverity complains about two overruns in process_tx_desc(). The complaints are false positives, but we might as well eliminate them. The problem is that hdr is defined as an unsigned int, but then used to offset an array of size 65536, and another of size 256 bytes. hdr will actually never be

Re: [Qemu-devel] [PATCH v5 0/4] Add i.MX25 support through the 3DS evaluation board

2013-06-03 Thread Peter Maydell
On 12 May 2013 14:14, Peter Maydell peter.mayd...@linaro.org wrote: On 12 May 2013 13:55, Jean-Christophe DUBOIS j...@tribudubois.net wrote: In your opinion, is this patch set good enough in its actual state for inclusion? Are you expecting anything else from me (beside maybe adding more i.MX

Re: [Qemu-devel] [PATCH 0/2] gdbstub runstate check follow-ups

2013-06-03 Thread Luiz Capitulino
On Mon, 3 Jun 2013 17:06:53 +0200 Paolo Bonzini pbonz...@redhat.com wrote: My patch committed at 87f25c12bfeaaa0c41fb857713bbc7e8a9b757dc was broken. These patches fix the problem in a better way. Looks good to me: Reviewed-by: Luiz Capitulino lcapitul...@redhat.com

Re: [Qemu-devel] [PATCH] blockdev: reset werror/rerror on drive_del

2013-06-03 Thread Paolo Bonzini
Il 03/06/2013 16:58, Stefan Hajnoczi ha scritto: Paolo Bonzini pbonz...@redhat.com suggested the following test case: 1. Launch a guest and wait at the GRUB boot menu: qemu-system-x86_64 -enable-kvm -m 1024 \ -drive if=none,cache=none,file=test.img,id=foo,werror=stop,rerror=stop

[Qemu-devel] [Bug 1186984] Re: large -initrd can wrap around in memory causing memory corruption

2013-06-03 Thread Richard Jones
** Summary changed: - large -initrd crashes qemu + large -initrd can wrap around in memory causing memory corruption -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1186984 Title: large -initrd can

Re: [Qemu-devel] [PATCH] linux-user: Allow getdents to be provided by getdents64

2013-06-03 Thread Peter Maydell
On 3 June 2013 16:15, Richard Henderson r...@twiddle.net wrote: On 06/02/2013 10:10 AM, Peter Maydell wrote: +tde-d_ino = tswapal(ino); +tde-d_off = tswapal(off); +tde-d_reclen = tswap16(treclen); +

Re: [Qemu-devel] [PATCH] s390/ipl: Fix spurious errors in virtio

2013-06-03 Thread Anthony Liguori
Applied. Thanks. Regards, Anthony Liguori

Re: [Qemu-devel] [PATCH v2] po/hu.po: Hungarian translation for the GTK+ interface

2013-06-03 Thread Anthony Liguori
Applied. Thanks. Regards, Anthony Liguori

[Qemu-devel] [PATCH 0/2] Fix NBD hostname parsing issues

2013-06-03 Thread Ján Tomko
Fix parsing of NBD URIs with IPv6 addresses, broken by v1.4.0-736-gf17c90b and fix parsing of NBD filenames with hostnames starting with a digit (fixed by the same commit, but only for NBD URIs): https://lists.gnu.org/archive/html/qemu-devel/2013-05/msg04719.html Ján Tomko (2): qemu-socket:

[Qemu-devel] [PATCH 1/2] qemu-socket: allow hostnames starting with a digit

2013-06-03 Thread Ján Tomko
According to RFC 1123 [1], hostnames can start with a digit too. [1] http://tools.ietf.org/html/rfc1123#page-13 Signed-off-by: Ján Tomko jto...@redhat.com --- util/qemu-sockets.c | 13 - 1 file changed, 4 insertions(+), 9 deletions(-) diff --git a/util/qemu-sockets.c

[Qemu-devel] [PATCH 2/2] nbd: strip braces from literal IPv6 address in URI

2013-06-03 Thread Ján Tomko
Otherwise they would get passed to getaddrinfo and fail with: address resolution failed for [::1]:1234: Name or service not known (Broken by commit v1.4.0-736-gf17c90b) Signed-off-by: Ján Tomko jto...@redhat.com --- block/nbd.c | 11 ++- 1 file changed, 10 insertions(+), 1 deletion(-)

Re: [Qemu-devel] [PATCH] linux-user: Allow getdents to be provided by getdents64

2013-06-03 Thread Richard Henderson
On 06/03/2013 08:45 AM, Peter Maydell wrote: +/* The target_dirent type is in what was formerly a padding + * byte at the end of the structure: + */ +*(((char *)tde) + treclen - 1) = type; Maybe easier

Re: [Qemu-devel] [PATCH 4/4] tcg/aarch64: implement tlb lookup fast path

2013-06-03 Thread Richard Henderson
On 06/03/2013 04:21 AM, Jani Kokkonen wrote: @ merge the tlb index into the env+tlb_offset add x1, x1, x0, lsl #3 For the record, oops. 3 should be CPU_TLB_ENTRY_BITS. r~

[Qemu-devel] [PATCH] target-i386: cpu: fix potential buffer overrun in get_register_name_32()

2013-06-03 Thread Igor Mammedov
spotted by Coverity, x86_reg_info_32[] is CPU_NB_REGS32 elements long, so accessing x86_reg_info_32[CPU_NB_REGS32] will be one element off array. Signed-off-by: Igor Mammedov imamm...@redhat.com --- target-i386/cpu.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git

[Qemu-devel] [Bug 1186935] Re: [1.5] QEMU monitor gets overlapped by GTK menu bar

2013-06-03 Thread Lekensteyn
This seems to be a bug when building against gtk3. When building against gtk 3.8.2: - monitor text gets hidden behind menu bar - a bar appears on the bottom, growing as the window is resized. When the contents overflows (a scrollbar appears), this bar is gone. Building against gtk 2.24.18 / vte

[Qemu-devel] VFIO and scheduled SR-IOV cards

2013-06-03 Thread Benoît Canet
Hello, I plan to write a PF driver for an SR-IOV card and make the VFs work with QEMU's VFIO passthrough so I am asking the following design question before trying to write and push code. After SR-IOV being enabled on this hardware only one VF function can be active at a given time. The PF

[Qemu-devel] [PULL 09/24] xilinx_spips: lqspi: Dont touch config register

2013-06-03 Thread Peter Maydell
From: Peter Crosthwaite peter.crosthwa...@xilinx.com The LQSPI mode is supposed to work via the automatic CS mode feature rather than manipulate CS lines itself. Now that auto CS is implemented remove LQSPIs CS mode override logic. There is still a need to manipulate the U_PAGE bit in LQSPI

[Qemu-devel] [PULL 10/24] xilinx_spips: Fix CTRL register RW bits

2013-06-03 Thread Peter Maydell
From: Peter Crosthwaite peter.crosthwa...@xilinx.com The CTRL register was RAZ/WI on some of the RW bits. Even though the function behind these bits is invalid in QEMU, they should still be guest accessible. Fix. Signed-off-by: Peter Crosthwaite peter.crosthwa...@xilinx.com Reviewed-by: Peter

Re: [Qemu-devel] [PATCH v19 0/7] Add pvpanic device to deal with guest panic event

2013-06-03 Thread Anthony Liguori
Applied. Thanks. Regards, Anthony Liguori

[Qemu-devel] [PULL 14/24] xilinx_spips: lqspi: Push more data to tx-fifo

2013-06-03 Thread Peter Maydell
From: Peter Crosthwaite peter.crosthwa...@xilinx.com Do 16 words per fifo flush. Increases performance and decreases debug verbosity. This data depth has no real hardware analogue, so just go with something that has reasonable performance. Signed-off-by: Peter Crosthwaite

Re: [Qemu-devel] [PATCH] linux-user: Allow getdents to be provided by getdents64

2013-06-03 Thread Peter Maydell
On 3 June 2013 16:58, Richard Henderson r...@twiddle.net wrote: On 06/03/2013 08:45 AM, Peter Maydell wrote: Dunno. It's not actually a char array, so I kind of prefer to use plain pointer arithmetic for this kind of thing. Then drop the unnecessary parenthesis *((char *)tde + trelen - 1)

[Qemu-devel] [PULL 24/24] i.MX: Improve EPIT timer code.

2013-06-03 Thread Peter Maydell
From: Jean-Christophe DUBOIS j...@tribudubois.net * Unify function and type naming * use dynamic cast whenever possible * simplify Debug printf. * use new style device intialization. Signed-off-by: Jean-Christophe DUBOIS j...@tribudubois.net Reviewed-by: Peter Chubb peter.ch...@nicta.com.au

[Qemu-devel] [PULL 12/24] xilinx_spips: Debug msgs for Snoop state

2013-06-03 Thread Peter Maydell
From: Peter Crosthwaite peter.crosthwa...@xilinx.com This is worth keeping track of when debugging the device model. Signed-off-by: Peter Crosthwaite peter.crosthwa...@xilinx.com Reviewed-by: Peter Maydell peter.mayd...@linaro.org Reviewed-by: Edgar E. Iglesias edgar.igles...@gmail.com

[Qemu-devel] [PULL 06/24] xilinx_spips: Trash LQ page cache on mode change

2013-06-03 Thread Peter Maydell
From: Peter Crosthwaite peter.crosthwa...@xilinx.com Invalidate the LQSPI cached page when transitioning into LQSPI mode. Otherwise there is a possibility that the controller will return stale data to the guest when transitioning back to LQ_MODE after a page program. Signed-off-by: Peter

[Qemu-devel] [PULL 13/24] xilinx_spips: Multiple debug verbosity levels

2013-06-03 Thread Peter Maydell
From: Peter Crosthwaite peter.crosthwa...@xilinx.com The debug printfs on every SPI operation is extremely verbose. Add a second level of debug for this. Signed-off-by: Peter Crosthwaite peter.crosthwa...@xilinx.com Reviewed-by: Peter Maydell peter.mayd...@linaro.org Reviewed-by: Edgar E.

[Qemu-devel] [PULL 15/24] xilinx_spips: lqspi: Fix byte/misaligned access

2013-06-03 Thread Peter Maydell
From: Peter Crosthwaite peter.crosthwa...@xilinx.com The LQSPI bus attachment supports byte/halfword and misaligned accesses. Fixed. Refactored the LQSPI cache to be byte-wise instead of word wise accordingly. Signed-off-by: Peter Crosthwaite peter.crosthwa...@xilinx.com Reviewed-by: Peter

[Qemu-devel] [PULL 19/24] sd/sdhci:ADMA: fix interrupt

2013-06-03 Thread Peter Maydell
From: Peter Crosthwaite peter.crosthwa...@xilinx.com The end of transfer check was occurring and potentially returning before the interrupt flag was checked. This means the interrupt will be missed if it occurs on the last packet. Fix by checking for the interrupt before checking for the end of

[Qemu-devel] [GIT PULL] Xen fixes and cleanups 20130603

2013-06-03 Thread Stefano Stabellini
Anthony, please pull the following changes since commit 6a4e17711442849bf2cc731ccddef5a2a2d92d29:

[Qemu-devel] [PULL 23/24] exynos4210.c: register rom_mem for memory migration

2013-06-03 Thread Peter Maydell
From: Igor Mitsyanko i.mitsya...@samsung.com Even if we do not register newly created RAM MemoryRegion for migration with vmstate_register_ram_global() function, ram_save_setup() still saves this region to snapshot file with empty idstr==. Consequently this results in error during VM loading in

[Qemu-devel] [PULL 16/24] sd/sdhci.c: Only reset data_count on new commands

2013-06-03 Thread Peter Maydell
From: Peter Crosthwaite peter.crosthwa...@xilinx.com The data_count variable was being reset on every transfer, including DMA transfer resumptions. This is incorrect, it should only be set on a new command. Manifests as a bug when using ADMA and there is a timer delay between ADMA frames where

Re: [Qemu-devel] [PATCH v5 06/11] block: add drive-backup QMP command

2013-06-03 Thread Eric Blake
On 05/30/2013 06:34 AM, Stefan Hajnoczi wrote: @drive-backup Start a point-in-time copy of a block device to a new destination. The status of ongoing drive-backup operations can be checked with query-block-jobs where the BlockJobInfo.type field has the value 'backup'. The operation can be

[Qemu-devel] [Bug 1186935] Re: [1.5] QEMU monitor gets overlapped by GTK menu bar

2013-06-03 Thread Lekensteyn
** Changed in: qemu Status: New = Confirmed -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1186935 Title: [1.5] QEMU monitor gets overlapped by GTK menu bar Status in QEMU: Confirmed Bug

Re: [Qemu-devel] [PATCH v5 04/11] blockdev: drop redundant proto_drv check

2013-06-03 Thread Eric Blake
On 05/30/2013 06:34 AM, Stefan Hajnoczi wrote: It is not necessary to check that we can find a protocol block driver since we create or open the image file. This produces the error that we need anyway. Besides, the QERR_INVALID_BLOCK_FORMAT is inappropriate since the protocol is incorrect

Re: [Qemu-devel] [PATCH v5 05/11] blockdev: use bdrv_getlength() in qmp_drive_mirror()

2013-06-03 Thread Eric Blake
On 05/30/2013 06:34 AM, Stefan Hajnoczi wrote: Use bdrv_getlength() for its byte units and error return instead of bdrv_get_geometry(). Reported-by: Kevin Wolf kw...@redhat.com Signed-off-by: Stefan Hajnoczi stefa...@redhat.com --- blockdev.c | 10 +++--- 1 file changed, 7

[Qemu-devel] [PATCH 1/2] MAINTAINERS: new maintainers for qapi-schema.json

2013-06-03 Thread Luiz Capitulino
I'm facing two problems lately wrt QMP patch review: increasingly lack of bandwidth and lack of background in so many different areas that are getting new QMP commands almost every week. In order to help me mitigate this problem, I'm adding Eric and Markus (besides me) as maintainers of the

[Qemu-devel] [PATCH 0/2] MAINTAINERS: QMP: entry update and new maintainers

2013-06-03 Thread Luiz Capitulino
Hi, This is an small update on QMP/HMP/QAPI mainternership. Please, check individual patches for details. Luiz Capitulino (2): MAINTAINERS: new maintainers for qapi-schema.json MAINTAINERS: split Monitor (QMP/HMP) entry MAINTAINERS | 24 ++-- 1 file changed, 22

Re: [Qemu-devel] [PATCH v2] qemu-char: don't issue CHR_EVENT_OPEN in a BH

2013-06-03 Thread Anthony Liguori
Michael Roth mdr...@linux.vnet.ibm.com writes: When CHR_EVENT_OPEN was initially added, it was CHR_EVENT_RESET, and it was issued as a bottom-half: 86e94dea5b740dad65446c857f6959eae43e0ba6 Which we basically used to print out a greeting/prompt for the monitor. AFAICT the only reason this

Re: [Qemu-devel] [PATCH 0/2] configure: improve handling of target lists

2013-06-03 Thread Peter Maydell
On 20 May 2013 16:16, Peter Maydell peter.mayd...@linaro.org wrote: These patches clean up configure's handling of the list of known targets. Firstly we autogenerate the default list based on the set of default-config files; this reduces by 1 the number of places that need editing when a new

[Qemu-devel] [Bug 1186935] Re: [1.5] QEMU monitor gets overlapped by GTK menu bar

2013-06-03 Thread Lekensteyn
gtk 3.8.2 -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1186935 Title: [1.5] QEMU monitor gets overlapped by GTK menu bar Status in QEMU: New Bug description: The QEMU minitor gets partially

[Qemu-devel] [PATCH 2/2] MAINTAINERS: split Monitor (QMP/HMP) entry

2013-06-03 Thread Luiz Capitulino
This entry doesn't reflect reality for a few years now. This commit splits it into Human Monitor (HMP), QAPI and QMP. Markus is dropped as a maintainer. This is what we have been for the last few years. Also, it's going to help me to offload some of this work to someone else in the near future.

Re: [Qemu-devel] [PATCH v5 09/11] blockdev: add DriveBackup transaction

2013-06-03 Thread Eric Blake
On 05/30/2013 06:34 AM, Stefan Hajnoczi wrote: This patch adds a transactional version of the drive-backup QMP command. It allows atomic snapshots of multiple drives along with automatic cleanup if there is a failure to start one of the backup jobs. Note that QMP events are emitted for block

[Qemu-devel] QEMU aborts since kvm: support using KVM_MEM_READONLY flag for regions

2013-06-03 Thread Peter Wu
Hi, With the current git source (f10acc8b38d65a66ffa0588a036489d7fa6a593e), `qemu-system-x86_64 -enable-kvm` fails to run with the following error: kvm_set_phys_mem: error unregistering overlapping slot: Invalid argument After this message, qemu aborts. Reverting the following commit on top

[Qemu-devel] [PULL 03/24] xilinx_spips: Inhibit interrupts in LQSPI mode

2013-06-03 Thread Peter Maydell
From: Peter Crosthwaite peter.crosthwa...@xilinx.com The real hardware does not produce interrupts in LQSPI mode. Inhibit generation of interrupts when the LQ_MODE bit is set. Signed-off-by: Peter Crosthwaite peter.crosthwa...@xilinx.com Reviewed-by: Peter Maydell peter.mayd...@linaro.org

Re: [Qemu-devel] [PATCH 2/2] MAINTAINERS: split Monitor (QMP/HMP) entry

2013-06-03 Thread Eric Blake
On 06/03/2013 11:17 AM, Luiz Capitulino wrote: This entry doesn't reflect reality for a few years now. This commit splits it into Human Monitor (HMP), QAPI and QMP. Markus is dropped as a maintainer. This is what we have been for the last few years. Also, it's going to help me to offload

Re: [Qemu-devel] [PATCH v2] qemu-char: don't issue CHR_EVENT_OPEN in a BH

2013-06-03 Thread mdroth
On Mon, Jun 03, 2013 at 12:17:00PM -0500, Anthony Liguori wrote: Michael Roth mdr...@linux.vnet.ibm.com writes: When CHR_EVENT_OPEN was initially added, it was CHR_EVENT_RESET, and it was issued as a bottom-half: 86e94dea5b740dad65446c857f6959eae43e0ba6 Which we basically used to

Re: [Qemu-devel] [PATCH 0/2] MAINTAINERS: QMP: entry update and new maintainers

2013-06-03 Thread Eric Blake
On 06/03/2013 11:17 AM, Luiz Capitulino wrote: Hi, This is an small update on QMP/HMP/QAPI mainternership. Please, check individual patches for details. Luiz Capitulino (2): MAINTAINERS: new maintainers for qapi-schema.json MAINTAINERS: split Monitor (QMP/HMP) entry Series:

Re: [Qemu-devel] [PATCH 2/2] MAINTAINERS: split Monitor (QMP/HMP) entry

2013-06-03 Thread Luiz Capitulino
On Mon, 03 Jun 2013 11:24:58 -0600 Eric Blake ebl...@redhat.com wrote: On 06/03/2013 11:17 AM, Luiz Capitulino wrote: This entry doesn't reflect reality for a few years now. This commit splits it into Human Monitor (HMP), QAPI and QMP. Markus is dropped as a maintainer. This is what

[Qemu-devel] [PULL 00/24] arm-devs queue

2013-06-03 Thread Peter Maydell
Hi; here's the latest arm-devs pullreq. I had a lot of cc'd mail to wade through putting this together so I wouldn't be surprised if I missed a patch somewhere; if so please ping the relevant patch. Please pull. thanks -- PMM The following changes since commit

Re: [Qemu-devel] [PATCH 0/2] MAINTAINERS: QMP: entry update and new maintainers

2013-06-03 Thread Luiz Capitulino
On Mon, 03 Jun 2013 11:27:51 -0600 Eric Blake ebl...@redhat.com wrote: On 06/03/2013 11:17 AM, Luiz Capitulino wrote: Hi, This is an small update on QMP/HMP/QAPI mainternership. Please, check individual patches for details. Luiz Capitulino (2): MAINTAINERS: new maintainers for

Re: [Qemu-devel] QEMU aborts since kvm: support using KVM_MEM_READONLY flag for regions

2013-06-03 Thread Jordan Justen
This should be fixed by Xiao's fix double free the memslot in kvm_set_phys_mem patch. On Mon, Jun 3, 2013 at 10:23 AM, Peter Wu lekenst...@gmail.com wrote: Hi, With the current git source (f10acc8b38d65a66ffa0588a036489d7fa6a593e), `qemu-system-x86_64 -enable-kvm` fails to run with the

Re: [Qemu-devel] [GIT PULL] Trivial patches for 2013-06-01

2013-06-03 Thread Anthony Liguori
Pulled. Thanks. Regards, Anthony Liguori

[Qemu-devel] [PULL 05/24] xilinx_spips: Fix QSPI FIFO size

2013-06-03 Thread Peter Maydell
From: Peter Crosthwaite peter.crosthwa...@xilinx.com QSPI has a bigger FIFO than the regular SPI controller. Differentiate between the two with correct FIFO sizes for each. This is the first piece of class data for SPIPS, so this patch sees the creation of the XilinxSPIPSClass definition and

[Qemu-devel] [PULL 04/24] xilinx_spips: Add verbose LQSPI debug output

2013-06-03 Thread Peter Maydell
From: Peter Crosthwaite peter.crosthwa...@xilinx.com You really need this is you want to track a guest banging on LQSPI. Signed-off-by: Peter Crosthwaite peter.crosthwa...@xilinx.com Reviewed-by: Peter Maydell peter.mayd...@linaro.org Reviewed-by: Edgar E. Iglesias edgar.igles...@gmail.com

[Qemu-devel] [PULL 21/24] i.MX: split GPT and EPIT timer implementation

2013-06-03 Thread Peter Maydell
From: Jean-Christophe DUBOIS j...@tribudubois.net There is no common code between these 2 timer implementation. So it is better to split them. Signed-off-by: Jean-Christophe DUBOIS j...@tribudubois.net Message-id: 1368990197-19694-1-git-send-email-...@tribudubois.net Signed-off-by: Peter Maydell

[Qemu-devel] [PATCH v2] qemu-char: don't issue CHR_EVENT_OPEN in a BH

2013-06-03 Thread Michael Roth
When CHR_EVENT_OPEN was initially added, it was CHR_EVENT_RESET, and it was issued as a bottom-half: 86e94dea5b740dad65446c857f6959eae43e0ba6 Which we basically used to print out a greeting/prompt for the monitor. AFAICT the only reason this was ever done in a BH was because in some cases we'd

[Qemu-devel] [PULL 17/24] sd/sdhci: Fix Buffer Write Ready interrupt

2013-06-03 Thread Peter Maydell
From: Peter Crosthwaite peter.crosthwa...@xilinx.com This interrupt is not risen after the last block is written to sd. It is mutually exclusive with the end of transfer conditions. Fix. Signed-off-by: Peter Crosthwaite peter.crosthwa...@xilinx.com Reviewed-by: Igor Mitsyanko

[Qemu-devel] [PULL 22/24] hw/arm/exynos4210.c: convert chipid_and_omr to an mmio region

2013-06-03 Thread Peter Maydell
From: Igor Mitsyanko i.mitsya...@samsung.com Exynos SoC was misusing memory_region_init_ram_ptr(): this interface can safely be used only for memory regions which size is a multiple of target page size. Change chipid_and_omr memory to an mmio region to fix this. Signed-off-by: Igor Mitsyanko

[Qemu-devel] [PULL 02/24] xilinx_spips: Make interrupts clear on read

2013-06-03 Thread Peter Maydell
From: Peter Crosthwaite peter.crosthwa...@xilinx.com By default these interrupts are clear on read. Signed-off-by: Peter Crosthwaite peter.crosthwa...@xilinx.com Reviewed-by: Peter Maydell peter.mayd...@linaro.org Reviewed-by: Edgar E. Iglesias edgar.igles...@gmail.com Message-id:

[Qemu-devel] [PULL 07/24] xilinx_spips: Add automatic start support

2013-06-03 Thread Peter Maydell
From: Peter Crosthwaite peter.crosthwa...@xilinx.com SPI has a mode where it automatically starts based on tx fifo occupancy. Implemented. Signed-off-by: Peter Crosthwaite peter.crosthwa...@xilinx.com Reviewed-by: Peter Maydell peter.mayd...@linaro.org Reviewed-by: Edgar E. Iglesias

Re: [Qemu-devel] VFIO and scheduled SR-IOV cards

2013-06-03 Thread Alex Williamson
On Mon, 2013-06-03 at 18:33 +0200, Benoît Canet wrote: Hello, I plan to write a PF driver for an SR-IOV card and make the VFs work with QEMU's VFIO passthrough so I am asking the following design question before trying to write and push code. After SR-IOV being enabled on this hardware

Re: [Qemu-devel] QEMU aborts since kvm: support using KVM_MEM_READONLY flag for regions

2013-06-03 Thread Peter Wu
On Monday 03 June 2013 10:35:28 Jordan Justen wrote: This should be fixed by Xiao's fix double free the memslot in kvm_set_phys_mem patch. Confirmed, with this one-liner patch the bug is gone. On Mon, Jun 3, 2013 at 10:23 AM, Peter Wu lekenst...@gmail.com wrote: Hi, With the current git

[Qemu-devel] [PULL 20/24] sd/sd.c: Fix inquiry ACMD41

2013-06-03 Thread Peter Maydell
From: Peter Crosthwaite peter.crosthwa...@xilinx.com QEMU models two (of the three) ACMD41 has two modes, inquiry and first. The selection logic for which of the two is incorrect - it compares != 0 for the entire argument value rather than only bits 23:0 as per the spec. Fix. Signed-off-by:

Re: [Qemu-devel] [PATCH] target-i386: Fix aflag logic for CODE64 and the 0x67 prefix

2013-06-03 Thread Anthony Liguori
Applied. Thanks. Regards, Anthony Liguori

[Qemu-devel] Re: Boot guest OS on many nodes?

2013-06-03 Thread Brian Jackson
On Sunday, June 2, 2013 9:37:55 PM CDT, Steven.G wrote: Dear all, I've a scene, I have many host nodes with linux kernels, and I want to boot one guest OS on all these nodes use QEMU/KVM. In order that I can use all the physical CPUs and Mems distributed on these linux nodes, and the Guest

Re: [Qemu-devel] [PATCH trivial] acpi: actually require either data= or file= for -acpitable

2013-06-03 Thread Laszlo Ersek
On 06/03/13 14:42, Michael Tokarev wrote: 03.06.2013 16:34, Eric Blake wrote: On 06/03/2013 03:20 AM, Michael Tokarev wrote: Initially the code ensured that we have exactly one of data= or file= option for -acpitable. But after some transformations, the condition becomes if (has_data ==

[Qemu-devel] [PULL 11/24] xilinx_spips: Fix striping behaviour

2013-06-03 Thread Peter Maydell
From: Peter Crosthwaite peter.crosthwa...@xilinx.com The QSPI controller was using byte-wide stripes when striping across the two flashes in dual parallel mode. The real hardware however uses individual bit striping. QEMU misbehaves in the (corner) case where data is written/read in dual-parallel

Re: [Qemu-devel] [GIT PULL] Xen fixes and cleanups 20130603

2013-06-03 Thread Anthony Liguori
Stefano Stabellini stefano.stabell...@eu.citrix.com writes: Anthony, please pull the following changes since commit 6a4e17711442849bf2cc731ccddef5a2a2d92d29:

Re: [Qemu-devel] VFIO and scheduled SR-IOV cards

2013-06-03 Thread Don Dutile
On 06/03/2013 02:02 PM, Alex Williamson wrote: On Mon, 2013-06-03 at 18:33 +0200, Benoît Canet wrote: Hello, I plan to write a PF driver for an SR-IOV card and make the VFs work with QEMU's VFIO passthrough so I am asking the following design question before trying to write and push code.

Re: [Qemu-devel] SR-IOV PF reset and QEMU VFs VFIO passthrough

2013-06-03 Thread Don Dutile
On 06/01/2013 08:13 AM, Benoît Canet wrote: Hello, I may have soon the PF driver of an SR-IOV card to code and make work with QEMU/KVM so I have the following questions. In an AMD64 setup where QEMU use VFIO to passthrough the VFs of an SR-IOV card to a guest will the consequences of a PF FLR

Re: [Qemu-devel] QEMU aborts since kvm: support using KVM_MEM_READONLY flag for regions

2013-06-03 Thread Anthony Liguori
Peter Wu lekenst...@gmail.com writes: Hi, With the current git source (f10acc8b38d65a66ffa0588a036489d7fa6a593e), `qemu-system-x86_64 -enable-kvm` fails to run with the following error: kvm_set_phys_mem: error unregistering overlapping slot: Invalid argument After this message, qemu

Re: [Qemu-devel] VFIO and scheduled SR-IOV cards

2013-06-03 Thread Alex Williamson
On Mon, 2013-06-03 at 14:34 -0400, Don Dutile wrote: On 06/03/2013 02:02 PM, Alex Williamson wrote: On Mon, 2013-06-03 at 18:33 +0200, Benoît Canet wrote: Hello, I plan to write a PF driver for an SR-IOV card and make the VFs work with QEMU's VFIO passthrough so I am asking the

[Qemu-devel] [PULL 01/24] xilinx_spips: seperate SPI and QSPI as two classes

2013-06-03 Thread Peter Maydell
From: Peter Crosthwaite peter.crosthwa...@xilinx.com Make SPI and QSPI different classes. QSPIPS is setup as a child of SPIPS. Only QSPI has the LQSPI functionality, so move all that to the child class. Signed-off-by: Peter Crosthwaite peter.crosthwa...@xilinx.com Reviewed-by: Peter Maydell

[Qemu-devel] [PULL 18/24] sd/sdhci.c: Fix bdata_read DPRINT message

2013-06-03 Thread Peter Maydell
From: Peter Crosthwaite peter.crosthwa...@xilinx.com This message was printing out the data in decimal only, which is not very friendly to the debugging developer. Add hex variant in parenthesis to make it consistent with other similar messages in this module. Signed-off-by: Peter Crosthwaite

Re: [Qemu-devel] [PATCH 0/2] MAINTAINERS: QMP: entry update and new maintainers

2013-06-03 Thread Anthony Liguori
Luiz Capitulino lcapitul...@redhat.com writes: Hi, This is an small update on QMP/HMP/QAPI mainternership. Please, check individual patches for details. Acked-by: Anthony Liguori aligu...@us.ibm.com Regards, Anthony Liguori Luiz Capitulino (2): MAINTAINERS: new maintainers for

Re: [Qemu-devel] [PATCH 2/2] MAINTAINERS: split Monitor (QMP/HMP) entry

2013-06-03 Thread Paolo Bonzini
Il 03/06/2013 19:17, Luiz Capitulino ha scritto: This entry doesn't reflect reality for a few years now. This commit splits it into Human Monitor (HMP), QAPI and QMP. Markus is dropped as a maintainer. This is what we have been for the last few years. Also, it's going to help me to offload

Re: [Qemu-devel] SR-IOV PF reset and QEMU VFs VFIO passthrough

2013-06-03 Thread Benoît Canet
to a guest will the consequences of a PF FLR be handled fine by QEMU and the guest ? the reset occurs long before the device is passed to the guest. I was asking this because the PF driver should reset the PF while the VF are used by VFIO/QEMU when the PF doesn't respond anymore. The PF

Re: [Qemu-devel] [PATCH 2/2] MAINTAINERS: split Monitor (QMP/HMP) entry

2013-06-03 Thread Luiz Capitulino
On Mon, 03 Jun 2013 21:13:30 +0200 Paolo Bonzini pbonz...@redhat.com wrote: Il 03/06/2013 19:17, Luiz Capitulino ha scritto: This entry doesn't reflect reality for a few years now. This commit splits it into Human Monitor (HMP), QAPI and QMP. Markus is dropped as a maintainer. This is

[Qemu-devel] OS crash while attempting to boot a 1TB guest.

2013-06-03 Thread Chegu Vinod
Hello, For guest sizes = 1TB RAM the guest OS is unable to boot up (please see attached GIF file for the Oops message). Wonder if this is a bug/regression in qemu/seabios or does one have to enable/disable something else in the qemu command line (pl. see below) ? Thanks Vinod Host and

Re: [Qemu-devel] [PATCH 2/2] MAINTAINERS: split Monitor (QMP/HMP) entry

2013-06-03 Thread mdroth
On Mon, Jun 03, 2013 at 03:54:48PM -0400, Luiz Capitulino wrote: On Mon, 03 Jun 2013 21:13:30 +0200 Paolo Bonzini pbonz...@redhat.com wrote: Il 03/06/2013 19:17, Luiz Capitulino ha scritto: This entry doesn't reflect reality for a few years now. This commit splits it into Human Monitor

<    1   2   3   >