[Qemu-devel] [Bug 604872] Re: qemu-system-arm segfaults emulating versatile machine after running debootstrap --second-stage inside vm

2011-02-16 Thread Loïc Minier
Ok; the commit you point at is in the current qemu-linaro package Fix released in Ubuntu is when we upload a fixed source package to the Ubuntu development release The usual way to close Ubuntu bugs fixed with an upload is via the debian/changelog, when Launchpad processes the .changes file with

[Qemu-devel] [Bug 604872] Re: qemu-system-arm segfaults emulating versatile machine after running debootstrap --second-stage inside vm

2011-02-15 Thread Loïc Minier
Hmm Ubuntu has 2011.02, but I think you had asked me to flip this bug back to Triaged; I'm confused now, is this fixed in Ubuntu? I didn't find http://patchwork.ozlabs.org/patch/77888/ applied in qemu- linaro 0.13.50-2011.02-0-0ubuntu1 which is based of 2011.02. -- You received this bug notifica

[Qemu-devel] [Bug 604872] Re: qemu-system-arm segfaults emulating versatile machine after running debootstrap --second-stage inside vm

2011-02-11 Thread Loïc Minier
** Changed in: qemu-linaro (Ubuntu) Status: Fix Released => Triaged -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/604872 Title: qemu-system-arm segfaults emulating versatile machine after ru

[Qemu-devel] [Bug 604872] Re: qemu-system-arm segfaults emulating versatile machine after running debootstrap --second-stage inside vm

2011-01-07 Thread Loïc Minier
qemu-kvm is what's currently in the Ubuntu archive; I'm sure Peter will also arrange for the ubuntu-qemu-omap branch to get these fixes once they are in suitable shape. In any case, these fixes are also going upstream and will eventually bubble up to derived trees -- You received this bug notifi

[Qemu-devel] [PATCH] Avoid exit in trap as it breaks with some shells

2010-09-25 Thread Loïc Minier
Don't call exit in the trap handler as it causes the return code to be zero with some buggy shells (dash and pdksh at least) and is useless here anyway. --- configure |4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/configure b/configure index 3bfc5e9..9d3acfc 100755 ---

Re: [Qemu-devel] [PATCH] Don't exit with zero in the trap handler

2010-09-25 Thread Loïc Minier
could you please pull the patch in QEMU? Without "exit" in the trap handler, the correct exit code will be returned. Thanks, -- Loïc Minier

Re: [Qemu-devel] [PATCH] Don't exit with zero in the trap handler

2010-09-25 Thread Loïc Minier
in a trap handler isn't needed here, I recommend including this as a bug fix. -- Loïc Minier

[Qemu-devel] [PATCH] Don't exit with zero in the trap handler

2010-09-24 Thread Loïc Minier
When configure runs "exit 1", the trap handler is run to cleanup any files created by configure, but this trap handler itself calls "exit" with no argument (which means zero exit code): [...] + echo Error: zlib check failed Error: zlib check failed + echo Make sure to have the zlib libs and headers

Re: [Qemu-devel] [PATCH] fat_chksum(): fix access above array bounds

2010-08-23 Thread Loïc Minier
t one accidental build on a modern gcc-4.4 tree because of the timing of my copy of the packages. I guess I should build them more regularly under a recent gcc. -- Loïc Minier

Re: [Qemu-devel] [PATCH 1/1] Added PMCR support

2010-08-22 Thread Loïc Minier
opaque, int version_id) > env->cp15.c6_data = qemu_get_be32(f); > env->cp15.c9_insn = qemu_get_be32(f); > env->cp15.c9_data = qemu_get_be32(f); > +env->cp15.c9_pmcr = qemu_get_be32(f); > env->cp15.c13_fcse = qemu_get_be32(f); > env->cp15.c13_context = qemu_get_be32(f); > env->cp15.c13_tls1 = qemu_get_be32(f); > -- > 1.7.0.4 > > -- Loïc Minier

[Qemu-devel] [PATCH] fat_chksum(): fix access above array bounds

