Hi Stephen, > -----Original Message----- > From: Stephen Hemminger <step...@networkplumber.org> > Sent: Wednesday, March 22, 2023 11:00 PM > To: Bing Zhao <bi...@nvidia.com> > Cc: Slava Ovsiienko <viachesl...@nvidia.com>; Matan Azrad > <ma...@nvidia.com>; dev@dpdk.org; Raslan Darawsheh > <rasl...@nvidia.com>; sta...@dpdk.org > Subject: Re: [PATCH v2] common/mlx5: fix the sysfs port name translation > > External email: Use caution opening links or attachments > > > On Wed, 22 Mar 2023 13:34:12 +0200 > Bing Zhao <bi...@nvidia.com> wrote: > > > + tail_nl = strchr(port_name, '\r'); > > + if (tail_nl) > > + *tail_nl = '\0'; > > This is a file provided by Linux kernel, there is no way it could have a CR > character.
Correct, thanks for pointing this. I was overanxious. I will remove this as well.