961934 - [DSDS][MMS] Unable to switch data call from primary SIM to 2nd if user tries to download MMS attachment of 2nd SIM.

Status: VERIFIED DUPLICATE of bug 961938

(Firefox OS Graveyard :: RIL, defect)

Description

[Enpei]

Attachment: time stamp: 12:54, 1/21 tries to change sim from 1 to 2.

Unable to switch data call from primary SIM to 2nd if user tries to download MMS attachment of 2nd SIM.

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

* Reproduce Steps
1. Send a MMS to non-primary data SIM.
2. Once receive the message, press download button within it.
3. When confirmation window pops up, select okay to download the attachment.

* Expected Result
Data call will be switched to non-primary data SIM and start downloading.

* Actual Result
Data call is not switched successfully, but the message still shows downloading and have no way to stop it.

* Occurrence rate
Only the first time works, after that, no matter SIM1 to SIM 2 or SIM 2 to SIM 1, unable to switch the data call.

Comment 1

[Enpei]

Attachment: time stamp: 13:09, 1/21 tries to change sim from 2 to 1.

Comment 2

[Gene Lian (I already quit Mozilla)]

I heard of this is by far a vendor's issue that we cannot switch Data connection among SIMs.

Comment 3

[Jessica Jong [:jessica] (inactive)]

I found something weird in the logs (attachment 8362795 [details]).

There are always two 'ril.data.defaultServiceId' commands send to RIL in 2-3 seconds:
01-21 12:59:44.384 I/Gecko   (  107): -*- RadioInterfaceLayer: 'ril.data.defaultServiceId' is now 1
01-21 12:59:46.484 I/Gecko   (  107): -*- RadioInterfaceLayer: 'ril.data.defaultServiceId' is now 0
and:
01-21 13:04:12.774 I/Gecko   (  107): -*- RadioInterfaceLayer: 'ril.data.defaultServiceId' is now 1
01-21 13:04:14.484 I/Gecko   (  107): -*- RadioInterfaceLayer: 'ril.data.defaultServiceId' is now 0
So the default sim for data call was not switched.
And finally:
01-21 13:08:28.724 I/Gecko   (  107): -*- RadioInterfaceLayer: 'ril.data.defaultServiceId' is now 1
default sim for data call was switched to SIM2.

I wonder if this is related to bug 961938.

Enpei, could you help verify if this issue can be reproduced if you switch default sim for data call manually? Many thanks.

Comment 4

[Enpei]

Hi Jessica,

Switch data sim from SIM manager has no such problem, thanks.

Comment 5

[Gene Lian (I already quit Mozilla)]

