Tom Stellard <t...@stellard.net> writes: > On Sun, Jul 21, 2013 at 07:46:36PM +0200, Francisco Jerez wrote: >> Tom Stellard <t...@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/ > > > This series looks good to me, and there are no regressions when compared > to my patches. Could you add > > Cc: mesa-sta...@lists.freedesktop.org > > to the patches when you commit, so they can be applied to the 9.2 > branch. > OK, pushed.
Thank you. > Thanks, > Tom
pgpTYCP6LpzBd.pgp
Description: PGP signature
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
