Re: [PATCH 8/8] Makefile: drop -D__CHECK_ENDIAN__ from cflags

2016-12-14 Thread Marcel Holtmann
Hi Michael, > That's the default now, no need for makefiles to set it. > > Signed-off-by: Michael S. Tsirkin > --- > drivers/bluetooth/Makefile| 2 -- > drivers/net/can/Makefile | 1 - >

[RFC 3/3] mac80211: Add receive path for ethernet frame format

2016-12-14 Thread Vasanthakumar Thiagarajan
Implement rx path which does fewer processing on the received data frame which has already gone through 802.11 header decapsulation and other functionalities which require 802.11 header in the low level driver or hardware. Currently this rx path is restricted to AP and STA mode, but can be

[RFC 2/3] mac80211: Implement data xmit for 802.11 encap offload

2016-12-14 Thread Vasanthakumar Thiagarajan
Driver (or hw) supporting 802.11 encapsulation offload for data frames can make use of this new xmit path. This patch defines new ndo_ops, all these callbacks are same as ieee80211_dataif_ops other than ndo_start_xmit() which does minimal processing leaving 802.11 encap related to driver. This

[RFC 1/3] mac80211: Add provision for 802.11 encap/decap offload

2016-12-14 Thread Vasanthakumar Thiagarajan
Drivers can have the capability to offload 802.11 encap/decap to firmware or hardware for data frames. This patch adds a new hw_flag for driver to advertise the offload support. Drivers advertising the support have also to implement new ieee80211_ops callback, get_vif_80211_hdr_offload(), to

[RFC 0/3] Add new data path for ethernet frame format

2016-12-14 Thread Vasanthakumar Thiagarajan
This patch set adds a new data path to offload 802.11 header encap/decap to driver or hardware. Drivers having support for ieee80211 header encap/decap and other offload functionalities which can't be done before encap or after decap can make use of this new data path. Currently it is implemented

[PATCH 8/8] Makefile: drop -D__CHECK_ENDIAN__ from cflags

2016-12-14 Thread Michael S. Tsirkin
That's the default now, no need for makefiles to set it. Signed-off-by: Michael S. Tsirkin --- drivers/bluetooth/Makefile| 2 -- drivers/net/can/Makefile | 1 - drivers/net/ethernet/altera/Makefile

[PATCH 5/8] linux: drop __bitwise__ everywhere

2016-12-14 Thread Michael S. Tsirkin
__bitwise__ used to mean "yes, please enable sparse checks unconditionally", but now that we dropped __CHECK_ENDIAN__ __bitwise is exactly the same. There aren't many users, replace it by __bitwise everywhere. Signed-off-by: Michael S. Tsirkin ---

Re: Problems getting mwifiex with sd8887 to work

2016-12-14 Thread Federico Pietro Briata
Hi All, I'm also using sd8887 and I'm trying to make it work in adhoc mode and encrypt the channel with wpa-none, but I got some trouble. If I make the same specific test with from the driver from marvel intranet, is working (maybe because the firmware is more recent ?) but anyway the general

Re: [Patch] NFC: trf7970a:

2016-12-14 Thread Mark Greer
On Wed, Dec 14, 2016 at 01:35:03PM -0500, Geoff Lansberry wrote: > On Wed, Dec 14, 2016 at 12:10 PM, Mark Greer wrote: > > On Wed, Dec 14, 2016 at 11:17:33AM -0500, Geoff Lansberry wrote: > >> On Wed, Dec 14, 2016 at 10:57 AM, Mark Greer > >>

Re: [RFC v2 05/11] ath10k: htc: refactorization

2016-12-14 Thread Erik Stromdahl
On 12/14/2016 02:46 PM, Valo, Kalle wrote: > Erik Stromdahl writes: > >> I have made a few updates since I submitted the original RFC and created >> a repo on github: >> >> https://github.com/erstrom/linux-ath >> >> I have a bunch of branches that are all based on the

Re: [PATCH v2] ath9k: do not return early to fix rcu unlocking

2016-12-14 Thread Felix Fietkau
On 2016-12-13 18:08, Tobias Klausmann wrote: > Starting with commit d94a461d7a7d ("ath9k: use ieee80211_tx_status_noskb > where possible") the driver uses rcu_read_lock() && rcu_read_unlock(), yet on > returning early in ath_tx_edma_tasklet() the unlock is missing leading to > stalls > and

Re: ath10k firmware sends probes on DFS channels without radar detection

2016-12-14 Thread Jouni Malinen
On Tue, Dec 06, 2016 at 06:02:52PM +0100, Jean-Pierre Tosoni wrote: > This follows on the previous discussion > "Client station sends probes on DFS channels" > > Problem: > The combination of QCA988X firmware v10.2.4.70-2 + ath10k + > wpa_supplicant do not comply with the norm ETSI/EN

[PATCH 01/10] mac80211: minstrel_ht: move supported bitrate mask out of group data

2016-12-14 Thread Felix Fietkau
Improves dcache footprint by ensuring that fewer cache lines need to be touched. Signed-off-by: Felix Fietkau --- net/mac80211/rc80211_minstrel_ht.c | 30 +++--- net/mac80211/rc80211_minstrel_ht.h | 6 +++---

[PATCH 02/10] mac80211: minstrel_ht: move short preamble check out of get_rate

2016-12-14 Thread Felix Fietkau
Test short preamble support in minstrel_ht_update_caps instead of looking at the per-packet flag. Makes the code more efficient. Signed-off-by: Felix Fietkau --- net/mac80211/rc80211_minstrel_ht.c | 22 +- 1 file changed, 5 insertions(+), 17 deletions(-) diff

[PATCH 03/10] mac80211: minstrel_ht: make att_hist and succ_hist u32 instead of u64

2016-12-14 Thread Felix Fietkau
They are only used for debugging purposes and take a very long time to overflow. Visibly reduces the size of the per-sta rate control data. Signed-off-by: Felix Fietkau --- net/mac80211/rc80211_minstrel.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 08/10] mac80211: minstrel: make prob_ewma u16 instead of u32

2016-12-14 Thread Felix Fietkau
Saves about 1.2 KiB memory per station Signed-off-by: Felix Fietkau --- net/mac80211/rc80211_minstrel.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/net/mac80211/rc80211_minstrel.h b/net/mac80211/rc80211_minstrel.h index ea86e67..be6c3f3 100644 ---

[PATCH 10/10] mac80211: minstrel: avoid port control frames for sampling

2016-12-14 Thread Felix Fietkau
From: Thomas Huehn Makes connections more reliable Signed-off-by: Thomas Huehn Signed-off-by: Felix Fietkau --- net/mac80211/rc80211_minstrel.c | 5 + 1 file changed, 5 insertions(+) diff --git

[PATCH 06/10] mac80211: minstrel: reduce MINSTREL_SCALE

2016-12-14 Thread Felix Fietkau
The loss of a bit of extra precision does not hurt the calculation, 12 bits is still enough to calculate probabilities well. Reducing the scale makes it easier to avoid overflows Signed-off-by: Felix Fietkau --- net/mac80211/rc80211_minstrel.h | 2 +- 1 file changed, 1

[PATCH 04/10] mac80211: check for MCS in ieee80211_duration before fetching chanctx

2016-12-14 Thread Felix Fietkau
Makes the code a bit more efficient Signed-off-by: Felix Fietkau --- net/mac80211/tx.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/net/mac80211/tx.c b/net/mac80211/tx.c index 058652d..4dea18b 100644 --- a/net/mac80211/tx.c +++ b/net/mac80211/tx.c

[PATCH 05/10] mac80211: minstrel: remove cur_prob from debugfs

2016-12-14 Thread Felix Fietkau
This field is redundant, because it is simply last success divided by last attempt count. Removing it from the rate stats struct saves about 1.2 KiB per HT station. Signed-off-by: Felix Fietkau --- net/mac80211/rc80211_minstrel.c| 10 ++

[PATCH 07/10] mac80211: minstrel: store probability variance instead of standard deviation

