Wed, Jun 14, 2017 at 08:32:22PM CEST, jb...@redhat.com wrote: >On Tue, 16 May 2017 19:29:36 +0200, Jiri Pirko wrote: >> From: Jiri Pirko <j...@mellanox.com> >> >> 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
