[Acked] [PATCH] [Vivid][SRU] hv_netvsc: Clean up two unused variables

[Andy Whitcroft]

On Fri, Dec 04, 2015 at 06:56:08AM -0700, Tim Gardner wrote:
> On 12/04/2015 05:02 AM, Andy Whitcroft wrote:
> > On Fri, Dec 04, 2015 at 12:19:09PM +0900, Seyeong Kim wrote:
> >> From: Haiyang Zhang <haiyangz at microsoft.com>
> >>
> >> BugLink: http://bugs.launchpad.net/bugs/1521053
> >>
> >> The commit
> >>
> >> hv_netvsc: Clean up two unused variables
> > 
> > As this commit only removes unused variables, it is not at all clear how
> > this could cause or fix network slowdowns.  The net effect is to remove
> > 64 bytes from one structure, but this seems unlikely enough to cause
> > significant performance issues.
> > 
> > How was it determined that this was the fix?
> > 
> > -apw
> 
> I'm with Andy on this one. How does this really fix the problem ?

It is sounding like this does actually fix something.  For that to be
true this layout change would need to radically affect cache
performance.  But if the testing can be confirmed then it has my ack.

Acked-by: Andy Whitcroft <apw at canonical.com>

-apw