Re: [PATCH v3 ipsec-next 3/3] xfrm: wrap xfrmdev_ops with offload config

2017-12-20 Thread Marcelo Ricardo Leitner
On Wed, Dec 20, 2017 at 08:22:40AM -0800, Shannon Nelson wrote: > On 12/20/2017 8:03 AM, Marcelo Ricardo Leitner wrote: > > On Tue, Dec 19, 2017 at 03:35:49PM -0800, Shannon Nelson wrote: > > > There's no reason to define netdev->xfrmdev_ops if > > > the offload facility is not CONFIG'd in. > > >

Re: [PATCH v3 ipsec-next 3/3] xfrm: wrap xfrmdev_ops with offload config

2017-12-20 Thread Shannon Nelson
On 12/20/2017 8:03 AM, Marcelo Ricardo Leitner wrote: On Tue, Dec 19, 2017 at 03:35:49PM -0800, Shannon Nelson wrote: There's no reason to define netdev->xfrmdev_ops if the offload facility is not CONFIG'd in. Signed-off-by: Shannon Nelson This one could use a

Re: [PATCH v3 ipsec-next 3/3] xfrm: wrap xfrmdev_ops with offload config

2017-12-20 Thread Marcelo Ricardo Leitner
On Tue, Dec 19, 2017 at 03:35:49PM -0800, Shannon Nelson wrote: > There's no reason to define netdev->xfrmdev_ops if > the offload facility is not CONFIG'd in. > > Signed-off-by: Shannon Nelson This one could use a Fixes tag perhaps: Fixes: d77e38e612a0 ("xfrm: Add an

[PATCH v3 ipsec-next 3/3] xfrm: wrap xfrmdev_ops with offload config

2017-12-19 Thread Shannon Nelson
There's no reason to define netdev->xfrmdev_ops if the offload facility is not CONFIG'd in. Signed-off-by: Shannon Nelson --- include/linux/netdevice.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/linux/netdevice.h