[Libva] reference frame array in mplayer vaapi

[Xiang, Haihao]

On Mon, 2010-03-29 at 18:58 +0800, Gwenole Beauchesne wrote:
> Hi,
> 
> On Mon, 29 Mar 2010, Xiang, Haihao wrote:
> 
> >   The reference frame array in mplayer-vaapi-20090828 is filled with h->ref_list however this array in mplayer-vaapi-20100224 is
> > filled with h->short_ref/h->long_ref except GMA500. My test against the branch i965_h264 on libva shows that
> > mplayer-vaapi-20090828 is right for us. Here I don’t understand your comments about GMA500 workaround in mplayer-vaapi-2010024.
> > Why is filling reference frame array with h->ref_list wrong?
> 
> Because this is RefPicList{0,1} and does not fully represent the DPB. This 
> also caused problems for other clips. Besides, FFmpeg's DPB is limited to 
> pictures marked as "used for reference", and capped to num_ref_frames as 
> reported in the bitstream.
> 
> h->short_ref/h->long_ref is what other HW accelerators use and work for 
> them. In my old testing with MPC-HC, this also works with the GMA500 
> driver for Windows 7.

So it is possible that a picture is in slice_param->RefPicList0/1, but it isn't in pic_param->ReferenceFrames, right?

Thanks
Haihao