On 2021-05-31 7:57 a.m., Davide Caratti wrote:
On Sun, May 30, 2021 at 02:40:49PM +0300, Boris Sukholitko wrote:
Currently vlan modification action checks existence of vlan priority by
comparing it to 0. Therefore it is impossible to modify existing vlan
tag to have priority 0.
hello Boris, thanks for following up!
Change Log:
v2 -> v3:
- Push assumes that the priority is being set
- tcf_vlan_get_fill_size accounts for priority existence
Reviewed-by: Davide Caratti <dcaratti@xxxxxxxxxx>
Looks good to me as well. And thanks for adding the tests!
Acked-by: Jamal Hadi Salim <jhs@xxxxxxxxxxxx>
cheers,
jamal
