lkml.org 
[lkml]   [2012]   [Mar]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
SubjectRe: [PATCH] Disintegrate asm/system.h for Unicore32 [based on ver #3, changed by gxt]
Date
Guan Xuetao <gxt@mprc.pku.edu.cn> wrote:

> Disintegrate asm/system.h for Unicore32. (Compilation successful)
> The implementation details are not changed, but only splitted.
> BTW, some codestyles are adjusted.
>
> Signed-off-by: David Howells <dhowells@redhat.com>
> Signed-off-by: Guan Xuetao <gxt@mprc.pku.edu.cn>

Aha! Much better. It mostly looks reasonable, apart from one thing. See the
attached patch, which I'll roll in if you're happy with it.

David
---
From: David Howells <dhowells@redhat.com>
Subject: [PATCH] Fix unicore xchg() to fail at build time with an unsupported pointer

Fix unicore xchg() to fail at build time with an unsupported pointer rather
then using panic().

Additionally, panic() should not be redeclared in this manner, but rather
should be brought in by #inclusion of <linux/kernel.h>.

Signed-off-by: David Howells <dhowells@redhat.com>
---

arch/unicore32/include/asm/cmpxchg.h | 10 +++++++---
1 files changed, 7 insertions(+), 3 deletions(-)


diff --git a/arch/unicore32/include/asm/cmpxchg.h b/arch/unicore32/include/asm/cmpxchg.h
index 37c431d..f39a59d 100644
--- a/arch/unicore32/include/asm/cmpxchg.h
+++ b/arch/unicore32/include/asm/cmpxchg.h
@@ -10,7 +10,11 @@
#ifndef __UNICORE_CMPXCHG_H__
#define __UNICORE_CMPXCHG_H__

-extern void panic(const char *fmt, ...);
+/*
+ * Generate a link failure on undefined symbol if the pointer points to a value
+ * of unsupported size.
+ */
+extern void __xchg_bad_pointer(void)

static inline unsigned long __xchg(unsigned long x, volatile void *ptr,
int size)
@@ -31,8 +35,8 @@ static inline unsigned long __xchg(unsigned long x, volatile void *ptr,
: "memory", "cc");
break;
default:
- panic("xchg: bad data size: ptr 0x%p, size %d\n",
- ptr, size);
+ __xchg_bad_pointer();
+ break;
}

return ret;


\
 
 \ /
  Last update: 2012-03-20 10:37    [W:0.036 / U:1.224 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site