Bug 91626

Summary:	[HSW..] vpp number of filters assertion
Product:	libva	Reporter:	Víctor Jáquez <vjaquez>
Component:	intel	Assignee:	PengChen <peng.c.chen>
Status:	ASSIGNED ---	QA Contact:	Sean V Kelley <seanvk>
Severity:	normal
Priority:	medium	CC:	bsreerenj
Version:	unspecified
Hardware:	Other
OS:	All
See Also:	https://bugs.freedesktop.org/show_bug.cgi?id=67150
Whiteboard:
i915 platform:		i915 features:

Description Víctor Jáquez 2015-08-13 15:10:09 UTC

While working in color balance interface in vaapipostroc [1], I ran across with an assertion that crashed the execution:

gen75_picture_process.c:169: VAStatus gen75_proc_picture(VADriverContextP, VAProfile, union codec_state *, struct hw_context *): Assertion `pipeline_param->num_filters <= 4' failed.
Aborted

In my opinion, this shouldn't be handled in this way. If the hardware offers only a limited number of filters to activate simultaneously, it should behave different (ignore the new filter, perhaps) but it shouldn't crash. 


1. https://bugzilla.gnome.org/show_bug.cgi?id=720376

Comment 1 PengChen 2016-12-14 00:50:05 UTC

whether return error is the better idea if pipeline_param->num_filters <= 4 is not true? it makes the user confused if the driver ignores the new filters at this case.

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.