Re: using verifier to ensure a BPF program uses certain metadata?

2017-10-17 Thread Johannes Berg
Hi Alexei, > > https://git.kernel.org/pub/scm/linux/kernel/git/jberg/mac80211-next > > .git/log/?h=bpf > > bpf bits looks pretty straightforward. Thanks for looking at this! > attach looks fine too. I'm assuming there is some rtnl or other lock, > so multiple assigns cannot race? Yes. > It's

Re: [PATCH v6 1/3] dt-bindings: net: add mt76 wireless device binding

2017-10-17 Thread Felix Fietkau
On 2017-10-18 04:02, Rob Herring wrote: > On Sat, Oct 14, 2017 at 10:43 AM, Christian Lamparter > wrote: >> On Saturday, October 14, 2017 9:20:46 AM CEST Felix Fietkau wrote: >>> On 2017-10-13 21:07, Rob Herring wrote: >>> > On Fri, Oct 06, 2017 at 01:02:47PM +0200, Felix Fietkau wrote: >>> >> Add

[PATCH v2 7/9] iwlwifi: mvm: add missing lq_color

2017-10-17 Thread Luca Coelho
From: Liad Kaufman In the compressed BA notif, the driver didn't parse out the LQ color, so statistics for the rates tried were always thrown out. Add it so it gets correctly used. While at it, fix the name of the relevant field in the struct. Fixes: c46e7724bfe9 ("iwlwifi: mvm: support new BA

Re: After upgrading to 4.11.1, wifi driver refuses to load after being unloaded once.

2017-10-17 Thread Luca Coelho
On Wed, 2017-10-18 at 07:59 +0300, Kalle Valo wrote: > Luca Coelho writes: > > > On Tue, 2017-10-17 at 14:23 -0700, Marc MERLIN wrote: > > > On Tue, Oct 17, 2017 at 05:05:57PM +0300, Luca Coelho wrote: > > > > Hi, > > > > > > > > On Tue, 2017-10-17 at 02:44 -0700, Marc MERLIN wrote: > > > > > Wa

Re: [PATCH 00/58] networking: Convert timers to use timer_setup()

2017-10-17 Thread Kalle Valo
Kees Cook writes: > On Tue, Oct 17, 2017 at 7:18 AM, Kalle Valo wrote: >> + linux-wireless >> >> Hi Kees, >> >> Kees Cook writes: >> >>> This is the current set of outstanding networking patches to perform >>> conversions to the new timer interface (rebased to -next). This is not >>> all expect

Re: Two rtlwifi drivers?

2017-10-17 Thread Kalle Valo
Pkshih writes: >> My recommendation is to avoid accumulating patches at all cost and start >> submitting them as soon as you can. This way you get patches committed >> much more smoother. So do not wait until _all_ patches are ready, >> instead start submitting patches as soon as you have _some_

Re: After upgrading to 4.11.1, wifi driver refuses to load after being unloaded once.

2017-10-17 Thread Kalle Valo
Luca Coelho writes: > On Tue, 2017-10-17 at 14:23 -0700, Marc MERLIN wrote: >> On Tue, Oct 17, 2017 at 05:05:57PM +0300, Luca Coelho wrote: >> > Hi, >> > >> > On Tue, 2017-10-17 at 02:44 -0700, Marc MERLIN wrote: >> > > Was broken in 4.11, still broken in 4.12. This is crippling, I'm >> > > not

Re: [PATCH v6 1/3] dt-bindings: net: add mt76 wireless device binding

2017-10-17 Thread Rob Herring
On Sat, Oct 14, 2017 at 10:43 AM, Christian Lamparter wrote: > On Saturday, October 14, 2017 9:20:46 AM CEST Felix Fietkau wrote: >> On 2017-10-13 21:07, Rob Herring wrote: >> > On Fri, Oct 06, 2017 at 01:02:47PM +0200, Felix Fietkau wrote: >> >> Add documentation describing how device tree can be

[PATCH] ath10k: Fix offchannel tx failure when no ath10k_mac_tx_frm_has_freq

2017-10-17 Thread greearb
From: Ben Greear This bug appears to have been added between 4.0 (which works for us), and 4.4, which does not work. I think this is because the tx-offchannel logic gets in a loop when ath10k_mac_tx_frm_has_freq(ar) is false, so pkt is never actually sent to the firmware for transmit. This patc

Re: iwlwifi crash with hostapd

