On Mon, May 12, 2014 at 10:51:37PM +0530, Rajkumar Manoharan wrote:
> On Mon, May 12, 2014 at 04:00:03PM +0200, Johannes Berg wrote:
> > On Fri, 2014-05-09 at 15:58 +0530, Rajkumar Manoharan wrote:
> >
> > > +static bool cfg80211_is_all_countryie_ignore(void)
> > > +{
> > > + struct cfg80211_registered_device *rdev;
> > > + struct wireless_dev *wdev;
> > > +
> > > + list_for_each_entry(rdev, &cfg80211_rdev_list, list) {
> > > + list_for_each_entry(wdev, &rdev->wdev_list, list) {
> > > + wdev_lock(wdev);
> > > + if (!(wdev->wiphy->regulatory_flags &
> > > + REGULATORY_COUNTRY_IE_IGNORE)) {
> > > + wdev_unlock(wdev);
> >
> > The locking here seems completely pointless.
> >
> locking is handled in same way of cfg80211_is_all_idle. Still
> want to remove locks?
>
> > Also, maybe you can do something about the long lines, maybe with a
> > temporary variable?
> >
sorry for the noise. sent v3 patch.
-Rajkumar
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
