From: "Lazar, Alexei Avshalom" <ailizaro@codeaurora.org>
To: Johannes Berg <johannes@sipsolutions.net>
Cc: linux-wireless@vger.kernel.org, wil6210@qca.qualcomm.com
Subject: Re: cfg80211: add set/get link loss profile
Date: Mon, 6 Mar 2017 15:54:14 +0200 [thread overview]
Message-ID: <ac1dc46d-25e3-f7a1-0d68-cc2f80515b5a@codeaurora.org> (raw)
In-Reply-To: <1483353931.4596.3.camel@sipsolutions.net>
On 1/2/2017 12:45 PM, Johannes Berg wrote:
> On Mon, 2016-12-19 at 12:58 +0200, Lazar, Alexei Avshalom wrote:
>
>>> What you're doing here is *extremely* vague. No definitions of what
>>> this means, no description of how it's intended to be implemented,
>>> no note even on whether or not this is for full-MAC chips or not,
>>> etc.
>>>
>>
>> The patch is intended for full-MAC chips and I will update the commit
>> message to include more details.
>>
>>> At least for mac80211, we can define signals to userspace regarding
>>> "link quality" and make decisions in wpa_s then?
>>>
>>
>> The policy is offloaded today to the FW and the decision being made
>> by the FW.
>> We are only modifying the behavior and not making the decision.
>
> Fair enough, but I stand by saying that it's extremely vague - how
> could userspace possibly know what "level" to pick, if you don't give
> it any information about what the levels really mean?
>
> johannes
>
I have updated the commit message for clarifying what the levels really mean.
--
Alexei Lazar
Qualcomm Israel, on behalf of Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum a Linux Foundation Collaborative Project
prev parent reply other threads:[~2017-03-06 14:20 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-11-10 11:33 cfg80211: add set/get link loss profile Lazar, Alexei Avshalom
2016-11-10 13:55 ` Kalle Valo
2016-11-17 13:50 ` Lazar, Alexei Avshalom
2016-11-18 11:05 ` Kalle Valo
2016-12-19 10:58 ` Lazar, Alexei Avshalom
2016-12-19 10:58 ` [PATCH v2] " Lazar, Alexei Avshalom
2016-12-20 17:00 ` Dan Williams
2017-03-06 13:54 ` [PATCH v3] " Lazar, Alexei Avshalom
2017-03-29 9:01 ` Johannes Berg
2016-11-28 14:52 ` Johannes Berg
2016-12-19 10:58 ` Lazar, Alexei Avshalom
2017-01-02 10:45 ` Johannes Berg
2017-03-06 13:54 ` Lazar, Alexei Avshalom [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=ac1dc46d-25e3-f7a1-0d68-cc2f80515b5a@codeaurora.org \
--to=ailizaro@codeaurora.org \
--cc=johannes@sipsolutions.net \
--cc=linux-wireless@vger.kernel.org \
--cc=wil6210@qca.qualcomm.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).