2016-12-14 Thread Felix Fietkau
This avoids the costly int_sqrt calls in the statistics update and moves it to the debugfs code instead. This also fixes an overflow in the previous standard deviation calculation. Signed-off-by: Thomas Huehn Signed-off-by: Felix Fietkau ---

[PATCH 09/10] mac80211: minstrel_ht: remove obsolete #if for >= 3 streams

2016-12-14 Thread Felix Fietkau
This was added during early development when 3x3 hardware was not very common yet. This is completely unnecessary now. Signed-off-by: Felix Fietkau --- net/mac80211/rc80211_minstrel_ht.c | 20 1 file changed, 20 deletions(-) diff --git

[PATCH v2] mac80211: fix legacy and invalid rx-rate rpt

2016-12-14 Thread greearb
From: Ben Greear This fixes obtaining the rate info via sta_set_sinfo when the rx rate is invalid (for instance, on IBSS interface that has received no frames from one of its peers). This also fixes a more general issue with rinfo->flags not being properly initialized

RE: ath10k firmware sends probes on DFS channels without radar detection

2016-12-14 Thread Jean-Pierre Tosoni
On 12/06/15 08:36 PM, Ben Greear wrote: > On 12/06/2016 09:02 AM, Jean-Pierre Tosoni wrote: > > This follows on the previous discussion > > "Client station sends probes on DFS channels" > > > > Problem: > > The combination of QCA988X firmware v10.2.4.70-2 + ath10k + > > wpa_supplicant do not

Re: [Patch] NFC: trf7970a:

2016-12-14 Thread Geoff Lansberry
On Wed, Dec 14, 2016 at 12:10 PM, Mark Greer wrote: > On Wed, Dec 14, 2016 at 11:17:33AM -0500, Geoff Lansberry wrote: >> On Wed, Dec 14, 2016 at 10:57 AM, Mark Greer wrote: >> > >> > On Tue, Dec 13, 2016 at 08:50:04PM -0500, Geoff Lansberry wrote:

Re: ath10k firmware sends probes on DFS channels without radar detection

2016-12-14 Thread Ben Greear
On 12/14/2016 10:14 AM, Jean-Pierre Tosoni wrote: On 12/06/15 08:36 PM, Ben Greear wrote: On 12/06/2016 09:02 AM, Jean-Pierre Tosoni wrote: This follows on the previous discussion "Client station sends probes on DFS channels" Problem: The combination of QCA988X firmware v10.2.4.70-2 +

Re: [Patch] NFC: trf7970a:

2016-12-14 Thread Mark Greer
On Wed, Dec 14, 2016 at 11:17:33AM -0500, Geoff Lansberry wrote: > On Wed, Dec 14, 2016 at 10:57 AM, Mark Greer wrote: > > > > On Tue, Dec 13, 2016 at 08:50:04PM -0500, Geoff Lansberry wrote: > > > Hi Mark - Thanks for getting back to me. It's funny that you ask, > > >

Re: [PATCH 1/2] ath10k: add accounting for the extended peer statistics

2016-12-14 Thread Christian Lamparter
On Wednesday, December 14, 2016 1:03:38 PM CET Mohammed Shafi Shajakhan wrote: > > On Wednesday, December 7, 2016 11:58:24 AM CET Mohammed Shafi Shajakhan > > wrote: > > > On Mon, Dec 05, 2016 at 10:52:45PM +0100, Christian Lamparter wrote: > > > > @@ -409,10 +410,12 @@ void

[PATCH] mac80211: only alloc mem if a station doesnt exist yet

2016-12-14 Thread Koen Vandeputte
This speeds up the function in case a station already exists by avoiding calling an expensive kzalloc just to free it again after the next check. Signed-off-by: Koen Vandeputte --- net/mac80211/sta_info.c | 14 +++--- 1 file changed, 7 insertions(+), 7

Re: [Patch] NFC: trf7970a:

2016-12-14 Thread Geoff Lansberry
On Wed, Dec 14, 2016 at 10:57 AM, Mark Greer wrote: > > On Tue, Dec 13, 2016 at 08:50:04PM -0500, Geoff Lansberry wrote: > > Hi Mark - Thanks for getting back to me. It's funny that you ask, > > because we are currently chasing a segfault that is happening in neard, but

