Closed Bug 1580112 Opened 3 months ago Closed 3 months ago

Short looping video produces frequent disk activity

Categories

(Core :: Audio/Video: Playback, defect, P2)

70 Branch
defect

Tracking

()

RESOLVED FIXED
mozilla71
Tracking Status
firefox-esr60 --- unaffected
firefox-esr68 --- unaffected
firefox69 --- unaffected
firefox70 --- fixed
firefox71 --- fixed

People

(Reporter: ke5trel, Assigned: achronop)

References

(Regression)

Details

(Keywords: perf, regression, Whiteboard: [fxperf])

Attachments

(1 file)

STR:

  1. Display a short looping video (eg https://imgur.com/a/WJfq5n6).

Expected:

No disk activity, silent HDD.

Actual:

Disk activity occurs every time the video loops. For HDDs this produces an audible ticking sound, indicating that writes are not being buffered. The prolonged nature of these frequent writes may cause unnecessary wear to storage devices.

Regression window:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=adc36d5fd87a2476edb63435c3ff9d19e731262c&tochange=eed72505cf9b3c968aced5a6488f2660e1f02da4

Regressed by Bug 1530996.

Assignee: nobody → achronop
Priority: -- → P2

In most of the cases, the value to be stored is the same as the existing value, which does not require an update to the database. Setting something in the database requires disk access. By using this we avoid accessing the disk in the majority of the cases.

Pushed by achronopoulos@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2e6f8b858af7
Stop updating the database when the value has not changed. r=alwu
Status: NEW → RESOLVED
Closed: 3 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla71

Please nominate this for Beta uplift when you get a chance.

Flags: needinfo?(achronop)

Comment on attachment 9092027 [details]
Bug 1580112 - Stop updating the database when the value has not changed. r?alwu

Beta/Release Uplift Approval Request

  • User impact if declined: Frequent disk activity when a short looping video plays on the loop
  • Is this code covered by automated tests?: No
  • Has the fix been verified in Nightly?: Yes
  • Needs manual test from QE?: No
  • If yes, steps to reproduce:
  • List of other uplifts needed: None
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): No risky, very small fix that improves the performance when a video is being played on a loop.
  • String changes made/needed: N/A
Flags: needinfo?(achronop)
Attachment #9092027 - Flags: approval-mozilla-beta?

Comment on attachment 9092027 [details]
Bug 1580112 - Stop updating the database when the value has not changed. r?alwu

Avoids doing useless work, approved for 70.0b8.

Attachment #9092027 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
You need to log in before you can comment on or make changes to this bug.