[Qemu-devel] [PATCH] configure: show debug-info option in --help output

2013-03-24 Thread riegamaths
From: Dunrong Huang huan...@cloud-times.com --enable-debug-info and --disable-debug-info were not shown in --help output. Signed-off-by: Dunrong Huang huan...@cloud-times.com --- configure | 2 ++ 1 file changed, 2 insertions(+) diff --git a/configure b/configure index f2af714..61e71fa 100755

[Qemu-devel] [PATCH] block: Add options QDict to bdrv_file_open() prototypes (fix MinGW build)

2013-03-24 Thread Stefan Weil
The new parameter is unused yet. This part was missing in commit 787e4a8500020695eb391e2f1cc4767ee071d441. Cc: Kevin Wolf kw...@redhat.com Cc: Eric Blake ebl...@redhat.com Signed-off-by: Stefan Weil s...@weilnetz.de --- block/raw-win32.c |6 -- 1 file changed, 4 insertions(+), 2

[Qemu-devel] [PATCH] rbd: fix compile error

2013-03-24 Thread Liu Yuan
From: Liu Yuan tailai...@taobao.com Commit 787e4a85 [block: Add options QDict to bdrv_file_open() prototypes] didn't update rbd.c accordingly. Cc: Kevin Wolf kw...@redhat.com Cc: Stefan Hajnoczi stefa...@redhat.com Signed-off-by: Liu Yuan tailai...@taobao.com --- block/rbd.c |3 ++- 1 file

[Qemu-devel] [PATCH] target-moxie: Fix pointer-to-integer conversion (MinGW-w64)

2013-03-24 Thread Stefan Weil
The type cast must use tcg_target_long instead of long. This makes a difference for hosts where sizeof(long) != sizeof(void *). Cc: Anthony Green gr...@moxielogic.com Cc: Blue Swirl blauwir...@gmail.com Signed-off-by: Stefan Weil s...@weilnetz.de --- target-moxie/translate.c |2 +- 1 file

Re: [Qemu-devel] [PATCH] rbd: fix compile error

2013-03-24 Thread Stefan Weil
Am 24.03.2013 08:41, schrieb Liu Yuan: From: Liu Yuan tailai...@taobao.com Commit 787e4a85 [block: Add options QDict to bdrv_file_open() prototypes] didn't update rbd.c accordingly. Cc: Kevin Wolf kw...@redhat.com Cc: Stefan Hajnoczi stefa...@redhat.com Signed-off-by: Liu Yuan

Re: [Qemu-devel] [PATCH] pcie: Enhance PCIe links

2013-03-24 Thread Michael S. Tsirkin
On Fri, Mar 22, 2013 at 09:25:13AM -0600, Alex Williamson wrote: On Thu, 2013-03-21 at 18:56 +0200, Michael S. Tsirkin wrote: On Tue, Mar 19, 2013 at 04:24:47PM -0600, Alex Williamson wrote: Enable PCIe devices to negotiate links. This upgrades our root ports and switches to advertising

Re: [Qemu-devel] [PATCH] linux-user: improve target_to_host_sock_type conversion for MIPS

2013-03-24 Thread Aurelien Jarno
I am reviewing that for the MIPS part, and I am not really used to the linux-user code, so a review from someone familiar with it would be appreciated. I added Richard Henderson and Blue Swirl in Cc: as alpha and sparc are also affected by this issue. On Mon, Mar 18, 2013 at 11:47:05PM +0100,

Re: [Qemu-devel] [PATCH] Remove device_tree.o from hw/moxie/Makefile.objs.

2013-03-24 Thread Aurelien Jarno
On Sun, Mar 24, 2013 at 12:43:38AM -0400, Anthony Green wrote: Here's a fix for the build problem identified by Aurelien Jano here: http://lists.gnu.org/archive/html/qemu-devel/2013-03/msg04177.html I hadn't tested with FDT enabled recently. Thanks! AG Signed-off-by: Anthony Green

[Qemu-devel] qemu-x86_64 on i386 host: SIGSEGV

