fs: Remove i_cindex from struct inode

The only user of the i_cindex element in the inode structure is used
is by the firewire drivers.  As part of an attempt to slim down the
inode structure to save memory --- since a typical Linux system will
have hundreds of thousands if not millions of inodes cached, a
reduction in the size inode has high leverage.

The firewire driver does not need i_cindex in any fast path, so it's
simple enough to calculate when it is needed, instead of wasting space
in the inode structure.

Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Cc: krh@redhat.com
Cc: stefanr@s5r6.in-berlin.de
Cc: linux-fsdevel@vger.kernel.org
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>

1 files changed, 3 insertions, 2 deletions

diff --git a/drivers/ieee1394/dv1394.c b/drivers/ieee1394/dv1394.c
index 823a6297a1af..2cd00b5b45b4 100644
--- a/drivers/ieee1394/dv1394.c
+++ b/drivers/ieee1394/dv1394.c
@@ -1789,12 +1789,13 @@ static int dv1394_open(struct inode *inode, struct file *file)
         } else {
                 /* look up the card by ID */
                 unsigned long flags;
+                int idx = ieee1394_file_to_instance(file);
 
                 spin_lock_irqsave(&dv1394_cards_lock, flags);
                 if (!list_empty(&dv1394_cards)) {
                         struct video_card *p;
                         list_for_each_entry(p, &dv1394_cards, list) {
-                                if ((p->id) == ieee1394_file_to_instance(file)) {
+                                if ((p->id) == idx) {
                                         video = p;
                                         break;
                                 }
@@ -1803,7 +1804,7 @@ static int dv1394_open(struct inode *inode, struct file *file)
                 spin_unlock_irqrestore(&dv1394_cards_lock, flags);
 
                 if (!video) {
-                        debug_printk("dv1394: OHCI card %d not found", ieee1394_file_to_instance(file));
+                        debug_printk("dv1394: OHCI card %d not found", idx);
                         return -ENODEV;
                 }