[Qemu-devel] qemu for android arm

2012-08-17 Thread Xin Tong
Does QEMU have all the supports to boot android on arm ? Xin

Re: [Qemu-devel] [PATCH] block: handle filenames with colons better

2012-08-17 Thread Iustin Pop
On Fri, Aug 17, 2012 at 12:15:41PM +0200, Kevin Wolf wrote: > Am 17.08.2012 12:05, schrieb Iustin Pop: > > On Fri, Aug 17, 2012 at 09:56:35AM +0200, Kevin Wolf wrote: > >> Am 17.08.2012 09:15, schrieb Iustin Pop: > >>> On Thu, Aug 16, 2012 at 11:24:11PM +0400, Michael Tokarev wrote: > On 16.08

[Qemu-devel] [PATCH] x86: enforce DPL checking on task gate switches invoked through IDT

2012-08-17 Thread Alex ZUEPKE
Hi, x86 software emulation (non-KVM mode) does not check privilege levels on task gate switches ... so one can invoke a kernel's double fault handler from user space -- very bad. Expected behaviour (testcase works with any linux distribution + gcc): $ cat test.c int main(void) { _

Re: [Qemu-devel] Windows slow boot: contractor wanted

2012-08-17 Thread Robert Vineyard
Richard, Not sure if you've tried this, but I noticed massive performance gains (easily booting 2-3 times as fast) by converting from RAW disk images to direct-mapped raw partitions and making sure that IOMMU support was enabled in the BIOS and in the kernel at boot time. The obvious downside

[Qemu-devel] [Bug 1038136] [NEW] lack of keycode 89 for br-abnt2 keyboards

2012-08-17 Thread Gilcio Amaral Martins
Public bug reported: qemu-kvm-1.1.1 host system: slackware64-13.37 Bug detailed description: Independent of Guest OS nothing happens when keycode 89 is pressed. If you select option "-k pt-br" at qemu commandline you get keycode 89 but there is no more keycode 26 (dead_acute dead_grave) and keyc

[Qemu-devel] [Bug 1038070] [NEW] > qemu-kvm-1.1.1-r1 - USB activkey doesn't work anymore

2012-08-17 Thread linuxale
Public bug reported: Linux Distro: Gentoo Smartcard Activkey doesn't work anymore. I use it without problem till version qemu-kvm-1.0.1. Follow a log extraction: 2012-08-14 16:27:34.751+: 5487: error : qemuProcessReadLogOutput:1298 : internal error Process exited while reading console log ou

[Qemu-devel] [PATCH] x86: enforce DPL checking on task gate switches invoked through IDT

2012-08-17 Thread Alex ZUEPKE
Hi, x86 software emulation (non-KVM mode) does not check privilege levels on task gate switches ... so one can invoke a kernel's double fault handler from user space -- very bad. Expected behaviour (testcase works with any linux distribution + gcc): $ cat test.c int main(void) { _

[Qemu-devel] Qemu phone modem emulation

2012-08-17 Thread Eric Koldeweij
Hello all, I am currently working on a serial telephone modem emulation implementation for Qemu. My question is if it would be a good idea to offer it for inclusion in the qemu main trunk when it's ready. These are the requirements I am working with: - The modem device is accessible through

[Qemu-devel] [PATCH 4/4] virtio-blk: hide VIRTIO_BLK_F_CONFIG_WCE from old machine types

2012-08-17 Thread Kevin Wolf
From: Stefan Hajnoczi QEMU has a policy of keeping a stable guest device ABI. When new guest device features are introduced they must not change hardware info seen by existing guests. This is important because operating systems or applications may "fingerprint" the hardware and refuse to run wh

[Qemu-devel] [PATCH 3/4] Documentation: Warn against qemu-img on active image

2012-08-17 Thread Kevin Wolf
People have repeatedly expected that you can do things like snapshotting an image with qemu-img while a qemu instance is running. Maybe we need to consider locking the files while they are in use, but having a warning in the qemu-img manpage is doable for 1.2 and can't hurt anyway. Signed-off-by:

[Qemu-devel] [PATCH 1/4] vmdk: Fix header structure

2012-08-17 Thread Kevin Wolf
Commit bb45ded9 swapped gd_offset and rgd_offset. This is wrong. Signed-off-by: Kevin Wolf --- block/vmdk.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/block/vmdk.c b/block/vmdk.c index daee426..9648398 100644 --- a/block/vmdk.c +++ b/block/vmdk.c @@ -57,8 +57,8 @@

[Qemu-devel] [PATCH 2/4] vmdk: Read footer for streamOptimized images

2012-08-17 Thread Kevin Wolf
The footer takes precedence over the header when it exists. It contains the real grain directory offset that is missing in the header. Without this patch, streamOptimized images with a footer cannot be read. Signed-off-by: Kevin Wolf Reviewed-by: Stefan Hajnoczi Reviewed-by: Jeff Cody --- bloc

[Qemu-devel] [PULL 0/4] Block patches for 1.2-rc1

2012-08-17 Thread Kevin Wolf
The following changes since commit 731dc9ecd4f2c3041538f7eb2d10eee0cb82da1b: Update version to 1.2.0-rc0 (2012-08-16 13:56:34 -0500) are available in the git repository at: http://repo.or.cz/r/qemu/kevin.git for-anthony Kevin Wolf (3): vmdk: Fix header structure vmdk: Read footer

Re: [Qemu-devel] [PATCHv3 2/2] envlist.c: handle strdup failure

2012-08-17 Thread Jim Meyering
Andreas Färber wrote: > Am 22.05.2012 12:16, schrieb Jim Meyering: >> From: Jim Meyering >> >> Without this, envlist_to_environ may silently fail to copy all >> strings into the destination buffer, and both callers would leak >> any env strings allocated after a failing strdup, because the >> free

[Qemu-devel] [PATCHv4 2/2] envlist.c: handle strdup failure

2012-08-17 Thread Jim Meyering
From: Jim Meyering Without this, envlist_to_environ may silently fail to copy all strings into the destination buffer, and both callers would leak any env strings allocated after a failing strdup, because the freeing code stops at the first NULL pointer. Signed-off-by: Jim Meyering --- envlist

[Qemu-devel] [PATCHv4 1/2] envlist.c: conform to QEMU's coding style

2012-08-17 Thread Jim Meyering
From: Jim Meyering Convert each TAB(width-4) to equivalent spaces. Put braces around each one-line if-body. Signed-off-by: Jim Meyering --- envlist.c | 268 -- 1 file changed, 140 insertions(+), 128 deletions(-) diff --git a/envlist.

[Qemu-devel] [PATCHv4 0/2] envlist.c: handle strdup failure

2012-08-17 Thread Jim Meyering
From: Jim Meyering Differences from v3 (no semantic change): - change 1/2 so this file conforms more closely to QEMU's coding style, by adding braces around each one-line if body (there was no one-line else- or while-block). - move an indentation correction from 2/2 into 1/2 Jim Meye

[Qemu-devel] [RFC 5/6] i386: cpu: eliminate duplicate feature names

2012-08-17 Thread Eduardo Habkost
Instead of having duplicate feature names on the ext2_feature array for the AMD feature bit aliases, we keep the feature names only on the feature_name[] array, and copy the corresponding bits to cpuid_ext2_features in case the CPU vendor is AMD. This will: - Make sure we don't set the feature bi

[Qemu-devel] [RFC 1/6] x86_cpudef_setup: coding style change

2012-08-17 Thread Eduardo Habkost
Make source code lines shorter. Signed-off-by: Eduardo Habkost --- target-i386/cpu.c | 15 +-- 1 file changed, 9 insertions(+), 6 deletions(-) diff --git a/target-i386/cpu.c b/target-i386/cpu.c index 96ec9e6..519a104 100644 --- a/target-i386/cpu.c +++ b/target-i386/cpu.c @@ -1500,20

[Qemu-devel] [RFC 6/6] i386: -cpu help: remove reference to specific CPUID leaves/registers

2012-08-17 Thread Eduardo Habkost
The -cpu configuration interface is based on a list of feature names or properties, on a single namespace, so there's no need to mention on which CPUID leaf/register each flag is located. Signed-off-by: Eduardo Habkost --- target-i386/cpu.c | 8 1 file changed, 4 insertions(+), 4 deleti

[Qemu-devel] [RFC 3/6] i386: kvm: bit 10 of CPUID[8000_0001].EDX is reserved

2012-08-17 Thread Eduardo Habkost
Bit 10 of CPUID[8000_0001].EDX is not defined as an alias of CPUID[1].EDX[10], so do not duplicate it on kvm_arch_get_supported_cpuid(). Signed-off-by: Eduardo Habkost --- target-i386/kvm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target-i386/kvm.c b/target-i386/kvm.c

[Qemu-devel] [RFC 0/6] i386: CPU: remove duplicate feature names

2012-08-17 Thread Eduardo Habkost
The problem: - Some features are report at the same time on both CPUID[1].EDX and CPUID[8000_0001].EDX on AMD CPUs (e.g. fpu, tsc, msr, pae, mmx). - "-cpu ,+feature" should enable the bit only on CPUID[1] if it's not an AMD CPU, but it should enable the bit on both CPUID[1] and CPUID[80

[Qemu-devel] [RFC 2/6] i386: kill cpudef config section support

2012-08-17 Thread Eduardo Habkost
It's nice to have a flexible system to maintain CPU models as data, but this is holding us from making improvements in the CPU code because it's not using the common infra-structure, and because the machine-type data is still inside C code. Users who want to configure CPU features directly may sim

[Qemu-devel] [RFC 4/6] i386: kvm: use a #define for the set of alias feature bits

