linux.git/fs/btrfs/ordered-data.c, branch v3.10

Btrfs: improve the performance of the csums lookup

2013-05-06T19:54:35+00:00

It is very likely that there are several blocks in bio, it is very
inefficient if we get their csums one by one. This patch improves
this problem by getting the csums in batch.

According to the result of the following test, the execute time of
__btrfs_lookup_bio_sums() is down by ~28%(300us -> 217us).

 # dd if=/file of=/dev/null bs=1M count=1024

Signed-off-by: Miao Xie 
Signed-off-by: Josef Bacik

Btrfs: hold the ordered operations mutex when waiting on ordered extents

2013-03-28T13:51:28+00:00

We need to hold the ordered_operations mutex while waiting on ordered extents
since we splice and run the ordered extents list.  We need to make sure anybody
else who wants to wait on ordered extents does actually wait for them to be
completed.  This will keep us from bailing out of flushing in case somebody is
already waiting on ordered extents to complete.  Thanks,

Signed-off-by: Josef Bacik

Btrfs: place ordered operations on a per transaction list

2013-02-20T17:59:57+00:00

Miao made the ordered operations stuff run async, which introduced a
deadlock where we could get somebody (sync) racing in and committing the
transaction while a commit was already happening.  The new committer would
try and flush ordered operations which would hang waiting for the commit to
finish because it is done asynchronously and no longer inherits the callers
trans handle.  To fix this we need to make the ordered operations list a per
transaction list.  We can get new inodes added to the ordered operation list
by truncating them and then having another process writing to them, so this
makes it so that anybody trying to add an ordered operation _must_ start a
transaction in order to add itself to the list, which will keep new inodes
from getting added to the ordered operations list after we start committing.
This should fix the deadlock and also keeps us from doing a lot more work
than we need to during commit.  Thanks,

Signed-off-by: Josef Bacik

Btrfs: don't traverse the ordered operation list repeatedly

2013-02-20T14:37:24+00:00

btrfs_run_ordered_operations() needn't traverse the ordered operation list
repeatedly, it is because the transaction commiter will invoke it again when
there is no other writer in this transaction, it can ensure that no one can
add new objects into the ordered operation list.

Signed-off-by: Miao Xie 
Signed-off-by: Josef Bacik

Btrfs: wait on ordered extents at the last possible moment

2013-02-20T14:37:04+00:00

Since we don't actually copy the extent information from the source tree in
the fast case we don't need to wait for ordered io to be completed in order
to fsync, we just need to wait for the io to be completed.  So when we're
logging our file just attach all of the ordered extents to the log, and then
when the log syncs just wait for IO_DONE on the ordered extents and then
write the super.  Thanks,

Signed-off-by: Josef Bacik

Btrfs: fix possible stale data exposure

2013-02-05T21:09:16+00:00

We specifically do not update the disk i_size if there are ordered extents
outstanding for any area between the current disk_i_size and our ordered
extent so that we do not expose stale data.  The problem is the check we
have only checks if the ordered extent starts at or after the current
disk_i_size, which doesn't take into account an ordered extent that starts
before the current disk_i_size and ends past the disk_i_size.  Fix this by
checking if the extent ends past the disk_i_size.  Thanks,

Signed-off-by: Josef Bacik

Btrfs: fix missing i_size update

2013-02-05T21:09:14+00:00

If we have an ordered extent before the ordered extent we are currently
completing that is after the current disk_i_size we will put our i_size
update into that ordered extent so that we do not expose stale data.  The
problem is that if our disk i_size is updated past the previous ordered
extent we won't update the i_size with the pending i_size update.  So check
the pending i_size update and if its above the current disk i_size we need
to go ahead and try to update.  Thanks,

Signed-off-by: Josef Bacik

Btrfs: cleanup for btrfs_wait_order_range

2012-12-12T22:15:19+00:00

Variable 'found' is no more used.

Signed-off-by: Liu Bo 
Signed-off-by: Chris Mason

Btrfs: make ordered extent be flushed by multi-task

2012-12-11T18:31:38+00:00

Though the process of the ordered extents is a bit different with the delalloc inode
flush, but we can see it as a subset of the delalloc inode flush, so we also handle
them by flush workers.

Signed-off-by: Miao Xie 
Signed-off-by: Chris Mason

Btrfs: make ordered operations be handled by multi-task

2012-12-11T18:31:37+00:00

The process of the ordered operations is similar to the delalloc inode flush, so
we handle them by flush workers.

Signed-off-by: Miao Xie 
Signed-off-by: Chris Mason