[HSW..] vpp number of filters assertion

[seanvk]

migrated from Bugzilla #91626
status ASSIGNED severity normal in component intel for ---
Reported in version unspecified on platform Other
Assigned to: PengChen

On 2015-08-13 15:10:09 +0000, Víctor Jáquez wrote:

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

On 2016-12-14 00:50:05 +0000, PengChen wrote:

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.