LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
* Re: [IA64] fix userspace compile error in gcc_intrin.h
       [not found] <200802052007.m15K7vuv005319@hera.kernel.org>
@ 2008-02-29 18:24 ` David Woodhouse
  2008-02-29 18:41   ` Doug Chapman
  0 siblings, 1 reply; 3+ messages in thread
From: David Woodhouse @ 2008-02-29 18:24 UTC (permalink / raw)
  To: Linux Kernel Mailing List; +Cc: Doug Chapman, Andrew Morton, Tony Luck


On Tue, 2008-02-05 at 20:07 +0000, Linux Kernel Mailing List wrote:
> Gitweb:     http://git.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=0df29025fd0379d5950d206314d0b10a2c8a9607
> Commit:     0df29025fd0379d5950d206314d0b10a2c8a9607
> Parent:     d2fc0bacd5c438cb459fdf531eff00ab18422a00
> Author:     Doug Chapman <doug.chapman@hp.com>
> AuthorDate: Mon Jan 28 15:33:28 2008 -0800
> Committer:  Tony Luck <tony.luck@intel.com>
> CommitDate: Mon Feb 4 15:08:25 2008 -0800
> 
>     [IA64] fix userspace compile error in gcc_intrin.h
>     
>     Fixes userspace build errors when linux/ipv6.h is included such as in the
>     dhcpv6 package under fedora.  Likely causes other userspace build errors as
>     well.  I found this in akpm's tree from 2.6.18 but could not find any case
>     of anyone proposing it for the main tree.
>     
>     Signed-off-by: Doug Chapman <doug.chapman@hp.com>
>     Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
>     Signed-off-by: Tony Luck <tony.luck@intel.com>
> ---
>  include/asm-ia64/gcc_intrin.h |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)
> 
> diff --git a/include/asm-ia64/gcc_intrin.h b/include/asm-ia64/gcc_intrin.h
> index 5b6665c..de2ed2c 100644
> --- a/include/asm-ia64/gcc_intrin.h
> +++ b/include/asm-ia64/gcc_intrin.h
> @@ -24,7 +24,9 @@
>  extern void ia64_bad_param_for_setreg (void);
>  extern void ia64_bad_param_for_getreg (void);
>  
> +#ifdef __KERNEL__
>  register unsigned long ia64_r13 asm ("r13") __used;
> +#endif
>  
>  #define ia64_setreg(regnum, val)						\
>  ({										\

You also want to move gcc_interin.h from $(header-y) to $(unifdef-y) in
include/asm-ia64/Kbuild.

-- 
dwmw2


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [IA64] fix userspace compile error in gcc_intrin.h
  2008-02-29 18:24 ` [IA64] fix userspace compile error in gcc_intrin.h David Woodhouse
@ 2008-02-29 18:41   ` Doug Chapman
  2008-03-01 14:49     ` David Woodhouse
  0 siblings, 1 reply; 3+ messages in thread
From: Doug Chapman @ 2008-02-29 18:41 UTC (permalink / raw)
  To: David Woodhouse; +Cc: Linux Kernel Mailing List, Andrew Morton, Tony Luck


On Fri, 2008-02-29 at 13:24 -0500, David Woodhouse wrote:
> On Tue, 2008-02-05 at 20:07 +0000, Linux Kernel Mailing List wrote:
> > Gitweb:     http://git.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=0df29025fd0379d5950d206314d0b10a2c8a9607
> > Commit:     0df29025fd0379d5950d206314d0b10a2c8a9607
> > Parent:     d2fc0bacd5c438cb459fdf531eff00ab18422a00
> > Author:     Doug Chapman <doug.chapman@hp.com>
> > AuthorDate: Mon Jan 28 15:33:28 2008 -0800
> > Committer:  Tony Luck <tony.luck@intel.com>
> > CommitDate: Mon Feb 4 15:08:25 2008 -0800
> > 
> >     [IA64] fix userspace compile error in gcc_intrin.h
> >     
> >     Fixes userspace build errors when linux/ipv6.h is included such as in the
> >     dhcpv6 package under fedora.  Likely causes other userspace build errors as
> >     well.  I found this in akpm's tree from 2.6.18 but could not find any case
> >     of anyone proposing it for the main tree.
> >     
> >     Signed-off-by: Doug Chapman <doug.chapman@hp.com>
> >     Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
> >     Signed-off-by: Tony Luck <tony.luck@intel.com>
> > ---
> >  include/asm-ia64/gcc_intrin.h |    2 ++
> >  1 files changed, 2 insertions(+), 0 deletions(-)
> > 
> > diff --git a/include/asm-ia64/gcc_intrin.h b/include/asm-ia64/gcc_intrin.h
> > index 5b6665c..de2ed2c 100644
> > --- a/include/asm-ia64/gcc_intrin.h
> > +++ b/include/asm-ia64/gcc_intrin.h
> > @@ -24,7 +24,9 @@
> >  extern void ia64_bad_param_for_setreg (void);
> >  extern void ia64_bad_param_for_getreg (void);
> >  
> > +#ifdef __KERNEL__
> >  register unsigned long ia64_r13 asm ("r13") __used;
> > +#endif
> >  
> >  #define ia64_setreg(regnum, val)						\
> >  ({										\
> 
> You also want to move gcc_interin.h from $(header-y) to $(unifdef-y) in
> include/asm-ia64/Kbuild.
> 

David,

Thanks for catching that.  I will post a patch for the Kbuild part later
today.

- Doug



^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [IA64] fix userspace compile error in gcc_intrin.h
  2008-02-29 18:41   ` Doug Chapman
@ 2008-03-01 14:49     ` David Woodhouse
  0 siblings, 0 replies; 3+ messages in thread
From: David Woodhouse @ 2008-03-01 14:49 UTC (permalink / raw)
  To: Doug Chapman; +Cc: Linux Kernel Mailing List, Andrew Morton, Tony Luck


On Fri, 2008-02-29 at 13:41 -0500, Doug Chapman wrote:
> Thanks for catching that.  I will post a patch for the Kbuild part
> later today.

There's also a bunch of other stuff in that file which looks like
perhaps it shouldn't be exposed to userspace.

-- 
dwmw2


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2008-03-01 14:50 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <200802052007.m15K7vuv005319@hera.kernel.org>
2008-02-29 18:24 ` [IA64] fix userspace compile error in gcc_intrin.h David Woodhouse
2008-02-29 18:41   ` Doug Chapman
2008-03-01 14:49     ` David Woodhouse

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).