From: Honnappa Nagarahalli <honnappa.nagaraha...@arm.com>

CONFIG_POOL_MAX_NUM is not used in linux-dpdk.

Signed-off-by: Honnappa Nagarahalli <honnappa.nagaraha...@arm.com>
Reviewed-by: Ola Liljedahl <ola.liljed...@arm.com>
---
/** Email created from pull request 312 (nagarahalli:2.0-pool-ring-alloc-mem)
 ** https://github.com/Linaro/odp/pull/312
 ** Patch: https://github.com/Linaro/odp/pull/312.patch
 ** Base sha: 00c7441fae53949dd87855d48102f932f8f64537
 ** Merge commit sha: 68fae56ca59e4dbc053c018ac8d4520b26e0d528
 **/
 platform/linux-dpdk/include/odp_config_internal.h |  5 -----
 platform/linux-dpdk/pool/dpdk.c                   | 21 +++------------------
 2 files changed, 3 insertions(+), 23 deletions(-)

diff --git a/platform/linux-dpdk/include/odp_config_internal.h 
b/platform/linux-dpdk/include/odp_config_internal.h
index 6aadc93f3..f36fcb929 100644
--- a/platform/linux-dpdk/include/odp_config_internal.h
+++ b/platform/linux-dpdk/include/odp_config_internal.h
@@ -119,11 +119,6 @@ extern "C" {
  */
 #define CONFIG_BURST_SIZE 16
 
-/*
- * Maximum number of events in a pool
- */
-#define CONFIG_POOL_MAX_NUM (1 * 1024 * 1024)
-
 /*
  * Maximum number of events in a thread local pool cache
  */
diff --git a/platform/linux-dpdk/pool/dpdk.c b/platform/linux-dpdk/pool/dpdk.c
index 90d2d7c61..94143dfe8 100644
--- a/platform/linux-dpdk/pool/dpdk.c
+++ b/platform/linux-dpdk/pool/dpdk.c
@@ -137,12 +137,12 @@ static int dpdk_pool_capability(odp_pool_capability_t 
*capa)
        capa->buf.max_pools = ODP_CONFIG_POOLS;
        capa->buf.max_align = ODP_CONFIG_BUFFER_ALIGN_MAX;
        capa->buf.max_size  = MAX_SIZE;
-       capa->buf.max_num   = CONFIG_POOL_MAX_NUM;
+       capa->buf.max_num   = 0;
 
        /* Packet pools */
        capa->pkt.max_pools        = ODP_CONFIG_POOLS;
        capa->pkt.max_len          = 0;
-       capa->pkt.max_num          = CONFIG_POOL_MAX_NUM;
+       capa->pkt.max_num          = 0;
        capa->pkt.min_headroom     = CONFIG_PACKET_HEADROOM;
        capa->pkt.min_tailroom     = CONFIG_PACKET_TAILROOM;
        capa->pkt.max_segs_per_pkt = CONFIG_PACKET_MAX_SEGS;
@@ -152,7 +152,7 @@ static int dpdk_pool_capability(odp_pool_capability_t *capa)
 
        /* Timeout pools */
        capa->tmo.max_pools = ODP_CONFIG_POOLS;
-       capa->tmo.max_num   = CONFIG_POOL_MAX_NUM;
+       capa->tmo.max_num   = 0;
 
        return 0;
 }
@@ -258,11 +258,6 @@ static int check_params(odp_pool_param_t *params)
 
        switch (params->type) {
        case ODP_POOL_BUFFER:
-               if (params->buf.num > capa.buf.max_num) {
-                       printf("buf.num too large %u\n", params->buf.num);
-                       return -1;
-               }
-
                if (params->buf.size > capa.buf.max_size) {
                        printf("buf.size too large %u\n", params->buf.size);
                        return -1;
@@ -276,12 +271,6 @@ static int check_params(odp_pool_param_t *params)
                break;
 
        case ODP_POOL_PACKET:
-               if (params->pkt.num > capa.pkt.max_num) {
-                       printf("pkt.num too large %u\n", params->pkt.num);
-
-                       return -1;
-               }
-
                if (params->pkt.seg_len > capa.pkt.max_seg_len) {
                        printf("pkt.seg_len too large %u\n",
                               params->pkt.seg_len);
@@ -297,10 +286,6 @@ static int check_params(odp_pool_param_t *params)
                break;
 
        case ODP_POOL_TIMEOUT:
-               if (params->tmo.num > capa.tmo.max_num) {
-                       printf("tmo.num too large %u\n", params->tmo.num);
-                       return -1;
-               }
                break;
 
        default:

Reply via email to