[Qemu-devel] [Bug 584083] [NEW] vvfat out of sync with host

2010-05-21 Thread David Campbell
Public bug reported: The vvfat directory in the guest os only synces up with the host when qemu is shutdown and started again. Running sync on the host does not help. Also, I can't cd into the vvfat directory; although, I can read from it. I start qemu using the following line in a bash script:

Re: [Qemu-devel] Problems changing dvdrom iso during execution

2010-05-21 Thread Natalia Portillo
Would you please test on SLES11 with another kernel revision? Possible, with various one, both lower and upper? El 22/05/2010, a las 00:45, Adnan Khaleel escribió: > It works in ubuntu 9.10. When I mount the CDROM the first time it mounts > fine. After I change the iso file and mount, it spits

Re: [Qemu-devel] Problems changing dvdrom iso during execution

2010-05-21 Thread Adnan Khaleel
It works in ubuntu 9.10. When I mount the CDROM the first time it mounts fine. After I change the iso file and mount, it spits out a bunch of messages but it does mount the drive. I think this problem might be specific to sles11. On May 21, 2010, at 5:37 PM, Natalia Portillo wrote:

Re: [Qemu-devel] Problems changing dvdrom iso during execution

2010-05-21 Thread Natalia Portillo
Have you tried any other operating system or kernel revision? I have just changed the iso with change ide1-cd0 command in Windows XP Upgrade (it asks to insert a previous Windows CD and then reinsert the XP one) without any kind of problem, in QEMU 0.12.4. El 21/05/2010, a las 20:42, Adnan Khal

Re: [Qemu-devel] [PATCH 1/2] trace: Add simple tracing support

2010-05-21 Thread Anthony Liguori
On 05/21/2010 04:41 PM, Jan Kiszka wrote: Anthony Liguori wrote: I'm not opposed to using a framework, but I'd rather have an equivalent to kvm_stat tomorrow than wait 3 years for LTTng to not get merged. So let's have a dirt-simple tracing mechanism and focus on adding useful trace points.

[Qemu-devel] [PATCH] sparc32 protect read-only bits in DMA CSR registers

2010-05-21 Thread Artyom Tarasenko
On a real hardware changing read-only bits has no effect Use a mask common for SCSI and Ethernet registers. The crucial bit is DMA_INTR, because setting or clearing it may produce spurious interrupts. This patch allows booting Solaris 2.3 Signed-off-by: Artyom Tarasenko --- hw/sparc32_dma.c |

Re: [Qemu-devel] [PATCH 1/2] trace: Add simple tracing support

2010-05-21 Thread Jan Kiszka
Anthony Liguori wrote: > I'm not opposed to using a framework, but I'd rather have an equivalent > to kvm_stat tomorrow than wait 3 years for LTTng to not get merged. > > So let's have a dirt-simple tracing mechanism and focus on adding useful > trace points. Then when we have a framework we can

Re: [Qemu-devel] [PATCH 1/2] trace: Add simple tracing support

