DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] examples/l2fwd: fix build warning with system wide install
@ 2019-11-12 12:37 David Marchand
  2019-11-12 17:09 ` Ferruh Yigit
  0 siblings, 1 reply; 3+ messages in thread
From: David Marchand @ 2019-11-12 12:37 UTC (permalink / raw)
  To: dev
  Cc: Marko Kovacevic, Ori Kam, Bruce Richardson, Radu Nicolau,
	Akhil Goyal, Tomasz Kantecki, Sunil Kumar Kori, Pavan Nikhilesh

Caught when compiling this example with pkg-config:

 ## Building l2fwd
 ...
 main.c: In function ‘main’:
 main.c:716:3: warning: ‘rte_eth_dev_set_ptypes’ is deprecated: Symbol
 is not yet part of stable ABI [-Wdeprecated-declarations]
   716 |   ret = rte_eth_dev_set_ptypes(portid, RTE_PTYPE_UNKNOWN, NULL,
       |   ^~~
 In file included from main.c:38:
 ...build-x86-default/install-root/usr/local/include/rte_ethdev.h:2661:5:
 note: declared here
  2661 | int rte_eth_dev_set_ptypes(uint16_t port_id, uint32_t
 ptype_mask,
       |     ^~~~~~~~~~~~~~~~~~~~~~
 ln -sf l2fwd-shared build/l2fwd

Fixes: 9731df2e7554 ("examples/l2fwd: disable packet type parsing")

Signed-off-by: David Marchand <david.marchand@redhat.com>
---
 examples/l2fwd/Makefile | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/examples/l2fwd/Makefile b/examples/l2fwd/Makefile
index 59b2b4a..9c50684 100644
--- a/examples/l2fwd/Makefile
+++ b/examples/l2fwd/Makefile
@@ -21,6 +21,8 @@ PKGCONF=pkg-config --define-prefix
 
 PC_FILE := $(shell $(PKGCONF) --path libdpdk)
 CFLAGS += -O3 $(shell $(PKGCONF) --cflags libdpdk)
+# Add flag to allow experimental API as l2fwd uses rte_ethdev_set_ptype API
+CFLAGS += -DALLOW_EXPERIMENTAL_API
 LDFLAGS_SHARED = $(shell $(PKGCONF) --libs libdpdk)
 LDFLAGS_STATIC = -Wl,-Bstatic $(shell $(PKGCONF) --static --libs libdpdk)
 
-- 
1.8.3.1


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [dpdk-dev] [PATCH] examples/l2fwd: fix build warning with system wide install
  2019-11-12 12:37 [dpdk-dev] [PATCH] examples/l2fwd: fix build warning with system wide install David Marchand
@ 2019-11-12 17:09 ` Ferruh Yigit
  2019-11-12 19:13   ` David Marchand
  0 siblings, 1 reply; 3+ messages in thread
From: Ferruh Yigit @ 2019-11-12 17:09 UTC (permalink / raw)
  To: David Marchand, dev
  Cc: Marko Kovacevic, Ori Kam, Bruce Richardson, Radu Nicolau,
	Akhil Goyal, Tomasz Kantecki, Sunil Kumar Kori, Pavan Nikhilesh

On 11/12/2019 12:37 PM, David Marchand wrote:
> Caught when compiling this example with pkg-config:
> 
>  ## Building l2fwd
>  ...
>  main.c: In function ‘main’:
>  main.c:716:3: warning: ‘rte_eth_dev_set_ptypes’ is deprecated: Symbol
>  is not yet part of stable ABI [-Wdeprecated-declarations]
>    716 |   ret = rte_eth_dev_set_ptypes(portid, RTE_PTYPE_UNKNOWN, NULL,
>        |   ^~~
>  In file included from main.c:38:
>  ...build-x86-default/install-root/usr/local/include/rte_ethdev.h:2661:5:
>  note: declared here
>   2661 | int rte_eth_dev_set_ptypes(uint16_t port_id, uint32_t
>  ptype_mask,
>        |     ^~~~~~~~~~~~~~~~~~~~~~
>  ln -sf l2fwd-shared build/l2fwd
> 
> Fixes: 9731df2e7554 ("examples/l2fwd: disable packet type parsing")
> 
> Signed-off-by: David Marchand <david.marchand@redhat.com>

Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [dpdk-dev] [PATCH] examples/l2fwd: fix build warning with system wide install
  2019-11-12 17:09 ` Ferruh Yigit
@ 2019-11-12 19:13   ` David Marchand
  0 siblings, 0 replies; 3+ messages in thread
From: David Marchand @ 2019-11-12 19:13 UTC (permalink / raw)
  To: Ferruh Yigit
  Cc: dev, Marko Kovacevic, Ori Kam, Bruce Richardson, Radu Nicolau,
	Akhil Goyal, Tomasz Kantecki, Sunil Kumar Kori, Pavan Nikhilesh

On Tue, Nov 12, 2019 at 6:09 PM Ferruh Yigit <ferruh.yigit@intel.com> wrote:
>
> On 11/12/2019 12:37 PM, David Marchand wrote:
> > Caught when compiling this example with pkg-config:
> >
> >  ## Building l2fwd
> >  ...
> >  main.c: In function ‘main’:
> >  main.c:716:3: warning: ‘rte_eth_dev_set_ptypes’ is deprecated: Symbol
> >  is not yet part of stable ABI [-Wdeprecated-declarations]
> >    716 |   ret = rte_eth_dev_set_ptypes(portid, RTE_PTYPE_UNKNOWN, NULL,
> >        |   ^~~
> >  In file included from main.c:38:
> >  ...build-x86-default/install-root/usr/local/include/rte_ethdev.h:2661:5:
> >  note: declared here
> >   2661 | int rte_eth_dev_set_ptypes(uint16_t port_id, uint32_t
> >  ptype_mask,
> >        |     ^~~~~~~~~~~~~~~~~~~~~~
> >  ln -sf l2fwd-shared build/l2fwd
> >
> > Fixes: 9731df2e7554 ("examples/l2fwd: disable packet type parsing")
> >
> > Signed-off-by: David Marchand <david.marchand@redhat.com>
>
> Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>

Thanks.
Applied.



--
David Marchand


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2019-11-12 19:14 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-11-12 12:37 [dpdk-dev] [PATCH] examples/l2fwd: fix build warning with system wide install David Marchand
2019-11-12 17:09 ` Ferruh Yigit
2019-11-12 19:13   ` David Marchand

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).