[Qemu-devel] [ICON] QEMU Mascot Contest

2011-11-11 Thread 陳韋任
Hi, all I am not a good painter or an artist, but here you go http://people.cs.nctu.edu.tw/~chenwj/slide/QEMU/QEMU_Mascot.png Let me know what you think. :) Regards, chenwj -- Wei-Ren Chen (陳韋任) Computer Systems Lab, Institute of Information Science, Academia Sinica, Taiwan (R.O.C.) Tel:

Re: [Qemu-devel] [PATCH v2 2/3] block: Introduce bdrv_co_flush_to_os

2011-11-11 Thread Kevin Wolf
Am 11.11.2011 05:06, schrieb Zhi Yong Wu: > On Fri, Nov 11, 2011 at 1:32 AM, Kevin Wolf wrote: >> qcow2 has a writeback metadata cache, so flushing a qcow2 image actually >> consists of writing back that cache to the protocol and only then flushes the >> protocol in order to get everything stable

Re: [Qemu-devel] Storage requirements for live migration

2011-11-11 Thread Kevin Wolf
Am 11.11.2011 07:27, schrieb Mark Wu: > On 11/11/2011 08:11 AM, Anthony Liguori wrote: >> I did a brain dump of my understanding of the various storage >> requirements for live migration. I think it's accurate but I may have >> misunderstand some details so I would appreciate review. >> >> I think

Re: [Qemu-devel] Storage requirements for live migration

2011-11-11 Thread Kevin Wolf
Am 11.11.2011 01:11, schrieb Anthony Liguori: > I did a brain dump of my understanding of the various storage requirements > for > live migration. I think it's accurate but I may have misunderstand some > details > so I would appreciate review. > > I think given sections (1) and (2), the only

Re: [Qemu-devel] [PATCH 0/2] net: socket backend uses uninitialized variables

2011-11-11 Thread Markus Armbruster
Ping? Markus Armbruster writes: > Compile-tested only, buyer beware. > > Markus Armbruster (2): > net: socket backend passes junk size to getsockname(), fix > net: socket backend shows junk in "info network", fix > > net/socket.c | 14 ++ > 1 files changed, 10 insertions(+), 4

[Qemu-devel] [PATCH 2/5] ui: Plug memory leaks on parse_keyboard_layout() error path

2011-11-11 Thread Markus Armbruster
Spotted by Coverity. Signed-off-by: Markus Armbruster --- ui/keymaps.c | 12 +++- 1 files changed, 7 insertions(+), 5 deletions(-) diff --git a/ui/keymaps.c b/ui/keymaps.c index f54a114..f55a2aa 100644 --- a/ui/keymaps.c +++ b/ui/keymaps.c @@ -92,15 +92,17 @@ static kbd_layout_t *pars

[Qemu-devel] [PATCH 0/5] Plug a few trivial resource leaks

2011-11-11 Thread Markus Armbruster
As usual, shit happens on error paths. Markus Armbruster (5): qemu-char: Plug memory leak on qemu_chr_open_pty() error path ui: Plug memory leaks on parse_keyboard_layout() error path qemu-sockets: Plug fd leak on unix_connect_opts() error path posix-aio-compat: Plug memory leak on paio_in

[Qemu-devel] [PATCH 1/5] qemu-char: Plug memory leak on qemu_chr_open_pty() error path

2011-11-11 Thread Markus Armbruster
Spotted by Coverity. Signed-off-by: Markus Armbruster --- qemu-char.c | 19 ++- 1 files changed, 10 insertions(+), 9 deletions(-) diff --git a/qemu-char.c b/qemu-char.c index 9fd94d1..b562bf8 100644 --- a/qemu-char.c +++ b/qemu-char.c @@ -985,7 +985,7 @@ static int qemu_chr_op

[Qemu-devel] [PATCH 3/5] qemu-sockets: Plug fd leak on unix_connect_opts() error path

