This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: PATCH: Remove amd64_linux_gregset64_reg_offset and regmap
On Thu, Apr 22, 2010 at 12:48 PM, Mark Kettenis <mark.kettenis@xs4all.nl> wrote:
>> Date: Wed, 21 Apr 2010 16:09:29 -0700
>> From: "H.J. Lu" <hongjiu.lu@intel.com>
>>
>> Hi,
>>
>> As we have discussed, this patch removes amd64_linux_gregset64_reg_offset
>> and regmap from i386/amd64 Linux targets. ?OK to install?
>
> The amd64 bits are ok. ?A small issue with the i386 bits below.
>> diff --git a/gdb/i386-linux-nat.c b/gdb/i386-linux-nat.c
>> index d559811..35bc3bd 100644
>> --- a/gdb/i386-linux-nat.c
>> +++ b/gdb/i386-linux-nat.c
>> @@ -245,18 +228,19 @@ void
>> ?fill_gregset (const struct regcache *regcache,
>> ? ? ? ? ? ? elf_gregset_t *gregsetp, int regno)
>> ?{
>> - ?elf_greg_t *regp = (elf_greg_t *) gregsetp;
>> + ?gdb_byte *regp = (gdb_byte *) gregsetp;
>> ? ?int i;
>>
>> ? ?for (i = 0; i < I386_NUM_GREGS; i++)
>> ? ? ?if (regno == -1 || regno == i)
>> - ? ? ?regcache_raw_collect (regcache, i, regp + regmap[i]);
>> + ? ? ?regcache_raw_collect (regcache, i,
>> + ? ? ? ? ? ? ? ? ? ? ? ? regp + i386_linux_gregset_reg_offset[i]);
>>
>> ? ?if ((regno == -1 || regno == I386_LINUX_ORIG_EAX_REGNUM)
>> ? ? ? ?&& I386_LINUX_ORIG_EAX_REGNUM
>> ? ? ? ? ?< gdbarch_num_regs (get_regcache_arch (regcache)))
>> ? ? ?regcache_raw_collect (regcache, I386_LINUX_ORIG_EAX_REGNUM,
>> - ? ? ? ? ? ? ? ? ? ? ? regp + ORIG_EAX);
>> + ? ? ? ? ? ? ? ? ? ? ? regp + 4 * ORIG_EAX);
>
> I think this should use an offset from i386_linux_gregset_reg_offset[]
> as well.
>
Like this?
diff --git a/gdb/i386-linux-nat.c b/gdb/i386-linux-nat.c
index 35bc3bd..eba0b6a 100644
--- a/gdb/i386-linux-nat.c
+++ b/gdb/i386-linux-nat.c
@@ -217,7 +217,7 @@ supply_gregset (struct regcache *regcache, const
elf_gregset_t *gregsetp)
if (I386_LINUX_ORIG_EAX_REGNUM
< gdbarch_num_regs (get_regcache_arch (regcache)))
regcache_raw_supply (regcache, I386_LINUX_ORIG_EAX_REGNUM,
- regp + ORIG_EAX);
+ regp + i386_linux_gregset_reg_offset[I386_LINUX_ORIG_EAX_REGNUM]);
}
/* Fill register REGNO (if it is a general-purpose register) in
@@ -240,7 +240,7 @@ fill_gregset (const struct regcache *regcache,
&& I386_LINUX_ORIG_EAX_REGNUM
< gdbarch_num_regs (get_regcache_arch (regcache)))
regcache_raw_collect (regcache, I386_LINUX_ORIG_EAX_REGNUM,
- regp + ORIG_EAX);
+ regp + i386_linux_gregset_reg_offset[I386_LINUX_ORIG_EAX_REGNUM]);
}
#ifdef HAVE_PTRACE_GETREGS
--
H.J.