On Thu, Oct 24, 2019 at 8:59 PM David Marchand
<david.march...@redhat.com> wrote:
>
> On Thu, Oct 24, 2019 at 5:18 PM Anatoly Burakov
> <anatoly.bura...@intel.com> wrote:
> >
> > We are using '--base-virtaddr' in a few places. We have a define for that,
> > so use it instead.
> >
> > Signed-off-by: Anatoly Burakov <anatoly.bura...@intel.com>
> > diff --git a/lib/librte_eal/freebsd/eal/eal_memory.c
> > b/lib/librte_eal/freebsd/eal/eal_memory.c
> > index cd31827c2b..a637127d5e 100644
> > --- a/lib/librte_eal/freebsd/eal/eal_memory.c
> > +++ b/lib/librte_eal/freebsd/eal/eal_memory.c
> > @@ -14,6 +14,8 @@
> > #include <rte_errno.h>
> > #include <rte_log.h>
> > #include <rte_string_fns.h>
> > +
> > +#include "eal_options.h"
> > #include "eal_private.h"
> > #include "eal_internal_cfg.h"
> > #include "eal_filesystem.h"
Hit build issue on FreeBSD:
In file included from ../lib/librte_eal/freebsd/eal/eal_memory.c:18:0:
../lib/librte_eal/common/eal_options.h:79:15: error: 'struct
internal_config' declared inside parameter list will not be visible
outside of this definition or declaration [-Werror]
struct internal_config *conf);
^~~~~~~~~~~~~~~
Fixed order of header inclusion.
Applied, thanks.
--
David Marchand
