Re: Patch resubmission: RFC2863 operstatus for 2.5.49

["David S. Miller" <davem@xxxxxxxxxx>]

This locking below achieves nothing.

+       read_lock_irqsave(&dev->operstate_lock, flags);
+       state = dev->operstate;
+       read_unlock_irqrestore(&dev->operstate_lock, flags);

In fact, the other side, locking when setting this value, can
be done with a simple spinlock.  Probably something else in
the device struct can be reused.

I also don't think this should be conditional, either we want
it or we don't.