[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <1198515457.4427.16.camel@localhost>
Date: Mon, 24 Dec 2007 11:57:37 -0500
From: jamal <hadi@...erus.ca>
To: Denys Fedoryshchenko <denys@...p.net.lb>
Cc: netdev@...r.kernel.org, Pablo Neira Ayuso <pablo@...filter.org>
Subject: Re: [PATCH] [IPROUTE2] Compatibility with iptables 1.4.0
I just submitted a tested, slightly reduced patch (example we dont need
to register matches) and it works as expected now. Thanks for the
excellent work Denys.
Pablo, below looks an oversight.
I think there are some libraries like the mark target need also to be
backward compatible.
cheers,
jamal
On Mon, 2007-24-12 at 11:30 -0500, jamal wrote:
> Grr. I tested with a different iptables target and thought the rest work
> in the same manner. iptables needs to be consistent.
>
> -----
> lilsol:~# export IPTABLES_LIB_DIR=/root/iptables-1.4.0/extensions/
> lilsol:~# tc actions add action ipt -j TOS --set-tos
> Maximize-Reliability
> tablename: mangle hook: NF_IP_POST_ROUTING
> target: TOS set Maximize-Reliability index 0
> lilsol:~# tc actions ls action ipt
>
> action order 0: tablename: mangle hook: NF_IP_POST_ROUTING
> target TOS set Maximize-Reliability
> index 2 ref 1 bind 0
> // the above is what i tested
> // below is what you did
> lilsol:~# tc actions add action ipt -j mark --set-mark 3
> /root/iptables-1.4.0/extensions/: cannot read file data: Is a directory
> failed to find target mark
>
> bad action parsing
> parse_action: bad value (5:ipt)!
> Illegal "action"
> Command "add" is unknown, try "tc actions help".
> lilsol:~#
> ------------
>
> Applying your patch ....
> Seems corrupt; let me fix it up, test it and resend it to Stephen.
>
> cheers,
> jamal
>
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists