eal: make max interrupt vectors configurable

Message ID 20190326125327.5264-1-pbhagavatula@marvell.com (mailing list archive)
State Superseded, archived
Delegated to: Thomas Monjalon
Headers
Series eal: make max interrupt vectors configurable |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation success Compilation OK
ci/mellanox-Performance-Testing fail Performance Testing issues
ci/intel-Performance-Testing success Performance Testing PASS

Commit Message

Pavan Nikhilesh Bhagavatula March 26, 2019, 12:54 p.m. UTC
  From: Pavan Nikhilesh <pbhagavatula@marvell.com>

Make max interrupt vectors configurable so that platforms can
choose interrupt vector limit.

Signed-off-by: Pavan Nikhilesh <pbhagavatula@marvell.com>
---
 config/common_base                                 | 1 +
 lib/librte_eal/common/include/rte_eal_interrupts.h | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)
  

Comments

Jerin Jacob Kollanukkaran March 26, 2019, 1:08 p.m. UTC | #1
On Tue, 2019-03-26 at 12:54 +0000, Pavan Nikhilesh Bhagavatula wrote:
> From: Pavan Nikhilesh <pbhagavatula@marvell.com>
> 
> Make max interrupt vectors configurable so that platforms can
> choose interrupt vector limit.
> 
> Signed-off-by: Pavan Nikhilesh <pbhagavatula@marvell.com>
> ---
>  config/common_base                                 

Please update meson config too.
  
Pavan Nikhilesh Bhagavatula March 26, 2019, 1:11 p.m. UTC | #2
> -----Original Message-----
> From: Jerin Jacob Kollanukkaran
> Sent: Tuesday, March 26, 2019 6:38 PM
> To: thomas@monjalon.net; Pavan Nikhilesh Bhagavatula
> <pbhagavatula@marvell.com>
> Cc: dev@dpdk.org
> Subject: Re: [dpdk-dev] [PATCH] eal: make max interrupt vectors
> configurable
> 
> On Tue, 2019-03-26 at 12:54 +0000, Pavan Nikhilesh Bhagavatula wrote:
> > From: Pavan Nikhilesh <pbhagavatula@marvell.com>
> >
> > Make max interrupt vectors configurable so that platforms can choose
> > interrupt vector limit.
> >
> > Signed-off-by: Pavan Nikhilesh <pbhagavatula@marvell.com>
> > ---
> >  config/common_base
> 
> Please update meson config too.

Will update meson in the next version.

Thanks,
Pavan.
  

Patch

diff --git a/config/common_base b/config/common_base
index 0b09a9348..e84f8a233 100644
--- a/config/common_base
+++ b/config/common_base
@@ -68,6 +68,7 @@  CONFIG_RTE_MAX_LCORE=128
 CONFIG_RTE_MAX_NUMA_NODES=8
 CONFIG_RTE_MAX_HEAPS=32
 CONFIG_RTE_MAX_MEMSEG_LISTS=64
+CONFIG_RTE_MAX_INTERRUPT_VECTORS=32
 # each memseg list will be limited to either RTE_MAX_MEMSEG_PER_LIST pages
 # or RTE_MAX_MEM_MB_PER_LIST megabytes worth of memory, whichever is smaller
 CONFIG_RTE_MAX_MEMSEG_PER_LIST=8192
diff --git a/lib/librte_eal/common/include/rte_eal_interrupts.h b/lib/librte_eal/common/include/rte_eal_interrupts.h
index 9d302f412..19e44019f 100644
--- a/lib/librte_eal/common/include/rte_eal_interrupts.h
+++ b/lib/librte_eal/common/include/rte_eal_interrupts.h
@@ -17,7 +17,7 @@ 
 #ifndef _RTE_EAL_INTERRUPTS_H_
 #define _RTE_EAL_INTERRUPTS_H_
 
-#define RTE_MAX_RXTX_INTR_VEC_ID     32
+#define RTE_MAX_RXTX_INTR_VEC_ID      RTE_MAX_INTERRUPT_VECTORS
 #define RTE_INTR_VEC_ZERO_OFFSET      0
 #define RTE_INTR_VEC_RXTX_OFFSET      1