[Qemu-devel] KVM call agenda for May 18

2010-05-17 Thread Chris Wright
Please send in any agenda items you are interested in covering. If we have a lack of agenda items I'll cancel the week's call. thanks, -chris

[Qemu-devel] qemu usage

2010-05-17 Thread K D
Hi I am able to boot my custom kernel off of bare hardware, but when I try to boot this with qemu it hangs after first two lines of print about BIOS. i tried booting through qemu image and also via '-kernel' arg. same result in both cases. i verified that with same qeumu-system-x86_64 i could b

Re: [Qemu-devel] [PATCH] Add QEMU DirectFB display driver

2010-05-17 Thread Anthony Liguori
On 05/17/2010 05:55 PM, Alexander Graf wrote: IMHO, the problem with an external GUI is that the interaction just gets too complicated. What interaction do you mean? I'm not advocating to move the GUI to a different project. I was more thinking of a separation like with perf where the ke

Re: [Qemu-devel] [PATCH] Add QEMU DirectFB display driver

2010-05-17 Thread Alexander Graf
On 18.05.2010, at 00:54, Anthony Liguori wrote: > On 05/17/2010 05:49 PM, Alexander Graf wrote: Then we could still offer a separate SDL based viewer that could do the same things it does now. But we'd also open up the gate for a whole new integration level with possible GUIs. >

Re: [Qemu-devel] [PATCH] Add QEMU DirectFB display driver

2010-05-17 Thread Alexander Graf
On 18.05.2010, at 00:47, Anthony Liguori wrote: > On 05/17/2010 05:42 PM, malc wrote: >>> In fact, couldn't we rather keep all graphic output out of qemu and >>> just expose VNC, possibly with self-made additions to the protocol >>> to speed up local rendering (thinking an SHM extension here)? Th

Re: [Qemu-devel] [PATCH] Add QEMU DirectFB display driver

2010-05-17 Thread Anthony Liguori
On 05/17/2010 05:49 PM, Alexander Graf wrote: Then we could still offer a separate SDL based viewer that could do the same things it does now. But we'd also open up the gate for a whole new integration level with possible GUIs. You could, but I think it introduces more complexity wh

Re: [Qemu-devel] [PATCH] Add QEMU DirectFB display driver

2010-05-17 Thread Alexander Graf
On 18.05.2010, at 00:46, Anthony Liguori wrote: > On 05/17/2010 05:26 PM, Alexander Graf wrote: >> I'm trying to think of a project where the clean separation between multiple >> video outputs implemented in the backend and a separate frontend worked out. >> So far the only case that has a stri

Re: [Qemu-devel] [PATCH] Add QEMU DirectFB display driver

2010-05-17 Thread Anthony Liguori
On 05/17/2010 05:42 PM, malc wrote: In fact, couldn't we rather keep all graphic output out of qemu and just expose VNC, possibly with self-made additions to the protocol to speed up local rendering (thinking an SHM extension here)? Then we could still offer a separate SDL based viewer that could

Re: [Qemu-devel] [PATCH] Add QEMU DirectFB display driver

2010-05-17 Thread Anthony Liguori
On 05/17/2010 05:26 PM, Alexander Graf wrote: I'm trying to think of a project where the clean separation between multiple video outputs implemented in the backend and a separate frontend worked out. So far the only case that has a strikingly similar architecture coming to my mind is mplayer.

Re: [Qemu-devel] [PATCH] Add QEMU DirectFB display driver

2010-05-17 Thread malc
On Tue, 18 May 2010, Alexander Graf wrote: > > On 17.05.2010, at 23:45, malc wrote: > > > On Mon, 17 May 2010, Anthony Liguori wrote: > > > >> On 05/17/2010 04:35 PM, malc wrote: > >>> There's one thing that SDL does marvelously well - it's just one fairly > >>> small and self contained library

Re: [Qemu-devel] [PATCH] Add QEMU DirectFB display driver

2010-05-17 Thread Alexander Graf
On 17.05.2010, at 23:45, malc wrote: > On Mon, 17 May 2010, Anthony Liguori wrote: > >> On 05/17/2010 04:35 PM, malc wrote: >>> There's one thing that SDL does marvelously well - it's just one fairly >>> small and self contained library that doesn't unleash dependency hell on >>> the user. >>>

Re: [Qemu-devel] [PATCH] Add QEMU DirectFB display driver

2010-05-17 Thread malc
On Mon, 17 May 2010, Anthony Liguori wrote: > On 05/17/2010 04:35 PM, malc wrote: > > There's one thing that SDL does marvelously well - it's just one fairly > > small and self contained library that doesn't unleash dependency hell on > > the user. > > > > > The fact that we have cocoa support

Re: [Qemu-devel] [PATCH] Add QEMU DirectFB display driver

2010-05-17 Thread Anthony Liguori
On 05/17/2010 04:35 PM, malc wrote: There's one thing that SDL does marvelously well - it's just one fairly small and self contained library that doesn't unleash dependency hell on the user. The fact that we have cocoa support in the tree is basically an admission of failure with SDL.

Re: [Qemu-devel] [PATCH] Add QEMU DirectFB display driver

2010-05-17 Thread malc
On Mon, 17 May 2010, Anthony Liguori wrote: > On 05/17/2010 03:20 PM, Gerd Hoffmann wrote: > > On 05/16/10 03:10, Paul Brook wrote: > > > > The other solution would be to use the DirectFB driver for SDL which > > > > would allow to do slightly the same as this patch. But that would mean > > > > ha

