netdev - RE: [EXT] Re: [PATCH net] octeontx2-pf: Fix holes in error code

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: 
 <MWHPR1801MB1918043F69F45C9D69656449D3DDA@MWHPR1801MB1918.namprd18.prod.outlook.com>
Date: Thu, 26 Oct 2023 10:37:48 +0000
From: Ratheesh Kannoth <rkannoth@...vell.com>
To: Wojciech Drewek <wojciech.drewek@...el.com>,
        "netdev@...r.kernel.org"
	<netdev@...r.kernel.org>,
        "linux-kernel@...r.kernel.org"
	<linux-kernel@...r.kernel.org>
CC: Sunil Kovvuri Goutham <sgoutham@...vell.com>,
        Geethasowjanya Akula
	<gakula@...vell.com>,
        Subbaraya Sundeep Bhatta <sbhatta@...vell.com>,
        Hariprasad Kelam <hkelam@...vell.com>,
        "davem@...emloft.net"
	<davem@...emloft.net>,
        "edumazet@...gle.com" <edumazet@...gle.com>,
        "kuba@...nel.org" <kuba@...nel.org>,
        "pabeni@...hat.com" <pabeni@...hat.com>
Subject: RE: [EXT] Re: [PATCH net] octeontx2-pf: Fix holes in error code

