Open
Bug 911582
Opened 11 years ago
Updated 2 years ago
cookies xpcshell tests (test_cookies_async_failure.js and test_cookies_read.js) perform a lot of write I/O
Categories
(Core :: Networking: Cookies, defect, P3)
Core
Networking: Cookies
Tracking
()
NEW
People
(Reporter: gps, Unassigned)
References
(Blocks 2 open bugs)
Details
(Whiteboard: [buildfaster:4][necko-backlog])
Awesome intern Mihnea hooked resource monitoring up to xpcshell tests to allow us to get per-test metrics for CPU, I/O, and memory. http://www.mihneadb.net/xpcshelltest-resource-usage/. As you can see, test_cookies_async_failure.js incurs 1.4 GB of write I/O (wchar)! Although - and this is somewhat strange - only ~11 MB are being written to disk (at least while the process is alive). The ~11 MB isn't that bad. But the 1.4 GB of raw write I/O is drastically more than any other xpcshell test (see graph). I suspect the test is doing something silly. (Please note that wchar encompasses all writes. This includes writing to the terminal and pipes) That, or the underlying functionality being tested is grossly inefficienct in some way (and might warrant perf investigation). test_cookies_read.js also incurs a lot of I/O - ~370 MB wchar. Now that xpcshell tests are running in parallel, tests are I/O bound on release infrastructure. So any work to decrease I/O in these tests will likely make tests run faster and increase automation capacity. (And yes, we should be running tests on an SSD or ramdisk, but that's another discussion.)
Updated•8 years ago
|
Whiteboard: [buildfaster:4] → [buildfaster:4][necko-backlog]
Comment 1•7 years ago
|
||
Junior, was this handled by your patch in bug 1370454?
Flags: needinfo?(juhsu)
Comment 2•7 years ago
|
||
(In reply to Shian-Yow Wu [:swu] from comment #1) > Junior, was this handled by your patch in bug 1370454? It's more or less handled. We comprise thousands transactions into one. It could reduce the amount of IO write and number of operations. However, test_cookies_async_failure.js requires lots of IO by design. I have no idea how much IO would be written after bug 1370454. FWIW, test_cookies_read.js should use Services.cookies.runInTransaction, too.
Flags: needinfo?(juhsu)
Comment 3•7 years ago
|
||
Bulk change to priority: https://bugzilla.mozilla.org/show_bug.cgi?id=1399258
Priority: -- → P1
Comment 4•7 years ago
|
||
Bulk change to priority: https://bugzilla.mozilla.org/show_bug.cgi?id=1399258
Priority: P1 → P3
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•