Closed Bug 1158024 Opened 9 years ago Closed 6 years ago

Create Preference value to Pre-cache Secure Element rules

Categories

(Firefox OS Graveyard :: NFC, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: dgarnerlee, Unassigned)

References

Details

Attachments

(1 file)

From Comment:
https://bugzilla.mozilla.org/show_bug.cgi?id=884594#c118

The Secure Element Access Control Enforcer pre-caches rules when the ACEService is first created by the SE. This may not be desirable if this happens to occur during phone startup, and startup time is extremely important (every millisecond counts).

It can be removed, and have the first SE DOM use pull in the access rules, and cache the rules then, or a device/system specific preference value can control it.
Depends on: 884594
Attached patch patchSplinter Review
This patch adds a flag to load the ace rules early (or not), based on a gecko preference.
Attachment #8608438 - Flags: feedback?(allstars.chh)
Comment on attachment 8608438 [details] [diff] [review]
patch

Review of attachment 8608438 [details] [diff] [review]:
-----------------------------------------------------------------

This still not seems to be a priority to me.
As there are still lots of SE bugs in the Bugzilla, and even the wallet app is not landed yet.
This should be a 2nd/3rd priority.
With considering FxOS phone should be a low-mid tie phone, it's still a question to me whether we should do this or not,

And the other question is, is the fetching rules fast enough ?
Are we pretty sure that all the code of fetching rules running on critical path and there's no more optimization could be done?

Also even we should do the pre-cache, there should be several ways to determine how to 'pre-cache' the rules,
do it in boot-time, or when wallet apps launches,
it could also do when the first time NFC is turn on,
with considering other types of SE(eSE, ASSD), 
this should be a more case-by-case discussion, instead of "just flipping the flag".

So I don't think we should spend time on this for now as this is still pre-mature.
There should be other bugs with more priority should be fixed.
Attachment #8608438 - Flags: feedback?(allstars.chh)
Firefox OS is not being worked on
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: