On 04/30/2013 06:12 PM, Alex Elder wrote:
On that, the only thing inefficient is looking up the snapshot id given its name. It is so inefficient it might be worth offering an op that does that for us, because as it is we have to do a sequential search to find out a snapshot's name. (Maybe it's not that important to track the name for parent snapshots but users might be interested.)
That would be more efficient, but I don't think it's worth going that far. The client would need a way to work with current osds too, and it's not that much data - it's mainly the extra latency from fetching it that would be the problem. Iterating over 1000 snapshot names in memory is much faster, and only done when mapping for the kernel. Userspace fetches all the snapshot metadata in one operation, and indexes it by name in memory (which is useful since all the librbd calls relating to snapshots deal with names). -- 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
