Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1491776

Re: [PATCH] brcmfmac: implement more accurate skb tracking

Show all headers | View raw

On 26-9-2016 14:38, Rafał Miłecki wrote:
> On 26 September 2016 at 14:13, Rafał Miłecki <zajec5@gmail.com> wrote:
>> On 26 September 2016 at 13:46, Arend Van Spriel
>> <arend.vanspriel@broadcom.com> wrote:
>>> On 26-9-2016 12:23, Rafał Miłecki wrote:
>>>> From: Rafał Miłecki <rafal@milecki.pl>
>>>>
>>>> We need to track 802.1x packets to know if there are any pending ones
>>>> for transmission. This is required for performing key update in the
>>>> firmware.
>>>
>>> The problem we are trying to solve is a pretty old one. The problem is
>>> that wpa_supplicant uses two separate code paths: EAPOL messaging
>>> through data path and key configuration though nl80211.
>>
>> Can I find it described/reported somewhere?
>>
>>
>>>> Unfortunately our old tracking code wasn't very accurate. It was
>>>> treating skb as pending as soon as it was passed by the netif. Actual
>>>> handling packet to the firmware was happening later as brcmfmac
>>>> internally queues them and uses its own worker(s).
>>>
>>> That does not seem right. As soon as we get a 1x packet we need to wait
>>> with key configuration regardless whether it is still in the driver or
>>> handed over to firmware already.
>>
>> OK, thanks.
> 
> Actually, it's not OK. I was trying to report/describe/discuss this
> problem for over a week. I couldn't get much of answer from you.
> 
> I had to come with a patch I worked on for quite some time. Only then
> you decided to react and reply with a reason for a nack. I see this
> patch may be wrong (but it's still hard to know what's going wrong
> without a proper hostapd bug report). I'd expect you to somehow work &
> communicate with open source community.

We do or at least make an honest attempt, but there is more on our plate
so responses may be delayed. It also does not help when you get anal and
preachy when we do respond. Also not OK. In this case the delay is
caused because I had to pick up the thread(s) as Hante is on vacation
(he needed a break :-p ). However, you started sending patches so I
decided to look at and respond to those. Sorry if you felt like we left
you hanging to dry.

Regards,
Arend

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH] brcmfmac: implement more accurate skb tracking Rafał Miłecki <zajec5@gmail.com> - 2016-09-26 12:30 +0200
  Re: [PATCH] brcmfmac: implement more accurate skb tracking Arend Van Spriel <arend.vanspriel@broadcom.com> - 2016-09-26 13:50 +0200
    Re: [PATCH] brcmfmac: implement more accurate skb tracking Rafał Miłecki <zajec5@gmail.com> - 2016-09-26 14:20 +0200
      Re: [PATCH] brcmfmac: implement more accurate skb tracking Arend Van Spriel <arend.vanspriel@broadcom.com> - 2016-09-26 14:30 +0200
      Re: [PATCH] brcmfmac: implement more accurate skb tracking Rafał Miłecki <zajec5@gmail.com> - 2016-09-26 14:40 +0200
        Re: [PATCH] brcmfmac: implement more accurate skb tracking Arend Van Spriel <arend.vanspriel@broadcom.com> - 2016-09-27 11:30 +0200
          Re: [PATCH] brcmfmac: implement more accurate skb tracking Rafał Miłecki <zajec5@gmail.com> - 2016-09-30 00:00 +0200
            Re: [PATCH] brcmfmac: implement more accurate skb tracking Arend Van Spriel <arend.vanspriel@broadcom.com> - 2016-09-30 10:40 +0200
      Re: [PATCH] brcmfmac: implement more accurate skb tracking Dan Williams <dcbw@redhat.com> - 2016-09-26 17:00 +0200
        Re: [PATCH] brcmfmac: implement more accurate skb tracking Arend Van Spriel <arend.vanspriel@broadcom.com> - 2016-09-27 11:10 +0200

csiph-web