[Qemu-devel] qemu gdb

2010-09-23 Thread chandra shekar
This is output when i use gdb with my img in which i installed windows xp and after some time i get cannot find bound of the current line or received SIGTRAP Trace/breakpoint trap and stops plz any one help $ gdb -args x86_64-softmmu/qemu-system-x86_64 -m 512 -cdrom guest1.img (gdb) r Starting p

[Qemu-devel] Trace Logical memory

2010-09-23 Thread vanson . dang
Dear All Currently, QEMU don't implement trace target memory. For example we trace Arm target on X86_64 PC. So, how to implement it? Can you propose methodology to trace logical memory access? Thansk ! "The information in this e-mail (including attachments) is confidential and is only in

[Qemu-devel] Re: Win2k host problem with {get, free}{addr, name}info()

2010-09-23 Thread Paolo Bonzini
On 09/22/2010 07:16 PM, Blue Swirl wrote: gnulib's submodule is never built directly. It only lives in the build tree so that it can be consulted by gnulib-tool, but it doesn't even make it to the release tarballs. Instead, gnulib-tool should be invoked after checking out superproject.git, and

[Qemu-devel] Problems with e1000 network card on qemu.git

2010-09-23 Thread Lucas Meneghel Rodrigues
Hi folks: As most of you might know, we run some daily sanity and functional tests with both qemu-kvm.git and qemu.git. I decided to write asking for help with regards to what it appears to be a problem with the e1000 nw card (the default). Here is a list of what it fails pretty much every day for

[Qemu-devel] Re: [PATCH] CMOS file support

2010-09-23 Thread Paolo Bonzini
On 09/22/2010 09:43 PM, Mathias Krause wrote: I managed to add the nvram option but how do I get a reference to the drive back in the RTC code? Would I just loop with drive_get(IF_NONE, 0, i) until the id of the returned drive is "nvram"? Doesn't sound right but I've found no better solution due

Re: [Qemu-devel] BUG - qdev - partial loss of network connectivity

2010-09-23 Thread Leszek Urbanski
<4c9a4c77.2080...@codemonkey.ws>; from Anthony Liguori on Wed, Sep 22, 2010 at 13:35:35 -0500 > >>Is the guest kernel vanilla 2.6.32.22 or is it a distro kernel? If the > >>later, what distro? > >> > >>The difference in the two invocations is that with the -device syntax, > >>you're getting offl

[Qemu-devel] [PATCH 3/7] cutils: qemu_iovec_copy and qemu_iovec_memset

2010-09-23 Thread Stefan Hajnoczi
From: Kevin Wolf This adds two functions that work on QEMUIOVectors and will be used by the next qcow2 patches. Signed-off-by: Kevin Wolf --- Kevin already has this queued up for mainline. QED uses qemu_iovec_memset(). cutils.c | 50 +-

[Qemu-devel] [PATCH 7/7] qed: Consistency check support

2010-09-23 Thread Stefan Hajnoczi
This patch adds support for the qemu-img check command. It also introduces a dirty bit in the qed header to mark modified images as needing a check. This bit is cleared when the image file is closed cleanly. If an image file is opened and it has the dirty bit set, a consistency check will run an

[Qemu-devel] [PATCH 0/7] qed: Add QEMU Enhanced Disk format

2010-09-23 Thread Stefan Hajnoczi
QEMU Enhanced Disk format is a disk image format that forgoes features found in qcow2 in favor of better levels of performance and data integrity. Due to its simpler on-disk layout, it is possible to safely perform metadata updates more efficiently. Installations, suspend-to-disk, and other alloc

[Qemu-devel] [PATCH 5/7] qed: Table, L2 cache, and cluster functions

2010-09-23 Thread Stefan Hajnoczi
This patch adds code to look up data cluster offsets in the image via the L1/L2 tables. The L2 tables are writethrough cached in memory for performance (each read/write requires a lookup so it is essential to cache the tables). With cluster lookup code in place it is possible to implement bdrv_is

Re: [Qemu-devel] [PATCH 1/6] qdev: Make qbus_walk_children() call busfn for root bus.

2010-09-23 Thread Markus Armbruster
Please excuse my late reply. I'm so much behind in this list, it's not funny anymore. I agree with Anthony, this series looks nice. A few remarks inline. Isaku Yamahata writes: > Make qbus_walk_children() call busfn for root bus. Please don't repeat the subject in the body. While I'm nit-p

