Search Linux Wireless

Re: [PATCH v2 1/7] mt76x02: use mask for vifs

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Jan 28, 2019 at 02:35:39PM +0100, Lorenzo Bianconi wrote:
> >  
> 
> Maybe we shoud grub dev->mt76.mutex here, since we can have multiple processes trying
> to add an interface

Interface creation and deletion are protected by rtnl_lock(), can not
be performed concurrently.

> > @@ -318,8 +326,10 @@ void mt76x02_remove_interface(struct ieee80211_hw *hw,
> >  			      struct ieee80211_vif *vif)
> >  {
> >  	struct mt76x02_dev *dev = hw->priv;
> > +	struct mt76x02_vif *mvif = (struct mt76x02_vif *)vif->drv_priv;
> >  
> >  	mt76_txq_remove(&dev->mt76, vif->txq);
> > +	dev->vif_mask &= ~BIT(mvif->idx);
> 
> If we are removing the first configured interface, should we have to update
> MT_MAC_ADDR_DW{0,1} regs?

What for ?

Thanks
Stanislaw 



[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Wireless Personal Area Network]     [Linux Bluetooth]     [Wireless Regulations]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite Hiking]     [MIPS Linux]     [ARM Linux]     [Linux RAID]

  Powered by Linux