]> git.karo-electronics.de Git - karo-tx-linux.git/commit
rbd: protect against duplicate client creation
authorAlex Elder <elder@inktank.com>
Wed, 29 May 2013 16:19:00 +0000 (11:19 -0500)
committerSage Weil <sage@inktank.com>
Thu, 13 Jun 2013 15:46:23 +0000 (08:46 -0700)
commit36c7125fa114af3e2dd80d5da43045db1ddd63ef
tree3352532bf135feb87f88e84d0388abfe493d9280
parentd2554242a43898c68ec09b1155176cda6d5e52bb
rbd: protect against duplicate client creation

If more than one rbd image has the same ceph cluster configuration
(same options, same set of monitors, same keys) they normally share
a single rbd client.

When an image is getting mapped, rbd looks to see if an existing
client can be used, and creates a new one if not.

The lookup and creation are not done under a common lock though, so
mapping two images concurrently could lead to duplicate clients
getting set up needlessly.  This isn't a major problem, but it's
wasteful and different from what's intended.

This patch fixes that by using the control mutex to protect
both the lookup and (if needed) creation of the client.  It
was previously used just when creating.

This resolves:
    http://tracker.ceph.com/issues/3094

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