Re: [Qemu-devel] [PATCH] virtio-net: Don't pass NULL peer to tap routines

2010-09-23 Thread Anthony Liguori
On 09/22/2010 02:52 PM, Alex Williamson wrote: During a hotplug, the netdev might be removed before the connected virtio device. When this happens, the guest might be running cleanup operations that can trigger a segfault in qemu. Avoid one set of these by checking whether the peer device is pr

Re: [Qemu-devel] [PATCH] virtio-net: Don't pass NULL peer to tap routines

2010-09-23 Thread Alex Williamson
On Thu, 2010-09-23 at 12:43 -0500, Anthony Liguori wrote: > On 09/22/2010 02:52 PM, Alex Williamson wrote: > > During a hotplug, the netdev might be removed before the > > connected virtio device. When this happens, the guest might > > be running cleanup operations that can trigger a segfault in >

[Qemu-devel] [PATCH] block: Use GCC_FMT_ATTR and fix a format error

2010-09-23 Thread Stefan Weil
Adding the gcc format attribute detects a format bug which is fixed here. Cc: Blue Swirl Cc: Kevin Wolf Signed-off-by: Stefan Weil --- block/blkverify.c |5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) diff --git a/block/blkverify.c b/block/blkverify.c index 8083464..b39fb67 100

[Qemu-devel] [PATCH 1/7] qcow2: Make get_bits_from_size() common

2010-09-23 Thread Stefan Hajnoczi
The get_bits_from_size() calculates the log base-2 of a number. This is useful in bit manipulation code working with power-of-2s. Currently used by qcow2 and needed by qed in a follow-on patch. Signed-off-by: Stefan Hajnoczi --- block/qcow2.c | 22 -- cutils.c | 26

[Qemu-devel] [PATCH 2/7] cutils: Add bytes_to_str() to format byte values

2010-09-23 Thread Stefan Hajnoczi
From: Anthony Liguori This common function converts byte counts to human-readable strings with proper units. Signed-off-by: Anthony Liguori Signed-off-by: Stefan Hajnoczi --- cutils.c | 15 +++ qemu-common.h |1 + 2 files changed, 16 insertions(+), 0 deletions(-) diff

[Qemu-devel] [PATCH] blockdev: Use GCC_FMT_ATTR (format checking)

2010-09-23 Thread Stefan Weil
Additional changes: * Removed 'extern' from drive_add (avoids too long line). * Removed 'extern' from other functions (makes declarations consistent with others in same header file). Cc: Blue Swirl Cc: Kevin Wolf Signed-off-by: Stefan Weil --- blockdev.h | 15 +++ 1 files chan

[Qemu-devel] Re: [PATCH] block: Use GCC_FMT_ATTR and fix a format error

2010-09-23 Thread Blue Swirl
On Thu, Sep 23, 2010 at 6:37 PM, Stefan Weil wrote: > Adding the gcc format attribute detects a format bug > which is fixed here. > > Cc: Blue Swirl > Cc: Kevin Wolf > Signed-off-by: Stefan Weil > --- >  block/blkverify.c |    5 +++-- >  1 files changed, 3 insertions(+), 2 deletions(-) > > diff

[Qemu-devel] [PATCH 4/7] qed: Add QEMU Enhanced Disk image format

2010-09-23 Thread Stefan Hajnoczi
This patch introduces the qed on-disk layout and implements image creation. Later patches add read/write and other functionality. Signed-off-by: Stefan Hajnoczi --- Makefile.objs |1 + block/qed.c | 530 + block/qed.h | 148 +

[Qemu-devel] Re: [PATCH] block: Use GCC_FMT_ATTR and fix a format error

2010-09-23 Thread Stefan Weil
Am 23.09.2010 20:53, schrieb Blue Swirl: On Thu, Sep 23, 2010 at 6:37 PM, Stefan Weil wrote: Adding the gcc format attribute detects a format bug which is fixed here. Cc: Blue Swirl Cc: Kevin Wolf Signed-off-by: Stefan Weil --- block/blkverify.c |5 +++-- 1 files changed, 3 insertio

[Qemu-devel] [PATCH 6/7] qed: Read/write support

