From: WANG Cong <xiyou.wangcong@gmail.com>
To: Greg KH <greg@kroah.com>
Cc: Rob Landley <rob@landley.net>,
linux-kernel@vger.kernel.org, Jeff Dike <jdike@addtoit.com>,
akpm@linux-foundation.org, Chris Wright <chrisw@sous-sol.org>
Subject: [Patch] Fix UML broken (was Re: User Mode Linux still broken in 2.6.23.1)
Date: Fri, 16 Nov 2007 11:08:32 +0800 [thread overview]
Message-ID: <20071116030832.GB2563@hacking> (raw)
In-Reply-To: <20071115210608.GA12213@kroah.com>
On Thu, Nov 15, 2007 at 01:06:08PM -0800, Greg KH wrote:
>On Thu, Nov 15, 2007 at 02:57:21PM -0600, Rob Landley wrote:
>> On Thursday 15 November 2007 00:02:55 Greg KH wrote:
>> > On Wed, Nov 14, 2007 at 11:58:15PM -0600, Rob Landley wrote:
>> > > On Wednesday 14 November 2007 12:54:44 Greg KH wrote:
>> > > > On Sat, Nov 03, 2007 at 11:51:50PM -0500, Rob Landley wrote:
>> > > > > Building with the attached .config on x86-64, it does this:
>> > > > >
>> > > > > CC arch/um/kernel/smp.o
>> > > > > In file included from include/asm/arch/tlb.h:11,
>> > > > > from include/asm/tlb.h:4,
>> > > > > from arch/um/kernel/smp.c:8:
>> > > > > include/asm-generic/tlb.h: In function ???tlb_flush_mmu???:
>> > > > > include/asm-generic/tlb.h:76: error: implicit declaration of function
>> > > > > ???release_pages??? include/asm-generic/tlb.h: In function
>> > > > > ???tlb_remove_page???:
>> > > > > include/asm-generic/tlb.h:105: error: implicit declaration of
>> > > > > function ???page_cache_release??? make[1]: *** [arch/um/kernel/smp.o]
>> > > > > Error 1 make: *** [arch/um/kernel] Error 2
>> > > > >
>> > > > > I've been doing the following to fix it. I know it's not the right
>> > > > > fix, (see the earlier thread about it at
>> > > > > http://lkml.org/lkml/2007/8/24/441 ) but could the one line fix go
>> > > > > into the -stable queue 2.6.23 while a proper fix goes into 2.6.24?
>> > > >
>> > > > I think the patches that I have just added to the stable queue for
>> > > > 2.6.23.2 will fix this. If not, please let me know after testing.
>> > >
>> > > Where do I find these patches to test? I know where to find the stable
>> > > releases, but not the "stable queue".
>> > >
>> > > Documentation/stable_kernel_rules.txt just says there _is_ a stable
>> > > queue, not where to access it. Google's first hit for "linux stable
>> > > queue" was
>> > > http://git.kernel.org/?p=linux/kernel/git/chrisw/stable-queue.git;a=short
>> > >log which apparently stopped updating in march...
>> > >
>> > > Happy to test the patch you mentioned, if I can figure out where to find
>> > > it...
>> >
>> > It's at:
>> > http://git.kernel.org/?p=linux/kernel/git/stable/stable-queue.git;a=summar
>> >y
>> >
>> > Chris and I used to have separate queues, but that got messy, I suppose
>> > we should just delete those old copies...
>>
>> In a freshly extracted 2.6.23 tarball, I applied the 4 uml-*.patch files from
>> stable-queue/review-2.6.23-2 in sequence (all applied cleanly), and then did
>> the following:
>>
>> cat > mini.conf << EOF &&
>> CONFIG_BINFMT_ELF=y
>> CONFIG_HOSTFS=y
>> CONFIG_LBD=y
>> CONFIG_BLK_DEV=y
>> CONFIG_BLK_DEV_LOOP=y
>> CONFIG_STDERR_CONSOLE=y
>> CONFIG_UNIX98_PTYS=y
>> CONFIG_EXT2_FS=y
>> EOF
>> make ARCH=um allnoconfig KCONFIG_ALLCONFIG=mini.conf
>>
>> Then I did:
>>
>> make -j 2 ARCH=um
>>
>> And it still breaks:
>>
>> CC arch/um/kernel/smp.o
>> In file included from include/asm/arch/tlb.h:11,
>> from include/asm/tlb.h:4,
>> from arch/um/kernel/smp.c:8:
>> include/asm-generic/tlb.h: In function ???tlb_flush_mmu???:
>> include/asm-generic/tlb.h:76: error: implicit declaration of
>> function ???release_pages???
>> include/asm-generic/tlb.h: In function ???tlb_remove_page???:
>> include/asm-generic/tlb.h:105: error: implicit declaration of
>> function ???page_cache_release???
>> make[1]: *** [arch/um/kernel/smp.o] Error 1
>> make: *** [arch/um/kernel] Error 2
>> make: *** Waiting for unfinished jobs....
>>
>> My .config is attached.
>
>Hm, not good. I'll let Jeff handle this :)
Could you please try this patch? Can it fix the error?
Signed-off-by: WANG Cong <xiyou.wangcong@gmail.com>
---
diff --git a/Makefile b/Makefile
diff --git a/include/linux/swap.h b/include/linux/swap.h
index 4f3838a..2c3ce4c 100644
--- a/include/linux/swap.h
+++ b/include/linux/swap.h
@@ -6,6 +6,7 @@
#include <linux/mmzone.h>
#include <linux/list.h>
#include <linux/sched.h>
+#include <linux/pagemap.h>
#include <asm/atomic.h>
#include <asm/page.h>
next prev parent reply other threads:[~2007-11-16 3:11 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-11-04 4:51 User Mode Linux still broken in 2.6.23.1 Rob Landley
2007-11-14 18:54 ` Greg KH
2007-11-15 5:58 ` Rob Landley
2007-11-15 6:02 ` Greg KH
2007-11-15 20:57 ` Rob Landley
2007-11-15 21:06 ` Greg KH
2007-11-16 3:08 ` WANG Cong [this message]
2007-11-16 5:01 ` [Patch] Fix UML broken (was Re: User Mode Linux still broken in 2.6.23.1) Rob Landley
2007-11-16 7:15 ` Greg KH
2007-11-16 15:51 ` Jeff Dike
2007-11-16 16:53 ` Greg KH
2007-11-16 22:00 ` Rob Landley
2007-11-17 18:53 ` Jeff Dike
2007-11-18 20:17 ` Rob Landley
2007-11-18 21:32 ` Sam Ravnborg
2007-11-19 7:14 ` Rob Landley
2007-11-16 18:00 ` Rob Landley
2007-11-16 15:12 ` Jeff Dike
2007-11-16 15:26 ` WANG Cong
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=20071116030832.GB2563@hacking \
--to=xiyou.wangcong@gmail.com \
--cc=akpm@linux-foundation.org \
--cc=chrisw@sous-sol.org \
--cc=greg@kroah.com \
--cc=jdike@addtoit.com \
--cc=linux-kernel@vger.kernel.org \
--cc=rob@landley.net \
/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