2010-08-21 Thread Loïc Minier
Signed-off-by: Loïc Minier --- block/vvfat.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/block/vvfat.c b/block/vvfat.c index 6d61c2e..365332a 100644 --- a/block/vvfat.c +++ b/block/vvfat.c @@ -512,7 +512,7 @@ static inline uint8_t fat_chksum(const direntry_t* entry

Re: [Qemu-devel] qemu cp15 access

2010-07-28 Thread Loïc Minier
d 1280086076-20649-1-git-send-email-loic.min...@linaro.org and thread http://article.gmane.org/gmane.comp.emulators.qemu/77092 -- Loïc Minier

Re: [Qemu-devel] [PATCH] Ignore writes of perf reg (cp15 with crm == 12)

2010-07-28 Thread Loïc Minier
I found out Matt Waddel has written a better looking patch, but I didn't test it; reviews welcome -- attached -- Loïc Minier --- Begin Message --- diff --git a/target-arm/cpu.h b/target-arm/cpu.h index 7440163..b5d8a6c 100644 --- a/target-arm/cpu.h +++ b/target-arm/cpu.h @@ -130,6 +

[Qemu-devel] [PATCH] Ignore writes of perf reg (cp15 with crm == 12)

2010-07-25 Thread Loïc Minier
On ARMv7, ignore writes to cp15 with crm == 12; these are to setup perf counters which we don't have. --- target-arm/helper.c |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/target-arm/helper.c b/target-arm/helper.c index 2dd64d9..865829f 100644 --- a/target-arm/helper.c

[Qemu-devel] [Bug 584131] Re: some guests hangs after migration (qemu-kvm-0.12)

2010-05-22 Thread Loïc Minier
** Bug watch added: Debian Bug tracker #580649 http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=580649 ** Also affects: debian via http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=580649 Importance: Unknown Status: Unknown -- some guests hangs after migration (qemu-kvm-0.12) http

[Qemu-devel] [Bug 584121] Re: migration always fails on 32bit qemu-kvm-0.12+ (sigsegv)

2010-05-22 Thread Loïc Minier
Sorry, I failed to realize that qemu-kvm uses the qemu project in launchpad to track bugs; so this is the right place to file these. Thanks! -- migration always fails on 32bit qemu-kvm-0.12+ (sigsegv) https://bugs.launchpad.net/bugs/584121 You received this bug notification because you are a mem

[Qemu-devel] [Bug 584155] Re: support horisontal mouse wheel

2010-05-22 Thread Loïc Minier
** Bug watch added: Debian Bug tracker #579968 http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=579968 ** Also affects: qemu-kvm (Debian) via http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=579968 Importance: Unknown Status: Unknown -- support horisontal mouse wheel https://bugs

[Qemu-devel] [Bug 584146] Re: Virtual fat breaks with -snapshot

2010-05-22 Thread Loïc Minier
** Bug watch added: Debian Bug tracker #504049 http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=504049 ** Also affects: qemu-kvm (Debian) via http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=504049 Importance: Unknown Status: Unknown -- Virtual fat breaks with -snapshot https://b

[Qemu-devel] [Bug 584153] Re: no useful error message when tap device open fails

2010-05-22 Thread Loïc Minier
** Bug watch added: Debian Bug tracker #578154 http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=578154 ** Also affects: qemu-kvm (Debian) via http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=578154 Importance: Unknown Status: Unknown -- no useful error message when tap device ope

[Qemu-devel] [Bug 584139] Re: keymapping error for usb keyboard (windows/menu keys)

2010-05-22 Thread Loïc Minier
** Bug watch added: Debian Bug tracker #578846 http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=578846 ** Also affects: qemu-kvm (Debian) via http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=578846 Importance: Unknown Status: Unknown -- keymapping error for usb keyboard (windows/

[Qemu-devel] [Bug 584143] Re: qemu fails to set hdd serial number

2010-05-22 Thread Loïc Minier
Patch still applies on top of qemu-kvm.git and qemu.git ** Bug watch added: Debian Bug tracker #573439 http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=573439 ** Also affects: qemu-kvm (Debian) via http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=573439 Importance: Unknown Status:

[Qemu-devel] [Bug 584121] Re: migration always fails on 32bit qemu-kvm-0.12+ (sigsegv)

2010-05-22 Thread Loïc Minier
Hi You're filing bugs against the QEMU project, which is used to track upstream issues (issues in the tarballs released by the QEMU project, or in their git tree). I think you intended to file these against the Ubuntu package of qemu, "qemu-kvm". AFAIK, QEMU doesn't have any upstream bug tracker

Re: [Qemu-devel] [PATCH 1/2] Detect and use GCC atomic builtins for locking

2010-02-22 Thread Loïc Minier
On Sat, Feb 20, 2010, malc wrote: > No, i'm against using locking GCC builtins for all the other targets (well > PPC) Do you have benchmarks with/without GCC atomic builtins? -- Loïc Minier

Re: [Qemu-devel] [PATCH 1/2] Detect and use GCC atomic builtins for locking

2010-02-20 Thread Loïc Minier
ld ask for new ones without the drawbacks if any. Do you have another option to implement locking on thumb-2? -- Loïc Minier

Re: [Qemu-devel] [PATCH 1/2] Detect and use GCC atomic builtins for locking

2010-02-20 Thread Loïc Minier
es purely political and don't relate to qemu. Which are your concerns with atomic builtins and are these in the above list? -- Loïc Minier

[Qemu-devel] [PATCH 2/2] Make spinlock_t types volatile

2010-02-19 Thread Loïc Minier
--- qemu-lock.h |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/qemu-lock.h b/qemu-lock.h index 5c8eb34..23e3442 100644 --- a/qemu-lock.h +++ b/qemu-lock.h @@ -34,7 +34,7 @@ #else #ifdef CONFIG_GCC_ATOMIC_BUILTINS -typedef int spinlock_t; +typedef volatile int spi

[Qemu-devel] [PATCH 1/2] Detect and use GCC atomic builtins for locking

2010-02-19 Thread Loïc Minier
--- configure | 17 + qemu-lock.h | 13 + 2 files changed, 30 insertions(+), 0 deletions(-) diff --git a/configure b/configure index 0a84b0e..b33f045 100755 --- a/configure +++ b/configure @@ -1823,6 +1823,20 @@ if compile_prog "" ""; then fi

Re: [Qemu-devel] [PATCH] fix the static compilation for sdl

2010-02-08 Thread Loïc Minier
tatic" = "yes" ; then > - sdl_libs=`sdl-config --static-libs 2>/dev/null` >if test $? = 0 && echo $sdl_libs | grep -- -laa > /dev/null; then > sdl_libs="$sdl_libs `aalib-config --static-libs >2 /dev/null`" > s

[Qemu-devel] [PATCH] Add -static earlier to LDFLAGS for compile_prog()

2010-01-31 Thread Loïc Minier
Add -static to LDFLAGS earlier as to run the compile_prog() tests with this flags, this will avoid turning on features for which a shared library is available but not a static one. Signed-off-by: Loïc Minier --- configure |5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) diff

[Qemu-devel] [PATCH] Fix missing symbols in .rel/.rela.plt sections

2010-01-31 Thread Loïc Minier
libc.a. This fixes the static build under i386. Apply similar logic to rela.plt/.iplt and __rela_iplt/_plt_start/_end to fix the static build under amd64. Signed-off-by: Loïc Minier --- i386.ld | 16 ++-- x86_64.ld | 16 ++-- 2 files changed, 28 insertions(+), 4

Re: [Qemu-devel] Re: Stop using "which" in ./configure

2010-01-28 Thread Loïc Minier
ed that /bin/sh on MilaX doesn't support "if ! foo", so I replaced these with "if foo; then :; else ...; fi" constructs. Please find an attached third patch. -- Loïc Minier >From 6fef73a3b890ec736cdfa1ff817230c8e6d760e3 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?L

Re: [Qemu-devel] Re: Stop using "which" in ./configure

2010-01-27 Thread Loïc Minier
On Tue, Jan 26, 2010, Blue Swirl wrote: > The patches didn't apply. Also please send only one patch per message, > git am can't handle multiple patches. They applied fine here, perhaps you didn't apply the sdl-config patch first? I rebased them and resent them. -- Loïc Minier

[Qemu-devel] [PATCH 3/3] Solaris: test for presence of commands with has()

2010-01-27 Thread Loïc Minier
--- configure |8 +++- 1 files changed, 3 insertions(+), 5 deletions(-) diff --git a/configure b/configure index 6bdd2b7..1d0915a 100755 --- a/configure +++ b/configure @@ -803,21 +803,19 @@ fi # Solaris specific configure tool chain decisions # if test "$solaris" = "yes" ; then - sol

[Qemu-devel] [PATCH 2/3] Add and use has() and path_of() funcs

2010-01-27 Thread Loïc Minier
Add has() and path_of() funcs and use them across configure; has() will test whether a command or builtin is available; path_of() will search the PATH for executables and return the full pathname if found. --- configure | 52 +++- 1 files changed,

[Qemu-devel] [PATCH 1/3] Check for sdl-config before calling it

2010-01-27 Thread Loïc Minier
Check whether sdl-config is available before calling it, otherwise ./configure triggers a warning: ./configure: 957: sdl-config: not found If neither the .pc file not sdl-config are present, disable SDL support. Signed-off-by: Loïc Minier --- configure |7 ++- 1 files changed, 6

Re: [Qemu-devel] Re: Stop using "which" in ./configure

2010-01-26 Thread Loïc Minier
ement over what's in ./configure. Thanks -- Loïc Minier

Re: [Qemu-devel] Re: Stop using "which" in ./configure

2010-01-21 Thread Loïc Minier
On Wed, Jan 20, 2010, Måns Rullgård wrote: > If you make that IFS=${local_ifs:-$(printf ' \t\n')} it should be safe. > Likewise if you set the value first. Ok; see attached patches -- Loïc Minier >From cccdcaeacc2214390c0c6c198ed875ac59d10669 Mon Sep 17 00:00:00 2001 From:

Re: [Qemu-devel] Re: Stop using "which" in ./configure

2010-01-20 Thread Loïc Minier
rhaps you missed the backup/restore bits: +local_ifs="$IFS" [...] +IFS=: [...] +IFS="$local_ifs" +return 0 [...] +IFS="$local_ifs" +return 1 Do you have an example of how that breaks if IFS isn't ever set in ./configure at all? -- Loïc Minier

Re: [Qemu-devel] Re: Stop using "which" in ./configure

2010-01-20 Thread Loïc Minier
On Wed, Jan 20, 2010, Loïc Minier wrote: > I'm attaching a new patch which changes the tests a bit; I would prefer > if someone with access to a Solaris build environment would do this > though. Sorry, there was a typo in the new patch, a test was reversed; updated patch attac

Re: [Qemu-devel] Re: Stop using "which" in ./configure

2010-01-20 Thread Loïc Minier
t;; then already pointed out, but -a is a feature of "test" and incorrect here; you want && > - if test -x "`which texi2html 2>/dev/null`" -a \ > - -x "`which pod2man 2>/dev/null`" ; then > + if prog_exist "texi2html" -a \ > + prog_exist "pod2man" ; then ditto Perhaps we should avoid work duplication though? -- Loïc Minier

Re: [Qemu-devel] Re: Stop using "which" in ./configure

2010-01-20 Thread Loïc Minier
On Wed, Jan 20, 2010, Paolo Bonzini wrote: > On 01/20/2010 12:37 PM, Loïc Minier wrote: > >+# not found > >+IFS="$local_ifs" > If you do this, you should set IFS to space-tab-lf at the beginning of > the script, like this: > > IFS=" "&qu

Re: [Qemu-devel] Re: Stop using "which" in ./configure

2010-01-20 Thread Loïc Minier
really important? Doesn't look like > it to me. I'm attaching a new patch which changes the tests a bit; I would prefer if someone with access to a Solaris build environment would do this though. -- Loïc Minier >From 27f151ef7be19fcffbbf9c2d3c6ee50750be854d Mon Sep 17 00:00:0

Re: [Qemu-devel] Stop using "which" in ./configure

2010-01-20 Thread Loïc Minier
On Tue, Jan 19, 2010, Stefan Weil wrote: > I did not test the whole patch, but I think this would be better: > +type "$local_command" >/dev/null 2>&1 Attaching an updated patch Thanks -- Loïc Minier >From 1c0b63fb9fc735a6d367a65a6ed1b998942fb6a4 Mon Sep 17