2013-03-24 Thread Michael Tokarev
$ ./x86_64-linux-user/qemu-x86_64 bash64 qemu: uncaught target signal 11 (Segmentation fault) - core dumped Segmentation Fault $ gdb x86_64-linux-user/qemu-x86_64 (gdb) ru bash64 Program received signal SIGSEGV, Segmentation fault. disas_insn (s=s@entry=0xcf98, pc_start=18446744073699066880)

Re: [Qemu-devel] qemu-x86_64 on i386 host: SIGSEGV

2013-03-24 Thread Peter Maydell
On 24 March 2013 10:43, Michael Tokarev m...@tls.msk.ru wrote: $ ./x86_64-linux-user/qemu-x86_64 bash64 qemu: uncaught target signal 11 (Segmentation fault) - core dumped Segmentation Fault Are 64 bit linux-user guests on 32 bit hosts supposed to work? I would expect them to be at best pretty

[Qemu-devel] [PATCH 02/10] versatile_pci: Expose PCI I/O region on Versatile PB

2013-03-24 Thread Peter Maydell
Comments in the QEMU source code claim that the version of the PCI controller on the VersatilePB board doesn't support the PCI I/O region, but this is incorrect; expose that region, map it in the correct location, and drop the misleading comments. This change removes the only currently

[Qemu-devel] [PATCH 01/10] versatile_pci: Fix hardcoded tabs

2013-03-24 Thread Peter Maydell
There is just one line in this source file with a hardcoded tab indent, so just fix it. Signed-off-by: Peter Maydell peter.mayd...@linaro.org --- hw/versatile_pci.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hw/versatile_pci.c b/hw/versatile_pci.c index

[Qemu-devel] [PATCH 08/10] versatile_pci: Implement the PCI controller's control registers

