This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH] Rewrite the codes for opcode 0x0f01 and add more instructions support
- From: Tom Tromey <tromey at redhat dot com>
- To: Jiang Jilin <freephp at gmail dot com>
- Cc: Yao Qi <yao at codesourcery dot com>, gdb-patches at sourceware dot org, Hui Zhu <teawater at gmail dot com>
- Date: Thu, 19 May 2011 13:26:37 -0600
- Subject: Re: [PATCH] Rewrite the codes for opcode 0x0f01 and add more instructions support
- References: <1255182393-15292-1-git-send-email-freephp@gmail.com> <4AD28E95.1050404@vmware.com> <7d77a27d0910111921v13a8e8acpb98c29a9aa72ed8@mail.gmail.com> <AANLkTimZi1dV7ao-0kpAfhDogtXnDwcuJPDwEr5hEGVq@mail.gmail.com> <4C929A9C.1050601@vmware.com> <AANLkTinrdVcu-5YpHZ+By99OcowB5fU1W7rs524PohoJ@mail.gmail.com> <4D91BE41.2010107@codesourcery.com> <AANLkTikq97fZLV3c+34WOY0OKphQs4sNvHfr=uA-qwUd@mail.gmail.com>
>>>>> "Jiang" == Jiang Jilin <freephp@gmail.com> writes:
I'm sorry you haven't gotten a reply to this.
Jiang> 2011-03-29 Jiang Jilin <freephp@gmail.com>
Jiang> * i386-tdep.c (i386_process_record): Rewrite the codes for
Jiang> opcode 0x0f01 and add more instructions support.
I am not the right person to review this.
I would have to look up all this stuff in the x86 references to try to
understand it. I'd appreciate it if Hui would reply about these details.
Most of the code seems to at least be in the same style as the
surrounding code, so at least on those grounds I find it basically ok.
Jiang> + switch (reg_rm)
Jiang> + {
Jiang> + /* vmcall */
Jiang> + case 0x01:
It looks like there is an extra space in the indentation here.
Jiang> + q = yquery (_("\
Jiang> + Process record ignores the memory change of \
Jiang> + instruction at address %s\n\
Jiang> + because it can't get the value of the segment \
Jiang> + register.\n Do you want to stop the program?"),
Reformatting this message like this is not ok.
The way it was before the patch is more correct.
Tom