Mon, Nov 13, 2017 at 08:53:57PM CET, xiyou.wangc...@gmail.com wrote: >On Mon, Nov 13, 2017 at 6:05 AM, Jiri Pirko <j...@resnulli.us> wrote: >> From: Jiri Pirko <j...@mellanox.com> >> >> Currently, user may choose to move device that is used by mirred action >> to another network namespace. That is wrong as the action still remains >> in the original namespace and references non-existing ifindex. > >It is a pure display issue, the action itself should function well >because we only use ifindex to lookup netdevice once and >we save the netdevice pointer in action. > >If you really want to fix it, just tell iprout2 to display netnsid together >with ifindex.
It is not only display issue. I think it is wrong to let a netdevice dissapear from underneath the mirred action. You certainly cannot add an action mirred with device from another net namespace. So should we allow that?