Closed Bug 1839615 Opened 1 year ago Closed 1 year ago

[Linux/VA-API] libva info log appears in terminal every time a video plays without logging enabled

Categories

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

Firefox 116
Unspecified
Linux
defect

Tracking

()

RESOLVED FIXED
118 Branch
Tracking Status
firefox-esr102 --- wontfix
firefox-esr115 --- wontfix
firefox114 --- wontfix
firefox115 --- wontfix
firefox116 --- wontfix
firefox117 --- wontfix
firefox118 --- fixed

People

(Reporter: ke5trel, Assigned: stransky)

References

(Blocks 1 open bug, Regression)

Details

(Keywords: regression)

Attachments

(1 file)

STR:

  1. Start with media.ffmpeg.vaapi.enabled = true from terminal on Ubuntu 23.04.
  2. Play videos with codec supported by HW decoding.

On startup and every time a video plays, the terminal outputs:

libva info: VA-API version 1.17.0
libva info: Trying to open /usr/lib/x86_64-linux-gnu/dri/r600_drv_video.so
libva info: Found init function __vaDriverInit_1_17
libva info: va_openDriver() returns 0

Regression window:
https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=add572d6012047244d022436e0b5c578b3dd7cf7&tochange=fb443d9a5f9cfaa17acc81c25473d7093d5cf696

Possibly regressed by Bug 1751987.

:stransky, since you are the author of the regressor, bug 1751987, could you take a look?

For more information, please visit BugBot documentation.

Flags: needinfo?(stransky)

That comes from libva which prints that to stdout when the library is initialized. So anytime we open and init the library we see it.

Flags: needinfo?(stransky)
Priority: -- → P5

Upstream libva issue:
https://github.com/intel/libva/issues/538

Rejected pull request to change the default verbosity due to Firefox initializing libva for every video causing excessive logging:
https://github.com/intel/libva/pull/564

Workaround is to add the environment variable LIBVA_MESSAGING_LEVEL=1 (0 - silent, 1 - errors, 2 - info (default)).

We could enable LIBVA_MESSAGING_LEVEL if logging is enabled and disable that otherwise.

Duplicate of this bug: 1841426

Set release status flags based on info from the regressing bug 1751987

The severity field is not set for this bug.
:jimm, could you have a look please?

For more information, please visit BugBot documentation.

Flags: needinfo?(jmathies)
Severity: -- → S3
Flags: needinfo?(jmathies)
Flags: needinfo?(ke5trel)
Flags: needinfo?(ke5trel) → needinfo?(stransky)
Assignee: nobody → stransky
Status: NEW → ASSIGNED
Flags: needinfo?(stransky)
Pushed by stransky@redhat.com: https://hg.mozilla.org/integration/autoland/rev/dd8ef46f5224 [Linux] Configure libva logging according to platform decoder log r=alwu
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 118 Branch

The patch landed in nightly and beta is affected.
:stransky, is this bug important enough to require an uplift?

  • If yes, please nominate the patch for beta approval.
  • If no, please set status-firefox117 to wontfix.

For more information, please visit BugBot documentation.

Flags: needinfo?(stransky)

Minor fix, no need to uplift.

Flags: needinfo?(stransky)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: