Modify

Opened 6 years ago

Closed 5 years ago

Last modified 4 years ago

#11914 closed defect (fixed)

Wifi Master and Client simultaneous no longer working

Reported by: Lars Bøgild Thomsen <lth@…> Owned by: blogic
Priority: normal Milestone: Barrier Breaker 14.07
Component: base system Version: Trunk
Keywords: Cc:

Description

I have been running a TP-Link WR2543ND on OpenWRT Trunk for a while. The wifi is configured running as client and master simultaneous on two different SSID's and this has been working fine until the latest upgrade. Now both interfaces are running with the SSID of the master and the client therefor does not connect.

I am not 100 % certain when this change happened, but I reckon within the past week or.

I did browse through the logs bug found nothing that could hint the reason.

Attachments (1)

bug.png (29.1 KB) - added by Lars Bøgild Thomsen <lth@…> 6 years ago.
Show the setup when it was working (trunk@32651)

Download all attachments as: .zip

Change History (14)

Changed 6 years ago by Lars Bøgild Thomsen <lth@…>

Show the setup when it was working (trunk@32651)

comment:1 Changed 6 years ago by Lars Bøgild Thomsen <lth@…>

As can be seen from attachment, the setup described was working in trunk@32651.

In trunk@32832 both networks switch to using the "ncp" ssid.

comment:2 Changed 6 years ago by anonymous

i got it working removing all wifi profiles, adding the master mode first and then the client mode ( be sure to uncheck the box to create an aditional network, or the previous configuration will be overwritten ).

best regards.

comment:3 Changed 6 years ago by Lars Bøgild Thomsen <lth@…>

Just tried a new image of trunk@32885 and still no go. Tried removing all and adding them in different order - including the one you describe - no go. They will both run with the SSID of the master.

As mentioned - using an image of trunk@32651 and the setup works fine.

comment:4 Changed 6 years ago by Lars Bøgild Thomsen <lth@…>

I can now narrow this down a bit. This problem has happened between trunk@32651 (working) and trunk@32775 (not working)

I suspect this might be related to changes made in commit 32760, but I have yet to verify that.

comment:5 Changed 6 years ago by Lars Bøgild Thomsen <lth@…>

Hmmm, cancel that last sentence :) 32760 worked. I will do a git bisect on this to establish what exactly went wrong.

comment:6 Changed 6 years ago by Lars Bøgild Thomsen <lth@…>

Ok - I have gone through a git bisect and narrowed it down to the first bad commit being trunk@32829.

lth@ncpws04:~/src/openwrt$ git bisect bad
ae557097050d5d5e3f53afb5a577ab87b068557f is the first bad commit
commit ae557097050d5d5e3f53afb5a577ab87b068557f
Author: blogic <blogic@3c298f89-4303-0410-b956-a3cf2f4a3e73>
Date: Tue Jul 24 23:07:15 2012 +0000

wpa_supplicant: add more parameters


Signed-off-by: Antonio Quartulli <ordex@…>


git-svn-id: svn://svn.openwrt.org/openwrt/trunk@32829 3c298f89-4303-0410-b956-a3cf2f4a3e73

comment:7 Changed 6 years ago by Lars Bøgild Thomsen <lth@…>

Fixed! The following tiny change fixes this problem (on trunk@32885):

--- a/package/hostapd/files/wpa_supplicant.sh
+++ b/package/hostapd/files/wpa_supplicant.sh
@@ -127,6 +127,8 @@ wpa_supplicant_setup_vif() {

local fixed_freq bssid1 beacon_interval brates mrate
config_get ifname "$vif" ifname
config_get bridge "$vif" bridge

+ config_get ssid "$vif" ssid
+ config_get bssid "$vif" bssid

bssid1=${bssid:+"bssid=$bssid"}
beacon_interval=${beacon_int:+"beacon_interval=$beacon_int"}

Those two lines disappeared in before mentioned commit. Nothing else needed change to make this work again.

comment:8 Changed 6 years ago by Lars Bøgild Thomsen <lth@…>

Sorry - forgot to quote that - here we go again:

--- a/package/hostapd/files/wpa_supplicant.sh
+++ b/package/hostapd/files/wpa_supplicant.sh
@@ -127,6 +127,8 @@ wpa_supplicant_setup_vif() {
        local fixed_freq bssid1 beacon_interval brates mrate
        config_get ifname "$vif" ifname
        config_get bridge "$vif" bridge
+       config_get ssid "$vif" ssid
+       config_get bssid "$vif" bssid
        bssid1=${bssid:+"bssid=$bssid"}
        beacon_interval=${beacon_int:+"beacon_interval=$beacon_int"}

comment:9 Changed 6 years ago by eleon216

I can reproduce this.
Had the same problem on a TL-WR741ND v4.21 running trunk@r23900
I wasn't able to apply the patch, but I added the two lines myself, and it's working now.
I guess some dev should look into why this 2 lines were removed.

comment:10 Changed 6 years ago by Lars Bøgild Thomsen <lth@…>

eleon216: Yes and definitely a developer should look into if readding those two lines could potentially cause any other problems. I just compared before and after and adding those two lines did work for me.

comment:11 Changed 6 years ago by jow

  • Owner changed from developers to blogic
  • Status changed from new to assigned

comment:12 Changed 5 years ago by jow

  • Resolution set to fixed
  • Status changed from assigned to closed

Offending change partly reverted with r33053, a more elaborate code rework will follow later.

comment:13 Changed 4 years ago by jow

  • Milestone changed from Attitude Adjustment 12.09 to Barrier Breaker 14.07

Milestone Attitude Adjustment 12.09 deleted

Add Comment

Modify Ticket

Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.