Re: [PATCH 3/6] dm raid45 target: introduce message parser

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, 2009-06-19 at 12:35 -0500, Jonathan Brassow wrote:
> On Jun 15, 2009, at 12:21 PM, heinzm@xxxxxxxxxx wrote:
> 
> > This patch adds the dm-message parser module.
> 
> Would it be ok to leave this out?  Would you be willing to implement  
> some simple parsing in 'raid_message' for now?

If arguments go the way, that this ain't buy us much code sharing with
other targets, sure.

More opinions from others ?

> 
> Maybe there are benefits that I am not seeing?

I did it to allow for argument parsing code to be sharable amongst
targets message methods aiming to simplify parsing in multiple targets.

Heinz

> 
>   brassow
> 
> --
> dm-devel mailing list
> dm-devel@xxxxxxxxxx
> https://www.redhat.com/mailman/listinfo/dm-devel

--
dm-devel mailing list
dm-devel@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/dm-devel

[Index of Archives]     [DM Crypt]     [Fedora Desktop]     [ATA RAID]     [Fedora Marketing]     [Fedora Packaging]     [Fedora SELinux]     [Yosemite Discussion]     [KDE Users]     [Fedora Docs]

  Powered by Linux