This should be fixed with the appended patch - please test.
** Changed in: qemu
Assignee: (unassigned) = Stefan Weil (ubuntu-weilnetz)
** Patch added: Always set locale C
https://bugs.launchpad.net/qemu/+bug/962880/+attachment/2924541/+files/0001-Makefile-Set-default-locale-C.patch
--
mem is an uint64_t value, so %lx was wrong.
Signed-off-by: Stefan Weil s...@weilnetz.de
---
trace-events |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/trace-events b/trace-events
index 70f059d..db2cd39 100644
--- a/trace-events
+++ b/trace-events
@@ -726,7 +726,7 @@
Dear All,
Now I want to moniter memory access on QEMU (guest virtual or guest
physical).
I found that QEMU will translate arm instructions into TCG instructions,
and the TCG instructions will be translated into X86 instructions.
The function tcg_out_op(tcg/i386/tcg_target.c) will translate the
On Wed, Mar 21, 2012 at 22:46, Michael Roth mdr...@linux.vnet.ibm.com wrote:
On Mon, Mar 19, 2012 at 11:57:44PM +0100, Juan Quintela wrote:
Signed-off-by: Juan Quintela quint...@redhat.com
---
hw/sun4u.c | 20 --
target-sparc/cpu.h | 9 +-
target-sparc/machine.c | 474
On Fri, Mar 23, 2012 at 15:22, Richard Henderson r...@twiddle.net wrote:
Patch 1 has been seen before but not picked up.
Patches 2-8 convert the target away from areg0.
Cool. Could you fix style issues while moving code around?
I used a three phase patching approach in Sparc: 1) fix style 2)
On Fri, Mar 23, 2012 at 15:25, Richard Henderson r...@twiddle.net wrote:
On 03/19/12 14:57, Blue Swirl wrote:
Add an explicit CPUARMState parameter instead of relying on AREG0
and move load and store helpers to helper.c. Remove AREG0 swapping in
tlb_fill(). Remove now empty op_helper.c.
On 23 March 2012 20:05, Stefan Weil s...@weilnetz.de wrote:
+# Set default locale for commands like tr and others.
+LANG=C
+LC_ALL=C
+LC_CTYPE=C
You don't export these, which leads to an obscure corner case:
if the user has no LANG variable set in their environment when
they run make then we
On Sat, Mar 17, 2012 at 08:04, Lee Essen lee.es...@nowonline.co.uk wrote:
(third email attempt, apologies if you get duplicates)
The patch does not apply:
git am:
Applying: Basic Illumos support
fatal: corrupt patch at line 19
Patch failed at 0001 Basic Illumos support
patch:
patching file
On Fri, Mar 23, 2012 at 17:27, Andreas Färber afaer...@suse.de wrote:
Am 14.03.2012 21:16, schrieb Blue Swirl:
On Wed, Mar 14, 2012 at 17:53, Andreas Färber afaer...@suse.de wrote:
diff --git a/target-sparc/cpu-qom.h b/target-sparc/cpu-qom.h
new file mode 100644
index 000..15dcf84
---
On Mon, Mar 19, 2012 at 21:12, Stefan Weil s...@weilnetz.de wrote:
QEMU host addresses must use uintptr_t to be portable for hosts with
an unusual size of long (w64).
tb_jmp_offset is an uint16_t value, therefore the local variable offset
in function tb_set_jmp_target was changed from
On Fri, Mar 09, 2012 at 04:07:43PM +, Stefan Hajnoczi wrote:
Paolo, your discard improvements in QEMU add FALLOC_FL_PUNCH_HOLE
support. XFS supports this fallocate() flag in current kernels,
thereby making the XFS-specific support obsolete.
I'm wondering whether it's worth expanding the
On Fri, Mar 09, 2012 at 06:16:54PM +0100, Paolo Bonzini wrote:
I'm wondering whether it's worth expanding the SELinux policy if we
will have no fstatfs(2) callers in QEMU. Are you planning to drop the
XFS code?
Chris Wedgwood said that on XFS you want to do discard even if the file
is
On 24 Mar 2012, at 12:56, Blue Swirl wrote:
On Sat, Mar 17, 2012 at 08:04, Lee Essen lee.es...@nowonline.co.uk wrote:
(third email attempt, apologies if you get duplicates)
The patch does not apply:
git am:
Applying: Basic Illumos support
fatal: corrupt patch at line 19
Patch failed at
On Sat, Mar 10, 2012 at 12:02:40PM -0600, Richard Laager wrote:
If we could probe for FALLOC_FL_PUNCH_HOLE support, then we could avoid
advertising discard support based on FALLOC_FL_PUNCH_HOLE when it is not
going to work. This would side step these problems. You said it wasn't
possible to
On Wed, Mar 14, 2012 at 01:01:35PM +0100, Kevin Wolf wrote:
Paolo mentioned a use case as a fast way for guests to write zeros, but
is it really faster than a normal write when we have to emulate it by a
bdrv_write with a temporary buffer of zeros? On the other hand we have
the cases where
On Wed, Mar 14, 2012 at 01:14:18PM +0100, Paolo Bonzini wrote:
Note that the discard granularity is only a hint, so it's really more a
maximum suggested value than a granularity. Outside of a cluster
boundary the format would still have to write zeros manually.
Also, Linux for example
On Wed, Mar 14, 2012 at 01:49:48PM +0100, Paolo Bonzini wrote:
It does make the distinction. I don't care is UNMAP (or WRITE
SAME(16) with the UNMAP bit set); I want to have zeroes is WRITE
SAME(10) or WRITE SAME(16) with an all-zero payload.
But once the taget sets the unmap zeroes data bit
On Thu, Mar 08, 2012 at 06:15:13PM +0100, Paolo Bonzini wrote:
Allow discard to fail, and fall back to the write operation. This
is needed because there's no simple way to probe for availability
of FALLOC_FL_PUNCH_HOLE.
So you switch on advertising TRIM support in the patch before, and then
On Fri, Mar 09, 2012 at 02:36:50PM -0600, Richard Laager wrote:
I'm not sure if fallocate() and/or BLKDISCARD always guarantee that the
discard has made it to stable storage. If they don't, does O_DIRECT or
O_DSYNC on open() cause them to make any such guarantee? If not, should
you be calling
On Thu, Mar 08, 2012 at 06:15:17PM +0100, Paolo Bonzini wrote:
SEEK_DATA and SEEK_HOLE can be used to implement the is_allocated
callback for raw files. These currently work on btrfs, with an XFS
implementation also coming soon.
Btw - if you're interested in a bit more kernel hacking it would
On Sun, Mar 11, 2012 at 04:03:01PM -0500, Leonardo E. Reiter wrote:
indeed mmap() is used in the code. This is unfortunate that it cannot be
used. It's a really high performance way to achieve what we want here, and
very safe for the use-case. Of course the only medium we support in the
Am 24.03.2012 15:51, schrieb Lee Essen:
On 24 Mar 2012, at 12:56, Blue Swirl wrote:
On Sat, Mar 17, 2012 at 08:04, Lee Essen lee.es...@nowonline.co.uk wrote:
(third email attempt, apologies if you get duplicates)
I did receive all three fine btw.
[...]
Me and patches just aren't working
On 24 Mar 2012, at 15:44, Andreas Färber wrote:
Am 24.03.2012 15:51, schrieb Lee Essen:
On 24 Mar 2012, at 12:56, Blue Swirl wrote:
On Sat, Mar 17, 2012 at 08:04, Lee Essen lee.es...@nowonline.co.uk wrote:
(third email attempt, apologies if you get duplicates)
I did receive all three
Thanks, applied.
On Sun, Mar 18, 2012 at 14:48, Avi Kivity a...@redhat.com wrote:
Add a script that enhances gdb to be aware of QEMU data structures.
This patch adds a single gdb command, 'qemu mtree'. The command is
similar to the monitor's 'info mtree', except that it prints MemoryRegion
It has to be set before configure. Because configure script creates wrong
.h file.
On Sat, Mar 24, 2012 at 2:44 PM, Peter Maydell peter.mayd...@linaro.orgwrote:
On 23 March 2012 20:05, Stefan Weil s...@weilnetz.de wrote:
+# Set default locale for commands like tr and others.
+LANG=C
Thanks, applied.
On Thu, Mar 1, 2012 at 07:34, Gerd Hoffmann kra...@redhat.com wrote:
Commit 45efb16124efef51de5157afc31984b5a47700f9 optimized a bit too
much. We can skip the vga_invalidate_display() in case no console
switch happened because we don't need a full redraw then. We can *not*
Thanks, applied.
On Mon, Mar 19, 2012 at 19:25, Richard Henderson r...@twiddle.net wrote:
This allows us to generate unwind info for the dynamicly generated
code in the code_gen_buffer. Only i386 is converted at this point.
Signed-off-by: Richard Henderson r...@twiddle.net
---
elf.h
Thanks, applied.
On Mon, Mar 19, 2012 at 22:25, Michael Tokarev m...@tls.msk.ru wrote:
We've at least one UTF8 char in the qemu texi doc:
$ grep Tibor qemu-doc.texi
by Tibor TS Schütz.
$ man ./qemu.1 | grep Tibor
by Tibor TS SchA~Xtz.
This patch allows utf8 in man/pod docs.
Thanks, applied.
On Thu, Mar 15, 2012 at 17:49, Peter Maydell peter.mayd...@linaro.org wrote:
From: Meador Inge mead...@codesourcery.com
Fix an issue where the GDB server implementation was sending GDB syscall
requests while the system CPU was still running. Syscall requests must
be sent
sigbus_reraise is used by the kvm_wait_io_event function and is
needed on both Linux and Solaris. This patch adds CONFIG_SOLARIS
to the current CONFIG_LINUX only ifdef.
Signed-off-by: Lee Essen lee.es...@nowonline.co.uk
---
cpus.c |4 +++-
1 files changed, 3 insertions(+), 1 deletions(-)
Dynticks was limited to linux. This patch adds Solaris support
and ensures a CLOCK_HIGHRES clock is used which is the optimal
setup for Solaris systems.
Signed-off-by: Lee Essen lee.es...@nowonline.co.uk
---
qemu-timer.c | 14 +-
1 files changed, 9 insertions(+), 5 deletions(-)
Solaris does not support the O_ASYNC option to open, this patch
adds the same functionality through the I_SETSIG ioctl.
Signed-off-by: Lee Essen lee.es...@nowonline.co.uk
---
qga/channel-posix.c | 16
1 files changed, 16 insertions(+), 0 deletions(-)
diff --git
libsocket and libxnet are required for base network functionality
used in os_dep.c, qemu-socket.c, qga/commands-posix.c and cutils.c
Signed-off-by: Lee Essen lee.es...@nowonline.co.uk
---
configure |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/configure b/configure
Changes v1-v2:
* Formatting fixed during the move to new files. It might have been
cleaner to fix it before the move, but it was easier to run the
existing patches through checkpatch.pl.
* Add noreturn marker, now that it's available.
r~
Richard Henderson (10):
Signed-off-by: Richard Henderson r...@twiddle.net
---
target-alpha/helper.c| 14 --
target-alpha/helper.h|4 ++--
target-alpha/op_helper.c | 10 --
target-alpha/translate.c |6 +++---
4 files changed, 17 insertions(+), 17 deletions(-)
diff --git
Signed-off-by: Richard Henderson r...@twiddle.net
---
Makefile.target |1 +
target-alpha/int_helper.c | 257 +
target-alpha/op_helper.c | 233
3 files changed, 258 insertions(+), 233 deletions(-)
Signed-off-by: Richard Henderson r...@twiddle.net
---
target-alpha/translate.c |3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)
diff --git a/target-alpha/translate.c b/target-alpha/translate.c
index b51fe5c..2c24619 100644
--- a/target-alpha/translate.c
+++ b/target-alpha/translate.c
Signed-off-by: Richard Henderson r...@twiddle.net
---
target-alpha/cpu.h |3 ++
target-alpha/helper.c| 39
target-alpha/helper.h|2 +-
target-alpha/op_helper.c | 73 --
target-alpha/translate.c |2
This softfp feature post-dates the last major update to the Alpha
fpu translation. We can make use of this to eliminate at least
one helper function that was performing this operation by hand.
Signed-off-by: Richard Henderson r...@twiddle.net
---
target-alpha/cpu.h|1 -
Signed-off-by: Richard Henderson r...@twiddle.net
---
target-alpha/helper.h | 12
target-alpha/int_helper.c | 63 +
target-alpha/op_helper.c | 62
target-alpha/translate.c | 45
Er, this is the v2 patch, which Richard updated based on review
comments, can you apply the later v3 one instead?
thanks
-- PMM
On 24 March 2012 16:11, Blue Swirl blauwir...@gmail.com wrote:
Thanks, applied.
On Mon, Mar 19, 2012 at 19:25, Richard Henderson r...@twiddle.net wrote:
This allows
Signed-off-by: Richard Henderson r...@twiddle.net
---
Makefile.target |2 +-
target-alpha/helper.h | 10 +++---
target-alpha/op_helper.c | 65 -
target-alpha/sys_helper.c | 87 +
This completes the transition away from AREG0. This patch must
be last because it requires CONFIG_TCG_PASS_AREG0 set too.
Signed-off-by: Richard Henderson r...@twiddle.net
---
Makefile.target |4 +-
configure |2 +-
target-alpha/helper.h |8 +-
Signed-off-by: Richard Henderson r...@twiddle.net
---
target-alpha/helper.h |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/target-alpha/helper.h b/target-alpha/helper.h
index 9f97c5d..a184def 100644
--- a/target-alpha/helper.h
+++ b/target-alpha/helper.h
@@ -1,6 +1,6 @@
Blue applied v2, not v3. This rebases against master, and also
adds support in the sparc and hppa backends.
I've tested sparc32. I've not been able to try the sparc64 version;
I'm having trouble getting glib2 compiled for 64-bit on the machine
to which I have access. Oh how I wish that was
---
tcg/tcg.c |8 +++-
1 files changed, 7 insertions(+), 1 deletions(-)
diff --git a/tcg/tcg.c b/tcg/tcg.c
index df4edc0..6b2e429 100644
--- a/tcg/tcg.c
+++ b/tcg/tcg.c
@@ -2360,6 +2360,12 @@ static void tcg_register_jit_int(void *buf_ptr, size_t
buf_size,
.e_shentsize =
---
tcg/hppa/tcg-target.c | 103 +---
1 files changed, 88 insertions(+), 15 deletions(-)
diff --git a/tcg/hppa/tcg-target.c b/tcg/hppa/tcg-target.c
index e579ef0..2885212 100644
--- a/tcg/hppa/tcg-target.c
+++ b/tcg/hppa/tcg-target.c
@@ -336,7 +336,7
Am 24.03.2012 17:11, schrieb Emre Ersin:
It has to be set before configure. Because configure script creates
wrong .h file.
On Sat, Mar 24, 2012 at 2:44 PM, Peter Maydell
peter.mayd...@linaro.org mailto:peter.mayd...@linaro.org wrote:
On 23 March 2012 20:05, Stefan Weil s...@weilnetz.de
This allows us to actually supply a function name in softmmu builds;
gdb doesn't pick up the minimal symbol table otherwise. Also add a
bit of documentation and statically generate more of the ELF image.
Signed-off-by: Richard Henderson r...@twiddle.net
---
tcg/tcg.c | 233
---
elf.h | 15 +++
tcg/sparc/tcg-target.c | 63
2 files changed, 78 insertions(+), 0 deletions(-)
diff --git a/elf.h b/elf.h
index 310e05a..36bcac4 100644
--- a/elf.h
+++ b/elf.h
@@ -346,6 +346,21 @@ typedef struct {
Thanks, applied all.
On Sat, Mar 24, 2012 at 16:51, Richard Henderson r...@twiddle.net wrote:
Changes v1-v2:
* Formatting fixed during the move to new files. It might have been
cleaner to fix it before the move, but it was easier to run the
existing patches through checkpatch.pl.
*
On Sat, Mar 24, 2012 at 17:47, Richard Henderson r...@twiddle.net wrote:
Blue applied v2, not v3.
Sorry!
This rebases against master, and also
adds support in the sparc and hppa backends.
I've tested sparc32. I've not been able to try the sparc64 version;
I'm having trouble getting glib2
v2: fix patch 1, tweak patch 2 and rebase to master.
URL git://repo.or.cz/qemu/blueswirl.git
http://repo.or.cz/r/qemu/blueswirl.git
Blue Swirl (6):
arm: move neon_tbl to neon_helper.c
arm: move saturating arithmetic to helper.c
arm: move other arithmetic to helper.c
arm: move
Add an explicit CPUARMState parameter instead of relying on AREG0
and move neon_tbl to neon_helper.c.
Signed-off-by: Blue Swirl blauwir...@gmail.com
---
target-arm/helper.h |2 +-
target-arm/neon_helper.c | 22 ++
target-arm/op_helper.c | 22
Add an explicit CPUARMState parameter instead of relying on AREG0
and move saturating arithmetic to helper.c.
Remove unused SIGNBIT64.
Signed-off-by: Blue Swirl blauwir...@gmail.com
---
target-arm/helper.c| 132 +++
target-arm/helper.h| 20
Add an explicit CPUARMState parameter instead of relying on AREG0
and move rest of the arithmetic to helper.c.
Reviewed-by: Richard Henderson r...@twiddle.net
Signed-off-by: Blue Swirl blauwir...@gmail.com
---
target-arm/helper.c| 146
Add an explicit CPUARMState parameter instead of relying on AREG0
and move cpsr and banked register access to helper.c.
Reviewed-by: Richard Henderson r...@twiddle.net
Signed-off-by: Blue Swirl blauwir...@gmail.com
---
target-arm/helper.c| 42 ++
Add an explicit CPUARMState parameter instead of relying on AREG0
and move exception and wfi helpers to helper.c. Merge raise_exception()
and helper_exception().
Reviewed-by: Richard Henderson r...@twiddle.net
Signed-off-by: Blue Swirl blauwir...@gmail.com
---
target-arm/helper.c| 13
Add an explicit CPUARMState parameter instead of relying on AREG0
and move load and store helpers to helper.c. Remove AREG0 swapping in
tlb_fill(). Remove now empty op_helper.c.
Switch to AREG0 free mode. Use cpu_ld{l,uw}_code in translation
and interrupt handling.
Reviewed-by: Richard Henderson
On 03/24/2012 11:41 AM, Blue Swirl wrote:
Patches 2 to 4 lack the SoB. I can add them by hand if that's OK?
Oops. Yes please.
r~
Thanks, applied all.
On Sat, Mar 24, 2012 at 19:23, Richard Henderson r...@twiddle.net wrote:
On 03/24/2012 11:41 AM, Blue Swirl wrote:
Patches 2 to 4 lack the SoB. I can add them by hand if that's OK?
Oops. Yes please.
r~
Signed-off-by: Hervé Poussineau hpous...@reactos.org
---
trace-events |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/trace-events b/trace-events
index 70f059d..8fa1c1c 100644
--- a/trace-events
+++ b/trace-events
@@ -726,7 +726,7 @@ ppm_save(const char *filename, void
Am 24.03.2012 23:56, schrieb Hervé Poussineau:
Signed-off-by: Hervé Poussineau hpous...@reactos.org
Cf. http://patchwork.ozlabs.org/patch/148493/ - looks identical.
Andreas
---
trace-events |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/trace-events
The Buildbot has detected a new failure on builder default_s390 while building
qemu.
Full details are available at:
http://buildbot.b1-systems.de/qemu/builders/default_s390/builds/200
Buildbot URL: http://buildbot.b1-systems.de/qemu/
Buildslave for this Build: qemu-s390.opensuse.org
Build
The Buildbot has detected a new failure on builder default_ppc while building
qemu.
Full details are available at:
http://buildbot.b1-systems.de/qemu/builders/default_ppc/builds/345
Buildbot URL: http://buildbot.b1-systems.de/qemu/
Buildslave for this Build: qemu-ppc.opensuse.org
Build
The Buildbot has detected a new failure on builder default_x86_64_debian_6_0
while building qemu.
Full details are available at:
http://buildbot.b1-systems.de/qemu/builders/default_x86_64_debian_6_0/builds/223
Buildbot URL: http://buildbot.b1-systems.de/qemu/
Buildslave for this Build: yuzuki
The Buildbot has detected a new failure on builder default_i386_debian_6_0
while building qemu.
Full details are available at:
http://buildbot.b1-systems.de/qemu/builders/default_i386_debian_6_0/builds/223
Buildbot URL: http://buildbot.b1-systems.de/qemu/
Buildslave for this Build: yuzuki
On 20/03/12 9:28 AM, Brad Smith wrote:
On 20/03/12 6:14 AM, Michal Privoznik wrote:
On 18.03.2012 02:09, Brad Smith wrote:
Michal,
http://git.qemu.org/?p=qemu.git;a=commit;h=3424fc9f16a1e7d1c48eb6d605eb0ca63e199ec2
This broke the build. Un-break the tree.
Can you please be more
void HELPER(wsr_ibreaka)(uint32_t i, uint32_t v)
{
if (env-sregs[IBREAKENABLE] (1 i) env-sregs[IBREAKA + i]
!= v) {
tb_invalidate_phys_page_range(
env-sregs[IBREAKA + i], env-sregs[IBREAKA + i] + 1, 0);
tb_invalidate_phys_page_range(v, v + 1, 0);
}
Instruction breakpoint/zero overhead loop handling code is built into
TBs pointed to by IBREAKA/LEND SRs. When these or related SRs get
changed TBs at virtual addresses corresponding to their old and their
new values must be invalidated.
Virtual address range is passed to the
70 matches
Mail list logo
