On Mon, Dec 19, 2016 at 06:25:26PM +0100, Simon Richter wrote:
> Date: Mon, 19 Dec 2016 18:25:26 +0100
> From: Simon Richter <[email protected]>
> To: [email protected]
> Subject: Re: [Beignet] [PATCH] Refine mem.h and improve the related macro
> defination.
>
> Hi,
>
> On 19.12.2016 10:21, [email protected] wrote:
>
> > --- a/src/cl_api_mem.c
> > +++ b/src/cl_api_mem.c
> > @@ -71,54 +71,54 @@ clGetMemObjectInfo(cl_mem memobj,
> > break;
> > case CL_MEM_HOST_PTR: {
> > ptr = 0;
> > - if (memobj->type == CL_MEM_IMAGE_TYPE) {
> > + if (!CL_OBJECT_IS_BUFFER(memobj)) {
> > ptr = (size_t)memobj->host_ptr;
> > } else {
>
> That would match pipes as well, is that intended?
>
> Simon
>
Already redefine CL_OBJECT_IS_BUFFER to just match buffer and subbuffer.
+#define CL_OBJECT_IS_BUFFER(mem) ((mem &&
\
+ ((cl_base_object)mem)->magic ==
CL_OBJECT_MEM_MAGIC && \
+ CL_OBJECT_GET_REF(mem) >= 1 &&
\
+ ((cl_mem)mem)->type <=
CL_MEM_SUBBUFFER_TYPE))
> _______________________________________________
> Beignet mailing list
> [email protected]
> https://lists.freedesktop.org/mailman/listinfo/beignet
_______________________________________________
Beignet mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/beignet
