* [PATCH] examples/ethtool: fix bug in drvinfo callback
@ 2016-11-18 9:40 Qiming Yang
2016-11-22 1:41 ` [PATCH v2] " Qiming Yang
0 siblings, 1 reply; 4+ messages in thread
From: Qiming Yang @ 2016-11-18 9:40 UTC (permalink / raw)
To: dev; +Cc: remy.horton, Qiming Yang
Function pcmd_drvinfo_callback uses struct info to get
the ethtool infomation of each port. Struct info will
store the infomation of previous port until this
infomation be updated. This patch fixes this issue.
Fixes: bda68ab9d1e7 ("examples/ethtool: add user-space ethtool sample application")
Signed-off-by: Qiming Yang <qiming.yang@intel.com>
---
examples/ethtool/ethtool-app/ethapp.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/examples/ethtool/ethtool-app/ethapp.c b/examples/ethtool/ethtool-app/ethapp.c
index 9b77385..192d941 100644
--- a/examples/ethtool/ethtool-app/ethapp.c
+++ b/examples/ethtool/ethtool-app/ethapp.c
@@ -177,6 +177,7 @@ pcmd_drvinfo_callback(__rte_unused void *ptr_params,
int id_port;
for (id_port = 0; id_port < rte_eth_dev_count(); id_port++) {
+ memset(&info, 0, sizeof(info));
if (rte_ethtool_get_drvinfo(id_port, &info)) {
printf("Error getting info for port %i\n", id_port);
return;
--
2.7.4
^ permalink raw reply related [flat|nested] 4+ messages in thread* [PATCH v2] examples/ethtool: fix bug in drvinfo callback
2016-11-18 9:40 [PATCH] examples/ethtool: fix bug in drvinfo callback Qiming Yang
@ 2016-11-22 1:41 ` Qiming Yang
2016-11-23 1:00 ` Remy Horton
0 siblings, 1 reply; 4+ messages in thread
From: Qiming Yang @ 2016-11-22 1:41 UTC (permalink / raw)
To: dev; +Cc: remy.horton, Qiming Yang
Function pcmd_drvinfo_callback uses struct info to get
the ethtool information of each port. Struct info will
store the information of previous port until this
information be updated. This patch fixes this issue.
Fixes: bda68ab9d1e7 ("examples/ethtool: add user-space ethtool sample application")
Signed-off-by: Qiming Yang <qiming.yang@intel.com>
---
v2 changes:
* fixed the spelling mistake in commit log
---
---
examples/ethtool/ethtool-app/ethapp.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/examples/ethtool/ethtool-app/ethapp.c b/examples/ethtool/ethtool-app/ethapp.c
index 9b77385..192d941 100644
--- a/examples/ethtool/ethtool-app/ethapp.c
+++ b/examples/ethtool/ethtool-app/ethapp.c
@@ -177,6 +177,7 @@ pcmd_drvinfo_callback(__rte_unused void *ptr_params,
int id_port;
for (id_port = 0; id_port < rte_eth_dev_count(); id_port++) {
+ memset(&info, 0, sizeof(info));
if (rte_ethtool_get_drvinfo(id_port, &info)) {
printf("Error getting info for port %i\n", id_port);
return;
--
2.7.4
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [PATCH v2] examples/ethtool: fix bug in drvinfo callback
2016-11-22 1:41 ` [PATCH v2] " Qiming Yang
@ 2016-11-23 1:00 ` Remy Horton
2016-12-20 17:21 ` Thomas Monjalon
0 siblings, 1 reply; 4+ messages in thread
From: Remy Horton @ 2016-11-23 1:00 UTC (permalink / raw)
To: Qiming Yang, dev
Not sure this is a problem in practice, as the same set if fields is
updated each time...at least for now.
On 22/11/2016 09:41, Qiming Yang wrote:
> Function pcmd_drvinfo_callback uses struct info to get
> the ethtool information of each port. Struct info will
> store the information of previous port until this
> information be updated. This patch fixes this issue.
>
> Fixes: bda68ab9d1e7 ("examples/ethtool: add user-space ethtool sample application")
>
> Signed-off-by: Qiming Yang <qiming.yang@intel.com>
Acked-by: Remy Horton <remy.horton@intel.com>
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH v2] examples/ethtool: fix bug in drvinfo callback
2016-11-23 1:00 ` Remy Horton
@ 2016-12-20 17:21 ` Thomas Monjalon
0 siblings, 0 replies; 4+ messages in thread
From: Thomas Monjalon @ 2016-12-20 17:21 UTC (permalink / raw)
To: Qiming Yang; +Cc: dev, Remy Horton
2016-11-23 09:00, Remy Horton:
> Not sure this is a problem in practice, as the same set if fields is
> updated each time...at least for now.
>
> On 22/11/2016 09:41, Qiming Yang wrote:
> > Function pcmd_drvinfo_callback uses struct info to get
> > the ethtool information of each port. Struct info will
> > store the information of previous port until this
> > information be updated. This patch fixes this issue.
> >
> > Fixes: bda68ab9d1e7 ("examples/ethtool: add user-space ethtool sample application")
> >
> > Signed-off-by: Qiming Yang <qiming.yang@intel.com>
>
> Acked-by: Remy Horton <remy.horton@intel.com>
Applied, thanks
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2016-12-20 17:21 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-11-18 9:40 [PATCH] examples/ethtool: fix bug in drvinfo callback Qiming Yang
2016-11-22 1:41 ` [PATCH v2] " Qiming Yang
2016-11-23 1:00 ` Remy Horton
2016-12-20 17:21 ` Thomas Monjalon
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.