Hi Stephen, > -----Original Message----- > From: Stephen Hemminger <[email protected]> > Sent: Tuesday, June 1, 2021 1:39 PM > To: Xia, Chenbo <[email protected]> > Cc: [email protected]; [email protected]; Liang, Cunming > <[email protected]>; Wu, Jingjing <[email protected]>; Burakov, > Anatoly <[email protected]>; Yigit, Ferruh <[email protected]>; > [email protected]; [email protected]; Richardson, Bruce > <[email protected]>; [email protected]; Ananyev, Konstantin > <[email protected]>; Tiwei Bie <[email protected]> > Subject: Re: [RFC v3 4/6] eal: add a helper for reading string from sysfs > > On Tue, 1 Jun 2021 11:06:42 +0800 > Chenbo Xia <[email protected]> wrote: > > > > > +int > > +rte_eal_parse_sysfs_str(const char *filename, char *buf, unsigned long sz) > > +{ > > + FILE *f; > > + > > + f = fopen(filename, "r"); > > + if (f == NULL) { > > + RTE_LOG(ERR, EAL, "%s(): cannot open sysfs file %s\n", > > + __func__, filename); > > Helpful to decode errno. > RTE_LOG(ERR, EAL, "%s(): cannot open sysfs file %s:%s\n", > __func__, filename, strerror(errno));
Yes. Will fix. Thanks, Chenbo
