Re: record mode supposed to work on x86_64-linux?

[Hui Zhu <teawater at gmail dot com>]

Hi Pedro,

What do you think about this one?

2009-05-17  Hui Zhu  <teawater@gmail.com>
	
	* i386-tdep.c (i386_process_record): Remove the static of it.
	(i386_gdbarch_init): Remove the set_gdbarch_process_record.
	* i386-tdep.c (i386_process_record): New extern.
	* i386-linux-tdep.c (i386_linux_init_abi): Call
	set_gdbarch_process_record.

Sorry for my always understand your mean so slow.  :)

Thanks,
Hui

On Sun, May 17, 2009 at 00:35, Pedro Alves <pedro@codesourcery.com> wrote:
> On Saturday 16 May 2009 10:51:42, Hui Zhu wrote:
>> Sorry. ?I made a mistake in gdbarch.sh.
>> How about this one?
>
> There's no need for a new gdbarch method if you
> implement what I suggested before:
>
> ?http://sourceware.org/ml/gdb-patches/2009-05/msg00267.html
>
>>
>> Thanks,
>> Hui
>>
>> On Sat, May 16, 2009 at 17:27, Andi Kleen <andi@firstfloor.org> wrote:
>> > On Sat, May 16, 2009 at 05:09:12PM +0800, Hui Zhu wrote:
>> >> Hi Andi,
>> >>
>> >> Thanks for remind me.
>> >> I make a patch for it. ?But I don't have a x86-64 close to me now.
>> >> Could you help me test it?
>> >
>> > Gets
>> >
>> > ../../src/gdb/gdbarch.c:648: internal-error: verify_gdbarch: the following are invalid ...
>> > ? ? ? ?support_process_record
>> > A problem internal to GDB has been detected,
>> > further debugging may prove unreliable.
>> >
>> > at startup. I haven't tried further.
>> >
>> > -Andi
>> >
>>
>
>
>
> --
> Pedro Alves
>

Attachment: prec_move_to_target.txt
Description: Text document