> +    atomic_t vlan_task_flag;

why is this atomic_t?  I only see:

 > +            atomic_set(&priv->vlan_task_flag, 1);
 > +            atomic_set(&priv->vlan_task_flag, 0);
 > +    iffup_value = atomic_read(&priv->vlan_task_flag) ? IFF_UP : 0;

so as far as I can tell you are not using anything atomic.  So if
there's a race you're worried about, it's still there...

 - R.
_______________________________________________
general mailing list
[email protected]
http://lists.openfabrics.org/cgi-bin/mailman/listinfo/general

To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general

Reply via email to