* [PATCH] xfs: turn off sign warnings
@ 2010-01-25 5:33 Dave Chinner
2010-01-25 8:34 ` Christoph Hellwig
0 siblings, 1 reply; 2+ messages in thread
From: Dave Chinner @ 2010-01-25 5:33 UTC (permalink / raw)
To: xfs
Because they cause warnings in static inline functions conditionally
compiled into XFS from the VFS (e.g. fsnotify).
Signed-off-by: Dave Chinner <david@fromorbit.com>
---
fs/xfs/Makefile | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/fs/xfs/Makefile b/fs/xfs/Makefile
index 1926701..5c5a366 100644
--- a/fs/xfs/Makefile
+++ b/fs/xfs/Makefile
@@ -16,7 +16,7 @@
# Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
#
-EXTRA_CFLAGS += -I$(src) -I$(src)/linux-2.6 -Wpointer-sign
+EXTRA_CFLAGS += -I$(src) -I$(src)/linux-2.6
XFS_LINUX := linux-2.6
--
1.6.5
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] xfs: turn off sign warnings
2010-01-25 5:33 [PATCH] xfs: turn off sign warnings Dave Chinner
@ 2010-01-25 8:34 ` Christoph Hellwig
0 siblings, 0 replies; 2+ messages in thread
From: Christoph Hellwig @ 2010-01-25 8:34 UTC (permalink / raw)
To: Dave Chinner; +Cc: xfs
On Mon, Jan 25, 2010 at 04:33:22PM +1100, Dave Chinner wrote:
> Because they cause warnings in static inline functions conditionally
> compiled into XFS from the VFS (e.g. fsnotify).
Ok. As I already mention I feared something like this could happen.
Would be great to get it on for the whole kernel.
Reviewed-by: Christoph Hellwig <hch@lst.de>
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2010-01-25 8:33 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-01-25 5:33 [PATCH] xfs: turn off sign warnings Dave Chinner
2010-01-25 8:34 ` Christoph Hellwig
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox