Re: [PATCH][Qemu-devel] Single stepping for PPC broken!

2008-02-13 Thread Marius Groeger
On Wed, 13 Feb 2008, Daniel Jacobowitz wrote: > The else block will be entered if !A, or if A && B. Yeah - oops - sorry :-) Regards Marius -- Marius Groeger SYSGO AG Embedded and Real-Time Software Voice: +49 6136 9948 0 FAX: +49 6136 9948 10 www.sysgo.com

Re: [Qemu-devel] FXSAVE doesn't #GP on alignment error

2008-02-13 Thread Michal Schulz
On Wednesday 13 February 2008, Vegard Nossum wrote: > I am not sure if this should be treated as an error or not (it > definitely isn't critical), but I'll post it as an extra information > if it is useful or you think it should be fixed. > > According to Intel manuals, the FXSAVE instruction requ

Re: [PATCH][Qemu-devel] Single stepping for PPC broken!

2008-02-13 Thread Daniel Jacobowitz
On Wed, Feb 13, 2008 at 04:52:22PM +0100, Marius Groeger wrote: > On Wed, 13 Feb 2008, Daniel Jacobowitz wrote: > > > On Wed, Feb 13, 2008 at 09:46:44AM +0100, Marius Groeger wrote: > > > if ((tb->pc & TARGET_PAGE_MASK) == (dest & TARGET_PAGE_MASK) && > > > !ctx->singlestep_enabled) { > >

[Qemu-devel] FXSAVE doesn't #GP on alignment error

2008-02-13 Thread Vegard Nossum
Hello, I am not sure if this should be treated as an error or not (it definitely isn't critical), but I'll post it as an extra information if it is useful or you think it should be fixed. According to Intel manuals, the FXSAVE instruction requires the argument to be 16-byte aligned. On a real mac

[Qemu-devel] x86_64 movl bug

2008-02-13 Thread jakllsch
Hi, http://mail-index.netbsd.org/port-amd64/2008/02/13/msg000124.html http://mail-index.netbsd.org/netbsd-bugs/2008/02/13/msg001312.html This looked like this in QEMU 0.9.1: 0x80115260: cmp%gs:0x2d4,%edi 0x80115268: jae0x80115288 IN: 0x

Re: [PATCH][Qemu-devel] Single stepping for PPC broken!

2008-02-13 Thread Marius Groeger
On Wed, 13 Feb 2008, Daniel Jacobowitz wrote: > On Wed, Feb 13, 2008 at 09:46:44AM +0100, Marius Groeger wrote: > > if ((tb->pc & TARGET_PAGE_MASK) == (dest & TARGET_PAGE_MASK) && > > !ctx->singlestep_enabled) { > > .. > > } else { > > gen_set_T1(dest); > > #if defined(TARGET_PPC64)

[Qemu-devel] Use SIGALRM for all timers

2008-02-13 Thread Anders Melchiorsen
I wonder if this patch makes sense, using SIGALRM for all kinds of timers, even those expiring through a file descriptor? It seems more clean to me to always use SIGALRM for alarms, but the actual trigger for this change was to free up SIGIO for KVM. Unfortunately, I have been unable to test it m

Re: [PATCH][Qemu-devel] Single stepping for PPC broken!

2008-02-13 Thread Daniel Jacobowitz
On Wed, Feb 13, 2008 at 09:46:44AM +0100, Marius Groeger wrote: > if ((tb->pc & TARGET_PAGE_MASK) == (dest & TARGET_PAGE_MASK) && > !ctx->singlestep_enabled) { > .. > } else { > gen_set_T1(dest); > #if defined(TARGET_PPC64) > if (ctx->sf_mode) > gen_op_b_T1_64(); > els

Fwd: Re: [Qemu-devel] What does code_copy_enabled do?

2008-02-13 Thread Felipe Contreras
Apparently I write in a spamy way. There's no contact for these kinds of issues, so I'm sending this here, sorry. -- Forwarded message -- From: Date: Feb 13, 2008 11:24 AM Subject: RE: Re: [Qemu-devel] What does code_copy_enabled do? To: [EMAIL PROTECTED] MDaemon has identifi

Re: [Qemu-devel] What does code_copy_enabled do?

2008-02-13 Thread Felipe Contreras
On Feb 13, 2008 10:22 AM, Rob Landley <[EMAIL PROTECTED]> wrote: > On Tuesday 12 February 2008 05:57:18 Ian Jackson wrote: > > hg is much better. OTOH it does have one severe problem: it expects > > you to check in before merging, which is the opposite of what cvs > > users expect. A cvs refugee,

Re: [PATCH][Qemu-devel] Single stepping for PPC broken!

2008-02-13 Thread Marius Groeger
On Mon, 11 Feb 2008, Rob Landley wrote: > On Thursday 10 January 2008 07:57:50 Marius Groeger wrote: > > The attached patch fixes the problem, but I have to admit I can't tell > > for sure if this doesn't break other things (such as qemu's built-in > > GDB server). Could some QEMU ppc expert pleas

Re: [Qemu-devel] What does code_copy_enabled do?

2008-02-13 Thread Rob Landley
On Tuesday 12 February 2008 05:57:18 Ian Jackson wrote: > hg is much better. OTOH it does have one severe problem: it expects > you to check in before merging, which is the opposite of what cvs > users expect. A cvs refugee, or anyone working on a project whose > leaders don't like merge changese