2012-08-17 Thread Eduardo Habkost
Instea of using a hardcoded hex constant, define CPUID_EXT2_AMD_ALIASES as the set of CPUID[8000_0001].EDX bits that on AMD are the same as the bits of CPUID[1].EDX. Signed-off-by: Eduardo Habkost --- target-i386/cpu.h | 12 target-i386/kvm.c | 2 +- 2 files changed, 13 insertions(

Re: [Qemu-devel] Q35 OS install status

2012-08-17 Thread Luiz Capitulino
On Wed, 15 Aug 2012 17:05:44 +0200 Alexander Graf wrote: > >> BSD > >> --- > >> Luiz tested these for me. > >> > >> OpenBSD 5.1, FreeBSD 9.0 and NetBSD all worked with the ide controller > >> passed on the command line. AHCI didn't work. > > > > Sorry if I wasn't clear in my report, I've only t

Re: [Qemu-devel] [PATCH 07/21] target-i386: convert cpuid features into properties

2012-08-17 Thread Eduardo Habkost
On Thu, Aug 16, 2012 at 03:10:50PM -0300, Eduardo Habkost wrote: > On Wed, Aug 15, 2012 at 06:13:27PM +0200, Igor Mammedov wrote: > > Signed-off-by: Igor Mammedov > > -- > > v2: > > * replaced mask/ffs tricks by plain 'for (bit = 0; bit < 32; bit++)' > > as suggested by Eduardo Habkost > > -

[Qemu-devel] qemu-ga : Guest Agent : Windows 2008 : Unknown command guest-fsfreeze-freeze

2012-08-17 Thread desi babu
Guest-Agent : Windows 2008 Error : Relase 1.1.90 error : internal error unable to execute QEMU command 'guest-fsfreeze-freeze': this feature or command is not currently supported. Guest-info shows the command is available.  Is there any information available on the list of commands supported in

Re: [Qemu-devel] [PATCH 1/3] usb: Halt ep queue en cancel pending packets on a packet error

2012-08-17 Thread Andreas Färber
Not being too familiar with the USB code I wonder if $subject was supposed to say "and cancel"? Andreas -- SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg

Re: [Qemu-devel] [RESEND][PATCH for-1.2] i82378: Remove bogus MMIO coalescing

2012-08-17 Thread Andreas Färber
Am 17.08.2012 12:56, schrieb Jan Kiszka: > This MMIO area is an entry gate to legacy PC ISA devices, addressed via > PIO over there. Quite a few of the PIO ports have side effects on access > like starting/stopping timers that must be executed properly ordered > /wrt the CPU. So we have to remove t

Re: [Qemu-devel] [PATCHv2 2/2] envlist.c: handle strdup failure

2012-08-17 Thread Andreas Färber
Am 17.08.2012 17:34, schrieb Jim Meyering: > Andreas Färber wrote: > >> Am 17.08.2012 15:35, schrieb Jim Meyering: >>> Jim Meyering wrote: From: Jim Meyering Without this, envlist_to_environ may silently fail to copy all strings into the destination buffer, and both callers wo

Re: [Qemu-devel] [PATCHv3 2/2] envlist.c: handle strdup failure

2012-08-17 Thread Andreas Färber
Am 22.05.2012 12:16, schrieb Jim Meyering: > From: Jim Meyering > > Without this, envlist_to_environ may silently fail to copy all > strings into the destination buffer, and both callers would leak > any env strings allocated after a failing strdup, because the > freeing code stops at the first N

Re: [Qemu-devel] [PATCHv3 1/2] envlist.c: convert each TAB(width-4) to equivalent spaces

2012-08-17 Thread Andreas Färber
Am 22.05.2012 12:16, schrieb Jim Meyering: > From: Jim Meyering > > > Signed-off-by: Jim Meyering > --- > envlist.c | 256 > +++--- > 1 file changed, 128 insertions(+), 128 deletions(-) > > diff --git a/envlist.c b/envlist.c > index f23

[Qemu-devel] [PATCH v7 1.2] qxl: add QXL_IO_MONITORS_CONFIG_ASYNC

2012-08-17 Thread Alon Levy
Revision bumped to 4 for new IO support, enabled for spice-server >= 0.11.1. New io enabled iff spice-server >= 0.11.1 && spice-protocol >= 0.12.0. On migration reissue spice_qxl_monitors_config_async. RHBZ: 770842 Signed-off-by: Alon Levy --- Fixed another defined I missed. This time used grep

Re: [Qemu-devel] [PATCHv2 2/2] envlist.c: handle strdup failure

2012-08-17 Thread Jim Meyering
Andreas Färber wrote: > Am 17.08.2012 15:35, schrieb Jim Meyering: >> Jim Meyering wrote: >>> From: Jim Meyering >>> >>> Without this, envlist_to_environ may silently fail to copy all >>> strings into the destination buffer, and both callers would leak >>> any env strings allocated after a failin

[Qemu-devel] [PATCH] usb-redir: Never return USB_RET_NAK for async handled packets

2012-08-17 Thread Hans de Goede
USB_RET_NAK is not a valid response for async handled packets (and will trigger an assert as such). Also drop the warning when receiving a status of cancelled for packets not cancelled by qemu itself, this can happen when a device gets unredirected by the usbredir-host while transfers are pending.

Re: [Qemu-devel] [PATCHv2 2/2] envlist.c: handle strdup failure

2012-08-17 Thread Andreas Färber
Am 17.08.2012 15:35, schrieb Jim Meyering: > Jim Meyering wrote: >> From: Jim Meyering >> >> Without this, envlist_to_environ may silently fail to copy all >> strings into the destination buffer, and both callers would leak >> any env strings allocated after a failing strdup, because the >> freein

Re: [Qemu-devel] qemu-kvm-1.0.1 - unable to exit if vcpu is in infinite loop

2012-08-17 Thread Jan Kiszka
On 2012-08-17 16:41, Jan Kiszka wrote: > On 2012-08-17 16:36, Jan Kiszka wrote: >> On 2012-08-17 15:11, Jan Kiszka wrote: >>> On 2012-08-06 17:11, Stefan Hajnoczi wrote: On Thu, Jun 28, 2012 at 2:05 PM, Peter Lieven wrote: > i debugged my initial problem further and found out that the pro

Re: [Qemu-devel] [PATCH] monitor: Fix warning from clang

2012-08-17 Thread Luiz Capitulino
On Fri, 17 Aug 2012 16:41:34 +0200 Markus Armbruster wrote: > Luiz Capitulino writes: > > > On Fri, 17 Aug 2012 16:10:12 +0200 > > Markus Armbruster wrote: > > > >> Stefan Weil writes: > >> > >> > ccc-analyzer reports these warnings: > >> > > >> > monitor.c:3532:21: warning: Division by zero

Re: [Qemu-devel] [PATCH] monitor: Fix warning from clang

2012-08-17 Thread Markus Armbruster
Luiz Capitulino writes: > On Fri, 17 Aug 2012 16:10:12 +0200 > Markus Armbruster wrote: > >> Stefan Weil writes: >> >> > ccc-analyzer reports these warnings: >> > >> > monitor.c:3532:21: warning: Division by zero >> > val %= val2; >> > ^ >> > monitor.c:3530:

Re: [Qemu-devel] qemu-kvm-1.0.1 - unable to exit if vcpu is in infinite loop

2012-08-17 Thread Jan Kiszka
On 2012-08-17 16:36, Jan Kiszka wrote: > On 2012-08-17 15:11, Jan Kiszka wrote: >> On 2012-08-06 17:11, Stefan Hajnoczi wrote: >>> On Thu, Jun 28, 2012 at 2:05 PM, Peter Lieven wrote: i debugged my initial problem further and found out that the problem happens to be that the m

Re: [Qemu-devel] qemu-kvm-1.0.1 - unable to exit if vcpu is in infinite loop

2012-08-17 Thread Jan Kiszka
On 2012-08-17 15:11, Jan Kiszka wrote: > On 2012-08-06 17:11, Stefan Hajnoczi wrote: >> On Thu, Jun 28, 2012 at 2:05 PM, Peter Lieven wrote: >>> i debugged my initial problem further and found out that the problem happens >>> to be that >>> the main thread is stuck in pause_all_vcpus() on reset or

Re: [Qemu-devel] [RFC PATCH] vfio: add fixup for broken PCI devices

2012-08-17 Thread Alexey Kardashevskiy
On Fri, Jun 22, 2012 at 6:16 PM, Alexey Kardashevskiy wrote: > On 07/06/12 09:17, Alex Williamson wrote: > > On Fri, 2012-05-25 at 17:35 +1000, Alexey Kardashevskiy wrote: > >> Some adapters (like NEC PCI USB controller) do not flush their config > >> on a sioftware reset and remember DMA config,

Re: [Qemu-devel] [PATCH] monitor: Fix warning from clang

2012-08-17 Thread Luiz Capitulino
On Fri, 17 Aug 2012 16:10:12 +0200 Markus Armbruster wrote: > Stefan Weil writes: > > > ccc-analyzer reports these warnings: > > > > monitor.c:3532:21: warning: Division by zero > > val %= val2; > > ^ > > monitor.c:3530:21: warning: Division by zero > >

Re: [Qemu-devel] [PATCH for 1.2] console: Fix warning from clang (and potential crash)

2012-08-17 Thread Jan Kiszka
On 2012-08-17 15:50, Stefan Weil wrote: > ccc-analyzer reports this warning: > > console.c:1090:29: warning: Dereference of null pointer > if (active_console->cursor_timer) { > ^ > > Function console_select allows active_console to be NULL, > but would crash wh

Re: [Qemu-devel] [PATCH] monitor: Fix warning from clang

2012-08-17 Thread Markus Armbruster
Stefan Weil writes: > ccc-analyzer reports these warnings: > > monitor.c:3532:21: warning: Division by zero > val %= val2; > ^ > monitor.c:3530:21: warning: Division by zero > val /= val2; > ^ > > Rewriting the code fixes thi

Re: [Qemu-devel] [PATCH 2/7] ppc: Make kvm_arch_put_registers() put *all* the registers

2012-08-17 Thread Alexander Graf
On 08/15/2012 06:33 AM, David Gibson wrote: At least when invoked with high enough 'level' arguments, kvm_arch_put_registers() is supposed to copy essentially all the cpu state as encoded in qemu's internal structures into the kvm state. Currently the ppc version does not do this - it never call

Re: [Qemu-devel] [PATCH] monitor: Fix warning from clang

2012-08-17 Thread Luiz Capitulino
On Fri, 17 Aug 2012 15:34:04 +0200 Stefan Weil wrote: > ccc-analyzer reports these warnings: > > monitor.c:3532:21: warning: Division by zero > val %= val2; > ^ > monitor.c:3530:21: warning: Division by zero > val /= val2; >

[Qemu-devel] [PATCH for 1.2] console: Fix warning from clang (and potential crash)

2012-08-17 Thread Stefan Weil
ccc-analyzer reports this warning: console.c:1090:29: warning: Dereference of null pointer if (active_console->cursor_timer) { ^ Function console_select allows active_console to be NULL, but would crash when accessing cursor_timer. Fix this. Signed-off-by: Ste

Re: [Qemu-devel] [PATCH 0/9] convert many more globals to "static"

2012-08-17 Thread Stefan Weil
Am 17.08.2012 15:37, schrieb Jim Meyering: Jim Meyering wrote: From: Jim Meyering Following up on discussion here, http://marc.info/?t=13375948768&r=1&w=2 here are patches to limit the scope of the remaining global variables. Most changes simply added a preceding "static". However, i

Re: [Qemu-devel] [PATCH 4/6] sheepdog: don't leak socket file descriptor upon connection failure

2012-08-17 Thread Jim Meyering
Kevin Wolf wrote: > Am 17.08.2012 15:30, schrieb Jim Meyering: >> Kevin Wolf wrote: >>> Am 16.05.2012 15:07, schrieb Jim Meyering: From: Jim Meyering Signed-off-by: Jim Meyering >>> >>> Acked-by: Kevin Wolf >> >> Hi Kevin, >> >> AFAICS, only one of these 6 patches has been applie

Re: [Qemu-devel] [PATCH 4/6] sheepdog: don't leak socket file descriptor upon connection failure

2012-08-17 Thread Kevin Wolf
Am 17.08.2012 15:30, schrieb Jim Meyering: > Kevin Wolf wrote: >> Am 16.05.2012 15:07, schrieb Jim Meyering: >>> From: Jim Meyering >>> >>> Signed-off-by: Jim Meyering >> >> Acked-by: Kevin Wolf > > Hi Kevin, > > AFAICS, only one of these 6 patches has been applied. > From what I recall (it's

Re: [Qemu-devel] [PATCH 0/9] convert many more globals to "static"

2012-08-17 Thread Jim Meyering
Jim Meyering wrote: > From: Jim Meyering > > Following up on discussion here, > > http://marc.info/?t=13375948768&r=1&w=2 > > here are patches to limit the scope of the remaining global variables. > Most changes simply added a preceding "static". However, in some cases, > I've made minor ad

Re: [Qemu-devel] [PATCHv2 2/2] envlist.c: handle strdup failure

2012-08-17 Thread Jim Meyering
Jim Meyering wrote: > From: Jim Meyering > > Without this, envlist_to_environ may silently fail to copy all > strings into the destination buffer, and both callers would leak > any env strings allocated after a failing strdup, because the > freeing code stops at the first NULL pointer. > > Signed-

[Qemu-devel] [PATCH] monitor: Fix warning from clang

2012-08-17 Thread Stefan Weil
ccc-analyzer reports these warnings: monitor.c:3532:21: warning: Division by zero val %= val2; ^ monitor.c:3530:21: warning: Division by zero val /= val2; ^ Rewriting the code fixes this (and also a style issue). Signed-off-

Re: [Qemu-devel] [PATCH 4/6] sheepdog: don't leak socket file descriptor upon connection failure

2012-08-17 Thread Jim Meyering
Kevin Wolf wrote: > Am 16.05.2012 15:07, schrieb Jim Meyering: >> From: Jim Meyering >> >> Signed-off-by: Jim Meyering > > Acked-by: Kevin Wolf Hi Kevin, AFAICS, only one of these 6 patches has been applied. >From what I recall (it's been nearly 3mo), there was good feedback and I posted at le

[Qemu-devel] [PATCH] usb: Halt ep queue en cancel pending packets on a packet error

2012-08-17 Thread Hans de Goede
For controllers which queue up more then 1 packet at a time, we must halt the ep queue, and inside the controller code cancel all pending packets on an error. There are multiple reasons for this: 1) Guests expect the controllers to halt ep queues on error, so that they get the opportunity to cance

[Qemu-devel] [PATCH] vdi: Fix warning from clang

2012-08-17 Thread Stefan Weil
ccc-analyzer reports these warnings: block/vdi.c:704:13: warning: Dereference of null pointer bmap[i] = VDI_UNALLOCATED; ^ block/vdi.c:702:13: warning: Dereference of null pointer bmap[i] = i; ^ Moving some code into the if block fixes this. It also

[Qemu-devel] [PATCH] ui: Fix spelling in comment (ressource -> resource)

2012-08-17 Thread Stefan Weil
The function is called interface_release_resource. Signed-off-by: Stefan Weil --- ui/spice-display.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ui/spice-display.c b/ui/spice-display.c index 3e8f0b3..277843c 100644 --- a/ui/spice-display.c +++ b/ui/spice-display.c @@ -

Re: [Qemu-devel] qemu-kvm-1.0.1 - unable to exit if vcpu is in infinite loop

2012-08-17 Thread Jan Kiszka
On 2012-08-06 17:11, Stefan Hajnoczi wrote: > On Thu, Jun 28, 2012 at 2:05 PM, Peter Lieven wrote: >> i debugged my initial problem further and found out that the problem happens >> to be that >> the main thread is stuck in pause_all_vcpus() on reset or quit commands in >> the monitor >> if one cp

Re: [Qemu-devel] [PATCH 1/3] usb: Halt ep queue en cancel pending packets on a packet error

2012-08-17 Thread Hans de Goede
Hi, On 08/17/2012 12:30 PM, Gerd Hoffmann wrote: Hi, Note this patch only touches the ehci and uhci controller changes, since AFAIK no other controllers actually queue up multiple transfer. If I'm wrong on this other controllers need to be updated too! xhci does it too (although it is har

[Qemu-devel] [PATCH v6 1.2] qxl: add QXL_IO_MONITORS_CONFIG_ASYNC

2012-08-17 Thread Alon Levy
Revision bumped to 4 for new IO support, enabled for spice-server >= 0.11.1. New io enabled iff spice-server >= 0.11.1 && spice-protocol >= 0.12.0. On migration reissue spice_qxl_monitors_config_async. RHBZ: 770842 Signed-off-by: Alon Levy --- Left in one defined, fixed here. configure

Re: [Qemu-devel] [PATCH] vmware_vga: Redraw only visible area

2012-08-17 Thread Marek Vasut
Dear Michael Tokarev, > On 17.08.2012 06:55, Marek Vasut wrote: > > Disallow negative value boundaries of the redraw rectangle. > > This fixes a segfault when using -vga vmware. > > > > Signed-off-by: Marek Vasut > > --- > > > > hw/vmware_vga.c |4 ++-- > > 1 file changed, 2 insertions(+),

Re: [Qemu-devel] Windows slow boot: contractor wanted

2012-08-17 Thread Richard Davies
Hi Avi, Thanks to you and several others for offering help. We will work with Avi at first, but are grateful for all the other offers of help. We have a number of other qemu-related projects which we'd be interested in getting done, and will get in touch with these names (and anyone else who comes

Re: [Qemu-devel] [PATCH] vmware_vga: Redraw only visible area

2012-08-17 Thread Michael Tokarev
On 17.08.2012 06:55, Marek Vasut wrote: > Disallow negative value boundaries of the redraw rectangle. > This fixes a segfault when using -vga vmware. > > Signed-off-by: Marek Vasut > --- > hw/vmware_vga.c |4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > NOTE: I tested this by

Re: [Qemu-devel] [RESEND][PATCH for 1.2] i82378: Remove bogus MMIO coalescing

2012-08-17 Thread Jan Kiszka
On 2012-08-17 13:13, Michael Tokarev wrote: > On 17.08.2012 14:56, Jan Kiszka wrote: >> This MMIO area is an entry gate to legacy PC ISA devices, addressed via >> PIO over there. Quite a few of the PIO ports have side effects on access >> like starting/stopping timers that must be executed properly

Re: [Qemu-devel] How does ARM VFP is emulated?

2012-08-17 Thread Peter Maydell
On 17 August 2012 12:27, 陳韋任 (Wei-Ren Chen) wrote: > On Fri, Aug 17, 2012 at 10:29:24AM +0200, Laurent Desnogues wrote: >> On Thursday, August 16, 2012, Oi Khote wrote: >> > So how exactly does VFP is being emulated. >> >> QEMU uses a library for FP computations, based on the softfloat package. >

Re: [Qemu-devel] qemu log function to print out the registers of the guest

2012-08-17 Thread Max Filippov
On Fri, Aug 17, 2012 at 3:14 PM, 陳韋任 (Wei-Ren Chen) wrote: >> > On Thu, Aug 16, 2012 at 7:49 PM, Steven wrote: >> > [...] >> >> I want to get the guest memory address in the instruction mov >> >> 0x4(%ebx) %eax, whic is 0x4(%ebx). >> >> Since %ebx is not resolved until the execution time, the co

[Qemu-devel] [PATCH v5 1.2 queue 3/4] qxl: add QXL_IO_MONITORS_CONFIG_ASYNC

2012-08-17 Thread Alon Levy
Revision bumped to 4 for new IO support, enabled for spice-server >= 0.11.1. New io enabled iff spice-server >= 0.11.1 && spice-protocol >= 0.12.0. On migration reissue spice_qxl_monitors_config_async. RHBZ: 770842 Signed-off-by: Alon Levy --- configure | 3 +++ hw/qxl.c |

[Qemu-devel] [PATCH v5 1.2 queue 2/4] qxl: disallow unknown revisions

2012-08-17 Thread Alon Levy
Signed-off-by: Alon Levy --- hw/qxl.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/hw/qxl.c b/hw/qxl.c index 6c48eb9..c978f5e 100644 --- a/hw/qxl.c +++ b/hw/qxl.c @@ -1797,10 +1797,13 @@ static int qxl_init_common(PCIQXLDevice *qxl) io_size = 16; break

[Qemu-devel] [PATCH v5 1.2 queue 4/4] configure: print spice-protocol and spice-server versions

2012-08-17 Thread Alon Levy
Signed-off-by: Alon Levy --- configure | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/configure b/configure index dbf3af6..af4f68d 100755 --- a/configure +++ b/configure @@ -2657,6 +2657,8 @@ EOF spice="yes" libs_softmmu="$libs_softmmu $spice_libs" QEMU_CFLA

[Qemu-devel] [PATCH v5 1.2 queue 1/4] qxl/update_area_io: guest_bug on invalid parameters

2012-08-17 Thread Alon Levy
Signed-off-by: Alon Levy --- hw/qxl.c | 12 1 file changed, 12 insertions(+) diff --git a/hw/qxl.c b/hw/qxl.c index c2dd3b4..6c48eb9 100644 --- a/hw/qxl.c +++ b/hw/qxl.c @@ -1385,6 +1385,18 @@ async_common: QXLCookie *cookie = NULL; QXLRect update = d->ram->update_

[Qemu-devel] [PATCH v5 1.2 queue 0/4] QXL_IO_MONITORS_CONFIG_ASYNC + misc

2012-08-17 Thread Alon Levy
Hi Gerd, Rebased on the lastest, redid ifdefs to use a single line, QXL_HAS_IO_MONITORS_CONFIG_ASYNC is 0 by default, 1 if spice-protocol is new enough. Also available at git://people.freedesktop.org/~alon/qemu qxl/pull Alon Levy (4): qxl/update_area_io: guest_bug on invalid parameter

Re: [Qemu-devel] How does ARM VFP is emulated?

2012-08-17 Thread Wei-Ren Chen
On Fri, Aug 17, 2012 at 10:29:24AM +0200, Laurent Desnogues wrote: > On Thursday, August 16, 2012, Oi Khote wrote: > > So how exactly does VFP is being emulated. > > QEMU uses a library for FP computations, based on the softfloat package. I thought QEMU emulates VFP itself, something like flo

Re: [Qemu-devel] [PATCH v2 2/2] vmdk: Read footer for streamOptimized images

2012-08-17 Thread Jeff Cody
On 08/16/2012 05:50 AM, Kevin Wolf wrote: > The footer takes precedence over the header when it exists. It contains > the real grain directory offset that is missing in the header. Without > this patch, streamOptimized images with a footer cannot be read. > > Signed-off-by: Kevin Wolf > --- > v2:

Re: [Qemu-devel] qemu log function to print out the registers of the guest

2012-08-17 Thread Wei-Ren Chen
> > On Thu, Aug 16, 2012 at 7:49 PM, Steven wrote: > > [...] > >> I want to get the guest memory address in the instruction mov > >> 0x4(%ebx) %eax, whic is 0x4(%ebx). > >> Since %ebx is not resolved until the execution time, the code in > >> softmmu_header.h does not generate any hit or miss inf

Re: [Qemu-devel] [RESEND][PATCH for 1.2] i82378: Remove bogus MMIO coalescing

2012-08-17 Thread Michael Tokarev
On 17.08.2012 14:56, Jan Kiszka wrote: > This MMIO area is an entry gate to legacy PC ISA devices, addressed via > PIO over there. Quite a few of the PIO ports have side effects on access > like starting/stopping timers that must be executed properly ordered > /wrt the CPU. So we have to remove the

[Qemu-devel] [RESEND][PATCH for 1.2] i82378: Remove bogus MMIO coalescing

2012-08-17 Thread Jan Kiszka
This MMIO area is an entry gate to legacy PC ISA devices, addressed via PIO over there. Quite a few of the PIO ports have side effects on access like starting/stopping timers that must be executed properly ordered /wrt the CPU. So we have to remove the coalescing mark. Acked-by: Hervé Poussineau

Re: [Qemu-devel] [PATCH v3 2/7] memory: Flush coalesced MMIO on selected region access

2012-08-17 Thread Jan Kiszka
On 2012-07-10 12:41, Jan Kiszka wrote: > On 2012-07-02 11:07, Avi Kivity wrote: >> On 06/29/2012 07:37 PM, Jan Kiszka wrote: >>> Instead of flushing pending coalesced MMIO requests on every vmexit, >>> this provides a mechanism to selectively flush when memory regions >>> related to the coalesced o

[Qemu-devel] [Bug 1037675] Re: Guest Kernel Panic if using "-cpu host" in qemu-kvm 1.1.1

2012-08-17 Thread Till Schäfer
sorry for the not very usefull information i provides above. i will try to reproduce the the failure with a vailla kernel (host) in a few days. currently the server is in use and cannot be restarted. the kernel panic was in the guest (thought this is clear by "my vm panic"). If it is reproducable

[Qemu-devel] [Bug 1037606] AlsaInfo.txt

2012-08-17 Thread Michal Suchanek
apport information ** Attachment added: "AlsaInfo.txt" https://bugs.launchpad.net/bugs/1037606/+attachment/3265233/+files/AlsaInfo.txt -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1037606 Title

[Qemu-devel] [Bug 1037606] BootDmesg.txt

2012-08-17 Thread Michal Suchanek
apport information ** Attachment added: "BootDmesg.txt" https://bugs.launchpad.net/bugs/1037606/+attachment/3265234/+files/BootDmesg.txt -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1037606 Tit

Re: [Qemu-devel] [PATCH 1/3] usb: Halt ep queue en cancel pending packets on a packet error

2012-08-17 Thread Gerd Hoffmann
Hi, > Note this patch only touches the ehci and uhci controller changes, since AFAIK > no other controllers actually queue up multiple transfer. If I'm wrong on this > other controllers need to be updated too! xhci does it too (although it is hard to test as xhci can happily submit 256k transfe

Re: [Qemu-devel] qemu log function to print out the registers of the guest

2012-08-17 Thread Wei-Ren Chen
> To verify what is translation time and what is the run time, I log the > register information before disassembling each guest code. I copied > some results from the log file, which is generated at run time of a > guest machine. > > EAX= EBX=6ffc > IN: > 0x

[Qemu-devel] [Bug 1037606] Lspci.txt

2012-08-17 Thread Michal Suchanek
apport information ** Attachment added: "Lspci.txt" https://bugs.launchpad.net/bugs/1037606/+attachment/3265236/+files/Lspci.txt -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1037606 Title: vmw

[Qemu-devel] [Bug 1037606] CurrentDmesg.txt

2012-08-17 Thread Michal Suchanek
apport information ** Attachment added: "CurrentDmesg.txt" https://bugs.launchpad.net/bugs/1037606/+attachment/3265235/+files/CurrentDmesg.txt -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/103760

[Qemu-devel] [Bug 1037606] UdevLog.txt

2012-08-17 Thread Michal Suchanek
apport information ** Attachment added: "UdevLog.txt" https://bugs.launchpad.net/bugs/1037606/+attachment/3265241/+files/UdevLog.txt -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1037606 Title:

Re: [Qemu-devel] [PATCH] block: handle filenames with colons better

2012-08-17 Thread Kevin Wolf
Am 17.08.2012 12:05, schrieb Iustin Pop: > On Fri, Aug 17, 2012 at 09:56:35AM +0200, Kevin Wolf wrote: >> Am 17.08.2012 09:15, schrieb Iustin Pop: >>> On Thu, Aug 16, 2012 at 11:24:11PM +0400, Michael Tokarev wrote: On 16.08.2012 18:58, Iustin Pop wrote: > Commit 947995c (block: protect pa

[Qemu-devel] [Bug 1037606] Re: vmwgfx does not work with kvm vmware vga

2012-08-17 Thread Michal Suchanek
apport information ** Tags added: apport-collected quantal ** Description changed: vmwgfx driver fails to initialize inside kvm. - tried: kvm -m 2048 -vga vmware -cdrom RebeccaBlackLinux.iso (Ubuntu - based, any Ubuntu live CD would do) + tried: kvm -m 2048 -vga vmware -cdrom RebeccaBlackLi

Re: [Qemu-devel] [PATCH] block: handle filenames with colons better

2012-08-17 Thread Iustin Pop
On Fri, Aug 17, 2012 at 09:56:35AM +0200, Kevin Wolf wrote: > Am 17.08.2012 09:15, schrieb Iustin Pop: > > On Thu, Aug 16, 2012 at 11:24:11PM +0400, Michael Tokarev wrote: > >> On 16.08.2012 18:58, Iustin Pop wrote: > >>> Commit 947995c (block: protect path_has_protocol from filenames with > >>> co

[Qemu-devel] [Bug 1037606] ProcInterrupts.txt

2012-08-17 Thread Michal Suchanek
apport information ** Attachment added: "ProcInterrupts.txt" https://bugs.launchpad.net/bugs/1037606/+attachment/3265238/+files/ProcInterrupts.txt -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/10

[Qemu-devel] [Bug 1037606] UdevDb.txt

2012-08-17 Thread Michal Suchanek
apport information ** Attachment added: "UdevDb.txt" https://bugs.launchpad.net/bugs/1037606/+attachment/3265240/+files/UdevDb.txt -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1037606 Title: v

[Qemu-devel] [Bug 1037606] ProcCpuinfo.txt

2012-08-17 Thread Michal Suchanek
apport information ** Attachment added: "ProcCpuinfo.txt" https://bugs.launchpad.net/bugs/1037606/+attachment/3265237/+files/ProcCpuinfo.txt -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1037606

[Qemu-devel] [Bug 1037606] WifiSyslog.txt

2012-08-17 Thread Michal Suchanek
apport information ** Attachment added: "WifiSyslog.txt" https://bugs.launchpad.net/bugs/1037606/+attachment/3265242/+files/WifiSyslog.txt ** Description changed: vmwgfx driver fails to initialize inside kvm. - tried: kvm -m 2048 -vga vmware -cdrom RebeccaBlackLinux.iso (Ubuntu based,

[Qemu-devel] [Bug 1037606] ProcModules.txt

2012-08-17 Thread Michal Suchanek
apport information ** Attachment added: "ProcModules.txt" https://bugs.launchpad.net/bugs/1037606/+attachment/3265239/+files/ProcModules.txt -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1037606

Re: [Qemu-devel] [PATCH 1/2] extract file_load() function from rom_add_file() for reusing

2012-08-17 Thread Yin Olivia-R63875
Hi Avi, Thanks. Exactly the second patch is more important which saves the QEMU memory. Best Regards, Olivia > -Original Message- > From: Avi Kivity [mailto:a...@redhat.com] > Sent: Thursday, August 16, 2012 7:23 PM > To: Yin Olivia-R63875 > Cc: qemu-...@nongnu.org; qemu-devel@nongnu.org

Re: [Qemu-devel] [PATCH 1/2] extract file_load() function from rom_add_file() for reusing

2012-08-17 Thread Yin Olivia-R63875
Hi Peter, Thanks for the reminder. I'll update the second patch to use g_file_get_contents(). Best Regards, Olivia > -Original Message- > From: Peter Maydell [mailto:peter.mayd...@linaro.org] > Sent: Thursday, August 16, 2012 7:36 PM > To: Yin Olivia-R63875 > Cc: qemu-...@nongnu.org; qe

[Qemu-devel] [PATCH 2/3] usb: controllers do not need to check for babble themselves

2012-08-17 Thread Hans de Goede
If an (emulated) usb-device tries to write more data to a packet then its iov len, this will trigger an assert in usb_packet_copy(), and if a driver somehow circumvents that check and writes more data to the iov then there is space, we have a much bigger problem then not correctly reporting babble

[Qemu-devel] [PATCH 3/3] ehci: simplify ehci_state_executing

2012-08-17 Thread Hans de Goede
ehci_state_executing does not need to check for p->usb_status == USB_RET_ASYNC or USB_RET_PROCERR, since ehci_execute_complete already does a similar check and will trigger an assert if either value is encountered. USB_RET_ASYNC should never be the packet status when execute_complete runs for obvi

[Qemu-devel] [PATCH 1/3] usb: Halt ep queue en cancel pending packets on a packet error

2012-08-17 Thread Hans de Goede
For controllers which queue up more then 1 packet at a time, we must halt the ep queue, and inside the controller code cancel all pending packets on an error. There are multiple reasons for this: 1) Guests expect the controllers to halt ep queues on error, so that they get the opportunity to cance

Re: [Qemu-devel] [PATCH V3 2/2] qemu-img: Add json output option to the info command.

2012-08-17 Thread Kevin Wolf
Am 15.08.2012 20:48, schrieb Benoît Canet: > This additionnal --machine=json option make qemu-img info output on > stdout a JSON formated representation of the image informations. > > --machine=json was choosen instead of --format=json because the > info command already have a -f parameter. Which

  1   2   >