Re: [Libva] [PATCH intel-driver 1/4] decoder: h264: don't deallocate surface storage of older frames.

2014-06-02 Thread Gwenole Beauchesne
2014-05-14 11:04 GMT+02:00 Gwenole Beauchesne gb.de...@gmail.com: Hi, 2014-05-12 8:17 GMT+02:00 Zhao, Yakui yakui.z...@intel.com: On Sun, 2014-05-11 at 23:31 -0600, Gwenole Beauchesne wrote: Hi, 2014-05-12 7:21 GMT+02:00 Zhao, Yakui yakui.z...@intel.com: On Sun, 2014-05-11 at 22:35 -0600,

Re: [Libva] [PATCH intel-driver 1/4] decoder: h264: don't deallocate surface storage of older frames.

2014-05-14 Thread Gwenole Beauchesne
Hi, 2014-05-12 8:17 GMT+02:00 Zhao, Yakui yakui.z...@intel.com: On Sun, 2014-05-11 at 23:31 -0600, Gwenole Beauchesne wrote: Hi, 2014-05-12 7:21 GMT+02:00 Zhao, Yakui yakui.z...@intel.com: On Sun, 2014-05-11 at 22:35 -0600, Gwenole Beauchesne wrote: Hi, 2014-05-12 2:55 GMT+02:00 Zhao

Re: [Libva] [PATCH intel-driver 1/4] decoder: h264: don't deallocate surface storage of older frames.

2014-05-11 Thread Zhao Yakui
On Fri, 2014-05-09 at 08:34 -0600, Gwenole Beauchesne wrote: Drop the optimization whereby surfaces that are no longer marked as reference and that were already displayed are to be destroyed. This is wrong mainly for two reasons: 1. The surface was displayed... once but it may still be

Re: [Libva] [PATCH intel-driver 1/4] decoder: h264: don't deallocate surface storage of older frames.

2014-05-11 Thread Gwenole Beauchesne
Hi, 2014-05-12 2:55 GMT+02:00 Zhao Yakui yakui.z...@intel.com: On Fri, 2014-05-09 at 08:34 -0600, Gwenole Beauchesne wrote: Drop the optimization whereby surfaces that are no longer marked as reference and that were already displayed are to be destroyed. This is wrong mainly for two reasons:

Re: [Libva] [PATCH intel-driver 1/4] decoder: h264: don't deallocate surface storage of older frames.

2014-05-11 Thread Gwenole Beauchesne
Hi, 2014-05-12 7:21 GMT+02:00 Zhao, Yakui yakui.z...@intel.com: On Sun, 2014-05-11 at 22:35 -0600, Gwenole Beauchesne wrote: Hi, 2014-05-12 2:55 GMT+02:00 Zhao Yakui yakui.z...@intel.com: On Fri, 2014-05-09 at 08:34 -0600, Gwenole Beauchesne wrote: Drop the optimization whereby surfaces

[Libva] [PATCH intel-driver 1/4] decoder: h264: don't deallocate surface storage of older frames.

2014-05-09 Thread Gwenole Beauchesne
Drop the optimization whereby surfaces that are no longer marked as reference and that were already displayed are to be destroyed. This is wrong mainly for two reasons: 1. The surface was displayed... once but it may still be needed for subsequent operations like displaying it again, using it