961918 - [DSDS] When airplane mode is on, reboot device, "SIM 2 searching..." will be displayed on lockscreen for about 5 seconds.

Status: VERIFIED WORKSFORME

(Firefox OS Graveyard :: RIL, defect)

Description

[Enpei]

Attachment: time stamp: 11:03, 1/21

When airplane mode is on, reboot device, "SIM 2 searching..." will be displayed on lockscreen for about 5 seconds then gone. Suppose no such wording should be seen when airplane mode is on.

* Build Number  
Fugu
Gaia      ee25b0e45649d2955f340ce4f71ad55712dd0fab
Gecko     913cf2b92845441c9578787362ddad6f2ac15df6
BuildID   20140121095108
Version   28.0a2

* Reproduce Steps
1. enable airplane mode.
2. Reboot device.
3. check lockscreen.

* Expected Result
It will only show "Airplane" on top.

* Actual Result
"SIM 2 searching..." will be displayed on lockscreen for about 5 seconds.

* Occurrence rate
100%

Comment 1

[Enpei]

Attachment: VID_0001.3gp

Comment 2

[Edgar Chen [:edgar]]

In bug 952386, we have some improvement for the radio control, I have quick tested the patch, it can fix this issue as well.

Comment 3

[shawn ku [:sku]]

Hi Aknow:
 Could you please help check this issue?
It looks like below [e-mail] will trun on radio during boot-up that cause this issue.

    /**
     * nsIObserver interface methods.
     */
    observe: function observe(subject, topic, data) {
      switch (topic) {
        case kSysMsgListenerReadyObserverTopic:
          Services.obs.removeObserver(this, kSysMsgListenerReadyObserverTopic);

          let numCards = this._getNumCards();
          for (let i = 0, N = this.ril.numRadioInterfaces; i < N; ++i) {
            if (this._isRadioAbleToEnableAtClient(i, numCards)) {
              let radioInterface = this.ril.getRadioInterface(i);
              radioInterface.setRadioEnabledInternal({enabled: true}, null);
            }
          }
          break;
        default:
          break;
      }
    }
  };

01-21 14:42:43.980   107   227 I Gecko   : RIL Worker[0]: Radio state changed from 'null' to 'null'
01-21 14:42:44.010   107   229 I Gecko   : RIL Worker[1]: Radio state changed from 'null' to 'null'
01-21 14:42:44.050   107   227 I Gecko   : RIL Worker[0]: Radio state changed from 'null' to 'off'
01-21 14:42:44.060   107   229 I Gecko   : RIL Worker[1]: Radio state changed from 'null' to 'off'

01-21 14:42:48.690   107   107 I Gecko   : -*- RadioInterfaceLayer: observe setRadioEnabledInternal true
01-21 14:42:48.690   107   107 I Gecko   : -*- RadioInterfaceLayer: setRadioEnabledInternal {"enabled":true}

01-21 14:42:55.020   107   227 I Gecko   : RIL Worker[0]: Radio state changed from 'off' to 'ready'
01-21 14:42:55.200   107   227 I Gecko   : RIL Worker[0]: Radio state changed from 'ready' to 'ready'
01-21 14:42:56.550   107   227 I Gecko   : RIL Worker[0]: Radio state changed from 'ready' to 'off'
01-21 14:43:04.220   107   229 I Gecko   : RIL Worker[1]: Radio state changed from 'off' to 'ready'
01-21 14:43:04.360   107   229 I Gecko   : RIL Worker[1]: Radio state changed from 'ready' to 'ready'
01-21 14:43:04.860   107   229 I Gecko   : RIL Worker[1]: Radio state changed from 'ready' to 'off'

Comment 4

[Edgar Chen [:edgar]]

(In reply to shawn ku [:sku] from comment #3)
> Hi Aknow:
>  Could you please help check this issue?
> It looks like below [e-mail] will trun on radio during
> boot-up that cause this issue.

Yes, in bug 952386, we have some improvements for radio control. it can fix this issue as well.
Thanks for your comments here. :)

> 
>     /**
>      * nsIObserver interface methods.
>      */
>     observe: function observe(subject, topic, data) {
>       switch (topic) {
>         case kSysMsgListenerReadyObserverTopic:
>           Services.obs.removeObserver(this,
> kSysMsgListenerReadyObserverTopic);
> 
>           let numCards = this._getNumCards();
>           for (let i = 0, N = this.ril.numRadioInterfaces; i < N; ++i) {
>             if (this._isRadioAbleToEnableAtClient(i, numCards)) {
>               let radioInterface = this.ril.getRadioInterface(i);
>               radioInterface.setRadioEnabledInternal({enabled: true}, null);
>             }
>           }
>           break;
>         default:
>           break;
>       }
>     }
>   };
> 
> 01-21 14:42:43.980   107   227 I Gecko   : RIL Worker[0]: Radio state
> changed from 'null' to 'null'
> 01-21 14:42:44.010   107   229 I Gecko   : RIL Worker[1]: Radio state
> changed from 'null' to 'null'
> 01-21 14:42:44.050   107   227 I Gecko   : RIL Worker[0]: Radio state
> changed from 'null' to 'off'
> 01-21 14:42:44.060   107   229 I Gecko   : RIL Worker[1]: Radio state
> changed from 'null' to 'off'
> 
> 01-21 14:42:48.690   107   107 I Gecko   : -*- RadioInterfaceLayer: observe
> setRadioEnabledInternal true
> 01-21 14:42:48.690   107   107 I Gecko   : -*- RadioInterfaceLayer:
> setRadioEnabledInternal {"enabled":true}
> 
> 01-21 14:42:55.020   107   227 I Gecko   : RIL Worker[0]: Radio state
> changed from 'off' to 'ready'
> 01-21 14:42:55.200   107   227 I Gecko   : RIL Worker[0]: Radio state
> changed from 'ready' to 'ready'
> 01-21 14:42:56.550   107   227 I Gecko   : RIL Worker[0]: Radio state
> changed from 'ready' to 'off'
> 01-21 14:43:04.220   107   229 I Gecko   : RIL Worker[1]: Radio state
> changed from 'off' to 'ready'
> 01-21 14:43:04.360   107   229 I Gecko   : RIL Worker[1]: Radio state
> changed from 'ready' to 'ready'
> 01-21 14:43:04.860   107   229 I Gecko   : RIL Worker[1]: Radio state
> changed from 'ready' to 'off'

Comment 5

[Edgar Chen [:edgar]]

In bug 952386, we have some improvements for radio power initialization. It can fix this issue as well. So mark this bug as a duplicate of bug 952386.

Comment 6

[Enpei]

Verified on Fugu.

Gaia      6fbeac2415f07f10de181f0877ddf67ee299b885
Gecko     086e69971516130ad297c7f950dfb42a995217e9
BuildID   20140123115514
Version   28.0a2