Re: [Qemu-devel] Re: [PATCH 4/4] gdbstub: x86: Switch 64/32 bit registers dynamically

[Jan Kiszka]

Paul Brook wrote:
>>>> No question, this is a gdb issue. But, as it was confirmed in several
>>>> discusssions with gdb people, it is a non-trivial thing to fix. So until
>>>> qemu finds a gdb version attach with a rework x86 support, we have to
>>>> work around it by switching the register layout as the guest switches
>>>> its execution mode between 16/32 and 64 bit.
>>> I still object to this patch. Especially as there's no indication that
>>> it's a harroble hack to workaround broken GDB.
>> Sorry, last sentence doesn't parse for me.
> 
> You've just deliberately broken the qemu gdbstub. After your patch it is 
> impossible to debug mixed 32/64-bit code.

Please give it a try as it's the other way around: You can't properly
debug mixed target code without my patch. You can only debug 32/16 bit
code with qemu (for i386) and 64 bit code with qemu-system-x86_64. But
those scenarios are not affected by my patch in any way.

> 
>> So what do you suggest for fixing this bug? 
> 
> Fix gdb.

It's not a one-liner, far more complex than this intermediate workaround.

> 
>> Do you have a patch at hand to fix gdb before 7.0?
> 
> No. I'm unconvinced by any argument that requires a specific GDB version. 
> You've known about this bug for a long time now.
> 

Please don't forget that it wasn't me to remove the workaround from qemu
before fixing gdb first. That workaround used to be there for a reason.

>> And a solution for older gdbs?
> 
> If you really care about old gdb, then you get to backport the changes.

To all standard distros out there in the field... (this is not just
about fixing my personal debug environment)

Jan

-- 
Siemens AG, Corporate Technology, CT SE 2
Corporate Competence Center Embedded Linux