>-----Original Message----- >From: Zhao, Yakui >Sent: Thursday, November 17, 2016 8:47 PM >To: Xiang, Haihao <haihao.xi...@intel.com> >Cc: libva@lists.freedesktop.org >Subject: Re: [Libva] [Libva-intel-driver][PATCH 06/17] Fix curbe length in >CMD_MEDIA_CURBE_LOAD on GEN8+ > >On 11/17/2016 04:35 PM, Xiang, Haihao wrote: >> It is multiple of 64 bytes > >Can we align the size of curbe_buffer when calling the gen8_gpe_context_init? >
The size is set by gpe user, so I prefer not to change it in gen8_gpe_context_init(). or we can add a requirement on the size which must be a multiple of 64 bytes. >> >> Signed-off-by: Xiang, Haihao<haihao.xi...@intel.com> >> --- >> src/i965_gpe_utils.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/src/i965_gpe_utils.c b/src/i965_gpe_utils.c index >> 139ab1b..a6d539b 100644 >> --- a/src/i965_gpe_utils.c >> +++ b/src/i965_gpe_utils.c >> @@ -1066,7 +1066,7 @@ gen8_gpe_curbe_load(VADriverContextP ctx, >> >> OUT_BATCH(batch, CMD_MEDIA_CURBE_LOAD | (4 - 2)); >> OUT_BATCH(batch, 0); >> - OUT_BATCH(batch, gpe_context->curbe.length); >> + OUT_BATCH(batch, ALIGN(gpe_context->curbe.length, 64)); >> OUT_BATCH(batch, gpe_context->curbe.offset); >> >> ADVANCE_BATCH(batch); _______________________________________________ Libva mailing list Libva@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libva