[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

[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

[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

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

2010-09-25 Thread Loïc Minier
including this as a bug fix. -- Loïc Minier

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

2010-09-25 Thread Loïc Minier
code will be returned. Thanks, -- Loïc Minier

[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

[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
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
-- 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 loic.min...@linaro.org --- 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

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 ---BeginMessage--- 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 +130,7

Re: [Qemu-devel] qemu cp15 access

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

[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 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,

[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

[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

[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

[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

[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)

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
are your concerns with atomic builtins and are these in the above list? -- Loïc Minier

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

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

[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

[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

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

2010-02-08 Thread Loïc Minier
` sdl_cflags=$sdl_cflags `aalib-config --cflags 2 /dev/null` Signed-off-by: Loïc Minier l...@dooz.org Ah right, the tests were being run on a shared libsdl no matter actual linking would happen against the static or shared libsdl; thanks for fixing the ordering. -- Loïc Minier

[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 l...@dooz.org --- i386.ld | 16 ++-- x86_64.ld | 16 ++-- 2 files changed, 28

[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 l...@dooz.org --- configure |5 +++-- 1 files changed, 3 insertions(+), 2 deletions

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

2010-01-28 Thread Loïc Minier
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?Lo=C3=AFc=20Minier?= l...@dooz.org Date: Wed, 20 Jan 2010 12:35:54

[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 l...@dooz.org --- configure |7 ++- 1 files

[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 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 -

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

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

2010-01-26 Thread Loïc Minier
-- 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: =?UTF-8?q?Lo=C3=AFc

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 21 Attaching an updated patch Thanks -- Loïc Minier From 1c0b63fb9fc735a6d367a65a6ed1b998942fb6a4 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Lo=C3

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

2010-01-20 Thread Loïc Minier
? 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:00 2001 From: =?UTF-8?q?Lo=C3=AFc=20Minier

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= Are you saying that I can't backup/restore IFS without

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

2010-01-20 Thread Loïc Minier
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, 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 attached. -- Loïc

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

2010-01-20 Thread Loïc Minier
=: [...] +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

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

2010-01-19 Thread Loïc Minier
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?= l...@dooz.org Date: Tue, 19 Jan 2010 11:05:00 +0100 Subject: [PATCH] Add and use has

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

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 PATH. This also

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 scripts

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

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

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

2010-01-17 Thread Loïc Minier
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-list; I also ran qemu for almost all combinations. Thanks, -- Loïc Minier

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

2010-01-17 Thread Loïc Minier
,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?= l...@dooz.org Date: Sun, 17 Jan 2010 12:43:21 +0100 Subject: [PATCH] Add

[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: =?UTF-8?q

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

2010-01-17 Thread Loïc Minier
(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?= l...@dooz.org Date: Sun, 17 Jan 2010 13:42:04 +0100 Subject: [PATCH] Check for sdl-config before calling it MIME-Version: 1.0

[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