From: Andrew Morton <akpm@linux-foundation.org>
To: Pekka Enberg <penberg@cs.helsinki.fi>, Ingo Molnar <mingo@elte.hu>
Cc: linux-mm@kvack.org
Subject: git-slab plus git-tip breaks i386 allnoconfig
Date: Thu, 9 Oct 2008 16:47:00 -0700 [thread overview]
Message-ID: <20081009164700.c9042902.akpm@linux-foundation.org> (raw)
In file included from include/linux/rcupdate.h:39,
from include/linux/marker.h:16,
from include/linux/kmemtrace.h:13,
from include/linux/slub_def.h:13,
from include/linux/slab.h:184,
from include/asm/pgtable_32.h:21,
from include/asm/pgtable.h:394,
from include/linux/mm.h:40,
from arch/x86/mm/pgtable.c:1:
include/linux/percpu.h: In function '__percpu_alloc_mask':
include/linux/percpu.h:108: error: implicit declaration of function 'kzalloc'
include/linux/percpu.h:108: warning: return makes pointer from integer without a cast
In file included from include/asm/pgtable_32.h:21,
from include/asm/pgtable.h:394,
from include/linux/mm.h:40,
from arch/x86/mm/pgtable.c:1:
include/linux/slab.h: At top level:
include/linux/slab.h:336: error: conflicting types for 'kzalloc'
include/linux/percpu.h:108: error: previous implicit declaration of 'kzalloc' was here
In file included from include/linux/rcupdate.h:39,
but that file includes slab.h, so I suspect we have some recursive
include snafu somewhere which caused the slab.h inclusion to get
skipped.
Shudder. I'll locally use this notapatch:
--- a/include/linux/percpu.h~a
+++ a/include/linux/percpu.h
@@ -103,15 +103,12 @@ extern void percpu_free(void *__pdata);
#define percpu_ptr(ptr, cpu) ({ (void)(cpu); (ptr); })
-static __always_inline void *__percpu_alloc_mask(size_t size, gfp_t gfp, cpumask_t *mask)
-{
- return kzalloc(size, gfp);
-}
-
-static inline void percpu_free(void *__pdata)
-{
- kfree(__pdata);
-}
+/*
+ * __percpu_alloc_mask() and percpu_free() are macros to simplify header
+ * dependencies
+ */
+#define __percpu_alloc_mask(size, gfp, maskp) kzalloc(size, gfp)
+#define percpu_free(pdata) kfree(pdata)
#endif /* CONFIG_SMP */
Whoever merges second gets to fix this for real - have fun ;)
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org. For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
next reply other threads:[~2008-10-09 23:47 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-10-09 23:47 Andrew Morton [this message]
2008-10-10 0:03 ` Andrew Morton
2008-10-10 6:45 ` Pekka Enberg
2008-10-10 7:18 ` Mathieu Desnoyers
2008-10-10 7:23 ` Ingo Molnar
2008-10-10 7:37 ` [PATCH] Markers : revert synchronize marker unregister static inline Mathieu Desnoyers
2008-10-10 7:43 ` Pekka Enberg
2008-10-10 7:48 ` [PATCH] Markers : revert synchronize marker unregister static inline (update) Mathieu Desnoyers
2008-10-10 8:36 ` Ingo Molnar
2008-10-10 9:06 ` [PATCH] Markers : revert synchronize marker unregister static inline Ingo Molnar
2008-10-10 7:44 ` git-slab plus git-tip breaks i386 allnoconfig Mathieu Desnoyers
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=20081009164700.c9042902.akpm@linux-foundation.org \
--to=akpm@linux-foundation.org \
--cc=linux-mm@kvack.org \
--cc=mingo@elte.hu \
--cc=penberg@cs.helsinki.fi \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox