Re: [Qemu-devel] [PATCH] Add single stepping option for all targets

[Stefan Weil <weil@mail.berlios.de>]

Laurent Desnogues schrieb:
> I have two comments:
>    - as Stuart previously wrote, user mode would be useful to
>      have
>   

Right. A quick-and-dirty solution to set the flag for user mode is very
simple
but also dirty: Today, bsd-user, darwin-user and linux-user would need the
same kind of patch in main.c. There is already too much code duplication
(or triplication) in these three directories, so a common code basis for all
user modes would help. I suggest to take the current patch first, then clean
the user mode code in a second step, then add single stepping for
user mode in a third step.

>    - shouldn't you also prevent block chaining from being done
>      for this feature to be really useful?
>   

The current implementation replaces conditional compilations
for several system emulations and adds the same feature to those
system emulations which did not have a conditional compilation.

I (and other users, especially those who added the conditional
compilation) use this feature, so it is really useful.

Maybe it can be improved the way you suggested. Do you have code
for this?

>
> Laurent
>
>
>
>   

Regards
Stefan