Fix packed headers attribute test in vaCreateConfig()

If packed headers are not supported then the only valid value for this
attribute is VA_ATTRIB_NOT_SUPPORTED.  If packed headers are supported,
then the provided value must be some (possibly empty) subset of the
supported headers.

This fixes config creation with an empty packed header set, which the
user may pass if they don't want to provide any packed headers.  This
pattern is used in at least libavcodec, where VP9 encoding is broken
prior to this change.

Fixes #362.

Signed-off-by: Mark Thompson <[email protected]>

Author: fhvwy

src/i965_drv_video.c

@@ -1431,8 +1431,13 @@ i965_CreateConfig(VADriverContextP ctx,
         if (attrib_found) {
             uint32_t enc_packed_attribs = i965_get_enc_packed_attributes(ctx, profile, entrypoint);
 
-            if (!(attrib_found->value & enc_packed_attribs))
-                vaStatus = VA_STATUS_ERROR_INVALID_VALUE;
+            if (enc_packed_attribs == VA_ATTRIB_NOT_SUPPORTED) {
+                if (attrib_found->value != VA_ATTRIB_NOT_SUPPORTED)
+                    vaStatus = VA_STATUS_ERROR_INVALID_VALUE;
+            } else {
+                if (attrib_found->value & ~enc_packed_attribs)
+                    vaStatus = VA_STATUS_ERROR_INVALID_VALUE;
+            }
         }
     }