2011-11-11 Thread Markus Armbruster
Spotted by Coverity. Signed-off-by: Markus Armbruster --- qemu-sockets.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/qemu-sockets.c b/qemu-sockets.c index 183a9cb..61b2247 100644 --- a/qemu-sockets.c +++ b/qemu-sockets.c @@ -572,6 +572,7 @@ int unix_connect_opts(Qemu

[Qemu-devel] [PATCH 4/5] posix-aio-compat: Plug memory leak on paio_init() error path

2011-11-11 Thread Markus Armbruster
Spotted by Coverity. Signed-off-by: Markus Armbruster --- posix-aio-compat.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/posix-aio-compat.c b/posix-aio-compat.c index d3c1174..0c0035c 100644 --- a/posix-aio-compat.c +++ b/posix-aio-compat.c @@ -667,6 +667,7 @@ int pa

[Qemu-devel] [PATCH 5/5] os-posix: Plug fd leak in qemu_create_pidfile()

2011-11-11 Thread Markus Armbruster
Spotted by Coverity. Signed-off-by: Markus Armbruster --- os-posix.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/os-posix.c b/os-posix.c index dbf3b24..dc4a6bb 100644 --- a/os-posix.c +++ b/os-posix.c @@ -372,13 +372,16 @@ int qemu_create_pidfile(const char *filena

Re: [Qemu-devel] Storage requirements for live migration

2011-11-11 Thread Daniel P. Berrange
On Fri, Nov 11, 2011 at 10:38:20AM +0100, Kevin Wolf wrote: > Am 11.11.2011 01:11, schrieb Anthony Liguori: > > I did a brain dump of my understanding of the various storage requirements > > for > > live migration. I think it's accurate but I may have misunderstand some > > details > > so I wo

Re: [Qemu-devel] Storage requirements for live migration

2011-11-11 Thread Kevin Wolf
Am 11.11.2011 10:55, schrieb Daniel P. Berrange: > On Fri, Nov 11, 2011 at 10:38:20AM +0100, Kevin Wolf wrote: >> Am 11.11.2011 01:11, schrieb Anthony Liguori: >>> I did a brain dump of my understanding of the various storage requirements >>> for >>> live migration. I think it's accurate but I m

Re: [Qemu-devel] [v9 Patch 1/6 - updated]Qemu: Enhance "info block" to display host cache setting

2011-11-11 Thread Supriya Kannery
On 11/11/2011 12:17 PM, Supriya Kannery wrote: > Enhance "info block" to display hostcache setting for each > block device. > > > ## > Index: qemu/qapi-types.h > === > --- qemu.orig/qapi-types.h > +++ qemu/qapi-types.h > @@ -383,6 +

Re: [Qemu-devel] qemu and qemu.git -> Migration + disk stress introduces qcow2 corruptions

2011-11-11 Thread Kevin Wolf
Am 10.11.2011 22:30, schrieb Anthony Liguori: > Live migration with qcow2 or any other image format is just not going to work > right now even with proper clustered storage. I think doing a block level > flush > cache interface and letting block devices decide how to do it is the best > approa

Re: [Qemu-devel] Issue Using netdev or vlan with qemu 15.

2011-11-11 Thread Stefan Hajnoczi
On Thu, Nov 10, 2011 at 5:12 PM, srinivasan Sundararajan wrote: > > Trying the following and can’t seem to make it work ?  This is with qemu-kvm > 15 .  What am I doing wrong here ? I get this id1 not found error. > > qemu-system-x86_64 \ > -enable-kvm  -cpu qemu64,+vmx -boot c -m 16000M \ > -driv

Re: [Qemu-devel] [Qemu-ppc] [PATCH] pseries: Correct RAM size check for SLOF

2011-11-11 Thread Alexander Graf
On 11/08/2011 11:12 AM, Thomas Huth wrote: Am Tue, 8 Nov 2011 16:52:16 +1100 schrieb David Gibson: The SLOF firmware used on the pseries machine needs a reasonable amount of (guest) RAM in order to run, so we have a check in the machine init function to check that this is available. However,

Re: [Qemu-devel] [ICON] QEMU Mascot Contest

2011-11-11 Thread Stefan Hajnoczi
On Fri, Nov 11, 2011 at 8:57 AM, 陳韋任 wrote: >  I am not a good painter or an artist, but here you go >  http://people.cs.nctu.edu.tw/~chenwj/slide/QEMU/QEMU_Mascot.png Neat idea to combine the emu with the Q shape. It would be nice to find a high resolution emu, this one is quite pixelated. Ste

Re: [Qemu-devel] [ICON] QEMU Mascot Contest

2011-11-11 Thread Alex Bradbury
On 11 November 2011 11:20, Stefan Hajnoczi wrote: > On Fri, Nov 11, 2011 at 8:57 AM, 陳韋任 wrote: >>  I am not a good painter or an artist, but here you go >>  http://people.cs.nctu.edu.tw/~chenwj/slide/QEMU/QEMU_Mascot.png > > Neat idea to combine the emu with the Q shape.  It would be nice to > f

Re: [Qemu-devel] [ICON] QEMU Mascot Contest

2011-11-11 Thread 陳韋任
> > Neat idea to combine the emu with the Q shape.  It would be nice to > > find a high resolution emu, this one is quite pixelated. > See my link from the other thread: > > http://commons.wikimedia.org/wiki/File:Emu_(PSF).png Actually, I borrow that picture and do some cut/paste. ;-) I am a M$

Re: [Qemu-devel] [ICON] QEMU Mascot Contest

2011-11-11 Thread Stefan Weil
Am 11.11.2011 12:31, schrieb 陳韋任: Neat idea to combine the emu with the Q shape. It would be nice to find a high resolution emu, this one is quite pixelated. See my link from the other thread: http://commons.wikimedia.org/wiki/File:Emu_(PSF).png Actually, I borrow that picture and do some

[Qemu-devel] [Bug 889053] Re: x86: FPU_MAX, FPU_MIN incorrect

2011-11-11 Thread Daniil Troshkov
** Attachment added: "patch" https://bugs.launchpad.net/qemu/+bug/889053/+attachment/2593058/+files/patch -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/889053 Title: x86: FPU_MAX, FPU_MIN incorr

[Qemu-devel] [Bug 889053] Re: x86: FPU_MAX, FPU_MIN incorrect

2011-11-11 Thread Daniil Troshkov
misprint: spec sphinx3 test data -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/889053 Title: x86: FPU_MAX, FPU_MIN incorrect Status in QEMU: New Bug description: Dear All, Bug was found in

[Qemu-devel] [Bug 889053] [NEW] x86: FPU_MAX, FPU_MIN incorrect

2011-11-11 Thread Daniil Troshkov
Public bug reported: Dear All, Bug was found in qemu.git. Now (0.15, 1.0) all fpu is softfpu. See target-i386/ops_sse.h: #define FPU_MIN(size, a, b) (a) < (b) ? (a) : (b) #define FPU_MAX(size, a, b) (a) > (b) ? (a) : (b) It is incorrect now, becouse float64 (or 32) is (typedef) uint64_t (or 32).

[Qemu-devel] [PATCH] make user networking hostfwd work with restrict=y

2011-11-11 Thread Gertjan Halkes
This patch allows the hostfwd option to override the restrict=y setting in the user network stack, as explicitly stated in the documentation on the restrict option: restrict=on|off If this option is enabled, the guest will be isolated, i.e. it will not be able to contact the

Re: [Qemu-devel] [PATCH 0/5] scsi/atapi: MMC fixes

2011-11-11 Thread Paolo Bonzini
On 11/10/2011 07:17 PM, Thomas Schmitt wrote: Hi, I only tested CD-RW DAO burning of an ISO image, plus invoking a bunch of commands from Thomas's logs). The burning succeeded but reading the resulting medium failed consistently at 26 MB. However, the same happened even when doing CD passthro

[Qemu-devel] [Bug 889053] Re: x86: FPU_MAX, FPU_MIN incorrect

2011-11-11 Thread Daniil Troshkov
** Patch added: "test.useprof.exe" https://bugs.launchpad.net/bugs/889053/+attachment/2593057/+files/test.useprof.exe -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/889053 Title: x86: FPU_MAX, F

[Qemu-devel] [Bug 889053] Re: x86: FPU_MAX, FPU_MIN incorrect

2011-11-11 Thread Daniil Troshkov
482.sphinx3: http://www.spec.org/cpu2006/CFP2006/ -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/889053 Title: x86: FPU_MAX, FPU_MIN incorrect Status in QEMU: New Bug description: Dear All,

[Qemu-devel] [Bug 889053] Re: x86: FPU_MAX, FPU_MIN incorrect

2011-11-11 Thread Peter Maydell
The attached patch is incorrect (using the softfloat _min/_max functions will give wrong answers for some special cases). The correct macros are #define FPU_MIN(size, a, b) float ## size ## _lt(a, b, &env->sse_status) ? (a) : (b) #define FPU_MAX(size, a, b) float ## size ## _lt(b, a, &env->sse_st

[Qemu-devel] [PATCH] Avoid SEGV when doing migration to an FD

2011-11-11 Thread Daniel P. Berrange
From: "Daniel P. Berrange" When doing migration to an FD, a NULL 'mon' parameter is being accessed, resulting in SEGV * migration.c: Always set 's->mon' Signed-off-by: Daniel P. Berrange NB, I've no idea if this is the right way to solve this, since I haven't taken the time to identify what c

[Qemu-devel] Long QEMU main loop pauses during migration (to file) under heavy load

2011-11-11 Thread Daniel P. Berrange
Libvirt recently introduced a change to the way it does 'save to file' with QEMU. Historically QEMU has a 32MB/s I/O limit on migration by default. When saving to file, we didn't want any artificial limit, but rather to max out the underlying storage. So when doing save to file, we set a large band

Re: [Qemu-devel] Long QEMU main loop pauses during migration (to file) under heavy load

2011-11-11 Thread Daniel P. Berrange
On Fri, Nov 11, 2011 at 01:03:20PM +, Daniel P. Berrange wrote: > Libvirt recently introduced a change to the way it does 'save to file' > with QEMU. Historically QEMU has a 32MB/s I/O limit on migration by > default. When saving to file, we didn't want any artificial limit, > but rather to max

Re: [Qemu-devel] [ICON] QEMU Mascot Contest v.2

2011-11-11 Thread 陳韋任
> You might use Inkscape for M$ from http://inkscape.org/download/ > and the highest resolution image from Wikipedia > (http://upload.wikimedia.org/wikipedia/commons/7/7d/Emu_%28PSF%29.png). I have tried my best. :-) http://people.cs.nctu.edu.tw/~chenwj/slide/QEMU/QEMU_Mascot.svg Regards, che

[Qemu-devel] [PATCH] hw/arm_timer.c: Fix bounds check for Integrator timer accesses

2011-11-11 Thread Peter Maydell
There are only three counter/timers on the integrator board: correct the bounds check to avoid an array overrun. (Spotted by Coverity, see bug 887883). Signed-off-by: Peter Maydell --- hw/arm_timer.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/hw/arm_timer.c b/hw/

Re: [Qemu-devel] Long QEMU main loop pauses during migration (to file) under heavy load

2011-11-11 Thread Paolo Bonzini
On 11/11/2011 02:03 PM, Daniel P. Berrange wrote: Or perhaps moving migration to a separate thread, out of the mainloop is what we need to strive for ? Yes, this is planned. This way the I/O does not even need to be non-blocking. Patches have been posted (and I have up-to-date versions) tha

Re: [Qemu-devel] [PATCH 1/5] atapi: kill MODE SENSE(6), fix MODE SENSE(10)

2011-11-11 Thread Kevin Wolf
Am 10.11.2011 17:01, schrieb Paolo Bonzini: > Mode page 2A of emulated ATAPI DVD-ROM should have page length 0x14 > like SCSI CD-ROM, rather than 0x12. > > Mode page length is off by 8, as it should contain the length of the > payload after the first two bytes. > > MODE SENSE(6) should be thrown

Re: [Qemu-devel] [PATCH 1/5] atapi: kill MODE SENSE(6), fix MODE SENSE(10)

2011-11-11 Thread Paolo Bonzini
On 11/11/2011 02:36 PM, Kevin Wolf wrote: > @@ -717,11 +712,10 @@ static void cmd_mode_sense(IDEState *s, uint8_t *buf) >buf[12] = 0x00; >buf[13] = 0x00; >buf[14] = 0x00; > -buf[15] = 0x00; Why did you drop this? It still seems to be

Re: [Qemu-devel] [PATCH 0/2] net: socket backend uses uninitialized variables

2011-11-11 Thread Anthony Liguori
On 11/11/2011 03:36 AM, Markus Armbruster wrote: Ping? I assumed "Compile-tested only, buyer beware." was code for, I'll test the patches later and post to the ML ;-) Regards, Anthony Liguori Markus Armbruster writes: Compile-tested only, buyer beware. Markus Armbruster (2): net:

[Qemu-devel] Open Source Virtualization and Cloud at FOSDEM

2011-11-11 Thread Mark McLoughlin
Hey, Here's one for the calendar - a 2 day track at FOSDEM devoted to Open Source Virtualization and Cloud. Hope to see you there! :) Cheers, Mark. Forwarded Message > From: Thierry Carrez > To: openst...@lists.launchpad.net > Subject: [Openstack] OpenStack at FOSDEM > Date:

[Qemu-devel] Patches for SMSC LAN911X driver

2011-11-11 Thread Cachet Bertrand
Hello, In the driver for the SMSC LAN9118 device (hw/lan9118.c), I modify the code to update the PM_CTRL register (switch PM_MODE bits to D0 and set (1b) READY bit ) when writing to the BYTE_TEST register. Writing to PM_CTRL was not permitted before this modification => raise an harware error

Re: [Qemu-devel] qemu and qemu.git -> Migration + disk stress introduces qcow2 corruptions

2011-11-11 Thread Anthony Liguori
On 11/11/2011 04:15 AM, Kevin Wolf wrote: Am 10.11.2011 22:30, schrieb Anthony Liguori: Live migration with qcow2 or any other image format is just not going to work right now even with proper clustered storage. I think doing a block level flush cache interface and letting block devices decide

Re: [Qemu-devel] [PATCH 2/5] scsi: update list of commands

2011-11-11 Thread Kevin Wolf
Am 10.11.2011 17:01, schrieb Paolo Bonzini: > Add more commands and their names, and remove SEEK(6) which is obsolete. > Instead, use SET_POSITION which is still in SSC. SET CAPACITY you mean? > > Signed-off-by: Paolo Bonzini > --- > hw/scsi-bus.c | 25 +++-- > hw/scsi-d

Re: [Qemu-devel] Storage requirements for live migration

2011-11-11 Thread Anthony Liguori
On 11/11/2011 03:38 AM, Kevin Wolf wrote: Am 11.11.2011 01:11, schrieb Anthony Liguori: I did a brain dump of my understanding of the various storage requirements for live migration. I think it's accurate but I may have misunderstand some details so I would appreciate review. I think given sec

Re: [Qemu-devel] Storage requirements for live migration

2011-11-11 Thread Anthony Liguori
On 11/11/2011 03:55 AM, Daniel P. Berrange wrote: On Fri, Nov 11, 2011 at 10:38:20AM +0100, Kevin Wolf wrote: Am 11.11.2011 01:11, schrieb Anthony Liguori: I did a brain dump of my understanding of the various storage requirements for live migration. I think it's accurate but I may have misund

Re: [Qemu-devel] Patches for SMSC LAN911X driver

2011-11-11 Thread Andreas Färber
Hi, Am 11.11.2011 14:44, schrieb Cachet Bertrand: > Patch is contained in the following commit : > https://bitbucket.org/bca/qemu-linaro/changeset/0aa1f76e5141 > > I have sent this patch to qemu-linaro but Peter Maydell (pm215 on #qemu > IRC channel) told me to send it here because it is against

Re: [Qemu-devel] [PATCH] make user networking hostfwd work with restrict=y

2011-11-11 Thread Anthony Liguori
On 11/11/2011 05:58 AM, Gertjan Halkes wrote: This patch allows the hostfwd option to override the restrict=y setting in the user network stack, as explicitly stated in the documentation on the restrict option: restrict=on|off If this option is enabled, the guest will be isolated

Re: [Qemu-devel] qemu and qemu.git -> Migration + disk stress introduces qcow2 corruptions

2011-11-11 Thread Kevin Wolf
Am 11.11.2011 15:03, schrieb Anthony Liguori: > On 11/11/2011 04:15 AM, Kevin Wolf wrote: >> Am 10.11.2011 22:30, schrieb Anthony Liguori: >>> Live migration with qcow2 or any other image format is just not going to >>> work >>> right now even with proper clustered storage. I think doing a block

Re: [Qemu-devel] [PATCH] Avoid SEGV when doing migration to an FD

2011-11-11 Thread Anthony Liguori
On 11/11/2011 06:44 AM, Daniel P. Berrange wrote: From: "Daniel P. Berrange" When doing migration to an FD, a NULL 'mon' parameter is being accessed, resulting in SEGV * migration.c: Always set 's->mon' Signed-off-by: Daniel P. Berrange NB, I've no idea if this is the right way to solve this,

Re: [Qemu-devel] qemu and qemu.git -> Migration + disk stress introduces qcow2 corruptions

2011-11-11 Thread Anthony Liguori
On 11/11/2011 08:29 AM, Kevin Wolf wrote: Am 11.11.2011 15:03, schrieb Anthony Liguori: On 11/11/2011 04:15 AM, Kevin Wolf wrote: Am 10.11.2011 22:30, schrieb Anthony Liguori: Live migration with qcow2 or any other image format is just not going to work right now even with proper clustered sto

Re: [Qemu-devel] qemu and qemu.git -> Migration + disk stress introduces qcow2 corruptions

2011-11-11 Thread Kevin Wolf
Am 11.11.2011 15:35, schrieb Anthony Liguori: > On 11/11/2011 08:29 AM, Kevin Wolf wrote: >> Am 11.11.2011 15:03, schrieb Anthony Liguori: >>> On 11/11/2011 04:15 AM, Kevin Wolf wrote: Am 10.11.2011 22:30, schrieb Anthony Liguori: > Live migration with qcow2 or any other image format is ju

Re: [Qemu-devel] QEMU Mascot Contest

2011-11-11 Thread Andreas Färber
Hi, Am 10.11.2011 23:10, schrieb Anthony Liguori: > [...] I thought it would be a fun part of the 1.0 release to have a > logo competition to create an official image of Q. > > So break out Gimp, Inkscape, or your favorite image editor and submit an > icon for consideration! If we get enough pr

[Qemu-devel] [PATCH 0/5] scsi/atapi: MMC fixes

2011-11-11 Thread Thomas Schmitt
Hi, > The READ TOC/PMA/ATIP at the end of burning gives (reformatted): > 43 02 02 00 00 00 00 00 51 00 > From drive: 81b > 00 4f 01 01 > 01 14 00 a0 00 00 00 00 01 00 00 > 01 14 00 a1 00 00 00 00 01 00 00 > 01 14 00 a2 00 00 00 00 18 39 32 <<< > 01 14 00 01 00 00 00 00 00 02 00 I was not aware yo

[Qemu-devel] [PATCH 4/6 v2] Makefile.target: Remove libqemu target

2011-11-11 Thread 陳韋任
From: Chen Wei-Ren Remove libqemu target from Makefile.target. I have ran the following test: - process mode: linux-user-test-0.3 - system mode: - arm-test-0.2.tar.gz - minix204.tar.bz2 - linux-0.2.img.bz2 Signed-off-by: Chen Wei-Ren --- Makefile.target |8 ---

[Qemu-devel] [PATCH 6/6 v2] LICENSE: There is no libqemu.a anymore

2011-11-11 Thread 陳韋任
From: Chen Wei-Ren Remove statement about libqemu.a from LICENSE. Signed-off-by: Chen Wei-Ren --- LICENSE |4 +--- 1 files changed, 1 insertions(+), 3 deletions(-) diff --git a/LICENSE b/LICENSE index cbd92c0..acae9a3 100644 --- a/LICENSE +++ b/LICENSE @@ -6,9 +6,7 @@ The following poin

[Qemu-devel] [PATCH 1/6 v2] tests/qruncom.c: Remove libqemu.a example

2011-11-11 Thread 陳韋任
From: Chen Wei-Ren Remove libqemu example since libqemu.a is not available anymore. Signed-off-by: Chen Wei-Ren --- tests/qruncom.c | 284 --- 1 files changed, 0 insertions(+), 284 deletions(-) delete mode 100644 tests/qruncom.c diff --g

[Qemu-devel] [PATCH 5/6 v2] Makefile.objs: Remove libqemu_common.a from the comment

2011-11-11 Thread 陳韋任
From: Chen Wei-Ren Remove libqemu_common.a from the comment. v1 -> v2: Reverse the deletion of common-obj-y. Only remove libqemu_common.a from the comment. Signed-off-by: Chen Wei-Ren --- Makefile.objs |7 +++ 1 files changed, 3 insertions(+), 4 deletions(-) diff --gi

[Qemu-devel] [PATCH 0/6 v2] Remove libqemu related stuff from QEMU source tree

2011-11-11 Thread 陳韋任
From: Chen Wei-Ren According to [1], libqemu is not available anymore. Remove libqemu related stuff from QEMU source tree. [1] http://www.mail-archive.com/address@hidden/msg49809.html Chen Wei-Ren (6): tests/qruncom.c: Remove libqemu.a example tests/Makefile: Remove qruncom target qemu-

[Qemu-devel] [PATCH 2/6 v2] tests/Makefile: Remove qruncom target

2011-11-11 Thread 陳韋任
From: Chen Wei-Ren Remove qruncom target from the Makefile file since we have removed libqemu example (qruncom.c). Signed-off-by: Chen Wei-Ren --- tests/Makefile |6 -- 1 files changed, 0 insertions(+), 6 deletions(-) diff --git a/tests/Makefile b/tests/Makefile index 430e0c1..15e36

[Qemu-devel] [PATCH 3/6 v2] qemu-tech.texi: Remove libqemu related stuff from the document

2011-11-11 Thread 陳韋任
From: Chen Wei-Ren Remove libqemu related stuff from the document since libqemu.a is not supported anymore. v1 -> v2: Remove entry "qruncom.c" from "3 Regression Tests". Signed-off-by: Chen Wei-Ren --- qemu-tech.texi | 10 -- 1 files changed, 0 insertions(+), 10 deletions(-)

Re: [Qemu-devel] A document about Windows Guest Debugging has been post on linux-kvm wiki

2011-11-11 Thread Mulyadi Santosa
On Fri, Nov 11, 2011 at 14:27, Cao,Bing Bu wrote: > Hi, > >    I have post a HOWTO document about windows guest debugging on: >    http://www.linux-kvm.org/page/WindowsGuestDrivers/UpdatedGuestDebugging > >    It include more details as a complement of > http://www.linux-kvm.org/page/WindowsGuestD

Re: [Qemu-devel] [PATCH 4/6 v2] Makefile.target: Remove libqemu target

2011-11-11 Thread Peter Maydell
On 11 November 2011 14:53, Chen Wei-Ren (陳韋任) wrote: >  Remove libqemu target from Makefile.target. > diff --git a/Makefile.target b/Makefile.target > index a111521..d1f7993 100644 > --- a/Makefile.target > +++ b/Makefile.target > @@ -92,14 +92,6 @@ tci-dis.o: QEMU_CFLAGS += -I$(SRC_PATH)/tcg >

Re: [Qemu-devel] [PATCH 2/6 v2] tests/Makefile: Remove qruncom target

2011-11-11 Thread Peter Maydell
On 11 November 2011 14:53, Chen Wei-Ren (陳韋任) wrote: > From: Chen Wei-Ren > >  Remove qruncom target from the Makefile file since we have > removed libqemu example (qruncom.c). This should be done before the patch removing qruncom.c, not after. Or do both in the same patch. -- PMM

Re: [Qemu-devel] [ICON] QEMU Mascot Contest v.2

2011-11-11 Thread Michael Roth
On 11/11/2011 07:24 AM, 陳韋任 wrote: You might use Inkscape for M$ from http://inkscape.org/download/ and the highest resolution image from Wikipedia (http://upload.wikimedia.org/wikipedia/commons/7/7d/Emu_%28PSF%29.png). I have tried my best. :-) http://people.cs.nctu.edu.tw/~chenwj/slide/

Re: [Qemu-devel] [PATCH] make user networking hostfwd work with restrict=y

2011-11-11 Thread Gertjan Halkes
On Fri, 11 Nov 2011 08:24:04 -0600, Anthony Liguori wrote: >Please submit against qemu.git master with a Signed-off-by. Ok, here goes: This patch allows the hostfwd option to override the restrict=y setting in the user network stack, as explicitly stated in the documentation on the restrict opt

Re: [Qemu-devel] [PATCH 2/6 v2] tests/Makefile: Remove qruncom target

2011-11-11 Thread 陳韋任
On Fri, Nov 11, 2011 at 03:02:16PM +, Peter Maydell wrote: > On 11 November 2011 14:53, Chen Wei-Ren (陳韋任) > wrote: > > From: Chen Wei-Ren > > > >  Remove qruncom target from the Makefile file since we have > > removed libqemu example (qruncom.c). > > This should be done before the patch re

[Qemu-devel] Re: [PATCH 1/5] atapi: kill MODE SENSE(6), fix MODE SENSE(10)

2011-11-11 Thread Thomas Schmitt
Hi, Paolo Bonzini wrote: > > > case MODE_PAGE_R_W_ERROR: /* error recovery */ > > > [...] > > > -buf[15] = 0x00; Kevin Wolf wrote: > > Why did you drop this? It still seems to be part of the buffer. Paolo Bonzini wrote: > Actually, I think it's best if these patches wait unt

Re: [Qemu-devel] [PATCH 4/6 v2] Makefile.target: Remove libqemu target

2011-11-11 Thread 陳韋任
> This commit isn't doing what the commit message claims it does. > > What you're actually removing is a set of manually inserted > dependencies of some object files on their source files and > on cpu.h. There is already no libqemu target in this makefile. > What you maybe wanted to remove is just

Re: [Qemu-devel] [PATCH 2/6 v2] tests/Makefile: Remove qruncom target

2011-11-11 Thread Peter Maydell
On 11 November 2011 15:07, 陳韋任 wrote: > On Fri, Nov 11, 2011 at 03:02:16PM +, Peter Maydell wrote: >> On 11 November 2011 14:53, Chen Wei-Ren (陳韋任) >> wrote: >> > From: Chen Wei-Ren >> > >> >  Remove qruncom target from the Makefile file since we have >> > removed libqemu example (qruncom.c

Re: [Qemu-devel] [PATCH 2/6 v2] tests/Makefile: Remove qruncom target

2011-11-11 Thread Andreas Färber
Am 11.11.2011 16:07, schrieb 陳韋任: > On Fri, Nov 11, 2011 at 03:02:16PM +, Peter Maydell wrote: >> On 11 November 2011 14:53, Chen Wei-Ren (陳韋任) >> wrote: >>> From: Chen Wei-Ren >>> >>> Remove qruncom target from the Makefile file since we have >>> removed libqemu example (qruncom.c). >> >>

[Qemu-devel] [Bug 888016] Re: RHEL 6.1 guest fails to boot with vhost

2011-11-11 Thread pradeep
** Changed in: qemu Assignee: (unassigned) => pradeep (psuriset) -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/888016 Title: RHEL 6.1 guest fails to boot with vhost Status in QEMU: New Bug

Re: [Qemu-devel] [PATCH] s390x: initialize virtio dev region

2011-11-11 Thread Alexander Graf
On 11/10/2011 03:19 AM, Peter Maydell wrote: On 10 November 2011 01:45, Alexander Graf wrote: On 10.11.2011, at 02:36, Peter Maydell wrote: This looks a bit fishy -- cpu_physical_memory_map() takes a target_phys_addr_t but you're passing it a ram_addr_t. Meh. Always those types ... :) In the

Re: [Qemu-devel] [PATCH] s390x: initialize virtio dev region

2011-11-11 Thread Peter Maydell
On 11 November 2011 15:59, Alexander Graf wrote: > This is the machine init function. The s390 virtio machine's ram layout is > defined to be exactly as I posted in the previous post. So there won't be > any ram starts at != 0 or multiple mappings :). That's the layout of the RAM in target_phys_a

Re: [Qemu-devel] [ICON] QEMU Mascot Contest v.2

2011-11-11 Thread Kevin Wolf
Am 11.11.2011 16:02, schrieb Michael Roth: > On 11/11/2011 07:24 AM, 陳韋任 wrote: >>> You might use Inkscape for M$ from http://inkscape.org/download/ >>> and the highest resolution image from Wikipedia >>> (http://upload.wikimedia.org/wikipedia/commons/7/7d/Emu_%28PSF%29.png). >> >>I have tried

Re: [Qemu-devel] [ICON] QEMU Mascot Contest v.2

2011-11-11 Thread 陳韋任
> This is awesome! I'm worried how well it will scale down though. If you > do the "emu" part on the right side, vertically, between the bottom of > its head and top of its foot, you can shrink the Q part a bit to make > the emu's body more proportional and leave more space for the "emu" text >

Re: [Qemu-devel] [PATCH] s390x: initialize virtio dev region

2011-11-11 Thread Alexander Graf
On 11/11/2011 05:11 PM, Peter Maydell wrote: On 11 November 2011 15:59, Alexander Graf wrote: This is the machine init function. The s390 virtio machine's ram layout is defined to be exactly as I posted in the previous post. So there won't be any ram starts at != 0 or multiple mappings :). Tha

Re: [Qemu-devel] [Xen-devel] [PATCH V3 07/10] Introduce Xen PCI Passthrough, qdevice (1/3)

2011-11-11 Thread Anthony PERARD
On Thu, 10 Nov 2011, Konrad Rzeszutek Wilk wrote: > On Fri, Oct 28, 2011 at 04:07:33PM +0100, Anthony PERARD wrote: > > From: Allen Kay > > > > This is going to be a bit lame review.. > [...] > > +return; > > +} > > + > > +/* find register group entry */ > > +reg_grp_entry =

