[1/2] net/ark: fix bug introduced during thread name addition

Message ID 20210511205854.24638-1-ed.czeck@atomicrules.com (mailing list archive)
State Accepted, archived
Delegated to: Ferruh Yigit
Headers
Series [1/2] net/ark: fix bug introduced during thread name addition |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/iol-testing success Testing PASS
ci/iol-abi-testing success Testing PASS
ci/iol-intel-Performance success Performance Testing PASS
ci/iol-intel-Functional success Functional Testing PASS
ci/iol-mellanox-Functional fail Functional Testing issues
ci/iol-mellanox-Performance success Performance Testing PASS

Commit Message

Ed Czeck May 11, 2021, 8:58 p.m. UTC
  Fixes: fdefe038eb9b ("net/ark: set generator delay thread name")
Cc: fengchengwen@huawei.com

Signed-off-by: Ed Czeck <ed.czeck@atomicrules.com>
---
 drivers/net/ark/ark_ethdev.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
  

Comments

Ferruh Yigit May 11, 2021, 10:48 p.m. UTC | #1
On 5/11/2021 9:58 PM, Ed Czeck wrote:
> Fixes: fdefe038eb9b ("net/ark: set generator delay thread name")
> Cc: fengchengwen@huawei.com
> 
> Signed-off-by: Ed Czeck <ed.czeck@atomicrules.com>

Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>

Applied to dpdk-next-net/main, thanks.
(changed the patch title and added a commit log, please check)
  
Ferruh Yigit May 11, 2021, 10:50 p.m. UTC | #2
On 5/11/2021 11:48 PM, Ferruh Yigit wrote:
> On 5/11/2021 9:58 PM, Ed Czeck wrote:
>> Fixes: fdefe038eb9b ("net/ark: set generator delay thread name")
>> Cc: fengchengwen@huawei.com
>>
>> Signed-off-by: Ed Czeck <ed.czeck@atomicrules.com>
> 
> Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>
> 
> Applied to dpdk-next-net/main, thanks.
> (changed the patch title and added a commit log, please check)
> 

The patch title is 1/2, is there a missing patch 2/2?
  
Ed Czeck May 11, 2021, 11:01 p.m. UTC | #3
There is only one patch for this submission.  (The second one remains under
development).
Thanks,


On Tue, May 11, 2021 at 6:50 PM Ferruh Yigit <ferruh.yigit@intel.com> wrote:

> On 5/11/2021 11:48 PM, Ferruh Yigit wrote:
> > On 5/11/2021 9:58 PM, Ed Czeck wrote:
> >> Fixes: fdefe038eb9b ("net/ark: set generator delay thread name")
> >> Cc: fengchengwen@huawei.com
> >>
> >> Signed-off-by: Ed Czeck <ed.czeck@atomicrules.com>
> >
> > Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>
> >
> > Applied to dpdk-next-net/main, thanks.
> > (changed the patch title and added a commit log, please check)
> >
>
> The patch title is 1/2, is there a missing patch 2/2?
>
  

Patch

diff --git a/drivers/net/ark/ark_ethdev.c b/drivers/net/ark/ark_ethdev.c
index 6372cf7740..154ccd7069 100644
--- a/drivers/net/ark/ark_ethdev.c
+++ b/drivers/net/ark/ark_ethdev.c
@@ -569,7 +569,7 @@  eth_ark_dev_start(struct rte_eth_dev *dev)
 		 * This is only used for sanity checking with internal generator
 		 */
 		if (rte_ctrl_thread_create(&thread, "ark-delay-pg", NULL,
-					   ark_pktgen_delay_start, ark) != 0) {
+					   ark_pktgen_delay_start, ark->pg)) {
 			ARK_PMD_LOG(ERR, "Could not create pktgen "
 				    "starter thread\n");
 			return -1;