All of lore.kernel.org
 help / color / mirror / Atom feed
From: behanw@converseincode.com (Behan Webster)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 3/4] arm64: LLVMLinux: Calculate current_thread_info from current_stack_pointer
Date: Thu, 31 Jul 2014 09:04:25 -0700	[thread overview]
Message-ID: <53DA6909.1050007@converseincode.com> (raw)
In-Reply-To: <53D9D4B2.8070008@suse.de>

On 07/30/14 22:31, Andreas F?rber wrote:
> Hi,
>
> Am 31.07.2014 01:57, schrieb behanw at converseincode.com:
>> From: Behan Webster <behanw@converseincode.com>
>>
>> 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@converseincode.com>
>> Signed-off-by: Mark Charlebois <charlebm@gmail.com>
>> Reviewed-by: Jan-Simon M??ller <dl9pf@gmx.de>
> 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 at converseincode.com

WARNING: multiple messages have this Message-ID (diff)
From: Behan Webster <behanw@converseincode.com>
To: "Andreas Färber" <afaerber@suse.de>,
	ard.biesheuvel@linaro.org, catalin.marinas@arm.com,
	k.khlebnikov@samsung.com, olof@lixom.net, peterz@infradead.org,
	takahiro.akashi@linaro.org, tglx@linutronix.de,
	will.deacon@arm.com
Cc: linux-kernel@vger.kernel.org,
	Mark Charlebois <charlebm@gmail.com>,
	dwmw2@infradead.org, mingo@kernel.org,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 3/4] arm64: LLVMLinux: Calculate current_thread_info from current_stack_pointer
Date: Thu, 31 Jul 2014 09:04:25 -0700	[thread overview]
Message-ID: <53DA6909.1050007@converseincode.com> (raw)
In-Reply-To: <53D9D4B2.8070008@suse.de>

On 07/30/14 22:31, Andreas Färber wrote:
> Hi,
>
> Am 31.07.2014 01:57, schrieb behanw@converseincode.com:
>> From: Behan Webster <behanw@converseincode.com>
>>
>> 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@converseincode.com>
>> Signed-off-by: Mark Charlebois <charlebm@gmail.com>
>> Reviewed-by: Jan-Simon M??ller <dl9pf@gmx.de>
> 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@converseincode.com


  reply	other threads:[~2014-07-31 16:04 UTC|newest]

Thread overview: 44+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-07-30 23:57 [PATCH 0/4] LLVMLinux: Patches to enable the kernel to be compiled with clang/LLVM behanw at converseincode.com
2014-07-30 23:57 ` behanw
2014-07-30 23:57 ` [PATCH 1/4] arm64: LLVMLinux: Add current_stack_pointer() for arm64 behanw at converseincode.com
2014-07-30 23:57   ` behanw
2014-07-30 23:57 ` [PATCH 2/4] arm64: LLVMLinux: Use current_stack_pointer in save_stack_trace_tsk behanw at converseincode.com
2014-07-30 23:57   ` behanw
2014-07-30 23:57 ` [PATCH 3/4] arm64: LLVMLinux: Calculate current_thread_info from current_stack_pointer behanw at converseincode.com
2014-07-30 23:57   ` behanw
2014-07-31  5:31   ` Andreas Färber
2014-07-31  5:31     ` Andreas Färber
2014-07-31 16:04     ` Behan Webster [this message]
2014-07-31 16:04       ` Behan Webster
2014-07-30 23:57 ` [PATCH 4/4] arm64: LLVMLinux: Use current_stack_pointer in kernel/traps.c behanw at converseincode.com
2014-07-30 23:57   ` behanw
2014-07-31  0:11 ` [PATCH 0/4] LLVMLinux: Patches to enable the kernel to be compiled with clang/LLVM Olof Johansson
2014-07-31  0:11   ` Olof Johansson
2014-07-31 10:33 ` Will Deacon
2014-07-31 10:33   ` Will Deacon
2014-07-31 15:39   ` Olof Johansson
2014-07-31 15:39     ` Olof Johansson
2014-07-31 15:48     ` Will Deacon
2014-07-31 15:48       ` Will Deacon
2014-07-31 15:52       ` Olof Johansson
2014-07-31 15:52         ` Olof Johansson
2014-08-01  4:11   ` Behan Webster
2014-08-01  4:11     ` Behan Webster
2014-08-26 14:16     ` Will Deacon
2014-08-26 14:16       ` Will Deacon
2014-08-27  4:26       ` Behan Webster
2014-08-27  4:26         ` Behan Webster
2014-08-27  4:29       ` [PATCH V2 0/6] " behanw at converseincode.com
2014-08-27  4:29         ` behanw
2014-08-27  4:29         ` [PATCH V2 1/6] arm64: LLVMLinux: Add current_stack_pointer() for arm64 behanw at converseincode.com
2014-08-27  4:29           ` behanw
2014-08-27  4:29         ` [PATCH V2 2/6] arm64: LLVMLinux: Use current_stack_pointer in save_stack_trace_tsk behanw at converseincode.com
2014-08-27  4:29           ` behanw
2014-08-27  4:29         ` [PATCH V2 3/6] arm64: LLVMLinux: Calculate current_thread_info from current_stack_pointer behanw at converseincode.com
2014-08-27  4:29           ` behanw
2014-08-27  4:29         ` [PATCH V2 4/6] arm64: LLVMLinux: Use current_stack_pointer in kernel/traps.c behanw at converseincode.com
2014-08-27  4:29           ` behanw
2014-08-27  4:29         ` [PATCH V2 5/6] arm64: LLVMLinux: Use global stack register variable for aarch64 behanw at converseincode.com
2014-08-27  4:29           ` behanw
2014-08-27  4:29         ` [PATCH V2 6/6] arm64: LLVMLinux: Use global stack pointer in return_address() behanw at converseincode.com
2014-08-27  4:29           ` behanw

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=53DA6909.1050007@converseincode.com \
    --to=behanw@converseincode.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.