2010-05-21 Thread Jan Kiszka
Stefan Hajnoczi wrote: > On Fri, May 21, 2010 at 5:52 PM, Jan Kiszka wrote: >> I would just like to avoid that too much efforts are spent on >> re-inventing smart trace buffers, trace daemons, or trace visualization >> tools. Then better pick up some semi-perfect approach (e.g. [1], it >> unfortun

Re: [Qemu-devel] [PATCH 1/2] trace: Add simple tracing support

2010-05-21 Thread Christoph Hellwig
On Fri, May 21, 2010 at 09:49:56PM +0100, Stefan Hajnoczi wrote: > http://sourceware.org/systemtap/wiki/AddingUserSpaceProbingToApps > > Requires kernel support - not sure if enough of utrace is in mainline > for this to work out-of-the-box across distros. Nothing of utrace is in mainline, neverm

[Qemu-devel] [PATCH -V3 3/7] virtio-9p: modify create/open2 and mkdir for new security model.

2010-05-21 Thread Venkateswararao Jujjuri (JV)
Add required infrastructure and modify create/open2 and mkdir per the new security model. Signed-off-by: Venkateswararao Jujjuri --- hw/file-op-9p.h | 23 - hw/virtio-9p-local.c | 139 +++--- hw/virtio-9p.c | 42 ++-

[Qemu-devel] [PATCH -V3 5/7] virtio-9p: Implemented security model for symlink and link.

2010-05-21 Thread Venkateswararao Jujjuri (JV)
Signed-off-by: Venkateswararao Jujjuri --- hw/file-op-9p.h |4 +- hw/virtio-9p-local.c | 87 +++-- hw/virtio-9p.c | 24 ++ 3 files changed, 88 insertions(+), 27 deletions(-) diff --git a/hw/file-op-9p.h b/hw/file-op-9p.h

[Qemu-devel] [PATCH -V3 1/7] virtio-9p: Introduces an option to specify the security model.

2010-05-21 Thread Venkateswararao Jujjuri (JV)
The new option is: -fsdev fstype,id=myid,path=/share_path/,security_model=[mapped|passthrough] -virtfs fstype,path=/share_path/,security_model=[mapped|passthrough],mnt_tag=tag In the case of mapped security model, files are created with QEMU user credentials and the client-user's credentials are

[Qemu-devel] [PATCH -V3 2/7] virtio-9p: Rearrange fileop structures

2010-05-21 Thread Venkateswararao Jujjuri (JV)
Signed-off-by: Venkateswararao Jujjuri --- hw/virtio-9p.c | 185 ++-- hw/virtio-9p.h | 92 2 files changed, 138 insertions(+), 139 deletions(-) diff --git a/hw/virtio-9p.c b/hw/virtio-9p.c index 8ecd39c..fda3c4a

[Qemu-devel] [PATCH -V3 6/7] virtio-9p: Implemented Security model for lstat and fstat

2010-05-21 Thread Venkateswararao Jujjuri (JV)
Signed-off-by: Venkateswararao Jujjuri --- hw/virtio-9p-local.c | 67 +++--- 1 files changed, 63 insertions(+), 4 deletions(-) diff --git a/hw/virtio-9p-local.c b/hw/virtio-9p-local.c index ca0f065..283ed5d 100644 --- a/hw/virtio-9p-local.c +++ b/hw/

[Qemu-devel] [PATCH -V3 4/7] virtio-9p: Implement Security model for mknod related files

2010-05-21 Thread Venkateswararao Jujjuri (JV)
In the mapped security model all the special files are created as regular files on the fileserver and appropriate mode bits are added to the extended attributes. These extended attributes are used to present this file as special file to the client. Signed-off-by: Venkateswararao Jujjuri --- hw/f

[Qemu-devel] [PATCH-V2 0/7] virtio-9p:Introducing security model for VirtFS

2010-05-21 Thread Venkateswararao Jujjuri (JV)
This patch series introduces the security model for VirtFS. Brief description of this patch series: It introduces two type of security models for VirtFS. They are: mapped and passthrough. The following is common to both security models. * Client's VFS determines/enforces the access control. L

[Qemu-devel] [PATCH -V3 7/7] virtio-9p: Implemented security model for chown and chgrp.

2010-05-21 Thread Venkateswararao Jujjuri (JV)
Signed-off-by: Venkateswararao Jujjuri --- hw/file-op-9p.h |4 ++-- hw/virtio-9p-local.c | 18 ++ hw/virtio-9p.c | 15 --- 3 files changed, 28 insertions(+), 9 deletions(-) diff --git a/hw/file-op-9p.h b/hw/file-op-9p.h index 0a9c10a..b3a320c 100644

[Qemu-devel] Re: [PATCH 1/2] Pad iommu with an empty slot (necessary for SunOS 4.1.4)

2010-05-21 Thread Blue Swirl
On Fri, May 21, 2010 at 5:23 PM, Artyom Tarasenko wrote: > 2010/5/10 Blue Swirl : >> On 5/10/10, Artyom Tarasenko wrote: >>> 2010/5/10 Blue Swirl : >>> >>> > On 5/10/10, Artyom Tarasenko wrote: >>>  >> 2010/5/9 Blue Swirl : >>>  >>  > On 5/9/10, Artyom Tarasenko wrote: >>>  >>  >> 2010/5/9 Blue

Re: [Qemu-devel] [PATCH 1/2] trace: Add simple tracing support

2010-05-21 Thread Anthony Liguori
On 05/21/2010 11:52 AM, Jan Kiszka wrote: Anthony Liguori wrote: On 05/21/2010 08:46 AM, Jan Kiszka wrote: Anthony Liguori wrote: On 05/21/2010 04:42 AM, Stefan Hajnoczi wrote: Trace events should be defined in trace.h. Events are written to /tmp/trace.log and c

Re: [Qemu-devel] [PATCH 1/2] trace: Add simple tracing support

2010-05-21 Thread Stefan Hajnoczi
On Fri, May 21, 2010 at 5:52 PM, Jan Kiszka wrote: > I would just like to avoid that too much efforts are spent on > re-inventing smart trace buffers, trace daemons, or trace visualization > tools. Then better pick up some semi-perfect approach (e.g. [1], it > unfortunately still seems to lack ker

Re: [Qemu-devel] [Bug 453617] Re: kvm hangs at 100% cpu when connecting to forwarded ports (when listed incorrectly on the command line)

2010-05-21 Thread Gleb Natapov
On Fri, May 21, 2010 at 02:44:46PM -0500, Anthony Liguori wrote: > On 05/21/2010 02:37 PM, Gleb Natapov wrote: > >Another thing. If my memory serves me right it was promised that message > >will contain enough info to contact bug reporter without going through > >launchpad web interface. In other w

Re: [Qemu-devel] [Bug 453617] Re: kvm hangs at 100% cpu when connecting to forwarded ports (when listed incorrectly on the command line)

2010-05-21 Thread Anthony Liguori
On 05/21/2010 02:37 PM, Gleb Natapov wrote: Another thing. If my memory serves me right it was promised that message will contain enough info to contact bug reporter without going through launchpad web interface. In other words email of bug reporter will be specified. I can't find it in the messa

Re: [Qemu-devel] Problems changing dvdrom iso during execution

2010-05-21 Thread Adnan Khaleel
Tried that, still the same. My current workaround is to mount all the DVD iso files as separate hard disks and mount those. This worked but its a workaround at best. Not sure what I'd do if ever had to access more then 3 dvd's at a time - which I doubt should happen anytime soon. _

Re: [Qemu-devel] [PATCH v2] Release usb devices on shutdown and usb_del command

2010-05-21 Thread Markus Armbruster
"David S. Ahern" writes: >> On 05/21/2010 1:33 AM, Markus Armbruster wrote: >> Note: usbdevice_init() is not for general initialization, just for >> dealing with the legacy -usbdevice command line. > > Perhaps the comment before usbdevice_create is misleading or has some > conversion not complete

Re: [Qemu-devel] [Bug 453617] Re: kvm hangs at 100% cpu when connecting to forwarded ports (when listed incorrectly on the command line)

2010-05-21 Thread Gleb Natapov
Another thing. If my memory serves me right it was promised that message will contain enough info to contact bug reporter without going through launchpad web interface. In other words email of bug reporter will be specified. I can't find it in the message. -- Gleb.

[Qemu-devel] Re: [[RfC PATCH]] linux fbdev display driver prototype.

2010-05-21 Thread Julian Pidancet
On 05/20/2010 09:20 PM, Gerd Hoffmann wrote: > Display works with 32 bpp (both host + guest) only. > Which surprisingly didn't cause much problems so far in my testing. > Host runs with kms and inteldrmfb. > > Mouse support isn't available yet. > I've cheated by passed through the hosts usb mouse

Re: [Qemu-devel] [Bug 453617] Re: kvm hangs at 100% cpu when connecting to forwarded ports (when listed incorrectly on the command line)

2010-05-21 Thread Gleb Natapov
On Fri, May 21, 2010 at 02:23:13PM -0500, Anthony Liguori wrote: > On 05/21/2010 10:42 AM, Gleb Natapov wrote: > >>ProblemType: Bug > >>Architecture: amd64 > >>Date: Fri Oct 16 17:19:59 2009 > >>DistroRelease: Ubuntu 9.10 > >What is the point forwarding distro bugs here? Can we have upstream bug >

Re: [Qemu-devel] [Bug 453617] Re: kvm hangs at 100% cpu when connecting to forwarded ports (when listed incorrectly on the command line)

2010-05-21 Thread Anthony Liguori
On 05/21/2010 10:42 AM, Gleb Natapov wrote: ProblemType: Bug Architecture: amd64 Date: Fri Oct 16 17:19:59 2009 DistroRelease: Ubuntu 9.10 What is the point forwarding distro bugs here? Can we have upstream bug tracker to report upstream issues? These bugs are being explicitly report

Re: [Qemu-devel] [PATCH 1/3] cursor: add cursor functions.

2010-05-21 Thread Blue Swirl
On Fri, May 21, 2010 at 12:29 PM, Anthony Liguori wrote: > On 05/21/2010 04:54 AM, Gerd Hoffmann wrote: >> >> Add a new cursor type to console.h and a bunch of functions to >> deal with cursors the (new) cursor.c file. >> >> Signed-off-by: Gerd Hoffmann >> > > I like this implementation.  Blue, ar

Re: [Qemu-devel] Problems changing dvdrom iso during execution

2010-05-21 Thread David S. Ahern
On 05/21/2010 10:10 AM, Adnan Khaleel wrote: > So do you have any idea whats causing the problem? Is there any other > way I can mount a dvd in Qemu? > > Adnan have you tried ejecting the cd in the guest before changing the file in the monitor? David

Re: [Qemu-devel] [PATCH v2] Release usb devices on shutdown and usb_del command

2010-05-21 Thread David S. Ahern
> On 05/21/2010 1:33 AM, Markus Armbruster wrote: > Note: usbdevice_init() is not for general initialization, just for > dealing with the legacy -usbdevice command line. Perhaps the comment before usbdevice_create is misleading or has some conversion not completed? Right now even devices entered t

[Qemu-devel] Re: [Bug 453617] Re: kvm hangs at 100% cpu when connecting to forwarded ports (when listed incorrectly on the command line)

2010-05-21 Thread Gleb Natapov
On Fri, May 21, 2010 at 09:22:23PM +0400, Michael Tokarev wrote: > But actually I were asking about this particular bugreport, > not about general forwarding - which - at least - should be > done with care :) > Ugh, I am talking about general forwarding not this particular bug report :) Although i

