[PATCH v10 01/21] eal: include header required for alloca

[Andre Muezerie]

On Wed, Nov 20, 2024 at 09:12:15AM -0800, Stephen Hemminger wrote:
> On Tue, 19 Nov 2024 19:13:01 -0800
> Andre Muezerie <andremue at linux.microsoft.com> wrote:
> 
> > From: Tyler Retzlaff <roretzla at linux.microsoft.com>
> > 
> > Include alloca.h for Linux and malloc.h for Windows to get declaration
> > of alloca().
> > 
> > Signed-off-by: Tyler Retzlaff <roretzla at linux.microsoft.com>
> > ---
> >  lib/eal/linux/include/rte_os.h   | 1 +
> >  lib/eal/windows/include/rte_os.h | 1 +
> >  2 files changed, 2 insertions(+)
> 
> What about FreeBSD?
> 	https://man.freebsd.org/cgi/man.cgi?alloca
> 
> Looks like to be pedantic stdlib.h should be included there.

Indeed. I'll include it.
Is FreeBSD tested when a patch is submitted? I did not see it in the list
of OSes/distros tested. I'm asking because it's always possible to break
something inadvertently if not tested.

Regards,
Andre