diff options
| author | Miao Xie | 2014-02-20 11:08:58 +0100 |
|---|---|---|
| committer | Josef Bacik | 2014-03-10 20:16:43 +0100 |
| commit | 8b050d350c7846462a21e9e054c9154ede9b43cf (patch) | |
| tree | 3e756c4aeefc74fafa7db71293e92dac20832ed3 /fs/btrfs/ctree.h | |
| parent | Btrfs: use signed integer instead of unsigned long integer for log transid (diff) | |
| download | kernel-qcow2-linux-8b050d350c7846462a21e9e054c9154ede9b43cf.tar.gz kernel-qcow2-linux-8b050d350c7846462a21e9e054c9154ede9b43cf.tar.xz kernel-qcow2-linux-8b050d350c7846462a21e9e054c9154ede9b43cf.zip | |
Btrfs: fix skipped error handle when log sync failed
It is possible that many tasks sync the log tree at the same time, but
only one task can do the sync work, the others will wait for it. But those
wait tasks didn't get the result of the log sync, and returned 0 when they
ended the wait. It caused those tasks skipped the error handle, and the
serious problem was they told the users the file sync succeeded but in
fact they failed.
This patch fixes this problem by introducing a log context structure,
we insert it into the a global list. When the sync fails, we will set
the error number of every log context in the list, then the waiting tasks
get the error number of the log context and handle the error if need.
Signed-off-by: Miao Xie <miaox@cn.fujitsu.com>
Signed-off-by: Josef Bacik <jbacik@fb.com>
Diffstat (limited to 'fs/btrfs/ctree.h')
| -rw-r--r-- | fs/btrfs/ctree.h | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/fs/btrfs/ctree.h b/fs/btrfs/ctree.h index 70c03f5f0953..906410719acb 100644 --- a/fs/btrfs/ctree.h +++ b/fs/btrfs/ctree.h @@ -1718,6 +1718,7 @@ struct btrfs_root { struct mutex log_mutex; wait_queue_head_t log_writer_wait; wait_queue_head_t log_commit_wait[2]; + struct list_head log_ctxs[2]; atomic_t log_writers; atomic_t log_commit[2]; atomic_t log_batch; |