Intermittent mainthreadio | File '{profile}\revocations.txt' was accessed and we were not expecting it: {'Count': 1, 'Duration': 0.031488, 'RunCount': 1}

RESOLVED INVALID

Status

Testing
Talos
RESOLVED INVALID
3 years ago
3 years ago

People

(Reporter: mgoodwin, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

3 years ago
A file was added as part of OneCRL; it's causing a test failure. Not sure of the correct course of action...
(Reporter)

Comment 1

3 years ago
Created attachment 8530250 [details] [diff] [review]
bug_1106021-add-revocations-to-profile.patch

I have no idea if this is the correct course of action; my apologies if not.
Attachment #8530250 - Flags: review?(jmaher)
(Reporter)

Comment 2

3 years ago
After digging around, It seems I can't test this since it requires xperf (which requires windows).

For more info, here's a sample failure: https://treeherder.mozilla.org/ui/logviewer.html#?job_id=4279101&repo=mozilla-inbound
If we determine that there is we need to have this done on the mainthread, then this patch will be the appropriate course of action.  I would like aklotz or vladan to chime in here.  In many cases this IO is not necessary and can be done async or coded slightly differently to get it out of the path.
Flags: needinfo?(aklotz)
Attachment #8530250 - Flags: review?(jmaher)
(Reporter)

Comment 4

3 years ago
(In reply to Joel Maher (:jmaher) from comment #3)
> In many cases this IO is not necessary
> and can be done async or coded slightly differently to get it out of the
> path.

I made changes to avoid the issue; this now happens the first time we attempt to verify a cert.
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → INVALID
Flags: needinfo?(aklotz)
You need to log in before you can comment on or make changes to this bug.