diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2012-06-06 09:15:33 -0500 |
|---|---|---|
| committer | Sage Weil <sage@inktank.com> | 2012-07-17 21:30:31 -0700 |
| commit | 6a3ca4f18873f950895cb64ddefafb51a732e3f7 (patch) | |
| tree | 2d2431b333e6ffd98037c8f1b6058ad5d5f17c3c | |
| parent | 236df3755d944c33120d77e84b5ff6f3917eb307 (diff) | |
| download | mm-6a3ca4f18873f950895cb64ddefafb51a732e3f7.tar.gz | |
rbd: endian bug in rbd_req_cb()
Sparse complains about this because:
drivers/block/rbd.c:996:20: warning: cast to restricted __le32
drivers/block/rbd.c:996:20: warning: cast from restricted __le16
These are set in osd_req_encode_op() and they are le16.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Reviewed-by: Alex Elder <elder@inktank.com>
(cherry picked from commit 895cfcc810e53d7d36639969c71efb9087221167)
| -rw-r--r-- | drivers/block/rbd.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/block/rbd.c b/drivers/block/rbd.c index 8b9c1734d80706..8f428a8ab003d8 100644 --- a/drivers/block/rbd.c +++ b/drivers/block/rbd.c @@ -977,7 +977,7 @@ static void rbd_req_cb(struct ceph_osd_request *req, struct ceph_msg *msg) op = (void *)(replyhead + 1); rc = le32_to_cpu(replyhead->result); bytes = le64_to_cpu(op->extent.length); - read_op = (le32_to_cpu(op->op) == CEPH_OSD_OP_READ); + read_op = (le16_to_cpu(op->op) == CEPH_OSD_OP_READ); dout("rbd_req_cb bytes=%lld readop=%d rc=%d\n", bytes, read_op, rc); |