2010-09-23 Thread Stefan Hajnoczi
This patch implements the read/write state machine. Operations are fully asynchronous and multiple operations may be active at any time. Allocating writes are serialized to make metadata updates consistent. For example, two write requests to the same unallocated cluster should only allocate the n

[Qemu-devel] Re: [PATCH] block: Use GCC_FMT_ATTR and fix a format error

2010-09-23 Thread Stefan Weil
Am 23.09.2010 21:03, schrieb Stefan Weil: Am 23.09.2010 20:53, schrieb Blue Swirl: On Thu, Sep 23, 2010 at 6:37 PM, Stefan Weil wrote: Adding the gcc format attribute detects a format bug which is fixed here. Cc: Blue Swirl Cc: Kevin Wolf Signed-off-by: Stefan Weil --- block/blkverify.c |

[Qemu-devel] [PATCH 2/3] Replace remaining gcc format attribute by macro GCC_FMT_ATTR (format checking)

2010-09-23 Thread Stefan Weil
Replace the remaining format attribute printf by macro GCC_FMT_ATTR which uses gnu_printf (if supported). This needs additional code changes: * Add qemu-common.h (which defined GCC_FMT_ATTR) were needed. * Remove standard includes when qemu-common.h was added. qemu-common.h already provides th

[Qemu-devel] [PATCH 1/3] Replace most gcc format attributes by macro GCC_FMT_ATTR (format checking)

