From patchwork Thu Jan 5 14:38:29 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jorge Ramirez X-Patchwork-Id: 711425 X-Patchwork-Delegate: davem@davemloft.net Return-Path: X-Original-To: patchwork-incoming@ozlabs.org Delivered-To: patchwork-incoming@ozlabs.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 3tvVfl1pQdz9t17 for ; Fri, 6 Jan 2017 01:38:39 +1100 (AEDT) Authentication-Results: ozlabs.org; dkim=pass (1024-bit key; unprotected) header.d=linaro.org header.i=@linaro.org header.b="W57z47u9"; dkim-atps=neutral Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1161631AbdAEOif (ORCPT ); Thu, 5 Jan 2017 09:38:35 -0500 Received: from mail-wj0-f179.google.com ([209.85.210.179]:36667 "EHLO mail-wj0-f179.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753617AbdAEOid (ORCPT ); Thu, 5 Jan 2017 09:38:33 -0500 Received: by mail-wj0-f179.google.com with SMTP id ew7so22741589wjc.3 for ; Thu, 05 Jan 2017 06:38:32 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=subject:to:references:cc:from:message-id:date:user-agent :mime-version:in-reply-to:content-transfer-encoding; bh=Sgw0hB3MYg25YCMP0pAkj4bDvlbEo0+6UqHl5ntYsE0=; b=W57z47u9EiVqdVDjypd2X8OP0/tExlmMydhw5SSXopMCzjxWFoUNMMV7jArHdUcKW+ /Zw1y4yfk15oQW0Fl1SJ9/N4ZSCt0I/HJ3ZWM7MlWe+EvvP31nAFaRUVfE5FP4wZls5F /XPmfK1KK5qKiJkH/GSkPzWbyB8kYrjfFi6vs= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:references:cc:from:message-id:date :user-agent:mime-version:in-reply-to:content-transfer-encoding; bh=Sgw0hB3MYg25YCMP0pAkj4bDvlbEo0+6UqHl5ntYsE0=; b=gunhbD8VlbYxjCpgGm+RK0ybAclGPZRrTwUzZWGg5Bel+VDVC91qgtRLcLnQ/e/tkF 2Zo3GOcbFwKgeoi8Z1UtHElWv4Zdps7WqaDWe5m8gCMZZpdIogQnhpggDMY7GKNhag5j vWK1L7QBT/oaQPhbjsy1i/Y7ODNCmOz78TTsXbas059CeVOQ1HfzApIJrcgv9ulSDlVQ JT8HfZLzXfsz8rIY4W4K0wY1GrQ5Un2yF/bu9G4VcZXcARC39kxGuoQZjoNpm6u3hmTi BEQ6623ObF3J9NIgcnuQIaQFLFW5yGFXdYuXSpUSX7Z9rU/vAg81vVahqNxsVMceIYdN MCdg== X-Gm-Message-State: AIkVDXLanU7tjn1olrCP0G2w6ICmbH55HJKd/ZYfl/7Jqgti25+dLyxxnXquGHxSTAEDQHsb X-Received: by 10.194.43.73 with SMTP id u9mr60801439wjl.109.1483627111621; Thu, 05 Jan 2017 06:38:31 -0800 (PST) Received: from [192.168.1.240] (178.red-79-157-58.dynamicip.rima-tde.net. [79.157.58.178]) by smtp.gmail.com with ESMTPSA id cl6sm104042313wjc.10.2017.01.05.06.38.30 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Thu, 05 Jan 2017 06:38:30 -0800 (PST) Subject: Re: SIOCSIWFREQ while in NL80211_IFTYPE_STATION To: Johannes Berg , netdev@vger.kernel.org, Daniel Lezcano References: <685811c3-6247-77fd-8c70-617951886451@linaro.org> <1483616301.4394.6.camel@sipsolutions.net> <92e60cb2-8bb4-df03-3434-4a055f575df0@linaro.org> <1483625204.4394.23.camel@sipsolutions.net> Cc: linux-wireless From: Jorge Ramirez Message-ID: Date: Thu, 5 Jan 2017 15:38:29 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.5.1 MIME-Version: 1.0 In-Reply-To: <1483625204.4394.23.camel@sipsolutions.net> Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org On 01/05/2017 03:06 PM, Johannes Berg wrote: >> I am not sure it matters - if I understood your reply, there is no >> valid use case to change the frequency in that mode (and all that >> code should be removed); > All of wext *is* being removed - slowly :) > It's not longer default in the kernel configuration now. ah thanks for the info. I should have checked! > > IIRC, there actually was a valid use case here - to set the frequency > before you set the SSID - to be able to force a connection on that > channel with wext. ok, yes I thought this would make sense. > >> it seems to me that it is also your view that userspace (iwconfig) >> should be fixed accordingly to not call the extensions? > iwconfig should just be deleted and iw be used :-) understood. > >> I am just trying to understand how the current code is supposed to >> work by exercising widely available user-space tools while debugging >> the kernel. > Heh, ok. > >> Actually the frequency gets programmed without errors when reverting >> your commit http://tinyurl.com/ho4urp8 (which comes as a surprise as >> well). > Well, it's not my commit, but that makes sense. I suppose we should > treat this as a regression, but reverting that doesn't seem like the > right fix. Actually, I'm not convinced we should do monitor channel > change anyway when you set the frequency with wext, so we can probably > just remove that call there - want to send a patch to that effect? do you mean this? [jramirez@igloo ~ (debian-qcom-dragonboard410c-16.09-local $)]$ git diff I tested the change above: we can now modify the channel/frequency when the SSID is not set in managed mode. When the SSID is set however iwconfig does not report any error but channel/frequency doesn't change. if you think this is acceptable I can submit a patch > johannes diff --git a/net/wireless/wext-sme.c b/net/wireless/wext-sme.c index a4e8af3..c56bac5 100644 --- a/net/wireless/wext-sme.c +++ b/net/wireless/wext-sme.c @@ -106,30 +106,7 @@ int cfg80211_mgd_wext_siwfreq(struct net_device *dev, goto out; } - wdev->wext.connect.channel = chan; - - /* - * SSID is not set, we just want to switch monitor channel, - * this is really just backward compatibility, if the SSID - * is set then we use the channel to select the BSS to use - * to connect to instead. If we were connected on another - * channel we disconnected above and reconnect below. - */ - if (chan && !wdev->wext.connect.ssid_len) { - struct cfg80211_chan_def chandef = { - .width = NL80211_CHAN_WIDTH_20_NOHT, - .center_freq1 = freq, - }; - - chandef.chan = ieee80211_get_channel(&rdev->wiphy, freq); - if (chandef.chan) - err = cfg80211_set_monitor_channel(rdev, &chandef); - else - err = -EINVAL; - goto out; - } - err = cfg80211_mgd_wext_connect(rdev, wdev); out: wdev_unlock(wdev);