aboutsummaryrefslogtreecommitdiffstats
path: root/net
diff options
context:
space:
mode:
authorSage Weil <sage@newdream.net>2011-05-12 18:48:16 -0400
committerSage Weil <sage@newdream.net>2011-05-19 14:25:05 -0400
commit9d6fcb081a4770c3772c51c59c7251c22716d7bb (patch)
tree8827d90aff9d7f463cb429aff1ecd22dde6a77a6 /net
parent6b4a3b517a767c483d16a200730b2967e0e23b83 (diff)
ceph: check return value for start_request in writepages
Since we pass the nofail arg, we should never get an error; BUG if we do. (And fix the function to not return an error if __map_request fails.) Signed-off-by: Sage Weil <sage@newdream.net>
Diffstat (limited to 'net')
-rw-r--r--net/ceph/osd_client.c8
1 files changed, 7 insertions, 1 deletions
diff --git a/net/ceph/osd_client.c b/net/ceph/osd_client.c
index 9adbb01d23ad..caa092eb0009 100644
--- a/net/ceph/osd_client.c
+++ b/net/ceph/osd_client.c
@@ -1677,8 +1677,14 @@ int ceph_osdc_start_request(struct ceph_osd_client *osdc,
1677 */ 1677 */
1678 if (req->r_sent == 0) { 1678 if (req->r_sent == 0) {
1679 rc = __map_request(osdc, req); 1679 rc = __map_request(osdc, req);
1680 if (rc < 0) 1680 if (rc < 0) {
1681 if (nofail) {
1682 dout("osdc_start_request failed map, "
1683 " will retry %lld\n", req->r_tid);
1684 rc = 0;
1685 }
1681 goto out_unlock; 1686 goto out_unlock;
1687 }
1682 if (req->r_osd == NULL) { 1688 if (req->r_osd == NULL) {
1683 dout("send_request %p no up osds in pg\n", req); 1689 dout("send_request %p no up osds in pg\n", req);
1684 ceph_monc_request_next_osdmap(&osdc->client->monc); 1690 ceph_monc_request_next_osdmap(&osdc->client->monc);