[Qemu-devel] [PULL 0/3] ppc patch queue 2011-11-11 for 1.0

2011-11-11 Thread Alexander Graf
Hi Blue, This is my current patch queue for ppc on QEMU 1.0. Please pull. Alex The following changes since commit 0e3b800e71cb7759d099eabbd8ad4c4fe848e381: Peter Maydell (1): hw/lan9118.c: Add missing 'break' to fix buffer overrun are available in the git repository at: git://repo

[Qemu-devel] [PATCH 1/3] pseries: Fix initialization of sPAPREnvironment structure

2011-11-11 Thread Alexander Graf
From: David Gibson Since we added PCI support to the pseries machine, we include a qlist of PCI host bridges in the sPAPREnvironment structure. However this list was never properly initialized it. Somehow we got away with this until some other recent change broke it, and we now segfault immedia

[Qemu-devel] [PATCH 2/3] tcg-ppc64: Fix compile errors for userspace only builds with gcc 4.6

2011-11-11 Thread Alexander Graf
From: David Gibson tcg/ppc64/tcg-target.c has a couple of places where variables are set unconditionally, but otherwise used only for softmmu builds, not userspace only builds. This causes compiler warnings (which are fatal by default) when compiling for a ppc64 host with gcc 4.6. This patch fi

[Qemu-devel] [PATCH 3/3] PPC: Fix for the gdb single step problem on an rfi instruction

