Search Linux Wireless

Re: hostapd with mac80211 progress

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, 2008-03-03 at 15:08 +0800, Zhu Yi wrote:
> On Fri, 2008-02-29 at 17:25 +0100, Johannes Berg wrote:
> > now that John merged more stuff you should only need the hostapd
> > patch and the allow-ap-vlan-modes patch from my kernel series :)
> 
> OK. This time is much better. Now I can at least ping my 4965 AP with
> below fix.
> 
> The removal of mac80211 IEEE80211_CHAN_W_SCAN flag causes channel flag
> incompatibility between hostapd and driver. Hostapd expects
> HOSTAPD_CHAN_W_SCAN (0x1) to be set for the configured channel while 0x1
> means IEEE80211_CHAN_DISABLED in the new mac80211. We should make
> ieee80211_channel_flags and HOSTAPD_CHAN_W_xxx compatible. What do you
> think? I now just workaround the problem with:
> 
> diff --git a/hostapd/hw_features.c b/hostapd/hw_features.c
> index c58e1e4..8a7b013 100644
> --- a/hostapd/hw_features.c
> +++ b/hostapd/hw_features.c
> @@ -294,8 +294,7 @@ static int select_hw_mode1(struct hostapd_iface
> *iface)
>  	for (j = 0; j < iface->current_mode->num_channels; j++) {
>  		struct hostapd_channel_data *chan =
>  			&iface->current_mode->channels[j];
> -		if ((chan->flag & HOSTAPD_CHAN_W_SCAN) &&
> -		    (chan->chan == iface->conf->channel)) {
> +		if (chan->chan == iface->conf->channel) {
>  			ok = 1;
>  			break;
>  		}

That's rather odd. ieee80211_channel_flags and HOSTAPD_CHAN_W_xx need
not be compatible because in driver_nl80211.c, phy_info_handler(),
they're translated into each other. What exactly is your setup?

johannes

Attachment: signature.asc
Description: This is a digitally signed message part


[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]
  Powered by Linux