* [dpdk-dev] [PATCH] drivers/net: document missing speed capabilities feature
@ 2017-05-10 13:10 Ferruh Yigit
2017-05-10 13:49 ` Mcnamara, John
` (2 more replies)
0 siblings, 3 replies; 17+ messages in thread
From: Ferruh Yigit @ 2017-05-10 13:10 UTC (permalink / raw)
To: Shepard Siegel, Ed Czeck, John Miller, John McNamara,
Harish Patil, Rasesh Mody, Rahul Lakkireddy, Hemant Agrawal,
Shreyansh Jain, Wenzhuo Lu, Marcin Wojtas, Michal Krawczyk,
Guy Tzalik, Evgeny Schemeilin, Jing Chen, Helin Zhang,
Jingjing Wu, Konstantin Ananyev, Adrien Mazarguil,
Nelio Laranjeiro, Matej Vido, Pascal Mazon, Yuanhan Liu,
Maxime Coquelin, Shrikrishna Khare
Cc: dev, Ferruh Yigit
Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
---
Please check other features of PMD and document missing ones in features
file.
Thanks.
---
doc/guides/nics/features/ark.ini | 1 +
doc/guides/nics/features/bnx2x.ini | 1 +
doc/guides/nics/features/bnx2x_vf.ini | 1 +
doc/guides/nics/features/cxgbe.ini | 1 +
doc/guides/nics/features/dpaa2.ini | 1 +
doc/guides/nics/features/e1000.ini | 1 +
doc/guides/nics/features/ena.ini | 1 +
doc/guides/nics/features/fm10k.ini | 1 +
doc/guides/nics/features/fm10k_vec.ini | 1 +
doc/guides/nics/features/fm10k_vf.ini | 1 +
doc/guides/nics/features/fm10k_vf_vec.ini | 1 +
doc/guides/nics/features/i40e.ini | 1 +
doc/guides/nics/features/i40e_vec.ini | 1 +
doc/guides/nics/features/igb.ini | 1 +
doc/guides/nics/features/ixgbe.ini | 1 +
doc/guides/nics/features/ixgbe_vec.ini | 1 +
doc/guides/nics/features/mlx4.ini | 1 +
doc/guides/nics/features/szedata2.ini | 1 +
doc/guides/nics/features/tap.ini | 2 +-
doc/guides/nics/features/virtio.ini | 1 +
doc/guides/nics/features/virtio_vec.ini | 1 +
doc/guides/nics/features/vmxnet3.ini | 1 +
22 files changed, 22 insertions(+), 1 deletion(-)
diff --git a/doc/guides/nics/features/ark.ini b/doc/guides/nics/features/ark.ini
index 31a3527..ec8a2b9 100644
--- a/doc/guides/nics/features/ark.ini
+++ b/doc/guides/nics/features/ark.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = Y
Queue start/stop = Y
Jumbo frame = Y
Scattered Rx = Y
diff --git a/doc/guides/nics/features/bnx2x.ini b/doc/guides/nics/features/bnx2x.ini
index 1ad8a3e..b41645a 100644
--- a/doc/guides/nics/features/bnx2x.ini
+++ b/doc/guides/nics/features/bnx2x.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = Y
Link status = Y
Link status event = Y
Promiscuous mode = Y
diff --git a/doc/guides/nics/features/bnx2x_vf.ini b/doc/guides/nics/features/bnx2x_vf.ini
index da9168e..7cd972d 100644
--- a/doc/guides/nics/features/bnx2x_vf.ini
+++ b/doc/guides/nics/features/bnx2x_vf.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = Y
Link status = Y
Link status event = Y
Promiscuous mode = Y
diff --git a/doc/guides/nics/features/cxgbe.ini b/doc/guides/nics/features/cxgbe.ini
index 2e72a10..3d0fde2 100644
--- a/doc/guides/nics/features/cxgbe.ini
+++ b/doc/guides/nics/features/cxgbe.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = Y
Link status = Y
Queue start/stop = Y
MTU update = Y
diff --git a/doc/guides/nics/features/dpaa2.ini b/doc/guides/nics/features/dpaa2.ini
index d43f404..259c2ee 100644
--- a/doc/guides/nics/features/dpaa2.ini
+++ b/doc/guides/nics/features/dpaa2.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = Y
Link status = Y
Queue start/stop = Y
MTU update = Y
diff --git a/doc/guides/nics/features/e1000.ini b/doc/guides/nics/features/e1000.ini
index 260d46d..4c8ac48 100644
--- a/doc/guides/nics/features/e1000.ini
+++ b/doc/guides/nics/features/e1000.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = Y
Link status = Y
Link status event = Y
Rx interrupt = Y
diff --git a/doc/guides/nics/features/ena.ini b/doc/guides/nics/features/ena.ini
index 74969fd..1b33d3b 100644
--- a/doc/guides/nics/features/ena.ini
+++ b/doc/guides/nics/features/ena.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = Y
Queue start/stop = Y
MTU update = Y
Jumbo frame = Y
diff --git a/doc/guides/nics/features/fm10k.ini b/doc/guides/nics/features/fm10k.ini
index 9e1035f..fb8214e 100644
--- a/doc/guides/nics/features/fm10k.ini
+++ b/doc/guides/nics/features/fm10k.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = Y
Rx interrupt = Y
Queue start/stop = Y
Jumbo frame = Y
diff --git a/doc/guides/nics/features/fm10k_vec.ini b/doc/guides/nics/features/fm10k_vec.ini
index 1384ab1..f1a22bf 100644
--- a/doc/guides/nics/features/fm10k_vec.ini
+++ b/doc/guides/nics/features/fm10k_vec.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = Y
Rx interrupt = Y
Queue start/stop = Y
Jumbo frame = Y
diff --git a/doc/guides/nics/features/fm10k_vf.ini b/doc/guides/nics/features/fm10k_vf.ini
index 15de536..92b5edb 100644
--- a/doc/guides/nics/features/fm10k_vf.ini
+++ b/doc/guides/nics/features/fm10k_vf.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = Y
Rx interrupt = Y
Queue start/stop = Y
Jumbo frame = Y
diff --git a/doc/guides/nics/features/fm10k_vf_vec.ini b/doc/guides/nics/features/fm10k_vf_vec.ini
index b32550c..046dcb9 100644
--- a/doc/guides/nics/features/fm10k_vf_vec.ini
+++ b/doc/guides/nics/features/fm10k_vf_vec.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = Y
Rx interrupt = Y
Queue start/stop = Y
Jumbo frame = Y
diff --git a/doc/guides/nics/features/i40e.ini b/doc/guides/nics/features/i40e.ini
index ecabce0..e862712 100644
--- a/doc/guides/nics/features/i40e.ini
+++ b/doc/guides/nics/features/i40e.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = Y
Link status = Y
Link status event = Y
Rx interrupt = Y
diff --git a/doc/guides/nics/features/i40e_vec.ini b/doc/guides/nics/features/i40e_vec.ini
index 206f348..7d7b3a9 100644
--- a/doc/guides/nics/features/i40e_vec.ini
+++ b/doc/guides/nics/features/i40e_vec.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = Y
Link status = Y
Link status event = Y
Rx interrupt = Y
diff --git a/doc/guides/nics/features/igb.ini b/doc/guides/nics/features/igb.ini
index 1145027..712063d 100644
--- a/doc/guides/nics/features/igb.ini
+++ b/doc/guides/nics/features/igb.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = Y
Link status = Y
Link status event = Y
Rx interrupt = Y
diff --git a/doc/guides/nics/features/ixgbe.ini b/doc/guides/nics/features/ixgbe.ini
index 4aa7af6..9ff5d8f 100644
--- a/doc/guides/nics/features/ixgbe.ini
+++ b/doc/guides/nics/features/ixgbe.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = Y
Link status = Y
Link status event = Y
Rx interrupt = Y
diff --git a/doc/guides/nics/features/ixgbe_vec.ini b/doc/guides/nics/features/ixgbe_vec.ini
index 4da8118..4d56df4 100644
--- a/doc/guides/nics/features/ixgbe_vec.ini
+++ b/doc/guides/nics/features/ixgbe_vec.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = Y
Link status = Y
Link status event = Y
Rx interrupt = Y
diff --git a/doc/guides/nics/features/mlx4.ini b/doc/guides/nics/features/mlx4.ini
index 285f0ec..1cf7979 100644
--- a/doc/guides/nics/features/mlx4.ini
+++ b/doc/guides/nics/features/mlx4.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = Y
Link status = Y
Link status event = Y
Removal event = Y
diff --git a/doc/guides/nics/features/szedata2.ini b/doc/guides/nics/features/szedata2.ini
index 624314d..cf41b6c 100644
--- a/doc/guides/nics/features/szedata2.ini
+++ b/doc/guides/nics/features/szedata2.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = Y
Link status = Y
Queue start/stop = Y
Scattered Rx = Y
diff --git a/doc/guides/nics/features/tap.ini b/doc/guides/nics/features/tap.ini
index 3efae75..d410409 100644
--- a/doc/guides/nics/features/tap.ini
+++ b/doc/guides/nics/features/tap.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = Y
Link status = Y
Link status event = Y
Jumbo frame = Y
@@ -13,7 +14,6 @@ Basic stats = Y
Flow API = Y
MTU update = Y
Multicast MAC filter = Y
-Speed capabilities = Y
Unicast MAC filter = Y
Packet type parsing = Y
Flow control = Y
diff --git a/doc/guides/nics/features/virtio.ini b/doc/guides/nics/features/virtio.ini
index 8e3aca1..ca77362 100644
--- a/doc/guides/nics/features/virtio.ini
+++ b/doc/guides/nics/features/virtio.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = Y
Link status = Y
Rx interrupt = Y
Queue start/stop = Y
diff --git a/doc/guides/nics/features/virtio_vec.ini b/doc/guides/nics/features/virtio_vec.ini
index ec93f5c..06e9089 100644
--- a/doc/guides/nics/features/virtio_vec.ini
+++ b/doc/guides/nics/features/virtio_vec.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = Y
Link status = Y
Rx interrupt = Y
Queue start/stop = Y
diff --git a/doc/guides/nics/features/vmxnet3.ini b/doc/guides/nics/features/vmxnet3.ini
index ef95932..c44ca96 100644
--- a/doc/guides/nics/features/vmxnet3.ini
+++ b/doc/guides/nics/features/vmxnet3.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = Y
Link status = Y
Link status event = Y
Queue start/stop = Y
--
2.9.3
^ permalink raw reply [flat|nested] 17+ messages in thread* Re: [dpdk-dev] [PATCH] drivers/net: document missing speed capabilities feature
2017-05-10 13:10 [dpdk-dev] [PATCH] drivers/net: document missing speed capabilities feature Ferruh Yigit
@ 2017-05-10 13:49 ` Mcnamara, John
2017-05-10 13:51 ` Thomas Monjalon
2017-05-15 10:59 ` [dpdk-dev] [PATCH v2] " Ferruh Yigit
2 siblings, 0 replies; 17+ messages in thread
From: Mcnamara, John @ 2017-05-10 13:49 UTC (permalink / raw)
To: Yigit, Ferruh, Shepard Siegel, Ed Czeck, John Miller,
Harish Patil, Rasesh Mody, Rahul Lakkireddy, Hemant Agrawal,
Shreyansh Jain, Lu, Wenzhuo, Marcin Wojtas, Michal Krawczyk,
Guy Tzalik, Evgeny Schemeilin, Chen, Jing D, Zhang, Helin, Wu,
Jingjing, Ananyev, Konstantin, Adrien Mazarguil,
Nelio Laranjeiro, Matej Vido, Pascal Mazon, Yuanhan Liu,
Maxime Coquelin, Shrikrishna Khare
Cc: dev
> -----Original Message-----
> From: Yigit, Ferruh
> Sent: Wednesday, May 10, 2017 2:11 PM
> To: Shepard Siegel <shepard.siegel@atomicrules.com>; Ed Czeck
> <ed.czeck@atomicrules.com>; John Miller <john.miller@atomicrules.com>;
> Mcnamara, John <john.mcnamara@intel.com>; Harish Patil
> <harish.patil@cavium.com>; Rasesh Mody <rasesh.mody@cavium.com>; Rahul
> Lakkireddy <rahul.lakkireddy@chelsio.com>; Hemant Agrawal
> <hemant.agrawal@nxp.com>; Shreyansh Jain <shreyansh.jain@nxp.com>; Lu,
> Wenzhuo <wenzhuo.lu@intel.com>; Marcin Wojtas <mw@semihalf.com>; Michal
> Krawczyk <mk@semihalf.com>; Guy Tzalik <gtzalik@amazon.com>; Evgeny
> Schemeilin <evgenys@amazon.com>; Chen, Jing D <jing.d.chen@intel.com>;
> Zhang, Helin <helin.zhang@intel.com>; Wu, Jingjing
> <jingjing.wu@intel.com>; Ananyev, Konstantin
> <konstantin.ananyev@intel.com>; Adrien Mazarguil
> <adrien.mazarguil@6wind.com>; Nelio Laranjeiro
> <nelio.laranjeiro@6wind.com>; Matej Vido <vido@cesnet.cz>; Pascal Mazon
> <pascal.mazon@6wind.com>; Yuanhan Liu <yuanhan.liu@linux.intel.com>;
> Maxime Coquelin <maxime.coquelin@redhat.com>; Shrikrishna Khare
> <skhare@vmware.com>
> Cc: dev@dpdk.org; Yigit, Ferruh <ferruh.yigit@intel.com>
> Subject: [PATCH] drivers/net: document missing speed capabilities feature
>
> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
>
> Please check other features of PMD and document missing ones in features
> file.
+1 to this suggestion. PMD maintainers should check the Nic Overview table
and update any supported features:
http://dpdk.org/doc/guides/nics/overview.html
Acked-by: John McNamara <john.mcnamara@intel.com>
^ permalink raw reply [flat|nested] 17+ messages in thread* Re: [dpdk-dev] [PATCH] drivers/net: document missing speed capabilities feature
2017-05-10 13:10 [dpdk-dev] [PATCH] drivers/net: document missing speed capabilities feature Ferruh Yigit
2017-05-10 13:49 ` Mcnamara, John
@ 2017-05-10 13:51 ` Thomas Monjalon
2017-05-12 10:49 ` Ferruh Yigit
2017-05-15 10:59 ` [dpdk-dev] [PATCH v2] " Ferruh Yigit
2 siblings, 1 reply; 17+ messages in thread
From: Thomas Monjalon @ 2017-05-10 13:51 UTC (permalink / raw)
To: Ferruh Yigit
Cc: dev, Shepard Siegel, Ed Czeck, John Miller, John McNamara,
Harish Patil, Rasesh Mody, Rahul Lakkireddy, Hemant Agrawal,
Shreyansh Jain, Wenzhuo Lu, Marcin Wojtas, Michal Krawczyk,
Guy Tzalik, Evgeny Schemeilin, Jing Chen, Helin Zhang,
Jingjing Wu, Konstantin Ananyev, Adrien Mazarguil,
Nelio Laranjeiro, Matej Vido, Pascal Mazon, Yuanhan Liu,
Maxime Coquelin, Shrikrishna Khare
10/05/2017 15:10, Ferruh Yigit:
> --- a/doc/guides/nics/features/bnx2x.ini
> +++ b/doc/guides/nics/features/bnx2x.ini
> @@ -4,6 +4,7 @@
> ; Refer to default.ini for the full list of available PMD features.
> ;
> [Features]
> +Speed capabilities = Y
We should validate this feature only if the driver advertise the
right speeds for the device.
Most of drivers advertise every possible speeds for the driver
without considering device limitations.
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [dpdk-dev] [PATCH] drivers/net: document missing speed capabilities feature
2017-05-10 13:51 ` Thomas Monjalon
@ 2017-05-12 10:49 ` Ferruh Yigit
2017-05-12 15:48 ` Thomas Monjalon
0 siblings, 1 reply; 17+ messages in thread
From: Ferruh Yigit @ 2017-05-12 10:49 UTC (permalink / raw)
To: Thomas Monjalon
Cc: dev, Shepard Siegel, Ed Czeck, John Miller, John McNamara,
Harish Patil, Rasesh Mody, Rahul Lakkireddy, Hemant Agrawal,
Shreyansh Jain, Wenzhuo Lu, Marcin Wojtas, Michal Krawczyk,
Guy Tzalik, Evgeny Schemeilin, Jing Chen, Helin Zhang,
Jingjing Wu, Konstantin Ananyev, Adrien Mazarguil,
Nelio Laranjeiro, Matej Vido, Pascal Mazon, Yuanhan Liu,
Maxime Coquelin, Shrikrishna Khare
On 5/10/2017 2:51 PM, Thomas Monjalon wrote:
> 10/05/2017 15:10, Ferruh Yigit:
>> --- a/doc/guides/nics/features/bnx2x.ini
>> +++ b/doc/guides/nics/features/bnx2x.ini
>> @@ -4,6 +4,7 @@
>> ; Refer to default.ini for the full list of available PMD features.
>> ;
>> [Features]
>> +Speed capabilities = Y
>
> We should validate this feature only if the driver advertise the
> right speeds for the device.
Hi Thomas,
Can you please clarify more, what is expected implementation in PMD?
And perhaps a good and a bad sample can be helpful.
Thanks,
ferruh
> Most of drivers advertise every possible speeds for the driver
> without considering device limitations.
>
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [dpdk-dev] [PATCH] drivers/net: document missing speed capabilities feature
2017-05-12 10:49 ` Ferruh Yigit
@ 2017-05-12 15:48 ` Thomas Monjalon
2017-05-15 8:40 ` Ferruh Yigit
2017-05-16 9:39 ` Alejandro Lucero
0 siblings, 2 replies; 17+ messages in thread
From: Thomas Monjalon @ 2017-05-12 15:48 UTC (permalink / raw)
To: Ferruh Yigit
Cc: dev, Shepard Siegel, Ed Czeck, John Miller, John McNamara,
Harish Patil, Rasesh Mody, Rahul Lakkireddy, Hemant Agrawal,
Shreyansh Jain, Wenzhuo Lu, Marcin Wojtas, Michal Krawczyk,
Guy Tzalik, Evgeny Schemeilin, Jing Chen, Helin Zhang,
Jingjing Wu, Konstantin Ananyev, Adrien Mazarguil,
Nelio Laranjeiro, Matej Vido, Pascal Mazon, Yuanhan Liu,
Maxime Coquelin, Shrikrishna Khare
12/05/2017 12:49, Ferruh Yigit:
> On 5/10/2017 2:51 PM, Thomas Monjalon wrote:
> > 10/05/2017 15:10, Ferruh Yigit:
> >> --- a/doc/guides/nics/features/bnx2x.ini
> >> +++ b/doc/guides/nics/features/bnx2x.ini
> >> @@ -4,6 +4,7 @@
> >> ; Refer to default.ini for the full list of available PMD features.
> >> ;
> >> [Features]
> >> +Speed capabilities = Y
> >
> > We should validate this feature only if the driver advertise the
> > right speeds for the device.
>
> Hi Thomas,
>
> Can you please clarify more, what is expected implementation in PMD?
It is expected to advertise only the speeds that the device is
capable to offer.
> And perhaps a good and a bad sample can be helpful.
Good example:
drivers/net/i40e/i40e_ethdev.c
if (I40E_PHY_TYPE_SUPPORT_40G(hw->phy.phy_types))
/* For XL710 */
dev_info->speed_capa = ETH_LINK_SPEED_40G;
else if (I40E_PHY_TYPE_SUPPORT_25G(hw->phy.phy_types))
/* For XXV710 */
dev_info->speed_capa = ETH_LINK_SPEED_25G;
else
/* For X710 */
dev_info->speed_capa = ETH_LINK_SPEED_1G | ETH_LINK_SPEED_10G;
Bad example:
drivers/net/bnx2x/bnx2x_ethdev.c
dev_info->speed_capa = ETH_LINK_SPEED_10G | ETH_LINK_SPEED_20G;
Looking at qlogic.com, only some 57840 adapters are capable of 20G.
^ permalink raw reply [flat|nested] 17+ messages in thread* Re: [dpdk-dev] [PATCH] drivers/net: document missing speed capabilities feature
2017-05-12 15:48 ` Thomas Monjalon
@ 2017-05-15 8:40 ` Ferruh Yigit
2017-05-15 9:33 ` Thomas Monjalon
2017-05-16 9:39 ` Alejandro Lucero
1 sibling, 1 reply; 17+ messages in thread
From: Ferruh Yigit @ 2017-05-15 8:40 UTC (permalink / raw)
To: Thomas Monjalon
Cc: dev, Shepard Siegel, Ed Czeck, John Miller, John McNamara,
Harish Patil, Rasesh Mody, Rahul Lakkireddy, Hemant Agrawal,
Shreyansh Jain, Wenzhuo Lu, Marcin Wojtas, Michal Krawczyk,
Guy Tzalik, Evgeny Schemeilin, Jing Chen, Helin Zhang,
Jingjing Wu, Konstantin Ananyev, Adrien Mazarguil,
Nelio Laranjeiro, Matej Vido, Pascal Mazon, Yuanhan Liu,
Maxime Coquelin, Shrikrishna Khare
On 5/12/2017 4:48 PM, Thomas Monjalon wrote:
> 12/05/2017 12:49, Ferruh Yigit:
>> On 5/10/2017 2:51 PM, Thomas Monjalon wrote:
>>> 10/05/2017 15:10, Ferruh Yigit:
>>>> --- a/doc/guides/nics/features/bnx2x.ini
>>>> +++ b/doc/guides/nics/features/bnx2x.ini
>>>> @@ -4,6 +4,7 @@
>>>> ; Refer to default.ini for the full list of available PMD features.
>>>> ;
>>>> [Features]
>>>> +Speed capabilities = Y
>>>
>>> We should validate this feature only if the driver advertise the
>>> right speeds for the device.
>>
>> Hi Thomas,
>>
>> Can you please clarify more, what is expected implementation in PMD?
>
> It is expected to advertise only the speeds that the device is
> capable to offer.
>
>> And perhaps a good and a bad sample can be helpful.
>
> Good example:
> drivers/net/i40e/i40e_ethdev.c
> if (I40E_PHY_TYPE_SUPPORT_40G(hw->phy.phy_types))
> /* For XL710 */
> dev_info->speed_capa = ETH_LINK_SPEED_40G;
> else if (I40E_PHY_TYPE_SUPPORT_25G(hw->phy.phy_types))
> /* For XXV710 */
> dev_info->speed_capa = ETH_LINK_SPEED_25G;
> else
> /* For X710 */
> dev_info->speed_capa = ETH_LINK_SPEED_1G | ETH_LINK_SPEED_10G;
>
> Bad example:
> drivers/net/bnx2x/bnx2x_ethdev.c
> dev_info->speed_capa = ETH_LINK_SPEED_10G | ETH_LINK_SPEED_20G;
> Looking at qlogic.com, only some 57840 adapters are capable of 20G.
OK, I will update the patch according.
Does it make sense to mark PMDs that report speed_capabilities, but
without checking actual hw, as partial "P", to differentiate them from
the ones that doesn't report at all, also this may help to PMD
maintainers about what to fix.
Thanks,
ferruh
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [dpdk-dev] [PATCH] drivers/net: document missing speed capabilities feature
2017-05-15 8:40 ` Ferruh Yigit
@ 2017-05-15 9:33 ` Thomas Monjalon
0 siblings, 0 replies; 17+ messages in thread
From: Thomas Monjalon @ 2017-05-15 9:33 UTC (permalink / raw)
To: Ferruh Yigit
Cc: dev, Shepard Siegel, Ed Czeck, John Miller, John McNamara,
Harish Patil, Rasesh Mody, Rahul Lakkireddy, Hemant Agrawal,
Shreyansh Jain, Wenzhuo Lu, Marcin Wojtas, Michal Krawczyk,
Guy Tzalik, Evgeny Schemeilin, Jing Chen, Helin Zhang,
Jingjing Wu, Konstantin Ananyev, Adrien Mazarguil,
Nelio Laranjeiro, Matej Vido, Pascal Mazon, Yuanhan Liu,
Maxime Coquelin, Shrikrishna Khare
15/05/2017 10:40, Ferruh Yigit:
> On 5/12/2017 4:48 PM, Thomas Monjalon wrote:
> > 12/05/2017 12:49, Ferruh Yigit:
> >> On 5/10/2017 2:51 PM, Thomas Monjalon wrote:
> >>> 10/05/2017 15:10, Ferruh Yigit:
> >>>> --- a/doc/guides/nics/features/bnx2x.ini
> >>>> +++ b/doc/guides/nics/features/bnx2x.ini
> >>>> @@ -4,6 +4,7 @@
> >>>> ; Refer to default.ini for the full list of available PMD features.
> >>>> ;
> >>>> [Features]
> >>>> +Speed capabilities = Y
> >>>
> >>> We should validate this feature only if the driver advertise the
> >>> right speeds for the device.
> >>
> >> Hi Thomas,
> >>
> >> Can you please clarify more, what is expected implementation in PMD?
> >
> > It is expected to advertise only the speeds that the device is
> > capable to offer.
> >
> >> And perhaps a good and a bad sample can be helpful.
> >
> > Good example:
> > drivers/net/i40e/i40e_ethdev.c
> > if (I40E_PHY_TYPE_SUPPORT_40G(hw->phy.phy_types))
> > /* For XL710 */
> > dev_info->speed_capa = ETH_LINK_SPEED_40G;
> > else if (I40E_PHY_TYPE_SUPPORT_25G(hw->phy.phy_types))
> > /* For XXV710 */
> > dev_info->speed_capa = ETH_LINK_SPEED_25G;
> > else
> > /* For X710 */
> > dev_info->speed_capa = ETH_LINK_SPEED_1G | ETH_LINK_SPEED_10G;
> >
> > Bad example:
> > drivers/net/bnx2x/bnx2x_ethdev.c
> > dev_info->speed_capa = ETH_LINK_SPEED_10G | ETH_LINK_SPEED_20G;
> > Looking at qlogic.com, only some 57840 adapters are capable of 20G.
>
> OK, I will update the patch according.
>
> Does it make sense to mark PMDs that report speed_capabilities, but
> without checking actual hw, as partial "P", to differentiate them from
> the ones that doesn't report at all, also this may help to PMD
> maintainers about what to fix.
Yes, we can use partial "P" here.
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [dpdk-dev] [PATCH] drivers/net: document missing speed capabilities feature
2017-05-12 15:48 ` Thomas Monjalon
2017-05-15 8:40 ` Ferruh Yigit
@ 2017-05-16 9:39 ` Alejandro Lucero
2017-05-16 10:02 ` Thomas Monjalon
1 sibling, 1 reply; 17+ messages in thread
From: Alejandro Lucero @ 2017-05-16 9:39 UTC (permalink / raw)
To: Thomas Monjalon
Cc: Ferruh Yigit, dev, Shepard Siegel, Ed Czeck, John Miller,
John McNamara, Harish Patil, Rasesh Mody, Rahul Lakkireddy,
Hemant Agrawal, Shreyansh Jain, Wenzhuo Lu, Marcin Wojtas,
Michal Krawczyk, Guy Tzalik, Evgeny Schemeilin, Jing Chen,
Helin Zhang, Jingjing Wu, Konstantin Ananyev, Adrien Mazarguil,
Nelio Laranjeiro, Matej Vido, Pascal Mazon, Yuanhan Liu,
Maxime Coquelin, Shrikrishna Khare
Maybe we need another feature for detected link speed report capability.
There is already support for ETH_SPEED_NUM_NONE if the speed is not defined
or it can not be specified by the device. Netronome has some firmwares
without that link speed report capability, although maybe we are not the
only ones with that problem.
What do you think about this?
On Fri, May 12, 2017 at 4:48 PM, Thomas Monjalon <thomas@monjalon.net>
wrote:
> 12/05/2017 12:49, Ferruh Yigit:
> > On 5/10/2017 2:51 PM, Thomas Monjalon wrote:
> > > 10/05/2017 15:10, Ferruh Yigit:
> > >> --- a/doc/guides/nics/features/bnx2x.ini
> > >> +++ b/doc/guides/nics/features/bnx2x.ini
> > >> @@ -4,6 +4,7 @@
> > >> ; Refer to default.ini for the full list of available PMD features.
> > >> ;
> > >> [Features]
> > >> +Speed capabilities = Y
> > >
> > > We should validate this feature only if the driver advertise the
> > > right speeds for the device.
> >
> > Hi Thomas,
> >
> > Can you please clarify more, what is expected implementation in PMD?
>
> It is expected to advertise only the speeds that the device is
> capable to offer.
>
> > And perhaps a good and a bad sample can be helpful.
>
> Good example:
> drivers/net/i40e/i40e_ethdev.c
> if (I40E_PHY_TYPE_SUPPORT_40G(hw->phy.phy_types))
> /* For XL710 */
> dev_info->speed_capa = ETH_LINK_SPEED_40G;
> else if (I40E_PHY_TYPE_SUPPORT_25G(hw->phy.phy_types))
> /* For XXV710 */
> dev_info->speed_capa = ETH_LINK_SPEED_25G;
> else
> /* For X710 */
> dev_info->speed_capa = ETH_LINK_SPEED_1G | ETH_LINK_SPEED_10G;
>
> Bad example:
> drivers/net/bnx2x/bnx2x_ethdev.c
> dev_info->speed_capa = ETH_LINK_SPEED_10G | ETH_LINK_SPEED_20G;
> Looking at qlogic.com, only some 57840 adapters are capable of 20G.
>
>
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [dpdk-dev] [PATCH] drivers/net: document missing speed capabilities feature
2017-05-16 9:39 ` Alejandro Lucero
@ 2017-05-16 10:02 ` Thomas Monjalon
2017-05-16 10:17 ` Alejandro Lucero
0 siblings, 1 reply; 17+ messages in thread
From: Thomas Monjalon @ 2017-05-16 10:02 UTC (permalink / raw)
To: Alejandro Lucero
Cc: Ferruh Yigit, dev, Shepard Siegel, Ed Czeck, John Miller,
John McNamara, Harish Patil, Rasesh Mody, Rahul Lakkireddy,
Hemant Agrawal, Shreyansh Jain, Wenzhuo Lu, Marcin Wojtas,
Michal Krawczyk, Guy Tzalik, Evgeny Schemeilin, Jing Chen,
Helin Zhang, Jingjing Wu, Konstantin Ananyev, Adrien Mazarguil,
Nelio Laranjeiro, Matej Vido, Pascal Mazon, Yuanhan Liu,
Maxime Coquelin, Shrikrishna Khare
16/05/2017 11:39, Alejandro Lucero:
> Maybe we need another feature for detected link speed report capability.
>
> There is already support for ETH_SPEED_NUM_NONE if the speed is not defined
> or it can not be specified by the device. Netronome has some firmwares
> without that link speed report capability, although maybe we are not the
> only ones with that problem.
>
>
> What do you think about this?
Yes, you mean you can advertise a speed capability,
but cannot report the real negotiated speed, right?
So we could add this feature.
There are also virtual drivers which cannot report some real speed.
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [dpdk-dev] [PATCH] drivers/net: document missing speed capabilities feature
2017-05-16 10:02 ` Thomas Monjalon
@ 2017-05-16 10:17 ` Alejandro Lucero
0 siblings, 0 replies; 17+ messages in thread
From: Alejandro Lucero @ 2017-05-16 10:17 UTC (permalink / raw)
To: Thomas Monjalon
Cc: Ferruh Yigit, dev, Shepard Siegel, Ed Czeck, John Miller,
John McNamara, Harish Patil, Rasesh Mody, Rahul Lakkireddy,
Hemant Agrawal, Shreyansh Jain, Wenzhuo Lu, Marcin Wojtas,
Michal Krawczyk, Guy Tzalik, Evgeny Schemeilin, Jing Chen,
Helin Zhang, Jingjing Wu, Konstantin Ananyev, Adrien Mazarguil,
Nelio Laranjeiro, Matej Vido, Pascal Mazon, Yuanhan Liu,
Maxime Coquelin, Shrikrishna Khare
On Tue, May 16, 2017 at 11:02 AM, Thomas Monjalon <thomas@monjalon.net>
wrote:
> 16/05/2017 11:39, Alejandro Lucero:
> > Maybe we need another feature for detected link speed report capability.
> >
> > There is already support for ETH_SPEED_NUM_NONE if the speed is not
> defined
> > or it can not be specified by the device. Netronome has some firmwares
> > without that link speed report capability, although maybe we are not the
> > only ones with that problem.
> >
> >
> > What do you think about this?
>
> Yes, you mean you can advertise a speed capability,
> but cannot report the real negotiated speed, right?
> So we could add this feature.
>
Perfect.
> There are also virtual drivers which cannot report some real speed.
>
>
Yes, that is right. Virtual drivers can not report speed.
We have a problem with a client using pktgen which assumes a device will
report the link speed always. By the way, I will send a patch for avoiding
this issue with pktgen.
Thanks
^ permalink raw reply [flat|nested] 17+ messages in thread
* [dpdk-dev] [PATCH v2] drivers/net: document missing speed capabilities feature
2017-05-10 13:10 [dpdk-dev] [PATCH] drivers/net: document missing speed capabilities feature Ferruh Yigit
2017-05-10 13:49 ` Mcnamara, John
2017-05-10 13:51 ` Thomas Monjalon
@ 2017-05-15 10:59 ` Ferruh Yigit
2017-05-15 12:17 ` Pascal Mazon
` (2 more replies)
2 siblings, 3 replies; 17+ messages in thread
From: Ferruh Yigit @ 2017-05-15 10:59 UTC (permalink / raw)
To: Shepard Siegel, Ed Czeck, John Miller, John McNamara,
Harish Patil, Rasesh Mody, Rahul Lakkireddy, Hemant Agrawal,
Shreyansh Jain, Wenzhuo Lu, Marcin Wojtas, Michal Krawczyk,
Guy Tzalik, Evgeny Schemeilin, Jing Chen, Helin Zhang,
Jingjing Wu, Konstantin Ananyev, Adrien Mazarguil,
Nelio Laranjeiro, Matej Vido, Pascal Mazon, Yuanhan Liu,
Maxime Coquelin, Shrikrishna Khare
Cc: dev, Ferruh Yigit
Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
---
Please fix partially support to fully supported speed capabilities.
Please check other features of PMD and document missing ones in features
file.
Thanks.
v2:
* Mark PMDs that return speed capabilities without checking real
hardware capabilities to Partially supported "P".
* Mark PMDs return fake speed capabilities to "P", like virtio and
vmxnet3.
---
doc/guides/nics/features/ark.ini | 1 +
doc/guides/nics/features/bnx2x.ini | 1 +
doc/guides/nics/features/bnx2x_vf.ini | 1 +
doc/guides/nics/features/cxgbe.ini | 1 +
doc/guides/nics/features/dpaa2.ini | 1 +
doc/guides/nics/features/e1000.ini | 1 +
doc/guides/nics/features/ena.ini | 1 +
doc/guides/nics/features/fm10k.ini | 1 +
doc/guides/nics/features/fm10k_vec.ini | 1 +
doc/guides/nics/features/fm10k_vf.ini | 1 +
doc/guides/nics/features/fm10k_vf_vec.ini | 1 +
doc/guides/nics/features/i40e.ini | 1 +
doc/guides/nics/features/i40e_vec.ini | 1 +
doc/guides/nics/features/igb.ini | 1 +
doc/guides/nics/features/ixgbe.ini | 1 +
doc/guides/nics/features/ixgbe_vec.ini | 1 +
doc/guides/nics/features/mlx4.ini | 1 +
doc/guides/nics/features/szedata2.ini | 1 +
doc/guides/nics/features/tap.ini | 2 +-
doc/guides/nics/features/virtio.ini | 1 +
doc/guides/nics/features/virtio_vec.ini | 1 +
doc/guides/nics/features/vmxnet3.ini | 1 +
22 files changed, 22 insertions(+), 1 deletion(-)
diff --git a/doc/guides/nics/features/ark.ini b/doc/guides/nics/features/ark.ini
index 31a3527..ec8a2b9 100644
--- a/doc/guides/nics/features/ark.ini
+++ b/doc/guides/nics/features/ark.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = Y
Queue start/stop = Y
Jumbo frame = Y
Scattered Rx = Y
diff --git a/doc/guides/nics/features/bnx2x.ini b/doc/guides/nics/features/bnx2x.ini
index 1ad8a3e..3e33e9a 100644
--- a/doc/guides/nics/features/bnx2x.ini
+++ b/doc/guides/nics/features/bnx2x.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = P
Link status = Y
Link status event = Y
Promiscuous mode = Y
diff --git a/doc/guides/nics/features/bnx2x_vf.ini b/doc/guides/nics/features/bnx2x_vf.ini
index da9168e..c270902 100644
--- a/doc/guides/nics/features/bnx2x_vf.ini
+++ b/doc/guides/nics/features/bnx2x_vf.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = P
Link status = Y
Link status event = Y
Promiscuous mode = Y
diff --git a/doc/guides/nics/features/cxgbe.ini b/doc/guides/nics/features/cxgbe.ini
index 2e72a10..2cf0af3 100644
--- a/doc/guides/nics/features/cxgbe.ini
+++ b/doc/guides/nics/features/cxgbe.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = P
Link status = Y
Queue start/stop = Y
MTU update = Y
diff --git a/doc/guides/nics/features/dpaa2.ini b/doc/guides/nics/features/dpaa2.ini
index d43f404..181f449 100644
--- a/doc/guides/nics/features/dpaa2.ini
+++ b/doc/guides/nics/features/dpaa2.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = P
Link status = Y
Queue start/stop = Y
MTU update = Y
diff --git a/doc/guides/nics/features/e1000.ini b/doc/guides/nics/features/e1000.ini
index 260d46d..51ca580 100644
--- a/doc/guides/nics/features/e1000.ini
+++ b/doc/guides/nics/features/e1000.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = P
Link status = Y
Link status event = Y
Rx interrupt = Y
diff --git a/doc/guides/nics/features/ena.ini b/doc/guides/nics/features/ena.ini
index 74969fd..691c1e3 100644
--- a/doc/guides/nics/features/ena.ini
+++ b/doc/guides/nics/features/ena.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = P
Queue start/stop = Y
MTU update = Y
Jumbo frame = Y
diff --git a/doc/guides/nics/features/fm10k.ini b/doc/guides/nics/features/fm10k.ini
index 9e1035f..f0f61a7 100644
--- a/doc/guides/nics/features/fm10k.ini
+++ b/doc/guides/nics/features/fm10k.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = P
Rx interrupt = Y
Queue start/stop = Y
Jumbo frame = Y
diff --git a/doc/guides/nics/features/fm10k_vec.ini b/doc/guides/nics/features/fm10k_vec.ini
index 1384ab1..4917e82 100644
--- a/doc/guides/nics/features/fm10k_vec.ini
+++ b/doc/guides/nics/features/fm10k_vec.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = P
Rx interrupt = Y
Queue start/stop = Y
Jumbo frame = Y
diff --git a/doc/guides/nics/features/fm10k_vf.ini b/doc/guides/nics/features/fm10k_vf.ini
index 15de536..32b93df 100644
--- a/doc/guides/nics/features/fm10k_vf.ini
+++ b/doc/guides/nics/features/fm10k_vf.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = P
Rx interrupt = Y
Queue start/stop = Y
Jumbo frame = Y
diff --git a/doc/guides/nics/features/fm10k_vf_vec.ini b/doc/guides/nics/features/fm10k_vf_vec.ini
index b32550c..6f4a639 100644
--- a/doc/guides/nics/features/fm10k_vf_vec.ini
+++ b/doc/guides/nics/features/fm10k_vf_vec.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = P
Rx interrupt = Y
Queue start/stop = Y
Jumbo frame = Y
diff --git a/doc/guides/nics/features/i40e.ini b/doc/guides/nics/features/i40e.ini
index ecabce0..e862712 100644
--- a/doc/guides/nics/features/i40e.ini
+++ b/doc/guides/nics/features/i40e.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = Y
Link status = Y
Link status event = Y
Rx interrupt = Y
diff --git a/doc/guides/nics/features/i40e_vec.ini b/doc/guides/nics/features/i40e_vec.ini
index 206f348..7d7b3a9 100644
--- a/doc/guides/nics/features/i40e_vec.ini
+++ b/doc/guides/nics/features/i40e_vec.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = Y
Link status = Y
Link status event = Y
Rx interrupt = Y
diff --git a/doc/guides/nics/features/igb.ini b/doc/guides/nics/features/igb.ini
index 1145027..d17cf18 100644
--- a/doc/guides/nics/features/igb.ini
+++ b/doc/guides/nics/features/igb.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = P
Link status = Y
Link status event = Y
Rx interrupt = Y
diff --git a/doc/guides/nics/features/ixgbe.ini b/doc/guides/nics/features/ixgbe.ini
index 4aa7af6..9ff5d8f 100644
--- a/doc/guides/nics/features/ixgbe.ini
+++ b/doc/guides/nics/features/ixgbe.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = Y
Link status = Y
Link status event = Y
Rx interrupt = Y
diff --git a/doc/guides/nics/features/ixgbe_vec.ini b/doc/guides/nics/features/ixgbe_vec.ini
index 4da8118..4d56df4 100644
--- a/doc/guides/nics/features/ixgbe_vec.ini
+++ b/doc/guides/nics/features/ixgbe_vec.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = Y
Link status = Y
Link status event = Y
Rx interrupt = Y
diff --git a/doc/guides/nics/features/mlx4.ini b/doc/guides/nics/features/mlx4.ini
index 285f0ec..fa9030f 100644
--- a/doc/guides/nics/features/mlx4.ini
+++ b/doc/guides/nics/features/mlx4.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = P
Link status = Y
Link status event = Y
Removal event = Y
diff --git a/doc/guides/nics/features/szedata2.ini b/doc/guides/nics/features/szedata2.ini
index 624314d..a0e6f6e 100644
--- a/doc/guides/nics/features/szedata2.ini
+++ b/doc/guides/nics/features/szedata2.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = P
Link status = Y
Queue start/stop = Y
Scattered Rx = Y
diff --git a/doc/guides/nics/features/tap.ini b/doc/guides/nics/features/tap.ini
index 3efae75..d410409 100644
--- a/doc/guides/nics/features/tap.ini
+++ b/doc/guides/nics/features/tap.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = Y
Link status = Y
Link status event = Y
Jumbo frame = Y
@@ -13,7 +14,6 @@ Basic stats = Y
Flow API = Y
MTU update = Y
Multicast MAC filter = Y
-Speed capabilities = Y
Unicast MAC filter = Y
Packet type parsing = Y
Flow control = Y
diff --git a/doc/guides/nics/features/virtio.ini b/doc/guides/nics/features/virtio.ini
index 8e3aca1..16e577d 100644
--- a/doc/guides/nics/features/virtio.ini
+++ b/doc/guides/nics/features/virtio.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = P
Link status = Y
Rx interrupt = Y
Queue start/stop = Y
diff --git a/doc/guides/nics/features/virtio_vec.ini b/doc/guides/nics/features/virtio_vec.ini
index ec93f5c..c06c860 100644
--- a/doc/guides/nics/features/virtio_vec.ini
+++ b/doc/guides/nics/features/virtio_vec.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = P
Link status = Y
Rx interrupt = Y
Queue start/stop = Y
diff --git a/doc/guides/nics/features/vmxnet3.ini b/doc/guides/nics/features/vmxnet3.ini
index ef95932..8f7f8e0 100644
--- a/doc/guides/nics/features/vmxnet3.ini
+++ b/doc/guides/nics/features/vmxnet3.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = P
Link status = Y
Link status event = Y
Queue start/stop = Y
--
2.9.3
^ permalink raw reply [flat|nested] 17+ messages in thread* Re: [dpdk-dev] [PATCH v2] drivers/net: document missing speed capabilities feature
2017-05-15 10:59 ` [dpdk-dev] [PATCH v2] " Ferruh Yigit
@ 2017-05-15 12:17 ` Pascal Mazon
2017-05-15 12:23 ` Ferruh Yigit
2017-05-15 12:30 ` [dpdk-dev] [PATCH v3] " Ferruh Yigit
2017-05-16 9:00 ` [dpdk-dev] [PATCH v2] " Mcnamara, John
2 siblings, 1 reply; 17+ messages in thread
From: Pascal Mazon @ 2017-05-15 12:17 UTC (permalink / raw)
To: Ferruh Yigit
Cc: Shepard Siegel, Ed Czeck, John Miller, John McNamara,
Harish Patil, Rasesh Mody, Rahul Lakkireddy, Hemant Agrawal,
Shreyansh Jain, Wenzhuo Lu, Marcin Wojtas, Michal Krawczyk,
Guy Tzalik, Evgeny Schemeilin, Jing Chen, Helin Zhang,
Jingjing Wu, Konstantin Ananyev, Adrien Mazarguil,
Nelio Laranjeiro, Matej Vido, Yuanhan Liu, Maxime Coquelin,
Shrikrishna Khare, dev
Hi,
In the case of tap PMD, I would say it's rather partially supported (P).
Best regards,
Pascal
On Mon, 15 May 2017 11:59:46 +0100
Ferruh Yigit <ferruh.yigit@intel.com> wrote:
> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
> ---
>
> Please fix partially support to fully supported speed capabilities.
>
> Please check other features of PMD and document missing ones in features
> file.
>
> Thanks.
>
> v2:
> * Mark PMDs that return speed capabilities without checking real
> hardware capabilities to Partially supported "P".
> * Mark PMDs return fake speed capabilities to "P", like virtio and
> vmxnet3.
> ---
> doc/guides/nics/features/ark.ini | 1 +
> doc/guides/nics/features/bnx2x.ini | 1 +
> doc/guides/nics/features/bnx2x_vf.ini | 1 +
> doc/guides/nics/features/cxgbe.ini | 1 +
> doc/guides/nics/features/dpaa2.ini | 1 +
> doc/guides/nics/features/e1000.ini | 1 +
> doc/guides/nics/features/ena.ini | 1 +
> doc/guides/nics/features/fm10k.ini | 1 +
> doc/guides/nics/features/fm10k_vec.ini | 1 +
> doc/guides/nics/features/fm10k_vf.ini | 1 +
> doc/guides/nics/features/fm10k_vf_vec.ini | 1 +
> doc/guides/nics/features/i40e.ini | 1 +
> doc/guides/nics/features/i40e_vec.ini | 1 +
> doc/guides/nics/features/igb.ini | 1 +
> doc/guides/nics/features/ixgbe.ini | 1 +
> doc/guides/nics/features/ixgbe_vec.ini | 1 +
> doc/guides/nics/features/mlx4.ini | 1 +
> doc/guides/nics/features/szedata2.ini | 1 +
> doc/guides/nics/features/tap.ini | 2 +-
> doc/guides/nics/features/virtio.ini | 1 +
> doc/guides/nics/features/virtio_vec.ini | 1 +
> doc/guides/nics/features/vmxnet3.ini | 1 +
> 22 files changed, 22 insertions(+), 1 deletion(-)
[...]
> diff --git a/doc/guides/nics/features/tap.ini b/doc/guides/nics/features/tap.ini
> index 3efae75..d410409 100644
> --- a/doc/guides/nics/features/tap.ini
> +++ b/doc/guides/nics/features/tap.ini
> @@ -4,6 +4,7 @@
> ; Refer to default.ini for the full list of available PMD features.
> ;
> [Features]
> +Speed capabilities = Y
> Link status = Y
> Link status event = Y
> Jumbo frame = Y
> @@ -13,7 +14,6 @@ Basic stats = Y
> Flow API = Y
> MTU update = Y
> Multicast MAC filter = Y
> -Speed capabilities = Y
> Unicast MAC filter = Y
> Packet type parsing = Y
> Flow control = Y
[...]
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [dpdk-dev] [PATCH v2] drivers/net: document missing speed capabilities feature
2017-05-15 12:17 ` Pascal Mazon
@ 2017-05-15 12:23 ` Ferruh Yigit
0 siblings, 0 replies; 17+ messages in thread
From: Ferruh Yigit @ 2017-05-15 12:23 UTC (permalink / raw)
To: Pascal Mazon
Cc: Shepard Siegel, Ed Czeck, John Miller, John McNamara,
Harish Patil, Rasesh Mody, Rahul Lakkireddy, Hemant Agrawal,
Shreyansh Jain, Wenzhuo Lu, Marcin Wojtas, Michal Krawczyk,
Guy Tzalik, Evgeny Schemeilin, Jing Chen, Helin Zhang,
Jingjing Wu, Konstantin Ananyev, Adrien Mazarguil,
Nelio Laranjeiro, Matej Vido, Yuanhan Liu, Maxime Coquelin,
Shrikrishna Khare, dev
On 5/15/2017 1:17 PM, Pascal Mazon wrote:
> Hi,
>
> In the case of tap PMD, I would say it's rather partially supported (P).
Hi Pascal,
Thanks, I will update it.
>
> Best regards,
>
> Pascal
>
^ permalink raw reply [flat|nested] 17+ messages in thread
* [dpdk-dev] [PATCH v3] drivers/net: document missing speed capabilities feature
2017-05-15 10:59 ` [dpdk-dev] [PATCH v2] " Ferruh Yigit
2017-05-15 12:17 ` Pascal Mazon
@ 2017-05-15 12:30 ` Ferruh Yigit
2017-05-17 13:28 ` Mcnamara, John
2017-05-16 9:00 ` [dpdk-dev] [PATCH v2] " Mcnamara, John
2 siblings, 1 reply; 17+ messages in thread
From: Ferruh Yigit @ 2017-05-15 12:30 UTC (permalink / raw)
To: Shepard Siegel, Ed Czeck, John Miller, John McNamara,
Harish Patil, Rasesh Mody, Rahul Lakkireddy, Hemant Agrawal,
Shreyansh Jain, Wenzhuo Lu, Marcin Wojtas, Michal Krawczyk,
Guy Tzalik, Evgeny Schemeilin, Jing Chen, Helin Zhang,
Jingjing Wu, Konstantin Ananyev, Adrien Mazarguil,
Nelio Laranjeiro, Matej Vido, Pascal Mazon, Yuanhan Liu,
Maxime Coquelin, Shrikrishna Khare
Cc: dev, Ferruh Yigit
Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
---
Please fix partially support to fully supported speed capabilities.
Please check other features of PMD and document missing ones in features
file.
Thanks.
v3:
* Update tap PMD speed capability to "P"
v2:
* Mark PMDs that return speed capabilities without checking real
hardware capabilities to Partially supported "P".
* Mark PMDs return fake speed capabilities to "P", like virtio and
vmxnet3.
---
doc/guides/nics/features/ark.ini | 1 +
doc/guides/nics/features/bnx2x.ini | 1 +
doc/guides/nics/features/bnx2x_vf.ini | 1 +
doc/guides/nics/features/cxgbe.ini | 1 +
doc/guides/nics/features/dpaa2.ini | 1 +
doc/guides/nics/features/e1000.ini | 1 +
doc/guides/nics/features/ena.ini | 1 +
doc/guides/nics/features/fm10k.ini | 1 +
doc/guides/nics/features/fm10k_vec.ini | 1 +
doc/guides/nics/features/fm10k_vf.ini | 1 +
doc/guides/nics/features/fm10k_vf_vec.ini | 1 +
doc/guides/nics/features/i40e.ini | 1 +
doc/guides/nics/features/i40e_vec.ini | 1 +
doc/guides/nics/features/igb.ini | 1 +
doc/guides/nics/features/ixgbe.ini | 1 +
doc/guides/nics/features/ixgbe_vec.ini | 1 +
doc/guides/nics/features/mlx4.ini | 1 +
doc/guides/nics/features/szedata2.ini | 1 +
doc/guides/nics/features/tap.ini | 2 +-
doc/guides/nics/features/virtio.ini | 1 +
doc/guides/nics/features/virtio_vec.ini | 1 +
doc/guides/nics/features/vmxnet3.ini | 1 +
22 files changed, 22 insertions(+), 1 deletion(-)
diff --git a/doc/guides/nics/features/ark.ini b/doc/guides/nics/features/ark.ini
index 31a3527..ec8a2b9 100644
--- a/doc/guides/nics/features/ark.ini
+++ b/doc/guides/nics/features/ark.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = Y
Queue start/stop = Y
Jumbo frame = Y
Scattered Rx = Y
diff --git a/doc/guides/nics/features/bnx2x.ini b/doc/guides/nics/features/bnx2x.ini
index 1ad8a3e..3e33e9a 100644
--- a/doc/guides/nics/features/bnx2x.ini
+++ b/doc/guides/nics/features/bnx2x.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = P
Link status = Y
Link status event = Y
Promiscuous mode = Y
diff --git a/doc/guides/nics/features/bnx2x_vf.ini b/doc/guides/nics/features/bnx2x_vf.ini
index da9168e..c270902 100644
--- a/doc/guides/nics/features/bnx2x_vf.ini
+++ b/doc/guides/nics/features/bnx2x_vf.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = P
Link status = Y
Link status event = Y
Promiscuous mode = Y
diff --git a/doc/guides/nics/features/cxgbe.ini b/doc/guides/nics/features/cxgbe.ini
index 2e72a10..2cf0af3 100644
--- a/doc/guides/nics/features/cxgbe.ini
+++ b/doc/guides/nics/features/cxgbe.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = P
Link status = Y
Queue start/stop = Y
MTU update = Y
diff --git a/doc/guides/nics/features/dpaa2.ini b/doc/guides/nics/features/dpaa2.ini
index d43f404..181f449 100644
--- a/doc/guides/nics/features/dpaa2.ini
+++ b/doc/guides/nics/features/dpaa2.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = P
Link status = Y
Queue start/stop = Y
MTU update = Y
diff --git a/doc/guides/nics/features/e1000.ini b/doc/guides/nics/features/e1000.ini
index 260d46d..51ca580 100644
--- a/doc/guides/nics/features/e1000.ini
+++ b/doc/guides/nics/features/e1000.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = P
Link status = Y
Link status event = Y
Rx interrupt = Y
diff --git a/doc/guides/nics/features/ena.ini b/doc/guides/nics/features/ena.ini
index 74969fd..691c1e3 100644
--- a/doc/guides/nics/features/ena.ini
+++ b/doc/guides/nics/features/ena.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = P
Queue start/stop = Y
MTU update = Y
Jumbo frame = Y
diff --git a/doc/guides/nics/features/fm10k.ini b/doc/guides/nics/features/fm10k.ini
index 9e1035f..f0f61a7 100644
--- a/doc/guides/nics/features/fm10k.ini
+++ b/doc/guides/nics/features/fm10k.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = P
Rx interrupt = Y
Queue start/stop = Y
Jumbo frame = Y
diff --git a/doc/guides/nics/features/fm10k_vec.ini b/doc/guides/nics/features/fm10k_vec.ini
index 1384ab1..4917e82 100644
--- a/doc/guides/nics/features/fm10k_vec.ini
+++ b/doc/guides/nics/features/fm10k_vec.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = P
Rx interrupt = Y
Queue start/stop = Y
Jumbo frame = Y
diff --git a/doc/guides/nics/features/fm10k_vf.ini b/doc/guides/nics/features/fm10k_vf.ini
index 15de536..32b93df 100644
--- a/doc/guides/nics/features/fm10k_vf.ini
+++ b/doc/guides/nics/features/fm10k_vf.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = P
Rx interrupt = Y
Queue start/stop = Y
Jumbo frame = Y
diff --git a/doc/guides/nics/features/fm10k_vf_vec.ini b/doc/guides/nics/features/fm10k_vf_vec.ini
index b32550c..6f4a639 100644
--- a/doc/guides/nics/features/fm10k_vf_vec.ini
+++ b/doc/guides/nics/features/fm10k_vf_vec.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = P
Rx interrupt = Y
Queue start/stop = Y
Jumbo frame = Y
diff --git a/doc/guides/nics/features/i40e.ini b/doc/guides/nics/features/i40e.ini
index ecabce0..e862712 100644
--- a/doc/guides/nics/features/i40e.ini
+++ b/doc/guides/nics/features/i40e.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = Y
Link status = Y
Link status event = Y
Rx interrupt = Y
diff --git a/doc/guides/nics/features/i40e_vec.ini b/doc/guides/nics/features/i40e_vec.ini
index 206f348..7d7b3a9 100644
--- a/doc/guides/nics/features/i40e_vec.ini
+++ b/doc/guides/nics/features/i40e_vec.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = Y
Link status = Y
Link status event = Y
Rx interrupt = Y
diff --git a/doc/guides/nics/features/igb.ini b/doc/guides/nics/features/igb.ini
index 1145027..d17cf18 100644
--- a/doc/guides/nics/features/igb.ini
+++ b/doc/guides/nics/features/igb.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = P
Link status = Y
Link status event = Y
Rx interrupt = Y
diff --git a/doc/guides/nics/features/ixgbe.ini b/doc/guides/nics/features/ixgbe.ini
index 4aa7af6..9ff5d8f 100644
--- a/doc/guides/nics/features/ixgbe.ini
+++ b/doc/guides/nics/features/ixgbe.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = Y
Link status = Y
Link status event = Y
Rx interrupt = Y
diff --git a/doc/guides/nics/features/ixgbe_vec.ini b/doc/guides/nics/features/ixgbe_vec.ini
index 4da8118..4d56df4 100644
--- a/doc/guides/nics/features/ixgbe_vec.ini
+++ b/doc/guides/nics/features/ixgbe_vec.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = Y
Link status = Y
Link status event = Y
Rx interrupt = Y
diff --git a/doc/guides/nics/features/mlx4.ini b/doc/guides/nics/features/mlx4.ini
index 285f0ec..fa9030f 100644
--- a/doc/guides/nics/features/mlx4.ini
+++ b/doc/guides/nics/features/mlx4.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = P
Link status = Y
Link status event = Y
Removal event = Y
diff --git a/doc/guides/nics/features/szedata2.ini b/doc/guides/nics/features/szedata2.ini
index 624314d..a0e6f6e 100644
--- a/doc/guides/nics/features/szedata2.ini
+++ b/doc/guides/nics/features/szedata2.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = P
Link status = Y
Queue start/stop = Y
Scattered Rx = Y
diff --git a/doc/guides/nics/features/tap.ini b/doc/guides/nics/features/tap.ini
index 3efae75..04211ad 100644
--- a/doc/guides/nics/features/tap.ini
+++ b/doc/guides/nics/features/tap.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = P
Link status = Y
Link status event = Y
Jumbo frame = Y
@@ -13,7 +14,6 @@ Basic stats = Y
Flow API = Y
MTU update = Y
Multicast MAC filter = Y
-Speed capabilities = Y
Unicast MAC filter = Y
Packet type parsing = Y
Flow control = Y
diff --git a/doc/guides/nics/features/virtio.ini b/doc/guides/nics/features/virtio.ini
index 8e3aca1..16e577d 100644
--- a/doc/guides/nics/features/virtio.ini
+++ b/doc/guides/nics/features/virtio.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = P
Link status = Y
Rx interrupt = Y
Queue start/stop = Y
diff --git a/doc/guides/nics/features/virtio_vec.ini b/doc/guides/nics/features/virtio_vec.ini
index ec93f5c..c06c860 100644
--- a/doc/guides/nics/features/virtio_vec.ini
+++ b/doc/guides/nics/features/virtio_vec.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = P
Link status = Y
Rx interrupt = Y
Queue start/stop = Y
diff --git a/doc/guides/nics/features/vmxnet3.ini b/doc/guides/nics/features/vmxnet3.ini
index ef95932..8f7f8e0 100644
--- a/doc/guides/nics/features/vmxnet3.ini
+++ b/doc/guides/nics/features/vmxnet3.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Speed capabilities = P
Link status = Y
Link status event = Y
Queue start/stop = Y
--
2.9.3
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [dpdk-dev] [PATCH v3] drivers/net: document missing speed capabilities feature
2017-05-15 12:30 ` [dpdk-dev] [PATCH v3] " Ferruh Yigit
@ 2017-05-17 13:28 ` Mcnamara, John
2017-06-12 15:48 ` Ferruh Yigit
0 siblings, 1 reply; 17+ messages in thread
From: Mcnamara, John @ 2017-05-17 13:28 UTC (permalink / raw)
To: Yigit, Ferruh, Shepard Siegel, Ed Czeck, John Miller,
Harish Patil, Rasesh Mody, Rahul Lakkireddy, Hemant Agrawal,
Shreyansh Jain, Lu, Wenzhuo, Marcin Wojtas, Michal Krawczyk,
Guy Tzalik, Evgeny Schemeilin, Chen, Jing D, Zhang, Helin, Wu,
Jingjing, Ananyev, Konstantin, Adrien Mazarguil,
Nelio Laranjeiro, Matej Vido, Pascal Mazon, Yuanhan Liu,
Maxime Coquelin, Shrikrishna Khare
Cc: dev
> -----Original Message-----
> From: Yigit, Ferruh
> Sent: Monday, May 15, 2017 1:31 PM
> To: Shepard Siegel <shepard.siegel@atomicrules.com>; Ed Czeck
> <ed.czeck@atomicrules.com>; John Miller <john.miller@atomicrules.com>;
> Mcnamara, John <john.mcnamara@intel.com>; Harish Patil
> <harish.patil@cavium.com>; Rasesh Mody <rasesh.mody@cavium.com>; Rahul
> Lakkireddy <rahul.lakkireddy@chelsio.com>; Hemant Agrawal
> <hemant.agrawal@nxp.com>; Shreyansh Jain <shreyansh.jain@nxp.com>; Lu,
> Wenzhuo <wenzhuo.lu@intel.com>; Marcin Wojtas <mw@semihalf.com>; Michal
> Krawczyk <mk@semihalf.com>; Guy Tzalik <gtzalik@amazon.com>; Evgeny
> Schemeilin <evgenys@amazon.com>; Chen, Jing D <jing.d.chen@intel.com>;
> Zhang, Helin <helin.zhang@intel.com>; Wu, Jingjing
> <jingjing.wu@intel.com>; Ananyev, Konstantin
> <konstantin.ananyev@intel.com>; Adrien Mazarguil
> <adrien.mazarguil@6wind.com>; Nelio Laranjeiro
> <nelio.laranjeiro@6wind.com>; Matej Vido <vido@cesnet.cz>; Pascal Mazon
> <pascal.mazon@6wind.com>; Yuanhan Liu <yuanhan.liu@linux.intel.com>;
> Maxime Coquelin <maxime.coquelin@redhat.com>; Shrikrishna Khare
> <skhare@vmware.com>
> Cc: dev@dpdk.org; Yigit, Ferruh <ferruh.yigit@intel.com>
> Subject: [PATCH v3] drivers/net: document missing speed capabilities
> feature
>
> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
Acked-by: John McNamara <john.mcnamara@intel.com>
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [dpdk-dev] [PATCH v3] drivers/net: document missing speed capabilities feature
2017-05-17 13:28 ` Mcnamara, John
@ 2017-06-12 15:48 ` Ferruh Yigit
0 siblings, 0 replies; 17+ messages in thread
From: Ferruh Yigit @ 2017-06-12 15:48 UTC (permalink / raw)
To: Mcnamara, John, Shepard Siegel, Ed Czeck, John Miller,
Harish Patil, Rasesh Mody, Rahul Lakkireddy, Hemant Agrawal,
Shreyansh Jain, Lu, Wenzhuo, Marcin Wojtas, Michal Krawczyk,
Guy Tzalik, Evgeny Schemeilin, Chen, Jing D, Zhang, Helin, Wu,
Jingjing, Ananyev, Konstantin, Adrien Mazarguil,
Nelio Laranjeiro, Matej Vido, Pascal Mazon, Yuanhan Liu,
Maxime Coquelin, Shrikrishna Khare
Cc: dev
On 5/17/2017 2:28 PM, Mcnamara, John wrote:
>
>
>> -----Original Message-----
>> From: Yigit, Ferruh
>> Sent: Monday, May 15, 2017 1:31 PM
>> To: Shepard Siegel <shepard.siegel@atomicrules.com>; Ed Czeck
>> <ed.czeck@atomicrules.com>; John Miller <john.miller@atomicrules.com>;
>> Mcnamara, John <john.mcnamara@intel.com>; Harish Patil
>> <harish.patil@cavium.com>; Rasesh Mody <rasesh.mody@cavium.com>; Rahul
>> Lakkireddy <rahul.lakkireddy@chelsio.com>; Hemant Agrawal
>> <hemant.agrawal@nxp.com>; Shreyansh Jain <shreyansh.jain@nxp.com>; Lu,
>> Wenzhuo <wenzhuo.lu@intel.com>; Marcin Wojtas <mw@semihalf.com>; Michal
>> Krawczyk <mk@semihalf.com>; Guy Tzalik <gtzalik@amazon.com>; Evgeny
>> Schemeilin <evgenys@amazon.com>; Chen, Jing D <jing.d.chen@intel.com>;
>> Zhang, Helin <helin.zhang@intel.com>; Wu, Jingjing
>> <jingjing.wu@intel.com>; Ananyev, Konstantin
>> <konstantin.ananyev@intel.com>; Adrien Mazarguil
>> <adrien.mazarguil@6wind.com>; Nelio Laranjeiro
>> <nelio.laranjeiro@6wind.com>; Matej Vido <vido@cesnet.cz>; Pascal Mazon
>> <pascal.mazon@6wind.com>; Yuanhan Liu <yuanhan.liu@linux.intel.com>;
>> Maxime Coquelin <maxime.coquelin@redhat.com>; Shrikrishna Khare
>> <skhare@vmware.com>
>> Cc: dev@dpdk.org; Yigit, Ferruh <ferruh.yigit@intel.com>
>> Subject: [PATCH v3] drivers/net: document missing speed capabilities
>> feature
>>
>> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
>
> Acked-by: John McNamara <john.mcnamara@intel.com>
Applied to dpdk-next-net/master, thanks.
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [dpdk-dev] [PATCH v2] drivers/net: document missing speed capabilities feature
2017-05-15 10:59 ` [dpdk-dev] [PATCH v2] " Ferruh Yigit
2017-05-15 12:17 ` Pascal Mazon
2017-05-15 12:30 ` [dpdk-dev] [PATCH v3] " Ferruh Yigit
@ 2017-05-16 9:00 ` Mcnamara, John
2 siblings, 0 replies; 17+ messages in thread
From: Mcnamara, John @ 2017-05-16 9:00 UTC (permalink / raw)
To: Yigit, Ferruh, Shepard Siegel, Ed Czeck, John Miller,
Harish Patil, Rasesh Mody, Rahul Lakkireddy, Hemant Agrawal,
Shreyansh Jain, Lu, Wenzhuo, Marcin Wojtas, Michal Krawczyk,
Guy Tzalik, Evgeny Schemeilin, Chen, Jing D, Zhang, Helin, Wu,
Jingjing, Ananyev, Konstantin, Adrien Mazarguil,
Nelio Laranjeiro, Matej Vido, Pascal Mazon, Yuanhan Liu,
Maxime Coquelin, Shrikrishna Khare
Cc: dev
> -----Original Message-----
> From: Yigit, Ferruh
> Sent: Monday, May 15, 2017 12:00 PM
> To: Shepard Siegel <shepard.siegel@atomicrules.com>; Ed Czeck
> <ed.czeck@atomicrules.com>; John Miller <john.miller@atomicrules.com>;
> Mcnamara, John <john.mcnamara@intel.com>; Harish Patil
> <harish.patil@cavium.com>; Rasesh Mody <rasesh.mody@cavium.com>; Rahul
> Lakkireddy <rahul.lakkireddy@chelsio.com>; Hemant Agrawal
> <hemant.agrawal@nxp.com>; Shreyansh Jain <shreyansh.jain@nxp.com>; Lu,
> Wenzhuo <wenzhuo.lu@intel.com>; Marcin Wojtas <mw@semihalf.com>; Michal
> Krawczyk <mk@semihalf.com>; Guy Tzalik <gtzalik@amazon.com>; Evgeny
> Schemeilin <evgenys@amazon.com>; Chen, Jing D <jing.d.chen@intel.com>;
> Zhang, Helin <helin.zhang@intel.com>; Wu, Jingjing
> <jingjing.wu@intel.com>; Ananyev, Konstantin
> <konstantin.ananyev@intel.com>; Adrien Mazarguil
> <adrien.mazarguil@6wind.com>; Nelio Laranjeiro
> <nelio.laranjeiro@6wind.com>; Matej Vido <vido@cesnet.cz>; Pascal Mazon
> <pascal.mazon@6wind.com>; Yuanhan Liu <yuanhan.liu@linux.intel.com>;
> Maxime Coquelin <maxime.coquelin@redhat.com>; Shrikrishna Khare
> <skhare@vmware.com>
> Cc: dev@dpdk.org; Yigit, Ferruh <ferruh.yigit@intel.com>
> Subject: [PATCH v2] drivers/net: document missing speed capabilities
> feature
>
> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
This needs to be checked/updated by the PMD maintainers but from a update/documentation point of view:
Acked-by: John McNamara <john.mcnamara@intel.com>
^ permalink raw reply [flat|nested] 17+ messages in thread
end of thread, other threads:[~2017-06-12 15:48 UTC | newest]
Thread overview: 17+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-05-10 13:10 [dpdk-dev] [PATCH] drivers/net: document missing speed capabilities feature Ferruh Yigit
2017-05-10 13:49 ` Mcnamara, John
2017-05-10 13:51 ` Thomas Monjalon
2017-05-12 10:49 ` Ferruh Yigit
2017-05-12 15:48 ` Thomas Monjalon
2017-05-15 8:40 ` Ferruh Yigit
2017-05-15 9:33 ` Thomas Monjalon
2017-05-16 9:39 ` Alejandro Lucero
2017-05-16 10:02 ` Thomas Monjalon
2017-05-16 10:17 ` Alejandro Lucero
2017-05-15 10:59 ` [dpdk-dev] [PATCH v2] " Ferruh Yigit
2017-05-15 12:17 ` Pascal Mazon
2017-05-15 12:23 ` Ferruh Yigit
2017-05-15 12:30 ` [dpdk-dev] [PATCH v3] " Ferruh Yigit
2017-05-17 13:28 ` Mcnamara, John
2017-06-12 15:48 ` Ferruh Yigit
2017-05-16 9:00 ` [dpdk-dev] [PATCH v2] " Mcnamara, John
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).