Subject: Re: [PATCH net] cxgb4: use port number to set mac addr

From: Anumula Murali Mohan Reddy <[email-masked]>
Date: Fri, 6 Dec 2024 01:00:12 -0500

O Wednesday, December 12/04/24, 2024 at 16:05:46 +0530, Michal Swiatkowski wrote:
> On Tue, Dec 03, 2024 at 08:11:49PM +0530, Anumula Murali Mohan Reddy wrote:
> > t4_set_vf_mac_acl() uses pf to set mac addr, but t4vf_get_vf_mac_acl()
> > uses port number to get mac addr, this leads to error when an attempt
> > to set MAC address on VF's of PF2 and PF3.
> > This patch fixes the issue by using port number to set mac address.
> > 
> > Signed-off-by: Anumula Murali Mohan Reddy <[email-masked]>
> > Signed-off-by: Potnuri Bharat Teja <[email-masked]>
> > ---
> >  drivers/net/ethernet/chelsio/cxgb4/cxgb4.h      | 2 +-
> >  drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c | 2 +-
> >  drivers/net/ethernet/chelsio/cxgb4/t4_hw.c      | 5 +++--
> >  3 files changed, 5 insertions(+), 4 deletions(-)
> > 
> > diff --git a/drivers/net/ethernet/chelsio/cxgb4/cxgb4.h b/drivers/net/ethernet/chelsio/cxgb4/cxgb4.h
> > index 75bd69ff61a8..c7c2c15a1815 100644
> > --- a/drivers/net/ethernet/chelsio/cxgb4/cxgb4.h
> > +++ b/drivers/net/ethernet/chelsio/cxgb4/cxgb4.h
> > @@ -2076,7 +2076,7 @@ void t4_idma_monitor(struct adapter *adapter,
> >  		     struct sge_idma_monitor_state *idma,
> >  		     int hz, int ticks);
> >  int t4_set_vf_mac_acl(struct adapter *adapter, unsigned int vf,
> > -		      unsigned int naddr, u8 *addr);
> > +		      u8 start, unsigned int naddr, u8 *addr);
> >  void t4_tp_pio_read(struct adapter *adap, u32 *buff, u32 nregs,
> >  		    u32 start_index, bool sleep_ok);
> >  void t4_tp_tm_pio_read(struct adapter *adap, u32 *buff, u32 nregs,
> > diff --git a/drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c b/drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c
> > index 97a261d5357e..bc3af0054406 100644
> > --- a/drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c
> > +++ b/drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c
> > @@ -3234,7 +3234,7 @@ static int cxgb4_mgmt_set_vf_mac(struct net_device *dev, int vf, u8 *mac)
> >  
> >  	dev_info(pi->adapter->pdev_dev,
> >  		 "Setting MAC %pM on VF %d\n", mac, vf);
> > -	ret = t4_set_vf_mac_acl(adap, vf + 1, 1, mac);
> > +	ret = t4_set_vf_mac_acl(adap, vf + 1, pi->lport, 1, mac);
> >  	if (!ret)
> >  		ether_addr_copy(adap->vfinfo[vf].vf_mac_addr, mac);
> >  	return ret;
> > diff --git a/drivers/net/ethernet/chelsio/cxgb4/t4_hw.c b/drivers/net/ethernet/chelsio/cxgb4/t4_hw.c
> > index 76de55306c4d..175bf9b13058 100644
> > --- a/drivers/net/ethernet/chelsio/cxgb4/t4_hw.c
> > +++ b/drivers/net/ethernet/chelsio/cxgb4/t4_hw.c
> > @@ -10215,11 +10215,12 @@ int t4_load_cfg(struct adapter *adap, const u8 *cfg_data, unsigned int size)
> >   *	t4_set_vf_mac_acl - Set MAC address for the specified VF
> >   *	@adapter: The adapter
> >   *	@vf: one of the VFs instantiated by the specified PF
> > + *	@start: The start port id associated with specified VF
> >   *	@naddr: the number of MAC addresses
> >   *	@addr: the MAC address(es) to be set to the specified VF
> >   */
> >  int t4_set_vf_mac_acl(struct adapter *adapter, unsigned int vf,
> > -		      unsigned int naddr, u8 *addr)
> > +		      u8 start, unsigned int naddr, u8 *addr)
> >  {
> >  	struct fw_acl_mac_cmd cmd;
> >  
> > @@ -10234,7 +10235,7 @@ int t4_set_vf_mac_acl(struct adapter *adapter, unsigned int vf,
> >  	cmd.en_to_len16 = cpu_to_be32((unsigned int)FW_LEN16(cmd));
> >  	cmd.nmac = naddr;
> >  
> > -	switch (adapter->pf) {
> > +	switch (start) {
> >  	case 3:
> >  		memcpy(cmd.macaddr3, addr, sizeof(cmd.macaddr3));
> You can use ether_addr_copy().
> 
> Beside that and fixes tag:
> Reviewed-by: Michal Swiatkowski <[email-masked]>
> 
> >  		break;
> > -- 
> > 2.39.3
there are multiple instances where memcpy needs to be replaced. I will
send a separate patch to replace memcpy with ethr_addr_copy().

Thanks