Re: [PATCH 3/4] arm64: LLVMLinux: Calculate current_thread_info from current_stack_pointer

[Behan Webster]

On 07/30/14 22:31, Andreas Färber wrote:
> Hi,
>
> Am 31.07.2014 01:57, schrieb behanw@xxxxxxxxxxxxxxxxxx:
> > From: Behan Webster <behanw@xxxxxxxxxxxxxxxxxx>
> >
> > Use the global current_stack_pointer to get the value of the stack pointer.
> > This change supports being able to compile the kernel with both gcc and clang.
> >
> > Signed-off-by: Behan Webster <behanw@xxxxxxxxxxxxxxxxxx>
> > Signed-off-by: Mark Charlebois <charlebm@xxxxxxxxx>
> > Reviewed-by: Jan-Simon M??ller <dl9pf@xxxxxx>
> Something went wrong with ö encoding here and in 2/4.
Yeah. That keeps happening. :(

Thanks.

> > ---
> >   arch/arm64/include/asm/thread_info.h | 4 ++--
> >   1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/arch/arm64/include/asm/thread_info.h b/arch/arm64/include/asm/thread_info.h
> > index e6b6094..c2432d2 100644
> > --- a/arch/arm64/include/asm/thread_info.h
> > +++ b/arch/arm64/include/asm/thread_info.h
> > @@ -80,8 +80,8 @@ static inline struct thread_info *current_thread_info(void) __attribute_const__;
> >   
> >   static inline struct thread_info *current_thread_info(void)
> >   {
> > -	register unsigned long sp asm ("sp");
> > -	return (struct thread_info *)(sp & ~(THREAD_SIZE - 1));
> > +	return (struct thread_info *) \
> This is not a macro, so \ seems superfluous.
Doh. Indeed. Will fix.

Behan

--
Behan Webster
behanw@xxxxxxxxxxxxxxxxxx

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/