[Mesa-dev] [PATCH 00/24] Massive clean up in t_dd_dmatmp.h

Predut, Marius marius.predut at intel.com
Tue Sep 22 06:59:52 PDT 2015


In the upstream code , 'nr' variable is used intensively in many places for things like :
for (j = 0; j + 2 < count; j += nr ) {
 nr = MIN2( currentsz, count - j );

But first time when it is used it isn't initialized.
I was unable to find  a place (a macro or something)  where  "nr" is initialized.
I suppose it is done somewhere because if NOT, THIS will be very strange how this code  works.
(Every time "nr" became "currentsz" because all the time nr is a big stack value, so this is the reason for why code works?)

-----Original Message-----
From: mesa-dev [mailto:mesa-dev-bounces at lists.freedesktop.org] On Behalf Of Ian Romanick
Sent: Tuesday, September 15, 2015 5:04 PM
To: Brian Paul; mesa-dev at lists.freedesktop.org
Subject: Re: [Mesa-dev] [PATCH 00/24] Massive clean up in t_dd_dmatmp.h

On 09/14/2015 07:04 PM, Brian Paul wrote:
> On 09/14/2015 07:01 PM, Ian Romanick wrote:
>> I looked at t_dd_dmatmp.h after the previous discussions about fixing 
>> the "count" problem.  It was a mess.  The first 5 patches fix the bug.
>> The remaining 19 patches delete dead code and make the file adhere to 
>> Mesa's coding standards.
>>
>> t_dd_triemit.h and t_dd_dmatmp2.h could probably use some similar 
>> clean up.
>>
>> I don't have easy access to i915, radeon, or r200 hardware right now, 
>> so I am only able to compile test this series.  I won't be able to 
>> test it on hardware until after XDC (next week).
> 
> I have a minor comment on patch 10 which could be addressed in a follow-on.
> 
> Patch 19 does more than formatting changes (the second to last hunk is 
> a code transformation, but looks correct).

I was putting that (and the changes from if-statements to ?: in other
patches) under the loose umbrella of "formatting fixes". :)

> Anyway, the series looks good to me (though I can't test it either).

Marius noticed a problem in one of the patches, so I'm definitely going to wait until I can test both i915 and radeon before pushing.

> Reviewed-by: Brian Paul <brianp at vmware.com>

_______________________________________________
mesa-dev mailing list
mesa-dev at lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev


More information about the mesa-dev mailing list