[emulator-l] [emulator-x86-l] reconsider the number of rild enabled

RESOLVED WONTFIX

Status

RESOLVED WONTFIX
3 years ago
3 years ago

People

(Reporter: hsinyi, Assigned: edgar)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(3 attachments)

(Reporter)

Description

3 years ago
We enabled 9 rild on emu-kk and emu-x86-kk. We enabled that many because at that time we heard that there could be a 6-sim device requirement. However, since then we don't see that happened.

I'm filing this bug to see if it's worth keeping 9 rild or it makes more sense to maintain just 2 rild for now.
(Assignee)

Comment 1

3 years ago
I don't have strong opinions on how many sim should emulator-kk support. However maintaining just 2 rild maybe could make emulator-kk run faster :p.
(Reporter)

Comment 2

3 years ago
Oh yes, I should have said more about why I raised this question ;)
1) apparently performance issue: I noticed even on emu-x86-kk that Callscreen transition is not smooth at all due to the need to check the status of 9 sims ... it implies that there's room for improvement in our gecko code but still wondering what if we simply reduce the unnecessary rild number .
2) still have a dream that emulator could be used more widely on gaia-ui tests. To achieve that, we need to have an emulator configured as our gaia has been aware
(Assignee)

Comment 3

3 years ago
(In reply to Hsin-Yi Tsai [:hsinyi] from comment #2)
> Oh yes, I should have said more about why I raised this question ;)
> 1) apparently performance issue: I noticed even on emu-x86-kk that
> Callscreen transition is not smooth at all due to the need to check the
> status of 9 sims ... it implies that there's room for improvement in our
> gecko code but still wondering what if we simply reduce the unnecessary rild
> number .
> 2) still have a dream that emulator could be used more widely on gaia-ui
> tests. To achieve that, we need to have an emulator configured as our gaia
> has been aware

Let's do this in emulator-l :)
Summary: [emu-kk] [emu-x86-kk] reconsider the number of rild enabled → [emulator-l] [emulator-x86-l] reconsider the number of rild enabled
(Assignee)

Updated

3 years ago
Blocks: 1177234
(Assignee)

Updated

3 years ago
Assignee: nobody → echen
(Assignee)

Comment 4

3 years ago
Created attachment 8640967 [details] [review]
[device_generic_goldfish:b2g-5.1.0_r1] PR #32, Maintain only 2 rild/rilproxy on emulator-l
(Assignee)

Comment 5

3 years ago
Created attachment 8641007 [details] [review]
[platform_build:b2g-5.1.0_r1] PR#124, Enable only 2 RIL services on emulator-l
(Assignee)

Comment 6

3 years ago
Created attachment 8641484 [details] [review]
[platform_external_qemu:b2g-studio-1.3] PR #155, Setup MAX_GSM_DEVICES from build configuration
(Assignee)

Comment 7

3 years ago
Comment on attachment 8641484 [details] [review]
[platform_external_qemu:b2g-studio-1.3] PR #155, Setup MAX_GSM_DEVICES from build configuration

This pull request changes to setup MAX_GSM_DEVICES from build configuration, `EMULATOR_MULTI_SIM`. In this way we could support both single-sim (still have dream about use same branch for emulator-ics :p) or multi-sim with different number of sim slot without modifying the code.

Hi Hsinyi, may I have your review? Thank you.
Attachment #8641484 - Flags: review?(htsai)
(Assignee)

Comment 8

3 years ago
Comment on attachment 8641007 [details] [review]
[platform_build:b2g-5.1.0_r1] PR#124, Enable only 2 RIL services on emulator-l

Configure EMULATOR_MULTI_SIM to 2 on emulator-l.
Attachment #8641007 - Flags: review?(htsai)
(Assignee)

Updated

3 years ago
Attachment #8640967 - Flags: review?(htsai)
(Assignee)

Updated

3 years ago
Duplicate of this bug: 1186889
(Assignee)

Comment 10

3 years ago
Comment on attachment 8640967 [details] [review]
[device_generic_goldfish:b2g-5.1.0_r1] PR #32, Maintain only 2 rild/rilproxy on emulator-l

Cancel the review request given that we are no longer working on emulator-l. Thank you.
Attachment #8640967 - Flags: review?(htsai)
(Assignee)

Updated

3 years ago
Attachment #8641007 - Flags: review?(htsai)
(Assignee)

Updated

3 years ago
Attachment #8641484 - Flags: review?(htsai)
(Reporter)

Updated

3 years ago
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.