[PATCH] Updated skeleton example code to better emit error
Thomas Monjalon
thomas at monjalon.net
Fri Jul 17 15:29:52 CEST 2026
12/05/2026 21:28, Arin Kharkar:
> Previously, the program would check if the number of ports is even and if not, would exit with an error stating "the number of ports must be even". To provide more information, the program now states how many ports were found. This is especially helpful if 0 ports were found, as it explicitly states DPDK wasn't able to find any ports, rather than making the user troubleshoot this.
> ---
> examples/skeleton/basicfwd.c | 6 ++----
> 1 file changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/examples/skeleton/basicfwd.c b/examples/skeleton/basicfwd.c
> index 133293cf15..381e4d4273 100644
> --- a/examples/skeleton/basicfwd.c
> +++ b/examples/skeleton/basicfwd.c
> @@ -188,11 +188,9 @@ main(int argc, char *argv[])
> /* Check that there is an even number of ports to send/receive on. */
> nb_ports = rte_eth_dev_count_avail();
> if (nb_ports < 2 || (nb_ports & 1))
> - rte_exit(EXIT_FAILURE, "Error: number of ports must be even\n");
> + rte_exit(EXIT_FAILURE, "Error: number of ports must be even, found %u ports\n", nb_ports);
>
> - /* Creates a new mempool in memory to hold the mbufs. */
> -
> - /* Allocates mempool to hold the mbufs. 8< */
> + /* Creates and allocates a mempool to hold the mbufs. 8< */
Changing this comment requires changing the doc referencing it.
I've made this change, added your signoff and other details,
and applied, thanks.
More information about the dev
mailing list