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 > [email protected] > http://lists.freedesktop.org/mailman/listinfo/libva _______________________________________________ Libva mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/libva
