[Intel-wired-lan] [dev-queue PATCH v2 2/3] i40e: Enable VF specific ethtool statistics via VF Port representor netdevs.

[Samudrala, Sridhar <sridhar.samudrala@intel.com>]

On 8/22/2016 4:13 PM, Jeff Kirsher wrote:
> On Mon, 2016-08-22 at 09:34 -0700, Samudrala, Sridhar wrote:
>>> So one change you need to this is to swap the rx_ and tx_ portion of
>>> the strings.  You have to keep in mind this is a switch port
>>> representation so the Rx and Tx values are swapped.  Every byte the VF
>>> receives is actually a byte that we transmitted to the VF.
>>>
>>> You could probably also add a comment above this block stating that
>>> the two were swapped and why.
>>>
>> Yes. As we are saying that this netdev represents the switch port
>> corresponding to the VF, it makes
>> sense to swap the rx and tx stats.
>>
>> Jeff,
>> Should i submit a patch on top of this patchset as it is already in
>> dev-queue branch?
>> Or resubmit the patchset as a v3 with this update.
> Alex is correct, please re-submit the series with the desired changes to
> this and any other patches, please.  I can drop this series from my tree if
> that helps you.
I submitted a v3 version of these patches that should apply cleanly 
after you drop the v2 series.
Amritha's switchid patch should apply cleanly on top this patchset.

Thanks
Sridhar