Re: [PATCH 10/12] vhost/crypto: fix build with GCC 12

2022-06-16 Thread David Marchand
On Tue, Jun 14, 2022 at 11:25 AM Bruce Richardson wrote: > > > > - if (unlikely(!src || !dlen)) > > > > + if (unlikely(!src || !dlen || dlen > left)) > > > > return -1; > > > > > > > > > > If this change is omitted, does the compiler still give warnings. Looking > > > through

Re: [PATCH 10/12] vhost/crypto: fix build with GCC 12

2022-06-14 Thread Bruce Richardson
On Tue, Jun 14, 2022 at 11:22:24AM +0200, David Marchand wrote: > On Thu, Jun 2, 2022 at 12:09 PM Bruce Richardson > wrote: > > > > On Wed, May 18, 2022 at 12:16:55PM +0200, David Marchand wrote: > > > GCC 12 raises the following warning: > > > > > > In file included from ../lib/mempool/rte_mempoo

Re: [PATCH 10/12] vhost/crypto: fix build with GCC 12

2022-06-14 Thread David Marchand
On Thu, Jun 2, 2022 at 12:09 PM Bruce Richardson wrote: > > On Wed, May 18, 2022 at 12:16:55PM +0200, David Marchand wrote: > > GCC 12 raises the following warning: > > > > In file included from ../lib/mempool/rte_mempool.h:46, > > from ../lib/mbuf/rte_mbuf.h:38, > >

Re: [PATCH 10/12] vhost/crypto: fix build with GCC 12

2022-06-11 Thread Stephen Hemminger
On Wed, 18 May 2022 12:16:55 +0200 David Marchand wrote: > GCC 12 raises the following warning: > > In file included from ../lib/mempool/rte_mempool.h:46, > from ../lib/mbuf/rte_mbuf.h:38, > from ../lib/vhost/vhost_crypto.c:7: > ../lib/vhost/vhost_crypto.c: In f

Re: [PATCH 10/12] vhost/crypto: fix build with GCC 12

2022-06-02 Thread Bruce Richardson
On Wed, May 18, 2022 at 12:16:55PM +0200, David Marchand wrote: > GCC 12 raises the following warning: > > In file included from ../lib/mempool/rte_mempool.h:46, > from ../lib/mbuf/rte_mbuf.h:38, > from ../lib/vhost/vhost_crypto.c:7: > ../lib/vhost/vhost_crypto.c:

[PATCH 10/12] vhost/crypto: fix build with GCC 12

2022-05-18 Thread David Marchand
GCC 12 raises the following warning: In file included from ../lib/mempool/rte_mempool.h:46, from ../lib/mbuf/rte_mbuf.h:38, from ../lib/vhost/vhost_crypto.c:7: ../lib/vhost/vhost_crypto.c: In function ‘rte_vhost_crypto_fetch_requests’: ../lib/eal/x86/include/rte_m