Re: [mac80211] mac80211: Support multiple VIFS per AP in debugfs.

[Ben Greear <greearb@candelatech.com>]

On 09/21/2010 10:49 AM, Johannes Berg wrote:
> On Tue, 2010-09-21 at 19:03 +0200, Johannes Berg wrote:
>
>>>>>> -	snprintf(mac, sizeof(mac), "%pM", sta->sta.addr);
>>>>>> +	snprintf(mac, sizeof(mac), "%pM-%s",
>>>>>> +		 sta->sta.addr, sta->sdata->name);
>>>>>>
>>>>>>     	/*
>>>>>>     	 * This might fail due to a race condition:
>>>>>
>>>>> I got to thinking:  Is there any reason to put the sta.addr in there?
>>>>>
>>>>> Maybe just use sta->sdata->name?  Seems it might be easier for
>>>>> scripts to deal with?
>>>>
>>>> APs have many STAs
>>>
>>> So, when in STA mode, sta_info are related to local VIFs, but
>>> in AP mode, they represent the remote wireless devices, with sdata pointing
>>> to the local AP interface?
>>
>> No, there's a sta_info for each peer, which in STA mode is only the AP.
>
> maybe the stations should be moved from phy0/stations/ to
> phy0/wlan0/stations/ instead?

It doesn't matter either way to me, but I'd like to get one or another
patch upstream.

If you want me to change it to put stations under netdev:wlan0/ instead,
please just let me know.

Thanks,
Ben

>
> johannes


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