[Qemu-devel] Re: [QEMU-KVM]: Megasas + TCM_Loop + SG_IO into Windows XP guests

2010-05-17 Thread Nicholas A. Bellinger
On Fri, 2010-05-14 at 02:42 -0700, Nicholas A. Bellinger wrote: > On Fri, 2010-05-14 at 09:22 +0200, Hannes Reinecke wrote: > > Nicholas A. Bellinger wrote: > > > Greetings Hannes and co, > > > > > > Let's see if I can find some time working on the megasas emulation. > > Maybe I find something. >

Re: [Qemu-devel] [PATCH] Add QEMU DirectFB display driver

2010-05-17 Thread Anthony Liguori
On 05/17/2010 03:20 PM, Gerd Hoffmann wrote: On 05/16/10 03:10, Paul Brook wrote: The other solution would be to use the DirectFB driver for SDL which would allow to do slightly the same as this patch. But that would mean having to deal with an additional layer in the graphical stack, which is

[Qemu-devel] Re: [PATCH 3/8] Add QBuffer

2010-05-17 Thread Jamie Lokier
Jan Kiszka wrote: > Jamie Lokier wrote: > > Anthony Liguori wrote: > >> Instead of encoding just as a string, it would be a good idea to encode > >> it as something like: > >> > >> {'__class__': 'base64', 'data': ...} > > > > Is there a benefit to the class indirection, over simply a keyword?: >

Re: [Qemu-devel] [PATCH] Add QEMU DirectFB display driver

2010-05-17 Thread Gerd Hoffmann
On 05/16/10 03:10, Paul Brook wrote: The other solution would be to use the DirectFB driver for SDL which would allow to do slightly the same as this patch. But that would mean having to deal with an additional layer in the graphical stack, which is not exactly what one wants from a performance o

Re: [Qemu-devel] Re: [PATCH] Add cache=volatile parameter to -drive

2010-05-17 Thread Jamie Lokier
Alexander Graf wrote: > > On 17.05.2010, at 18:26, Anthony Liguori wrote: > > > On 05/17/2010 11:23 AM, Paul Brook wrote: > I don't see a difference between the results. Apparently the barrier > option doesn't change a thing. > > >>> Ok. I don't like it, but I can see how i

[Qemu-devel] A20 line control (was Re: [PATCH 0/2] pckbd improvements)

2010-05-17 Thread Jamie Lokier
Blue Swirl wrote: > On 5/16/10, Jamie Lokier wrote: > > Blue Swirl wrote: > > > On 5/16/10, Paolo Bonzini wrote: > > > > On 05/15/2010 11:49 AM, Blue Swirl wrote: > > > > > > > > > In 2/2, A20 logic changes a bit but I doubt any guest would be broken > > > > > if A20 line written through I/O

Re: [Qemu-devel] [PATCH 00/22] tcg-i386 cleanup and improvement, v2

2010-05-17 Thread Aurelien Jarno
Pong. Sorry -ENOTIME for QEMU On Mon, May 17, 2010 at 11:26:37AM -0700, Richard Henderson wrote: > Ping? > > On 04/28/2010 11:24 AM, Richard Henderson wrote: > > Changes v1->v2: > > * Dropped controversial bswap changes; bswap16 continues to use rolw. > > * Tidy data16 as the last of the har

Re: [Qemu-devel] [PATCH] Add QEMU DirectFB display driver

2010-05-17 Thread Gerd Hoffmann
On 05/17/10 14:04, Julian Pidancet wrote: On 05/17/2010 11:53 AM, Gerd Hoffmann wrote: +directfb="no" Should be ="" (aka autodetect). +if test "$directfb" = "yes" ; then + directfb_libs=`directfb-config --libs` + directfb_cflags=`directfb-config --cflags` + libs_softmmu="$directfb_lib

Re: [Qemu-devel] [PATCH 1/2] QMP: Introduce commands doc

2010-05-17 Thread Gerd Hoffmann
Hi, IMO, that's a serious bug. A slot is a user visible entity, both in that devices can only be hotplugged only as slots, not functions, and to determine the maximum number of devices you can add. If the user knows about it, qemu should too. We can easily represent a slot/device as a qbus wi

[Qemu-devel] [PATCH 2/2] pckbd: improve debugging

2010-05-17 Thread Blue Swirl
Signed-off-by: Blue Swirl --- hw/pckbd.c | 28 +++- 1 files changed, 15 insertions(+), 13 deletions(-) diff --git a/hw/pckbd.c b/hw/pckbd.c index 5d268b4..3812284 100644 --- a/hw/pckbd.c +++ b/hw/pckbd.c @@ -29,6 +29,12 @@ /* debug PC keyboard */ //#define DEBUG_KBD

Re: [Qemu-devel] [PATCH 0/2] two tcg improvements

2010-05-17 Thread Richard Henderson
Ping? Update: The TYPE parameter to tcg_out_mov would be helpful for the S390 port as well. The 32-bit LR is 2 bytes, while the 64-bit LGR is 4 bytes. r~ On 05/03/2010 04:30 PM, Richard Henderson wrote: > The first patch allows the x86-64 port to avoid the REX.W prefix > on moves, by allowing

Re: [Qemu-devel] [PATCH] Fix --enable-user-pie compilation.

