[dpdk-dev] [PATCH] common/cnxk: fix build failure

[dpdk-dev] [PATCH] common/cnxk: fix build failure

[Tejasree Kondoj]

Fixing build failure with EXTRA_CFLAGS='-O1'.

Fixes: d85f9749f915 ("common/cnxk: add hash generation API")

Reported-by: Longfeng Liang <longfengx.liang@intel.com>
Signed-off-by: Tejasree Kondoj <ktejasree@marvell.com>
---
 drivers/common/cnxk/roc_hash.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/common/cnxk/roc_hash.c b/drivers/common/cnxk/roc_hash.c
index 092286e41e..4a34c7fbf8 100644
--- a/drivers/common/cnxk/roc_hash.c
+++ b/drivers/common/cnxk/roc_hash.c
@@ -23,7 +23,7 @@ roc_hash_sha1_gen(uint8_t *msg, uint32_t *hash)
 			       0x67452301, 0xEFCDAB89, 0x98BADCFE, 0x10325476,
 			       0xC3D2E1F0};
 	int i;
-	uint32_t temp;		/* Temporary word value */
+	uint32_t temp = 0;	/* Temporary word value */
 	uint32_t W[80];		/* Word sequence */
 	uint32_t A, B, C, D, E; /* Word buffers */
 
-- 
2.27.0

Re: [dpdk-dev] [PATCH] common/cnxk: fix build failure

[Anoob Joseph]

> Subject: [PATCH] common/cnxk: fix build failure
> 
> Fixing build failure with EXTRA_CFLAGS='-O1'.
> 
> Fixes: d85f9749f915 ("common/cnxk: add hash generation API")
> 
> Reported-by: Longfeng Liang <longfengx.liang@intel.com>
> Signed-off-by: Tejasree Kondoj <ktejasree@marvell.com>

Acked-by: Anoob Joseph <anoobj@marvell.com>

Re: [dpdk-dev] [PATCH] common/cnxk: fix build failure

[Jerin Jacob]

On Thu, Oct 28, 2021 at 12:31 PM Anoob Joseph <anoobj@marvell.com> wrote:
>
> > Subject: [PATCH] common/cnxk: fix build failure
> >
> > Fixing build failure with EXTRA_CFLAGS='-O1'.
> >
> > Fixes: d85f9749f915 ("common/cnxk: add hash generation API")
> >
> > Reported-by: Longfeng Liang <longfengx.liang@intel.com>
> > Signed-off-by: Tejasree Kondoj <ktejasree@marvell.com>
>
> Acked-by: Anoob Joseph <anoobj@marvell.com>


Changed subject to: common/cnxk: fix build with -O1

Applied to dpdk-next-net-mrvl/for-next-net. Thanks

Re: [dpdk-dev] [PATCH] common/cnxk: fix build failure

[Ferruh Yigit]

On 10/27/2021 2:12 PM, Tejasree Kondoj wrote:
> Fixing build failure with EXTRA_CFLAGS='-O1'.
> 

For future, can you please provide a little more detail in the commit
log related to the build error,
like which compiler or compiler version cause the issue, what is the
build error and put part of the log in commit log that can help to
people searching for the error log.
Plus if the build error is genuine build error, or if you are just
silencing compiler false positive.

> Fixes: d85f9749f915 ("common/cnxk: add hash generation API")
> 
> Reported-by: Longfeng Liang <longfengx.liang@intel.com>
> Signed-off-by: Tejasree Kondoj <ktejasree@marvell.com>
> ---
>   drivers/common/cnxk/roc_hash.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/common/cnxk/roc_hash.c b/drivers/common/cnxk/roc_hash.c
> index 092286e41e..4a34c7fbf8 100644
> --- a/drivers/common/cnxk/roc_hash.c
> +++ b/drivers/common/cnxk/roc_hash.c
> @@ -23,7 +23,7 @@ roc_hash_sha1_gen(uint8_t *msg, uint32_t *hash)
>   			       0x67452301, 0xEFCDAB89, 0x98BADCFE, 0x10325476,
>   			       0xC3D2E1F0};
>   	int i;
> -	uint32_t temp;		/* Temporary word value */
> +	uint32_t temp = 0;	/* Temporary word value */
>   	uint32_t W[80];		/* Word sequence */
>   	uint32_t A, B, C, D, E; /* Word buffers */
>   
> 

Re: [dpdk-dev] [EXT] Re: [PATCH] common/cnxk: fix build failure

[Tejasree Kondoj]

Sure Ferruh.

Thanks
Tejasree

> -----Original Message-----
> From: Ferruh Yigit <ferruh.yigit@intel.com>
> Sent: Thursday, October 28, 2021 11:02 PM
> To: Tejasree Kondoj <ktejasree@marvell.com>; Akhil Goyal
> <gakhil@marvell.com>; Jiang@mx0b-0016f401.pphosted.com; YuX
> <yux.jiang@intel.com>; Longfeng Liang <longfengx.liang@intel.com>
> Cc: Anoob Joseph <anoobj@marvell.com>; dev@dpdk.org
> Subject: [EXT] Re: [dpdk-dev] [PATCH] common/cnxk: fix build failure
> 
> External Email
> 
> ----------------------------------------------------------------------
> On 10/27/2021 2:12 PM, Tejasree Kondoj wrote:
> > Fixing build failure with EXTRA_CFLAGS='-O1'.
> >
> 
> For future, can you please provide a little more detail in the commit log
> related to the build error, like which compiler or compiler version cause the
> issue, what is the build error and put part of the log in commit log that can
> help to people searching for the error log.
> Plus if the build error is genuine build error, or if you are just silencing
> compiler false positive.
> 
> > Fixes: d85f9749f915 ("common/cnxk: add hash generation API")
> >
> > Reported-by: Longfeng Liang <longfengx.liang@intel.com>
> > Signed-off-by: Tejasree Kondoj <ktejasree@marvell.com>
> > ---
> >   drivers/common/cnxk/roc_hash.c | 2 +-
> >   1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/common/cnxk/roc_hash.c
> > b/drivers/common/cnxk/roc_hash.c index 092286e41e..4a34c7fbf8 100644
> > --- a/drivers/common/cnxk/roc_hash.c
> > +++ b/drivers/common/cnxk/roc_hash.c
> > @@ -23,7 +23,7 @@ roc_hash_sha1_gen(uint8_t *msg, uint32_t *hash)
> >   			       0x67452301, 0xEFCDAB89, 0x98BADCFE,
> 0x10325476,
> >   			       0xC3D2E1F0};
> >   	int i;
> > -	uint32_t temp;		/* Temporary word value */
> > +	uint32_t temp = 0;	/* Temporary word value */
> >   	uint32_t W[80];		/* Word sequence */
> >   	uint32_t A, B, C, D, E; /* Word buffers */
> >
> >