2017-10-17 Thread James Cameron
On Tue, Oct 17, 2017 at 09:35:39PM +0200, Mario Theodoridis wrote: > On 16.10.2017 05:37, James Cameron wrote: > >On Sun, Oct 15, 2017 at 06:21:36PM +0200, Mario Theodoridis wrote: > >>Thanks for the pointers, James. > >> > >>On 12.10.2017 23:24, James Cameron wrote: > >>>There's a good chance this

[PATCH] net: mac80211: mark expected switch fall-throughs

2017-10-17 Thread Gustavo A. R. Silva
In preparation to enabling -Wimplicit-fallthrough, mark switch cases where we are expecting to fall through. Notice that in some cases I replaced "fall through on else" and "otherwise fall through" comments with just a "fall through" comment, which is what GCC is expecting to find. Signed-off-by:

Re: using verifier to ensure a BPF program uses certain metadata?

2017-10-17 Thread Alexei Starovoitov
On Mon, Oct 16, 2017 at 09:38:44AM +0200, Johannes Berg wrote: > Hi, > > As we discussed in April already (it's really been that long...), I'd > wanted to allow using BPF to filter wireless monitor frames, to enable > new use cases and higher performance in monitoring. I have some code, > at > >

Re: After upgrading to 4.11.1, wifi driver refuses to load after being unloaded once.

2017-10-17 Thread Luca Coelho
On Tue, 2017-10-17 at 14:23 -0700, Marc MERLIN wrote: > On Tue, Oct 17, 2017 at 05:05:57PM +0300, Luca Coelho wrote: > > Hi, > > > > On Tue, 2017-10-17 at 02:44 -0700, Marc MERLIN wrote: > > > Was broken in 4.11, still broken in 4.12. This is crippling, I'm > > > not > > > running linux so that I

Re: After upgrading to 4.11.1, wifi driver refuses to load after being unloaded once.

2017-10-17 Thread Marc MERLIN
On Tue, Oct 17, 2017 at 05:05:57PM +0300, Luca Coelho wrote: > Hi, > > On Tue, 2017-10-17 at 02:44 -0700, Marc MERLIN wrote: > > Was broken in 4.11, still broken in 4.12. This is crippling, I'm not > > running linux so that I have to reboot it to reload an intel wireless > > driver :-/ > > Can yo

[PATCH] mac80211: aggregation: Convert timers to use timer_setup()

2017-10-17 Thread Kees Cook
In preparation for unconditionally passing the struct timer_list pointer to all timer callbacks, switch to using the new timer_setup() and from_timer() to pass the timer pointer explicitly. This removes the tid mapping array and expands the tid structures to add a pointer back to the station, alon

[PATCH] cfg80211: fix connect/disconnect edge cases

2017-10-17 Thread Johannes Berg
From: Johannes Berg If we try to connect while already connected/connecting, but this fails, we set ssid_len=0 but leave current_bss hanging, leading to errors. Check all of this better, first of all ensuring that we can't try to connect to a different SSID while connected/ing; ensure that prev_

[PATCH] nl80211: don't expose wdev->ssid for most interfaces

2017-10-17 Thread Johannes Berg
From: Johannes Berg For mesh, this is simply wrong - there's no SSID, only the mesh ID, so don't expose it at all. For (P2P) client, it's wrong, because it exposes an internal value that's only used when certain APIs are used. For AP, it's actually the only correct case, so leave that. All other

Re: [PATCH 00/58] networking: Convert timers to use timer_setup()

2017-10-17 Thread Kees Cook
On Tue, Oct 17, 2017 at 7:18 AM, Kalle Valo wrote: > + linux-wireless > > Hi Kees, > > Kees Cook writes: > >> This is the current set of outstanding networking patches to perform >> conversions to the new timer interface (rebased to -next). This is not >> all expected conversions, but it contains

[PATCH] wil6210: disallow changing RSN in beacon change

2017-10-17 Thread Johannes Berg
From: Johannes Berg This is a code path that will never really get hit anyway, since it's nonsense to change the beacon of an existing BSS to suddenly include or no longer include the RSN IE. Reject this instead of having the dead code, and get rid of accessing wdev->ssid/_len by way of that. Si

Re: iwlwifi crash with hostapd

2017-10-17 Thread Mario Theodoridis
On 16.10.2017 05:37, James Cameron wrote: On Sun, Oct 15, 2017 at 06:21:36PM +0200, Mario Theodoridis wrote: Thanks for the pointers, James. On 12.10.2017 23:24, James Cameron wrote: There's a good chance this problem has been fixed already. You are using a v4.4 kernel with many patches app

[PATCH] libertas: don't write wdev->ssid/_len

2017-10-17 Thread Johannes Berg
From: Johannes Berg When joining an IBSS network, wdev->ssid/_len will already be set, so there's no need to write them. In any case, they are internal cfg80211 values, and have very little user-visible impact. Signed-off-by: Johannes Berg --- drivers/net/wireless/marvell/libertas/cfg.c | 3 --

[PATCH] wil6210: remove wil6210_uapi.h from MAINTAINERS

2017-10-17 Thread Johannes Berg
From: Johannes Berg This file doesn't exist. Signed-off-by: Johannes Berg --- MAINTAINERS | 1 - 1 file changed, 1 deletion(-) diff --git a/MAINTAINERS b/MAINTAINERS index 3944f1626911..7051e2ae9971 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -14565,7 +14565,6 @@ L: wil6...@qca.qualcom

[PATCH] wil6210: remove SSID debugfs

2017-10-17 Thread Johannes Berg
From: Johannes Berg This driver shouldn't be using wdev->ssid to start with, as it's more or less an internal field in cfg80211 used for various purposes. Reading it is possible through nl80211, even if that's not really what we should be doing there for anything but AP type interfaces. It *real

[PATCH] mac80211: use constant time comparison with keys

2017-10-17 Thread Jason A. Donenfeld
Otherwise we risk leaking information via timing side channel. Fixes: fdf7cb4185b6 ("mac80211: accept key reinstall without changing anything") Signed-off-by: Jason A. Donenfeld --- net/mac80211/key.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/net/mac80211/key.c b/net/

Re: pull-request: mac80211 2017-10-16

2017-10-17 Thread Jason A. Donenfeld
On Tue, Oct 17, 2017 at 7:46 AM, Johannes Berg wrote: > If it's not equal, you execute so much code > beneath, going to the driver etc., that I'd think this particular time > is in the noise. Usually presumptions like this get you in trouble when some crafty academic has a smart idea about that n

Re: [PATCH v2] ath10k: Retry pci probe on failure.

2017-10-17 Thread Ben Greear
On 10/17/2017 01:45 AM, Kalle Valo wrote: Ben Greear writes: On 10/13/2017 08:50 AM, Adrian Chadd wrote: On 13 October 2017 at 05:41, Kalle Valo wrote: gree...@candelatech.com writes: From: Ben Greear This works around a problem we see when sometimes the wifi NIC does not respond the fi

Re: Commit 0711d638 breaks mwifiex

2017-10-17 Thread Jesse Sung
On Tue, Oct 17, 2017 at 11:10 PM, Johannes Berg wrote: > Hi, > >> > > https://p.sipsolutions.net/8f9d5917a5cbe4b3.txt >> > > >> > > Can you also give that a spin? > > Thanks for testing! > >> > Issued four reassociate, the network interface is still alive. Also >> > with this patch it stays with B

Re: Commit 0711d638 breaks mwifiex

2017-10-17 Thread Johannes Berg
Hi, > > > https://p.sipsolutions.net/8f9d5917a5cbe4b3.txt > > > > > > Can you also give that a spin? Thanks for testing! > > Issued four reassociate, the network interface is still alive. Also > > with this patch it stays with BSSID1 and I don't see any > > "Association request to the driver fa

Re: bcma: use bcma_debug and pr_cont in MIPS driver

2017-10-17 Thread Kalle Valo
Rafał Miłecki wrote: > From: Rafał Miłecki > > Using bcma_debug gives a device-specific prefix for messages and pr_cont > is a common helper for continuing a line. > > Signed-off-by: Rafał Miłecki > Acked-By: Hauke Mehrtens Patch applied to wireless-drivers-next.git, thanks. 66cc04424960 bc

Re: rtlwifi: Fix typo in if ... else if ... else construct

2017-10-17 Thread Kalle Valo
Larry Finger wrote: > The kbuild test robot reports two conditions with no effect (if == else). > These are the result of copy and paste typographical errors. > > Signed-off-by: Larry Finger > Cc: Ping-Ke Shih > Cc: Yan-Hsuan Chuang > Cc: Birming Chiu > Cc: Shaofu > Cc: Steven Ting > Cc: k

Re: [PATCH 00/58] networking: Convert timers to use timer_setup()

2017-10-17 Thread Kalle Valo
+ linux-wireless Hi Kees, Kees Cook writes: > This is the current set of outstanding networking patches to perform > conversions to the new timer interface (rebased to -next). This is not > all expected conversions, but it contains everything needed in networking > to eliminate init_timer(), an

Re: Help with bug "Black screen freeze on suspend to ram (rtl8188ee)"

2017-10-17 Thread Larry Finger
On 10/17/2017 09:03 AM, Lauro Costa wrote: Dear maintainer, I have opened a bug report on bugzilla [1] regarding this issue I find when I attempt to suspend to ram, unfortunately no one has replied to me yet. I assumed this issue is related to the driver rtl8188ee since the black screen does

Re: Commit 0711d638 breaks mwifiex

2017-10-17 Thread Jesse Sung
On Tue, Oct 17, 2017 at 10:08 PM, Jesse Sung wrote: > On Tue, Oct 17, 2017 at 9:13 PM, Johannes Berg > wrote: >> >> On Tue, 2017-10-17 at 21:07 +0800, Jesse Sung wrote: >> > >> > > Not really quite sure about it yet, but that should address the >> > > issue? >> > >> > A very rough test by issuing

Re: Commit 0711d638 breaks mwifiex

2017-10-17 Thread Jesse Sung
On Tue, Oct 17, 2017 at 9:13 PM, Johannes Berg wrote: > > On Tue, 2017-10-17 at 21:07 +0800, Jesse Sung wrote: > > > > > Not really quite sure about it yet, but that should address the > > > issue? > > > > A very rough test by issuing reassociate in wpa_cli: > > mwifiex works after reassociate - l

Re: After upgrading to 4.11.1, wifi driver refuses to load after being unloaded once.

2017-10-17 Thread Luca Coelho
Hi, On Tue, 2017-10-17 at 02:44 -0700, Marc MERLIN wrote: > Was broken in 4.11, still broken in 4.12. This is crippling, I'm not > running linux so that I have to reboot it to reload an intel wireless > driver :-/ Can you report a bug in https://bugzilla.kernel.org so it's easier to track this?

Re: Commit 0711d638 breaks mwifiex

2017-10-17 Thread Johannes Berg
On Tue, 2017-10-17 at 21:07 +0800, Jesse Sung wrote: > > > Not really quite sure about it yet, but that should address the > > issue? > > A very rough test by issuing reassociate in wpa_cli: > mwifiex works after reassociate - looks good Ok. Discussing this with Ilan, we realized that this was b

Re: Commit 0711d638 breaks mwifiex

2017-10-17 Thread Jesse Sung
On Tue, Oct 17, 2017 at 6:48 PM, Johannes Berg wrote: > On Tue, 2017-10-17 at 18:18 +0800, Jesse Sung wrote: > >> > Does mwifiex treat this -EALREADY as *keeping* an old connection, >> > or tearing it down entirely? >> >> From the call trace: > > Well, the call trace can't really answer that :-) >

Re: AP6335 with mainline kernel

2017-10-17 Thread Vanessa Maegima
Hi Arend, On Qui, 2017-09-21 at 12:30 -0300, Vanessa Ayumi Maegima wrote: > Hi Arend, > > On Thu, Sep 21, 2017 at 4:26 AM, Arend van Spriel > wrote: > > > > On 20-09-17 21:33, Vanessa Ayumi Maegima wrote: > > > > > > > > > Hi, > > > > > > I am trying to enable Wifi on imx7d-pico using mainli

Re: Commit 0711d638 breaks mwifiex

2017-10-17 Thread Johannes Berg
On Tue, 2017-10-17 at 18:18 +0800, Jesse Sung wrote: > > Does mwifiex treat this -EALREADY as *keeping* an old connection, > > or tearing it down entirely? > > From the call trace: Well, the call trace can't really answer that :-) Does mwifiex firmware stay connected? > 139.451318: nl80211_get

Re: Commit 0711d638 breaks mwifiex

2017-10-17 Thread Jesse Sung
Hi Johannes, On Tue, Oct 17, 2017 at 5:51 PM, Johannes Berg wrote: > Hi, > >> While working on an issue that marvell module stops connecting to AP, >> bisect reveals that the issue starts to happen from commit 0711d638, >> which uses wdev->ssid_len instead of wdev->current_bss to determine >> if

Re: [PATCH 2/2] mac80211: Add airtime accounting and scheduling to TXQs

2017-10-17 Thread Toke Høiland-Jørgensen
Johannes Berg writes: > On Tue, 2017-10-17 at 09:34 +0200, Toke Høiland-Jørgensen wrote: > >> Yeah, I did that initially. The reason I ended up squashing them is >> that >> this patch moved the per-station 'airtime' debugfs-entry that was >> previously created by ath9k into mac80211. I assumed it

Re: [PATCH 2/2] mac80211: Add airtime accounting and scheduling to TXQs

2017-10-17 Thread Johannes Berg
On Tue, 2017-10-17 at 09:34 +0200, Toke Høiland-Jørgensen wrote: > Yeah, I did that initially. The reason I ended up squashing them is > that > this patch moved the per-station 'airtime' debugfs-entry that was > previously created by ath9k into mac80211. I assumed it would create > problems if bot

Re: Commit 0711d638 breaks mwifiex

2017-10-17 Thread Johannes Berg
Hi, > While working on an issue that marvell module stops connecting to AP, > bisect reveals that the issue starts to happen from commit 0711d638, > which uses wdev->ssid_len instead of wdev->current_bss to determine > if driver's .disconnect() should be called. > > It happens because mwifiex_cfg

Re: After upgrading to 4.11.1, wifi driver refuses to load after being unloaded once.

2017-10-17 Thread Marc MERLIN
Was broken in 4.11, still broken in 4.12. This is crippling, I'm not running linux so that I have to reboot it to reload an intel wireless driver :-/ I currently have: -rw-rw-r-- 1 merlin merlin 1745176 Jun 18 2015 /lib/firmware/iwlwifi-8000C-13.ucode -rw-r--r-- 1 root root 2345768 Dec 6 2

RE: [2/2] ath10k: handle tdls peer events

2017-10-17 Thread Manikanta Pubbisetty
>This patch introduces new warnings: > >$ ath10k-check >drivers/net/wireless/ath/ath10k/wmi.c:4470:39: warning: incorrect type in >argument 2 (different base types) >drivers/net/wireless/ath/ath10k/wmi.c:4470:39:expected int [signed] >vdev_id >drivers/net/wireless/ath/ath10k/wmi.c:4470:39:g

Commit 0711d638 breaks mwifiex

2017-10-17 Thread Jesse Sung
Hi, While working on an issue that marvell module stops connecting to AP, bisect reveals that the issue starts to happen from commit 0711d638, which uses wdev->ssid_len instead of wdev->current_bss to determine if driver's .disconnect() should be called. It happens because mwifiex_cfg80211_connec

Re: [PATCH v2] ath10k: Retry pci probe on failure.

2017-10-17 Thread Kalle Valo
Ben Greear writes: > On 10/13/2017 08:50 AM, Adrian Chadd wrote: >> On 13 October 2017 at 05:41, Kalle Valo wrote: >>> gree...@candelatech.com writes: >>> From: Ben Greear This works around a problem we see when sometimes the wifi NIC does not respond the first time. This s

Re: [PATCH] wireless: qtnfmac: Convert timers to use timer_setup()

2017-10-17 Thread Sergey Matyukevich
> In preparation for unconditionally passing the struct timer_list pointer to > all timer callbacks, switch to using the new timer_setup() and from_timer() > to pass the timer pointer explicitly. > > Cc: Igor Mitsyanko > Cc: Avinash Patil > Cc: Sergey Matyukevich > Cc: Kalle Valo > Cc: Kamlesh

Re: [PATCH 2/2] mac80211: Add airtime accounting and scheduling to TXQs

2017-10-17 Thread Toke Høiland-Jørgensen
Johannes Berg writes: >> Only ath9k currently sets the AIRTIME_ACCOUNTING flag. > > I think you should actually make that a separate patch. > > In the previous patch that's not possible or it breaks things, but > this patch just adds a new feature that drivers _may_ use, they don't > have to sinc

Re: [PATCH 2/2] mac80211: Add airtime accounting and scheduling to TXQs

2017-10-17 Thread Johannes Berg
> Only ath9k currently sets the AIRTIME_ACCOUNTING flag. I think you should actually make that a separate patch. In the previous patch that's not possible or it breaks things, but this patch just adds a new feature that drivers _may_ use, they don't have to since it doesn't change the API, so yo