rte_device->name copied into eth_dev->name, right now size is same for both but the requirement is not clear.
This patch highlights the relation without changing actual sizes. Signed-off-by: Ferruh Yigit <ferruh.yi...@intel.com> --- lib/librte_eal/common/include/rte_dev.h | 2 ++ lib/librte_eal/common/include/rte_devargs.h | 2 +- lib/librte_ether/rte_ethdev.h | 2 +- 3 files changed, 4 insertions(+), 2 deletions(-) diff --git a/lib/librte_eal/common/include/rte_dev.h b/lib/librte_eal/common/include/rte_dev.h index de20c06..fdd5b11 100644 --- a/lib/librte_eal/common/include/rte_dev.h +++ b/lib/librte_eal/common/include/rte_dev.h @@ -132,6 +132,8 @@ struct rte_driver { const char *alias; /**< Driver alias. */ }; +#define RTE_DEV_NAME_MAX_LEN (32) + /** * A structure describing a generic device. */ diff --git a/lib/librte_eal/common/include/rte_devargs.h b/lib/librte_eal/common/include/rte_devargs.h index 88120a1..b11cbfc 100644 --- a/lib/librte_eal/common/include/rte_devargs.h +++ b/lib/librte_eal/common/include/rte_devargs.h @@ -86,7 +86,7 @@ struct rte_devargs { /** Used if type is RTE_DEVTYPE_VIRTUAL. */ struct { /** Driver name. */ - char drv_name[32]; + char drv_name[RTE_DEV_NAME_MAX_LEN]; } virt; }; /** Arguments string as given by user or "" for no argument. */ diff --git a/lib/librte_ether/rte_ethdev.h b/lib/librte_ether/rte_ethdev.h index 97c3b8e..d1da369 100644 --- a/lib/librte_ether/rte_ethdev.h +++ b/lib/librte_ether/rte_ethdev.h @@ -1689,7 +1689,7 @@ struct rte_eth_dev_sriov { }; #define RTE_ETH_DEV_SRIOV(dev) ((dev)->data->sriov) -#define RTE_ETH_NAME_MAX_LEN (32) +#define RTE_ETH_NAME_MAX_LEN RTE_DEV_NAME_MAX_LEN /** * @internal -- 2.9.3