[ofa-general][PATCH] mlx4_core: Multi Protocol support
    Roland Dreier 
    rdreier at cisco.com
       
    Wed Apr 16 11:52:27 PDT 2008
    
    
  
 > +	if (vector == 0) {
 > +		vector = priv->eq_table.last_comp_eq %
 > +			priv->eq_table.num_comp_eqs + 1;
 > +		priv->eq_table.last_comp_eq = vector;
 > +	}
The current IB code is written assuming that 0 is a normal completion
vector I think.  Making 0 be a special "round robin" value is a pretty
big change of policy.
Also there is no locking of last_comp_eq that I can see here, although
maybe it doesn't matter.
 > +	req_eqs = (dev->flags & MLX4_FLAG_MSI_X) ? num_online_cpus() : 1;
I don't think num_online_cpus() is the right thing really... what if a
CPU is hot-plugged later?  num_possible_cpus() seems better to me.
 - R.
    
    
More information about the general
mailing list