On Thu, 2016-01-07 at 13:52 +0100, Arend van Spriel wrote:

Agree. However, one of the reasons I added the ext_feature is because
people gave feedback they wanted it for testing purposes so they know
what to expect from the driver. That same argument could used for the
supported selection criteria. I have no strong opinion though.

Yeah, that's true - if we play the "advisory" card then we don't really
need to add it at all... Not sure what to do really. I'd love to
specify it very clearly if we need to at all.

Perhaps for features we can add NL80211_ATTR_BSS_SELECT to the wiphy
information, and within that add each of the supported
ATTR_BSS_SELECT_* as an NLA_FLAG? That way at least we don't need to
define all kinds of new flags in the API.

quoted

Right. So realistically, writing this a bit more verbosely, you
have

1) rssi_preference, band_preference(band)
2) rssi_adjust(band, delta), rssi_preference

and perhaps

3) rssi_preference

as the default?

Good point.

quoted

As for 1), you said it was "band, rssi" but it seems you really
meant
the other way around since before you said "band" was a tie-
breaker.

My bad. When using "band_preference()" the selected BSS is in specified 
band even though the BSS in other band would be stronger. If there is no 
proper BSS in the specified band the preference is obviously
discarded.

So 1) really becomes just
 1) band_preference(band)

without any RSSI at all? Ah, I get it. You have to kind of reword that
in terms of the logic used:

 1) select_highest_rssi_in_band(band), select_highest_rssi()
 2) adjust_rssi_if_in_band(band, delta), select_highest_rssi()
 3) select_highest_rssi()

But if you look at that, then "select_highest_rssi_in_band(band)" is
really the same as "adjust_rssi_if_in_band(band, 1000)" since you can't
have real RSSI that high, which leaves us with just a single primitive
and perhaps a standard value for that "1000"?

quoted

Perhaps then, the API should just expose the two "primitives"
  * band_preference(band)
  * rssi_adjust(band, delta)?

I guess it makes even more sense as I just worded it.

quoted

And for use-case number #3:

   * rssi_preference(void)

Yeah although you'd assume that's the default anyway?

johannes