Bug 90945

Summary:	wrong assert for BGRA surfaces in vpp
Product:	libva	Reporter:	Rainer Hochecker <fernetmenta>
Component:	intel	Assignee:	haihao <haihao.xiang>
Status:	RESOLVED FIXED	QA Contact:	Sean V Kelley <seanvk>
Severity:	blocker
Priority:	medium	CC:	fritsch, gb.devel, haihao.xiang
Version:	unspecified
Hardware:	x86-64 (AMD64)
OS:	Linux (All)
Whiteboard:
i915 platform:		i915 features:
Attachments:	Fix support of BGRA in GEN75 Don't assert on output and input v2

Description Rainer Hochecker 2015-06-12 05:28:26 UTC

Interop Vaapi / EGL requires BGRA surfaces on vaapi side. driver asserts when this format is set: http://cgit.freedesktop.org/vaapi/intel-driver/tree/src/gen75_vpp_vebox.c#n1452

BGRA is supported: http://cgit.freedesktop.org/vaapi/intel-driver/commit/src/i965_drv_video.c?id=8d2077666fa71cda672e0f5476f9b91369398a6a

Comment 1 Peter Frühberger 2015-06-19 18:43:29 UTC

Created attachment 116601 [details] [review]
Fix support of BGRA in GEN75

@Haihao for review please

Comment 2 haihao 2015-06-24 07:30:21 UTC

As BGRA is also supported for input surface, could you remove the assert() for input BGRA surface too ?

Comment 3 Peter Frühberger 2015-06-24 07:51:13 UTC

Yes, will do. Will post an updated patch this evening.

Comment 4 Peter Frühberger 2015-06-24 08:20:47 UTC

Created attachment 116685 [details] [review]
Don't assert on output and input v2

Comment 5 haihao 2015-06-25 05:36:48 UTC

Applied, thanks

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.