[Libva] [PATCH 2/3] gen6_mfc_common: Do not use the reserved nal unit types

Gwenole Beauchesne gb.devel at gmail.com
Fri Aug 29 07:49:03 PDT 2014


Hi,

2014-08-29 16:24 GMT+02:00  <sreerenj.balachandran at intel.com>:
> From: Sreerenj Balachandran <sreerenj.balachandran at intel.com>
>
> The nal_unit_type 21 is reserved, don't take this into account
> while doing the nal unit header bytes counting.

This is valid extension for MVCD. So, should someone, somehow,
generates MVCD packed headers, that could be needed. This is
hypothetitical, but at least, this is readily future proof. e.g. we
could imagine someone generates and stiches in the MVCD bits through
additional SW support.

> ---
>  src/gen6_mfc_common.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/gen6_mfc_common.c b/src/gen6_mfc_common.c
> index fe41dac..e68528e 100644
> --- a/src/gen6_mfc_common.c
> +++ b/src/gen6_mfc_common.c
> @@ -494,7 +494,7 @@ static int intel_avc_find_skipemulcnt(unsigned char *buf, int bits_length)
>      nal_unit_type = (buf[skip_cnt]) & NAL_UNIT_TYPE_MASK;
>      skip_cnt += 1;
>
> -    if (nal_unit_type == 14 || nal_unit_type == 20 || nal_unit_type == 21) {
> +    if (nal_unit_type == 14 || nal_unit_type == 20) {
>          /* more unit header bytes are accounted for MVC/SVC */
>          skip_cnt += 3;
>      }
> --
> 1.9.1
>
> _______________________________________________
> Libva mailing list
> Libva at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/libva


Thanks,
-- 
Gwenole Beauchesne
Intel Corporation SAS / 2 rue de Paris, 92196 Meudon Cedex, France
Registration Number (RCS): Nanterre B 302 456 199


More information about the Libva mailing list