linux-kernel - Re: Cant compile 2.6.26-rc5-mm2 kernel

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20080612171919.GD12367@duck.suse.cz>
Date:	Thu, 12 Jun 2008 19:19:19 +0200
From:	Jan Kara <jack@...e.cz>
To:	Vegard Nossum <vegard.nossum@...il.com>
Cc:	arjenpool@...il.com, Andrew Morton <akpm@...ux-foundation.org>,
	linux-kernel@...r.kernel.org
Subject: Re: Cant compile 2.6.26-rc5-mm2 kernel

On Wed 11-06-08 22:03:42, Vegard Nossum wrote:
> Hi,
> 
> Thanks for the report.
> 
> On Wed, Jun 11, 2008 at 8:08 PM,  <arjenpool@...il.com> wrote:
> > Today i`ve tried to compile a linux-2.6.26-rc5-mm2 kernel obtained with the
> > Ketchup tool.
> >
> > It got halfway interupted with the following error messages:
> >
> > CC [M] fs/xfs/support/debug.o
> >  CC [M] fs/xfs/support/uuid.o
> >  LD [M] fs/xfs/xfs.o
> >  CC fs/quota.o
> > fs/quota.c: In function 'quota_sync_sb':
> > fs/quota.c:174: error: implicit declaration of function 'sb_dqopt'
> 
> It seems that this definition is needed regardless of CONFIG_QUOTA.
> 
> I'm attaching a patch that fixes it for me. I'm not 100% it's the right
> thing.
  Yes, it is the right thing. I forgot to test compilation with XFS and
quotas off... Thanks for fixing this.
  You can add: Acked-by: Jan Kara <jack@...e.cz>

									Honza
 
> From 5fc276129d8a3e2a70980104cc374bdc0d61afa3 Mon Sep 17 00:00:00 2001
> From: Vegard Nossum <vegard.nossum@...il.com>
> Date: Wed, 11 Jun 2008 21:58:32 +0200
> Subject: [PATCH] fs: fix build breakage
> 
> The patch
> 
> | commit 9651a656c9fbf3e8183b5eb21740edc1eb9ca16b
> | Author: Jan Kara <jack@...e.cz>
> | Date:   Thu Jun 5 19:53:42 2008 +0000
> |
> |     quota-move-function-macros-from-quotah-to-quotaopsh
> |
> 
> causes the build to break for CONFIG_QUOTA=n.
> 
> Reported-by: Arjen Pool <arjenpool@...il.com>
> Cc: Jan Kara <jack@...e.cz>
> Signed-off-by: Vegard Nossum <vegard.nossum@...il.com>
> ---
>  include/linux/quotaops.h |   10 +++++-----
>  1 files changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/include/linux/quotaops.h b/include/linux/quotaops.h
> index 5644280..eb575d1 100644
> --- a/include/linux/quotaops.h
> +++ b/include/linux/quotaops.h
> @@ -14,6 +14,11 @@
>  
>  #include <linux/fs.h>
>  
> +static inline struct quota_info *sb_dqopt(struct super_block *sb)
> +{
> +	return &sb->s_dquot;
> +}
> +
>  #if defined(CONFIG_QUOTA)
>  
>  /*
> @@ -52,11 +57,6 @@ void vfs_dq_drop(struct inode *inode);
>  int vfs_dq_transfer(struct inode *inode, struct iattr *iattr);
>  int vfs_dq_quota_on_remount(struct super_block *sb);
>  
> -static inline struct quota_info *sb_dqopt(struct super_block *sb)
> -{
> -	return &sb->s_dquot;
> -}
> -
>  static inline struct mem_dqinfo *sb_dqinfo(struct super_block *sb, int type)
>  {
>  	return sb_dqopt(sb)->info + type;
> -- 
> 1.5.4.1
> 
-- 
Jan Kara <jack@...e.cz>
SUSE Labs, CR
--
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/