From mboxrd@z Thu Jan 1 00:00:00 1970 From: Olivier Matz Subject: [PATCH 04/11] kvargs: remove useless size field Date: Tue, 28 Jan 2014 17:06:37 +0100 Message-ID: <1390925204-10800-5-git-send-email-olivier.matz@6wind.com> References: <1390925204-10800-1-git-send-email-olivier.matz@6wind.com> To: dev-VfR2kkLFssw@public.gmane.org Return-path: In-Reply-To: <1390925204-10800-1-git-send-email-olivier.matz-pdR9zngts4EAvxtiuMwx3w@public.gmane.org> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces-VfR2kkLFssw@public.gmane.org Sender: "dev" This value was not very useful as the size of the table is fixed (equals RTE_KVARGS_MAX). By the way, the memset in the initialization function was wrong (size too short). Even if it was not really an issue since we rely on the "count" field, it is now fixed by this patch. Signed-off-by: Olivier Matz --- lib/librte_kvargs/rte_kvargs.c | 5 ++--- lib/librte_kvargs/rte_kvargs.h | 1 - 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/lib/librte_kvargs/rte_kvargs.c b/lib/librte_kvargs/rte_kvargs.c index 935698c..0bf46fe 100644 --- a/lib/librte_kvargs/rte_kvargs.c +++ b/lib/librte_kvargs/rte_kvargs.c @@ -48,8 +48,7 @@ int rte_kvargs_init(struct rte_kvargs *kvlist) { kvlist->count = 0; - kvlist->size = RTE_KVARGS_MAX; - memset(kvlist->pairs, 0, kvlist->size); + memset(kvlist->pairs, 0, sizeof(kvlist->pairs)); return 0; } @@ -64,7 +63,7 @@ rte_kvargs_add_pair(struct rte_kvargs *kvlist, char *key, char *val) struct rte_kvargs_pair* entry; /* is the list full? */ - if (kvlist->count >= kvlist->size) { + if (kvlist->count >= RTE_KVARGS_MAX) { RTE_LOG(ERR, PMD, "Couldn't add %s, key/value list is full\n", key); return -1; } diff --git a/lib/librte_kvargs/rte_kvargs.h b/lib/librte_kvargs/rte_kvargs.h index 804ea1d..577f00b 100644 --- a/lib/librte_kvargs/rte_kvargs.h +++ b/lib/librte_kvargs/rte_kvargs.h @@ -71,7 +71,6 @@ struct rte_kvargs_pair { /** Store a list of key/value associations */ struct rte_kvargs { unsigned count; /**< number of entries in the list */ - size_t size; /**< maximum number of entries */ struct rte_kvargs_pair pairs[RTE_KVARGS_MAX]; /**< list of key/values */ }; -- 1.8.4.rc3