Re: [Qemu-devel] [Bug 453617] Re: kvm hangs at 100% cpu when connecting to forwarded ports (when listed incorrectly on the command line)

2010-05-21 Thread Gleb Natapov
On Fri, May 21, 2010 at 08:31:41PM +0400, Michael Tokarev wrote: > 21.05.2010 19:42, Gleb Natapov wrote: > >>ProblemType: Bug > >>Architecture: amd64 > >>Date: Fri Oct 16 17:19:59 2009 > >>DistroRelease: Ubuntu 9.10 > >What is the point forwarding distro bugs here? Can we have upstream bug > >track

Re: [Qemu-devel] [PATCH 0/6]: QMP: Fix issues in parser/lexer

2010-05-21 Thread Luiz Capitulino
On Thu, 20 May 2010 10:35:52 -0300 Luiz Capitulino wrote: > > > 2. QMP doesn't check the return of json_message_parser_feed() > > > > > > Which means we don't handle JSON syntax errors. While the fix might > > > seem > > > trivial (ie. just return an error!), I'm not sure what's the best

[Qemu-devel] Re: [PATCH] Release usb devices on shutdown and usb_del command

2010-05-21 Thread Shahar Havivi
Remove usb_host_device_release and using usb_host_close to handle usb_del command. Gerd, What do you think about the usb_cleanup()? If it will be in usb-linux.c we will have to ad atexit to each device, if we usb-bus.c we will have to implement in bsd and stub... Signed-off-by: Shahar Havivi ---

[Qemu-devel] [PATCH 1/2] Use calloc in qemu_mallocz.

2010-05-21 Thread Richard Henderson
Avoids the memset if the allocator has gotten new zeroed storage from the operating system. Signed-off-by: Richard Henderson --- qemu-malloc.c |8 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/qemu-malloc.c b/qemu-malloc.c index 6cdc5de..1b33e04 100644 --- a/qemu-ma

Re: [Qemu-devel] linux-user mmap bug

2010-05-21 Thread Richard Henderson
On 05/21/2010 09:38 AM, Richard Henderson wrote: > I have a patch series that attempts to clean this up, but it > isn't quite optimal. I'll post it for reference, however. Bah, the patch sequence no longer applies since Paul removed PAGE_RESERVED. r~

[Qemu-devel] [PATCH 2/2] linux-user: Use qemu-malloc.c.

2010-05-21 Thread Richard Henderson
Since we're no longer setting PAGE_RESERVED, there's no need to implement qemu_malloc via mmap. Signed-off-by: Richard Henderson --- Makefile.target |3 ++- linux-user/mmap.c | 52 2 files changed, 2 insertions(+), 53 deletions(-) dif

[Qemu-devel] [PATCH 0/2] Tidy qemu_malloc

