rbd: don't bother setting snapid in rbd_do_request()
authorAlex Elder <elder@inktank.com>
Fri, 9 Nov 2012 18:50:10 +0000 (12:50 -0600)
committerAlex Elder <elder@inktank.com>
Thu, 17 Jan 2013 21:52:03 +0000 (15:52 -0600)
commit7c3d22cf16f1bbcb37a73e88338c042bb49ff112
tree5f12e6272e0d6c1f309d398e4a954e91981670b5
parent25704ac9de30ac3e73c123e7b2734f7ca744c8d8
rbd: don't bother setting snapid in rbd_do_request()

For some reason, the snapid field of the osd request header is
explicitly set to CEPH_NOSNAP in rbd_do_request().  Just a few lines
later--with no code that would access this field in between--a call
is made to ceph_calc_raw_layout() passing the snapid provided to
rbd_do_request(), which encodes the snapid value it is provided into
that field instead.

In other words, there is no need to fill in CEPH_NOSNAP, and doing
so suggests it might be necessary.  Don't do that any more.

Signed-off-by: Alex Elder <elder@inktank.com>
Reviewed-by: Josh Durgin <josh.durgin@inktank.com>
drivers/block/rbd.c