Could you explain more how to reproduce the regressions? And if possible, a bisect should be helpful to root cause the regression. Thanks.
On Tue, May 13, 2014 at 03:16:20PM +0800, He Junyan wrote: > This patch will cause some regression in buffer tests > > > On Mon, 2014-05-12 at 23:11 +0800, Yang Rong wrote: > > HSW: Byte scattered Read/Write require that the buffer size must be a > > multiple of 4 bytes. > > So simply alignment all buffer size to 4. Pass utest > > compiler_function_constant0. > > > > Because it is very light work around, align it without not check device. > > > > Signed-off-by: Yang Rong <rong.r.y...@intel.com> > > --- > > src/cl_mem.c | 4 ++++ > > 1 file changed, 4 insertions(+) > > > > diff --git a/src/cl_mem.c b/src/cl_mem.c > > index 44482f7..5feda74 100644 > > --- a/src/cl_mem.c > > +++ b/src/cl_mem.c > > @@ -334,6 +334,10 @@ cl_mem_new_buffer(cl_context ctx, > > goto error; > > } > > > > + /* HSW: Byte scattered Read/Write has limitation that > > + the buffer size must be a multiple of 4 bytes. */ > > + sz = ALIGN(sz, 4); > > + > > /* Create the buffer in video memory */ > > mem = cl_mem_allocate(CL_MEM_BUFFER_TYPE, ctx, flags, sz, CL_FALSE, > > &err); > > if (mem == NULL || err != CL_SUCCESS) > > > > _______________________________________________ > Beignet mailing list > Beignet@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/beignet _______________________________________________ Beignet mailing list Beignet@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/beignet