2013-03-24 Thread Peter Maydell
The versatile_pci PCI controller has a set of control registers which handle the mapping between PCI and system address spaces. Implement these registers (though for now they have no effect since we don't implement mapping PCI space into system memory at all). The most natural order for our

[Qemu-devel] [PATCH 07/10] versatile_pci: Implement the correct PCI IRQ mapping

2013-03-24 Thread Peter Maydell
Implement the correct IRQ mapping for the Versatile PCI controller; it differs between realview and versatile boards, but the previous QEMU implementation was correct only for the first PCI card on a versatile board, since we weren't swizzling IRQs based on the slot number. Note that this change

[Qemu-devel] [PATCH 06/10] versatile_pci: Put the host bridge PCI device at slot 29

2013-03-24 Thread Peter Maydell
On real hardware the host bridge appears as a PCI device in slot 29, so make QEMU put its host bridge in that slot too. Signed-off-by: Peter Maydell peter.mayd...@linaro.org --- hw/versatile_pci.c |2 ++ 1 file changed, 2 insertions(+) diff --git a/hw/versatile_pci.c b/hw/versatile_pci.c

[Qemu-devel] [PATCH 03/10] versatile_pci: Update to realize and instance init functions

2013-03-24 Thread Peter Maydell
Update the Versatile PCI controller to use a realize function rather than SysBusDevice::init. To reflect the fact that the 'realview_pci' class is taking most of its implementation from 'versatile_pci' (and to make the QOM casts work) we make 'realview_pci' a subclass of 'versatile_pci'.

[Qemu-devel] [PATCH 05/10] versatile_pci: Use separate PCI I/O space rather than system I/O space

2013-03-24 Thread Peter Maydell
Rather than overloading the system I/O space (which doesn't even make any sense on ARM) for PCI I/O, create an memory region in the PCI controller and use that to represent the I/O space. Signed-off-by: Peter Maydell peter.mayd...@linaro.org --- hw/versatile_pci.c | 17 + 1

[Qemu-devel] [PATCH 09/10] arm/realview: Fix mapping of PCI regions

2013-03-24 Thread Peter Maydell
Fix the mapping of the PCI regions for the realview board, which were all incorrect. (This was never noticed because the Linux kernel doesn't actually include a PCI driver for the realview boards.) Signed-off-by: Peter Maydell peter.mayd...@linaro.org --- hw/arm/realview.c | 18

[Qemu-devel] [PATCH 00/10] Fix versatile_pci (and break versatilepb linux guests!)

2013-03-24 Thread Peter Maydell
This patch series fixes a number of serious bugs in our emulation of the PCI controller found on VersatilePB and the early Realview boards: * our interrupt mapping was totally wrong * we weren't implementing the PCI memory windows * the I/O window wasn't mapped on VersatilePB * realview mapped

[Qemu-devel] [PATCH 04/10] versatile_pci: Change to subclassing TYPE_PCI_HOST_BRIDGE

2013-03-24 Thread Peter Maydell
Change versatile_pci to subclass TYPE_PCI_HOST_BRIDGE and generally handle PCI in a more QOM-like fashion. Signed-off-by: Peter Maydell peter.mayd...@linaro.org --- hw/versatile_pci.c | 41 ++--- 1 file changed, 30 insertions(+), 11 deletions(-) diff --git

[Qemu-devel] [PATCH 10/10] versatile_pci: Expose PCI memory space to system

2013-03-24 Thread Peter Maydell
The VersatilePB's PCI controller exposes the PCI memory space to the system via three regions controlled by the mapping control registers. Implement this so that guests can actually use MMIO-BAR PCI cards. Signed-off-by: Peter Maydell peter.mayd...@linaro.org --- hw/arm/realview.c|3 +++

Re: [Qemu-devel] [PATCH 1/2] char: add qemu_chr_be_is_fe_connected

2013-03-24 Thread Hans de Goede
Hi, On 03/22/2013 06:11 PM, Anthony Liguori wrote: Hans de Goede hdego...@redhat.com writes: snip If the qemu-char.c code did: int qemu_chr_fe_write(...) { if (!s-fe_open) { qemu_chr_fe_open(s); } ... } That would be one thing. It's a hack, but a more reasonable

[Qemu-devel] [PATCH 1/8] RFC: chardev frontend open handling cleanup

2013-03-24 Thread Hans de Goede
This patch-series is the result of the [PATCH 1/2] char: add qemu_chr_be_is_fe_connected discussion thread. This patch series (tries to) make(s) the frontend open concept both more explicit and generic, and significantly cleans up the surrounding code. Regards, Hans

[Qemu-devel] [PATCH 1/8] qemu-char: Rename opened to be_open

2013-03-24 Thread Hans de Goede
Rename the opened variable to be_open to reflect that it contains the opened state of the backend. Signed-off-by: Hans de Goede hdego...@redhat.com --- hw/usb/dev-serial.c | 2 +- hw/usb/redirect.c | 2 +- include/char/char.h | 2 +- qemu-char.c | 6 +++--- spice-qemu-char.c | 4 ++--

[Qemu-devel] [PATCH 2/8] qemu-char: Rename qemu_chr_generic_open to qemu_chr_be_generic_open

2013-03-24 Thread Hans de Goede
To better reflect that it is for handling a backend being opened. Signed-off-by: Hans de Goede hdego...@redhat.com --- backends/baum.c | 2 +- include/char/char.h | 2 +- qemu-char.c | 24 ui/console.c| 2 +- ui/gtk.c| 2 +- 5 files

[Qemu-devel] [PATCH 4/8] qemu-char: Automatically do fe_open / fe_close on qemu_chr_add_handlers

2013-03-24 Thread Hans de Goede
Most frontends can't really determine if the guest actually has the frontend side open. So lets automatically generate fe_open / fe_close as soon as a frontend becomes ready (as signalled by calling qemu_chr_add_handlers) / becomes non ready (as signalled by setting all handlers to NULL). And

[Qemu-devel] [PATCH 7/8] virtio-serial: Consolidate guest_open/guest_close into set_guest_connected

2013-03-24 Thread Hans de Goede
Signed-off-by: Hans de Goede hdego...@redhat.com --- hw/virtio-console.c| 23 +-- hw/virtio-serial-bus.c | 15 +-- hw/virtio-serial.h | 6 ++ 3 files changed, 12 insertions(+), 32 deletions(-) diff --git a/hw/virtio-console.c b/hw/virtio-console.c

[Qemu-devel] [PATCH 6/8] qemu-char: Consolidate guest_close/guest_open into a set_fe_open callback

2013-03-24 Thread Hans de Goede
Signed-off-by: Hans de Goede hdego...@redhat.com --- include/char/char.h | 3 +-- qemu-char.c | 10 +++--- spice-qemu-char.c | 17 +++-- 3 files changed, 11 insertions(+), 19 deletions(-) diff --git a/include/char/char.h b/include/char/char.h index 3c8dd28..1457e80

[Qemu-devel] [PATCH 3/8] qemu-char: Add fe_open tracking

2013-03-24 Thread Hans de Goede
Add tracking of the fe_open state to struct CharDriverState. Signed-off-by: Hans de Goede hdego...@redhat.com --- include/char/char.h | 1 + qemu-char.c | 2 ++ 2 files changed, 3 insertions(+) diff --git a/include/char/char.h b/include/char/char.h index dd8f39a..3174575 100644 ---

[Qemu-devel] [PATCH 5/8] qemu-char: Cleanup: consolidate fe_open/fe_close into fe_set_open

2013-03-24 Thread Hans de Goede
Signed-off-by: Hans de Goede hdego...@redhat.com --- hw/virtio-console.c | 4 ++-- include/char/char.h | 17 - qemu-char.c | 19 +-- 3 files changed, 11 insertions(+), 29 deletions(-) diff --git a/hw/virtio-console.c b/hw/virtio-console.c index

[Qemu-devel] [PATCH 8/8] spice-qemu-char: Drop hackish vmc_register on spice_chr_write

2013-03-24 Thread Hans de Goede
Now that the core takes care of fe_open tracking we no longer need this hack. Signed-off-by: Hans de Goede hdego...@redhat.com --- spice-qemu-char.c | 1 - 1 file changed, 1 deletion(-) diff --git a/spice-qemu-char.c b/spice-qemu-char.c index ba59374..7e6bd2d 100644 --- a/spice-qemu-char.c +++

[Qemu-devel] [PATCH 0/1] usb-serial: Remove double call to qemu_chr_add_handlers( NULL )

2013-03-24 Thread Hans de Goede
usb-serial has a qdev chardev property, and hw/qdev-properties-system.c already contains: static void release_chr(Object *obj, const char *name, void *opaque) { DeviceState *dev = DEVICE(obj); Property *prop = opaque; CharDriverState **ptr = qdev_get_prop_ptr(dev, prop); if

[Qemu-devel] [PATCH] usb-serial: Remove double call to qemu_chr_add_handlers( NULL )

2013-03-24 Thread Hans de Goede
usb-serial has a qdev chardev property, and hw/qdev-properties-system.c already contains: static void release_chr(Object *obj, const char *name, void *opaque) { DeviceState *dev = DEVICE(obj); Property *prop = opaque; CharDriverState **ptr = qdev_get_prop_ptr(dev, prop); if

[Qemu-devel] [PATCH 0/1] virtio-serial: propagate guest_connected to the port on post_load

2013-03-24 Thread Hans de Goede
Here is a new attempt on fixing the spicevmc needing the guest_connected value post-migration, as discussed in various other threads. Note this patch applies on top of my frontend-open handling cleanup series I just posted. Signed-off-by: Hans de Goede hdego...@redhat.com

[Qemu-devel] [PATCH] virtio-serial: propagate guest_connected to the port on post_load

2013-03-24 Thread Hans de Goede
From: Alon Levy al...@redhat.com When migrating a host with with a spice agent running the mouse becomes non operational after the migration due to the agent state being inconsistent between the guest and the client. After migration the spicevmc backend on the destination has never been notified

Re: [Qemu-devel] [PATCH v2] ifname=xxx for -netdev bridge

2013-03-24 Thread Alexandre Kandalintsev
Eric Blake ebl...@redhat.com wrote: You also need to add a line documenting this field: # @ifname: #optional Set the interface name that will be used (since 1.5). Done. I also added an example with -netdev and -device (seems this question is not covered in docs). Signed-off-by: Alexandre

Re: [Qemu-devel] qemu-x86_64 on i386 host: SIGSEGV

2013-03-24 Thread Michael Tokarev
24.03.2013 14:59, Peter Maydell wrote: On 24 March 2013 10:43, Michael Tokarev m...@tls.msk.ru wrote: $ ./x86_64-linux-user/qemu-x86_64 bash64 qemu: uncaught target signal 11 (Segmentation fault) - core dumped Segmentation Fault Are 64 bit linux-user guests on 32 bit hosts supposed to work?

[Qemu-devel] [PATCHv2] ibverbs: add a new IBV_ACCESS_GIFT option

2013-03-24 Thread Michael S. Tsirkin
At the moment registering an MR breaks COW. If the application does not care that adapter sees stale data (for example, it tracks writes reregisters and resends), it can use a new IBV_ACCESS_GIFT flag to prevent registration from breaking COW. The semantics are similar to that of SPLICE_F_GIFT

[Qemu-devel] [PATCHv2] rdma: add a new IB_ACCESS_GIFT flag

2013-03-24 Thread Michael S. Tsirkin
At the moment registering an MR breaks COW. This breaks memory overcommit for users such as KVM: we have a lot of COW pages, e.g. instances of the zero page or pages shared using KSM. If the application does not care that adapter sees stale data (for example, it tracks writes reregisters and

Re: [Qemu-devel] [PATCH 00/10] Fix versatile_pci (and break versatilepb linux guests!)

2013-03-24 Thread Aurelien Jarno
On Sun, Mar 24, 2013 at 11:32:30AM +, Peter Maydell wrote: This patch series fixes a number of serious bugs in our emulation of the PCI controller found on VersatilePB and the early Realview boards: * our interrupt mapping was totally wrong * we weren't implementing the PCI memory

Re: [Qemu-devel] [PATCH 00/10] Fix versatile_pci (and break versatilepb linux guests!)

2013-03-24 Thread Peter Maydell
On 24 March 2013 15:58, Aurelien Jarno aurel...@aurel32.net wrote: On Sun, Mar 24, 2013 at 11:32:30AM +, Peter Maydell wrote: This patch series fixes a number of serious bugs in our emulation of the PCI controller found on VersatilePB and the early Realview boards: Patchset tested on both

[Qemu-devel] [PATCHv2 0/6] ARM dump-guest-memory support

2013-03-24 Thread Rabin Vincent
A second patchset to add dump-guest-memory support for ARM. This version of the patchset addresses the following comments from the previous posting, and also adds some more patches to the core dump code (patch #4 and #6): - memset prstatus to 0 in x86_64_write_elf64_note() - handle big endian

[Qemu-devel] [PATCHv2 1/6] dump: create writable files

2013-03-24 Thread Rabin Vincent
The files dump-guest-memory are created as read-only even for the owner. This non-standard behaviour makes it annoying to deal with the dump files (eg. rm -f is needed to delete them or saving a new dump by overwriting the previous one is not possible). Change the code to generate files with

[Qemu-devel] [PATCHv2 3/6] dump: extract out get note size function

2013-03-24 Thread Rabin Vincent
Extract out the ELF note size function from i386 so we can use it from other targets. Signed-off-by: Rabin Vincent ra...@rab.in --- dump.c | 15 +++ include/sysemu/dump.h |2 ++ target-i386/arch_dump.c | 14 ++ 3 files changed, 19 insertions(+),

[Qemu-devel] [PATCHv2 2/6] dump: extract out note helper

2013-03-24 Thread Rabin Vincent
Make a common helper function out of the x86 code to add ELF notes. Signed-off-by: Rabin Vincent ra...@rab.in --- dump.c | 51 include/sysemu/dump.h |4 + target-i386/arch_dump.c | 208 +++ 3 files changed, 104

[Qemu-devel] [PATCHv2 4/6] dump: fix up memory mapping dependencies / stub

2013-03-24 Thread Rabin Vincent
dump.c won't build without the functions from memory_mapping.c (and memory_mapping-stub.c does not help there), so build memory_mapping.c when CONFIG_HAVE_CORE_DUMP is set. dump.c:84: undefined reference to `memory_mapping_list_free' dump.c:819: undefined reference to

[Qemu-devel] [PATCHv2 6/6] dump: fix memory region handling

2013-03-24 Thread Rabin Vincent
RAMBlock.offset does not provide the physical address of the memory region. This is available in the MemoryRegion's address. The wrong usage leads to incorrect physical addreses in the ELF. Fix it. Signed-off-by: Rabin Vincent ra...@rab.in --- dump.c| 19 +++

[Qemu-devel] [PATCHv2 5/6] target-arm: add dump-guest-memory support

2013-03-24 Thread Rabin Vincent
Enable support for the dump-guest-memory monitor command for ARM. Cc: Peter Maydell peter.mayd...@linaro.org Signed-off-by: Rabin Vincent ra...@rab.in --- configure|2 +- target-arm/Makefile.objs |2 +- target-arm/arch_dump.c | 61

Re: [Qemu-devel] [PATCH v2] target-i386: Improve x86_cpu_list output

2013-03-24 Thread Andreas Färber
Am 27.02.2013 10:37, schrieb Jan Kiszka: On 2013-02-27 10:33, Andreas Färber wrote: Am 27.02.2013 10:15, schrieb Jan Kiszka: Several issues fixed: - We were missing a bunch of feature lists. Fix this by simply dumping the meta list feature_word_info. - kvm_enabled() cannot be true at

Re: [Qemu-devel] [PULL 0/5] ipxe: add efi support

2013-03-24 Thread Jan Kiszka
On 2013-03-18 12:12, Gerd Hoffmann wrote: Hi, This pull adds efi support to our nic roms. Patches have been posted here for review: https://lists.nongnu.org/archive/html/qemu-devel/2013-03/msg00344.html As there have been no changes (other than a rebase to master) I'm not posting the

Re: [Qemu-devel] [PATCH 00/19] chardev flow control

2013-03-24 Thread Jan Kiszka
On 2013-02-18 22:47, Anthony Liguori wrote: This series implements an idea from Paolo to introduce flow control in the char layer by converting all char backends to use GIOChannels internally. Then we can just use the existing IO watch support in glib to implement flow control. This is

Re: [Qemu-devel] [PATCH v2] gtk: Fix accelerator filtering

2013-03-24 Thread Jan Kiszka
On 2013-02-25 16:44, Jan Kiszka wrote: On 2013-02-25 16:39, Anthony Liguori wrote: Jan Kiszka jan.kis...@siemens.com writes: This is in fact very simply: When the input in grabbed, everything should be exclusively passed to the guest - except it has our magic CTRL-ALT modifier set. Then let

[Qemu-devel] [PATCH v3] gtk: Release modifier when graphic console loses keyboard focus

2013-03-24 Thread Jan Kiszka
From: Jan Kiszka jan.kis...@siemens.com This solves, e.g., sticky ALT when selecting a GTK menu, switching to a different window or selecting a different virtual console. Signed-off-by: Jan Kiszka jan.kis...@siemens.com --- Changes in v3: - rebased ui/gtk.c | 53

Re: [Qemu-devel] [PATCHv2 5/6] target-arm: add dump-guest-memory support

2013-03-24 Thread Peter Maydell
On 24 March 2013 17:27, Rabin Vincent ra...@rab.in wrote: Enable support for the dump-guest-memory monitor command for ARM. Hi. I'm afraid I'm not really familiar with the dump-guest-memory command/implementation so I have some possibly dumb comments below: --- /dev/null +++

Re: [Qemu-devel] [PATCHv2 6/6] dump: fix memory region handling

2013-03-24 Thread Peter Maydell
On 24 March 2013 17:27, Rabin Vincent ra...@rab.in wrote: /** + * memory_region_get_addr: Get the address of a memory region + * + * @mr: the memory region + */ +hwaddr memory_region_get_addr(MemoryRegion *mr); I'm afraid this doesn't make sense. A MemoryRegion by itself has no address --

Re: [Qemu-devel] [PATCH 00/10] Fix versatile_pci (and break versatilepb linux guests!)

2013-03-24 Thread Michael S. Tsirkin
On Sun, Mar 24, 2013 at 04:58:37PM +0100, Aurelien Jarno wrote: On Sun, Mar 24, 2013 at 11:32:30AM +, Peter Maydell wrote: This patch series fixes a number of serious bugs in our emulation of the PCI controller found on VersatilePB and the early Realview boards: * our interrupt mapping

Re: [Qemu-devel] [PATCHv2 5/6] target-arm: add dump-guest-memory support

2013-03-24 Thread Rabin Vincent
2013/3/24 Peter Maydell peter.mayd...@linaro.org: On 24 March 2013 17:27, Rabin Vincent ra...@rab.in wrote: --- /dev/null +++ b/target-arm/arch_dump.c @@ -0,0 +1,61 @@ +#include cpu.h +#include sysemu/dump.h +#include elf.h + +typedef struct { +char pad1[24]; +uint32_t pid; +

Re: [Qemu-devel] [PATCHv2 6/6] dump: fix memory region handling

2013-03-24 Thread Rabin Vincent
2013/3/24 Peter Maydell peter.mayd...@linaro.org: On 24 March 2013 17:27, Rabin Vincent ra...@rab.in wrote: /** + * memory_region_get_addr: Get the address of a memory region + * + * @mr: the memory region + */ +hwaddr memory_region_get_addr(MemoryRegion *mr); I'm afraid this doesn't

Re: [Qemu-devel] [PATCH 00/10] Fix versatile_pci (and break versatilepb linux guests!)

2013-03-24 Thread Aurelien Jarno
On Sun, Mar 24, 2013 at 04:49:37PM +, Peter Maydell wrote: On 24 March 2013 15:58, Aurelien Jarno aurel...@aurel32.net wrote: On Sun, Mar 24, 2013 at 11:32:30AM +, Peter Maydell wrote: This patch series fixes a number of serious bugs in our emulation of the PCI controller found on

Re: [Qemu-devel] [PATCHv2 6/6] dump: fix memory region handling

2013-03-24 Thread Peter Maydell
On 24 March 2013 19:35, Rabin Vincent ra...@rab.in wrote: 2013/3/24 Peter Maydell peter.mayd...@linaro.org: On 24 March 2013 17:27, Rabin Vincent ra...@rab.in wrote: /** + * memory_region_get_addr: Get the address of a memory region + * + * @mr: the memory region + */ +hwaddr

Re: [Qemu-devel] [PATCHv2 5/6] target-arm: add dump-guest-memory support

2013-03-24 Thread Peter Maydell
On 24 March 2013 19:26, Rabin Vincent ra...@rab.in wrote: 2013/3/24 Peter Maydell peter.mayd...@linaro.org: On 24 March 2013 17:27, Rabin Vincent ra...@rab.in wrote: So I guess I should prefix this email by saying that quite a bit of it is really comments on the existing dump code rather than

Re: [Qemu-devel] [PATCH 00/10] Fix versatile_pci (and break versatilepb linux guests!)

2013-03-24 Thread Peter Maydell
On 24 March 2013 19:17, Michael S. Tsirkin m...@redhat.com wrote: On Sun, Mar 24, 2013 at 04:58:37PM +0100, Aurelien Jarno wrote: On Sun, Mar 24, 2013 at 11:32:30AM +, Peter Maydell wrote: I've provided a property for enabling the old broken IRQ mapping; this can be enabled with the

Re: [Qemu-devel] [PATCH 00/10] Fix versatile_pci (and break versatilepb linux guests!)

2013-03-24 Thread Arnd Bergmann
On Sunday 24 March 2013, Peter Maydell wrote: Yeah, ideally being able to detect the buggy kernel would be good; I can't see anything at the controller level that would do though, and I don't really know enough about PCI to know about generic PCI stuff that would work. (Why would the OS need

Re: [Qemu-devel] [PATCH 00/10] Fix versatile_pci (and break versatilepb linux guests!)

2013-03-24 Thread Peter Maydell
On 24 March 2013 21:16, Arnd Bergmann a...@arndb.de wrote: On Sunday 24 March 2013, Peter Maydell wrote: Yeah, ideally being able to detect the buggy kernel would be good; I can't see anything at the controller level that would do though, and I don't really know enough about PCI to know about

Re: [Qemu-devel] [PATCH 00/10] Fix versatile_pci (and break versatilepb linux guests!)

2013-03-24 Thread Michael S. Tsirkin
On Sun, Mar 24, 2013 at 08:53:33PM +, Peter Maydell wrote: On 24 March 2013 19:17, Michael S. Tsirkin m...@redhat.com wrote: On Sun, Mar 24, 2013 at 04:58:37PM +0100, Aurelien Jarno wrote: On Sun, Mar 24, 2013 at 11:32:30AM +, Peter Maydell wrote: I've provided a property for

Re: [Qemu-devel] [PATCH 00/10] Fix versatile_pci (and break versatilepb linux guests!)

2013-03-24 Thread Michael S. Tsirkin
On Sun, Mar 24, 2013 at 09:16:28PM +, Arnd Bergmann wrote: On Sunday 24 March 2013, Peter Maydell wrote: Yeah, ideally being able to detect the buggy kernel would be good; I can't see anything at the controller level that would do though, and I don't really know enough about PCI to know

Re: [Qemu-devel] [PATCH 00/10] Fix versatile_pci (and break versatilepb linux guests!)

2013-03-24 Thread Arnd Bergmann
On Sunday 24 March 2013, Peter Maydell wrote: OK, I'll give that a go tomorrow. While you're here, do you know what the point of the PCI_SELFID register is? The h/w docs are clear that the OS has to write the slot number of the board itself in to this register, but again I don't see why the

Re: [Qemu-devel] [PATCH 00/10] Fix versatile_pci (and break versatilepb linux guests!)

2013-03-24 Thread Arnd Bergmann
On Sunday 24 March 2013, Michael S. Tsirkin wrote: For future kernels, let's build in some hook that let qemu detect a non broken guest. How about writing some magic value into revision ID or some other readonly field? Yes, makes sense.

Re: [Qemu-devel] [PATCH] hw/arm_mptimer: Save the timer state

2013-03-24 Thread Peter Crosthwaite
Hi Peter, On Sat, Mar 23, 2013 at 1:49 AM, Peter Maydell peter.mayd...@linaro.org wrote: Add a missing VMSTATE_TIMER() entry to the arm_mptimer vmstate description; this omission meant that we would probably hang on reload when the timer failed to fire. I know load-save does not work for

Re: [Qemu-devel] [PATCH 02/10] versatile_pci: Expose PCI I/O region on Versatile PB

2013-03-24 Thread Peter Crosthwaite
Hi Peter, On Sun, Mar 24, 2013 at 9:32 PM, Peter Maydell peter.mayd...@linaro.org wrote: Comments in the QEMU source code claim that the version of the PCI controller on the VersatilePB board doesn't support the PCI I/O region, but this is incorrect; expose that region, map it in the correct

Re: [Qemu-devel] [RFC qemu PATCH] only writing out the last byte of MAC makes it have effect

2013-03-24 Thread Amos Kong
On Fri, Mar 22, 2013 at 10:45:09AM +1030, Rusty Russell wrote: Michael S. Tsirkin m...@redhat.com writes: On Thu, Mar 21, 2013 at 02:44:50PM +0800, Amos Kong wrote: The lengcy guests don't have mac programming command, we don't know when it's safe to use MAC. This patch changed qemu to