2010-05-21 Thread Richard Henderson
I was reminded of this by the discussion on the list recently of using calloc for qemu_mallocz. I'd vaguely remembered that I already had a patch that did this. The second patch is a cleanup enabled by Paul's patch: 2e9a5713f0567fffaa3518f495b8d16a2b74f84a which removed PAGE_RESERVED. r~ Ri

[Qemu-devel] Re: [Bug 453617] Re: kvm hangs at 100% cpu when connecting to forwarded ports (when listed incorrectly on the command line)

2010-05-21 Thread Jan Kiszka
Michael Tokarev wrote: > 21.05.2010 21:16, Jan Kiszka wrote: >> Michael Tokarev wrote: >>> 21.05.2010 19:42, Gleb Natapov wrote: > ProblemType: Bug > Architecture: amd64 > Date: Fri Oct 16 17:19:59 2009 > DistroRelease: Ubuntu 9.10 What is the point forwarding distro bugs here?

[Qemu-devel] Re: [Bug 453617] Re: kvm hangs at 100% cpu when connecting to forwarded ports (when listed incorrectly on the command line)

2010-05-21 Thread Jan Kiszka
Jan Kiszka wrote: > Michael Tokarev wrote: >> 21.05.2010 19:42, Gleb Natapov wrote: ProblemType: Bug Architecture: amd64 Date: Fri Oct 16 17:19:59 2009 DistroRelease: Ubuntu 9.10 >>> What is the point forwarding distro bugs here? Can we have upstream bug >>> tracker to report up

[Qemu-devel] Re: [Bug 453617] Re: kvm hangs at 100% cpu when connecting to forwarded ports (when listed incorrectly on the command line)

2010-05-21 Thread Jan Kiszka
Michael Tokarev wrote: > 21.05.2010 19:42, Gleb Natapov wrote: >>> ProblemType: Bug >>> Architecture: amd64 >>> Date: Fri Oct 16 17:19:59 2009 >>> DistroRelease: Ubuntu 9.10 >> What is the point forwarding distro bugs here? Can we have upstream bug >> tracker to report upstream issues? > > Um, thi

[Qemu-devel] Re: [PATCH 1/2] Pad iommu with an empty slot (necessary for SunOS 4.1.4)

2010-05-21 Thread Artyom Tarasenko
2010/5/10 Blue Swirl : > On 5/10/10, Artyom Tarasenko wrote: >> 2010/5/10 Blue Swirl : >> >> > On 5/10/10, Artyom Tarasenko wrote: >>  >> 2010/5/9 Blue Swirl : >>  >>  > On 5/9/10, Artyom Tarasenko wrote: >>  >>  >> 2010/5/9 Blue Swirl : >>  >>  >> >>  >>  >> > On 5/8/10, Artyom Tarasenko wrote

[Qemu-devel] Re: [Bug 453617] Re: kvm hangs at 100% cpu when connecting to forwarded ports (when listed incorrectly on the command line)

2010-05-21 Thread Michael Tokarev
21.05.2010 21:16, Jan Kiszka wrote: Michael Tokarev wrote: 21.05.2010 19:42, Gleb Natapov wrote: ProblemType: Bug Architecture: amd64 Date: Fri Oct 16 17:19:59 2009 DistroRelease: Ubuntu 9.10 What is the point forwarding distro bugs here? Can we have upstream bug tracker to report upstream iss

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

2010-05-21 Thread Aurelien Jarno
On Fri, May 21, 2010 at 08:30:20AM -0700, Richard Henderson wrote: > Changes v2->v3: > * ext8u and ext8s operate on 'q' register inputs only. > * pushi is inline and loses the ifdef. > * trailing whitespace cleanup > > > r~ > > > Richard Henderson (15): > tcg-i386: Tidy ext8u and ext16u

Re: [Qemu-devel] [PATCH 1/2] trace: Add simple tracing support

2010-05-21 Thread Jan Kiszka
Anthony Liguori wrote: > On 05/21/2010 08:46 AM, Jan Kiszka wrote: >> Anthony Liguori wrote: >> >>> On 05/21/2010 04:42 AM, Stefan Hajnoczi wrote: >>> Trace events should be defined in trace.h. Events are written to /tmp/trace.log and can be formatted using trace.py. Remember

Re: [Qemu-devel] linux-user mmap bug

