Re: [dpdk-dev] [PATCH 2/3] mbuf: add a non fatal sanity check helper

2018-10-09 Thread Olivier Matz
On Mon, Sep 10, 2018 at 11:33:44AM +0300, Andrew Rybchenko wrote: > On 09/10/2018 11:24 AM, David Marchand wrote: > > On Mon, Sep 10, 2018 at 10:12 AM, Andrew Rybchenko > > wrote: > > > +/** > > > + * Sanity checks on a mbuf. > > > + * > > > + * Almost like rte_mbuf_sanity_check(), but this functi

Re: [dpdk-dev] [PATCH 2/3] mbuf: add a non fatal sanity check helper

2018-09-10 Thread Andrew Rybchenko
On 09/10/2018 11:24 AM, David Marchand wrote: On Mon, Sep 10, 2018 at 10:12 AM, Andrew Rybchenko wrote: +/** + * Sanity checks on a mbuf. + * + * Almost like rte_mbuf_sanity_check(), but this function gives the reason + * if corruption is detected rather than panic. + * + * @param m + * The m

Re: [dpdk-dev] [PATCH 2/3] mbuf: add a non fatal sanity check helper

2018-09-10 Thread David Marchand
On Mon, Sep 10, 2018 at 10:12 AM, Andrew Rybchenko wrote: > +/** > + * Sanity checks on a mbuf. > + * > + * Almost like rte_mbuf_sanity_check(), but this function gives the reason > + * if corruption is detected rather than panic. > + * > + * @param m > + * The mbuf to be checked. > + * @param i

Re: [dpdk-dev] [PATCH 2/3] mbuf: add a non fatal sanity check helper

2018-09-10 Thread Andrew Rybchenko
On 09/10/2018 08:45 AM, David Marchand wrote: Let's add a little helper that does the same as rte_mbuf_sanity_check but without the panic. Signed-off-by: David Marchand --- <...> diff --git a/lib/librte_mbuf/rte_mbuf.h b/lib/librte_mbuf/rte_mbuf.h index a50b05c64..e12a4c765 100644 --- a/lib

Re: [dpdk-dev] [PATCH 2/3] mbuf: add a non fatal sanity check helper

2018-09-09 Thread David Marchand
On Mon, Sep 10, 2018 at 7:45 AM, David Marchand wrote: > Let's add a little helper that does the same as rte_mbuf_sanity_check but > without the panic. > > Signed-off-by: David Marchand > --- > lib/librte_mbuf/Makefile | 2 + > lib/librte_mbuf/meson.build | 2 + > lib/libr

[dpdk-dev] [PATCH 2/3] mbuf: add a non fatal sanity check helper

2018-09-09 Thread David Marchand
Let's add a little helper that does the same as rte_mbuf_sanity_check but without the panic. Signed-off-by: David Marchand --- lib/librte_mbuf/Makefile | 2 + lib/librte_mbuf/meson.build | 2 + lib/librte_mbuf/rte_mbuf.c | 74 lib/lib