On Tue, Mar 17, 2015 at 04:37:07PM +0000, Sami Tolvanen wrote: > +<mode> > + Optional. The format needs to remain easily extendable. Two alternatives: Document this as a required field going forward, but say that if it is missing the default will be assumed. eio|logging|restart (but those names are confusing) Or (my preference) use a 'features' approach like most other targets do: <number of feature> <feature arguments> "error" is the existing default, so doesn't need stating. The other two could be: restart_on_corruption ignore_corruption Logging and events would happen in all cases, but if they did need controlling independently, additional feature arguments could do that. Alasdair -- dm-devel mailing list dm-devel@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/dm-devel