[dpdk-dev] [PATCH v3 0/3] test: fix timeout in flags autotest

David Marchand david.marchand at redhat.com
Fri Nov 8 12:11:22 CET 2019


On Fri, Nov 8, 2019 at 12:05 PM David Marchand
<david.marchand at redhat.com> wrote:
>
> On Fri, Nov 8, 2019 at 11:21 AM <kkanas at marvell.com> wrote:
> >
> > Hi David,
> >
> > Thanks for review, hopefully this patch will addresses most of the sutff.
> > Rest I will address here.
> >
> > >
> > > > +       const char *procdir = "/proc/self/fd/";
> > >
> > > self is a Linux thing.
> > > This won't work on FreeBSD.
> >
> > IMHO original code didn't worked on FreeBSD as well.
> > I have created function to adress in third patch
>
> Indeed...
>
> Well, wait.
> Why do we need to close those file descriptors?
> FreeBSD has been like this for quite some time.
>
> Can't we just remove this code?
> Maybe someone from Intel has an idea of why it was like this?

Great...

        /* get file for config (fd is always 3) */
        snprintf(path, sizeof(path), "/proc/self/fd/%d", 3);

And this only applies to linux (getting the current file prefix).

-- 
David Marchand



More information about the dev mailing list