[<prev] [next>] [day] [month] [year] [list]
Message-Id: <20090924012756.69d858e2.akpm@linux-foundation.org>
Date: Thu, 24 Sep 2009 01:27:56 -0700
From: Andrew Morton <akpm@...ux-foundation.org>
To: Michal Simek <michal.simek@...alogix.com>
Cc: stefani@...bold.net,
Linus Torvalds <torvalds@...ux-foundation.org>,
Linux Kernel list <linux-kernel@...r.kernel.org>,
Ingo Molnar <mingo@...e.hu>,
John Williams <john.williams@...alogix.com>
Subject: Re: Microblaze noMMU compilation fault - mm/Makefile
On Thu, 24 Sep 2009 10:12:20 +0200 Michal Simek <michal.simek@...alogix.com> wrote:
> 2009/9/24 Andrew Morton <akpm@...ux-foundation.org>
>
> > On Thu, 24 Sep 2009 09:51:14 +0200 Michal Simek <
> > michal.simek@...alogix.com> wrote:
> >
> > > Hi Stefani,
> > >
> > > Commit d899bf7b55f503ba7d3d07ed27c3a37e270fa7db break Microblaze noMMU
> > kernel
> > >
> > > There is problem that you setup compilation pagewalk.c which is MMU
> > specific.
> > >
> > > And Andrew add there this comment:
> > > [akpm@...ux-foundation.org: fs/proc/array.c now needs walk_page_range()]
> > >
> > > that mean that the second problem is with compilation of proc/array.c for
> > noMMU.
> > >
> >
> > I don't understand.
> >
> > Please send us a copy of the compiler error messages.
> >
> >
> http://www.monstr.eu/wiki/doku.php?id=log:2009-09-24_09_51_35
>
> CC mm/pagewalk.o
> mm/pagewalk.c: In function 'walk_pte_range':
> mm/pagewalk.c:11: error: implicit declaration of function 'pte_offset_map'
> mm/pagewalk.c:11: warning: assignment makes pointer from integer without a cast
> mm/pagewalk.c:22: error: implicit declaration of function 'pte_unmap'
> mm/pagewalk.c: In function 'walk_pmd_range':
> mm/pagewalk.c:35: error: implicit declaration of function 'pmd_addr_end'
> mm/pagewalk.c:36: error: implicit declaration of function
> 'pmd_none_or_clear_bad'
> mm/pagewalk.c: In function 'walk_pud_range':
> mm/pagewalk.c:61: error: implicit declaration of function 'pud_offset'
> mm/pagewalk.c:61: warning: assignment makes pointer from integer without a cast
> mm/pagewalk.c:63: error: implicit declaration of function 'pud_addr_end'
> mm/pagewalk.c:64: error: implicit declaration of function
> 'pud_none_or_clear_bad'
> mm/pagewalk.c: In function 'walk_page_range':
> mm/pagewalk.c:117: error: implicit declaration of function 'pgd_offset'
> mm/pagewalk.c:117: warning: assignment makes pointer from integer without a cast
> mm/pagewalk.c:119: error: implicit declaration of function 'pgd_addr_end'
> mm/pagewalk.c:120: error: implicit declaration of function
> 'pgd_none_or_clear_bad'
> make[1]: *** [mm/pagewalk.o] Error 1
> make[1]: *** Waiting for unfinished jobs....
>
OK, I wonder how that snuck through.
I suppose the feature isn't going to work at all on mmuless systems.
Does this work OK?
fs/proc/array.c | 7 +++++++
mm/Makefile | 4 ++--
2 files changed, 9 insertions(+), 2 deletions(-)
diff -puN fs/proc/array.c~a fs/proc/array.c
--- a/fs/proc/array.c~a
+++ a/fs/proc/array.c
@@ -322,6 +322,8 @@ static inline void task_context_switch_c
p->nivcsw);
}
+#ifdef CONFIG_MMU
+
struct stack_stats {
struct vm_area_struct *vma;
unsigned long startpage;
@@ -402,6 +404,11 @@ static inline void task_show_stack_usage
mmput(mm);
}
}
+#else
+static void task_show_stack_usage(struct seq_file *m, struct task_struct *task)
+{
+}
+#endif /* CONFIG_MMU */
int proc_pid_status(struct seq_file *m, struct pid_namespace *ns,
struct pid *pid, struct task_struct *task)
diff -puN mm/Makefile~a mm/Makefile
--- a/mm/Makefile~a
+++ a/mm/Makefile
@@ -5,14 +5,14 @@
mmu-y := nommu.o
mmu-$(CONFIG_MMU) := fremap.o highmem.o madvise.o memory.o mincore.o \
mlock.o mmap.o mprotect.o mremap.o msync.o rmap.o \
- vmalloc.o
+ vmalloc.o pagewalk.o
obj-y := bootmem.o filemap.o mempool.o oom_kill.o fadvise.o \
maccess.o page_alloc.o page-writeback.o \
readahead.o swap.o truncate.o vmscan.o shmem.o \
prio_tree.o util.o mmzone.o vmstat.o backing-dev.o \
page_isolation.o mm_init.o mmu_context.o \
- pagewalk.o $(mmu-y)
+ $(mmu-y)
obj-y += init-mm.o
obj-$(CONFIG_BOUNCE) += bounce.o
_
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists