Skip to content

Require extra info for Linux hardware accel#491

Merged
RytoEX merged 1 commit into
obsproject:masterfrom
tytan652:linux_hwaccel_pr_fix
Jul 9, 2025
Merged

Require extra info for Linux hardware accel#491
RytoEX merged 1 commit into
obsproject:masterfrom
tytan652:linux_hwaccel_pr_fix

Conversation

@tytan652

@tytan652 tytan652 commented Jul 6, 2025

Copy link
Copy Markdown
Contributor

Description

Disable hardware accel on Linux if the extra field is not provided by CEF API.

A CEF 127 build with the API backported can be used if CEF_OSR_EXTRA_INFO is defined by the build itself. (https://github.com/tytan652/cef/tree/6533-fix-stutter-and-osr-extra-info)
Otherwise CEF 133 is required to enable it without backport required.

Motivation and Context

To properly do DMA-BUF we need CEF to provide the size of the image and not rely on the source settings.

How Has This Been Tested?

It works on my machine.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)

Checklist:

  • My code has been run through clang-format.
  • I have read the contributing document.
  • My code is not on the master branch.
  • The code has been tested.
  • All commit messages are properly formatted and commits squashed where appropriate.
  • I have included updates to all appropriate documentation.

Comment thread browser-client.cpp

@RytoEX RytoEX left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks fine at a glance.

@RytoEX RytoEX merged commit 82092ce into obsproject:master Jul 9, 2025
1 check passed
@tytan652 tytan652 deleted the linux_hwaccel_pr_fix branch July 12, 2025 18:20
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.

2 participants