[Qemu-devel] PowerPC code generation and the program counter

2010-09-12 Thread Stu Grossman
I've been using qemu-12.4 to trace accesses to non-existent addresses, but I've found that the PC is incorrect when cpu_abort() is called from within the unassigned memory helper routines (unassigned_mem_read[bwl] and unassigned_mem_write[bwl]).  Even nearby instructions (plus or minus 10 instructi

Re: [Qemu-devel] Arm : How to investigate Qemu blank screen on target system problem?

2010-09-12 Thread Mulyadi Santosa
On Sun, Sep 12, 2010 at 02:43, Robert Smith wrote: > Sirs, > > I've built kernel  v2.6.33.2 and tryintg to simulate realview (or integrator) > arm board on qemu under Ubuntu 9.10. > > make ARCH=arm realview_defconfig > make ARCH=arm CROSS_COMPILE=arm-linux- > qemu-system-arm -M realview -kernel z

[Qemu-devel] [Bug 636446] Re: prep ppc "machine" no more working

2010-09-12 Thread Massimo Montecchi
I want to emulate PReP machine to be able to install AIX for testing purpose. I will ask on openbios list. -- prep ppc "machine" no more working https://bugs.launchpad.net/bugs/636446 You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. Sta

Re: [Qemu-devel] Issue with compiling qemu-0.13.0.-rc1

2010-09-12 Thread Adhyas Avasthi
I tried qemu-0.13.0-rc1 and it has the same issues on this machine. Let me try what you suggest. Thanks, Adhyas On Sun, Sep 12, 2010 at 3:21 PM, Andreas Färber wrote: > Try giving an explicit --target-list= to configure, to narrow down what > build step causes this. There may be subtle dependenc

Re: [Qemu-devel] Issue with compiling qemu-0.13.0.-rc1

2010-09-12 Thread malc
On Fri, 10 Sep 2010, Adhyas Avasthi wrote: > When I try to run the following commands on rc1 or fresh git (synced 5 > minutes back), I get the following error: > > bash-3.2$ ./configure --disable-kvm --disable-xen > bash-3.2$ make > Makefile:24: no file name for `-include' > make-3.79.1-p7[1]: **

Re: [Qemu-devel] Issue with compiling qemu-0.13.0.-rc1

2010-09-12 Thread Anthony Liguori
On 09/12/2010 04:16 PM, Andreas Färber wrote: Am 12.09.2010 um 21:26 schrieb Anthony Liguori: On 09/12/2010 01:06 PM, Adhyas Avasthi wrote: Thanks for the response. I tried './configure' without any arguments and still the same issue. Find the output from ./configure and make attached. You'

Re: [Qemu-devel] Issue with compiling qemu-0.13.0.-rc1

2010-09-12 Thread Andreas Färber
Am 13.09.2010 um 00:08 schrieb Adhyas Avasthi: Is there a workaround that can unblock me for the time being ? Try giving an explicit --target-list= to configure, to narrow down what build step causes this. There may be subtle dependency issues, cf. my recent patch "trace: Fix user emulator

Re: [Qemu-devel] Issue with compiling qemu-0.13.0.-rc1

2010-09-12 Thread Adhyas Avasthi
bash-3.2$ make --version GNU Make version 3.79.1-p7, by Richard Stallman and Roland McGrath. Built for i686-pc-linux-gnu Copyright (C) 1988, 89, 90, 91, 92, 93, 94, 95, 96, 97, 98, 99, 2000 Free Software Foundation, Inc. This is free software; see the source for copying conditions. There is

Re: [Qemu-devel] [Bug 636315] [NEW] configure and build errors on Solaris 10 due to /bin/sh usage

2010-09-12 Thread Andreas Färber
Am 12.09.2010 um 23:05 schrieb Blue Swirl: On Sun, Sep 12, 2010 at 5:58 PM, Andreas Färber > wrote: Am 12.09.2010 um 19:47 schrieb Blue Swirl: nfields=$((nfields + 1)) ./tracetool: syntax error at line 53: `nfields=$' unexpected That looks like fully standards compliant, so Solaris'

[Qemu-devel] [PATCH 3/5] virtio-blk: propagate the required alignment

2010-09-12 Thread Christoph Hellwig
Signed-off-by: Christoph Hellwig Index: qemu/hw/virtio-blk.c === --- qemu.orig/hw/virtio-blk.c 2010-09-12 14:42:46.721759377 -0300 +++ qemu/hw/virtio-blk.c2010-09-12 14:51:09.737759375 -0300 @@ -540,6 +540,7 @@ VirtIODevice

[Qemu-devel] [PATCH 5/5] ide: propagate the required alignment

2010-09-12 Thread Christoph Hellwig
IDE is a bit ugly in this respect. For one it doesn't really keep track of a sector size - most of the protocol is in units of 512 bytes, and we assume 2048 bytes for CDROMs which is correct most of the time. Second IDE allocates an I/O buffer long before we know if we're dealing with a CDROM or

[Qemu-devel] [PATCH 4/5] scsi-disk: propagate the required alignment

2010-09-12 Thread Christoph Hellwig
Signed-off-by: Christoph Hellwig Index: qemu/hw/scsi-disk.c === --- qemu.orig/hw/scsi-disk.c2010-09-12 14:43:04.694759377 -0300 +++ qemu/hw/scsi-disk.c 2010-09-12 14:50:19.049759376 -0300 @@ -1178,6 +1178,7 @@ static int scsi_dis

[Qemu-devel] [PATCH 2/5] raw-posix: handle > 512 byte alignment correctly

2010-09-12 Thread Christoph Hellwig
Replace the hardcoded handling of 512 byte alignment with bs->buffer_alignment to handle larger sector size devices correctly. Note that we can not rely on it to be initialize in bdrv_open, so deal with the worst case there. Signed-off-by: Christoph Hellwig Index: qemu/block/raw-posix.c ===

[Qemu-devel] [PATCH 1/5] use qemu_blockalign consistently

2010-09-12 Thread Christoph Hellwig
Use qemu_blockalign for all allocations in the block layer. This allows increasing the required alignment, which is need to support O_DIRECT on devices with large block sizes. Signed-off-by: Christoph Hellwig Index: qemu/hw/scsi-disk.c ===

Re: [Qemu-devel] Issue with compiling qemu-0.13.0.-rc1

2010-09-12 Thread Andreas Färber
Am 12.09.2010 um 21:26 schrieb Anthony Liguori: On 09/12/2010 01:06 PM, Adhyas Avasthi wrote: Thanks for the response. I tried './configure' without any arguments and still the same issue. Find the output from ./configure and make attached. You've got a weird version of make installed. It's

Re: [Qemu-devel] [Bug 636315] [NEW] configure and build errors on Solaris 10 due to /bin/sh usage

2010-09-12 Thread Blue Swirl
On Sun, Sep 12, 2010 at 5:58 PM, Andreas Färber wrote: > Am 12.09.2010 um 19:47 schrieb Blue Swirl: > >> On Sun, Sep 12, 2010 at 5:35 PM, Andreas Färber >> wrote: >>> >>> Am 12.09.2010 um 19:22 schrieb Blue Swirl: >>> What is the output of "sh ./tracetool --nop --check-backend"? >>> >>> ./tr

Re: [Qemu-devel] [RFC] qed: Add QEMU Enhanced Disk format

2010-09-12 Thread Anthony Liguori
On 09/12/2010 12:51 PM, Avi Kivity wrote: On 09/12/2010 07:09 PM, Anthony Liguori wrote: On 09/12/2010 10:56 AM, Avi Kivity wrote: No, the worst case is 0.003% allocated disk, with the allocated clusters distributed uniformly. That means all your L2s are allocated, but almost none of your cl

Re: [Qemu-devel] Issue with compiling qemu-0.13.0.-rc1

2010-09-12 Thread Anthony Liguori
On 09/12/2010 01:06 PM, Adhyas Avasthi wrote: Thanks for the response. I tried './configure' without any arguments and still the same issue. Find the output from ./configure and make attached. You've got a weird version of make installed. It's throwing an error when no file is specified fo

[Qemu-devel] [Bug 636446] Re: prep ppc "machine" no more working

2010-09-12 Thread Andreas Färber
** Tags added: powerpc -- prep ppc "machine" no more working https://bugs.launchpad.net/bugs/636446 You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. Status in QEMU: Confirmed Bug description: I have tried qemu from 0.11 to latest git (a

Re: [Qemu-devel] Issue with compiling qemu-0.13.0.-rc1

2010-09-12 Thread Adhyas Avasthi
Thanks for the response. I tried './configure' without any arguments and still the same issue. Find the output from ./configure and make attached. Thanks, Adhyas On Sat, Sep 11, 2010 at 10:37 AM, Anthony Liguori wrote: > Please  include the full output of configure. > > Why are you explicitly us

Re: [Qemu-devel] [Bug 636315] [NEW] configure and build errors on Solaris 10 due to /bin/sh usage

2010-09-12 Thread Andreas Färber
Am 12.09.2010 um 19:47 schrieb Blue Swirl: On Sun, Sep 12, 2010 at 5:35 PM, Andreas Färber > wrote: Am 12.09.2010 um 19:22 schrieb Blue Swirl: What is the output of "sh ./tracetool --nop --check-backend"? ./tracetool: syntax error at line 51: `$' unexpected Does this patch fix the problem?

Re: [Qemu-devel] [PATCH 1/3] block: allow migration to work with image files

2010-09-12 Thread Avi Kivity
On 09/12/2010 07:10 PM, Anthony Liguori wrote: On 09/12/2010 11:06 AM, Avi Kivity wrote: On 09/12/2010 05:26 PM, Anthony Liguori wrote: On 09/12/2010 08:28 AM, Avi Kivity wrote: On 09/12/2010 03:06 PM, Anthony Liguori wrote: Backing files and logical size shouldn't change during live mig

Re: [Qemu-devel] [RFC] qed: Add QEMU Enhanced Disk format

2010-09-12 Thread Avi Kivity
On 09/12/2010 07:09 PM, Anthony Liguori wrote: On 09/12/2010 10:56 AM, Avi Kivity wrote: No, the worst case is 0.003% allocated disk, with the allocated clusters distributed uniformly. That means all your L2s are allocated, but almost none of your clusters are. But in this case, you're so s

Re: [Qemu-devel] [Bug 636315] [NEW] configure and build errors on Solaris 10 due to /bin/sh usage

2010-09-12 Thread Blue Swirl
On Sun, Sep 12, 2010 at 5:35 PM, Andreas Färber wrote: > Am 12.09.2010 um 19:22 schrieb Blue Swirl: > >> On Sun, Sep 12, 2010 at 11:26 AM, Andreas Färber >> <636...@bugs.launchpad.net> wrote: >>> >>> Error: invalid trace backend >>> Please choose a supported trace backend. >> >> What is the output

Re: [Qemu-devel] [Bug 636315] [NEW] configure and build errors on Solaris 10 due to /bin/sh usage

2010-09-12 Thread Andreas Färber
Am 12.09.2010 um 19:22 schrieb Blue Swirl: On Sun, Sep 12, 2010 at 11:26 AM, Andreas Färber <636...@bugs.launchpad.net> wrote: Error: invalid trace backend Please choose a supported trace backend. What is the output of "sh ./tracetool --nop --check-backend"? ./tracetool: syntax error at lin

[Qemu-devel] [Bug 636446] Re: prep ppc "machine" no more working

2010-09-12 Thread Andreas Färber
This is a known issue, nobody has volunteered to update PReP emulation yet. The last partially working version was v0.9.1, using an OpenHackWare ROM file. Depending on why you want to emulate a PReP machine, you can either try supplying some proprietary ROM file, or contribute PReP support to Open

Re: [Qemu-devel] QEMU interfaces for image streaming and post-copy block migration

2010-09-12 Thread Avi Kivity
On 09/12/2010 07:19 PM, Anthony Liguori wrote: On 09/12/2010 11:45 AM, Avi Kivity wrote: Streaming relies on copy-on-read to do the writing. Ah. You can avoid the copy-on-read implementation in the block format driver and do it completely in generic code. Copy on read takes advantage of

[Qemu-devel] Re: [PATCH v3] Introduce qemu_madvise()

2010-09-12 Thread Blue Swirl
On Sun, Sep 12, 2010 at 12:55 PM, Andreas Färber wrote: > From: Andreas Färber > > vl.c has a Sun-specific hack to supply a prototype for madvise(), > but the call site has apparently moved to arch_init.c. > > Haiku doesn't implement madvise() in favor of posix_madvise(). > OpenBSD and Solaris 10

Re: [Qemu-devel] [Bug 636315] [NEW] configure and build errors on Solaris 10 due to /bin/sh usage

2010-09-12 Thread Blue Swirl
On Sun, Sep 12, 2010 at 11:26 AM, Andreas Färber <636...@bugs.launchpad.net> wrote: > Public bug reported: > > Running `LANG=C LC_ALL=C ./configure --prefix=... > --install=/usr/ucb/install` on Solaris 10 amd64 results in the following > errors: > > ./configure: bad substitution > ./configure: !: n

Re: [Qemu-devel] QEMU interfaces for image streaming and post-copy block migration

2010-09-12 Thread Anthony Liguori
On 09/12/2010 11:45 AM, Avi Kivity wrote: Streaming relies on copy-on-read to do the writing. Ah. You can avoid the copy-on-read implementation in the block format driver and do it completely in generic code. Copy on read takes advantage of temporal locality. You wouldn't want to stream

[Qemu-devel] [Bug 611640] Re: snapshot mode is broken for raw images

2010-09-12 Thread Victor Shkamerda
** Changed in: qemu Status: New => Fix Committed -- snapshot mode is broken for raw images https://bugs.launchpad.net/bugs/611640 You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. Status in QEMU: Fix Committed Bug description: The

Re: [Qemu-devel] [PATCH 01/14] trace: Add trace-events file for declaring trace events

2010-09-12 Thread Stefan Hajnoczi
On Sat, Sep 11, 2010 at 10:53 PM, Andreas Färber wrote: > Am 06.09.2010 um 17:13 schrieb Stefan Hajnoczi: > >> diff --git a/Makefile b/Makefile >> index f95cc2f..3c5e6a0 100644 >> --- a/Makefile >> +++ b/Makefile >> @@ -1,6 +1,6 @@ >> # Makefile for QEMU. >> >> -GENERATED_HEADERS = config-host.h >

[Qemu-devel] [Bug 636446] Re: prep ppc "machine" no more working

2010-09-12 Thread Andreas Färber
** Changed in: qemu Status: New => Confirmed -- prep ppc "machine" no more working https://bugs.launchpad.net/bugs/636446 You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. Status in QEMU: Confirmed Bug description: I have tried qe

[Qemu-devel] [Bug 611646] Re: isa bus not working

2010-09-12 Thread Victor Shkamerda
** Changed in: qemu Status: New => Fix Committed -- isa bus not working https://bugs.launchpad.net/bugs/611646 You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. Status in QEMU: Fix Committed Bug description: isa bus emulation not

Re: [Qemu-devel] [PATCH 1/3] block: allow migration to work with image files

2010-09-12 Thread Anthony Liguori
On 09/12/2010 11:06 AM, Avi Kivity wrote: On 09/12/2010 05:26 PM, Anthony Liguori wrote: On 09/12/2010 08:28 AM, Avi Kivity wrote: On 09/12/2010 03:06 PM, Anthony Liguori wrote: Backing files and logical size shouldn't change during live migration. Why not? To make our lives easier. I

Re: [Qemu-devel] [RFC] qed: Add QEMU Enhanced Disk format

2010-09-12 Thread Anthony Liguori
On 09/12/2010 10:56 AM, Avi Kivity wrote: No, the worst case is 0.003% allocated disk, with the allocated clusters distributed uniformly. That means all your L2s are allocated, but almost none of your clusters are. But in this case, you're so sparse that your metadata is pretty much co-locat

Re: [Qemu-devel] QEMU interfaces for image streaming and post-copy block migration

2010-09-12 Thread Avi Kivity
On 09/12/2010 05:23 PM, Anthony Liguori wrote: On 09/12/2010 08:40 AM, Avi Kivity wrote: Why would it serialize all I/O operations? It's just like another vcpu issuing reads. Because the block layer isn't re-entrant. A threaded block layer is reentrant. Of course pushing the thing into a

Re: [Qemu-devel] [PATCH 3/3] disk: don't read from disk until the guest starts

2010-09-12 Thread Avi Kivity
On 09/12/2010 05:29 PM, Anthony Liguori wrote: On 09/12/2010 08:26 AM, Avi Kivity wrote: On 09/12/2010 03:08 PM, Anthony Liguori wrote: This can cause a disk read, no? Shouldn't it be made asynchronous? Yes, it should. I'm not sure there's an easy way to make it asynchronous though not

Re: [Qemu-devel] [PATCH 1/3] block: allow migration to work with image files

2010-09-12 Thread Avi Kivity
On 09/12/2010 05:26 PM, Anthony Liguori wrote: On 09/12/2010 08:28 AM, Avi Kivity wrote: On 09/12/2010 03:06 PM, Anthony Liguori wrote: Backing files and logical size shouldn't change during live migration. Why not? To make our lives easier. It means management needs to block volume re

Re: [Qemu-devel] [RFC] qed: Add QEMU Enhanced Disk format

2010-09-12 Thread Avi Kivity
On 09/12/2010 05:13 PM, Anthony Liguori wrote: On 09/12/2010 08:24 AM, Avi Kivity wrote: Not atexit, just when we close the image. Just a detail, but we need an atexit() handler to make sure block devices get closed because we have too many exit()s in the code today. Right. So when you

[Qemu-devel] [Bug 636446] [NEW] prep ppc "machine" no more working

2010-09-12 Thread Massimo Montecchi
Public bug reported: I have tried qemu from 0.11 to latest git (as of 11/09/2010), but if i want to use PREP PPC "machine" the error is the same: qemu: hardware error: PowerPC 601 / 620 / 970 need a 1MB BIOS CPU #0: NIP LR CTR XER MSR HID0

[Qemu-devel] [Bug 623852] Re: PPC emulation loops on booting a FreeBSD kernel

2010-09-12 Thread Massimo Montecchi
Also I can confirm that I have this problem on QEMU. I had tried booting FreeBSD8.1-ppc under QEMU (Linux x86_64 host; PPC guest) but there seems to be a problem with whatever the FreeBSD8.1 kernel does, that QEMU's PPC emulation can't handle. I am using the latest version of QEMU from GIT as of

Re: [Qemu-devel] [PATCH 3/3] disk: don't read from disk until the guest starts

2010-09-12 Thread Anthony Liguori
On 09/12/2010 08:26 AM, Avi Kivity wrote: On 09/12/2010 03:08 PM, Anthony Liguori wrote: This can cause a disk read, no? Shouldn't it be made asynchronous? Yes, it should. I'm not sure there's an easy way to make it asynchronous though not because of the block layer but because of how th

[Qemu-devel] Re: [PATCH 3/3] pci: improve signature of pci_register_bar().

2010-09-12 Thread Michael S. Tsirkin
On Thu, Sep 09, 2010 at 11:48:57AM +0900, Isaku Yamahata wrote: > Make type uint8_t from int because PCIIORegion::type is uint8_t. > > Signed-off-by: Isaku Yamahata > --- > hw/pci.c |2 +- > hw/pci.h |2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) Applied, thanks. > diff --g

[Qemu-devel] Re: [PATCH 2/3] pci: don't ignore invalid parameter for pci_register_bar().

2010-09-12 Thread Michael S. Tsirkin
On Thu, Sep 09, 2010 at 11:48:56AM +0900, Isaku Yamahata wrote: > Abort when invalid value for region_num is passed to pci_register_bar. > That is caller's bug. Abort instead of silently ignoring invalid value. > > Signed-off-by: Isaku Yamahata > --- > hw/pci.c |5 ++--- > 1 files changed, 2

Re: [Qemu-devel] [PATCH 1/3] block: allow migration to work with image files

2010-09-12 Thread Anthony Liguori
On 09/12/2010 08:28 AM, Avi Kivity wrote: On 09/12/2010 03:06 PM, Anthony Liguori wrote: Backing files and logical size shouldn't change during live migration. Why not? To make our lives easier. Regards, Anthony Liguori But even so, I think the interface make sense. It's basically, d

[Qemu-devel] Re: [PATCH 1/3] pci: sorting out type confusion in pci_register_bar().

2010-09-12 Thread Michael S. Tsirkin
On Thu, Sep 09, 2010 at 11:48:55AM +0900, Isaku Yamahata wrote: > This patch sorts out invalid use of pcibus_t. > > In pci_register_bar(), pcibus_t wmask is used. > It should, However, be uint64_t because it is used to set > pci configuration space value(PCIDevice::wmask) by pci_set_quad() > or pc

Re: [Qemu-devel] QEMU interfaces for image streaming and post-copy block migration

2010-09-12 Thread Anthony Liguori
On 09/12/2010 08:40 AM, Avi Kivity wrote: Why would it serialize all I/O operations? It's just like another vcpu issuing reads. Because the block layer isn't re-entrant. What you basically do is: stream_step_three(): complete() stream_step_two(offset, length): bdrv_aio_readv(offset,

[Qemu-devel] [PATCH] trace: Fix user emulator dependency on trace objects

2010-09-12 Thread Andreas Färber
On a clean build, after generating trace.h, make would recurse into *-*-user without a clue how to build ../trace.o (added to $(obj-y) in Makefile.target) since its generation rule is in the main Makefile. The softmmus are seemingly unaffected because the $(TOOLS), which each have a dependency on $

Re: [Qemu-devel] Stable patch tracking

2010-09-12 Thread Anthony Liguori
On 09/12/2010 08:42 AM, Avi Kivity wrote: On 09/12/2010 03:04 PM, Anthony Liguori wrote: Can the captcha for logged in users be removed? Can you be more specific about what you're seeing? There should only be a captcha for creating an account. I've never seen it since I created my account

Re: [Qemu-devel] [RFC] qed: Add QEMU Enhanced Disk format

2010-09-12 Thread Anthony Liguori
On 09/12/2010 08:24 AM, Avi Kivity wrote: Not atexit, just when we close the image. Just a detail, but we need an atexit() handler to make sure block devices get closed because we have too many exit()s in the code today. Right. So when you click the 'X' on the qemu window, we get to wait

Re: [Qemu-devel] [PATCH 2/2] qemu-kvm: Add svm cpuid features

2010-09-12 Thread Joerg Roedel
On Sun, Sep 12, 2010 at 10:01:28AM +0200, Avi Kivity wrote: > On 09/12/2010 09:16 AM, Alexander Graf wrote: >> Either way, I don't think we need a phenom2 type. The features >> additional are minor enough to not really matter and all use cases I >> can come up with require either -cpu host (local

Re: [Qemu-devel] [PATCH 2/2] qemu-kvm: Add svm cpuid features

2010-09-12 Thread Avi Kivity
On 09/12/2010 04:30 PM, Joerg Roedel wrote: Either way, I don't think we need a phenom2 type. The features additional are minor enough to not really matter and all use cases I can come up with require either -cpu host (local virt) or -cpu phenom (migration). I'm fine with this (or with adding

[Qemu-devel] Copy, Cut, Paste - Files/Text

2010-09-12 Thread Dhananjay Goel
Hi there! I'm using qemu for quite a while and was searching for clipboard sharing between guests and host. I got some info of copy/paste using VNC. I would appreciate if somebody can help me in finding if the following things are supported directly using Qemu: 1) Copy/Paste Files between Host an

Re: [Qemu-devel] Stable patch tracking

2010-09-12 Thread Avi Kivity
On 09/12/2010 03:42 PM, Avi Kivity wrote: On 09/12/2010 03:04 PM, Anthony Liguori wrote: Can the captcha for logged in users be removed? Can you be more specific about what you're seeing? There should only be a captcha for creating an account. I've never seen it since I created my accoun

Re: [Qemu-devel] [PATCH 3/3] disk: don't read from disk until the guest starts

2010-09-12 Thread Avi Kivity
On 09/12/2010 03:08 PM, Anthony Liguori wrote: This can cause a disk read, no? Shouldn't it be made asynchronous? Yes, it should. I'm not sure there's an easy way to make it asynchronous though not because of the block layer but because of how these functions are called. Sorry to harp o

Re: [Qemu-devel] Stable patch tracking

2010-09-12 Thread Avi Kivity
On 09/12/2010 03:04 PM, Anthony Liguori wrote: Can the captcha for logged in users be removed? Can you be more specific about what you're seeing? There should only be a captcha for creating an account. I've never seen it since I created my account. I updated the page (logged in as avi)

Re: [Qemu-devel] [PATCH 1/3] block: allow migration to work with image files

2010-09-12 Thread Avi Kivity
On 09/12/2010 03:06 PM, Anthony Liguori wrote: Backing files and logical size shouldn't change during live migration. Why not? But even so, I think the interface make sense. It's basically, drop anything you have cached that may change during migration. What needs to be read is depende

[Qemu-devel] Re: [PATCH v2 2/9] pcie: helper functions for pcie extended capability.

2010-09-12 Thread Michael S. Tsirkin
On Wed, Sep 08, 2010 at 04:39:35PM +0900, Isaku Yamahata wrote: > +#define PCI_EXP_SLTCTL_AIC_SHIFT6 > +#define PCI_EXP_SLTCTL_AIC_ON (PCI_EXP_HP_IND_ON << > PCI_EXP_SLTCTL_AIC_SHIFT) > +#define PCI_EXP_SLTCTL_AIC_BLINK(PCI_EXP_HP_IND_BLINK << > PCI_EXP_SLTCTL_AIC_SHIFT)

Re: [Qemu-devel] QEMU interfaces for image streaming and post-copy block migration

2010-09-12 Thread Avi Kivity
On 09/12/2010 03:25 PM, Anthony Liguori wrote: On 09/12/2010 07:41 AM, Avi Kivity wrote: On 09/07/2010 05:57 PM, Anthony Liguori wrote: I agree that streaming should be generic, like block migration. The trivial generic implementation is: void bdrv_stream(BlockDriverState* bs) { for (s

Re: [Qemu-devel] QEMU interfaces for image streaming and post-copy block migration

2010-09-12 Thread Anthony Liguori
On 09/12/2010 07:41 AM, Avi Kivity wrote: On 09/07/2010 05:57 PM, Anthony Liguori wrote: I agree that streaming should be generic, like block migration. The trivial generic implementation is: void bdrv_stream(BlockDriverState* bs) { for (sector = 0; sector< bdrv_getlength(bs); sector +=

Re: [Qemu-devel] [RFC] qed: Add QEMU Enhanced Disk format

2010-09-12 Thread Avi Kivity
On 09/10/2010 08:07 PM, Anthony Liguori wrote: On 09/10/2010 10:49 AM, Avi Kivity wrote: If I do a qemu-img create -f qcow2 foo.img 10GB, and then do a naive copy of the image file and end up with a 2GB image when there's nothing in it, that's badness. Only if you crash in the middle. If

Re: [Qemu-devel] Stable patch tracking

2010-09-12 Thread Anthony Liguori
On 09/12/2010 05:08 AM, Avi Kivity wrote: On 09/07/2010 09:24 PM, Anthony Liguori wrote: A lot of folks tend to reply to patches and suggest them for stable which is certainly appreciated but is very difficult for me to track reliably. So here's another way to nominate a patch for inclusion

Re: [Qemu-devel] [PATCH 3/3] disk: don't read from disk until the guest starts

2010-09-12 Thread Anthony Liguori
On 09/12/2010 05:42 AM, Avi Kivity wrote: On 09/11/2010 05:04 PM, Anthony Liguori wrote: This fixes a couple nasty problems relating to live migration. 1) When dealing with shared storage with weak coherence (i.e. NFS), even if we re-read, we may end up with undesired caching. By delayin

Re: [Qemu-devel] [PATCH 1/3] block: allow migration to work with image files

2010-09-12 Thread Anthony Liguori
On 09/12/2010 05:37 AM, Avi Kivity wrote: On 09/11/2010 05:04 PM, Anthony Liguori wrote: Image files have two types of data: immutable data that describes things like image size, backing files, etc. and mutable data that includes offset and reference count tables. Note: even the logical si

Re: [Qemu-devel] [RFC][PATCH 0/3] Fix caching issues with live migration

2010-09-12 Thread Anthony Liguori
On 09/12/2010 05:46 AM, Avi Kivity wrote: On 09/11/2010 05:04 PM, Anthony Liguori wrote: Today, live migration only works when using shared storage that is fully cache coherent using raw images. The failure case with weak coherent (i.e. NFS) is subtle but nontheless still exists. NFS only gu

[Qemu-devel] [PATCH v3] Introduce qemu_madvise()

2010-09-12 Thread Andreas Färber
From: Andreas Färber vl.c has a Sun-specific hack to supply a prototype for madvise(), but the call site has apparently moved to arch_init.c. Haiku doesn't implement madvise() in favor of posix_madvise(). OpenBSD and Solaris 10 don't implement posix_madvise() but madvise(). Check for madvise()

Re: [Qemu-devel] QEMU interfaces for image streaming and post-copy block migration

2010-09-12 Thread Avi Kivity
On 09/07/2010 05:57 PM, Anthony Liguori wrote: I agree that streaming should be generic, like block migration. The trivial generic implementation is: void bdrv_stream(BlockDriverState* bs) { for (sector = 0; sector< bdrv_getlength(bs); sector += n) { if (!bdrv_is_allocated(bs, s

Re: [Qemu-devel] [PATCH 2/2] qemu-kvm: Add svm cpuid features

2010-09-12 Thread Avi Kivity
On 09/12/2010 01:14 PM, Alexander Graf wrote: Uh, something in the power management module. Just give it a try on an Istambul system. Just use the 11.1 iso - the kernels are close enough. Oh, probably some silly msr. -- error compiling committee.c: too many arguments to function

[Qemu-devel] [Bug 636315] [NEW] configure and build errors on Solaris 10 due to /bin/sh usage

2010-09-12 Thread Andreas Färber
Public bug reported: Running `LANG=C LC_ALL=C ./configure --prefix=... --install=/usr/ucb/install` on Solaris 10 amd64 results in the following errors: ./configure: bad substitution ./configure: !: not found ./configure: curl-config: not found ./configure: curl-config: not found Error: invalid t

Re: [Qemu-devel] [PATCH 2/2] qemu-kvm: Add svm cpuid features

2010-09-12 Thread Alexander Graf
Am 12.09.2010 um 12:22 schrieb Avi Kivity : > On 09/12/2010 12:06 PM, Alexander Graf wrote: For the Phenom type, I honestly don't remember why, but there was also a good reason to add it. In fact, I use it today to have nested virt without -cpu host on hardware that's too new for

Re: [Qemu-devel] QEMU interfaces for image streaming and post-copy block migration

2010-09-12 Thread Avi Kivity
On 09/07/2010 04:41 PM, Anthony Liguori wrote: Hi, We've got copy-on-read and image streaming working in QED and before going much further, I wanted to bounce some interfaces off of the libvirt folks to make sure our final interface makes sense. Here's the basic idea: Today, you can create

Re: [Qemu-devel] [RFC][PATCH 0/3] Fix caching issues with live migration

2010-09-12 Thread Avi Kivity
On 09/11/2010 05:04 PM, Anthony Liguori wrote: Today, live migration only works when using shared storage that is fully cache coherent using raw images. The failure case with weak coherent (i.e. NFS) is subtle but nontheless still exists. NFS only guarantees close-to-open coherence and when pe

Re: [Qemu-devel] [PATCH 3/3] disk: don't read from disk until the guest starts

2010-09-12 Thread Avi Kivity
On 09/11/2010 05:04 PM, Anthony Liguori wrote: This fixes a couple nasty problems relating to live migration. 1) When dealing with shared storage with weak coherence (i.e. NFS), even if we re-read, we may end up with undesired caching. By delaying any reads until we absolutely have to,

Re: [Qemu-devel] [PATCH 1/3] block: allow migration to work with image files

2010-09-12 Thread Avi Kivity
On 09/11/2010 05:04 PM, Anthony Liguori wrote: Image files have two types of data: immutable data that describes things like image size, backing files, etc. and mutable data that includes offset and reference count tables. Note: even the logical size is, in principle, mutable. If we introduc

Re: [Qemu-devel] [PATCH 2/2] qemu-kvm: Add svm cpuid features

2010-09-12 Thread Avi Kivity
On 09/12/2010 12:06 PM, Alexander Graf wrote: For the Phenom type, I honestly don't remember why, but there was also a good reason to add it. In fact, I use it today to have nested virt without -cpu host on hardware that's too new for my guests. Curious, what guests balk at modern hardware bu

Re: [Qemu-devel] Stable patch tracking

2010-09-12 Thread Avi Kivity
On 09/07/2010 09:24 PM, Anthony Liguori wrote: A lot of folks tend to reply to patches and suggest them for stable which is certainly appreciated but is very difficult for me to track reliably. So here's another way to nominate a patch for inclusion in -stable: http://wiki.qemu.org/Releases/

Re: [Qemu-devel] [PATCH 2/2] qemu-kvm: Add svm cpuid features

2010-09-12 Thread Alexander Graf
Am 12.09.2010 um 10:01 schrieb Avi Kivity : > On 09/12/2010 09:16 AM, Alexander Graf wrote: Depends on which Phenom you have. A Phenom II has NRIPSAVE but the old Phenoms don't have it. For the SVM features it is not that important what the host hardware supports but what KVM

[Qemu-devel] Re: [PATCH] Introduce qemu_madvise()

2010-09-12 Thread Blue Swirl
On Sun, Sep 12, 2010 at 8:50 AM, Andreas Färber wrote: > Am 12.09.2010 um 09:20 schrieb Blue Swirl: > >> On Sat, Sep 11, 2010 at 5:05 PM, Andreas Färber >> wrote: >>> >>> Use qemu_madvise() in arch_init.c's ram_load(). >>> >>> >>> http://www.opengroup.org/onlinepubs/9699919799/functions/posix_mad

[Qemu-devel] Re: [PATCH] Introduce qemu_madvise()

2010-09-12 Thread Andreas Färber
Am 12.09.2010 um 09:20 schrieb Blue Swirl: On Sat, Sep 11, 2010 at 5:05 PM, Andreas Färber > wrote: Use qemu_madvise() in arch_init.c's ram_load(). http://www.opengroup.org/onlinepubs/9699919799/functions/posix_madvise.html Remaining madvise() users: exec.c: limited to __linux__ and/or MADV_ME

Re: [Qemu-devel] [PATCH 2/2] qemu-kvm: Add svm cpuid features

2010-09-12 Thread Avi Kivity
On 09/12/2010 09:16 AM, Alexander Graf wrote: Depends on which Phenom you have. A Phenom II has NRIPSAVE but the old Phenoms don't have it. For the SVM features it is not that important what the host hardware supports but what KVM can emulate. VMCBCLEAN can be emulated without supporting it in

Re: [Qemu-devel] Re: [PATCH v2 2/9] pcie: helper functions for pcie extended capability.

2010-09-12 Thread Blue Swirl
On Wed, Sep 8, 2010 at 5:38 PM, Wei Xu wrote: > Isaku: > > For binary constants below, to achieve max compatibility with gcc versions, > I recommend to change to hex (0x...): Yes, binary constants were only added to GCC 4.3.x. Since they are also GCC extensions with no obvious way to circumvent t

[Qemu-devel] Re: [PATCH] Introduce qemu_madvise()

2010-09-12 Thread Blue Swirl
On Sat, Sep 11, 2010 at 5:05 PM, Andreas Färber wrote: > From: Andreas Färber > > vl.c has a Sun-specific hack to supply a prototype for madvise(), > but the call site has apparently moved to arch_init.c. > The underlying issue is that madvise() is not a POSIX function, > therefore Solaris' _POSI

Re: [Qemu-devel] [PATCH 2/2] qemu-kvm: Add svm cpuid features

2010-09-12 Thread Alexander Graf
On 12.09.2010, at 08:05, Avi Kivity wrote: > On 09/11/2010 05:20 PM, Joerg Roedel wrote: >> On Sat, Sep 11, 2010 at 03:43:02PM +0200, Alexander Graf wrote: @@ -305,6 +322,8 @@ static x86_def_t builtin_x86_defs[] = { CPUID_EXT3_OSVW, CPUID_EXT3_IBS */ .ex