diff mbox series

[3/4] app/testpmd: fix build with gcc 11

Message ID 20200921133830.1575872-3-ferruh.yigit@intel.com (mailing list archive)
State Accepted, archived
Delegated to: David Marchand
Headers
Series [1/4] net/qede: fix build with gcc 11 |

Checks

Context Check Description
ci/checkpatch success coding style OK

Commit Message

Ferruh Yigit Sept. 21, 2020, 1:38 p.m. UTC 
  Error observed with gcc 11 under development
gcc (GCC) 11.0.0 20200920 (experimental)

../app/test-pmd/config.c:1777:61:
	warning: argument 3 of type ‘const uint32_t[n]’
	{aka ‘const unsigned int[n]’} declared as a variable length
	array [-Wvla-parameter]
 1777 | port_flow_list(portid_t port_id, uint32_t n,
 			const uint32_t group[n])
      |                 ~~~~~~~~~~~~~~~^~~~~~~~
In file included from ../app/test-pmd/config.c:53:
../app/test-pmd/testpmd.h:764:67:
	note: previously declared as a pointer ‘const uint32_t *’
	{aka ‘const unsigned int *’}
  764 | void port_flow_list(portid_t port_id, uint32_t n,
  				const uint32_t *group);
      |                         ~~~~~~~~~~~~~~~~^~~~~

Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
---
 app/test-pmd/config.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

David Marchand Sept. 24, 2020, 9:31 a.m. UTC | #1 
On Mon, Sep 21, 2020 at 3:39 PM Ferruh Yigit <ferruh.yigit@intel.com> wrote:
>
> Error observed with gcc 11 under development
> gcc (GCC) 11.0.0 20200920 (experimental)
>
> ../app/test-pmd/config.c:1777:61:
>         warning: argument 3 of type ‘const uint32_t[n]’
>         {aka ‘const unsigned int[n]’} declared as a variable length
>         array [-Wvla-parameter]
>  1777 | port_flow_list(portid_t port_id, uint32_t n,
>                         const uint32_t group[n])
>       |                 ~~~~~~~~~~~~~~~^~~~~~~~
> In file included from ../app/test-pmd/config.c:53:
> ../app/test-pmd/testpmd.h:764:67:
>         note: previously declared as a pointer ‘const uint32_t *’
>         {aka ‘const unsigned int *’}
>   764 | void port_flow_list(portid_t port_id, uint32_t n,
>                                 const uint32_t *group);
>       |                         ~~~~~~~~~~~~~~~~^~~~~
>

It won't hurt to backport this.

Fixes: 938a184a1870 ("app/testpmd: implement basic support for flow API")
Cc: stable@dpdk.org

> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
> ---
>  app/test-pmd/config.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/app/test-pmd/config.c b/app/test-pmd/config.c
> index b6eb2a57a..380f3b5f6 100644
> --- a/app/test-pmd/config.c
> +++ b/app/test-pmd/config.c
> @@ -1774,7 +1774,7 @@ port_flow_aged(portid_t port_id, uint8_t destroy)
>
>  /** List flow rules. */
>  void
> -port_flow_list(portid_t port_id, uint32_t n, const uint32_t group[n])
> +port_flow_list(portid_t port_id, uint32_t n, const uint32_t *group)
>  {
>         struct rte_port *port;
>         struct port_flow *pf;
> --
> 2.26.2
>

Reviewed-by: David Marchand <david.marchand@redhat.com>
Phil Yang Sept. 24, 2020, 11:01 a.m. UTC | #2 
<snip>

> Subject: [dpdk-dev] [PATCH 3/4] app/testpmd: fix build with gcc 11
> 
> Error observed with gcc 11 under development
> gcc (GCC) 11.0.0 20200920 (experimental)
> 
> ../app/test-pmd/config.c:1777:61:
> 	warning: argument 3 of type ‘const uint32_t[n]’
> 	{aka ‘const unsigned int[n]’} declared as a variable length
> 	array [-Wvla-parameter]
>  1777 | port_flow_list(portid_t port_id, uint32_t n,
>  			const uint32_t group[n])
>       |                 ~~~~~~~~~~~~~~~^~~~~~~~
> In file included from ../app/test-pmd/config.c:53:
> ../app/test-pmd/testpmd.h:764:67:
> 	note: previously declared as a pointer ‘const uint32_t *’
> 	{aka ‘const unsigned int *’}
>   764 | void port_flow_list(portid_t port_id, uint32_t n,
>   				const uint32_t *group);
>       |                         ~~~~~~~~~~~~~~~~^~~~~
> 
> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>


Verified with gcc version 11.0.0 20200920 (experimental).
Reviewed-by: Phil Yang <phil.yang@arm.com>


> ---
>  app/test-pmd/config.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/app/test-pmd/config.c b/app/test-pmd/config.c
> index b6eb2a57a..380f3b5f6 100644
> --- a/app/test-pmd/config.c
> +++ b/app/test-pmd/config.c
> @@ -1774,7 +1774,7 @@ port_flow_aged(portid_t port_id, uint8_t destroy)
> 
>  /** List flow rules. */
>  void
> -port_flow_list(portid_t port_id, uint32_t n, const uint32_t group[n])
> +port_flow_list(portid_t port_id, uint32_t n, const uint32_t *group)
>  {
>  	struct rte_port *port;
>  	struct port_flow *pf;
> --
> 2.26.2
Asaf Penso Oct. 4, 2020, 8:57 a.m. UTC | #3 
>-----Original Message-----
>From: dev <dev-bounces@dpdk.org> On Behalf Of David Marchand
>Sent: Thursday, September 24, 2020 12:31 PM
>To: Ferruh Yigit <ferruh.yigit@intel.com>
>Cc: Wenzhuo Lu <wenzhuo.lu@intel.com>; Beilei Xing
><beilei.xing@intel.com>; Bernard Iremonger
><bernard.iremonger@intel.com>; dev <dev@dpdk.org>; NBU-Contact-Adrien
>Mazarguil <adrien.mazarguil@6wind.com>
>Subject: Re: [dpdk-dev] [PATCH 3/4] app/testpmd: fix build with gcc 11
>
>On Mon, Sep 21, 2020 at 3:39 PM Ferruh Yigit <ferruh.yigit@intel.com> wrote:
>>
>> Error observed with gcc 11 under development gcc (GCC) 11.0.0 20200920
>> (experimental)
>>
>> ../app/test-pmd/config.c:1777:61:
>>         warning: argument 3 of type ‘const uint32_t[n]’
>>         {aka ‘const unsigned int[n]’} declared as a variable length
>>         array [-Wvla-parameter]
>>  1777 | port_flow_list(portid_t port_id, uint32_t n,
>>                         const uint32_t group[n])
>>       |                 ~~~~~~~~~~~~~~~^~~~~~~~
>> In file included from ../app/test-pmd/config.c:53:
>> ../app/test-pmd/testpmd.h:764:67:
>>         note: previously declared as a pointer ‘const uint32_t *’
>>         {aka ‘const unsigned int *’}
>>   764 | void port_flow_list(portid_t port_id, uint32_t n,
>>                                 const uint32_t *group);
>>       |                         ~~~~~~~~~~~~~~~~^~~~~
>>
>
>It won't hurt to backport this.
>
>Fixes: 938a184a1870 ("app/testpmd: implement basic support for flow API")
>Cc: stable@dpdk.org
>
>> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
>> ---
>>  app/test-pmd/config.c | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/app/test-pmd/config.c b/app/test-pmd/config.c index
>> b6eb2a57a..380f3b5f6 100644
>> --- a/app/test-pmd/config.c
>> +++ b/app/test-pmd/config.c
>> @@ -1774,7 +1774,7 @@ port_flow_aged(portid_t port_id, uint8_t
>> destroy)
>>
>>  /** List flow rules. */
>>  void
>> -port_flow_list(portid_t port_id, uint32_t n, const uint32_t group[n])
>> +port_flow_list(portid_t port_id, uint32_t n, const uint32_t *group)
>>  {
>>         struct rte_port *port;
>>         struct port_flow *pf;
>> --
>> 2.26.2
>>
>
>Reviewed-by: David Marchand <david.marchand@redhat.com>
>
>--
>David Marchand


Reviewed-by: Asaf Penso <asafp@nvidia.com>
David Marchand Oct. 8, 2020, 10:48 a.m. UTC | #4 
On Mon, Sep 21, 2020 at 3:39 PM Ferruh Yigit <ferruh.yigit@intel.com> wrote:
>
> Error observed with gcc 11 under development
> gcc (GCC) 11.0.0 20200920 (experimental)
>
> ../app/test-pmd/config.c:1777:61:
>         warning: argument 3 of type ‘const uint32_t[n]’
>         {aka ‘const unsigned int[n]’} declared as a variable length
>         array [-Wvla-parameter]
>  1777 | port_flow_list(portid_t port_id, uint32_t n,
>                         const uint32_t group[n])
>       |                 ~~~~~~~~~~~~~~~^~~~~~~~
> In file included from ../app/test-pmd/config.c:53:
> ../app/test-pmd/testpmd.h:764:67:
>         note: previously declared as a pointer ‘const uint32_t *’
>         {aka ‘const unsigned int *’}
>   764 | void port_flow_list(portid_t port_id, uint32_t n,
>                                 const uint32_t *group);
>       |                         ~~~~~~~~~~~~~~~~^~~~~
>

Fixes: 938a184a1870 ("app/testpmd: implement basic support for flow API")
Cc: stable@dpdk.org

> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
Reviewed-by: David Marchand <david.marchand@redhat.com>
Reviewed-by: Phil Yang <phil.yang@arm.com>
Reviewed-by: Asaf Penso <asafp@nvidia.com>

Applied, thanks.
diff mbox series

Patch

diff --git a/app/test-pmd/config.c b/app/test-pmd/config.c
index b6eb2a57a..380f3b5f6 100644
--- a/app/test-pmd/config.c
+++ b/app/test-pmd/config.c
@@ -1774,7 +1774,7 @@  port_flow_aged(portid_t port_id, uint8_t destroy)
 
 /** List flow rules. */
 void
-port_flow_list(portid_t port_id, uint32_t n, const uint32_t group[n])
+port_flow_list(portid_t port_id, uint32_t n, const uint32_t *group)
 {
 	struct rte_port *port;
 	struct port_flow *pf;