[dm-devel] multipath-tool udev rules require missing dmsetup features

Guido Guenther agx at sigxcpu.org
Mon Aug 13 11:21:00 UTC 2007


On Sun, Aug 12, 2007 at 08:21:44PM +0200, Bastian Blank wrote:
> On Sun, Aug 12, 2007 at 06:27:17PM +0200, Guido Guenther wrote:
> > +		[state]       = "DM_STATE=\"%s%s\"\n",
> > +		[tablestate]  = "DM_TABLE_STATE=\"%s%s%s\"\n",
> 
> Why use freetext fields if you can set it in a splitted way?
> 
> DM_STATE for example shows both suspended and read-only state without
> making in properly parsable.
This is just does what the patch I based this on does, I'd rather split
this into separate fields too.
Cheers,
 -- Guido




More information about the dm-devel mailing list