diff options
| author | Miao Xie <miaox@cn.fujitsu.com> | 2013-05-15 09:48:29 +0200 |
|---|---|---|
| committer | Josef Bacik <jbacik@fusionio.com> | 2013-06-14 17:29:48 +0200 |
| commit | 3f1e3fa65c44b8ecdf2d6f14956c2cfe3a462a03 (patch) | |
| tree | 63d404df29d4fc1d0d9c215736484e6cf1f6bcb5 /fs/btrfs/transaction.h | |
| parent | Btrfs: don't flush the delalloc inodes in the while loop if flushoncommit is set (diff) | |
| download | linux-3f1e3fa65c44b8ecdf2d6f14956c2cfe3a462a03.tar.xz linux-3f1e3fa65c44b8ecdf2d6f14956c2cfe3a462a03.zip | |
Btrfs: remove unnecessary varient ->num_joined in btrfs_transaction structure
We used ->num_joined track if there were some writers which join the current
transaction when the committer was sleeping. If some writers joined the current
transaction, we has to continue the while loop to do some necessary stuff, such
as flush the ordered operations. But it is unnecessary because we will do it
after the while loop.
Besides that, tracking ->num_joined would make the committer drop into the while
loop when there are lots of internal writers(TRANS_JOIN).
So we remove ->num_joined and don't track if there are some writers which join
the current transaction when the committer is sleeping.
Signed-off-by: Miao Xie <miaox@cn.fujitsu.com>
Signed-off-by: Josef Bacik <jbacik@fusionio.com>
Diffstat (limited to 'fs/btrfs/transaction.h')
| -rw-r--r-- | fs/btrfs/transaction.h | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/fs/btrfs/transaction.h b/fs/btrfs/transaction.h index 5cc77b085427..0fc45e2a5139 100644 --- a/fs/btrfs/transaction.h +++ b/fs/btrfs/transaction.h @@ -37,8 +37,6 @@ struct btrfs_transaction { atomic_t num_writers; atomic_t use_count; - unsigned long num_joined; - spinlock_t commit_lock; int in_commit; int commit_done; |