2010-05-17 Thread Richard Henderson
Ping? This options really doesn't work atm... r~ On 05/03/2010 03:54 PM, Richard Henderson wrote: > We forgot to propagate -fpie to the libdis-user directory. > > Signed-off-by: Richard Henderson > --- > configure |3 +++ > 1 files changed, 3 insertions(+), 0 deletions(-) > > diff --git

Re: [Qemu-devel] [PATCH 00/22] tcg-i386 cleanup and improvement, v2

2010-05-17 Thread Richard Henderson
Ping? On 04/28/2010 11:24 AM, Richard Henderson wrote: > Changes v1->v2: > * Dropped controversial bswap changes; bswap16 continues to use rolw. > * Tidy data16 as the last of the hard-coded constants. > > > r~ > > > Richard Henderson (22): > tcg-i386: Allocate call-saved registers first

[Qemu-devel] Re: [PULL 00/21] Block patches

2010-05-17 Thread Anthony Liguori
On 05/14/2010 12:10 PM, Kevin Wolf wrote: The following changes since commit 14ac15d3ac8e0ef1c91204e2ac772b6412a6b99e: Anthony Liguori (1): Update SeaBIOS are available in the git repository at: git://repo.or.cz/qemu/kevin.git block Pulled, thanks. Regards, Anthony Liguor

Re: [Qemu-devel] [PATCH 1/2] QMP: Introduce commands doc

2010-05-17 Thread Avi Kivity
On 05/17/2010 09:10 PM, Anthony Liguori wrote: The protocol must be forward looking, or we will need endless fixes for it. But we can always add a domain property to extend the address (with a default domain of 0). That's what I meant by endless fixes. Each one is reasonable by itself,

Re: [Qemu-devel] [PATCH 00/12] [PULL] qemu-kvm.git uq/master queue

2010-05-17 Thread Anthony Liguori
On 05/12/2010 04:24 PM, Marcelo Tosatti wrote: The following changes since commit 54d7cf136f040713095cbc064f62d753bff6f9d2: Markus Armbruster (1): doc: Clean up monitor command function index Pulled. Thanks. Regards, Anthony Liguori are available in the git repository at:

Re: [Qemu-devel] [PATCH 1/2] QMP: Introduce commands doc

2010-05-17 Thread Anthony Liguori
On 05/15/2010 01:19 AM, Avi Kivity wrote: On 05/15/2010 01:54 AM, Luiz Capitulino wrote: On Fri, 14 May 2010 19:03:36 +0200 Markus Armbruster wrote: What about PCI domains? Good point. Better to provide for them neatly now, instead of kludging them in later. When I did this conversion I a

Re: [Qemu-devel] [PATCH 1/2] QMP: Introduce commands doc

2010-05-17 Thread Anthony Liguori
On 05/17/2010 06:19 AM, Markus Armbruster wrote: Avi Kivity writes: On 05/17/2010 11:27 AM, Markus Armbruster wrote: A slot is the hotpluggable entity. Open your computer and you can actually see them. QEMU doesn't really know that. How can

Re: [Qemu-devel] [PULL] virtio, pci fixes

2010-05-17 Thread Anthony Liguori
On 05/12/2010 05:37 PM, Michael S. Tsirkin wrote: The following changes since commit 54d7cf136f040713095cbc064f62d753bff6f9d2: doc: Clean up monitor command function index (2010-05-10 11:36:04 -0500) Pulled. Thanks. Regards, Anthony Liguori are available in the git repository at:

[Qemu-devel] [PATCH 1/2] Compile pckbd only once

2010-05-17 Thread Blue Swirl
Use a qemu_irq to indicate A20 line changes. Move I/O port 92 to pckbd.c. Signed-off-by: Blue Swirl --- Makefile.objs|1 + Makefile.target |8 ++-- default-configs/i386-softmmu.mak |1 + default-configs/mips-softmmu.mak |1 + d

[Qemu-devel] [PATCH 0/2] pckbd improvements v2

2010-05-17 Thread Blue Swirl
This version should not change the i8042 vs. port 92 logic. Blue Swirl (2): Compile pckbd only once pckbd: improve debugging Makefile.objs|1 + Makefile.target |8 +- default-configs/i386-softmmu.mak |1 + default-configs/mips-soft

Re: [Qemu-devel] [PATCH 1/3] Fix %lld or %llx printf format use

2010-05-17 Thread Markus Armbruster
Blue Swirl writes: > Signed-off-by: Blue Swirl > --- > audio/audio_template.h |2 +- > block/curl.c|9 --- > block/parallels.c |7 - > block/qcow2.c |8 -- > darwin-user/commpage.c |2 +- > darwin-user/syscall.c |2 +- > hw/vga

Re: [Qemu-devel] [PATCH 0/3] mingw32 compile fixes

2010-05-17 Thread Markus Armbruster
Blue Swirl writes: > On 5/17/10, Markus Armbruster wrote: >> Blue Swirl writes: >> >> > On 5/16/10, Stefan Weil wrote: >> >> Am 15.05.2010 22:49, schrieb Blue Swirl: >> >> >> >> >> >> > Hi, >> >> > >> >> > With this mingw32 compiler: >> >> > >> >> > $ i586-mingw32msvc-gcc -v >> >> >

[Qemu-devel] Re: [PATCH 0/2] pckbd improvements

2010-05-17 Thread Blue Swirl
On 5/17/10, Paolo Bonzini wrote: > > > > In 2/2, A20 logic changes a bit but I doubt any guest would be broken > > > > if A20 line written through I/O port 92 couldn't be read via i8042. > > > > The reverse (write using i8042 and read port 92) will work. > > > > > > > > > > Why take the risk

