mbox series

[v5,0/4] Support ipv6 flow rules

Message ID cover.1588991713.git.cloud.wangxiaoyun@huawei.com (mailing list archive)
Headers
Series Support ipv6 flow rules |

Message

Wangxiaoyun (Cloud) May 9, 2020, 4:04 a.m. UTC
  This patch adds jumbo frame offload flag, increases
judgment whether Tx/Rx queues is non-null when free
resources, and if card mode does not support nic, 
driver will be initialized failed, supports ipv6
flow director for BGP or ICMP packets.

--
v4->v5:
  - increase Tx/Rx queues non-null judgment
  - add jumbo frame offload flag
  - increase judgment for support NIC or not
  - support ipv6 flow rules
  
v3->v4:
  - fix PF firmware hotactive problem
  - optimize log style
  - adds txq xstats members
  
v2->v3:
  - fix FW hotactive problem
  - optimize log style
  
v1->v2:
  - Fix LRO problems
  - Fix hotupdate firmware problem
  - Optimize doorbell area initialization 
  - Remove 40GE Mezz card id
  - Add Fdir filter type
  - Optimize log files
  - Support pause flow control

v1:
  - Allocate IO memory with socketid

Xiaoyun wang (4):
  net/hinic: increase Tx/Rx queues non-null judgment
  net/hinic: add jumbo frame offload flag
  net/hinic: increase judgment for support NIC or not
  net/hinic/base: support ipv6 flow rules

 drivers/net/hinic/base/hinic_pmd_cfg.c   |   3 +-
 drivers/net/hinic/base/hinic_pmd_cmdq.h  |   2 +-
 drivers/net/hinic/base/hinic_pmd_hwif.c  |   2 +-
 drivers/net/hinic/base/hinic_pmd_nicio.c |   1 -
 drivers/net/hinic/hinic_pmd_ethdev.c     |  32 +++--
 drivers/net/hinic/hinic_pmd_ethdev.h     |  57 ++++++++-
 drivers/net/hinic/hinic_pmd_flow.c       | 209 ++++++++++++++++++++++++++++---
 drivers/net/hinic/hinic_pmd_rx.c         |   3 +-
 drivers/net/hinic/hinic_pmd_tx.c         |   5 +-
 9 files changed, 281 insertions(+), 33 deletions(-)
  

Comments

Ferruh Yigit May 11, 2020, 7:04 p.m. UTC | #1
On 5/9/2020 5:04 AM, Xiaoyun wang wrote:
> This patch adds jumbo frame offload flag, increases
> judgment whether Tx/Rx queues is non-null when free
> resources, and if card mode does not support nic, 
> driver will be initialized failed, supports ipv6
> flow director for BGP or ICMP packets.
> 
> --
> v4->v5:
>   - increase Tx/Rx queues non-null judgment
>   - add jumbo frame offload flag
>   - increase judgment for support NIC or not
>   - support ipv6 flow rules
>   
> v3->v4:
>   - fix PF firmware hotactive problem
>   - optimize log style
>   - adds txq xstats members
>   
> v2->v3:
>   - fix FW hotactive problem
>   - optimize log style
>   
> v1->v2:
>   - Fix LRO problems
>   - Fix hotupdate firmware problem
>   - Optimize doorbell area initialization 
>   - Remove 40GE Mezz card id
>   - Add Fdir filter type
>   - Optimize log files
>   - Support pause flow control
> 
> v1:
>   - Allocate IO memory with socketid
> 
> Xiaoyun wang (4):
>   net/hinic: increase Tx/Rx queues non-null judgment
>   net/hinic: add jumbo frame offload flag
>   net/hinic: increase judgment for support NIC or not
>   net/hinic/base: support ipv6 flow rules
> 

Hi Xiaoyun,

Some of above are separate patchsets, and previous ones already merged. I guess
there is a confusion about versioning (vN) in patchsets.

The versioning is a new version of patchset if a new version required/requested.
It is not incremental changes to a driver (or a component)

The previous patchset has been applied in v4, for this new patchset it should
start from v1 again.

(Also previous v2 and v3 were different patchsets, and should reset the
versioning ... )
  
Ferruh Yigit May 11, 2020, 8:20 p.m. UTC | #2
On 5/11/2020 8:04 PM, Ferruh Yigit wrote:
> On 5/9/2020 5:04 AM, Xiaoyun wang wrote:
>> This patch adds jumbo frame offload flag, increases
>> judgment whether Tx/Rx queues is non-null when free
>> resources, and if card mode does not support nic, 
>> driver will be initialized failed, supports ipv6
>> flow director for BGP or ICMP packets.
>>
>> --
>> v4->v5:
>>   - increase Tx/Rx queues non-null judgment
>>   - add jumbo frame offload flag
>>   - increase judgment for support NIC or not
>>   - support ipv6 flow rules
>>   
>> v3->v4:
>>   - fix PF firmware hotactive problem
>>   - optimize log style
>>   - adds txq xstats members
>>   
>> v2->v3:
>>   - fix FW hotactive problem
>>   - optimize log style
>>   
>> v1->v2:
>>   - Fix LRO problems
>>   - Fix hotupdate firmware problem
>>   - Optimize doorbell area initialization 
>>   - Remove 40GE Mezz card id
>>   - Add Fdir filter type
>>   - Optimize log files
>>   - Support pause flow control
>>
>> v1:
>>   - Allocate IO memory with socketid
>>
>> Xiaoyun wang (4):
>>   net/hinic: increase Tx/Rx queues non-null judgment
>>   net/hinic: add jumbo frame offload flag
>>   net/hinic: increase judgment for support NIC or not
>>   net/hinic/base: support ipv6 flow rules
>>
> 
> Hi Xiaoyun,
> 
> Some of above are separate patchsets, and previous ones already merged. I guess
> there is a confusion about versioning (vN) in patchsets.
> 
> The versioning is a new version of patchset if a new version required/requested.
> It is not incremental changes to a driver (or a component)
> 
> The previous patchset has been applied in v4, for this new patchset it should
> start from v1 again.
> 
> (Also previous v2 and v3 were different patchsets, and should reset the
> versioning ... )
> 

Hi Xiaoyun,

It looks patches in the set don't depend on each other, so I will get the set
partially, except patch 1/4, to be able to include them into the -rc2 release.
(Otherwise it may not be possible to merge flow API patch in -rc3 or later.)

Please send the patch 1/4 separately.

Except 1/4,
Series applied to dpdk-next-net/master, thanks.