netdev - RE: [EXT] [PATCH net-next 3/3] octeontx2-af: Remove redundant functions rvu_npc_exact

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: 
 <DM6PR18MB2602CE6D00C9A6080BA14893CD12A@DM6PR18MB2602.namprd18.prod.outlook.com>
Date: Wed, 9 Aug 2023 03:31:07 +0000
From: Geethasowjanya Akula <gakula@...vell.com>
To: Li Zetao <lizetao1@...wei.com>,
        Sunil Kovvuri Goutham
	<sgoutham@...vell.com>,
        Linu Cherian <lcherian@...vell.com>,
        Jerin Jacob
 Kollanukkaran <jerinj@...vell.com>,
        Hariprasad Kelam <hkelam@...vell.com>,
        Subbaraya Sundeep Bhatta <sbhatta@...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>
CC: "netdev@...r.kernel.org" <netdev@...r.kernel.org>
Subject: RE: [EXT] [PATCH net-next 3/3] octeontx2-af: Remove redundant
 functions rvu_npc_exact_mac2u64()



> -----Original Message-----
> From: Li Zetao <lizetao1@...wei.com>
> Sent: Tuesday, August 8, 2023 5:15 PM
> To: Sunil Kovvuri Goutham <sgoutham@...vell.com>; Linu Cherian
> <lcherian@...vell.com>; Geethasowjanya Akula <gakula@...vell.com>;
> Jerin Jacob Kollanukkaran <jerinj@...vell.com>; Hariprasad Kelam
> <hkelam@...vell.com>; Subbaraya Sundeep Bhatta
> <sbhatta@...vell.com>; davem@...emloft.net; edumazet@...gle.com;
> kuba@...nel.org; pabeni@...hat.com
> Cc: lizetao1@...wei.com; netdev@...r.kernel.org
> Subject: [EXT] [PATCH net-next 3/3] octeontx2-af: Remove redundant
> functions rvu_npc_exact_mac2u64()
> 
> External Email
> 
> ----------------------------------------------------------------------
> The rvu_npc_exact_mac2u64() is used to convert an Ethernet MAC address
> into a u64 value, as this is exactly what ether_addr_to_u64() does.
> Use ether_addr_to_u64() to replace the rvu_npc_exact_mac2u64().
> 
> Signed-off-by: Li Zetao <lizetao1@...wei.com>
> ---
>  .../marvell/octeontx2/af/rvu_npc_hash.c       | 20 ++-----------------
>  1 file changed, 2 insertions(+), 18 deletions(-)
> 
> diff --git a/drivers/net/ethernet/marvell/octeontx2/af/rvu_npc_hash.c
> b/drivers/net/ethernet/marvell/octeontx2/af/rvu_npc_hash.c
> index 7e20282c12d0..d2661e7fabdb 100644
> --- a/drivers/net/ethernet/marvell/octeontx2/af/rvu_npc_hash.c
> +++ b/drivers/net/ethernet/marvell/octeontx2/af/rvu_npc_hash.c
> @@ -390,22 +390,6 @@ int
> rvu_mbox_handler_npc_get_field_hash_info(struct rvu *rvu,
>  	return 0;
>  }
> 
> -/**
> - *	rvu_npc_exact_mac2u64 - utility function to convert mac address to
> u64.
> - *	@mac_addr: MAC address.
> - *	Return: mdata for exact match table.
> - */
> -static u64 rvu_npc_exact_mac2u64(u8 *mac_addr) -{
> -	u64 mac = 0;
> -	int index;
> -
> -	for (index = ETH_ALEN - 1; index >= 0; index--)
> -		mac |= ((u64)*mac_addr++) << (8 * index);
> -
> -	return mac;
> -}
> -
>  /**
>   *	rvu_exact_prepare_mdata - Make mdata for mcam entry
>   *	@mac: MAC address
> @@ -416,7 +400,7 @@ static u64 rvu_npc_exact_mac2u64(u8 *mac_addr)
>   */
>  static u64 rvu_exact_prepare_mdata(u8 *mac, u16 chan, u16 ctype, u64
> mask)  {
> -	u64 ldata = rvu_npc_exact_mac2u64(mac);
> +	u64 ldata = ether_addr_to_u64(mac);
> 
>  	/* Please note that mask is 48bit which excludes chan and ctype.
>  	 * Increase mask bits if we need to include them as well.
> @@ -604,7 +588,7 @@ static u64 rvu_exact_prepare_table_entry(struct rvu
> *rvu, bool enable,
>  					 u8 ctype, u16 chan, u8 *mac_addr)
> 
>  {
> -	u64 ldata = rvu_npc_exact_mac2u64(mac_addr);
> +	u64 ldata = ether_addr_to_u64(mac_addr);
> 
>  	/* Enable or disable */
>  	u64 mdata = FIELD_PREP(GENMASK_ULL(63, 63), enable ? 1 : 0);
> --
> 2.34.1
Ack. Thanks for the patch.