James Hogan writes:
> On 14/03/13 04:00, Rusty Russell wrote:
>> From: Rusty Russell
>> Subject: CONFIG_SYMBOL_PREFIX: cleanup.
...
> Reviewed-by: James Hogan
> Tested-by: James Hogan (metag)
Thanks.
> The only other special case of symbol prefixing I'm aware of is in
>
On 14/03/13 04:00, Rusty Russell wrote:
> From: Rusty Russell
> Subject: CONFIG_SYMBOL_PREFIX: cleanup.
>
> We have CONFIG_SYMBOL_PREFIX, which three archs define to the string
> "_". But Al Viro broke this in "consolidate cond_syscall and
> SYSCALL_ALIAS declarations" (in linux-next), and he's
Sam Ravnborg writes:
>> actually, y'know, proof-read it.
> Hmm..
>> + $(KERNELRELEASE) "$(patsubst
>> y,_,$(CONFIG_HAVE_SYMBOL_PREFIX_UNDERSCORE))"
>>
>> +config HAVE_UNDERSCORE_SYMBOL_PREFIX
>
> HAVE_UNDERSCORE_... or HAVE_SYMBOL_... confusion.
> I prefer the
Sam Ravnborg s...@ravnborg.org writes:
actually, y'know, proof-read it.
Hmm..
+ $(KERNELRELEASE) $(patsubst
y,_,$(CONFIG_HAVE_SYMBOL_PREFIX_UNDERSCORE))
+config HAVE_UNDERSCORE_SYMBOL_PREFIX
HAVE_UNDERSCORE_... or HAVE_SYMBOL_... confusion.
I prefer the
On 14/03/13 04:00, Rusty Russell wrote:
From: Rusty Russell ru...@rustcorp.com.au
Subject: CONFIG_SYMBOL_PREFIX: cleanup.
We have CONFIG_SYMBOL_PREFIX, which three archs define to the string
_. But Al Viro broke this in consolidate cond_syscall and
SYSCALL_ALIAS declarations (in
James Hogan james.ho...@imgtec.com writes:
On 14/03/13 04:00, Rusty Russell wrote:
From: Rusty Russell ru...@rustcorp.com.au
Subject: CONFIG_SYMBOL_PREFIX: cleanup.
...
Reviewed-by: James Hogan james.ho...@imgtec.com
Tested-by: James Hogan james.ho...@imgtec.com (metag)
Thanks.
The only
On Wed, Mar 13, 2013 at 09:21:01AM +, James Hogan wrote:
> Hi Sam,
>
> On 13/03/13 06:31, Sam Ravnborg wrote:
> >> /* Some toolchains use a `_' prefix for all user symbols. */
> >> -#ifdef CONFIG_SYMBOL_PREFIX
> >> -#define MODULE_SYMBOL_PREFIX CONFIG_SYMBOL_PREFIX
> >> +#ifdef
Hi Sam,
On 13/03/13 06:31, Sam Ravnborg wrote:
>> /* Some toolchains use a `_' prefix for all user symbols. */
>> -#ifdef CONFIG_SYMBOL_PREFIX
>> -#define MODULE_SYMBOL_PREFIX CONFIG_SYMBOL_PREFIX
>> +#ifdef CONFIG_HAVE_UNDERSCORE_SYMBOL_PREFIX
>> +#define __VMLINUX_SYMBOL(x) _##x
>> +#define
> actually, y'know, proof-read it.
Hmm..
> + $(KERNELRELEASE) "$(patsubst
> y,_,$(CONFIG_HAVE_SYMBOL_PREFIX_UNDERSCORE))"
>
> +config HAVE_UNDERSCORE_SYMBOL_PREFIX
HAVE_UNDERSCORE_... or HAVE_SYMBOL_... confusion.
I prefer the HAVE_SYMBOL_... variant but no strong
actually, y'know, proof-read it.
Hmm..
+ $(KERNELRELEASE) $(patsubst
y,_,$(CONFIG_HAVE_SYMBOL_PREFIX_UNDERSCORE))
+config HAVE_UNDERSCORE_SYMBOL_PREFIX
HAVE_UNDERSCORE_... or HAVE_SYMBOL_... confusion.
I prefer the HAVE_SYMBOL_... variant but no strong feelings..
+ *
Hi Sam,
On 13/03/13 06:31, Sam Ravnborg wrote:
/* Some toolchains use a `_' prefix for all user symbols. */
-#ifdef CONFIG_SYMBOL_PREFIX
-#define MODULE_SYMBOL_PREFIX CONFIG_SYMBOL_PREFIX
+#ifdef CONFIG_HAVE_UNDERSCORE_SYMBOL_PREFIX
+#define __VMLINUX_SYMBOL(x) _##x
+#define
On Wed, Mar 13, 2013 at 09:21:01AM +, James Hogan wrote:
Hi Sam,
On 13/03/13 06:31, Sam Ravnborg wrote:
/* Some toolchains use a `_' prefix for all user symbols. */
-#ifdef CONFIG_SYMBOL_PREFIX
-#define MODULE_SYMBOL_PREFIX CONFIG_SYMBOL_PREFIX
+#ifdef
Stephen Rothwell writes:
> Hi Rusty,
>
> Looks partly better. You seem to be using
> CONFIG_SYMBOL_PREFIX_UNDERSCORE but selecting
> CONFIG_HAVE_UNDERSCORE_SYMBOL_PREFIX. One trivial comment below.
>
> Maybe this was an unfinished version of the patch?
Indeed. It was crap now I've
Hi Rusty,
On 12/03/13 04:48, Rusty Russell wrote:
> v2: Rename CONFIG_SYMBOL_PREFIX_UNDERSCORE to
> CONFIG_HAVE_UNDERSCORE_SYMBOL_PREFIX,
> which is defined in arch/Kconfig and selected by the 3 archs which need
> it.
Sorry I didn't get a chance to try your patch yesterday.
> Subject:
Hi Rusty,
Looks partly better. You seem to be using
CONFIG_SYMBOL_PREFIX_UNDERSCORE but selecting
CONFIG_HAVE_UNDERSCORE_SYMBOL_PREFIX. One trivial comment below.
Maybe this was an unfinished version of the patch?
On Tue, 12 Mar 2013 15:18:15 +1030 Rusty Russell wrote:
>
> diff --git
Stephen Rothwell writes:
> Hi Rusty,
>
> On Mon, 11 Mar 2013 17:05:17 +1030 Rusty Russell
> wrote:
>>
>> diff --git a/arch/blackfin/Kconfig b/arch/blackfin/Kconfig
>> index 600494c..cd0f7c7 100644
>> --- a/arch/blackfin/Kconfig
>> +++ b/arch/blackfin/Kconfig
>> @@ -1,6 +1,5 @@
>> -config
Stephen Rothwell s...@canb.auug.org.au writes:
Hi Rusty,
On Mon, 11 Mar 2013 17:05:17 +1030 Rusty Russell ru...@rustcorp.com.au
wrote:
diff --git a/arch/blackfin/Kconfig b/arch/blackfin/Kconfig
index 600494c..cd0f7c7 100644
--- a/arch/blackfin/Kconfig
+++ b/arch/blackfin/Kconfig
@@
Hi Rusty,
Looks partly better. You seem to be using
CONFIG_SYMBOL_PREFIX_UNDERSCORE but selecting
CONFIG_HAVE_UNDERSCORE_SYMBOL_PREFIX. One trivial comment below.
Maybe this was an unfinished version of the patch?
On Tue, 12 Mar 2013 15:18:15 +1030 Rusty Russell ru...@rustcorp.com.au wrote:
Hi Rusty,
On 12/03/13 04:48, Rusty Russell wrote:
v2: Rename CONFIG_SYMBOL_PREFIX_UNDERSCORE to
CONFIG_HAVE_UNDERSCORE_SYMBOL_PREFIX,
which is defined in arch/Kconfig and selected by the 3 archs which need
it.
Sorry I didn't get a chance to try your patch yesterday.
Subject:
Stephen Rothwell s...@canb.auug.org.au writes:
Hi Rusty,
Looks partly better. You seem to be using
CONFIG_SYMBOL_PREFIX_UNDERSCORE but selecting
CONFIG_HAVE_UNDERSCORE_SYMBOL_PREFIX. One trivial comment below.
Maybe this was an unfinished version of the patch?
Indeed. It was crap
Hi Rusty,
On Mon, 11 Mar 2013 17:05:17 +1030 Rusty Russell wrote:
>
> diff --git a/arch/blackfin/Kconfig b/arch/blackfin/Kconfig
> index 600494c..cd0f7c7 100644
> --- a/arch/blackfin/Kconfig
> +++ b/arch/blackfin/Kconfig
> @@ -1,6 +1,5 @@
> -config SYMBOL_PREFIX
> - string
> - default
James Hogan writes:
> Hi Rusty,
>
> On 08/03/13 00:03, Rusty Russell wrote:
>> James Hogan writes:
>>> Also the definition of SYMBOL_PREFIX in is removed as
>>> it conflicts, isn't used anywhere, and is defined as a string so differs
>>> from the assembly definition.
>>
>> So now, if
James Hogan james.ho...@imgtec.com writes:
Hi Rusty,
On 08/03/13 00:03, Rusty Russell wrote:
James Hogan james.ho...@imgtec.com writes:
Also the definition of SYMBOL_PREFIX in linux/kernel.h is removed as
it conflicts, isn't used anywhere, and is defined as a string so differs
from the
Hi Rusty,
On Mon, 11 Mar 2013 17:05:17 +1030 Rusty Russell ru...@rustcorp.com.au wrote:
diff --git a/arch/blackfin/Kconfig b/arch/blackfin/Kconfig
index 600494c..cd0f7c7 100644
--- a/arch/blackfin/Kconfig
+++ b/arch/blackfin/Kconfig
@@ -1,6 +1,5 @@
-config SYMBOL_PREFIX
- string
-
Hi Rusty,
On 08/03/13 00:03, Rusty Russell wrote:
> James Hogan writes:
>> Also the definition of SYMBOL_PREFIX in is removed as
>> it conflicts, isn't used anywhere, and is defined as a string so differs
>> from the assembly definition.
>
> So now, if CONFIG_SYMBOL_PREFIX, SYMBOL_PREFIX is
Hi Rusty,
On 08/03/13 00:03, Rusty Russell wrote:
James Hogan james.ho...@imgtec.com writes:
Also the definition of SYMBOL_PREFIX in linux/kernel.h is removed as
it conflicts, isn't used anywhere, and is defined as a string so differs
from the assembly definition.
So now, if
James Hogan writes:
> The commit "consolidate cond_syscall and SYSCALL_ALIAS declarations"
> broke the build on blackfin and metag due to the following code:
>
> #ifndef SYMBOL_NAME
> #ifdef CONFIG_SYMBOL_PREFIX
> #define SYMBOL_NAME(x) CONFIG_SYMBOL_PREFIX ## x
> #else
> #define
The commit "consolidate cond_syscall and SYSCALL_ALIAS declarations"
broke the build on blackfin and metag due to the following code:
#ifndef SYMBOL_NAME
#ifdef CONFIG_SYMBOL_PREFIX
#define SYMBOL_NAME(x) CONFIG_SYMBOL_PREFIX ## x
#else
#define SYMBOL_NAME(x) x
#endif
#endif
The commit consolidate cond_syscall and SYSCALL_ALIAS declarations
broke the build on blackfin and metag due to the following code:
#ifndef SYMBOL_NAME
#ifdef CONFIG_SYMBOL_PREFIX
#define SYMBOL_NAME(x) CONFIG_SYMBOL_PREFIX ## x
#else
#define SYMBOL_NAME(x) x
#endif
#endif
#define
James Hogan james.ho...@imgtec.com writes:
The commit consolidate cond_syscall and SYSCALL_ALIAS declarations
broke the build on blackfin and metag due to the following code:
#ifndef SYMBOL_NAME
#ifdef CONFIG_SYMBOL_PREFIX
#define SYMBOL_NAME(x) CONFIG_SYMBOL_PREFIX ## x
#else
30 matches
Mail list logo