summaryrefslogtreecommitdiffstats
path: root/fs/btrfs/btrfs_inode.h
diff options
context:
space:
mode:
authorJosef Bacik <josef@redhat.com>2011-05-11 15:26:06 -0400
committerJosef Bacik <josef@redhat.com>2011-05-23 13:03:12 -0400
commitd82a6f1d7e8b61ed5996334d0db66651bb43641d (patch)
tree5738ec75f7679bdc1b97aecbc5cad3464cc65541 /fs/btrfs/btrfs_inode.h
parent7e2355ba1a11649f0b212a29fdb9f47476f1248e (diff)
downloadop-kernel-dev-d82a6f1d7e8b61ed5996334d0db66651bb43641d.zip
op-kernel-dev-d82a6f1d7e8b61ed5996334d0db66651bb43641d.tar.gz
Btrfs: kill BTRFS_I(inode)->block_group
Originally this was going to be used as a way to give hints to the allocator, but frankly we can get much better hints elsewhere and it's not even used at all for anything usefull. In addition to be completely useless, when we initialize an inode we try and find a freeish block group to set as the inodes block group, and with a completely full 40gb fs this takes _forever_, so I imagine with say 1tb fs this is just unbearable. So just axe the thing altoghether, we don't need it and it saves us 8 bytes in the inode and saves us 500 microseconds per inode lookup in my testcase. Thanks, Signed-off-by: Josef Bacik <josef@redhat.com>
Diffstat (limited to 'fs/btrfs/btrfs_inode.h')
-rw-r--r--fs/btrfs/btrfs_inode.h3
1 files changed, 0 insertions, 3 deletions
diff --git a/fs/btrfs/btrfs_inode.h b/fs/btrfs/btrfs_inode.h
index 57c3bb2..4bc852d 100644
--- a/fs/btrfs/btrfs_inode.h
+++ b/fs/btrfs/btrfs_inode.h
@@ -120,9 +120,6 @@ struct btrfs_inode {
*/
u64 index_cnt;
- /* the start of block group preferred for allocations. */
- u64 block_group;
-
/* the fsync log has some corner cases that mean we have to check
* directories to see if any unlinks have been done before
* the directory was logged. See tree-log.c for all the
OpenPOWER on IntegriCloud