diff options
| author | Sage Weil <sage@newdream.net> | 2009-10-09 13:31:32 -0400 |
|---|---|---|
| committer | Sage Weil <sage@newdream.net> | 2009-10-09 14:58:20 -0400 |
| commit | 266673db423e6ab247170d6c6d72ec36e530a911 (patch) | |
| tree | d86b9d3b3e34fbf2dcc64b21d73145fd962237ac /fs/ceph/osd_client.c | |
| parent | 81b024e70fed635a2cf5a4bf911db1649bb005f5 (diff) | |
ceph: cancel osd requests before resending them
This ensures we don't submit the same request twice if we are kicking a
specific osd (as with an osd_reset), or when we hit a transient error and
resend.
Signed-off-by: Sage Weil <sage@newdream.net>
Diffstat (limited to 'fs/ceph/osd_client.c')
| -rw-r--r-- | fs/ceph/osd_client.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/fs/ceph/osd_client.c b/fs/ceph/osd_client.c index 4a8b4f08d4ae..8e33928647f4 100644 --- a/fs/ceph/osd_client.c +++ b/fs/ceph/osd_client.c | |||
| @@ -813,10 +813,13 @@ static void kick_requests(struct ceph_osd_client *osdc, | |||
| 813 | 813 | ||
| 814 | if (req->r_resend) { | 814 | if (req->r_resend) { |
| 815 | dout(" r_resend set on tid %llu\n", req->r_tid); | 815 | dout(" r_resend set on tid %llu\n", req->r_tid); |
| 816 | __cancel_request(req); | ||
| 816 | goto kick; | 817 | goto kick; |
| 817 | } | 818 | } |
| 818 | if (req->r_osd && kickosd == req->r_osd) | 819 | if (req->r_osd && kickosd == req->r_osd) { |
| 820 | __cancel_request(req); | ||
| 819 | goto kick; | 821 | goto kick; |
| 822 | } | ||
| 820 | 823 | ||
| 821 | err = __map_osds(osdc, req); | 824 | err = __map_osds(osdc, req); |
| 822 | if (err == 0) | 825 | if (err == 0) |
