On Fri, Feb 01, 2013 at 07:30:19AM -0800, Tejun Heo wrote: > On Fri, Feb 01, 2013 at 07:27:43AM -0800, Kent Overstreet wrote: > > > > Kent was talking about using MD (and though he isn't opposed to DM he > > > > doesn't care to integrate with DM himself). Either DM or MD would > > > > implicitly enable bcache to use this interface. But in the near-term I > > > > cannot see why Kent shouldn't be able to use bd_link_disk_holder too. > > > > > > Being part of dm or dm should make this mostly irrelevant, no? > > > > Yeah, but who knows when that'll actually happen and since this is for > > userspace I'm just going to call it. The refcounting won't affect me, > > and using it in bcache won't affect ripping that out. > > Yeah, I don't see any problem regarding user-visible behavior. Please > go ahead. It's just gross internally and I wanted someone to do > something about it before spreading its misuse (depending on the > refs). Could add a new, fixed version that doesn't do the refcounting, bcache and I imagine md could use that right away (maybe even just split the refcounting out into different functions and have dm call those directly, probably an easy way to refactor it anyways) -- dm-devel mailing list dm-devel@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/dm-devel