netdev - RE: [EXT] [PATCH net-next 1/3] octeontx2-af: Remove redundant functions mac2u64() and cfg2mac()

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: 
 <DM6PR18MB260258A8526A64945E3ED4E4CD12A@DM6PR18MB2602.namprd18.prod.outlook.com>
Date: Wed, 9 Aug 2023 03:27:22 +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 1/3] octeontx2-af: Remove redundant
 functions mac2u64() and cfg2mac()



> -----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 1/3] octeontx2-af: Remove redundant
> functions mac2u64() and cfg2mac()
> 
> External Email
> 
> ----------------------------------------------------------------------
> The mac2u64() is used to convert an Ethernet MAC address into a u64 value,
> as this is exactly what ether_addr_to_u64() does. Similarly, the cfg2mac() is
> also the case. Use ether_addr_to_u64() and u64_to_ether_addr() instead of
> these two.
> 
> Signed-off-by: Li Zetao <lizetao1@...wei.com>
> ---
>  .../net/ethernet/marvell/octeontx2/af/cgx.c   | 26 +++----------------
>  1 file changed, 4 insertions(+), 22 deletions(-)
> 
> diff --git a/drivers/net/ethernet/marvell/octeontx2/af/cgx.c
> b/drivers/net/ethernet/marvell/octeontx2/af/cgx.c
> index 592037f4e55b..988383e20bb8 100644
> --- a/drivers/net/ethernet/marvell/octeontx2/af/cgx.c
> +++ b/drivers/net/ethernet/marvell/octeontx2/af/cgx.c
> @@ -223,24 +223,6 @@ int cgx_get_link_info(void *cgxd, int lmac_id,
>  	return 0;
>  }
> 
> -static u64 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;
> -}
> -
> -static void cfg2mac(u64 cfg, u8 *mac_addr) -{
> -	int i, index = 0;
> -
> -	for (i = ETH_ALEN - 1; i >= 0; i--, index++)
> -		mac_addr[i] = (cfg >> (8 * index)) & 0xFF;
> -}
> -
>  int cgx_lmac_addr_set(u8 cgx_id, u8 lmac_id, u8 *mac_addr)  {
>  	struct cgx *cgx_dev = cgx_get_pdata(cgx_id); @@ -255,7 +237,7 @@
> int cgx_lmac_addr_set(u8 cgx_id, u8 lmac_id, u8 *mac_addr)
>  	/* copy 6bytes from macaddr */
>  	/* memcpy(&cfg, mac_addr, 6); */
> 
> -	cfg = mac2u64 (mac_addr);
> +	cfg = ether_addr_to_u64(mac_addr);
> 
>  	id = get_sequence_id_of_lmac(cgx_dev, lmac_id);
> 
> @@ -322,7 +304,7 @@ int cgx_lmac_addr_add(u8 cgx_id, u8 lmac_id, u8
> *mac_addr)
> 
>  	index = id * lmac->mac_to_index_bmap.max + idx;
> 
> -	cfg = mac2u64 (mac_addr);
> +	cfg = ether_addr_to_u64(mac_addr);
>  	cfg |= CGX_DMAC_CAM_ADDR_ENABLE;
>  	cfg |= ((u64)lmac_id << 49);
>  	cgx_write(cgx_dev, 0, (CGXX_CMRX_RX_DMAC_CAM0 + (index *
> 0x8)), cfg); @@ -405,7 +387,7 @@ int cgx_lmac_addr_update(u8 cgx_id, u8
> lmac_id, u8 *mac_addr, u8 index)
> 
>  	cfg = cgx_read(cgx_dev, 0, (CGXX_CMRX_RX_DMAC_CAM0 + (index *
> 0x8)));
>  	cfg &= ~CGX_RX_DMAC_ADR_MASK;
> -	cfg |= mac2u64 (mac_addr);
> +	cfg |= ether_addr_to_u64(mac_addr);
> 
>  	cgx_write(cgx_dev, 0, (CGXX_CMRX_RX_DMAC_CAM0 + (index *
> 0x8)), cfg);
>  	return 0;
> @@ -441,7 +423,7 @@ int cgx_lmac_addr_del(u8 cgx_id, u8 lmac_id, u8
> index)
>  	/* Read MAC address to check whether it is ucast or mcast */
>  	cfg = cgx_read(cgx_dev, 0, (CGXX_CMRX_RX_DMAC_CAM0 + (index *
> 0x8)));
> 
> -	cfg2mac(cfg, mac);
> +	u64_to_ether_addr(cfg, mac);
>  	if (is_multicast_ether_addr(mac))
>  		lmac->mcast_filters_count--;
> 
> --
> 2.34.1
Ack. Thanks for the patch.