Re: [PATCH net-next V3 3/3] net: add a sysctl to make auto flowlabel consistent

2017-12-04 Thread kbuild test robot
Hi Shaohua, I love your patch! Perhaps something to improve: [auto build test WARNING on net-next/master] url: https://github.com/0day-ci/linux/commits/Shaohua-Li/net-fix-flowlabel-inconsistency-in-reset-packet/20171205-072626 config: i386-randconfig-x002-12041112 (attached as .config) compi

Re: [PATCH net-next V3 3/3] net: add a sysctl to make auto flowlabel consistent

2017-12-04 Thread kbuild test robot
Hi Shaohua, I love your patch! Yet something to improve: [auto build test ERROR on net-next/master] url: https://github.com/0day-ci/linux/commits/Shaohua-Li/net-fix-flowlabel-inconsistency-in-reset-packet/20171205-072626 config: x86_64-randconfig-x004-201749 (attached as .config) compiler: g

Re: [PATCH net-next V3 3/3] net: add a sysctl to make auto flowlabel consistent

2017-12-01 Thread Tom Herbert
On Fri, Dec 1, 2017 at 3:31 PM, Shaohua Li wrote: > From: Shaohua Li > > Currently if there is negative routing, we change sock's txhash, so the > sock will have a different flowlabel and route to different path. > According to Tom, we'd better to have option to enable this, because some > router

[PATCH net-next V3 3/3] net: add a sysctl to make auto flowlabel consistent

2017-12-01 Thread Shaohua Li
From: Shaohua Li Currently if there is negative routing, we change sock's txhash, so the sock will have a different flowlabel and route to different path. According to Tom, we'd better to have option to enable this, because some routers require flowlabel consistent. By default, we maintain consis