Re: [Qemu-devel] [PATCH] Add QEMU DirectFB display driver

2010-05-17 Thread Anthony Liguori
On 05/17/2010 10:09 AM, Julian Pidancet wrote: On 05/17/2010 02:30 PM, Anthony Liguori wrote: On 05/15/2010 08:10 PM, Paul Brook wrote: The other solution would be to use the DirectFB driver for SDL which would allow to do slightly the same as this patch. But that would mean having to

Re: [Qemu-devel] Re: [PATCH] Add cache=volatile parameter to -drive

2010-05-17 Thread Alexander Graf
On 17.05.2010, at 18:26, Anthony Liguori wrote: > On 05/17/2010 11:23 AM, Paul Brook wrote: I don't see a difference between the results. Apparently the barrier option doesn't change a thing. >>> Ok. I don't like it, but I can see how it's compelling. I'd like to >>> see t

Re: [Qemu-devel] Qemu-KVM 0.12.4 Guest entered Paused State

2010-05-17 Thread Anthony Liguori
On 05/17/2010 10:16 AM, Peter Lieven wrote: Hi, i have a VM where I just did a dist-upgrade in Ubuntu and the VM entered paused state without any obvious reason. Is there a way to debug why this happened? I have not touched the VM by now to leave the opportunity to debug. You have run out

Re: [Qemu-devel] Re: [PATCH] Add cache=volatile parameter to -drive

2010-05-17 Thread Anthony Liguori
On 05/17/2010 11:23 AM, Paul Brook wrote: I don't see a difference between the results. Apparently the barrier option doesn't change a thing. Ok. I don't like it, but I can see how it's compelling. I'd like to see the documentation improved though. I also think a warning printed on st

Re: [Qemu-devel] Re: [PATCH] Add cache=volatile parameter to -drive

2010-05-17 Thread Paul Brook
> > I don't see a difference between the results. Apparently the barrier > > option doesn't change a thing. > > Ok. I don't like it, but I can see how it's compelling. I'd like to > see the documentation improved though. I also think a warning printed > on stdio about the safety of the option w

Re: [Qemu-devel] [PATCH 0/3] mingw32 compile fixes

2010-05-17 Thread Blue Swirl
On 5/17/10, Markus Armbruster wrote: > Blue Swirl writes: > > > On 5/16/10, Stefan Weil wrote: > >> Am 15.05.2010 22:49, schrieb Blue Swirl: > >> > >> > >> > Hi, > >> > > >> > With this mingw32 compiler: > >> > > >> > $ i586-mingw32msvc-gcc -v > >> > Using built-in specs. > >> > Targe

[Qemu-devel] Re: [PATCH] Add cache=volatile parameter to -drive

2010-05-17 Thread Anthony Liguori
On 05/17/2010 09:04 AM, Alexander Graf wrote: Anthony Liguori wrote: On 05/17/2010 08:17 AM, Alexander Graf wrote: On 17.05.2010, at 15:09, Anthony Liguori wrote: On 05/17/2010 08:02 AM, Alexander Graf wrote: My concern is that ext3 exaggerates the cost of fsync

[Qemu-devel] Qemu-KVM 0.12.4 Guest entered Paused State

2010-05-17 Thread Peter Lieven
Hi, i have a VM where I just did a dist-upgrade in Ubuntu and the VM entered paused state without any obvious reason. Is there a way to debug why this happened? I have not touched the VM by now to leave the opportunity to debug. Peter

[Qemu-devel] Re: [PATCH] Add cache=volatile parameter to -drive

2010-05-17 Thread Anthony Liguori
On 05/17/2010 05:14 AM, Alexander Graf wrote: Usually the guest can tell the host to flush data to disk. In some cases we don't want to flush though, but try to keep everything in cache. So let's add a new cache value to -drive that allows us to set the cache policy to most aggressive, disabling

Re: [Qemu-devel] [PATCH] Add QEMU DirectFB display driver

2010-05-17 Thread Julian Pidancet
On 05/17/2010 02:30 PM, Anthony Liguori wrote: > On 05/15/2010 08:10 PM, Paul Brook wrote: >>> The other solution would be to use the DirectFB driver for SDL which >>> would allow to do slightly the same as this patch. But that would mean >>> having to deal with an additional layer in the graphical

Re: [Qemu-devel] Re: [PATCH] Add cache=volatile parameter to -drive

2010-05-17 Thread Alexander Graf
Alexander Graf wrote: > Anthony Liguori wrote: > >> On 05/17/2010 08:17 AM, Alexander Graf wrote: >> >>> On 17.05.2010, at 15:09, Anthony Liguori wrote: >>> >>> >>> On 05/17/2010 08:02 AM, Alexander Graf wrote: >> My concern is that ext3 exaggerate

[Qemu-devel] Re: [PATCH] Add cache=volatile parameter to -drive

2010-05-17 Thread Alexander Graf
Anthony Liguori wrote: > On 05/17/2010 08:17 AM, Alexander Graf wrote: >> On 17.05.2010, at 15:09, Anthony Liguori wrote: >> >> >>> On 05/17/2010 08:02 AM, Alexander Graf wrote: >>> > My concern is that ext3 exaggerates the cost of fsync() which will > result in diminishing value ov

Re: [Qemu-devel] [PATCH] Add QEMU DirectFB display driver

