Re: [PATCH net v3 03/11] bonding: fix unexpected IFF_BONDING bit unset

2019-09-16 Thread Jay Vosburgh
Jiri Pirko wrote: >Mon, Sep 16, 2019 at 03:47:54PM CEST, ap420...@gmail.com wrote: >>The IFF_BONDING means bonding master or bonding slave device. >>->ndo_add_slave() sets IFF_BONDING flag and ->ndo_del_slave() unsets >>IFF_BONDING flag. >> >>bond0<--bond1 >> >>Both bond0 and bond1 are bonding de

Re: [PATCH net v3 03/11] bonding: fix unexpected IFF_BONDING bit unset

2019-09-16 Thread Jiri Pirko
Mon, Sep 16, 2019 at 03:47:54PM CEST, ap420...@gmail.com wrote: >The IFF_BONDING means bonding master or bonding slave device. >->ndo_add_slave() sets IFF_BONDING flag and ->ndo_del_slave() unsets >IFF_BONDING flag. > >bond0<--bond1 > >Both bond0 and bond1 are bonding device and these should keep h

Re: [PATCH net v3 03/11] bonding: fix unexpected IFF_BONDING bit unset

2019-09-16 Thread Jay Vosburgh
Taehee Yoo wrote: >The IFF_BONDING means bonding master or bonding slave device. >->ndo_add_slave() sets IFF_BONDING flag and ->ndo_del_slave() unsets >IFF_BONDING flag. > >bond0<--bond1 > >Both bond0 and bond1 are bonding device and these should keep having >IFF_BONDING flag until they are remov

[PATCH net v3 03/11] bonding: fix unexpected IFF_BONDING bit unset

2019-09-16 Thread Taehee Yoo
The IFF_BONDING means bonding master or bonding slave device. ->ndo_add_slave() sets IFF_BONDING flag and ->ndo_del_slave() unsets IFF_BONDING flag. bond0<--bond1 Both bond0 and bond1 are bonding device and these should keep having IFF_BONDING flag until they are removed. But bond1 would lose IFF