This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [patch] Disassembly improvements
- From: Doug Evans <dje at google dot com>
- To: "Abid, Hafiz" <Hafiz_Abid at mentor dot com>
- Cc: "Pedro Alves <palves at redhat dot com> (palves at redhat dot com)" <palves at redhat dot com>, "gdb-patches at sourceware dot org" <gdb-patches at sourceware dot org>, "Mirza, Taimoor" <Taimoor_Mirza at mentor dot com>
- Date: Fri, 11 Oct 2013 14:34:19 -0700
- Subject: Re: [patch] Disassembly improvements
- Authentication-results: sourceware.org; auth=none
- References: <EB3B29AD43CA924DA27099BC85192376E0705106 at EU-MBX-03 dot mgc dot mentorg dot com>
On Thu, Oct 10, 2013 at 6:14 AM, Abid, Hafiz <Hafiz_Abid@mentor.com> wrote:
> Hi Pedro,
> I am attaching the patch that was mentioned in the following thread. I resurrected it from our internal repo, did a bit of manual testing and run the regression suite without any problem. It basically reads memory from the target in a buffer in gdb_disassembly and tries to use this buffer in dis_asm_read_memory instead of reading from the target. This saves us on repeated memory read calls. The problem was noted when eclipse was trying to fill its disassembly view.
> https://sourceware.org/ml/gdb-patches/2013-10/msg00221.html
>
> Regards,
> Abid
>
> 2013-10-10 Taimoor Mirza <taimoor_mirza@mentor.com>
>
> * disasm.c (DIS_BUF_SIZE): New define.
> (dis_asm_read_memory): Read from the disassembly buffer instead
> of target memory directly.
> (gdb_disassembly): Fill the disassembly buffer with a chunk of
> the memory to disassemble.
This is a specific fix to a general problem.
Question: How much more of the general problem can we fix without
having a fix baked into the disassembler?