Re: aufs3: remove include of linux/fs.h from linux/mm.h

2014-07-21 Thread sfjro
Ian Campbell: > There were some other errors in the original buildd log at > http://buildd.debian-ports.org/status/fetch.php?pkg=linux&arch=arm64&ver=3.14.12-1&stamp=1405234443 > but they were resolved as part of this patch too so I didn't investigate > much further. Some of them were missing pr

Re: aufs3: remove include of linux/fs.h from linux/mm.h

2014-07-21 Thread Ian Campbell
On Mon, 2014-07-21 at 16:01 +0900, sf...@users.sourceforge.net wrote: > Ian Campbell: > > In file included from /=ABPKGBUILDDIR=BB/include/linux/mm.h:23:0, > > from /=ABPKGBUILDDIR=BB/include/linux/pid_namespace.h:6, > > from /=ABPKGBUILDDIR=BB/include/linux/ptrace

Re: aufs3: remove include of linux/fs.h from linux/mm.h

2014-07-21 Thread sfjro
Ian Campbell: > In file included from /=ABPKGBUILDDIR=BB/include/linux/mm.h:23:0, > from /=ABPKGBUILDDIR=BB/include/linux/pid_namespace.h:6, > from /=ABPKGBUILDDIR=BB/include/linux/ptrace.h:9, > from /=ABPKGBUILDDIR=BB/arch/arm64/include/asm/compa

Re: aufs3: remove include of linux/fs.h from linux/mm.h

2014-07-20 Thread sfjro
> Re-reading several makefiles, > - fs/proc/Makefile builds nommu.o and task_nommu.o unconditionally. > - mm/Makeilfe builds nommu.o unconditionally. > > So I change my mind and stop adding ifndef CONFIG_MMU. Oops! I was wrong. Sorry. Please ignore my previous mail. J. R. Okajima -- To UNSUB

Re: aufs3: remove include of linux/fs.h from linux/mm.h

2014-07-20 Thread sfjro
Ian Campbell: > > #ifndef CONFIG_MMU was less important since the built kernel doesn't > > contain the functions defined in the header as long as they are unused. > > But by moving them into a source file, the built kernel will contain > > them. So ifndef CONFIG_MMU is necessary this time. > > Yes

Re: aufs3: remove include of linux/fs.h from linux/mm.h

2014-07-20 Thread Ian Campbell
On Mon, 2014-07-21 at 04:47 +0900, sf...@users.sourceforge.net wrote: > Ian Campbell: > > The following patch which does what I think you are suggesting works OK > > for me too. > > Thanks for the patch. > #ifndef CONFIG_MMU was less important since the built kernel doesn't > contain the functions

Re: aufs3: remove include of linux/fs.h from linux/mm.h

2014-07-20 Thread sfjro
Ian Campbell: > The following patch which does what I think you are suggesting works OK > for me too. Thanks for the patch. #ifndef CONFIG_MMU was less important since the built kernel doesn't contain the functions defined in the header as long as they are unused. But by moving them into a source

Re: aufs3: remove include of linux/fs.h from linux/mm.h

2014-07-20 Thread Ian Campbell
; > That sounds like a plausible plan to me. The following patch which does what I think you are suggesting works OK for me too. 8<-- From: Ian Campbell Subject: aufs3: remove include of linux/fs.h from linux/mm.h This include is added by aufs3-mmap.patch but causes cir

Re: aufs3: remove include of linux/fs.h from linux/mm.h

2014-07-19 Thread Ian Campbell
On Sun, 2014-07-20 at 02:38 +0900, sf...@users.sourceforge.net wrote: > Hello Ian, > > Ian Campbell: > > This include is added by aufs3-mmap.patch but causes circular > > dependencies on arm64 as seen with the Debian kernel packages in > ::: > > According to http://article.gmane.org/gmane.li

Re: aufs3: remove include of linux/fs.h from linux/mm.h

2014-07-19 Thread sfjro
Hello Ian, Ian Campbell: > This include is added by aufs3-mmap.patch but causes circular > dependencies on arm64 as seen with the Debian kernel packages in ::: > According to http://article.gmane.org/gmane.linux.ports.arm.kernel/342042 > > The added mm.h->fs.h looks like a mistake, it sho

aufs3: remove include of linux/fs.h from linux/mm.h

2014-07-19 Thread Ian Campbell
(resending since original post was rejected by aufs-users) This include is added by aufs3-mmap.patch but causes circular dependencies on arm64 as seen with the Debian kernel packages in http://buildd.debian-ports.org/status/fetch.php?pkg=linux&arch=arm64&ver=3.14.12-1&stamp=1405234443 which conta

aufs3: remove include of linux/fs.h from linux/mm.h

2014-07-19 Thread Ian Campbell
This include is added by aufs3-mmap.patch but causes circular dependencies on arm64 as seen with the Debian kernel packages in http://buildd.debian-ports.org/status/fetch.php?pkg=linux&arch=arm64&ver=3.14.12-1&stamp=1405234443 which contains: In file included from /«PKGBUILDDIR»/include/linux/mm.