Search Linux Wireless

signal quality names

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sun Apr 15 06:22:46 CEST 2007 Larry Finger wrote:
> The variables in mac80211 are confusing and should be renamed; however,
> that may take, some time to reach a consensus. 

this was seven month ago. has there been any consensus reached?

the current names are very misleading:

/**
 * struct ieee80211_rx_status - receive status
 * [...]
 * @ssi: signal strength when receiving this frame
 * @signal: used as 'qual' in statistics reporting
 * @noise: PHY noise when receiving this frame

no units are specified. so this increases the confusion we have already 
because of poorly defined wireless extensions. please let's not repeat this 
mistake and let's try to standardize the reporting of signal and noise values 
across drivers at least up to mac80211.

i see that most of the drivers already try to report signal and noise ('ssi' 
and 'noise' according to the current naming) in dBm values and calculate a 
quality percentage (current name: 'signal') based on different factors.

so may i suggest to use variable names like

signal_dbm
noise_dbm

then? or leave them as 'signal' and 'noise' and make it clear in the kerneldoc 
that we expect dBm.

if there are drivers which cannot convert their internal values into dBm, we 
could provide an alternative 'ssi' (or similar, positive, unit-less, 
un-defined) to allow for that and handle the differences in reporting (to 
wext and for the radiotap headers) inside mac80211.

and could we move the quality calculation up into mac80211 to have consistency 
and comparable values across drivers?

i am willing to do the work.

bruno
-
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]
  Powered by Linux