2010-05-21 Thread Richard Henderson
On 05/21/2010 06:28 AM, Edgar E. Iglesias wrote: > ptr = mmap(g2h(addr), size, PROT_NONE, > - MAP_ANONYMOUS|MAP_PRIVATE|MAP_NORESERVE, -1, 0); > + /* When the kernel returns addresses that the guest > + cannot use we might need to fa

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

2010-05-21 Thread Richard Henderson
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 a/configure b/configure index 3cd2c5f..ba3aaac 100755 --- a/configure +++ b/configure @@ -2355,6 +2355,9 @@ for d in

Re: [Qemu-devel] Re: lsi: Handle removal of selected devices

2010-05-21 Thread Jan Kiszka
Aurelien Jarno wrote: > On Fri, May 21, 2010 at 05:49:26PM +0200, Aurelien Jarno wrote: >> This patch: >> >> 64d564094cac5f72eeaeb950c442b773a00d3586 is the first bad commit >> commit 64d564094cac5f72eeaeb950c442b773a00d3586 >> Author: Jan Kiszka >> Date: Tue May 4 14:21:03 2010 +0200 >> >>

[Qemu-devel] [PATCH] alpha-linux-user: Fill in SI_CODE for SIGSEGV.

2010-05-21 Thread Richard Henderson
Signed-off-by: Richard Henderson --- linux-user/main.c |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/linux-user/main.c b/linux-user/main.c index b240f29..de1076b 100644 --- a/linux-user/main.c +++ b/linux-user/main.c @@ -2433,7 +2433,8 @@ void cpu_loop (CPUState *env)

Re: [Qemu-devel] Re: lsi: Handle removal of selected devices

2010-05-21 Thread Aurelien Jarno
Jan Kiszka a écrit : > Aurelien Jarno wrote: >> On Fri, May 21, 2010 at 05:49:26PM +0200, Aurelien Jarno wrote: >>> This patch: >>> >>> 64d564094cac5f72eeaeb950c442b773a00d3586 is the first bad commit >>> commit 64d564094cac5f72eeaeb950c442b773a00d3586 >>> Author: Jan Kiszka >>> Date: Tue May 4

[Qemu-devel] [PATCH] lsi: Fix value overflow in request tag processing

2010-05-21 Thread Jan Kiszka
This fixes a mismerge of 64d564094cac5f72eeaeb950c442b773a00d3586 (wrong patch version): We need to mask the tag value properly to obtain its device ID. Signed-off-by: Jan Kiszka --- hw/lsi53c895a.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/hw/lsi53c895a.c b/hw/

[Qemu-devel] Re: [PATCH] add support for protocol driver create_options

2010-05-21 Thread Kevin Wolf
Am 20.05.2010 07:36, schrieb MORITA Kazutaka: > This patch enables protocol drivers to use their create options which > are not supported by the format. For example, protcol drivers can use > a backing_file option with raw format. > > Signed-off-by: MORITA Kazutaka > --- > block.c |7

Re: [Qemu-devel] Re: lsi: Handle removal of selected devices

2010-05-21 Thread Aurelien Jarno
On Fri, May 21, 2010 at 05:49:26PM +0200, Aurelien Jarno wrote: > This patch: > > 64d564094cac5f72eeaeb950c442b773a00d3586 is the first bad commit > commit 64d564094cac5f72eeaeb950c442b773a00d3586 > Author: Jan Kiszka > Date: Tue May 4 14:21:03 2010 +0200 > > lsi: Handle removal of selecte

Re: [Qemu-devel] [PATCH] [S390] Remove warning in tcg stub (tcg_out_reloc)

2010-05-21 Thread Richard Henderson
On 05/21/2010 09:46 AM, Aurelien Jarno wrote: >> +tcg_out_reloc(NULL, NULL, 0, 0, 0); >> } >> > > What about declaring tcg_out_reloc static inline? I think we're not far away from a mergable s390 port. I think the smallest local change is best in the interim. r~

Re: [Qemu-devel] [PATCH 0/5] tcg-i386: tidy softmmu code

2010-05-21 Thread Richard Henderson
On 05/21/2010 09:02 AM, Richard Henderson wrote: > This is sort-of part 2 of the patch series that I just edited and > re-posted for you. The majority of the patch series deals with > the SOFTMMU code. Gah. Sorry for the re-post of parts 10-15 of the previous series. r~

Re: [Qemu-devel] [PATCH 0/5] tcg: Initialize prologue after guest_base fixed

2010-05-21 Thread Aurelien Jarno
On Thu, May 06, 2010 at 08:50:40AM -0700, Richard Henderson wrote: > By doing this we can make any number of decisions about code generation > during the prologue. For instance, we can decide whether or not to > reserve a register to hold the value of GUEST_BASE. > > This fixes a latent bug in th

Re: [Qemu-devel] [PATCH 0/7] alpha-linux syscall fixes

2010-05-21 Thread Aurelien Jarno
On Mon, May 03, 2010 at 10:07:48AM -0700, Richard Henderson wrote: > I've been doing a bit of testing of the alpha TCG port by running > QEMU compiled for alpha on QEMU compiled for x86-64. Which is an > interesting challenge for the linux-user code, and has found a > few bugs. > > > r~ > > >

Re: [Qemu-devel] [PATCH] linux-user: do not warn for missing pselect6

2010-05-21 Thread Aurelien Jarno
On Fri, May 07, 2010 at 12:28:05PM +, Riku Voipio wrote: > Libc will fallback gracefully if pselect6 is not available. Thus put > pselect6 to nowarn until the atomicity issues of the original pselect6 > patch are dealt with. > > Signed-off-by: Riku Voipio > Cc: Michael Casadevall Thanks, ap

[Qemu-devel] [PATCH 4/5] tcg-i386: Split out tlb load function.

2010-05-21 Thread Richard Henderson
Share some code between qemu_ld and qemu_st. Signed-off-by: Richard Henderson --- tcg/i386/tcg-target.c | 203 +++- 1 files changed, 97 insertions(+), 106 deletions(-) diff --git a/tcg/i386/tcg-target.c b/tcg/i386/tcg-target.c index 0d85ec0..cf621da

Re: [Qemu-devel] [Bug 453617] Re: kvm hangs at 100% cpu when connecting to forwarded ports (when listed incorrectly on the command line)

2010-05-21 Thread Michael Tokarev
21.05.2010 19:42, Gleb Natapov wrote: ProblemType: Bug Architecture: amd64 Date: Fri Oct 16 17:19:59 2009 DistroRelease: Ubuntu 9.10 What is the point forwarding distro bugs here? Can we have upstream bug tracker to report upstream issues? Um, this _is_ upstream issue, as far as I can see. It

Re: [Qemu-devel] [PATCH] [S390] Remove warning in tcg stub (tcg_out_reloc)

2010-05-21 Thread Aurelien Jarno
On Tue, May 11, 2010 at 06:23:32PM +0200, Alexander Graf wrote: > On S390 we don't have a real TCG implementation but use a stub instead. This > stub obviously doesn't call any of the TCG helper functions that are usually > used by the other TCG targets. > > If such a helper function is static tho

[Qemu-devel] [Bug 583462] Re: qemu disables screensaver

2010-05-21 Thread Michael Tokarev
Debian bugreport about this: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=578672 (#578672). I tried it here again, with your program (using 0.12.4 from debian) - and it works as expected for me, no resets every 4 secs, screen saver triggers after configured time even if the input is grabbed

Re: [Qemu-devel] Re: lsi: Handle removal of selected devices

2010-05-21 Thread Jan Kiszka
Jan Kiszka wrote: > Aurelien Jarno wrote: >> On Fri, May 21, 2010 at 05:49:26PM +0200, Aurelien Jarno wrote: >>> This patch: >>> >>> 64d564094cac5f72eeaeb950c442b773a00d3586 is the first bad commit >>> commit 64d564094cac5f72eeaeb950c442b773a00d3586 >>> Author: Jan Kiszka >>> Date: Tue May 4 14:

Re: [Qemu-devel] Re: lsi: Handle removal of selected devices

2010-05-21 Thread Jan Kiszka
Jan Kiszka wrote: > Jan Kiszka wrote: >> Aurelien Jarno wrote: >>> On Fri, May 21, 2010 at 05:49:26PM +0200, Aurelien Jarno wrote: This patch: 64d564094cac5f72eeaeb950c442b773a00d3586 is the first bad commit commit 64d564094cac5f72eeaeb950c442b773a00d3586 Author: Jan Kiszka

[Qemu-devel] [PATCH 5/5] tcg-i386: Remove some ifdefs in qemu_ld/st.

2010-05-21 Thread Richard Henderson
Tidy some code by replacing ifdefs by C ifs. Signed-off-by: Richard Henderson --- tcg/i386/tcg-target.c | 85 ++--- 1 files changed, 38 insertions(+), 47 deletions(-) diff --git a/tcg/i386/tcg-target.c b/tcg/i386/tcg-target.c index cf621da..8a9122c

[Qemu-devel] [PATCH 1/5] tcg-i386: Tidy data16 prefixes.

2010-05-21 Thread Richard Henderson
Include it in the opcode as an extension, as with P_EXT or the REX bits in the x86-64 port. Signed-off-by: Richard Henderson --- tcg/i386/tcg-target.c | 28 ++-- 1 files changed, 18 insertions(+), 10 deletions(-) diff --git a/tcg/i386/tcg-target.c b/tcg/i386/tcg-target

[Qemu-devel] [PATCH 2/5] tcg-i386: Split out TLB Hit path from qemu_ld/st.

2010-05-21 Thread Richard Henderson
Splitting out these functions will allow further cleanups. Signed-off-by: Richard Henderson --- tcg/i386/tcg-target.c | 197 + 1 files changed, 102 insertions(+), 95 deletions(-) diff --git a/tcg/i386/tcg-target.c b/tcg/i386/tcg-target.c index 92

[Qemu-devel] Re: lsi: Handle removal of selected devices

2010-05-21 Thread Jan Kiszka
Aurelien Jarno wrote: > This patch: > > 64d564094cac5f72eeaeb950c442b773a00d3586 is the first bad commit > commit 64d564094cac5f72eeaeb950c442b773a00d3586 > Author: Jan Kiszka > Date: Tue May 4 14:21:03 2010 +0200 > > lsi: Handle removal of selected devices > > We must not store refer

[Qemu-devel] [PATCH 3/5] tcg-i386: Swap order of TLB hit and miss paths.

2010-05-21 Thread Richard Henderson
Make fallthru be TLB hit and branch be TLB miss. Doing this both improves branch prediction and will allow further cleanup. Signed-off-by: Richard Henderson --- tcg/i386/tcg-target.c | 172 +++-- 1 files changed, 80 insertions(+), 92 deletions(-) di

Re: [Qemu-devel] Problems changing dvdrom iso during execution

2010-05-21 Thread Adnan Khaleel
So do you have any idea whats causing the problem? Is there any other way I can mount a dvd in Qemu? Adnan _ From: David S. Ahern [mailto:daah...@cisco.com] To: ad...@khaleel.us Cc: Qemu-devel@nongnu.org Sent: Thu, 20 May 2010 17:45:11 -0500 Subject: Re: [Qemu-devel] Problems changing d

[Qemu-devel] [PATCH 0/5] tcg-i386: tidy softmmu code

2010-05-21 Thread Richard Henderson
This is sort-of part 2 of the patch series that I just edited and re-posted for you. The majority of the patch series deals with the SOFTMMU code. I think you saw a patch about the data16 prefix before, but this one's different. It's written the way it is here because I have a third patch serie

[Qemu-devel] [PATCH 15/15] tcg-i386: Nuke trailing whitespace.

2010-05-21 Thread Richard Henderson
Signed-off-by: Richard Henderson --- tcg/i386/tcg-target.c | 52 1 files changed, 26 insertions(+), 26 deletions(-) diff --git a/tcg/i386/tcg-target.c b/tcg/i386/tcg-target.c index fc61e80..396a2f1 100644 --- a/tcg/i386/tcg-target.c +++ b/tcg/i3

[Qemu-devel] Re: lsi: Handle removal of selected devices

2010-05-21 Thread Aurelien Jarno
This patch: 64d564094cac5f72eeaeb950c442b773a00d3586 is the first bad commit commit 64d564094cac5f72eeaeb950c442b773a00d3586 Author: Jan Kiszka Date: Tue May 4 14:21:03 2010 +0200 lsi: Handle removal of selected devices We must not store references to selected devices as they may be

[Qemu-devel] [PATCH 11/15] tcg-i386: Tidy multiply.

2010-05-21 Thread Richard Henderson
Define and use OPC_IMUL_GvEv{,Ib,Iz}. Signed-off-by: Richard Henderson --- tcg/i386/tcg-target.c |9 ++--- 1 files changed, 6 insertions(+), 3 deletions(-) diff --git a/tcg/i386/tcg-target.c b/tcg/i386/tcg-target.c index 07da8c7..8eb88da 100644 --- a/tcg/i386/tcg-target.c +++ b/tcg/i386

Re: [Qemu-devel] [Bug 453617] Re: kvm hangs at 100% cpu when connecting to forwarded ports (when listed incorrectly on the command line)

2010-05-21 Thread Gleb Natapov
> ProblemType: Bug > Architecture: amd64 > Date: Fri Oct 16 17:19:59 2009 > DistroRelease: Ubuntu 9.10 What is the point forwarding distro bugs here? Can we have upstream bug tracker to report upstream issues? -- Gleb.

[Qemu-devel] [PATCH 13/15] tcg-i386: Tidy lea.

2010-05-21 Thread Richard Henderson
Implement full modrm+sib addressing mode processing. Use that in qemu_ld/st to output the LEA. Signed-off-by: Richard Henderson --- tcg/i386/tcg-target.c | 91 - 1 files changed, 60 insertions(+), 31 deletions(-) diff --git a/tcg/i386/tcg-target

[Qemu-devel] [PATCH 10/15] tcg-i386: Tidy unary arithmetic.

2010-05-21 Thread Richard Henderson
Define OPC_GRP3 and EXT3_FOO to match. Use them instead of bare constants. Define OPC_GRP5 and rename the existing EXT_BAR to EXT5_BAR to make it clear which extension should be used with which opcode. Signed-off-by: Richard Henderson --- tcg/i386/tcg-target.c | 38 +-

[Qemu-devel] [PATCH 12/15] tcg-i386: Tidy xchg.

2010-05-21 Thread Richard Henderson
Define and use OPC_XCHG_ax_r32. Signed-off-by: Richard Henderson --- tcg/i386/tcg-target.c |4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/tcg/i386/tcg-target.c b/tcg/i386/tcg-target.c index 8eb88da..09a56e8 100644 --- a/tcg/i386/tcg-target.c +++ b/tcg/i386/tcg-target

[Qemu-devel] [PATCH 08/15] tcg-i386: Tidy ret.

2010-05-21 Thread Richard Henderson
Define and use OPC_RET. Signed-off-by: Richard Henderson --- tcg/i386/tcg-target.c |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/tcg/i386/tcg-target.c b/tcg/i386/tcg-target.c index 28a01f3..c258775 100644 --- a/tcg/i386/tcg-target.c +++ b/tcg/i386/tcg-target.c @@ -18

[Qemu-devel] [PATCH 14/15] tcg-i386: Use lea for three-operand add.

2010-05-21 Thread Richard Henderson
The result is shorter than the mov+add that TCG would otherwise generate for us. Signed-off-by: Richard Henderson --- tcg/i386/tcg-target.c | 23 --- 1 files changed, 20 insertions(+), 3 deletions(-) diff --git a/tcg/i386/tcg-target.c b/tcg/i386/tcg-target.c index 6562a5d.

[Qemu-devel] [PATCH 05/15] tcg-i386: Tidy movi.

2010-05-21 Thread Richard Henderson
Define and use OPC_MOVL_Iv. Signed-off-by: Richard Henderson --- tcg/i386/tcg-target.c |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/tcg/i386/tcg-target.c b/tcg/i386/tcg-target.c index bc5985f..1d227db 100644 --- a/tcg/i386/tcg-target.c +++ b/tcg/i386/tcg-target.c @@

[Qemu-devel] [PATCH 07/15] tcg-i386: Tidy calls.

2010-05-21 Thread Richard Henderson
Define OPC_CALL_Jz, generated by tcg_out_calli; use the later throughout. Unify the calls within qemu_st; adjust the stack with a single pop if applicable. Define and use EXT_CALLN_Ev for indirect calls. Signed-off-by: Richard Henderson --- tcg/i386/tcg-target.c | 49

[Qemu-devel] [PATCH 04/15] tcg-i386: Tidy non-immediate arithmetic operations.

2010-05-21 Thread Richard Henderson
Add more OPC values, and tgen_arithr. Use the later throughout. Note that normal reg/reg arithmetic now uses the Gv,Ev opcode form instead of the Ev,Gv opcode form used previously. Both forms disassemble properly, and so there's no visible change when diffing log files before and after the chang

[Qemu-devel] [PATCH 09/15] tcg-i386: Tidy setcc.

2010-05-21 Thread Richard Henderson
Define and use OPC_SETCC. Signed-off-by: Richard Henderson --- tcg/i386/tcg-target.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/tcg/i386/tcg-target.c b/tcg/i386/tcg-target.c index c258775..a43dbfb 100644 --- a/tcg/i386/tcg-target.c +++ b/tcg/i386/tcg-target.c @@

[Qemu-devel] [PATCH 02/15] tcg-i386: Tidy ext8s and ext16s operations.

2010-05-21 Thread Richard Henderson
Define OPC_MOVSBL and OPC_MOVSWL. Factor opcode emission to separate functions. Signed-off-by: Richard Henderson --- tcg/i386/tcg-target.c | 35 --- 1 files changed, 24 insertions(+), 11 deletions(-) diff --git a/tcg/i386/tcg-target.c b/tcg/i386/tcg-target.c i

[Qemu-devel] [PATCH 03/15] tcg-i386: Tidy immediate arithmetic operations.

2010-05-21 Thread Richard Henderson
Define OPC_ARITH_EvI[bz]; use throughout. Use tcg_out_ext8u directly in setcond. Use tgen_arithi in qemu_ld/st. Signed-off-by: Richard Henderson --- tcg/i386/tcg-target.c | 26 ++ 1 files changed, 10 insertions(+), 16 deletions(-) diff --git a/tcg/i386/tcg-target.c b

[Qemu-devel] [PATCH 06/15] tcg-i386: Tidy push/pop.

2010-05-21 Thread Richard Henderson
Move tcg_out_push/pop up in the file so that they can be used by qemu_ld/st. Define a tcg_out_pushi to be used as well. Signed-off-by: Richard Henderson --- tcg/i386/tcg-target.c | 48 ++-- 1 files changed, 30 insertions(+), 18 deletions(-) diff --

[Qemu-devel] [PATCH 01/15] tcg-i386: Tidy ext8u and ext16u operations.

2010-05-21 Thread Richard Henderson
Define OPC_MOVZBL and OPC_MOVZWL. Factor opcode emission to separate functions. Signed-off-by: Richard Henderson --- tcg/i386/tcg-target.c | 57 ++-- 1 files changed, 31 insertions(+), 26 deletions(-) diff --git a/tcg/i386/tcg-target.c b/tcg/i386/t

[Qemu-devel] [PATCH 00/15] tcg-i386 cleanup and improvement, v3

2010-05-21 Thread Richard Henderson
Changes v2->v3: * ext8u and ext8s operate on 'q' register inputs only. * pushi is inline and loses the ifdef. * trailing whitespace cleanup r~ Richard Henderson (15): tcg-i386: Tidy ext8u and ext16u operations. tcg-i386: Tidy ext8s and ext16s operations. tcg-i386: Tidy immediate ari

[Qemu-devel] [PATCH] resent: x86/cpuid: propagate further CPUID leafs when -cpu host

2010-05-21 Thread Andre Przywara
-cpu host currently only propagates the CPU's family/model/stepping, the brand name and the feature bits. Add a whitelist of safe CPUID leafs to let the guest see the actual CPU's cache details and other things. Signed-off-by: Andre Przywara --- target-i386/cpu.h |6 +- target-i386/cpu

Re: [Qemu-devel] [PATCH 1/2] trace: Add simple tracing support

2010-05-21 Thread Anthony Liguori
On 05/21/2010 08:46 AM, Jan Kiszka wrote: Anthony Liguori wrote: On 05/21/2010 04:42 AM, Stefan Hajnoczi wrote: Trace events should be defined in trace.h. Events are written to /tmp/trace.log and can be formatted using trace.py. Remember to add events to trace.py for pretty-printing

Re: [Qemu-devel] [PATCH 1/2] trace: Add simple tracing support

2010-05-21 Thread Jan Kiszka
Anthony Liguori wrote: > On 05/21/2010 04:42 AM, Stefan Hajnoczi wrote: >> Trace events should be defined in trace.h. Events are written to >> /tmp/trace.log and can be formatted using trace.py. Remember to add >> events to trace.py for pretty-printing. >> >> Signed-off-by: Stefan Hajnoczi >> ---

[Qemu-devel] linux-user mmap bug

2010-05-21 Thread Edgar E. Iglesias
Hi I ran into an mmap problem linux-user emulating CRIS (32bit) on x86_64 hosts. Guest asks for a non fixed mmap, QEMU tries the mmap but the kernel returns a high 64bit address. QEMU notices that it wont fit in the guests 32bit ptr size and retries with a low address but doesn't set the MAP_FIXED

[Qemu-devel] Re: vgabios plans ( Re: [PATCH 1/5] Makefile cleanup)

2010-05-21 Thread Gerd Hoffmann
On 05/20/10 15:04, Anthony Liguori wrote: Hmm. No response for weeks from vgabios folks on this patch series. How to go forward best with vgabios bits? Just upgrade http://git.qemu.org/vgabios.git/ to 0.6c, then apply patches there? Yeah, I think the long term goal should be to move to SeaBIOS

[Qemu-devel] Re: [PATCH 1/2] trace: Add simple tracing support

2010-05-21 Thread Stefan Hajnoczi
On Fri, May 21, 2010 at 12:13 PM, Jan Kiszka wrote: > Stefan Hajnoczi wrote: >> Trace events should be defined in trace.h.  Events are written to >> /tmp/trace.log and can be formatted using trace.py.  Remember to add >> events to trace.py for pretty-printing. > > When already writing to a file, w

[Qemu-devel] Re: [PATCH 1/2] trace: Add simple tracing support

2010-05-21 Thread Jan Kiszka
Stefan Hajnoczi wrote: > On Fri, May 21, 2010 at 12:13 PM, Jan Kiszka wrote: >> Stefan Hajnoczi wrote: >>> Trace events should be defined in trace.h. Events are written to >>> /tmp/trace.log and can be formatted using trace.py. Remember to add >>> events to trace.py for pretty-printing. >> When

[Qemu-devel] [PATCH] resent: x86/cpuid: Add kvm32 CPU model

2010-05-21 Thread Andre Przywara
Create a kvm32 CPU model that describes a least common denominator for KVM capable guest CPUs. Useful for migration purposes. Signed-off-by: Andre Przywara --- target-i386/cpuid.c | 14 ++ 1 files changed, 14 insertions(+), 0 deletions(-) diff --git a/target-i386/cpuid.c b/target-

Re: [Qemu-devel] forking i386 binaries on arm linux user mode

2010-05-21 Thread Damion Yates
On Tue, 11 May 2010, Richard Henderson wrote: > On 05/11/2010 10:08 AM, Damion Yates wrote: > > Also is there some magic in gnemul-x86 beyond being a set of x86 > > libs? > > No. Okay, good to know. > > Does it do any shortcutting to system calls in native rather than > > sticking with the lib

Re: [Qemu-devel] [PATCH] fix curses update - v2

2010-05-21 Thread Bernhard Kauer
On Fri, May 21, 2010 at 02:02:43PM +0200, andrzej zaborowski wrote: > I pushed a modified patch to preserve attributes such as background > colour. Good idea. > Please check if this works for you. Yes, this works. Thanks, Bernhard Kauer

  1   2   >