2010-09-23 Thread Stefan Weil
Since version 4.4.x, gcc supports additional format attributes. __attribute__ ((format (gnu_printf, 1, 2))) should be used instead of __attribute__ ((format (printf, 1, 2)) because QEMU always uses standard format strings (even with mingw32). The patch replaces format attribute printf / __

[Qemu-devel] [PATCH 3/3] Use GCC_FMT_ATTR (format checking)

2010-09-23 Thread Stefan Weil
Signed-off-by: Stefan Weil --- audio/audio.h |2 +- disas.c|3 ++- hw/mips_fulong2e.c |3 ++- hw/mips_malta.c|3 ++- json-parser.c |3 ++- monitor.c |3 ++- monitor.h |3 ++- qemu-char.h|3 ++- qemu-error.h

[Qemu-devel] Re: [PATCH] block: Use GCC_FMT_ATTR and fix a format error

2010-09-23 Thread Blue Swirl
On Thu, Sep 23, 2010 at 7:11 PM, Stefan Weil wrote: > Am 23.09.2010 21:03, schrieb Stefan Weil: >> >> Am 23.09.2010 20:53, schrieb Blue Swirl: >>> >>> On Thu, Sep 23, 2010 at 6:37 PM, Stefan Weil >>>  wrote: Adding the gcc format attribute detects a format bug which is fixed here. >

[Qemu-devel] Re: [PATCH 2/3] Replace remaining gcc format attribute by macro GCC_FMT_ATTR (format checking)

2010-09-23 Thread Blue Swirl
On Thu, Sep 23, 2010 at 7:28 PM, Stefan Weil wrote: > Replace the remaining format attribute printf by macro > GCC_FMT_ATTR which uses gnu_printf (if supported). > > This needs additional code changes: > > * Add qemu-common.h (which defined GCC_FMT_ATTR) were needed. > > * Remove standard includes

[Qemu-devel] Re: [PATCH 2/3] Replace remaining gcc format attribute by macro GCC_FMT_ATTR (format checking)

2010-09-23 Thread Stefan Weil
Am 23.09.2010 22:33, schrieb Blue Swirl: On Thu, Sep 23, 2010 at 7:28 PM, Stefan Weil wrote: Replace the remaining format attribute printf by macro GCC_FMT_ATTR which uses gnu_printf (if supported). This needs additional code changes: * Add qemu-common.h (which defined GCC_FMT_ATTR) were need

[Qemu-devel] [Snapshot Mode]

2010-09-23 Thread Benjamin Kormann
Hi there, we've been using qemu for quite a while on different platforms and we'd like to perform snapshots on running qemu machines and tar them onto a backup server. The qemu-img snapshot can only be applied to inactive qemu machines (images), hence this is not applicable. The savevm command w

[Qemu-devel] Re: [PATCH] block: Use GCC_FMT_ATTR and fix a format error

2010-09-23 Thread Stefan Weil
Am 23.09.2010 22:24, schrieb Blue Swirl: On Thu, Sep 23, 2010 at 7:11 PM, Stefan Weil wrote: Am 23.09.2010 21:03, schrieb Stefan Weil: Am 23.09.2010 20:53, schrieb Blue Swirl: On Thu, Sep 23, 2010 at 6:37 PM, Stefan Weil wrote: Adding the gcc format attribute detects a format bug which i

Re: [Qemu-devel] [PATCH 1/7] qcow2: Make get_bits_from_size() common

2010-09-23 Thread malc
On Thu, 23 Sep 2010, Stefan Hajnoczi wrote: > The get_bits_from_size() calculates the log base-2 of a number. This is > useful in bit manipulation code working with power-of-2s. > > Currently used by qcow2 and needed by qed in a follow-on patch. int ilog2 (size_t size) { if (size & (size -

[Qemu-devel] [Bug 646427] [NEW] VirtFS mapped symlinks resolved wrong

2010-09-23 Thread Moshroum
Public bug reported: I tried to map a folder with qemu-kvm qemu-kvm-0.13.0-rc1-3-gc9c2179 (this is v0.13.0-rc1-16667-gc9c2179). I mounted it first in passthrough mode and saw all symlinks as expected. Then I used mapped and noticed that the target of a symlink changed to the actual data inside the

[Qemu-devel] Re: [PATCH v3 05/13] pcie: helper functions for pcie capability and extended capability.

2010-09-23 Thread Isaku Yamahata
On Sun, Sep 19, 2010 at 01:45:33PM +0200, Michael S. Tsirkin wrote: > On Sun, Sep 19, 2010 at 01:56:23PM +0900, Isaku Yamahata wrote: > > On Wed, Sep 15, 2010 at 02:43:10PM +0200, Michael S. Tsirkin wrote: > > > > +/*** > > > >

[Qemu-devel] Re: [PATCH v3 06/13] pcie/aer: helper functions for pcie aer capability.

2010-09-23 Thread Isaku Yamahata
On Wed, Sep 22, 2010 at 01:50:16PM +0200, Michael S. Tsirkin wrote: > > +} > > + > > +/* capability & control */ > > +if (ranges_overlap(addr, len, pos + PCI_ERR_CAP, 4)) { > > +uint32_t err_cap = pci_get_long(dev->config + pos + PCI_ERR_CAP); > > +if (!(err_cap & PCI_E

Re: [Qemu-devel] [PATCH RFC V3 04/12] xen: Add the Xen platform pci device

2010-09-23 Thread Isaku Yamahata
On Fri, Sep 17, 2010 at 12:14:59PM +0100, anthony.per...@citrix.com wrote: > +static int xen_platform_initfn(PCIDevice *dev) > +{ > +PCIXenPlatformState *d = DO_UPCAST(PCIXenPlatformState, pci_dev, dev); > +uint8_t *pci_conf; > + > +pci_conf = d->pci_dev.config; > + > +pci_config_se

Re: [Qemu-devel] [PATCH RFC V3 05/12] piix_pci: Introduces Xen specific call for irq.

2010-09-23 Thread Isaku Yamahata
On Fri, Sep 17, 2010 at 12:15:00PM +0100, anthony.per...@citrix.com wrote: > From: Anthony PERARD > > This patch introduces Xen specific call in piix_pci. > > The specific part for Xen is in write_config, set_irq and get_pirq. > > Signed-off-by: Anthony PERARD > Signed-off-by: Stefano Stabelli

[Qemu-devel] Re: [PATCH v3 08/13] pcie root port: implement pcie root port.

2010-09-23 Thread Isaku Yamahata
On Wed, Sep 22, 2010 at 01:25:59PM +0200, Michael S. Tsirkin wrote: > > +PCIESlot *pcie_root_init(PCIBus *bus, int devfn, bool multifunction, > > + const char *bus_name, pci_map_irq_fn map_irq, > > + uint8_t port, uint8_t chassis, uint16_t slot); > >

Re: [Qemu-devel] [PATCH RFC V3 04/12] xen: Add the Xen platform pci device

2010-09-23 Thread Isaku Yamahata
On Fri, Sep 17, 2010 at 12:14:59PM +0100, anthony.per...@citrix.com wrote: > +static uint32_t platform_mmio_read(void *opaque, target_phys_addr_t addr) > +{ > +static int warnings = 0; > + > +if (warnings < 5) { > +DPRINTF("Warning: attempted read from physical address " > +