aboutsummaryrefslogtreecommitdiffstats
path: root/fs/gfs2/incore.h
diff options
context:
space:
mode:
authorSteven Whitehouse <swhiteho@redhat.com>2009-08-27 10:51:07 -0400
committerSteven Whitehouse <swhiteho@redhat.com>2009-08-27 10:51:07 -0400
commit8d8291ae93ecb4a246e87e452d55cca412373300 (patch)
treeccf67c703fb9f9b2e4ee945f3221b8b20c1804db /fs/gfs2/incore.h
parent307cf6e63cfa5025589ea1a06db44439a43819ff (diff)
GFS2: Remove no_formal_ino generating code
The inum structure used throughout GFS2 has two fields. One no_addr is the disk block number of the inode in question and is used everywhere as the inode number. The other, no_formal_ino, is used only as the generation number for NFS. Historically the no_formal_ino field was set using a complicated system of one global and one per-node file containing inode numbers in order to ensure that each no_formal_ino was unique. Also this code made no provision for what would happen when eventually the (64 bit) numbers ran out. Now I know that is pretty unlikely to happen given the large space of numbers, but it is possible nevertheless. The only guarantee required for no_formal_ino is that, for any single inode, the same number doesn't get reused too quickly. We already have a generation number which is kept in the inode and initialised from a counter in the resource group (almost no overhead, since we have to touch the resource group anyway in order to allocate an inode in the first place). Aside from ensuring that we never use the value 0 in the no_formal_ino field, we can use that counter directly. As a result of that change, we lose about 200 lines of code and also gain about 10 creates/sec on the postmark benchmark (on my test machine). Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Diffstat (limited to 'fs/gfs2/incore.h')
-rw-r--r--fs/gfs2/incore.h7
1 files changed, 0 insertions, 7 deletions
diff --git a/fs/gfs2/incore.h b/fs/gfs2/incore.h
index 1d11e6e0373e..52436abc3b4d 100644
--- a/fs/gfs2/incore.h
+++ b/fs/gfs2/incore.h
@@ -548,18 +548,12 @@ struct gfs2_sbd {
548 struct dentry *sd_root_dir; 548 struct dentry *sd_root_dir;
549 549
550 struct inode *sd_jindex; 550 struct inode *sd_jindex;
551 struct inode *sd_inum_inode;
552 struct inode *sd_statfs_inode; 551 struct inode *sd_statfs_inode;
553 struct inode *sd_ir_inode;
554 struct inode *sd_sc_inode; 552 struct inode *sd_sc_inode;
555 struct inode *sd_qc_inode; 553 struct inode *sd_qc_inode;
556 struct inode *sd_rindex; 554 struct inode *sd_rindex;
557 struct inode *sd_quota_inode; 555 struct inode *sd_quota_inode;
558 556
559 /* Inum stuff */
560
561 struct mutex sd_inum_mutex;
562
563 /* StatFS stuff */ 557 /* StatFS stuff */
564 558
565 spinlock_t sd_statfs_spin; 559 spinlock_t sd_statfs_spin;
@@ -587,7 +581,6 @@ struct gfs2_sbd {
587 struct gfs2_holder sd_journal_gh; 581 struct gfs2_holder sd_journal_gh;
588 struct gfs2_holder sd_jinode_gh; 582 struct gfs2_holder sd_jinode_gh;
589 583
590 struct gfs2_holder sd_ir_gh;
591 struct gfs2_holder sd_sc_gh; 584 struct gfs2_holder sd_sc_gh;
592 struct gfs2_holder sd_qc_gh; 585 struct gfs2_holder sd_qc_gh;
593 586