diff options
author | Christoph Hellwig <hch@infradead.org> | 2010-03-03 09:05:01 -0500 |
---|---|---|
committer | Jan Kara <jack@suse.cz> | 2010-03-04 18:20:28 -0500 |
commit | 63936ddaa16b9486e2d426ed7b09f559a5c60f87 (patch) | |
tree | 4cb1c4581799e10c26dd71d1a7d420de3c2cfd05 /Documentation/filesystems/Locking | |
parent | 5dd4056db84387975140ff2568eaa0406f07985e (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/Locking | 8 |
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. | |||
462 | prototypes: | 462 | prototypes: |
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 |
480 | initialize: yes maybe dqonoff_sem | 478 | initialize: yes maybe dqonoff_sem |
481 | drop: yes - | 479 | drop: yes - |
482 | alloc_inode: ->mark_dirty() - | ||
483 | free_inode: ->mark_dirty() - | ||
484 | transfer: yes - | 480 | transfer: yes - |
485 | write_dquot: yes dqonoff_sem or dqptr_sem | 481 | write_dquot: yes dqonoff_sem or dqptr_sem |
486 | acquire_dquot: yes dqonoff_sem or dqptr_sem | 482 | acquire_dquot: yes dqonoff_sem or dqptr_sem |
@@ -491,10 +487,6 @@ write_info: yes dqonoff_sem | |||
491 | FS recursion means calling ->quota_read() and ->quota_write() from superblock | 487 | FS recursion means calling ->quota_read() and ->quota_write() from superblock |
492 | operations. | 488 | operations. |
493 | 489 | ||
494 | ->alloc_inode(), ->free_inode() are called | ||
495 | only directly by the filesystem and do not call any fs functions only | ||
496 | the ->mark_dirty() operation. | ||
497 | |||
498 | More details about quota locking can be found in fs/dquot.c. | 490 | More details about quota locking can be found in fs/dquot.c. |
499 | 491 | ||
500 | --------------------------- vm_operations_struct ----------------------------- | 492 | --------------------------- vm_operations_struct ----------------------------- |