[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [Cluster-devel] Cluster Project branch, master, updated. gfs-kernel_0_1_22-156-gd2a926d




Hi Abhijith,

On Fri, 11 Apr 2008, adas sourceware org wrote:

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Cluster Project".

http://sources.redhat.com/git/gitweb.cgi?p=cluster.git;a=commitdiff;h=d2a926d2122c23e6175a62326b5e2b421b842a93

The branch, master has been updated
      via  d2a926d2122c23e6175a62326b5e2b421b842a93 (commit)
     from  56e13f474bc8abf02324e294462527755f8752f4 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit d2a926d2122c23e6175a62326b5e2b421b842a93
Author: Abhijith Das <adas redhat com>
Date:   Fri Apr 11 09:10:47 2008 -0500

   gfs2_tool: Fix build warnings in misc.c bz 441636

   gfs2_tool used to include both linux/fs.h and sys/mount.h that caused some symbols to be defined twice and hence caused some build warnings. This patch uses linux/ext3_fs.h instead of /linux/fs.h and uses EXT3_XXX_FL inode flags instead of the respective FS_XXX_FL flags.
   This patch also removes the SYSTEM and DIRECTIO flags as they are not used anymore.

-----------------------------------------------------------------------

Summary of changes:
gfs2/tool/misc.c |   28 +++++++++++-----------------
1 files changed, 11 insertions(+), 17 deletions(-)

[SNIP]

It looks like we will need to find another way to fix those build warnings because with latest kernel from git I get the following errors:

In file included from /usr/src/upstream/cluster/rhcluster/cluster/gfs2/tool/misc.c:32: /usr/src/upstream/kernel/linux-2.6/include/linux/ext3_fs.h: In function ext3_rec_len_from_disk: /usr/src/upstream/kernel/linux-2.6/include/linux/ext3_fs.h:663: warning: implicit declaration of function le16_to_cpu /usr/src/upstream/kernel/linux-2.6/include/linux/ext3_fs.h: In function ext3_rec_len_to_disk: /usr/src/upstream/kernel/linux-2.6/include/linux/ext3_fs.h:673: warning: implicit declaration of function cpu_to_le16 /usr/src/upstream/kernel/linux-2.6/include/linux/ext3_fs.h:675: warning: implicit declaration of function BUG /usr/src/upstream/cluster/rhcluster/cluster/gfs2/tool/misc.c: In function print_lockdump: /usr/src/upstream/cluster/rhcluster/cluster/gfs2/tool/misc.c:155: warning: format not a string literal and no format arguments /usr/src/upstream/cluster/rhcluster/cluster/gfs2/tool/misc.c: In function set_flag: /usr/src/upstream/cluster/rhcluster/cluster/gfs2/tool/misc.c:273: error: FS_IOC_GETFLAGS undeclared (first use in this function) /usr/src/upstream/cluster/rhcluster/cluster/gfs2/tool/misc.c:273: error: (Each undeclared identifier is reported only once /usr/src/upstream/cluster/rhcluster/cluster/gfs2/tool/misc.c:273: error: for each function it appears in.) /usr/src/upstream/cluster/rhcluster/cluster/gfs2/tool/misc.c:279: error: FS_IOC_SETFLAGS undeclared (first use in this function)
make: *** [misc.o] Error 1

and FS_IOC_GETFLAGS and SETFLAGS are defined only linux/fs.h so we are back the original problem.

Fabio

--
I'm going to make him an offer he can't refuse.


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]