[Qemu-devel] [PATCH] MIPS single stepping

[Qemu-devel] [PATCH] MIPS single stepping

[Dirk Behme]

[-- Attachment #1: Type: text/plain, Size: 398 bytes --]

Hi,

patch below makes GDB single stepping for MIPS work.

For debugging (breakpoint set), in
target-mips/translate.c/gen_intermediate_code_internal()
gen_opc_pc[] isn't initialized. So in debugging case don't call
cpu_restore_state() which then accesses uninitalized
gen_opc_pc[]. do_raise_exception_direct() indirectly calls
cpu_restore_state(), do_raise_exception() doesn't.

Best regards

Dirk

[-- Attachment #2: qemu-mips-single-step.txt --]
[-- Type: text/plain, Size: 331 bytes --]

--- ./target-mips/op.c_orig	2006-04-19 18:14:00.000000000 +0200
+++ ./target-mips/op.c	2006-04-19 18:14:29.000000000 +0200
@@ -604,7 +604,7 @@ void op_trap (void)
 
 void op_debug (void)
 {
-  CALL_FROM_TB1(do_raise_exception_direct, EXCP_DEBUG);
+  CALL_FROM_TB1(do_raise_exception, EXCP_DEBUG);
 }
 
 void op_set_lladdr (void)

Re: [Qemu-devel] [PATCH] MIPS single stepping

[Stefan Weil]

Great, it works fine. It should be integrated in CVS HEAD,
so all MIPS users get it...

Many thanks!
Stefan

Re: [Qemu-devel] [PATCH] MIPS single stepping

[Thiemo Seufer]

Stefan Weil wrote:
> Great, it works fine. It should be integrated in CVS HEAD,
> so all MIPS users get it...

FWIW, I have some rather massive MIPS update (e.g. MIPS32R2 support)
in the works and hope to get it finished enough the next days to
make a quilt patchset of it. I plan to integrate the other MIPS patches
into it for the time being.


Thiemo

[Qemu-devel] MIPS patches, was: MIPS single stepping

[Dirk Behme]

Thiemo Seufer wrote:
> Stefan Weil wrote:
> 
>>Great, it works fine. 

Thanks :)

> FWIW, I have some rather massive MIPS update (e.g. MIPS32R2 support)
> in the works and hope to get it finished enough the next days to
> make a quilt patchset of it. I plan to integrate the other MIPS patches
> into it for the time being.

My list of MIPS patches:

MIPS Bugs:
=========

1. [PATCH] Fix overflow conditions for MIPS add / subtract
http://lists.gnu.org/archive/html/qemu-devel/2006-04/msg00270.html

*OR*

[PATCH] Fix test for two's complement overflow
http://lists.gnu.org/archive/html/qemu-devel/2006-02/msg00154.html

(not sure which is the prefered one. Are both equal?)

2. [PATCH] MIPS CP0 not usable in kernel mode?
http://lists.gnu.org/archive/html/qemu-devel/2006-03/msg00148.html

3. [PATCH] MIPS single stepping
http://lists.gnu.org/archive/html/qemu-devel/2006-04/msg00336.html

MIPS Improvements:
==================

1. [PATCH] Huge TLB performance improvement
http://lists.gnu.org/archive/html/qemu-devel/2006-03/msg00034.html

2. [PATCH] Add MIPS ELF loader
http://lists.gnu.org/archive/html/qemu-devel/2006-03/msg00185.html

3. [PATCH][MIPS] add "lwu" instruction
http://lists.gnu.org/archive/html/qemu-devel/2006-04/msg00326.html

4. [PATCH][MIPS] add basic FPU support
http://lists.gnu.org/archive/html/qemu-devel/2006-04/msg00327.html

Anything missing?

Dirk

Re: [Qemu-devel] MIPS patches, was: MIPS single stepping

[Raphaël Rigo]

Dirk Behme wrote:
> Thiemo Seufer wrote:
>> Stefan Weil wrote:
>>
>>> Great, it works fine. 
> 
> Thanks :)
> 
>> FWIW, I have some rather massive MIPS update (e.g. MIPS32R2 support)
>> in the works and hope to get it finished enough the next days to
>> make a quilt patchset of it. I plan to integrate the other MIPS patches
>> into it for the time being.
> 
> My list of MIPS patches:
> 
> MIPS Bugs:
> =========
> 
> 1. [PATCH] Fix overflow conditions for MIPS add / subtract
> http://lists.gnu.org/archive/html/qemu-devel/2006-04/msg00270.html
> 
> *OR*
> 
> [PATCH] Fix test for two's complement overflow
> http://lists.gnu.org/archive/html/qemu-devel/2006-02/msg00154.html
> 
> (not sure which is the prefered one. Are both equal?)
> 
> 2. [PATCH] MIPS CP0 not usable in kernel mode?
> http://lists.gnu.org/archive/html/qemu-devel/2006-03/msg00148.html
> 
> 3. [PATCH] MIPS single stepping
> http://lists.gnu.org/archive/html/qemu-devel/2006-04/msg00336.html
> 
> MIPS Improvements:
> ==================
> 
> 1. [PATCH] Huge TLB performance improvement
> http://lists.gnu.org/archive/html/qemu-devel/2006-03/msg00034.html
> 
> 2. [PATCH] Add MIPS ELF loader
> http://lists.gnu.org/archive/html/qemu-devel/2006-03/msg00185.html
> 
> 3. [PATCH][MIPS] add "lwu" instruction
> http://lists.gnu.org/archive/html/qemu-devel/2006-04/msg00326.html
> 
> 4. [PATCH][MIPS] add basic FPU support
> http://lists.gnu.org/archive/html/qemu-devel/2006-04/msg00327.html
> 
> Anything missing?
> 
> Dirk
> 
> 
> _______________________________________________
> Qemu-devel mailing list
> Qemu-devel@nongnu.org
> http://lists.nongnu.org/mailman/listinfo/qemu-devel
You might want to include my patch adding some sockets-related syscalls.

Raph

Re: [Qemu-devel] MIPS patches, was: MIPS single stepping

[Marius Groeger]

On Thu, 20 Apr 2006, Dirk Behme wrote:

> Thiemo Seufer wrote:
>
>>  FWIW, I have some rather massive MIPS update (e.g. MIPS32R2 support)
>>  in the works and hope to get it finished enough the next days to
>>  make a quilt patchset of it. I plan to integrate the other MIPS patches
>>  into it for the time being.

Excellent news, Thiemo! :-)

> My list of MIPS patches:
...
> 4. [PATCH][MIPS] add basic FPU support
> http://lists.gnu.org/archive/html/qemu-devel/2006-04/msg00327.html

Thanks for that list, Dirk.

FYI, I continued work on that one and will soonish be posting an 
updated version which actually will do some more CP1 ops (cvt, trunc, 
add/sub/mul/div, m*c1, c, bc1*) as well as handling better the 
Status.FR setting, etc.

Regards,
Marius

-- 
Marius Groeger <mgroeger@sysgo.com>
SYSGO AG                      Embedded and Real-Time Software
Voice: +49 6136 9948 0                  FAX: +49 6136 9948 10
www.sysgo.com | www.elinos.com | www.osek.de | www.pikeos.com