2010-05-17 Thread Anthony Liguori
On 05/15/2010 08:10 PM, Paul Brook wrote: The other solution would be to use the DirectFB driver for SDL which would allow to do slightly the same as this patch. But that would mean having to deal with an additional layer in the graphical stack, which is not exactly what one wants from a performa

[Qemu-devel] Re: [PATCH] Add cache=volatile parameter to -drive

2010-05-17 Thread Anthony Liguori
On 05/17/2010 08:17 AM, Alexander Graf wrote: On 17.05.2010, at 15:09, Anthony Liguori wrote: On 05/17/2010 08:02 AM, Alexander Graf wrote: My concern is that ext3 exaggerates the cost of fsync() which will result in diminishing value over time for this feature as people move to ext

[Qemu-devel] Re: [PATCH 1/2] QMP: Introduce commands doc

2010-05-17 Thread Luiz Capitulino
On Sat, 15 May 2010 10:42:44 +0200 Jan Kiszka wrote: > Luiz Capitulino wrote: > > On Fri, 14 May 2010 19:08:07 +0200 > > Jan Kiszka wrote: > > > >> Avi Kivity wrote: > >>> On 05/14/2010 08:01 PM, Avi Kivity wrote: > On 05/14/2010 07:52 PM, Jan Kiszka wrote: > >> In order not to comprom

[Qemu-devel] Re: [PATCH] Add cache=volatile parameter to -drive

2010-05-17 Thread Alexander Graf
On 17.05.2010, at 15:09, Anthony Liguori wrote: > On 05/17/2010 08:02 AM, Alexander Graf wrote: >>> My concern is that ext3 exaggerates the cost of fsync() which will result >>> in diminishing value over time for this feature as people move to >>> ext4/btrfs. >>> >> There will be ext3 file

[Qemu-devel] Re: [PATCH] Add cache=volatile parameter to -drive

2010-05-17 Thread Anthony Liguori
On 05/17/2010 08:02 AM, Alexander Graf wrote: My concern is that ext3 exaggerates the cost of fsync() which will result in diminishing value over time for this feature as people move to ext4/btrfs. There will be ext3 file systems for years out. Just because people can use better and fast

[Qemu-devel] Re: [PATCH 3/8] Add QBuffer

2010-05-17 Thread Anthony Liguori
On 05/17/2010 02:45 AM, Avi Kivity wrote: On 05/17/2010 10:40 AM, Jan Kiszka wrote: The alternative is to have a schema. Sun RPC/XDR doesn't carry any type information (you can't even distinguish between a number and text) yet C clients have to problem extracting typed information from it.

Re: [Qemu-devel] Re: [RFC PATCH v3 2/3] block: call the snapshot handlers of the protocol drivers

