Re: [ovs-dev] [PATCH v2] datapath: Remove tunnel header caching.

2012-10-29 Thread Pravin Shelar
On Fri, Oct 26, 2012 at 5:15 PM, Jesse Gross wrote: > On Thu, Oct 25, 2012 at 11:14 AM, Pravin B Shelar wrote: >> Tunnel caching was added to reduce CPU utilization on TX path >> by caching packet header, So performance gain is directly proportional >> to number of skbs transferred. But with help

Re: [ovs-dev] [PATCH v2] datapath: Remove tunnel header caching.

2012-10-26 Thread Jesse Gross
On Thu, Oct 25, 2012 at 11:14 AM, Pravin B Shelar wrote: > Tunnel caching was added to reduce CPU utilization on TX path > by caching packet header, So performance gain is directly proportional > to number of skbs transferred. But with help of offloads skb are getting > larger. So there are less n

Re: [ovs-dev] [PATCH v2] datapath: Remove tunnel header caching.

2012-10-26 Thread Jesse Gross
On Fri, Oct 26, 2012 at 11:21 AM, Ben Pfaff wrote: > On Fri, Oct 26, 2012 at 11:19:37AM -0700, Ben Pfaff wrote: >> On Thu, Oct 25, 2012 at 11:14:38AM -0700, Pravin B Shelar wrote: >> > v1-v2: >> > - Fixed capwap fragment case. >> > - simplified tnl_send. >> > --8<--

Re: [ovs-dev] [PATCH v2] datapath: Remove tunnel header caching.

2012-10-26 Thread Ben Pfaff
On Fri, Oct 26, 2012 at 11:19:37AM -0700, Ben Pfaff wrote: > On Thu, Oct 25, 2012 at 11:14:38AM -0700, Pravin B Shelar wrote: > > v1-v2: > > - Fixed capwap fragment case. > > - simplified tnl_send. > > --8<--cut here-->8-- > > > > Tunnel cach

Re: [ovs-dev] [PATCH v2] datapath: Remove tunnel header caching.

2012-10-26 Thread Ben Pfaff
On Thu, Oct 25, 2012 at 11:14:38AM -0700, Pravin B Shelar wrote: > v1-v2: > - Fixed capwap fragment case. > - simplified tnl_send. > --8<--cut here-->8-- > > Tunnel caching was added to reduce CPU utilization on TX path > by caching packe