2011-11-11 Thread Alexander Graf
From: Sebastian Bauer When using gdb to single step a ppc interrupt routine, the execution flow passes the rfi instruction without actually returning from the interrupt. The patch fixes this by avoiding to update the nip when the debug exception is raised and a previous POWERPC_EXCP_SYNC was set

Re: [Qemu-devel] [PATCH] s390x: initialize virtio dev region

2011-11-11 Thread Peter Maydell
On 11 November 2011 16:24, Alexander Graf wrote: > On 11/11/2011 05:11 PM, Peter Maydell wrote: >> Yes, so that's option (2) and you need to be using a target_phys_addr_t. > > But ram_size is ram_addr_t and is the ram size that I have available to use, > so it's exactly the address that I want. I

Re: [Qemu-devel] [PATCH] s390x: initialize virtio dev region

2011-11-11 Thread Alexander Graf
On 11/11/2011 05:44 PM, Peter Maydell wrote: On 11 November 2011 16:24, Alexander Graf wrote: On 11/11/2011 05:11 PM, Peter Maydell wrote: Yes, so that's option (2) and you need to be using a target_phys_addr_t. But ram_size is ram_addr_t and is the ram size that I have available to use, so i

[Qemu-devel] [PATCH 03/10] qed: convert to .bdrv_co_is_allocated()

