aboutsummaryrefslogtreecommitdiffstats
path: root/Documentation/filesystems/Locking
diff options
context:
space:
mode:
authorChristoph Hellwig <hch@infradead.org>2010-03-03 09:05:01 -0500
committerJan Kara <jack@suse.cz>2010-03-04 18:20:28 -0500
commit63936ddaa16b9486e2d426ed7b09f559a5c60f87 (patch)
tree4cb1c4581799e10c26dd71d1a7d420de3c2cfd05 /Documentation/filesystems/Locking
parent5dd4056db84387975140ff2568eaa0406f07985e (diff)
dquot: cleanup inode allocation / freeing routines
Get rid of the alloc_inode and free_inode dquot operations - they are always called from the filesystem and if a filesystem really needs their own (which none currently does) it can just call into it's own routine directly. Also get rid of the vfs_dq_alloc/vfs_dq_free wrappers and always call the lowlevel dquot_alloc_inode / dqout_free_inode routines directly, which now lose the number argument which is always 1. Signed-off-by: Christoph Hellwig <hch@lst.de> Signed-off-by: Jan Kara <jack@suse.cz>
Diffstat (limited to 'Documentation/filesystems/Locking')
-rw-r--r--Documentation/filesystems/Locking8
1 files changed, 0 insertions, 8 deletions
diff --git a/Documentation/filesystems/Locking b/Documentation/filesystems/Locking
index 1192fde11638..4428f55f2131 100644
--- a/Documentation/filesystems/Locking
+++ b/Documentation/filesystems/Locking
@@ -462,8 +462,6 @@ in sys_read() and friends.
462prototypes: 462prototypes:
463 int (*initialize) (struct inode *, int); 463 int (*initialize) (struct inode *, int);
464 int (*drop) (struct inode *); 464 int (*drop) (struct inode *);
465 int (*alloc_inode) (const struct inode *, unsigned long);
466 int (*free_inode) (const struct inode *, unsigned long);
467 int (*transfer) (struct inode *, struct iattr *); 465 int (*transfer) (struct inode *, struct iattr *);
468 int (*write_dquot) (struct dquot *); 466 int (*write_dquot) (struct dquot *);
469 int (*acquire_dquot) (struct dquot *); 467 int (*acquire_dquot) (struct dquot *);
@@ -479,8 +477,6 @@ What filesystem should expect from the generic quota functions:
479 FS recursion Held locks when called 477 FS recursion Held locks when called
480initialize: yes maybe dqonoff_sem 478initialize: yes maybe dqonoff_sem
481drop: yes - 479drop: yes -
482alloc_inode: ->mark_dirty() -
483free_inode: ->mark_dirty() -
484transfer: yes - 480transfer: yes -
485write_dquot: yes dqonoff_sem or dqptr_sem 481write_dquot: yes dqonoff_sem or dqptr_sem
486acquire_dquot: yes dqonoff_sem or dqptr_sem 482acquire_dquot: yes dqonoff_sem or dqptr_sem
@@ -491,10 +487,6 @@ write_info: yes dqonoff_sem
491FS recursion means calling ->quota_read() and ->quota_write() from superblock 487FS recursion means calling ->quota_read() and ->quota_write() from superblock
492operations. 488operations.
493 489
494->alloc_inode(), ->free_inode() are called
495only directly by the filesystem and do not call any fs functions only
496the ->mark_dirty() operation.
497
498More details about quota locking can be found in fs/dquot.c. 490More details about quota locking can be found in fs/dquot.c.
499 491
500--------------------------- vm_operations_struct ----------------------------- 492--------------------------- vm_operations_struct -----------------------------