On 09/09/2013 02:16 AM, Josh Durgin wrote: > Patches 1-3 fix races between device removal and notify processing. > > Patch 2 has an improved summary, fixes reference counting, and > renames the function as suggested by Alex. > > Patch 3 is a reworked and simplified version that uses the existing > rbd_dev->flags and lock instead of adding new ones. It also restricts > where it holds the lock to avoid an inversion with bdev->lock. I found all but this one were ready to go. If you can, please just pull this patch out to be resolved separately. There's no sense in holding up the other four until this one is perfect. Nice work. -Alex > Patch 4 is the same. > > Patch 5 fixes an inconsistency noticed by Alex in his review of patch 4. > > Josh Durgin (5): > rbd: complete notifies before cleaning up osd_client and rbd_dev > rbd: make rbd_obj_notify_ack() synchronous > rbd: fix use-after free of rbd_dev->disk > rbd: ignore unmapped snapshots that no longer exist > rbd: fix error handling from rbd_snap_name() > > drivers/block/rbd.c | 71 ++++++++++++++++++++++++++++++++++++++------------- > 1 files changed, 53 insertions(+), 18 deletions(-) > -- To unsubscribe from this list: send the line "unsubscribe ceph-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
