[Mesa-stable] [Mesa-dev] [PATCH v1.1 2/2] clover: fix getting scalar args api size
Vedran Miletić
vedran at miletic.net
Sun Jun 5 23:07:26 UTC 2016
On 06/04/2016 04:27 AM, Francisco Jerez wrote:
> Serge Martin <edb+mesa at sigluy.net> writes:
>
>> This fix getting the size of a struct arg. vec3 types still work ok.
>> Only buit-in args need to have power of two alignment, getTypeAllocSize
>> reports the correct size.
>>
> Is there any guarantee that the alloc size of the type will match what
> the CL API expects? (which is the only thing arg_api_size is used for
> IIRC) Isn't it fully dependent on the data layout which is pretty much
> up to the target?
>
It is dependent on the target. The issue is that right now we get the
struct size wrongly, as we round it here to the next power of two, store
that in scalar arg object, and later compare that rounded value to
sizeof(struct_arg_type) which user has passed via the API.
Regards,
Vedran
--
Vedran Miletić
vedran.miletic.net
More information about the mesa-stable
mailing list