Skip to content

fix(gstreamer): harden log validation for successful decode pipelines#398

Merged
bhargav0610 merged 1 commit intoqualcomm-linux:mainfrom
smuppand:gstreamer
Apr 9, 2026
Merged

fix(gstreamer): harden log validation for successful decode pipelines#398
bhargav0610 merged 1 commit intoqualcomm-linux:mainfrom
smuppand:gstreamer

Conversation

@smuppand
Copy link
Copy Markdown
Contributor

@smuppand smuppand commented Apr 9, 2026

This fixes #397 the GStreamer log validation helpers to avoid false failures on successful decode pipelines that emit benign downstream CRITICAL warnings.

What changed:

  • ignore known non-fatal warnings seen on successful V4L2 decode paths such as:
    • gst_video_info_dma_drm_to_caps assertion on invalid DRM fourcc
    • gst_structure_remove_field assertion on immutable structures
  • keep hard failures for genuine ERROR/FATAL, negotiation, state-change, and file/resource issues
  • validate remaining CRITICAL messages against positive success evidence such as PLAYING transition, caps negotiation, and EOS completion
  • improve failure reporting by surfacing matched fatal log lines

Why:
Video decode pipelines were completing successfully with exit code 0, caps negotiation, PLAYING state, and EOS, but the testcase was still marked as failed because the validator treated all CRITICAL log messages as fatal.

Impact:

  • avoids false FAIL results for successful H.264, H.265, and VP9 decode testcases
  • preserves detection of genuine GStreamer pipeline failures
  • makes multimedia validation more robust on downstream V4L2 stacks

…lines

Improve GStreamer log validation so successful decode pipelines are not
marked as failed due to benign downstream CRITICAL warnings.

What this fixes:
- ignore known non-fatal V4L2 decode warnings seen on successful H.264,
  H.265, and VP9 decode paths
- keep hard failures for real ERROR/FATAL, negotiation, state-change,
  and resource issues
- use pipeline success evidence such as PLAYING, caps negotiation, and
  EOS completion before treating remaining CRITICAL messages as fatal

Signed-off-by: Srikanth Muppandam <smuppand@qti.qualcomm.com>
@smuppand smuppand requested review from Vkenjam and bhargav0610 April 9, 2026 16:47
Copy link
Copy Markdown
Contributor

@bhargav0610 bhargav0610 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@bhargav0610 bhargav0610 merged commit e421daa into qualcomm-linux:main Apr 9, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

imsdk video decode testcase failed in axiom

2 participants