[v1,6/7] eal_interrupts: don't reinitialize threads

Message ID 20230815013826.1288972-7-okaya@kernel.org (mailing list archive)
State Superseded, archived
Delegated to: Thomas Monjalon
Headers
Series support reinit flow |

Checks

Context Check Description
ci/checkpatch warning coding style issues

Commit Message

Sinan Kaya Aug. 15, 2023, 1:38 a.m. UTC
  From: Sinan Kaya <okaya@kernel.org>

Signed-off-by: Sinan Kaya <okaya@kernel.org>
---
 lib/eal/linux/eal_interrupts.c | 5 +++++
 1 file changed, 5 insertions(+)
  

Comments

Stephen Hemminger Aug. 15, 2023, 2:20 a.m. UTC | #1
On Mon, 14 Aug 2023 21:38:25 -0400
okaya@kernel.org wrote:

> diff --git a/lib/eal/linux/eal_interrupts.c b/lib/eal/linux/eal_interrupts.c
> index d52ec8eb4c..aa296f153b 100644
> --- a/lib/eal/linux/eal_interrupts.c
> +++ b/lib/eal/linux/eal_interrupts.c
> @@ -1174,6 +1174,10 @@ int
>  rte_eal_intr_init(void)
>  {
>  	int ret = 0;
> +	static int initialized = 0;
> +
> +	if (initialized)
> +		return 0;
>  
>  	/* init the global interrupt source head */
>  	TAILQ_INIT(&intr_sources);
> @@ -1196,6 +1200,7 @@ rte_eal_intr_init(void)
>  			"Failed to create thread for interrupt handling\n");
>  	}
>  
> +	initialized = true;

If you want a bool, use a bool
  

Patch

diff --git a/lib/eal/linux/eal_interrupts.c b/lib/eal/linux/eal_interrupts.c
index d52ec8eb4c..aa296f153b 100644
--- a/lib/eal/linux/eal_interrupts.c
+++ b/lib/eal/linux/eal_interrupts.c
@@ -1174,6 +1174,10 @@  int
 rte_eal_intr_init(void)
 {
 	int ret = 0;
+	static int initialized = 0;
+
+	if (initialized)
+		return 0;
 
 	/* init the global interrupt source head */
 	TAILQ_INIT(&intr_sources);
@@ -1196,6 +1200,7 @@  rte_eal_intr_init(void)
 			"Failed to create thread for interrupt handling\n");
 	}
 
+	initialized = true;
 	return ret;
 }