Re: [PATCH] iproute2 -- add fwmarkmask

2006-02-23 Thread Patrick McHardy
Michael Richardson wrote: > > >>>"Patrick" == Patrick McHardy <[EMAIL PROTECTED]> writes: > > Patrick> The normal way to display masks is with a "/". Also I think > Patrick> it shouldn't display the default mask to avoid breaking > Patrick> scripts that parse the output. > > I

Re: [PATCH] iproute2 -- add fwmarkmask

2006-02-23 Thread Michael Richardson
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 > "Patrick" == Patrick McHardy <[EMAIL PROTECTED]> writes: Patrick> The normal way to display masks is with a "/". Also I think Patrick> it shouldn't display the default mask to avoid breaking Patrick> scripts that parse the output.

Re: [PATCH] iproute2 -- add fwmarkmask

2006-02-20 Thread Patrick McHardy
Michael Richardson wrote: > - if (tb[RTA_PROTOINFO]) { > - fprintf(fp, "fwmark %#x ", > *(__u32*)RTA_DATA(tb[RTA_PROTOINFO])); > + if (tb[RTA_FWMARK]) { > + __u32 value = *(__u32*)RTA_DATA(tb[RTA_PROTOINFO]); > + if (tb[RTA_FWMARK_MASK]) { > +

[PATCH] iproute2 -- add fwmarkmask

2006-02-20 Thread Michael Richardson
{see discussion about RTA_FWMARK in other posting} diff-tree 573e55d29ef2f014218508d01eee678d82c7c9b7 (from cb420780b79a1d6f5db2082bdac0b310dbc5ebcc) Author: Michael Richardson <[EMAIL PROTECTED]> Date: Mon Feb 20 10:21:05 2006 -0500 this patch adds a mask operation to the fwmark rule cas