search for: libnbd_context_qemu_dirty_bitmap

...PACE_BASE ":" in the C code. >> >> - whether this should live in a separate (still included in libnbd.h) file, > >Single file is fine by me for now; we can always split later if it gets >huge. > >> >> - and more... > >I'd love to have LIBNBD_CONTEXT_QEMU_DIRTY_BITMAP(foo) produce the >string "qemu:dirty-bitmap:foo". I'm not sure how to wire that in on top >of your patches, so it doesn't have to happen today, but it's food for >thought. > You mean, for example: #define LIBNBD_CONTEXT_QEMU_DIRTY_BITMAP(foo) "qemu:dirty-bit...

This just defines the namespace, its contexts and related constants and the only supported one is currently base:allocation. The names of the constants are not very future-proof, but shorter than LIBNBD_META_NS_CONTEXT_BASE_ALLOCATION or similar. Currently the output looks like this: /* "base" namespace */ /* "base" namespace contexts */ /* "base:allocation"

..." when querying which contexts are available in that namespace. > > - whether this should live in a separate (still included in libnbd.h) file, Single file is fine by me for now; we can always split later if it gets huge. > > - and more... I'd love to have LIBNBD_CONTEXT_QEMU_DIRTY_BITMAP(foo) produce the string "qemu:dirty-bitmap:foo". I'm not sure how to wire that in on top of your patches, so it doesn't have to happen today, but it's food for thought. Here's what I'm pushing as a followup to your patch, to add more documentation about the constants,...