After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 785663 - kmssink: need check scaleable when start
kmssink: need check scaleable when start
Status: RESOLVED OBSOLETE
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
git master
Other Linux
: Normal normal
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2017-08-01 03:08 UTC by Haihua Hu
Modified: 2018-11-03 14:11 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
kmssink: add atomic check for scale capability (5.22 KB, patch)
2017-08-11 15:47 UTC, Haihua Hu
none Details | Review

Description Haihua Hu 2017-08-01 03:08:54 UTC
I create a new thread for this issue. Please refer to this link for detail info:https://bugzilla.gnome.org/show_bug.cgi?id=784599

There is a bug when driver not support scale. when video size is same as display resolution, show frame will not do retry because the setplane will never fail. But the scale result is not correct. We can do this check by set a fake plane.

This bug will impact the videooverlay interface, because the implementation is based on driver scaleable capability.
Comment 1 Haihua Hu 2017-08-11 15:47:36 UTC
Created attachment 357423 [details] [review]
kmssink: add atomic check for scale capability

Use atomic check can get the scale capability of driver, will keep retry setplane to be compatible with the driver not support atomic check
Comment 2 Haihua Hu 2017-08-21 03:10:43 UTC
Hi all,

Any Update?
Comment 3 Víctor Manuel Jáquez Leal 2017-08-21 14:14:36 UTC
Hi,

From my side, I have been very busy lately, and my knowledge on the atomic API is none, so I would need to study it. Thus, from my side, I need more time to assess this patch.
Comment 4 Nicolas Dufresne (ndufresne) 2017-09-29 13:24:51 UTC
I ask Daniel S. about mixing APIs, he didn't think it's a great idea. Feature sets between atomic/non-atomic could vary due to porting still being in progress on some drivers.
Comment 5 Haihua Hu 2017-10-09 07:51:37 UTC
(In reply to Nicolas Dufresne (stormer) from comment #4)
> I ask Daniel S. about mixing APIs, he didn't think it's a great idea.
> Feature sets between atomic/non-atomic could vary due to porting still being
> in progress on some drivers.

Not feature setting just testing and check.
Comment 6 GStreamer system administrator 2018-11-03 14:11:40 UTC
-- GitLab Migration Automatic Message --

This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/issues/591.