Re: [PATCH v12 2/2] cfg80211: P2P find phase offload

[Johannes Berg <johannes@sipsolutions.net>]

Looking good ... :)

On Tue, 2013-06-25 at 16:42 +0300, Vladimir Kondratiev wrote:

> + * @channels: channels to operate on
> + *     may be NULL, in this case driver suppose to scan for all
> social
> + *     channels; or channel set should be non-empty (n_channels > 0)

Does that make sense, or should cfg80211 just build the channel list in
the case the user didn't specify anything, rather than leave it NULL? It
seems that every driver implementing it really needs the list, and
having multiple drivers create it seems wasteful?

> +static int nl80211_stop_p2p_find(struct sk_buff *skb, struct genl_info *info)
> +{
> +	struct cfg80211_registered_device *rdev = info->user_ptr[0];
> +	struct wireless_dev *wdev = info->user_ptr[1];
> +
> +	if (wdev->iftype != NL80211_IFTYPE_P2P_DEVICE)
> +		return -EOPNOTSUPP;
> +
> +	if (!rdev->ops->start_p2p_find || !rdev->ops->stop_p2p_find)
> +		return -EOPNOTSUPP;
> +
> +	rdev_stop_p2p_find(rdev, wdev);

You don't track whether this p2p find is ongoing, so userspace can abort
it many times. I think it would be worth documenting that, and also that
in that case cfg80211_p2p_find_notify_end() probably shouldn't be
called?

johannes