Re: [Mesa-dev] [PATCH] etnaviv: Avoid infinite loop in find_frame()

2017-01-30 Thread Christian Gmeiner
2017-01-30 15:47 GMT+01:00 Rhys Kidd : > > On Sun, Jan 22, 2017 at 1:49 PM Rhys Kidd wrote: >> >> On 20 January 2017 at 17:06, Christian Gmeiner >> wrote: >>> >>> Hi Rhys, >>> >>> 2017-01-19 7:02 GMT+01:00 Rhys Kidd : >>> > Use of unsigned loop control variable with '>= 0' would lead to >>> > inf

Re: [Mesa-dev] [PATCH] etnaviv: Avoid infinite loop in find_frame()

2017-01-30 Thread Rhys Kidd
On Sun, Jan 22, 2017 at 1:49 PM Rhys Kidd wrote: > On 20 January 2017 at 17:06, Christian Gmeiner < > christian.gmei...@gmail.com> wrote: > > Hi Rhys, > > 2017-01-19 7:02 GMT+01:00 Rhys Kidd : > > Use of unsigned loop control variable with '>= 0' would lead to infinite > loop. > > > > Reported by

Re: [Mesa-dev] [PATCH] etnaviv: Avoid infinite loop in find_frame()

2017-01-22 Thread Rhys Kidd
On 20 January 2017 at 17:06, Christian Gmeiner wrote: > Hi Rhys, > > 2017-01-19 7:02 GMT+01:00 Rhys Kidd : > > Use of unsigned loop control variable with '>= 0' would lead to infinite > loop. > > > > Reported by clang: > > > > etnaviv_compiler.c:1024:39: warning: comparison of unsigned expression

Re: [Mesa-dev] [PATCH] etnaviv: Avoid infinite loop in find_frame()

2017-01-20 Thread Christian Gmeiner
Hi Rhys, 2017-01-19 7:02 GMT+01:00 Rhys Kidd : > Use of unsigned loop control variable with '>= 0' would lead to infinite loop. > > Reported by clang: > > etnaviv_compiler.c:1024:39: warning: comparison of unsigned expression >= 0 is > always true [-Wtautological-compare] >for (unsigned

[Mesa-dev] [PATCH] etnaviv: Avoid infinite loop in find_frame()

2017-01-18 Thread Rhys Kidd
Use of unsigned loop control variable with '>= 0' would lead to infinite loop. Reported by clang: etnaviv_compiler.c:1024:39: warning: comparison of unsigned expression >= 0 is always true [-Wtautological-compare] for (unsigned sp = c->frame_sp; sp >= 0; sp--)