Re: [PATCH net-next] bonding: check slave set command firstly

2019-02-03 Thread David Miller
From: xiangxia.m@gmail.com Date: Fri, 1 Feb 2019 06:07:15 -0800 > From: Tonghao Zhang > > This patch is a little improvement. If user use the > command shown as below, we should print the info [1] > instead of [2]. The eth0 exists actually, and it may > confuse user. > > $ echo "eth0" > /s

Re: [PATCH net-next] bonding: check slave set command firstly

2019-02-02 Thread Tonghao Zhang
On Sat, Feb 2, 2019 at 10:38 PM kbuild test robot wrote: > > Hi Tonghao, > > Thank you for the patch! Perhaps something to improve: > > [auto build test WARNING on net-next/master] > > url: > https://github.com/0day-ci/linux/commits/xiangxia-m-yue-gmail-com/bonding-check-slave-set-command-firs

Re: [PATCH net-next] bonding: check slave set command firstly

2019-02-02 Thread kbuild test robot
Hi Tonghao, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on net-next/master] url: https://github.com/0day-ci/linux/commits/xiangxia-m-yue-gmail-com/bonding-check-slave-set-command-firstly/20190202-215305 config: xtensa-allyesconfig (attached as .config) com

[PATCH net-next] bonding: check slave set command firstly

2019-02-01 Thread xiangxia . m . yue
From: Tonghao Zhang This patch is a little improvement. If user use the command shown as below, we should print the info [1] instead of [2]. The eth0 exists actually, and it may confuse user. $ echo "eth0" > /sys/class/net/bond4/bonding/slaves [1] "bond4: no command found in slaves file - use +