2011-11-11 Thread Stefan Hajnoczi
The bdrv_qed_is_allocated() function is a synchronous wrapper around qed_find_cluster(), which performs the cluster lookup. In order to convert the synchronous function to a coroutine function we yield instead of using qemu_aio_wait(). Note that QED's cache is already safe for parallel requests s

[Qemu-devel] [PATCH 00/10] block: replace .bdrv_is_allocated() with .bdrv_co_is_allocated()

2011-11-11 Thread Stefan Hajnoczi
The bdrv_is_allocated() interface is not suitable for use while the VM is running. It is a synchronous interface so it may block the running VM for arbitrary amounts of time. It also assumes it is the only block driver operation and there is a risk that internal state could be corrupted if asynch

[Qemu-devel] [PATCH 05/10] vvfat: use public block layer interface

2011-11-11 Thread Stefan Hajnoczi
For some reason vvfat.c invokes BlockDriver functions directly and does not go through block.h functions. Change all direct calls except for the .bdrv_make_empty() call for which there is no public interface. This change makes the conversion to .bdrv_co_is_allocated() possible. Signed-off-by: St

[Qemu-devel] [PATCH 04/10] block: convert qcow2, qcow2, and vmdk to .bdrv_co_is_allocated()

2011-11-11 Thread Stefan Hajnoczi
The qcow2, qcow, and vmdk block drivers are based on coroutines. They have a coroutine mutex which protects internal state. We can convert the .bdrv_is_allocated() function to .bdrv_co_is_allocated() by holding the mutex around the cluster lookup operation. Signed-off-by: Stefan Hajnoczi --- b

