rbd: set snapc->seq only when refreshing header
authorAlex Elder <elder@inktank.com>
Thu, 19 Jul 2012 13:49:18 +0000 (08:49 -0500)
committerSage Weil <sage@inktank.com>
Tue, 31 Jul 2012 01:15:43 +0000 (18:15 -0700)
commit505cbb9bedc8c609c31d86ff4f8f656e5a0f9c49
tree460fca717d94ec4b897078e8201dca35b3fdcdcc
parent78dc447d3ca3701206a1dd813c901556a3fad451
rbd: set snapc->seq only when refreshing header

In rbd_header_add_snap() there is code to set snapc->seq to the
just-added snapshot id.  This is the only remnant left of the
use of that field for recording which snapshot an rbd_dev was
associated with.  That functionality is no longer supported,
so get rid of that final bit of code.

Doing so means we never actually set snapc->seq any more.  On the
server, the snapshot context's sequence value represents the highest
snapshot id ever issued for a particular rbd image.  So we'll make
it have that meaning here as well.  To do so, set this value
whenever the rbd header is (re-)read.  That way it will always be
consistent with the rest of the snapshot context we maintain.

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