Re: [Libva] [Libva-intel-driver][PATCH] Needn't reset brc if the bitrate setting isn't changed in the Begin/Render/End sequence

2017-01-03 Thread Mark Thompson
On 03/01/17 06:42, Xiang, Haihao wrote: > On Fri, 2016-12-30 at 17:44 +, Mark Thompson wrote: >> On 30/12/16 00:51, Xiang, Haihao wrote: >>> On Thu, 2016-12-29 at 18:52 +, Mark Thompson wrote: On 29/12/16 07:09, Zhao Yakui wrote: > On 12/29/2016 03:08 PM, Xiang, Haihao wrote:

Re: [Libva] [Libva-intel-driver][PATCH] Needn't reset brc if the bitrate setting isn't changed in the Begin/Render/End sequence

2017-01-02 Thread Xiang, Haihao
On Fri, 2016-12-30 at 17:44 +, Mark Thompson wrote: > On 30/12/16 00:51, Xiang, Haihao wrote: > > On Thu, 2016-12-29 at 18:52 +, Mark Thompson wrote: > > > On 29/12/16 07:09, Zhao Yakui wrote: > > > > On 12/29/2016 03:08 PM, Xiang, Haihao wrote: > > > > > > > > > > > On 12/29/2016 01:22

Re: [Libva] [Libva-intel-driver][PATCH] Needn't reset brc if the bitrate setting isn't changed in the Begin/Render/End sequence

2017-01-02 Thread Xiang, Haihao
On Tue, 2017-01-03 at 09:50 +0800, Zhao Yakui wrote: > On 12/31/2016 01:54 AM, Mark Thompson wrote: > > On 30/12/16 00:14, Zhao Yakui wrote: > > > On 12/30/2016 02:52 AM, Mark Thompson wrote: > > > > On 29/12/16 07:09, Zhao Yakui wrote: > > > > > On 12/29/2016 03:08 PM, Xiang, Haihao wrote: > > >

Re: [Libva] [Libva-intel-driver][PATCH] Needn't reset brc if the bitrate setting isn't changed in the Begin/Render/End sequence

2017-01-02 Thread Zhao Yakui
On 12/31/2016 01:54 AM, Mark Thompson wrote: On 30/12/16 00:14, Zhao Yakui wrote: On 12/30/2016 02:52 AM, Mark Thompson wrote: On 29/12/16 07:09, Zhao Yakui wrote: On 12/29/2016 03:08 PM, Xiang, Haihao wrote: On 12/29/2016 01:22 PM, Xiang, Haihao wrote: User can use

Re: [Libva] [Libva-intel-driver][PATCH] Needn't reset brc if the bitrate setting isn't changed in the Begin/Render/End sequence

2016-12-30 Thread Mark Thompson
On 30/12/16 00:14, Zhao Yakui wrote: > On 12/30/2016 02:52 AM, Mark Thompson wrote: >> On 29/12/16 07:09, Zhao Yakui wrote: >>> On 12/29/2016 03:08 PM, Xiang, Haihao wrote: > On 12/29/2016 01:22 PM, Xiang, Haihao wrote: >> User can use VAEncMiscParameterRateControl to update bitrate,

Re: [Libva] [Libva-intel-driver][PATCH] Needn't reset brc if the bitrate setting isn't changed in the Begin/Render/End sequence

2016-12-30 Thread Mark Thompson
On 30/12/16 00:51, Xiang, Haihao wrote: > On Thu, 2016-12-29 at 18:52 +, Mark Thompson wrote: >> On 29/12/16 07:09, Zhao Yakui wrote: >>> On 12/29/2016 03:08 PM, Xiang, Haihao wrote: > On 12/29/2016 01:22 PM, Xiang, Haihao wrote: >> User can use VAEncMiscParameterRateControl to

Re: [Libva] [Libva-intel-driver][PATCH] Needn't reset brc if the bitrate setting isn't changed in the Begin/Render/End sequence

2016-12-29 Thread Xiang, Haihao
On Thu, 2016-12-29 at 18:52 +, Mark Thompson wrote: > On 29/12/16 07:09, Zhao Yakui wrote: > > On 12/29/2016 03:08 PM, Xiang, Haihao wrote: > > > > > > > On 12/29/2016 01:22 PM, Xiang, Haihao wrote: > > > > > User can use VAEncMiscParameterRateControl to update bitrate, > > > > > so we > > >

Re: [Libva] [Libva-intel-driver][PATCH] Needn't reset brc if the bitrate setting isn't changed in the Begin/Render/End sequence

2016-12-29 Thread Zhao Yakui
On 12/30/2016 02:52 AM, Mark Thompson wrote: On 29/12/16 07:09, Zhao Yakui wrote: On 12/29/2016 03:08 PM, Xiang, Haihao wrote: On 12/29/2016 01:22 PM, Xiang, Haihao wrote: User can use VAEncMiscParameterRateControl to update bitrate, so we should ignore the bitrate in the sequence parameter

Re: [Libva] [Libva-intel-driver][PATCH] Needn't reset brc if the bitrate setting isn't changed in the Begin/Render/End sequence

2016-12-29 Thread Mark Thompson
On 29/12/16 07:09, Zhao Yakui wrote: > On 12/29/2016 03:08 PM, Xiang, Haihao wrote: >> >>> On 12/29/2016 01:22 PM, Xiang, Haihao wrote: User can use VAEncMiscParameterRateControl to update bitrate, so we should ignore the bitrate in the sequence parameter if

Re: [Libva] [Libva-intel-driver][PATCH] Needn't reset brc if the bitrate setting isn't changed in the Begin/Render/End sequence

2016-12-28 Thread Zhao Yakui
On 12/29/2016 03:08 PM, Xiang, Haihao wrote: On 12/29/2016 01:22 PM, Xiang, Haihao wrote: User can use VAEncMiscParameterRateControl to update bitrate, so we should ignore the bitrate in the sequence parameter if VAEncMiscParameterRateControl is present. This makes sense. The

Re: [Libva] [Libva-intel-driver][PATCH] Needn't reset brc if the bitrate setting isn't changed in the Begin/Render/End sequence

2016-12-28 Thread Xiang, Haihao
> On 12/29/2016 01:22 PM, Xiang, Haihao wrote: > > User can use VAEncMiscParameterRateControl to update bitrate, so we > > should ignore > > the bitrate in the sequence parameter if > > VAEncMiscParameterRateControl is present. > > This makes sense. The MiscRateControl mentions that it can

Re: [Libva] [Libva-intel-driver][PATCH] Needn't reset brc if the bitrate setting isn't changed in the Begin/Render/End sequence

2016-12-28 Thread Zhao Yakui
On 12/29/2016 01:22 PM, Xiang, Haihao wrote: User can use VAEncMiscParameterRateControl to update bitrate, so we should ignore the bitrate in the sequence parameter if VAEncMiscParameterRateControl is present. This makes sense. The MiscRateControl mentions that it can override the bps