> From: Wojciech Drewek <wojciech.drewek@...el.com>
> Sent: Thursday, October 26, 2023 3:44 PM
> Subject: [EXT] Re: [PATCH net] octeontx2-pf: Fix holes in error code
> 
> >  static char *nix_snd_status_e_str[NIX_SND_STATUS_MAX] =  {
> > -	"NIX_SND_STATUS_GOOD",
> > -	"NIX_SND_STATUS_SQ_CTX_FAULT",
> > -	"NIX_SND_STATUS_SQ_CTX_POISON",
> > -	"NIX_SND_STATUS_SQB_FAULT",
> > -	"NIX_SND_STATUS_SQB_POISON",
> > -	"NIX_SND_STATUS_HDR_ERR",
> > -	"NIX_SND_STATUS_EXT_ERR",
> > -	"NIX_SND_STATUS_JUMP_FAULT",
> > -	"NIX_SND_STATUS_JUMP_POISON",
> > -	"NIX_SND_STATUS_CRC_ERR",
> > -	"NIX_SND_STATUS_IMM_ERR",
> > -	"NIX_SND_STATUS_SG_ERR",
> > -	"NIX_SND_STATUS_MEM_ERR",
> > -	"NIX_SND_STATUS_INVALID_SUBDC",
> > -	"NIX_SND_STATUS_SUBDC_ORDER_ERR",
> > -	"NIX_SND_STATUS_DATA_FAULT",
> > -	"NIX_SND_STATUS_DATA_POISON",
> > -	"NIX_SND_STATUS_NPC_DROP_ACTION",
> > -	"NIX_SND_STATUS_LOCK_VIOL",
> > -	"NIX_SND_STATUS_NPC_UCAST_CHAN_ERR",
> > -	"NIX_SND_STATUS_NPC_MCAST_CHAN_ERR",
> > -	"NIX_SND_STATUS_NPC_MCAST_ABORT",
> > -	"NIX_SND_STATUS_NPC_VTAG_PTR_ERR",
> > -	"NIX_SND_STATUS_NPC_VTAG_SIZE_ERR",
> > -	"NIX_SND_STATUS_SEND_STATS_ERR",
> > +	[NIX_SND_STATUS_GOOD] = "NIX_SND_STATUS_GOOD",
> > +	[NIX_SND_STATUS_SQ_CTX_FAULT] =
> "NIX_SND_STATUS_SQ_CTX_FAULT",
> > +	[NIX_SND_STATUS_SQ_CTX_POISON] =
> "NIX_SND_STATUS_SQ_CTX_POISON",
> > +	[NIX_SND_STATUS_SQB_FAULT] = "NIX_SND_STATUS_SQB_FAULT",
> > +	[NIX_SND_STATUS_SQB_POISON] =
> "NIX_SND_STATUS_SQB_POISON",
> > +	[NIX_SND_STATUS_HDR_ERR] = "NIX_SND_STATUS_HDR_ERR",
> > +	[NIX_SND_STATUS_EXT_ERR] = "NIX_SND_STATUS_EXT_ERR",
> > +	[NIX_SND_STATUS_JUMP_FAULT] =
> "NIX_SND_STATUS_JUMP_FAULT",
> > +	[NIX_SND_STATUS_JUMP_POISON] =
> "NIX_SND_STATUS_JUMP_POISON",
> > +	[NIX_SND_STATUS_CRC_ERR] = "NIX_SND_STATUS_CRC_ERR",
> > +	[NIX_SND_STATUS_IMM_ERR] = "NIX_SND_STATUS_IMM_ERR",
> > +	[NIX_SND_STATUS_SG_ERR] = "NIX_SND_STATUS_SG_ERR",
> > +	[NIX_SND_STATUS_MEM_ERR] = "NIX_SND_STATUS_MEM_ERR",
> > +	[NIX_SND_STATUS_INVALID_SUBDC] =
> "NIX_SND_STATUS_INVALID_SUBDC",
> > +	[NIX_SND_STATUS_SUBDC_ORDER_ERR] =
> "NIX_SND_STATUS_SUBDC_ORDER_ERR",
> > +	[NIX_SND_STATUS_DATA_FAULT] =
> "NIX_SND_STATUS_DATA_FAULT",
> > +	[NIX_SND_STATUS_DATA_POISON] =
> "NIX_SND_STATUS_DATA_POISON",
> > +	[NIX_SND_STATUS_NPC_DROP_ACTION] =
> "NIX_SND_STATUS_NPC_DROP_ACTION",
> > +	[NIX_SND_STATUS_LOCK_VIOL] = "NIX_SND_STATUS_LOCK_VIOL",
> > +	[NIX_SND_STATUS_NPC_UCAST_CHAN_ERR] =
> "NIX_SND_STATUS_NPC_UCAST_CHAN_ERR",
> > +	[NIX_SND_STATUS_NPC_MCAST_CHAN_ERR] =
> "NIX_SND_STATUS_NPC_MCAST_CHAN_ERR",
> > +	[NIX_SND_STATUS_NPC_MCAST_ABORT] =
> "NIX_SND_STATUS_NPC_MCAST_ABORT",
> > +	[NIX_SND_STATUS_NPC_VTAG_PTR_ERR] =
> "NIX_SND_STATUS_NPC_VTAG_PTR_ERR",
> > +	[NIX_SND_STATUS_NPC_VTAG_SIZE_ERR] =
> "NIX_SND_STATUS_NPC_VTAG_SIZE_ERR",
> > +	[NIX_SND_STATUS_SEND_MEM_FAULT] =
> "NIX_SND_STATUS_SEND_MEM_FAULT",
> > +	[NIX_SND_STATUS_SEND_STATS_ERR] =
> "NIX_SND_STATUS_SEND_STATS_ERR",
> >  };
> >
> > @@ -318,23 +318,23 @@ enum nix_snd_status_e {
> >  	NIX_SND_STATUS_EXT_ERR = 0x6,
> >  	NIX_SND_STATUS_JUMP_FAULT = 0x7,
> >  	NIX_SND_STATUS_JUMP_POISON = 0x8,
> > -	NIX_SND_STATUS_CRC_ERR = 0x9,
> > -	NIX_SND_STATUS_IMM_ERR = 0x10,
> > -	NIX_SND_STATUS_SG_ERR = 0x11,
> > -	NIX_SND_STATUS_MEM_ERR = 0x12,
> > -	NIX_SND_STATUS_INVALID_SUBDC = 0x13,
> > -	NIX_SND_STATUS_SUBDC_ORDER_ERR = 0x14,
> > -	NIX_SND_STATUS_DATA_FAULT = 0x15,
> > -	NIX_SND_STATUS_DATA_POISON = 0x16,
> > -	NIX_SND_STATUS_NPC_DROP_ACTION = 0x17,
> > -	NIX_SND_STATUS_LOCK_VIOL = 0x18,
> > -	NIX_SND_STATUS_NPC_UCAST_CHAN_ERR = 0x19,
> > -	NIX_SND_STATUS_NPC_MCAST_CHAN_ERR = 0x20,
> > -	NIX_SND_STATUS_NPC_MCAST_ABORT = 0x21,
> > -	NIX_SND_STATUS_NPC_VTAG_PTR_ERR = 0x22,
> > -	NIX_SND_STATUS_NPC_VTAG_SIZE_ERR = 0x23,
> > -	NIX_SND_STATUS_SEND_MEM_FAULT = 0x24,
> > -	NIX_SND_STATUS_SEND_STATS_ERR = 0x25,
> > +	NIX_SND_STATUS_CRC_ERR = 0x10,> +
> 	NIX_SND_STATUS_IMM_ERR = 0x11,
> > +	NIX_SND_STATUS_SG_ERR = 0x12,
> > +	NIX_SND_STATUS_MEM_ERR = 0x13,
> > +	NIX_SND_STATUS_INVALID_SUBDC = 0x14,
> > +	NIX_SND_STATUS_SUBDC_ORDER_ERR = 0x15,
> > +	NIX_SND_STATUS_DATA_FAULT = 0x16,
> > +	NIX_SND_STATUS_DATA_POISON = 0x17,
> 
> There is a gap here, is it intended?
[Ratheesh Kannoth] Yes.

> And in general, why error codes were changed?
[Ratheesh Kannoth] it was a bug. 
 
> Starting from NIX_SND_STATUS_CRC_ERR which was 0x09 and now it's 0x10.
[Ratheesh Kannoth] Yes. It should be 0x10 .  The real issue is - >enum was not in sequence.  (Eg ,,after 0x9 it should be 0xa).  But 
"static char *nix_snd_status_e_str" was assigned sequential strings.  

> 
> > +	NIX_SND_STATUS_NPC_DROP_ACTION = 0x20,
> > +	NIX_SND_STATUS_LOCK_VIOL = 0x21,
> > +	NIX_SND_STATUS_NPC_UCAST_CHAN_ERR = 0x22,
> > +	NIX_SND_STATUS_NPC_MCAST_CHAN_ERR = 0x23,
> > +	NIX_SND_STATUS_NPC_MCAST_ABORT = 0x24,
> > +	NIX_SND_STATUS_NPC_VTAG_PTR_ERR = 0x25,
> > +	NIX_SND_STATUS_NPC_VTAG_SIZE_ERR = 0x26,
> > +	NIX_SND_STATUS_SEND_MEM_FAULT = 0x27,
> > +	NIX_SND_STATUS_SEND_STATS_ERR = 0x28,
> >  	NIX_SND_STATUS_MAX,
> >  };
> >