[v2] net/octeontx: fix vdev name
Checks
Commit Message
The octeontx driver is creating vdev with name "OCTEONTX_PMD"
which is an artifact from how RTE_PMD_REGISTER_VDEV arguments
work.
Change to use the same convention as all the other network
drivers ie "net_octeontx").
Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
Reviewed-by: Ramil Rosen <ramirose@gmail.com>
---
v2 - fix typo in commit message and name should be net_octeontx
not net_octeon
drivers/net/octeontx/octeontx_ethdev.h | 1 +
1 file changed, 1 insertion(+)
Comments
On Wed, 2019-03-13 at 14:58 -0700, Stephen Hemminger wrote:
> The octeontx driver is creating vdev with name "OCTEONTX_PMD"
> which is an artifact from how RTE_PMD_REGISTER_VDEV arguments
> work.
>
> Change to use the same convention as all the other network
> drivers ie "net_octeontx").
>
> Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
> Reviewed-by: Ramil Rosen <ramirose@gmail.com>
> ---
Acked-by: Jerin Jacob <jerinj@marvell.com>
On 3/14/2019 3:35 PM, Jerin Jacob Kollanukkaran wrote:
> On Wed, 2019-03-13 at 14:58 -0700, Stephen Hemminger wrote:
>> The octeontx driver is creating vdev with name "OCTEONTX_PMD"
>> which is an artifact from how RTE_PMD_REGISTER_VDEV arguments
>> work.
>>
>> Change to use the same convention as all the other network
>> drivers ie "net_octeontx").
>>
>> Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
>> Reviewed-by: Ramil Rosen <ramirose@gmail.com>
>> ---
>
> Acked-by: Jerin Jacob <jerinj@marvell.com>
>
Hi Jerin, Stephen,
Can you please provide fixes line, (which commit broke the pmd name)?
Thanks,
ferruh
On Tue, 19 Mar 2019 18:59:21 +0000
Ferruh Yigit <ferruh.yigit@intel.com> wrote:
> On 3/14/2019 3:35 PM, Jerin Jacob Kollanukkaran wrote:
> > On Wed, 2019-03-13 at 14:58 -0700, Stephen Hemminger wrote:
> >> The octeontx driver is creating vdev with name "OCTEONTX_PMD"
> >> which is an artifact from how RTE_PMD_REGISTER_VDEV arguments
> >> work.
> >>
> >> Change to use the same convention as all the other network
> >> drivers ie "net_octeontx").
> >>
> >> Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
> >> Reviewed-by: Ramil Rosen <ramirose@gmail.com>
> >> ---
> >
> > Acked-by: Jerin Jacob <jerinj@marvell.com>
> >
>
> Hi Jerin, Stephen,
>
> Can you please provide fixes line, (which commit broke the pmd name)?
It goes all the way back to original submission:
Fixes: f7be70e5130e ("net/octeontx: add net device probe and remove")
On 3/19/2019 10:19 PM, Stephen Hemminger wrote:
> On Tue, 19 Mar 2019 18:59:21 +0000
> Ferruh Yigit <ferruh.yigit@intel.com> wrote:
>
>> On 3/14/2019 3:35 PM, Jerin Jacob Kollanukkaran wrote:
>>> On Wed, 2019-03-13 at 14:58 -0700, Stephen Hemminger wrote:
>>>> The octeontx driver is creating vdev with name "OCTEONTX_PMD"
>>>> which is an artifact from how RTE_PMD_REGISTER_VDEV arguments
>>>> work.
>>>>
>>>> Change to use the same convention as all the other network
>>>> drivers ie "net_octeontx").
>>>>
>>>> Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
>>>> Reviewed-by: Ramil Rosen <ramirose@gmail.com>
>>>> ---
>>>
>>> Acked-by: Jerin Jacob <jerinj@marvell.com>
>>>
>>
>> Hi Jerin, Stephen,
>>
>> Can you please provide fixes line, (which commit broke the pmd name)?
>
> It goes all the way back to original submission:
>
> Fixes: f7be70e5130e ("net/octeontx: add net device probe and remove")
>
Indeed, I was assuming it has been broken somewhere on the way, but is seems no.
Applied to dpdk-next-net/master, thanks.
@@ -21,6 +21,7 @@
#include "base/octeontx_pkovf.h"
#include "base/octeontx_io.h"
+#define OCTEONTX_PMD net_octeontx
#define OCTEONTX_VDEV_DEFAULT_MAX_NR_PORT 12
#define OCTEONTX_VDEV_NR_PORT_ARG ("nr_port")
#define OCTEONTX_MAX_NAME_LEN 32