From: Andreas Dilger <adilger@whamcloud.com>
Date: Mon, 3 Oct 2011 21:16:28 +0000 (-0700)
Subject: LU-735 Remove fixme() macro
X-Git-Tag: 2.1.51~2
X-Git-Url: https://git.whamcloud.com/?p=fs%2Flustre-release.git;a=commitdiff_plain;h=b05ed4fef2a84f7130ef069e71e93df6155bccb8

LU-735 Remove fixme() macro

This NULL export warning is the only consumer of the fixme()
macro.  Change it to a CWARN and remove the fixme() macro.

Change-Id: I194a9d92369a6a4dce35701fe261631420f9894d
Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
Reviewed-on: http://review.whamcloud.com/1472
Tested-by: Hudson
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
---

diff --git a/lustre/include/obd_support.h b/lustre/include/obd_support.h
index 85537da..9e40c43 100644
--- a/lustre/include/obd_support.h
+++ b/lustre/include/obd_support.h
@@ -455,8 +455,6 @@ static inline void obd_fail_write(int id, struct super_block *sb)
 #define OBD_FAIL_WRITE(id, sb) obd_fail_write(id, sb)
 #endif
 
-#define fixme() CDEBUG(D_OTHER, "FIXME\n");
-
 extern cfs_atomic_t libcfs_kmemory;
 
 #ifdef LPROCFS
diff --git a/lustre/obdclass/genops.c b/lustre/obdclass/genops.c
index 36d8b44..177c0d0 100644
--- a/lustre/obdclass/genops.c
+++ b/lustre/obdclass/genops.c
@@ -1144,8 +1144,7 @@ int class_disconnect(struct obd_export *export)
         ENTRY;
 
         if (export == NULL) {
-                fixme();
-                CDEBUG(D_IOCTL, "attempting to free NULL export %p\n", export);
+                CWARN("attempting to free NULL export %p\n", export);
                 RETURN(-EINVAL);
         }