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 tri
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 t
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 following
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 t
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 re
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 form
-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 co
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 t
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 conv
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 part
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 wor
On Tue, Nov 18, 2014 at 1:42 PM, Jose Fonseca 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 were.
>>
>>
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 mailto:jfons...@vmware.com>>
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
norma
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 wrote:
> > The idea is that we have a lot of format conversion code s
> 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 skimm
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 fr
17 matches
Mail list logo
