Re: [SH][PATCH] fix headers for new kernel headers

[Kaz Kojima <kkojima at rr dot iij4u dot or dot jp>]

SUGIOKA Toshinobu <sugioka@itonet.co.jp> wrote:
>>Did you see the thread
>>
>>http://sourceware.org/ml/libc-alpha/2009-02/msg00004.html
>>
>>?  Does Andrew's patch fix your problem?
>>
> 
> Thanks for the reply.
> I tried Andrew's patch. It worked for compiling glibc itself, but
> 'struct user' is needed for compile native gdb. So I modified the patch as follows
> and then succeeded.

It looks that you've dropped REG_* definitions from Andrew's
latest patch:

http://sourceware.org/ml/libc-alpha/2009-02/msg00018.html

Is it intentional?

BTW, please see also

http://sourceware.org/ml/libc-alpha/2009-02/msg00034.html
http://sourceware.org/ml/libc-alpha/2009-02/msg00035.html

and the indentation for struct user has to be matched with
the others.  Simply

struct user
  {
    struct pt_regs regs;
    struct user_fpu_struct fpu;
    int	u_fpvalid;
    size_t u_tsize;
    size_t u_dsize;
    size_t u_ssize;
    unsigned long start_code;
    unsigned long start_data;
    unsigned long start_stack;
    long int signal;
    unsigned long u_ar0;
    struct user_fpu_struct *u_fpstate;
    unsigned long magic;
    char u_comm[32];
  };

will be ok.

Regards,
	kaz