[Libva] [PATCH 0/2] Fix and simplify REF_IDX_STATE

[Xiang, Haihao]

On Mon, 2012-01-30 at 19:26 +0100, Gwenole Beauchesne wrote: 
> Hi,
> 
> The following patches factor out and fix REF_IDX_STATE construction.
> 
> 1. Bit5 set to 1 normally represents a frame, not a picture ;
> 2. RefPicList0/1 size is controlled by num_ref_idx_l0/1_minus1 + 1. So, don't
>    traverse the whole array that may not contain valid data.
> 3. Cope for cases where decoder represents frame with TOP|BOTTOM.
> 
> Tested on ILK, SNB, IVB with gstreamer-vaapi. I will push that if nobody finds
> out a regression by the end of week.

Your patches look good to me. Thanks.

Regards
Haihao

> 
> Regards,
> Gwenole.
> 
> Gwenole Beauchesne (2):
>   Factor out type definitions (GenFrameStore, GenBuffer).
>   h264: fix and simplify REF_IDX_STATE (ILK, SNB, IVB).
> 
>  NEWS                     |    5 ++-
>  src/Makefile.am          |    3 +
>  src/gen6_mfd.c           |   77 +++-------------------------
>  src/gen6_mfd.h           |   49 +++--------------
>  src/gen7_mfd.c           |   77 +++-------------------------
>  src/gen7_mfd.h           |   49 +++--------------
>  src/i965_avc_bsd.c       |   60 +++++----------------
>  src/i965_decoder.h       |   46 ++++++++++++++++
>  src/i965_decoder_utils.c |  130 ++++++++++++++++++++++++++++++++++++++++++++++
>  src/i965_decoder_utils.h |   45 ++++++++++++++++
>  src/i965_media_h264.h    |    6 +--
>  11 files changed, 277 insertions(+), 270 deletions(-)
>  create mode 100644 src/i965_decoder.h
>  create mode 100644 src/i965_decoder_utils.c
>  create mode 100644 src/i965_decoder_utils.h
> 
> _______________________________________________
> Libva mailing list
> Libva at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/libva