d/numam-dpdk
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

rib: constify arguments

Browse Source 
The getter functions should take a constant pointer
to make it clear that node is not modified.

The rib create functions do not modify their config structure.
Mark the config as constant so that programs can pass
simple constant data.

Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
Acked-by: Vladimir Medvedkin <vladimir.medvedkin@intel.com>
This commit is contained in:
Stephen Hemminger 2020-06-25 13:32:06 -07:00 committed by Thomas Monjalon
parent 041a3971c8
commit 7bd83e60bf
4 changed files with 22 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
lib/librte_rib/rte_rib.c
View File

@ -342,7 +342,7 @@ rte_rib_insert(struct rte_rib *rib, uint32_t ip, uint8_t depth)
}
int
rte_rib_get_ip(struct rte_rib_node *node, uint32_t *ip)
rte_rib_get_ip(const struct rte_rib_node *node, uint32_t *ip)
{
if ((node == NULL) || (ip == NULL)) {
rte_errno = EINVAL;
@ -353,7 +353,7 @@ rte_rib_get_ip(struct rte_rib_node *node, uint32_t *ip)
}
int
rte_rib_get_depth(struct rte_rib_node *node, uint8_t *depth)
rte_rib_get_depth(const struct rte_rib_node *node, uint8_t *depth)
{
if ((node == NULL) || (depth == NULL)) {
rte_errno = EINVAL;
@ -370,7 +370,7 @@ rte_rib_get_ext(struct rte_rib_node *node)
}
int
rte_rib_get_nh(struct rte_rib_node *node, uint64_t *nh)
rte_rib_get_nh(const struct rte_rib_node *node, uint64_t *nh)
{
if ((node == NULL) || (nh == NULL)) {
rte_errno = EINVAL;
@ -392,7 +392,7 @@ rte_rib_set_nh(struct rte_rib_node *node, uint64_t nh)
}
struct rte_rib *
rte_rib_create(const char *name, int socket_id, struct rte_rib_conf *conf)
rte_rib_create(const char *name, int socket_id, const struct rte_rib_conf *conf)
{
char mem_name[RTE_RIB_NAMESIZE];
struct rte_rib *rib = NULL;

9
lib/librte_rib/rte_rib.h
View File

@ -178,7 +178,7 @@ rte_rib_insert(struct rte_rib *rib, uint32_t ip, uint8_t depth);
*/
__rte_experimental
int
rte_rib_get_ip(struct rte_rib_node *node, uint32_t *ip);
rte_rib_get_ip(const struct rte_rib_node *node, uint32_t *ip);
/**
* Get a depth from rte_rib_node
@ -193,7 +193,7 @@ rte_rib_get_ip(struct rte_rib_node *node, uint32_t *ip);
*/
__rte_experimental
int
rte_rib_get_depth(struct rte_rib_node *node, uint8_t *depth);
rte_rib_get_depth(const struct rte_rib_node *node, uint8_t *depth);
/**
* Get ext field from the rib node
@ -222,7 +222,7 @@ rte_rib_get_ext(struct rte_rib_node *node);
*/
__rte_experimental
int
rte_rib_get_nh(struct rte_rib_node *node, uint64_t *nh);
rte_rib_get_nh(const struct rte_rib_node *node, uint64_t *nh);
/**
* Set nexthop into the rib node
@ -254,7 +254,8 @@ rte_rib_set_nh(struct rte_rib_node *node, uint64_t nh);
*/
__rte_experimental
struct rte_rib *
rte_rib_create(const char *name, int socket_id, struct rte_rib_conf *conf);
rte_rib_create(const char *name, int socket_id,
const struct rte_rib_conf *conf);
/**
* Find an existing RIB object and return a pointer to it.

10
lib/librte_rib/rte_rib6.c
View File

@ -399,7 +399,8 @@ rte_rib6_insert(struct rte_rib6 *rib,
}
int
rte_rib6_get_ip(struct rte_rib6_node *node, uint8_t ip[RTE_RIB6_IPV6_ADDR_SIZE])
rte_rib6_get_ip(const struct rte_rib6_node *node,
uint8_t ip[RTE_RIB6_IPV6_ADDR_SIZE])
{
if ((node == NULL) || (ip == NULL)) {
rte_errno = EINVAL;
@ -410,7 +411,7 @@ rte_rib6_get_ip(struct rte_rib6_node *node, uint8_t ip[RTE_RIB6_IPV6_ADDR_SIZE])
}
int
rte_rib6_get_depth(struct rte_rib6_node *node, uint8_t *depth)
rte_rib6_get_depth(const struct rte_rib6_node *node, uint8_t *depth)
{
if ((node == NULL) || (depth == NULL)) {
rte_errno = EINVAL;
@ -427,7 +428,7 @@ rte_rib6_get_ext(struct rte_rib6_node *node)
}
int
rte_rib6_get_nh(struct rte_rib6_node *node, uint64_t *nh)
rte_rib6_get_nh(const struct rte_rib6_node *node, uint64_t *nh)
{
if ((node == NULL) || (nh == NULL)) {
rte_errno = EINVAL;
@ -449,7 +450,8 @@ rte_rib6_set_nh(struct rte_rib6_node *node, uint64_t nh)
}
struct rte_rib6 *
rte_rib6_create(const char *name, int socket_id, struct rte_rib6_conf *conf)
rte_rib6_create(const char *name, int socket_id,
const struct rte_rib6_conf *conf)
{
char mem_name[RTE_RIB6_NAMESIZE];
struct rte_rib6 *rib = NULL;

13
lib/librte_rib/rte_rib6.h
View File

@ -77,7 +77,7 @@ rte_rib6_copy_addr(uint8_t *dst, const uint8_t *src)
* 0 otherwise
*/
static inline int
rte_rib6_is_equal(uint8_t *ip1, uint8_t *ip2) {
rte_rib6_is_equal(const uint8_t *ip1, const uint8_t *ip2) {
int i;
if ((ip1 == NULL) || (ip2 == NULL))
@ -234,8 +234,8 @@ rte_rib6_insert(struct rte_rib6 *rib,
*/
__rte_experimental
int
rte_rib6_get_ip(struct rte_rib6_node *node,
uint8_t ip[RTE_RIB6_IPV6_ADDR_SIZE]);
rte_rib6_get_ip(const struct rte_rib6_node *node,
uint8_t ip[RTE_RIB6_IPV6_ADDR_SIZE]);
/**
* Get a depth from rte_rib6_node
@ -250,7 +250,7 @@ rte_rib6_get_ip(struct rte_rib6_node *node,
*/
__rte_experimental
int
rte_rib6_get_depth(struct rte_rib6_node *node, uint8_t *depth);
rte_rib6_get_depth(const struct rte_rib6_node *node, uint8_t *depth);
/**
* Get ext field from the rte_rib6_node
@ -279,7 +279,7 @@ rte_rib6_get_ext(struct rte_rib6_node *node);
*/
__rte_experimental
int
rte_rib6_get_nh(struct rte_rib6_node *node, uint64_t *nh);
rte_rib6_get_nh(const struct rte_rib6_node *node, uint64_t *nh);
/**
* Set nexthop into the rte_rib6_node
@ -311,7 +311,8 @@ rte_rib6_set_nh(struct rte_rib6_node *node, uint64_t nh);
*/
__rte_experimental
struct rte_rib6 *
rte_rib6_create(const char *name, int socket_id, struct rte_rib6_conf *conf);
rte_rib6_create(const char *name, int socket_id,
const struct rte_rib6_conf *conf);
/**
* Find an existing RIB object and return a pointer to it.