[Mesa-dev] clover: kernel argument improvements

[Francisco Jerez]

Tom Stellard <tom at stellard.net> writes:

> Hi,
>
> This series improves clover's kernel argument handling by allowing drivers to
> specify an alignment for the input buffer and also by sign/zero extending the
> arguments when required.  In order to test these patches on r600g and radeonsi,
> you will need to also apply the LLVM patches which were just sent to the list.
>
> -Tom

Hi Tom,

can you have a look at this branch? [1] It addresses some concerns I had
about this series.  I've dropped patches 1, 4 and 5, and rewritten 6 and
7.

I've removed the KERNEL_ARG_ALIGNMENT cap and pad arguments, instead all
the alignment information is obtained from the compiler, which
simplifies things considerably but I'm not really sure if it's going to
work for you.

The "noext" argument extension type seemed redundant so I dropped it
too.

I've also added support for byte-swapping for the case where host and
device have different endianness, and fixed all argument subclasses so
they can deal with arbitrary data sizes -- Letting the compiler specify
argument::__size isn't enough, we also needed some way to transform the
base data type to that size.

Thanks.

[1] http://cgit.freedesktop.org/~currojerez/mesa/log/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 229 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20130721/1ce79de4/attachment.pgp>