[dpdk-dev] [PATCH v7 21/25] examples/ntb: new link status print format

[Yigit, Ferruh]

> -----Original Message-----
> From: Ivan Dyukov <i.dyukov at samsung.com>
> Sent: Friday, July 10, 2020 8:02 AM
> To: dev at dpdk.org; i.dyukov at samsung.com; v.kuramshin at samsung.com;
> thomas at monjalon.net; david.marchand at redhat.com; Yigit, Ferruh
> <ferruh.yigit at intel.com>; arybchenko at solarflare.com; Zhao1, Wei
> <wei.zhao1 at intel.com>; Guo, Jia <jia.guo at intel.com>; Xing, Beilei
> <beilei.xing at intel.com>; Yang, Qiming <qiming.yang at intel.com>; Lu,
> Wenzhuo <wenzhuo.lu at intel.com>; mb at smartsharesystems.com;
> stephen at networkplumber.org; Chautru, Nicolas
> <nicolas.chautru at intel.com>; Richardson, Bruce
> <bruce.richardson at intel.com>; Ananyev, Konstantin
> <konstantin.ananyev at intel.com>; Dumitrescu, Cristian
> <cristian.dumitrescu at intel.com>; Nicolau, Radu <radu.nicolau at intel.com>;
> akhil.goyal at nxp.com; Doherty, Declan <declan.doherty at intel.com>;
> skori at marvell.com; pbhagavatula at marvell.com; jerinj at marvell.com;
> kirankumark at marvell.com; Hunt, David <david.hunt at intel.com>; Burakov,
> Anatoly <anatoly.burakov at intel.com>; Li, Xiaoyun <xiaoyun.li at intel.com>;
> Wu, Jingjing <jingjing.wu at intel.com>; Mcnamara, John
> <john.mcnamara at intel.com>; Singh, Jasvinder
> <jasvinder.singh at intel.com>; Marohn, Byron <byron.marohn at intel.com>;
> Wang, Yipeng1 <yipeng1.wang at intel.com>
> Subject: [PATCH v7 21/25] examples/ntb: new link status print format
> 
> Add usage of rte_eth_link_strf function to example applications
> 
> Signed-off-by: Ivan Dyukov <i.dyukov at samsung.com>
> ---
>  examples/ntb/ntb_fwd.c | 10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/examples/ntb/ntb_fwd.c b/examples/ntb/ntb_fwd.c index
> eba8ebf9f..84fe374c4 100644
> --- a/examples/ntb/ntb_fwd.c
> +++ b/examples/ntb/ntb_fwd.c
> @@ -729,6 +729,7 @@ start_pkt_fwd(void)
>  	struct rte_eth_link eth_link;
>  	uint32_t lcore_id;
>  	int ret, i;
> +	char link_status_text[60];
> 
>  	ret = ntb_fwd_config_setup();
>  	if (ret < 0) {
> @@ -747,11 +748,10 @@ start_pkt_fwd(void)
>  				return;
>  			}
>  			if (eth_link.link_status) {
> -				printf("Eth%u Link Up. Speed %u Mbps -
> %s\n",
> -					eth_port_id, eth_link.link_speed,
> -					(eth_link.link_duplex ==
> -					 ETH_LINK_FULL_DUPLEX) ?
> -					("full-duplex") : ("half-duplex"));
> +				rte_eth_link_strf(link_status_text, 60, NULL,
> +						    &link);

s/link/eth_link/

.../examples/ntb/ntb_fwd.c:752:11: error: passing argument 4 of ‘rte_eth_link_strf’ from incompatible pointer type [-Werror=incompatible-pointer-types]
  752 |           &link);
      |           ^~~~~
      |           |
      |           int (*)(const char *, const char *)

> +				printf("Eth%u %s", eth_port_id,
> +				       link_status_text);
>  				break;
>  			}
>  		}
> --
> 2.17.1