2010-05-17 Thread MORITA Kazutaka
On Mon, May 17, 2010 at 9:20 PM, Kevin Wolf wrote: > Am 17.05.2010 14:19, schrieb MORITA Kazutaka: >> At Mon, 17 May 2010 13:08:08 +0200, >> Kevin Wolf wrote: >>> >>> Am 17.05.2010 12:19, schrieb MORITA Kazutaka:  int bdrv_snapshot_goto(BlockDriverState *bs,                        

[Qemu-devel] Re: [PATCH] Add cache=volatile parameter to -drive

2010-05-17 Thread Alexander Graf
On 17.05.2010, at 14:58, Anthony Liguori wrote: > On 05/17/2010 05:14 AM, Alexander Graf wrote: >> Usually the guest can tell the host to flush data to disk. In some cases we >> don't want to flush though, but try to keep everything in cache. >> >> So let's add a new cache value to -drive that a

[Qemu-devel] Re: [PATCH] Add cache=volatile parameter to -drive

2010-05-17 Thread Anthony Liguori
On 05/17/2010 05:14 AM, Alexander Graf wrote: Usually the guest can tell the host to flush data to disk. In some cases we don't want to flush though, but try to keep everything in cache. So let's add a new cache value to -drive that allows us to set the cache policy to most aggressive, disabling

Re: [Qemu-devel] Re: [PATCH 2/2] Add flush=off parameter to -drive

2010-05-17 Thread Alexander Graf
On 17.05.2010, at 14:41, Christoph Hellwig wrote: > On Fri, May 14, 2010 at 11:16:47AM +0200, Markus Armbruster wrote: >> If you change -drive, please keep the coming separation in mind, and >> avoid parameters that mix up guest and host. > > This new option seems to be exactly that, unfortunate

Re: [Qemu-devel] Re: [PATCH 2/2] Add flush=off parameter to -drive

2010-05-17 Thread Christoph Hellwig
On Fri, May 14, 2010 at 11:16:47AM +0200, Markus Armbruster wrote: > If you change -drive, please keep the coming separation in mind, and > avoid parameters that mix up guest and host. This new option seems to be exactly that, unfortunately..

Re: [Qemu-devel] Re: [PATCH 2/2] Add flush=off parameter to -drive

2010-05-17 Thread Christoph Hellwig
On Tue, May 11, 2010 at 11:11:12PM +0100, Paul Brook wrote: > .. though it may be a kernel bug rather that a qemu bug, depending on the > exact details. Either way, I consider any mode that inhibits host filesystem > write cache but not volatile drive cache to be pretty worthless. Either we >

Re: [Qemu-devel] [PATCH] Add QEMU DirectFB display driver

2010-05-17 Thread Christoph Hellwig
On Mon, May 17, 2010 at 01:14:05PM +0100, Julian Pidancet wrote: > I have not heard about that, sorry. > My test configuration has an i915-like Intel graphic card, and a nearly > standard 2.6.32 linux kernel. No specific patching was required. You're right. I've checked it and the horrible kerne

[Qemu-devel] [PATCH] MIPS DINSU

2010-05-17 Thread Dmitry Antipov
Hello, shouldn't it be in that way? Dmitry --- qemu-0.12.4/target-mips/translate.c 2010-05-17 16:12:58.048661610 +0400 +++ qemu-0.12.4/target-mips/translate.c 2010-05-17 16:13:12.281656754 +0400 @@ -2761,7 +2761,7 @@ case OPC_DINSU: if (lsb > msb) goto fail; -

Re: [Qemu-devel] Re: [RFC PATCH v3 2/3] block: call the snapshot handlers of the protocol drivers

2010-05-17 Thread Kevin Wolf
Am 17.05.2010 14:19, schrieb MORITA Kazutaka: > At Mon, 17 May 2010 13:08:08 +0200, > Kevin Wolf wrote: >> >> Am 17.05.2010 12:19, schrieb MORITA Kazutaka: >>> >>> int bdrv_snapshot_goto(BlockDriverState *bs, >>> const char *snapshot_id) >>> { >>> BlockDriver *drv =

Re: [Qemu-devel] [PATCH] Add QEMU DirectFB display driver

2010-05-17 Thread Julian Pidancet
On 05/17/2010 12:44 PM, Christoph Hellwig wrote: > On Fri, May 14, 2010 at 05:58:50PM +0100, Julian Pidancet wrote: >> This patch implements a DirectFB driver for QEMU. It allows Qemu to >> draw a VM graphic output directly in the framebuffer of the host, >> without having to rely on X11. >> Direct

Re: [Qemu-devel] [PATCH] Add QEMU DirectFB display driver

2010-05-17 Thread Julian Pidancet
On 05/17/2010 11:53 AM, Gerd Hoffmann wrote: >> +directfb="no" > > Should be ="" (aka autodetect). > >> +if test "$directfb" = "yes" ; then >> + directfb_libs=`directfb-config --libs` >> + directfb_cflags=`directfb-config --cflags` >> + libs_softmmu="$directfb_libs $libs_softmmu" >> +fi > >

[Qemu-devel] Re: [RFC PATCH v3 2/3] block: call the snapshot handlers of the protocol drivers

2010-05-17 Thread MORITA Kazutaka
At Mon, 17 May 2010 13:08:08 +0200, Kevin Wolf wrote: > > Am 17.05.2010 12:19, schrieb MORITA Kazutaka: > > > > int bdrv_snapshot_goto(BlockDriverState *bs, > > const char *snapshot_id) > > { > > BlockDriver *drv = bs->drv; > > +int ret, open_ret; > > + > >

Re: [Qemu-devel] [PATCH] Add QEMU DirectFB display driver

2010-05-17 Thread Christoph Hellwig
On Fri, May 14, 2010 at 05:58:50PM +0100, Julian Pidancet wrote: > This patch implements a DirectFB driver for QEMU. It allows Qemu to > draw a VM graphic output directly in the framebuffer of the host, > without having to rely on X11. > DirectFB also provides with a generic interface take advantag

[Qemu-devel] Re: [PATCH 0/2] pckbd improvements

2010-05-17 Thread Paolo Bonzini
> > > In 2/2, A20 logic changes a bit but I doubt any guest would be broken > > > if A20 line written through I/O port 92 couldn't be read via i8042. > > > The reverse (write using i8042 and read port 92) will work. > > > > > > > Why take the risk? > > The alternative is to route a signal from po

Re: [Qemu-devel] Re: [PATCH 00/26] split out piix specific part from pc emulator and some clean ups

2010-05-17 Thread Markus Armbruster
Blue Swirl writes: > Thanks, applied all. Thanks indeed! 13 versions and I-don't-remember-how-many months, must have taxed your patience.

Re: [Qemu-devel] [PATCH 1/2] QMP: Introduce commands doc

2010-05-17 Thread Markus Armbruster
Avi Kivity writes: > On 05/17/2010 11:27 AM, Markus Armbruster wrote: >> >> > A slot is the hotpluggable entity. Open your computer and you can > actually see them. > > QEMU doesn't really know that. >>> How can that be? Do we signal hotplug

Re: [Qemu-devel] [RFC PATCH v3 2/3] block: call the snapshot handlers of the protocol drivers

2010-05-17 Thread Kevin Wolf
Am 17.05.2010 12:19, schrieb MORITA Kazutaka: > When snapshot handlers are not defined in the format driver, it is > better to call the ones of the protocol driver. This enables us to > implement snapshot support in the protocol driver. > > We need to call bdrv_close() and bdrv_open() handlers of

Re: [Qemu-devel] [PATCH] Add QEMU DirectFB display driver

2010-05-17 Thread Gerd Hoffmann
Hi, Can you provide some performance data to justify this since SDL provides the same ability? IMHO no performance data is needed to justify this because SDL running on top of the linux framebuffer is simply unusable IMHO. cheers, Gerd

Re: [Qemu-devel] [PATCH] Add QEMU DirectFB display driver

2010-05-17 Thread Gerd Hoffmann
+directfb="no" Should be ="" (aka autodetect). +if test "$directfb" = "yes" ; then + directfb_libs=`directfb-config --libs` + directfb_cflags=`directfb-config --cflags` + libs_softmmu="$directfb_libs $libs_softmmu" +fi use pkgconfig here. directfb-config most likely is just a pkgconfig

[Qemu-devel] Re: [PATCH] Add cache=volatile parameter to -drive

2010-05-17 Thread Kevin Wolf
Am 17.05.2010 12:14, schrieb Alexander Graf: > Usually the guest can tell the host to flush data to disk. In some cases we > don't want to flush though, but try to keep everything in cache. > > So let's add a new cache value to -drive that allows us to set the cache > policy to most aggressive, di

[Qemu-devel] Re: Qemu-KVM Livate Migration 0.12.2 -> 0.12.3/4 broken?

2010-05-17 Thread Jan Kiszka
Juan Quintela wrote: > Jan Kiszka wrote: >> Juan Quintela wrote: >>> Lack of "proper" subsections. IDE is something like: >>> >>> const VMStateDescription vmstate_ide_drive = { >>> .version_id = 4, >>> >>> } >>> >>> static const VMStateDescription vmstate_bmdma = { >>> .name = "ide b

[Qemu-devel] Re: [RFC PATCH v2 3/3] block: add sheepdog driver for distributed storage support

2010-05-17 Thread MORITA Kazutaka
Hi, Thank you very much for the reviewing! At Fri, 14 May 2010 13:08:06 +0200, Kevin Wolf wrote: > > + > > +struct sd_req { > > + uint8_t proto_ver; > > + uint8_t opcode; > > + uint16_tflags; > > + uint32_tepoch; > > + uint32_tid; > > + uint32_

[Qemu-devel] [PATCH] Add cache=volatile parameter to -drive

2010-05-17 Thread Alexander Graf
Usually the guest can tell the host to flush data to disk. In some cases we don't want to flush though, but try to keep everything in cache. So let's add a new cache value to -drive that allows us to set the cache policy to most aggressive, disabling flushes. We call this mode "volatile", as guest

[Qemu-devel] [RFC PATCH v3 3/3] block: add sheepdog driver for distributed storage support

2010-05-17 Thread MORITA Kazutaka
Sheepdog is a distributed storage system for QEMU. It provides highly available block level storage volumes to VMs like Amazon EBS. This patch adds a qemu block driver for Sheepdog. Sheepdog features are: - No node in the cluster is special (no metadata node, no control node, etc) - Linear scal

[Qemu-devel] [RFC PATCH v3 0/3] Sheepdog: distributed storage system for QEMU

2010-05-17 Thread MORITA Kazutaka
Hi all, This patch adds a block driver for Sheepdog distributed storage system. Changes from v2 to v3 are: - add drv->bdrv_close() and drv->bdrv_open() before and after bdrv_snapshot_goto() call of the protocol. - address the review comments on the sheepdog driver code. I'll send the det

[Qemu-devel] [RFC PATCH v3 1/3] close all the block drivers before the qemu process exits

2010-05-17 Thread MORITA Kazutaka
This patch calls the close handler of the block driver before the qemu process exits. This is necessary because the sheepdog block driver releases the lock of VM images in the close handler. Signed-off-by: MORITA Kazutaka --- block.c |9 + block.h |1 + vl.c|1 + 3 files

[Qemu-devel] [RFC PATCH v3 2/3] block: call the snapshot handlers of the protocol drivers

2010-05-17 Thread MORITA Kazutaka
When snapshot handlers are not defined in the format driver, it is better to call the ones of the protocol driver. This enables us to implement snapshot support in the protocol driver. We need to call bdrv_close() and bdrv_open() handlers of the format driver before and after bdrv_snapshot_goto()

Re: [Qemu-devel] Re: Qemu-KVM Livate Migration 0.12.2 -> 0.12.3/4 broken?

2010-05-17 Thread Alexander Graf
On 17.05.2010, at 11:11, Michael Tokarev wrote: > 17.05.2010 13:07, Juan Quintela wrote: >> Michael Tokarev wrote: > [] >>> http://www.mail-archive.com/k...@vger.kernel.org/msg34051.html >>> I wonder why it is not noticed before -- it's broken since 0.12... >> >> People has become rich and ever

[Qemu-devel] Re: [PATCH 3/8] Add QBuffer

2010-05-17 Thread Avi Kivity
On 05/17/2010 12:17 PM, Jan Kiszka wrote: Avi Kivity wrote: On 05/17/2010 11:55 AM, Jan Kiszka wrote: The names of fields are also type information. Not in the case of device_show. The clients have no idea of the vmstate structures before they were transfered. Granted, tha

[Qemu-devel] Re: [PATCH 3/8] Add QBuffer

2010-05-17 Thread Jan Kiszka
Avi Kivity wrote: > On 05/17/2010 11:55 AM, Jan Kiszka wrote: >>> The names of fields are also type information. >>> >> Not in the case of device_show. The clients have no idea of the vmstate >> structures before they were transfered. Granted, that will likely remain >> a special case in the

[Qemu-devel] Re: Qemu-KVM Livate Migration 0.12.2 -> 0.12.3/4 broken?

2010-05-17 Thread Michael Tokarev
17.05.2010 13:07, Juan Quintela wrote: Michael Tokarev wrote: [] http://www.mail-archive.com/k...@vger.kernel.org/msg34051.html I wonder why it is not noticed before -- it's broken since 0.12... People has become rich and everybody has a 64 bit hardware :-) Actually I don't think there's a

[Qemu-devel] Re: Qemu-KVM Livate Migration 0.12.2 -> 0.12.3/4 broken?

2010-05-17 Thread Juan Quintela
Michael Tokarev wrote: > 17.05.2010 11:00, Avi Kivity wrote: >> On 05/16/2010 11:04 PM, Juan Quintela wrote: > [] We've regressed from failing some migrations to failing all migrations. >>> Humm, 0.12.4 -> 0.12.4 should work. My advise is just revert the patch >>> and live with it for another

[Qemu-devel] Re: [PATCH] block: fix aio_flush segfaults for read-only protocols (e.g. curl)

2010-05-17 Thread Kevin Wolf
Am 16.05.2010 13:59, schrieb Avi Kivity: > Not all block format drivers expose an io_flush method (reasonable for > read-only protocols), so calling io_flush there will immediately segfault. > > Fix by checking for the method's existence before calling it. > > Signed-off-by: Avi Kivity Thanks,

Re: [Qemu-devel] [PATCH 1/2] QMP: Introduce commands doc

2010-05-17 Thread Avi Kivity
On 05/17/2010 11:27 AM, Markus Armbruster wrote: A slot is the hotpluggable entity. Open your computer and you can actually see them. QEMU doesn't really know that. How can that be? Do we signal hotplug notifications to a function or to a slot? Can we hotplug a sing

[Qemu-devel] Re: [PATCH 3/8] Add QBuffer

2010-05-17 Thread Avi Kivity
On 05/17/2010 11:55 AM, Jan Kiszka wrote: The names of fields are also type information. Not in the case of device_show. The clients have no idea of the vmstate structures before they were transfered. Granted, that will likely remain a special case in the QMP command set. For that

[Qemu-devel] Re: [PATCH] net: Remove dead code from net/socket.c

2010-05-17 Thread Jan Kiszka
Miguel Di Ciurcio Filho wrote: > On Fri, May 14, 2010 at 2:03 PM, Jan Kiszka wrote: >>> In both cases the info_str string is written inside >>> net_socket_fd_init_(stream|dgram), and after that, it is >>> overwritten on a subsequent snprintf() in net_socket_accept(). >>> >> There is non-zero time

[Qemu-devel] Re: [PATCH 3/8] Add QBuffer

2010-05-17 Thread Jan Kiszka
Avi Kivity wrote: > On 05/17/2010 10:57 AM, Jan Kiszka wrote: >> Avi Kivity wrote: >> >>> On 05/17/2010 10:40 AM, Jan Kiszka wrote: >>> > The alternative is to have a schema. Sun RPC/XDR doesn't carry any type > information (you can't even distinguish between a number and text) y

Re: [Qemu-devel] [PATCH 0/3] mingw32 compile fixes

2010-05-17 Thread Markus Armbruster
Blue Swirl writes: > On 5/16/10, Stefan Weil wrote: >> Am 15.05.2010 22:49, schrieb Blue Swirl: >> >> >> > Hi, >> > >> > With this mingw32 compiler: >> > >> > $ i586-mingw32msvc-gcc -v >> > Using built-in specs. >> > Target: i586-mingw32msvc >> > Configured with: [...] >> > build will not succee

[Qemu-devel] [PATCH] Fix error handling in qemu_read_config_file

2010-05-17 Thread Kevin Wolf
We need to close the file even in error case. While at it, make the callers catch all kind of errors. ENOENT is allowed for default config files, they are optional. Reported-by: Luiz Capitulino Signed-off-by: Kevin Wolf --- qemu-config.c | 12 vl.c |4 ++-- 2 files c

Re: [Qemu-devel] [PATCH 0/3] mingw32 compile fixes

2010-05-17 Thread Markus Armbruster
Would you mind sending patch series with proper References: headers, so that the parts of the series are properly threaded? You can ask either git-format-patch or git-send-email to add them.

Re: [Qemu-devel] [PATCH 1/2] QMP: Introduce commands doc

2010-05-17 Thread Markus Armbruster
Avi Kivity writes: > On 05/15/2010 01:54 AM, Luiz Capitulino wrote: >> On Fri, 14 May 2010 19:03:36 +0200 >> Markus Armbruster wrote: >> >> What about PCI domains? >>> Good point. Better to provide for them neatly now, instead of kludging >>> them in later. >>> >>

Re: [Qemu-devel] [PATCH 1/2] QMP: Introduce commands doc

2010-05-17 Thread Markus Armbruster
Avi Kivity writes: > On 05/14/2010 08:03 PM, Markus Armbruster wrote: >> Avi Kivity writes: >> >> >>> On 05/14/2010 11:50 AM, Markus Armbruster wrote: [...] We have a list of buses, each containing a list of device functions. Not sure the additional level of nesting you propose buy

[Qemu-devel] Re: Qemu-KVM Livate Migration 0.12.2 -> 0.12.3/4 broken?

2010-05-17 Thread Michael Tokarev
17.05.2010 11:00, Avi Kivity wrote: On 05/16/2010 11:04 PM, Juan Quintela wrote: [] We've regressed from failing some migrations to failing all migrations. Humm, 0.12.4 -> 0.12.4 should work. My advise is just revert the patch and live with it for another week, what do you think? A week is f

[Qemu-devel] Re: [PATCH 3/8] Add QBuffer

2010-05-17 Thread Avi Kivity
On 05/17/2010 11:10 AM, Avi Kivity wrote: Another way of looking at it: if the client sees { __class__: foo, f1: 10, f2: 9 }, it cannot derive any information from __class__ unless it was aware of foo beforehand. If that's the case, let's make it part of the schema so it is available at comp

  1   2   >