920388 - [B2G][Helix][WIFI][wangyuguo]Why the wifi enabled too slowly?

Status: RESOLVED WORKSFORME

(Firefox OS Graveyard :: General, defect, P2)

Description

[lecky]

User Agent: Mozilla/4.0 (compatible; MSIE 8.0; Windows NT 6.1; Trident/4.0; SLCC2; .NET CLR 2.0.50727; .NET CLR 3.5.30729; .NET CLR 3.0.30729; Media Center PC 6.0; aff-kingsoft-ciba; .NET4.0C; .NET4.0E)

Steps to reproduce:

1) Go to Settings; 
2) Select WiFi option;
3) enable the wifi toggle;
4) disalbe the wifi toggle after the device searched AP;
5) repro the step 3)



Actual results:

The time of wifi enabled need 6 seconds to 12 seconds.


Expected results:

we compared with FFOS on V1.1, the time is not exceed 3 seconds. And we compared with other device, the wifi enabled speed is almost quikly.

Comment 1

[lecky]

Form the wifi log: we found that the file of WifiWorker.js in the path of "gecko\dom\wifi" control the wifi state. In the function of manager.setWifiEnabled = function(enable, callback), there is function as following which is called in disable the wifi:
...
unloadDriver(callback);
...
That is to say, when close the wifi, the wifi manager will unload the wifi driver. It will load the wifi driver again on next time which is the main reason of cause long time to enable the wifi.

Comment 2

[Vincent Chang[:vchang][changyihsin]]

Which branches do you use for this ? We have improved Wifi startup speed on Bug 897871. But it's merged to mozilla-central only.

Comment 3

[lecky]

The branches we use is V1.1 HD.ok, we will try murge the Bug 897871 patch to our code. 
Thanks!

Comment 4

[lecky]

Hi Vincent:
   The bug of 897871 contain the file of "attachment 792134 [details] [diff] [review]", I merge this to our project, but the system can not start up. I found the file of WifiWorker.js has other modify not only about bug 897871, I want to know the modify of bug of 897871 whether rely on other modify?

the gecko we use:
gaia：8059864bd278645ab5d2598b5ed595154136c253
gecko：0265cd51b4dc64ca100353482ada2025df6f5f53
gonk-misc：ca9192d7ef6037016d4723647d86b72f7a410633

Comment 5

[lecky]

from the log, the error is following:
...
[JavaScript Error: "gNetworkManager.setWifiOperationMode is not a function""{file: "jar:file:///system/b2g/omni.ja!/components/WifiWorker.js" line: 1189}]
...

In our project, we don't have the function of gNetworkManager.setWifiOperationMode, please help me to check this function is necessarily to this bug in the patch of attachment 792134 [details] [diff] [review]?

Comment 6

[Vincent Chang[:vchang][changyihsin]]

setWifiOperationMode is mandatory for android JB 4.3. So you may not need it in ICS.

Comment 7

[lecky]

Yes, I removed the function of gNetworkManager.setWifiOperationMode, and the wifi work normally. Now the main delay of enable wifi is the delay of "DRIVER_READY_WAIT" in file of WifiWorker.js, the default delay is 2s, I short the 2s to 0.2s and test it well. How to define this value according to the device?

Comment 8

[Vincent Chang[:vchang][changyihsin]]

(In reply to lecky from comment #7)
> Yes, I removed the function of gNetworkManager.setWifiOperationMode, and the
> wifi work normally. Now the main delay of enable wifi is the delay of
> "DRIVER_READY_WAIT" in file of WifiWorker.js, the default delay is 2s, I
> short the 2s to 0.2s and test it well. How to define this value according to
> the device?

It can be controlled by Android's property "ro.moz.wifi.driverDelay".