(In reply to Jessica Jong [:jjong] [:jessica] from comment #3)
> I found something weird in the logs (attachment 8362795 [details]).
> 
> There are always two 'ril.data.defaultServiceId' commands send to RIL in 2-3
> seconds:
> 01-21 12:59:44.384 I/Gecko   (  107): -*- RadioInterfaceLayer:
> 'ril.data.defaultServiceId' is now 1
> 01-21 12:59:46.484 I/Gecko   (  107): -*- RadioInterfaceLayer:
> 'ril.data.defaultServiceId' is now 0
> and:
> 01-21 13:04:12.774 I/Gecko   (  107): -*- RadioInterfaceLayer:
> 'ril.data.defaultServiceId' is now 1
> 01-21 13:04:14.484 I/Gecko   (  107): -*- RadioInterfaceLayer:
> 'ril.data.defaultServiceId' is now 0
> So the default sim for data call was not switched.
> And finally:
> 01-21 13:08:28.724 I/Gecko   (  107): -*- RadioInterfaceLayer:
> 'ril.data.defaultServiceId' is now 1
> default sim for data call was switched to SIM2.
> 
> I wonder if this is related to bug 961938.

Yes, that's it. Because data connection cannot switch to the other SIM, so the confirmation prompt would keep popping up until the data connection is switched.

Comment 6

[Preeti Raghunath(:Preeti)]

ken

Please look at it from RIL perspective.

Comment 7

[Ken Chang[:kenkai]]

Jessica, I wonder if it is possible to fix 922584 before this week. If not, we should discuss with Steve to see if we can have a alternative solution.

Comment 8

[Hsin-Yi Tsai (she/her) [:hsinyi]]

(In reply to Ken Chang[:ken] from comment #7)
> Jessica, I wonder if it is possible to fix 922584 before this week. If not,
> we should discuss with Steve to see if we can have a alternative solution.

I talked with Steve yesterday. The root cause seems bug 961938 which is kinda blocked by bug 922584.
We agreed that if this bug is urgent (1.3+), then gaia could provide a workaround at this moment. However, I strongly suggest that we move bug 922584 much more faster.

Comment 9

[Hsin-Yi Tsai (she/her) [:hsinyi]]

(In reply to Hsin-Yi Tsai  [:hsinyi] from comment #8)
> (In reply to Ken Chang[:ken] from comment #7)
> > Jessica, I wonder if it is possible to fix 922584 before this week. If not,
> > we should discuss with Steve to see if we can have a alternative solution.
> 
> I talked with Steve yesterday. The root cause seems bug 961938 which is
> kinda blocked by bug 922584.
> We agreed that if this bug is urgent (1.3+), then gaia could provide a
> workaround at this moment. However, I strongly suggest that we move bug
> 922584 much more faster.

Cool, Jessica provided a very good solution (I won't say it's a workaround ;) ) for gaia. And we could move those bugs on based on her suggestion! Thanks, Jessica :D

Comment 10

[Ken Chang[:kenkai]]

(In reply to Hsin-Yi Tsai  [:hsinyi] from comment #9)
> (In reply to Hsin-Yi Tsai  [:hsinyi] from comment #8)
> > (In reply to Ken Chang[:ken] from comment #7)
> > > Jessica, I wonder if it is possible to fix 922584 before this week. If not,
> > > we should discuss with Steve to see if we can have a alternative solution.
> > 
> > I talked with Steve yesterday. The root cause seems bug 961938 which is
> > kinda blocked by bug 922584.
> > We agreed that if this bug is urgent (1.3+), then gaia could provide a
> > workaround at this moment. However, I strongly suggest that we move bug
> > 922584 much more faster.
> 
> Cool, Jessica provided a very good solution (I won't say it's a workaround
> ;) ) for gaia. And we could move those bugs on based on her suggestion!
> Thanks, Jessica :D
Thanks, Jessica and Hsinyi, by the way, who will provide a patch for this bug for Gaia?

Comment 11

[Jessica Jong [:jessica] (inactive)]

(In reply to Ken Chang[:ken] from comment #10)
> (In reply to Hsin-Yi Tsai  [:hsinyi] from comment #9)
> > (In reply to Hsin-Yi Tsai  [:hsinyi] from comment #8)
> > > (In reply to Ken Chang[:ken] from comment #7)
> > > > Jessica, I wonder if it is possible to fix 922584 before this week. If not,
> > > > we should discuss with Steve to see if we can have a alternative solution.
> > > 
> > > I talked with Steve yesterday. The root cause seems bug 961938 which is
> > > kinda blocked by bug 922584.
> > > We agreed that if this bug is urgent (1.3+), then gaia could provide a
> > > workaround at this moment. However, I strongly suggest that we move bug
> > > 922584 much more faster.
> > 
> > Cool, Jessica provided a very good solution (I won't say it's a workaround
> > ;) ) for gaia. And we could move those bugs on based on her suggestion!
> > Thanks, Jessica :D
> Thanks, Jessica and Hsinyi, by the way, who will provide a patch for this
> bug for Gaia?

I think Steve is working on this, see bug 961938, which seems to be the root cause of this bug.

Comment 12

[Ken Chang[:kenkai]]

Steve, this bug seems not a blocker for bug 961938. should we set this bug as a duplicate of bug 961938?

Comment 13

[Steve Chung [:steveck]]

Attachment: dsds.patch

Hi Jessica, here is the WIP patch which should trigger the mms retrieval at the correct timing. Please verify the issue again after patch applied, thanks.

Comment 14

[Steve Chung [:steveck]]

I updated the patch in attachment 8365825 [details] [review] and Enpei is testing for it. We can verify it when patch landed, but I think we can duplicate this one with her positive feedback first.

Comment 15

[Kai-Chih Hu [:khu]]

(In reply to Steve Chung [:steveck] from comment #13)
> Created attachment 8365826 [details] [diff] [review]
> dsds.patch
> 
> Hi Jessica, here is the WIP patch which should trigger the mms retrieval at
> the correct timing. Please verify the issue again after patch applied,
> thanks.

Jessica, can you help to provide feedback?

Comment 16

[Jessica Jong [:jessica] (inactive)]

Sorry for not giving feedback yet. Enpei is helping us testing the patch provided by Steve, we found some issues while doing the tests and wanted to make sure it is not related to the original issue. Will sync with her today.

Comment 17

[Enpei]

During the test I met bug 965107 but the occurrence rate is not 100%. I assume that it's not related to this bug but need to be tracked after this bug is landed on 1.3. So I open that bug first for tracking. 

But if anyone has concern, feel free to comment it.

Comment 18

[Jessica Jong [:jessica] (inactive)]

Comment on attachment 8365826 [details] [diff] [review]
dsds.patch

From Enpei's comment 17, the original issue can be solved with this patch (or the latest patch in bug 961938). So I think we can mark this as duplicated. Thank you.

Comment 19

[Wesley Huang [:wesley_huang] (EPM) (NI me)]

Set as duplicate per comment 14, comment 17, and comment 18

Comment 20

[Enpei]

Verified.

Fugu - 
Gaia      bb36b4164d3e51ca04b612e507e1178f80bf240d
Gecko     4227240a4d3e1e418ced160c2a48cb454d7b5645
BuildID   20140204104742
Version   28.0