lkml.org 
[lkml]   [2014]   [Jul]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] microblaze: fixes includes in sigcontext.h
On Thu, 3 Jul 2014, Nick Krause wrote:

> You may be right I don't have a compiler that supports microblaze therefore I
> would suggest removing this FIXME message.

The FIXME here is obviously stating that the code needs to be fixed so
that it can include linux/ptrace.h rather than asm/ptrace.h since we want
to include the former rather than the latter. It doesn't imply, in any
way, that you can just do the conversion without some due diligence
(compilation) to ensure it works; otherwise it would have been including
linux/ptrace.h since the beginning.

It's bad practice to mess with per-arch header files for platforms that
you cannot compile yourself.

> Cheers Nick
>
> On Thu, Jul 3, 2014 at 5:57 PM, David Rientjes <rientjes@google.com> wrote:
> > On Thu, 3 Jul 2014, Nicholas Krause wrote:
> >
> >> diff --git a/arch/microblaze/include/uapi/asm/sigcontext.h b/arch/microblaze/include/uapi/asm/sigcontext.h
> >> index 55873c8..1180d9b 100644
> >> --- a/arch/microblaze/include/uapi/asm/sigcontext.h
> >> +++ b/arch/microblaze/include/uapi/asm/sigcontext.h
> >> @@ -9,8 +9,7 @@
> >> #ifndef _ASM_MICROBLAZE_SIGCONTEXT_H
> >> #define _ASM_MICROBLAZE_SIGCONTEXT_H
> >>
> >> -/* FIXME should be linux/ptrace.h */
> >> -#include <asm/ptrace.h>
> >> +#include <linux/ptrace.h>
> >>
> >> struct sigcontext {
> >> struct pt_regs regs;
> >
> > Umm, does this even compile? How does sigcontext.h get the definition of
> > struct pt_regs? Seems like it would be better to just do a forward
> > declaration.
>


\
 
 \ /
  Last update: 2014-07-04 01:01    [W:0.113 / U:0.304 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site