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 ----------------------------- |
