> > Hi > I found that OVS DPDK fails to build with DPDK 16.07 after commit > 01961bb. I am not sure if this is because of some expectations about > DPDK configuration or problems in the code. I could fix this fairly > easily with 4 additions to lib/dpdk.c (patch below), but am unsure > about whether this is what was intended.
Hi, Apologies, I just submitted a very similar patch. I hadn't seen yours! I think the pdump include should be conditional, depending on whether DPDK_PDUMP is detected. It can also be taken out of netdev-dpdk.c as it's no longer required there. For my build I didn't require adding the ring and mempool includes to this file. Do you have additional DPDK config options set that require these headers to be included here? Thanks, Ciara > > Panda > > > > diff --git a/lib/dpdk.c b/lib/dpdk.c > index caea0f4..e998b65 100644 > --- a/lib/dpdk.c > +++ b/lib/dpdk.c > @@ -22,12 +22,16 @@ > #include <getopt.h> > > #include <rte_memzone.h> > +#include <rte_mempool.h> > +#include <rte_ring.h> > +#include <rte_pdump.h> > > #include "dirs.h" > #include "netdev-dpdk.h" > #include "openvswitch/dynamic-string.h" > #include "openvswitch/vlog.h" > #include "smap.h" > +#include "fatal-signal.h" > > VLOG_DEFINE_THIS_MODULE(dpdk); > _______________________________________________ > dev mailing list > dev@openvswitch.org > http://openvswitch.org/mailman/listinfo/dev _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev