lkml.org 
[lkml]   [2008]   [May]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH 5/67] aufs Makefile
    Hi Junjiro.

    A few comments to your Makefile.

    On Fri, May 16, 2008 at 11:32:19PM +0900, hooanon05@yahoo.co.jp wrote:
    > From: Junjiro Okajima <hooanon05@yahoo.co.jp>
    >
    > initial commit
    > aufs Makefile
    >
    > Signed-off-by: Junjiro Okajima <hooanon05@yahoo.co.jp>
    > ---
    > fs/aufs/Makefile | 57 ++++++++++++++++++++++++++++++++++++++++++++++++++++++
    > 1 files changed, 57 insertions(+), 0 deletions(-)
    >
    > diff --git a/fs/aufs/Makefile b/fs/aufs/Makefile
    > new file mode 100644
    > index 0000000..05e0864
    > --- /dev/null
    > +++ b/fs/aufs/Makefile
    > @@ -0,0 +1,57 @@
    > +# AUFS Makefile for the Linux 2.6.25 and later
    > +# $Id: Makefile,v 1.3 2008/04/28 03:03:10 sfjro Exp $
    Please drop these two lines

    > +
    > +# the environment variables are not inherited since 2.6.23
    > +ifdef AUFS_EXTRA_CFLAGS
    > +EXTRA_CFLAGS += ${AUFS_EXTRA_CFLAGS}
    > +endif
    And this as well. If this is needed do like the others do.

    > +
    > +########################################
    > +
    > +ifdef CONFIG_AUFS_WORKAROUND_FUSE
    > +# defined in ${srctree}/fs/fuse/inode.c
    > +EXTRA_CFLAGS += -DFUSE_SUPER_MAGIC=0x65735546
    > +endif
    Use:
    ccflags-$(CONFIG_AUFS_WORKAROUND_FUSE) += -DFUSE_SUPER_MAGIC=0x65735546

    > +
    > +ifdef CONFIG_XFS_FS
    > +# defined in ${srctree}/fs/xfs/xfs_sb.h
    > +EXTRA_CFLAGS += -DXFS_SB_MAGIC=0x58465342
    > +endif
    ccflags-$(CONFIG_XFS_FS) += -DXFS_SB_MAGIC=0x58465342

    > +
    > +ifdef CONFIG_TMPFS
    > +# defined in ${srctree}/mm/shmem.c
    > +EXTRA_CFLAGS += -DTMPFS_MAGIC=0x01021994
    > +endif
    ccflags-$(CONFIG_TMPFS) += -DTMPFS_MAGIC=0x01021994

    > +
    > +-include $(dir $(lastword ${MAKEFILE_LIST}))priv.mk

    Please drop this.

    > +#$(warning ${EXTRA_CFLAGS})
    Not used - drop it.

    > +
    > +########################################
    > +
    > +obj-$(CONFIG_AUFS) += aufs.o
    > +aufs-y := module.o super.o sbinfo.o branch.o xino.o sysaufs.o opts.o \
    > + wkq.o vfsub.o dcsub.o \
    > + cpup.o whout.o plink.o wbr_policy.o \
    > + dentry.o dinfo.o \
    > + file.o f_op.o finfo.o \
    > + dir.o vdir.o \
    > + inode.o i_op.o i_op_add.o i_op_del.o i_op_ren.o iinfo.o \
    > + misc.o
    > +
    > +#xattr.o
    > +aufs-$(CONFIG_SYSFS) += sysfs.o
    > +aufs-$(CONFIG_AUFS_BR_NFS) += br_nfs.o
    > +aufs-$(CONFIG_AUFS_BR_XFS) += br_xfs.o
    > +aufs-$(CONFIG_AUFS_WORKAROUND_FUSE) += br_fuse.o
    > +
    > +aufs-$(CONFIG_AUFS_DLGT) += dlgt.o
    > +aufs-$(CONFIG_AUFS_HINOTIFY) += hinotify.o hin_or_dlgt.o

    > +# dirty
    > +ifndef CONFIG_AUFS_HINOTIFY
    > +aufs-$(CONFIG_AUFS_DLGT) += hin_or_dlgt.o
    > +endif
    Can we have this dependency expressed in your Kconfig file?

    > +
    > +aufs-$(CONFIG_AUFS_EXPORT) += export.o
    > +aufs-$(CONFIG_AUFS_ROBR) += robr.o
    > +aufs-$(CONFIG_AUFS_DEBUG) += debug.o
    > +aufs-$(CONFIG_AUFS_MAGIC_SYSRQ) += sysrq.o


    Sam


    \
     
     \ /
      Last update: 2008-05-16 19:27    [W:4.077 / U:0.148 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site