[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <Pine.LNX.4.64.1009281241450.16237@cobra.newdream.net>
Date: Tue, 28 Sep 2010 12:42:10 -0700 (PDT)
From: Sage Weil <sage@...dream.net>
To: Randy Dunlap <randy.dunlap@...cle.com>
cc: Stephen Rothwell <sfr@...b.auug.org.au>,
linux-next@...r.kernel.org, LKML <linux-kernel@...r.kernel.org>,
ceph-devel@...r.kernel.org
Subject: Re: [PATCH -next] ceph: fix debugfs warnings
Thanks, Randy. This is in my for-next branch now!
sage
On Tue, 28 Sep 2010, Randy Dunlap wrote:
> From: Randy Dunlap <randy.dunlap@...cle.com>
>
> Include "super.h" outside of CONFIG_DEBUG_FS to eliminate a compiler warning:
>
> fs/ceph/debugfs.c:266: warning: 'struct ceph_fs_client' declared inside parameter list
> fs/ceph/debugfs.c:266: warning: its scope is only this definition or declaration, which is probably not what you want
> fs/ceph/debugfs.c:271: warning: 'struct ceph_fs_client' declared inside parameter list
>
> Signed-off-by: Randy Dunlap <randy.dunlap@...cle.com>
> ---
> fs/ceph/debugfs.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> --- linux-next-20100927.orig/fs/ceph/debugfs.c
> +++ linux-next-20100927/fs/ceph/debugfs.c
> @@ -12,9 +12,10 @@
> #include <linux/ceph/auth.h>
> #include <linux/ceph/debugfs.h>
>
> +#include "super.h"
> +
> #ifdef CONFIG_DEBUG_FS
>
> -#include "super.h"
> #include "mds_client.h"
>
> static int mdsmap_show(struct seq_file *s, void *p)
> --
> To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
> the body of a message to majordomo@...r.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
>
--
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