Wed, Jun 14, 2017 at 08:32:22PM CEST, jbenc@redhat.com wrote:

On Tue, 16 May 2017 19:29:36 +0200, Jiri Pirko wrote:

quoted

From: Jiri Pirko <redacted>

Each tc action is terminated by a control action. Each action parses and
prints then intividually. Introduce set of helpers and allow to share
this code.

This patch, as indicated by git bisect, broke tc if there are multiple
actions specified. This somehow non-real life reproducer (the real case
where this broke for me was more complex) works fine with 732f03461bc4:

# tc filter add dev eth0 parent 1: matchall action vlan push id 1 action drop

while with both the current net-next and with e67aba559581, the result
is:

# tc filter add dev eth0 parent 1: matchall action vlan push id 1 action drop
Bad action type action

Isn't it fixed by:

https://git.kernel.org/pub/scm/linux/kernel/git/shemminger/iproute2.git/commit/?h=net-next&id=18f05d06016d9492c87fd105d831de0d6d858f43

?

I will try this tomorrow