Re: [Mesa-dev] [PATCH 00/20] Auto-generate pack/unpack functions

2014-11-25 Thread Michel Dänzer
On 21.11.2014 23:57, Samuel Iglesias Gonsálvez wrote: On Wed, 2014-11-19 at 17:09 +0900, Michel Dänzer wrote: On 18.11.2014 17:43, Iago Toral Quiroga wrote: For software drivers we worked with a trimmed set of piglit tests (related to format conversion), ~5700 tests selected with the

Re: [Mesa-dev] [PATCH 00/20] Auto-generate pack/unpack functions

2014-11-25 Thread Samuel Iglesias Gonsálvez
On 25/11/14 09:59, Michel Dänzer wrote: On 21.11.2014 23:57, Samuel Iglesias Gonsálvez wrote: On Wed, 2014-11-19 at 17:09 +0900, Michel Dänzer wrote: On 18.11.2014 17:43, Iago Toral Quiroga wrote: For software drivers we worked with a trimmed set of piglit tests (related to format

Re: [Mesa-dev] [PATCH 00/20] Auto-generate pack/unpack functions

2014-11-25 Thread Jose Fonseca
On 25/11/14 09:43, Samuel Iglesias Gonsálvez wrote: On 25/11/14 09:59, Michel Dänzer wrote: On 21.11.2014 23:57, Samuel Iglesias Gonsálvez wrote: On Wed, 2014-11-19 at 17:09 +0900, Michel Dänzer wrote: On 18.11.2014 17:43, Iago Toral Quiroga wrote: For software drivers we worked with a

Re: [Mesa-dev] [PATCH 00/20] Auto-generate pack/unpack functions

2014-11-21 Thread Samuel Iglesias Gonsálvez
On Wed, 2014-11-19 at 17:09 +0900, Michel Dänzer wrote: On 18.11.2014 17:43, Iago Toral Quiroga wrote: For software drivers we worked with a trimmed set of piglit tests (related to format conversion), ~5700 tests selected with the following filter: -t format -t color -t tex -t image

Re: [Mesa-dev] [PATCH 00/20] Auto-generate pack/unpack functions

2014-11-19 Thread Michel Dänzer
On 18.11.2014 17:43, Iago Toral Quiroga wrote: For software drivers we worked with a trimmed set of piglit tests (related to format conversion), ~5700 tests selected with the following filter: -t format -t color -t tex -t image -t swizzle -t clamp -t rgb -t lum -t pix -t fbo -t frame Any

Re: [Mesa-dev] [PATCH 00/20] Auto-generate pack/unpack functions

2014-11-19 Thread Iago Toral
On Tue, 2014-11-18 at 13:41 -0800, Jason Ekstrand wrote: Iago, Most of this looks pretty good to me. The one primary concern I have is in the handling of integer formats. I made the comment in a couple of patches, but I'll make it in general here. In a lot of the code, when you convert

Re: [Mesa-dev] [PATCH 00/20] Auto-generate pack/unpack functions

2014-11-19 Thread Iago Toral
On Wed, 2014-11-19 at 17:09 +0900, Michel Dänzer wrote: On 18.11.2014 17:43, Iago Toral Quiroga wrote: For software drivers we worked with a trimmed set of piglit tests (related to format conversion), ~5700 tests selected with the following filter: -t format -t color -t tex -t image

Re: [Mesa-dev] [PATCH 00/20] Auto-generate pack/unpack functions

2014-11-19 Thread Samuel Iglesias Gonsálvez
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 19/11/14 09:25, Iago Toral wrote: On Tue, 2014-11-18 at 13:41 -0800, Jason Ekstrand wrote: Iago, Most of this looks pretty good to me. The one primary concern I have is in the handling of integer formats. I made the comment in a couple of

Re: [Mesa-dev] [PATCH 00/20] Auto-generate pack/unpack functions

2014-11-19 Thread Jose Fonseca
On 19/11/14 08:29, Iago Toral wrote: On Wed, 2014-11-19 at 17:09 +0900, Michel Dänzer wrote: On 18.11.2014 17:43, Iago Toral Quiroga wrote: For software drivers we worked with a trimmed set of piglit tests (related to format conversion), ~5700 tests selected with the following filter: -t

Re: [Mesa-dev] [PATCH 00/20] Auto-generate pack/unpack functions

2014-11-19 Thread Samuel Iglesias Gonsálvez
On Wed, 2014-11-19 at 01:15 +, Emil Velikov wrote: Hi Iago, On 18/11/14 08:43, Iago Toral Quiroga wrote: [snip] Summary of the patches: * Patches 1-7 are general fixes to the current code that were found while working on this. Have you noticed if any of those fixes resolves a

[Mesa-dev] [PATCH 00/20] Auto-generate pack/unpack functions

2014-11-18 Thread Iago Toral Quiroga
This is the fist of two series of patches to address: https://bugs.freedesktop.org/show_bug.cgi?id=84566 The idea is that we have a lot of format conversion code scattered through different files in the repository, a lot of that is redundant / duplicated, so this intends to address that issue.

Re: [Mesa-dev] [PATCH 00/20] Auto-generate pack/unpack functions

2014-11-18 Thread Iago Toral
I forgot to say that the series is available for testing here: https://github.com/Igalia/mesa/tree/itoral-autogen-packing-review Also, one of the patches was held for review by the list owner due to its size (patch 12, which handles auto-generation of format_unpack.c), so reviewers can fetch it

Re: [Mesa-dev] [PATCH 00/20] Auto-generate pack/unpack functions

2014-11-18 Thread Jose Fonseca
The idea is that we have a lot of format conversion code scattered through different files in the repository, a lot of that is redundant / duplicated, so this intends to address that issue. First, I think this is a great goal. And while I haven't reviewed them in detail, just from

Re: [Mesa-dev] [PATCH 00/20] Auto-generate pack/unpack functions

2014-11-18 Thread Jason Ekstrand
Jose, I haven't had time to fully review Iago and Samuel's code, so I can't 100% comment on it right now. However, let me make a few comments on the overarching plan as it were. On Tue, Nov 18, 2014 at 2:36 AM, Jose Fonseca jfons...@vmware.com wrote: The idea is that we have a lot of format

Re: [Mesa-dev] [PATCH 00/20] Auto-generate pack/unpack functions

2014-11-18 Thread Jason Ekstrand
Iago, Most of this looks pretty good to me. The one primary concern I have is in the handling of integer formats. I made the comment in a couple of patches, but I'll make it in general here. In a lot of the code, when you convert from integer formats to float, you treat them as if they are

Re: [Mesa-dev] [PATCH 00/20] Auto-generate pack/unpack functions

2014-11-18 Thread Jose Fonseca
On 18/11/14 17:10, Jason Ekstrand wrote: Jose, I haven't had time to fully review Iago and Samuel's code, so I can't 100% comment on it right now. However, let me make a few comments on the overarching plan as it were. On Tue, Nov 18, 2014 at 2:36 AM, Jose Fonseca jfons...@vmware.com

Re: [Mesa-dev] [PATCH 00/20] Auto-generate pack/unpack functions

2014-11-18 Thread Jason Ekstrand
On Tue, Nov 18, 2014 at 1:42 PM, Jose Fonseca jfons...@vmware.com wrote: On 18/11/14 17:10, Jason Ekstrand wrote: Jose, I haven't had time to fully review Iago and Samuel's code, so I can't 100% comment on it right now. However, let me make a few comments on the overarching plan as it

Re: [Mesa-dev] [PATCH 00/20] Auto-generate pack/unpack functions

2014-11-18 Thread Emil Velikov
Hi Iago, On 18/11/14 08:43, Iago Toral Quiroga wrote: [snip] Summary of the patches: * Patches 1-7 are general fixes to the current code that were found while working on this. Have you noticed if any of those fixes resolves a piglit and/or a real world application ? If so it might be worth