[Qemu-devel] [PATCH 01/10] block: use public bdrv_is_allocated() interface

2011-11-11 Thread Stefan Hajnoczi
There is no need for bdrv_commit() to use the BlockDriver .bdrv_is_allocated() interface directly. Converting to the public interface gives us the freedom to drop .bdrv_is_allocated() entirely in favor of a new .bdrv_co_is_allocated() in the future. Signed-off-by: Stefan Hajnoczi --- block.c |

Re: [Qemu-devel] [PATCH 05/10] vvfat: use public block layer interface

2011-11-11 Thread Kevin Wolf
Am 11.11.2011 17:47, schrieb Stefan Hajnoczi: > For some reason vvfat.c invokes BlockDriver functions directly and does > not go through block.h functions. Change all direct calls except for > the .bdrv_make_empty() call for which there is no public interface. > > This change makes the conversion

[Qemu-devel] [PATCH 10/10] block: add bdrv_co_is_allocated() interface

2011-11-11 Thread Stefan Hajnoczi
This patch introduce the public bdrv_co_is_allocated() interface which can be used to query image allocation status while the VM is running. Signed-off-by: Stefan Hajnoczi --- block.c | 37 - block.h |2 ++ 2 files changed, 26 insertions(+), 13 deletions

[Qemu-devel] [PATCH 08/10] cow: convert to .bdrv_co_is_allocated()

