Subject: Re: [PATCH 1/2] wireless: Support auth-at-ms timer in sta-info
To:     Johannes Berg <johannes@sipsolutions.net>,
        linux-wireless@vger.kernel.org
References: <20190412213710.17292-1-greearb@candelatech.com>
 <cf95c43bcfe1bd4d8fbc8aca8ecc65dac9eeae0c.camel@sipsolutions.net>
From:   Ben Greear <greearb@candelatech.com>
Organization: Candela Technologies
Message-ID: <a8865096-f19e-608f-13c2-08036876d316@candelatech.com>
Date:   Mon, 15 Apr 2019 09:32:28 -0700
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101
 Thunderbird/60.4.0
MIME-Version: 1.0
In-Reply-To: <cf95c43bcfe1bd4d8fbc8aca8ecc65dac9eeae0c.camel@sipsolutions.net>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Sender: linux-wireless-owner@vger.kernel.org
Precedence: bulk

On 4/15/19 4:15 AM, Johannes Berg wrote:
> On Fri, 2019-04-12 at 14:37 -0700, greearb@candelatech.com wrote:
>> From: Ben Greear <greearb@candelatech.com>
>>
>> Report time stamp of when sta became authenticated.
> 
> You didn't actually implement this ;-)
> 
> Did you mean associated?
> 
>> + * @auth_at_ms: time in ms of the last assoc -> auth transition.
> 
> There's commonly not really such a transition?

Oh, I guess AUTHORIZED is only when doing .1x authentication?  My test case
was (luckly?) doing that, so I didn't notice the bug.

I'll change to setting the timer in the ASSOC state instead.

Thanks,
Ben

-- 
Ben Greear <greearb@candelatech.com>
Candela Technologies Inc  http://www.candelatech.com