diff mbox series

net/ark: support multi-port pkt generation

Message ID 20220226124454.68600-1-john.miller@atomicrules.com (mailing list archive)
State Superseded, archived
Delegated to: Ferruh Yigit
Headers
Series net/ark: support multi-port pkt generation |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/iol-mellanox-Performance success Performance Testing PASS
ci/iol-broadcom-Performance success Performance Testing PASS
ci/github-robot: build success github build: passed
ci/iol-broadcom-Functional success Functional Testing PASS
ci/iol-intel-Performance success Performance Testing PASS
ci/iol-intel-Functional success Functional Testing PASS
ci/iol-x86_64-compile-testing success Testing PASS
ci/iol-aarch64-unit-testing success Testing PASS
ci/iol-aarch64-compile-testing success Testing PASS
ci/iol-x86_64-unit-testing success Testing PASS
ci/iol-abi-testing success Testing PASS
ci/Intel-compilation success Compilation OK
ci/intel-Testing success Testing PASS

Commit Message

John Miller Feb. 26, 2022, 12:44 p.m. UTC 
  Added support for packet generation in
multi-port Arkville implementations. The packet
generator is a singleton within the device but is
capable of generating packets for any port within
one device.

Signed-off-by: John Miller <john.miller@atomicrules.com>
---
 drivers/net/ark/ark_ethdev.c | 4 +++-
 drivers/net/ark/ark_global.h | 1 +
 2 files changed, 4 insertions(+), 1 deletion(-)

Comments

Ferruh Yigit Feb. 28, 2022, 2:51 p.m. UTC | #1 
On 2/26/2022 12:44 PM, John Miller wrote:
> Added support for packet generation in
> multi-port Arkville implementations. The packet
> generator is a singleton within the device but is
> capable of generating packets for any port within
> one device.
> 
> Signed-off-by: John Miller <john.miller@atomicrules.com>
> ---
>   drivers/net/ark/ark_ethdev.c | 4 +++-
>   drivers/net/ark/ark_global.h | 1 +
>   2 files changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/net/ark/ark_ethdev.c b/drivers/net/ark/ark_ethdev.c
> index 230a1272e9..980e1a4a3b 100644
> --- a/drivers/net/ark/ark_ethdev.c
> +++ b/drivers/net/ark/ark_ethdev.c
> @@ -441,6 +441,7 @@ ark_config_device(struct rte_eth_dev *dev)
>   	 * known state
>   	 */
>   	ark->start_pg = 0;
> +	ark->pg_running = 0;
>   	ark->pg = ark_pktgen_init(ark->pktgen.v, 0, 1);
>   	if (ark->pg == NULL)
>   		return -1;
> @@ -562,7 +563,7 @@ eth_ark_dev_start(struct rte_eth_dev *dev)
>   	if (ark->start_pg)
>   		ark_pktchkr_run(ark->pc);
>   
> -	if (ark->start_pg && (dev->data->port_id == 0)) {
> +	if (ark->start_pg && !ark->pg_running) {
>   		pthread_t thread;
>   
>   		/* Delay packet generatpr start allow the hardware to be ready
> @@ -574,6 +575,7 @@ eth_ark_dev_start(struct rte_eth_dev *dev)
>   				    "starter thread\n");
>   			return -1;
>   		}
> +		ark->pg_running = 1;

Should there be a place 'ark->pg_running' set back to '0',
like 'eth_ark_dev_stop()'?

Or is this delay thread is only required once in first init
per port?


Btw, if there will change, you may consider thread name,
"ark-delay-pg", get port_id to differentiate multiple threads,
like "ark-delay-pg_0", "ark-delay-pg_1" ...

>   	}
>   
>   	if (ark->user_ext.dev_start)
> diff --git a/drivers/net/ark/ark_global.h b/drivers/net/ark/ark_global.h
> index 49193ac5b3..3c3a712bc8 100644
> --- a/drivers/net/ark/ark_global.h
> +++ b/drivers/net/ark/ark_global.h
> @@ -107,6 +107,7 @@ struct ark_adapter {
>   
>   	/* Pointers to packet generator and checker */
>   	int start_pg;
> +	uint16_t pg_running;
>   	ark_pkt_gen_t pg;
>   	ark_pkt_chkr_t pc;
>   	ark_pkt_dir_t pd;
diff mbox series

Patch

diff --git a/drivers/net/ark/ark_ethdev.c b/drivers/net/ark/ark_ethdev.c
index 230a1272e9..980e1a4a3b 100644
--- a/drivers/net/ark/ark_ethdev.c
+++ b/drivers/net/ark/ark_ethdev.c
@@ -441,6 +441,7 @@  ark_config_device(struct rte_eth_dev *dev)
 	 * known state
 	 */
 	ark->start_pg = 0;
+	ark->pg_running = 0;
 	ark->pg = ark_pktgen_init(ark->pktgen.v, 0, 1);
 	if (ark->pg == NULL)
 		return -1;
@@ -562,7 +563,7 @@  eth_ark_dev_start(struct rte_eth_dev *dev)
 	if (ark->start_pg)
 		ark_pktchkr_run(ark->pc);
 
-	if (ark->start_pg && (dev->data->port_id == 0)) {
+	if (ark->start_pg && !ark->pg_running) {
 		pthread_t thread;
 
 		/* Delay packet generatpr start allow the hardware to be ready
@@ -574,6 +575,7 @@  eth_ark_dev_start(struct rte_eth_dev *dev)
 				    "starter thread\n");
 			return -1;
 		}
+		ark->pg_running = 1;
 	}
 
 	if (ark->user_ext.dev_start)
diff --git a/drivers/net/ark/ark_global.h b/drivers/net/ark/ark_global.h
index 49193ac5b3..3c3a712bc8 100644
--- a/drivers/net/ark/ark_global.h
+++ b/drivers/net/ark/ark_global.h
@@ -107,6 +107,7 @@  struct ark_adapter {
 
 	/* Pointers to packet generator and checker */
 	int start_pg;
+	uint16_t pg_running;
 	ark_pkt_gen_t pg;
 	ark_pkt_chkr_t pc;
 	ark_pkt_dir_t pd;