This is the mail archive of the libc-hacker@sourceware.cygnus.com mailing list for the glibc project.

Note that libc-hacker is a closed list. You may look at the archives of this list, but subscription and posting are not open.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Re: Patch for linuxthreads_db/td_thr_getgregs.c


On Wed, Jan 12, 2000 at 01:40:38PM +0100, Andreas Schwab wrote:

> |> > Uli, if those function declarations are indeed correct (which I
> |> > doubt), you need to change td_thr_getgregs.c accordingly (memset
> |> > (&greg,...)).
> |> 
> |> Well, the form I have in the moment is acutally what Solaris has.  It
> |> was strange when I saw it but I haven't really spend time looking at
> |> it.  But you are right, it cannot be correct.
> 
> The problem is that gregset_t is sometimes an array (i386), sometimes a
> struct (mips).  The array decays to a pointer, so you won't see the
> different types, but the sizeof in td_thr_getregs will return the wrong
> value (size of a pointer instead of the whole array).

I've taken care that on gpregset_t can also be accessed using one of the
indices defined in <asm/reg.h>.  So if it makes things easier it's ok
to define a userland gpregset_t that is an array.

  Ralf

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]