From: Prakash Surya <surya1@llnl.gov>
Date: Mon, 28 Jan 2013 16:41:32 +0000 (-0800)
Subject: LU-2158 fsfilt: Remove unused opcodes
X-Git-Tag: 2.3.61~14
X-Git-Url: https://git.whamcloud.com/gitweb?a=commitdiff_plain;h=e163c90f0d1c754a21498d2248f8e21e9f9713da;p=fs%2Flustre-release.git

LU-2158 fsfilt: Remove unused opcodes

This change removes the unused fsfilt opcode definitions and the
accompanying code in fsfilt_ext3_start. This is strictly cleanup.

Signed-off-by: Prakash Surya <surya1@llnl.gov>
Change-Id: Ifccb6e7c9c728bc98e4ddfa52093e660dd117b8b
Reviewed-on: http://review.whamcloud.com/5192
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: James Simmons <uja.ornl@gmail.com>
---

diff --git a/lustre/include/linux/lustre_fsfilt.h b/lustre/include/linux/lustre_fsfilt.h
index 94fc528..6b3e8b6 100644
--- a/lustre/include/linux/lustre_fsfilt.h
+++ b/lustre/include/linux/lustre_fsfilt.h
@@ -88,14 +88,6 @@ static inline char *fsfilt_get_label(struct obd_device *obd,
 }
 
 #define FSFILT_OP_UNLINK                1
-#define FSFILT_OP_RMDIR                 2
-#define FSFILT_OP_RENAME                3
-#define FSFILT_OP_CREATE                4
-#define FSFILT_OP_MKDIR                 5
-#define FSFILT_OP_SYMLINK               6
-#define FSFILT_OP_MKNOD                 7
-#define FSFILT_OP_SETATTR               8
-#define FSFILT_OP_LINK                  9
 #define FSFILT_OP_CANCEL_UNLINK         10
 
 #define __fsfilt_check_slow(obd, start, msg)                              \
diff --git a/lustre/lvfs/fsfilt_ext3.c b/lustre/lvfs/fsfilt_ext3.c
index fa46534..7d1e5ac 100644
--- a/lustre/lvfs/fsfilt_ext3.c
+++ b/lustre/lvfs/fsfilt_ext3.c
@@ -135,46 +135,12 @@ static void *fsfilt_ext3_start(struct inode *inode, int op, void *desc_private,
         }
 
         switch(op) {
-        case FSFILT_OP_RMDIR:
         case FSFILT_OP_UNLINK:
 		/* delete one file + create/update logs for each stripe */
 		nblocks += EXT3_DELETE_TRANS_BLOCKS(inode->i_sb);
 		nblocks += (EXT3_INDEX_EXTRA_TRANS_BLOCKS +
 			    FSFILT_SINGLEDATA_TRANS_BLOCKS(inode->i_sb)) * logs;
 		break;
-        case FSFILT_OP_RENAME:
-                /* modify additional directory */
-                nblocks += FSFILT_SINGLEDATA_TRANS_BLOCKS(inode->i_sb);
-                /* no break */
-        case FSFILT_OP_SYMLINK:
-                /* additional block + block bitmap + GDT for long symlink */
-                nblocks += 3;
-                /* no break */
-        case FSFILT_OP_CREATE: {
-                /* no break */
-        }
-        case FSFILT_OP_MKDIR:
-        case FSFILT_OP_MKNOD:
-                /* modify one inode + block bitmap + GDT */
-                nblocks += 3;
-                /* no break */
-        case FSFILT_OP_LINK:
-                /* modify parent directory */
-		nblocks += EXT3_INDEX_EXTRA_TRANS_BLOCKS +
-			   EXT3_DATA_TRANS_BLOCKS(inode->i_sb);
-                /* create/update logs for each stripe */
-                nblocks += (EXT3_INDEX_EXTRA_TRANS_BLOCKS +
-                            FSFILT_SINGLEDATA_TRANS_BLOCKS(inode->i_sb)) * logs;
-                break;
-        case FSFILT_OP_SETATTR:
-                /* Setattr on inode */
-		nblocks += 1;
-		nblocks += EXT3_INDEX_EXTRA_TRANS_BLOCKS +
-			   EXT3_DATA_TRANS_BLOCKS(inode->i_sb);
-                /* quota chown log for each stripe */
-                nblocks += (EXT3_INDEX_EXTRA_TRANS_BLOCKS +
-                            FSFILT_SINGLEDATA_TRANS_BLOCKS(inode->i_sb)) * logs;
-                break;
         case FSFILT_OP_CANCEL_UNLINK:
 		LASSERT(logs == 1);