diff options
| author | Henry C Chang <henry.cy.chang@gmail.com> | 2011-03-15 05:18:02 -0400 |
|---|---|---|
| committer | Sage Weil <sage@newdream.net> | 2011-03-21 15:24:25 -0400 |
| commit | 49bcb93236ce1c60d9b7eb21a0aea1999f4d8709 (patch) | |
| tree | e8dc94cf5c106fc1067411bc38e16e43ab15318e | |
| parent | 78a255654fa7f01945dea0dcedcf5113b3ad9f93 (diff) | |
ceph: add request to the tail of unsafe write list
In sync_write_wait(), we assume that the newest request is at the
tail of unsafe write list. We should maintain the semantics here.
Signed-off-by: Henry C Chang <henry_c_chang@tcloudcomputing.com>
Signed-off-by: Sage Weil <sage@newdream.net>
| -rw-r--r-- | fs/ceph/file.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/fs/ceph/file.c b/fs/ceph/file.c index db5d86309744..159b512d5a27 100644 --- a/fs/ceph/file.c +++ b/fs/ceph/file.c | |||
| @@ -564,7 +564,8 @@ more: | |||
| 564 | * start_request so that a tid has been assigned. | 564 | * start_request so that a tid has been assigned. |
| 565 | */ | 565 | */ |
| 566 | spin_lock(&ci->i_unsafe_lock); | 566 | spin_lock(&ci->i_unsafe_lock); |
| 567 | list_add(&req->r_unsafe_item, &ci->i_unsafe_writes); | 567 | list_add_tail(&req->r_unsafe_item, |
| 568 | &ci->i_unsafe_writes); | ||
| 568 | spin_unlock(&ci->i_unsafe_lock); | 569 | spin_unlock(&ci->i_unsafe_lock); |
| 569 | ceph_get_cap_refs(ci, CEPH_CAP_FILE_WR); | 570 | ceph_get_cap_refs(ci, CEPH_CAP_FILE_WR); |
| 570 | } | 571 | } |