2011-11-11 Thread Stefan Hajnoczi
The cow block driver does not keep internal state for cluster lookups. This means it is safe to perform cluster lookups in coroutine context without risk of race conditions that corrupt internal state. Signed-off-by: Stefan Hajnoczi --- block/cow.c |8 1 files changed, 4 insertions(

Re: [Qemu-devel] [ICON] QEMU Mascot Contest v.2

2011-11-11 Thread Michael Roth
On 11/11/2011 10:20 AM, Kevin Wolf wrote: Am 11.11.2011 16:02, schrieb Michael Roth: On 11/11/2011 07:24 AM, 陳韋任 wrote: You might use Inkscape for M$ from http://inkscape.org/download/ and the highest resolution image from Wikipedia (http://upload.wikimedia.org/wikipedia/commons/7/7d/Emu_%28PSF

[Qemu-devel] [PATCH 07/10] vdi: convert to .bdrv_co_is_allocated()

2011-11-11 Thread Stefan Hajnoczi
It is trivial to switch from the synchronous .bdrv_is_allocated() interface to .bdrv_co_is_allocated() since vdi_is_allocated() does not block. Signed-off-by: Stefan Hajnoczi --- block/vdi.c |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/block/vdi.c b/block/vdi.c i

[Qemu-devel] [PATCH 09/10] block: drop .bdrv_is_allocated() interface

2011-11-11 Thread Stefan Hajnoczi
Now that all block drivers have been converted to .bdrv_co_is_allocated() we can drop .bdrv_is_allocated(). Note that the public bdrv_is_allocated() interface is still available but is in fact a synchronous wrapper around .bdrv_co_is_allocated(). Signed-off-by: Stefan Hajnoczi --- block.c |

[Qemu-devel] [PATCH 06/10] vvfat: convert to .bdrv_co_is_allocated()

2011-11-11 Thread Stefan Hajnoczi
It is trivial to switch from the synchronous .bdrv_is_allocated() interface to .bdrv_co_is_allocated() since vvfat_is_allocated() does not block. Signed-off-by: Stefan Hajnoczi --- block/vvfat.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/block/vvfat.c b/block/vvf

Re: [Qemu-devel] [ICON] QEMU Mascot Contest v.2

2011-11-11 Thread Michael Roth
On 11/11/2011 10:18 AM, 陳韋任 wrote: This is awesome! I'm worried how well it will scale down though. If you do the "emu" part on the right side, vertically, between the bottom of its head and top of its foot, you can shrink the Q part a bit to make the emu's body more proportional and leave more s

[Qemu-devel] [PATCH 02/10] block: add .bdrv_co_is_allocated()

2011-11-11 Thread Stefan Hajnoczi
This patch adds the .bdrv_co_is_allocated() interface which is identical to .bdrv_is_allocated() but runs in coroutine context. Running in coroutine context implies that other coroutines might be performing I/O at the same time. Therefore it must be safe to run while the following BlockDriver fu

[Qemu-devel] [PULL 00/16] Block patches for 1.0

2011-11-11 Thread Kevin Wolf
The following changes since commit 0e3b800e71cb7759d099eabbd8ad4c4fe848e381: hw/lan9118.c: Add missing 'break' to fix buffer overrun (2011-11-10 12:29:50 +) are available in the git repository at: git://repo.or.cz/qemu/kevin.git block-stable Charles Arnold (1): block: Fix vpc init

[Qemu-devel] [PATCH 01/16] vvfat: Fix read-write mode

2011-11-11 Thread Kevin Wolf
vvfat used to directly call into the qcow2 block driver instead of using the block.c wrappers. With the coroutine conversion, this stopped working. Signed-off-by: Kevin Wolf Reviewed-by: Paolo Bonzini --- block/vvfat.c | 44 +++- 1 files changed, 23 ins

[Qemu-devel] [PATCH 03/16] atapi: implement eject requests

2011-11-11 Thread Kevin Wolf
From: Paolo Bonzini Signed-off-by: Paolo Bonzini Signed-off-by: Kevin Wolf --- hw/ide/atapi.c | 11 --- hw/ide/core.c | 13 + 2 files changed, 21 insertions(+), 3 deletions(-) diff --git a/hw/ide/atapi.c b/hw/ide/atapi.c index 90b6729..1fed359 100644 --- a/hw/ide/atap

[Qemu-devel] [PATCH 04/16] scsi-disk: implement eject requests

2011-11-11 Thread Kevin Wolf
From: Paolo Bonzini Signed-off-by: Paolo Bonzini Signed-off-by: Kevin Wolf --- hw/scsi-disk.c | 24 +--- 1 files changed, 21 insertions(+), 3 deletions(-) diff --git a/hw/scsi-disk.c b/hw/scsi-disk.c index 1c04872..62f538f 100644 --- a/hw/scsi-disk.c +++ b/hw/scsi-disk.c

  1   2   >