Re: [PATCH 00/13] cxgbe: add CXGBE VF PMD and updates

[Ferruh Yigit <ferruh.yigit@intel.com>]

On 3/10/2018 10:48 PM, Rahul Lakkireddy wrote:
> Patches 1 - 9 add support for cxgbe VF driver.  Patches 10 - 12 fix
> bugs and convert license in cxgbe files to SPDX License Tag.  Patch
> 13 adds compile time option to keep outer vlan tag in Q-in-Q packet.
> 
> Patch 1 adds minimal cxgbe VF driver.

Can you please update driver documentation with new supported device?

> 
> Patch 2 adds firmware mailbox support for VF.
> 
> Patch 3 adds base functions to enable VF ports in subsequent patches.
> 
> Patch 4 adds cxgbe VF probe to initialize VF devices.
> 
> Patch 5 initializes SGE and queues for VF.
> 
> Patch 6 enables RSS for VF.
> 
> Patch 7 updates TX and RX path for VF.
> 
> Patch 8 adds support for VF port statistics.
> 
> Patch 9 adds support to set mac address.
> 
> Patch 10 fixes bug where the other ports under same PF are not closed
> properly.
> 
> Patch 11 exports RSS hash functions in device info and adds check
> to prevent configuring unsupported hash functions.
> 
> Patch 12 converts all cxgbe files to use SPDX license tag.
> 
> Patch 13 adds compile time option to keep outer vlan tag in Q-in-Q
> packet.

We are trying to reduce the config options, is it possible to provide this
functionality with a runtime option (devargs) ?

Or there is already an offload option DEV_RX_OFFLOAD_QINQ_STRIP, I guess this is
different (is it?), if so does it make sense to have another offload option to
cover your case?


> 
> Thanks,
> Rahul
> 
> Kumar Sanghvi (10):
>   cxgbe: add skeleton VF driver
>   cxgbe: add VF firmware mailbox support
>   cxgbe: add base for enabling VF ports
>   cxgbe: add probe to initialize VF devices
>   cxgbe: initialize SGE and queues for VF
>   cxgbe: enable RSS for VF
>   cxgbe: update TX and RX path for VF
>   cxgbe: add VF port statistics
>   cxgbe: add support to set mac address
>   cxgbe: add option to keep outer VLAN tag in Q-in-Q
> 
> Rahul Lakkireddy (3):
>   cxgbe: fix check to close other ports properly
>   cxgbe: export supported RSS hash functions
>   cxgbe: convert to SPDX license tags

<...>