Re: [Patch] NFC: trf7970a:

2016-12-14 Thread Mark Greer
On Tue, Dec 13, 2016 at 08:50:04PM -0500, Geoff Lansberry wrote: > Hi Mark - Thanks for getting back to me. It's funny that you ask, > because we are currently chasing a segfault that is happening in neard, but > may end up back in the trf7970a driver. Have you ever heard on anyone > having

[PATCH 2/5] mwifiex: cleanup in PCIe flr code path

2016-12-14 Thread Amitkumar Karwar
From: Xinming Hu adapter and card variables don't get freed during PCIe function level reset. "adapter->ext_scan" variable need not be re-initialized. fw_name and tx_buf_size initialization is moved to pcie specific code so that mwifiex_reinit_sw() can be used by SDIO.

[PATCH 3/5] mwifiex: sdio card reset enhancement

2016-12-14 Thread Amitkumar Karwar
From: Xinming Hu Commit b4336a282db8 ("mwifiex: sdio: reset adapter using mmc_hw_reset") introduces a simple sdio card reset solution based on card remove and re-probe. This solution has proved to be vulnerable, as card and adapter structures are not protected, concurrent

[PATCH 5/5] mwifiex: get rid of global save_adapter and sdio_work

2016-12-14 Thread Amitkumar Karwar
From: Xinming Hu This patch moves sdio_work to card structure, in this way we can get adapter structure in the work, so save_adapter won't be needed. Signed-off-by: Xinming Hu Signed-off-by: Amitkumar Karwar ---

[PATCH 4/5] mwifiex: get rid of __mwifiex_sdio_remove helper

2016-12-14 Thread Amitkumar Karwar
From: Xinming Hu __mwifiex_sdio_remove helper is not needed after our enhancements in SDIO card reset. Signed-off-by: Xinming Hu Signed-off-by: Amitkumar Karwar --- drivers/net/wireless/marvell/mwifiex/sdio.c | 11 +++ 1 file

[PATCH 1/5] mwifiex: get rid of mwifiex_do_flr wrapper

2016-12-14 Thread Amitkumar Karwar
From: Xinming Hu This patch gets rid of mwifiex_do_flr. We will call mwifiex_shutdown_sw() and mwifiex_reinit_sw() directly. These two general purpose functions will be useful for sdio card reset handler. Signed-off-by: Xinming Hu Signed-off-by: Amitkumar

Re: [RFC v2 05/11] ath10k: htc: refactorization

2016-12-14 Thread Valo, Kalle
Michal Kazior writes: >> I have made a few updates since I submitted the original RFC and created >> a repo on github: >> >> https://github.com/erstrom/linux-ath >> >> I have a bunch of branches that are all based on the tags on the ath master. >> >> As of this moment

Re: [RFC v2 05/11] ath10k: htc: refactorization

2016-12-14 Thread Valo, Kalle
Erik Stromdahl writes: > I have made a few updates since I submitted the original RFC and created > a repo on github: > > https://github.com/erstrom/linux-ath > > I have a bunch of branches that are all based on the tags on the ath master. > > As of this moment the

Re: [RFC V3 04/11] nl80211: add driver api for gscan notifications

2016-12-14 Thread Arend Van Spriel
On 13-12-2016 17:20, Johannes Berg wrote: > On Mon, 2016-12-12 at 11:59 +, Arend van Spriel wrote: >> The driver can indicate gscan results are available or gscan >> operation has stopped. > > This patch is renumbering the previous patches' nl80211 API, which is > best avoided, even if I do

Re: [RFC V3 03/11] nl80211: add support for gscan

2016-12-14 Thread Arend Van Spriel
On 13-12-2016 23:29, Johannes Berg wrote: > On Tue, 2016-12-13 at 21:09 +0100, Arend Van Spriel wrote: >> >>> There's a bit of a weird hard-coded restriction to 16 channels too, >>> that's due to the bucket map? >> >> Uhm. Is there? I will check, but if you can give me a pointer where >> to