diff options
author | Eric Sandeen <sandeen@redhat.com> | 2010-05-16 09:00:00 -0400 |
---|---|---|
committer | Theodore Ts'o <tytso@mit.edu> | 2010-05-16 09:00:00 -0400 |
commit | 56246f9ae4cfa95b460f9dfbcfb1b772d85db046 (patch) | |
tree | 2a11530806fd384c7a482d7d05f7966af773fef2 /fs/quota | |
parent | 84061e07c5fbbbf9dc8aef8fb750fc3a2dfc31f3 (diff) |
quota: use flags interface for dquot alloc/free space
Switch __dquot_alloc_space and __dquot_free_space to take flags
to indicate whether to warn and/or to reserve (or free reserve).
This is slightly more readable at the callpoints, and makes it
cleaner to add a "nofail" option in the next patch.
Signed-off-by: Eric Sandeen <sandeen@redhat.com>
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Diffstat (limited to 'fs/quota')
-rw-r--r-- | fs/quota/dquot.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/fs/quota/dquot.c b/fs/quota/dquot.c index 788b5802a7ce..8c40c07797ad 100644 --- a/fs/quota/dquot.c +++ b/fs/quota/dquot.c | |||
@@ -1488,11 +1488,12 @@ static void inode_decr_space(struct inode *inode, qsize_t number, int reserve) | |||
1488 | /* | 1488 | /* |
1489 | * This operation can block, but only after everything is updated | 1489 | * This operation can block, but only after everything is updated |
1490 | */ | 1490 | */ |
1491 | int __dquot_alloc_space(struct inode *inode, qsize_t number, | 1491 | int __dquot_alloc_space(struct inode *inode, qsize_t number, int flags) |
1492 | int warn, int reserve) | ||
1493 | { | 1492 | { |
1494 | int cnt, ret = 0; | 1493 | int cnt, ret = 0; |
1495 | char warntype[MAXQUOTAS]; | 1494 | char warntype[MAXQUOTAS]; |
1495 | int warn = flags & DQUOT_SPACE_WARN; | ||
1496 | int reserve = flags & DQUOT_SPACE_RESERVE; | ||
1496 | 1497 | ||
1497 | /* | 1498 | /* |
1498 | * First test before acquiring mutex - solves deadlocks when we | 1499 | * First test before acquiring mutex - solves deadlocks when we |
@@ -1612,10 +1613,11 @@ EXPORT_SYMBOL(dquot_claim_space_nodirty); | |||
1612 | /* | 1613 | /* |
1613 | * This operation can block, but only after everything is updated | 1614 | * This operation can block, but only after everything is updated |
1614 | */ | 1615 | */ |
1615 | void __dquot_free_space(struct inode *inode, qsize_t number, int reserve) | 1616 | void __dquot_free_space(struct inode *inode, qsize_t number, int flags) |
1616 | { | 1617 | { |
1617 | unsigned int cnt; | 1618 | unsigned int cnt; |
1618 | char warntype[MAXQUOTAS]; | 1619 | char warntype[MAXQUOTAS]; |
1620 | int reserve = flags & DQUOT_SPACE_RESERVE; | ||
1619 | 1621 | ||
1620 | /* First test before acquiring mutex - solves deadlocks when we | 1622 | /* First test before acquiring mutex - solves deadlocks when we |
1621 | * re-enter the quota code and are already holding the mutex */ | 1623 | * re-enter the quota code and are already holding the mutex */ |