Thread (30 messages) 30 messages, 2 authors, 2018-08-28

Re: [PATCH 2/7] cfg80211: Add new NL command to configure peer specific rssi threshold

From: Johannes Berg <johannes@sipsolutions.net>
Date: 2018-07-06 11:40:55

On Wed, 2018-07-04 at 11:39 +0530, Tamizh chelvam wrote:
On 2018-06-29 14:59, Johannes Berg wrote:
quoted
On Wed, 2018-06-13 at 16:15 +0530, Tamizh chelvam wrote:
quoted
+ * @NL80211_ATTR_STA_MON: Station's connection monitor configuration 
in a
+ *	nested attribute with %NL80211_ATTR_STA_MON_* sub-attributes.
Can't we reuse the existing attributes in this nesting?
This new attributes introduced as per the previous discussion.
https://patchwork.kernel.org/patch/10240697/
Is my understanding correct ?
Yeah ok ... I'm thinking perhaps we could share some part of it, but
it's not really worth making it more complex due to that.
quoted
Wait, you went to all this effort in the first patch and now you don't
add this to the list?
No, the list and the rssi_config structure is for multi threshold value 
and not for single threshold.
Yeah, though I'm still not sure why we need the single threshold thing
at all. List should always be a superset.

johannes
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help