[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