Re: [Qemu-devel] [PATCH] Fix missing symbols in .rela.plt sections

2010-01-19 Thread Loïc Minier
On Sun, Jan 17, 2010, Loïc Minier wrote: > Static builds of qemu on x86-64 (and probably i386) fail After actually checking in an i386 Ubuntu lucid chroot, I found out that ld uses .rel.plt and .rel.iplt instead of .rela.plt and .rela.iplt. I've applied the same fixes to the two .ld

Re: [Qemu-devel] Stop using "which" in ./configure

2010-01-19 Thread Loïc Minier
On Tue, Jan 19, 2010, Loïc Minier wrote: > Following the thread on the sdl-config patch, please find attached a > patch to add a couple of portable shell functions which allow testing > whehter a command/builtin is available and to find the full pathname of > an executable in the

Re: [Qemu-devel] Stop using "which" in ./configure

2010-01-19 Thread Loïc Minier
On Tue, Jan 19, 2010, Laurent Vivier wrote: > Why don't you use "type -P" for "path_of" ?" I don't think that's portable enough: $ sh -c 'type -P ls' -P: not found ls is /bin/ls -- Loïc Minier

[Qemu-devel] Stop using "which" in ./configure

2010-01-19 Thread Loïc Minier
h" in ./configure. (This should be applied on top of the sdl-config patch.) Thanks, -- Loïc Minier >From 815ca36e9b3e2c56040d7fc7709c95d600ed6496 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Lo=C3=AFc=20Minier?= Date: Tue, 19 Jan 2010 11:05:00 +0100 Subject: [PATCH] Add and use has(

Re: [Qemu-devel] [PATCH] Check for sdl-config before calling it

2010-01-17 Thread Loïc Minier
s wrong: configure should abort with > an error message (feature_not_found). Ack; how about the attached one instead? -- Loïc Minier >From eb4ba45b0d71d13cb89f6362443ffbe50b65eba0 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Lo=C3=AFc=20Minier?= Date: Sun, 17 Jan 2010 13:42:04 +0100 Subject

[Qemu-devel] [PATCH] Check for sdl-config before calling it

2010-01-17 Thread Loïc Minier
Hi On systems were sdl-config isn't installed, ./configure triggers this warning: ./configure: 957: sdl-config: not found The attached patch fixes the warning for me. Thanks, -- Loïc Minier >From 94876939db7f46cf8d920e289d0d4f929d3b7df1 Mon Sep 17 00:00:00 2001 From:

[Qemu-devel] [PATCH] Add -static earlier to LDFLAGS for compile_prog()

2010-01-17 Thread Loïc Minier
ftmmu,x86_64-softmmu,arm-linux-user,i386-linux-user,x86_64-linux-user and again without --static. Thanks, -- Loïc Minier >From 3df135de4babc35849578cde901ef9ffb04a2ab3 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Lo=C3=AFc=20Minier?= Date: Sun, 17 Jan 2010 12:43:21 +0100 Subject: [PATCH] Add

[Qemu-devel] [PATCH] Fix missing symbols in .rela.plt sections

2010-01-17 Thread Loïc Minier
pts weren't copying the .rela.iplt section at all, so I included this section and the __rela_iplt_start and __rela_iplt_end arount it. Tested by building qemu in static and shared mode for the arm-softmmu,i386-softmmu,x86_64-softmmu,arm-linux-user,i386-linux-user,x86_64-linux-user target-lis

Re: [Qemu-devel] [PATCH] linux-user: adapt uname machine to emulated CPU

2010-01-17 Thread Loïc Minier
version. ] Thanks, -- Loïc Minier >From c4396cafac0041cfa2fed686f935e9df3636e05b Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Lo=C3=AFc=20Minier?= Date: Tue, 29 Dec 2009 22:06:13 +0100 Subject: [PATCH] linux-user: adapt uname machine to emulated CPU --- Makefile.target|2 +- lin

[Qemu-devel] [PATCH] linux-user: adapt uname machine to emulated CPU

2009-12-29 Thread Loïc Minier
/null +++ b/linux-user/cpu-uname.c @@ -0,0 +1,72 @@ +/* + * cpu to uname machine name map + * + * Copyright (c) 2009 Loïc Minier + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free