RE: [PATCH net-next] net: increase LL_MAX_HEADER if HYPERV_NET is enabled

2015-11-03 Thread KY Srinivasan
icrosoft.com> > Subject: Re: [PATCH net-next] net: increase LL_MAX_HEADER if HYPERV_NET > is enabled > > From: Eric Dumazet <eric.duma...@gmail.com> > Date: Sun, 01 Nov 2015 09:20:59 -0800 > > > From: Eric Dumazet <eduma...@google.com> > > > > My recen

Re: [PATCH net-next] net: increase LL_MAX_HEADER if HYPERV_NET is enabled

2015-11-03 Thread David Miller
From: KY Srinivasan Date: Tue, 3 Nov 2015 07:59:36 + > I have implemented the scheme we had discussed a few weeks ago. In > this new implementation our driver is NOT requesting addition > headroom - rndis header and the per packet state is being maintained > outside of

RE: [PATCH net-next] net: increase LL_MAX_HEADER if HYPERV_NET is enabled

2015-11-03 Thread Haiyang Zhang
google.com; > netdev@vger.kernel.org > Subject: Re: [PATCH net-next] net: increase LL_MAX_HEADER if HYPERV_NET > is enabled > > On Tue, 2015-11-03 at 10:33 -0500, David Miller wrote: > > From: KY Srinivasan <k...@microsoft.com> > > Date: Tue, 3 Nov 2015 07:59:36

RE: [PATCH net-next] net: increase LL_MAX_HEADER if HYPERV_NET is enabled

2015-11-03 Thread KY Srinivasan
@vger.kernel.org > Subject: Re: [PATCH net-next] net: increase LL_MAX_HEADER if HYPERV_NET > is enabled > > From: KY Srinivasan <k...@microsoft.com> > Date: Tue, 3 Nov 2015 07:59:36 + > > > I have implemented the scheme we had discussed a few weeks ago. In >

Re: [PATCH net-next] net: increase LL_MAX_HEADER if HYPERV_NET is enabled

2015-11-03 Thread David Miller
From: Haiyang Zhang Date: Tue, 3 Nov 2015 17:34:47 + > But we still keep this busy return in our code, just for "weird corner cases". The queue_stopped condition must be precise. If you cannot enqueue a maximally segmented SKB, you must stop the queue. -- To

RE: [PATCH net-next] net: increase LL_MAX_HEADER if HYPERV_NET is enabled

2015-11-03 Thread Haiyang Zhang
@vger.kernel.org > Subject: Re: [PATCH net-next] net: increase LL_MAX_HEADER if HYPERV_NET > is enabled > > From: Haiyang Zhang <haiya...@microsoft.com> > Date: Tue, 3 Nov 2015 17:34:47 + > > > But we still keep this busy return in our code, just for "weird cor

Re: [PATCH net-next] net: increase LL_MAX_HEADER if HYPERV_NET is enabled

2015-11-03 Thread David Miller
Cc: eric.duma...@gmail.com; KY Srinivasan <k...@microsoft.com>; >> eduma...@google.com; netdev@vger.kernel.org >> Subject: Re: [PATCH net-next] net: increase LL_MAX_HEADER if HYPERV_NET >> is enabled >> >> From: Haiyang Zhang <haiya...@microsoft.com> >&g

RE: [PATCH net-next] net: increase LL_MAX_HEADER if HYPERV_NET is enabled

2015-11-03 Thread Haiyang Zhang
mailto:da...@davemloft.net] > >> Sent: Tuesday, November 3, 2015 1:20 PM > >> To: Haiyang Zhang <haiya...@microsoft.com> > >> Cc: eric.duma...@gmail.com; KY Srinivasan <k...@microsoft.com>; > >> eduma...@google.com; netdev@vger.kernel.org > >

Re: [PATCH net-next] net: increase LL_MAX_HEADER if HYPERV_NET is enabled

2015-11-03 Thread Eric Dumazet
On Tue, 2015-11-03 at 10:33 -0500, David Miller wrote: > From: KY Srinivasan > Date: Tue, 3 Nov 2015 07:59:36 + > > > I have implemented the scheme we had discussed a few weeks ago. In > > this new implementation our driver is NOT requesting addition > > headroom - rndis

Re: [PATCH net-next] net: increase LL_MAX_HEADER if HYPERV_NET is enabled

2015-11-01 Thread David Miller
From: Eric Dumazet Date: Sun, 01 Nov 2015 09:20:59 -0800 > From: Eric Dumazet > > My recent commit, attaching SYNACK messages to request sockets > exposed a too small LL_MAX_HEADER when netvsc_drv.c is in use, > because this driver sets a

Re: [PATCH net-next] net: increase LL_MAX_HEADER if HYPERV_NET is enabled

2015-11-01 Thread Eric Dumazet
On Sun, 2015-11-01 at 15:58 -0500, David Miller wrote: > From: Eric Dumazet > Date: Sun, 01 Nov 2015 09:20:59 -0800 > > > From: Eric Dumazet > > > > My recent commit, attaching SYNACK messages to request sockets > > exposed a too small LL_MAX_HEADER