<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jul 5, 2017 at 7:26 AM, Christian Ehrhardt <span dir="ltr"><<a href="mailto:christian.ehrhardt@canonical.com" target="_blank">christian.ehrhardt@canonical.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div class="gmail_extra"><br></div><div class="gmail_extra">So I'll spin  a few more tries around adapting OVS as needed.</div><div class="gmail_extra">Current revision: <span style="font-family:monospace"><span style="color:rgb(0,0,0)"><a href="http://paste.ubuntu.com/25023344/" target="_blank">http://paste.ubuntu.<wbr>com/25023344/</a></span></span></div></blockquote></div><br>That wasn't enough yet.</div><div class="gmail_extra"><div class="gmail_extra">The compile works just fine, analyzing the build logs:</div><div class="gmail_extra"><br></div><div class="gmail_extra">Old auto-detect vs pkg-config:<br></div><div class="gmail_extra">1. lost</div><div class="gmail_extra">   -I/usr/include</div><div class="gmail_extra">   -I/usr/local/include/dpdk</div><div class="gmail_extra">2. stays as-is</div><div class="gmail_extra">   -I/usr/include/dpdk</div><div class="gmail_extra">3. added:</div><div class="gmail_extra">   -I/usr/include/x86_64-linux-gnu/dpdk</div><div class="gmail_extra">   -include /usr/include/x86_64-linux-gnu/dpdk/rte_config.h</div><div class="gmail_extra"><br></div><div class="gmail_extra">That should be just what we want and works.</div><div class="gmail_extra"><br></div><div class="gmail_extra">But later it fails on linking:</div><div class="gmail_extra">Error is:</div><div class="gmail_extra">./libtool: line 7471: cd: -l: invalid option</div><div class="gmail_extra"><br></div><div class="gmail_extra">Likely another double option ...</div><div class="gmail_extra"><br></div><div class="gmail_extra">It has the same modification to the includes.</div><div class="gmail_extra">But then:</div><div class="gmail_extra">1 lost:</div><div class="gmail_extra">  -L/usr/lib</div><div class="gmail_extra">2. stays as-is</div><div class="gmail_extra">  It tails the options with -ldpdk -pcap in both cases</div><div class="gmail_extra">  Remember that pulls in the linker script which is fine to expand the group.</div><div class="gmail_extra">3. it gained the pkg-config "libs"</div><div class="gmail_extra">  -L-lrte_acl -lrte_cfgfile -lrte_cmdline -lrte_cryptodev -lrte_distributor -lrte_eal -lrte_ethdev -lrte_hash -lrte_ip_frag -lrte_jobstats -lrte_kni -lrte_kvargs -lrte_lpm -lrte_mbuf -lrte_mempool -lrte_meter -lrte_net -lrte_pdump -lrte_pipeline -lrte_pmd_af_packet -lrte_pmd_bnxt -lrte_pmd_bond -lrte_pmd_cxgbe -lrte_pmd_e1000 -lrte_pmd_ena -lrte_pmd_enic -lrte_pmd_fm10k -lrte_pmd_i40e -lrte_pmd_ixgbe -lrte_pmd_null -lrte_pmd_null_crypto -lrte_pmd_pcap -lrte_pmd_qede -lrte_pmd_ring -lrte_pmd_thunderx_nicvf -lrte_pmd_vhost -lrte_pmd_virtio -lrte_pmd_vmxnet3_uio -lrte_pmd_xenvirt -lrte_port -lrte_power -lrte_reorder -lrte_ring -lrte_sched -lrte_table -lrte_timer -lrte_vhost -ldl -lm -lpthread -lz </div><div class="gmail_extra"><br></div><div class="gmail_extra">Two things are wrong with that:</div><div class="gmail_extra">1. it again does combine args the wrong way "-L-lrte_acl" in LDFLAGS="$LDFLAGS -L${DPDK_LIB_DIR}"</div><div class="gmail_extra">2. it seems it actually wanted a -L<path> and not real -l statements.</div><div class="gmail_extra">    Also it only wants that if it is a non standard path, and we clearly are at a standard path.</div><div class="gmail_extra"><br></div><div class="gmail_extra">For the path we would actually need more like:<br></div><div class="gmail_extra">pkg-config --variable=libdir libdpdk</div><div class="gmail_extra">But I don't know how to fetch that in acinclude m4 code.</div><div class="gmail_extra"><br></div><div class="gmail_extra">Anyway we clearly use a default libdir path and unless we change we don't need to mess with ovs's config of it.</div><div class="gmail_extra">So dropping DPDK_LIBS from our change actually should do it.</div><div class="gmail_extra">I also adapted a few more things to follow the coding style the overall file has.</div><div class="gmail_extra"><br></div><div class="gmail_extra">Next revision currently trying to build in ppas: <a href="http://paste.ubuntu.com/25023546/">http://paste.ubuntu.com/25023546/</a></div><div class="gmail_extra"><br></div><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><span style="color:rgb(136,136,136);font-size:12.8px">Christian Ehrhardt</span><div style="color:rgb(136,136,136);font-size:12.8px">Software Engineer, Ubuntu Server</div><div style="color:rgb(136,136,136);font-size:12.8px">Canonical Ltd</div></div></div></div></div>
</div></div>