mbox series

[0/4] features for hns3 PMD

Message ID 1623553508-5478-1-git-send-email-humin29@huawei.com (mailing list archive)
Headers
Series features for hns3 PMD |

Message

humin (Q) June 13, 2021, 3:05 a.m. UTC
  This patch set contains 4 features for hns3 PMD:
add query basic info support for VF
support for VF modify VLAN filter state
support multiple TC MAC pause
supports disabling PFC by dev configure API

Chengchang Tang (2):
  net/hns3: add query basic info support for VF
  net/hns3: support for VF modify VLAN filter state

Huisong Li (2):
  net/hns3: support multiple TC MAC pause
  net/hns3: supports disabling PFC by dev configure API

 drivers/net/hns3/hns3_cmd.h       |  9 ++++
 drivers/net/hns3/hns3_dcb.c       | 61 +++++++++++++------------
 drivers/net/hns3/hns3_ethdev.c    |  5 +-
 drivers/net/hns3/hns3_ethdev.h    |  7 +++
 drivers/net/hns3/hns3_ethdev_vf.c | 96 ++++++++++++++++++++++++++++++---------
 drivers/net/hns3/hns3_mbx.h       | 11 ++++-
 6 files changed, 138 insertions(+), 51 deletions(-)
  

Comments

Andrew Rybchenko June 14, 2021, 3 p.m. UTC | #1
On 6/13/21 6:05 AM, Min Hu (Connor) wrote:
> This patch set contains 4 features for hns3 PMD:
> add query basic info support for VF
> support for VF modify VLAN filter state
> support multiple TC MAC pause
> supports disabling PFC by dev configure API
> 
> Chengchang Tang (2):
>    net/hns3: add query basic info support for VF
>    net/hns3: support for VF modify VLAN filter state
> 
> Huisong Li (2):
>    net/hns3: support multiple TC MAC pause
>    net/hns3: supports disabling PFC by dev configure API
> 
>   drivers/net/hns3/hns3_cmd.h       |  9 ++++
>   drivers/net/hns3/hns3_dcb.c       | 61 +++++++++++++------------
>   drivers/net/hns3/hns3_ethdev.c    |  5 +-
>   drivers/net/hns3/hns3_ethdev.h    |  7 +++
>   drivers/net/hns3/hns3_ethdev_vf.c | 96 ++++++++++++++++++++++++++++++---------
>   drivers/net/hns3/hns3_mbx.h       | 11 ++++-
>   6 files changed, 138 insertions(+), 51 deletions(-)
> 

Overall LGTM, but build breakage must be fixed.
  
humin (Q) July 9, 2021, 1:32 a.m. UTC | #2
Thanks Andrew,
"net/hns3: supports disabling PFC by dev configure API"
this patch could be abandoned.

The other three can be applied.


在 2021/6/14 23:00, Andrew Rybchenko 写道:
> On 6/13/21 6:05 AM, Min Hu (Connor) wrote:
>> This patch set contains 4 features for hns3 PMD:
>> add query basic info support for VF
>> support for VF modify VLAN filter state
>> support multiple TC MAC pause
>> supports disabling PFC by dev configure API
>>
>> Chengchang Tang (2):
>>    net/hns3: add query basic info support for VF
>>    net/hns3: support for VF modify VLAN filter state
>>
>> Huisong Li (2):
>>    net/hns3: support multiple TC MAC pause
>>    net/hns3: supports disabling PFC by dev configure API
>>
>>   drivers/net/hns3/hns3_cmd.h       |  9 ++++
>>   drivers/net/hns3/hns3_dcb.c       | 61 +++++++++++++------------
>>   drivers/net/hns3/hns3_ethdev.c    |  5 +-
>>   drivers/net/hns3/hns3_ethdev.h    |  7 +++
>>   drivers/net/hns3/hns3_ethdev_vf.c | 96 
>> ++++++++++++++++++++++++++++++---------
>>   drivers/net/hns3/hns3_mbx.h       | 11 ++++-
>>   6 files changed, 138 insertions(+), 51 deletions(-)
>>
> 
> Overall LGTM, but build breakage must be fixed.
> .
  
Andrew Rybchenko July 9, 2021, 9:20 a.m. UTC | #3
On 7/9/21 4:32 AM, Min Hu (Connor) wrote:
> Thanks Andrew,
> "net/hns3: supports disabling PFC by dev configure API"
> this patch could be abandoned.
> 
> The other three can be applied.
> 
> 
> 在 2021/6/14 23:00, Andrew Rybchenko 写道:
>> On 6/13/21 6:05 AM, Min Hu (Connor) wrote:
>>> This patch set contains 4 features for hns3 PMD:
>>> add query basic info support for VF
>>> support for VF modify VLAN filter state
>>> support multiple TC MAC pause
>>> supports disabling PFC by dev configure API
>>>
>>> Chengchang Tang (2):
>>>    net/hns3: add query basic info support for VF
>>>    net/hns3: support for VF modify VLAN filter state
>>>
>>> Huisong Li (2):
>>>    net/hns3: support multiple TC MAC pause
>>>    net/hns3: supports disabling PFC by dev configure API
>>>
>>>   drivers/net/hns3/hns3_cmd.h       |  9 ++++
>>>   drivers/net/hns3/hns3_dcb.c       | 61 +++++++++++++------------
>>>   drivers/net/hns3/hns3_ethdev.c    |  5 +-
>>>   drivers/net/hns3/hns3_ethdev.h    |  7 +++
>>>   drivers/net/hns3/hns3_ethdev_vf.c | 96
>>> ++++++++++++++++++++++++++++++---------
>>>   drivers/net/hns3/hns3_mbx.h       | 11 ++++-
>>>   6 files changed, 138 insertions(+), 51 deletions(-)
>>>
>>
>> Overall LGTM, but build breakage must be fixed.
>> .

Please, send a new version with the patch dropped.
I'd like patchwork automation to check it once
again.
  
humin (Q) July 10, 2021, 1:59 a.m. UTC | #4
在 2021/7/9 17:20, Andrew Rybchenko 写道:
> On 7/9/21 4:32 AM, Min Hu (Connor) wrote:
>> Thanks Andrew,
>> "net/hns3: supports disabling PFC by dev configure API"
>> this patch could be abandoned.
>>
>> The other three can be applied.
>>
>>
>> 在 2021/6/14 23:00, Andrew Rybchenko 写道:
>>> On 6/13/21 6:05 AM, Min Hu (Connor) wrote:
>>>> This patch set contains 4 features for hns3 PMD:
>>>> add query basic info support for VF
>>>> support for VF modify VLAN filter state
>>>> support multiple TC MAC pause
>>>> supports disabling PFC by dev configure API
>>>>
>>>> Chengchang Tang (2):
>>>>     net/hns3: add query basic info support for VF
>>>>     net/hns3: support for VF modify VLAN filter state
>>>>
>>>> Huisong Li (2):
>>>>     net/hns3: support multiple TC MAC pause
>>>>     net/hns3: supports disabling PFC by dev configure API
>>>>
>>>>    drivers/net/hns3/hns3_cmd.h       |  9 ++++
>>>>    drivers/net/hns3/hns3_dcb.c       | 61 +++++++++++++------------
>>>>    drivers/net/hns3/hns3_ethdev.c    |  5 +-
>>>>    drivers/net/hns3/hns3_ethdev.h    |  7 +++
>>>>    drivers/net/hns3/hns3_ethdev_vf.c | 96
>>>> ++++++++++++++++++++++++++++++---------
>>>>    drivers/net/hns3/hns3_mbx.h       | 11 ++++-
>>>>    6 files changed, 138 insertions(+), 51 deletions(-)
>>>>
>>>
>>> Overall LGTM, but build breakage must be fixed.
>>> .
> 
> Please, send a new version with the patch dropped.
> I'd like patchwork automation to check it once
> again.
> 
Hi, v2 has been sent, please check it out.
> .
>