Hi, > -----Original Message----- > From: Bing Zhao <[email protected]> > Sent: Thursday, March 23, 2023 12:54 PM > To: Slava Ovsiienko <[email protected]>; Matan Azrad > <[email protected]>; [email protected] > Cc: [email protected]; Raslan Darawsheh <[email protected]>; > [email protected] > Subject: [PATCH v3] net/mlx5: fix the sysfs port name translation > > With some OFED or upstream kernel of mlx5, the port name fetched from > "/sys/class/net/[DEV]/phys_port_name" may have a tailing "\n" as the > EOL. The sscanf() will return the scanned items number with this EOL. > > In such case, the "equal to" condition is considered as false and > the function mlx5_translate_port_name() will recognize the port type > wrongly with UNKNOWN result. > > The tailing carriage return character should be removed before > calling the mlx5_translate_port_name(), this was already done in the > NL message handling. In the meanwhile, the possible incorrect line > feed character is also taken into consideration. > > Fixes: 654810b56828 ("common/mlx5: share Netlink commands") > Fixes: 420bbdae89f2 ("net/mlx5: fix host physical function representor > naming") > Cc: [email protected] > > Signed-off-by: Bing Zhao <[email protected]>
Patch applied to next-net-mlx, Kindest regards, Raslan Darawsheh

