On Thu, Nov 14, 2019 at 8:32 PM Kevin Traynor <ktray...@redhat.com> wrote:
>
> On 12/11/2019 20:31, David Marchand wrote:
> > From: Krzysztof Kanas <kka...@marvell.com>
> >
> > Caught while investigating timeouts on a ARM64 server.
> >
> > Stracing a test process running the eal_flags_autotest, we can see that
> > the fork helper is checking all possible file descriptors from
> > getdtablesize() to 2, and close the existing ones.
> > We can do better by inspecting this forked process /proc/self/fd
> > directory.
> >
> > Besides, checking file descriptors via /proc/self/fd only makes sense for
> > Linux. This code was a noop on FreeBSD.
> >
> > Fixes: af75078fece3 ("first public release")
> > Cc: sta...@dpdk.org
> >
> > Signed-off-by: Krzysztof Kanas <kka...@marvell.com>
> > Signed-off-by: David Marchand <david.march...@redhat.com>Tested-by: Krzysztof Kanas <kka...@marvell.com> > Seems to be an improvement on what is existing. > > Acked-by: Kevin Traynor <ktray...@redhat.com> > Applied, thanks. -- David Marchand
