Re: [Mesa-dev] [PATCH v3] i915: fixing driver crashes if too few vertices are submitted

2015-09-14 Thread Predut, Marius
14, 2015 11:18 AM > To: Eirik Byrkjeflot Anonsen; Ilia Mirkin; Ian Romanick > Cc: mesa-dev@lists.freedesktop.org; Romanick, Ian D > Subject: Re: [Mesa-dev] [PATCH v3] i915: fixing driver crashes if too > few vertices are submitted > > > > -Original Message- >

Re: [Mesa-dev] [PATCH v3] i915: fixing driver crashes if too few vertices are submitted

2015-09-14 Thread Predut, Marius
-- > From: mesa-dev [mailto:mesa-dev-boun...@lists.freedesktop.org] On Behalf Of > Predut, Marius > Sent: Monday, September 14, 2015 11:25 AM > To: Ian Romanick; Ilia Mirkin > Cc: mesa-dev@lists.freedesktop.org; Romanick, Ian D > Subject: Re: [Mesa-dev] [PATCH v3] i915: fixing driver crashes if too

Re: [Mesa-dev] [PATCH v3] i915: fixing driver crashes if too few vertices are submitted

2015-09-14 Thread Predut, Marius
t Anonsen; Ilia Mirkin; Ian Romanick > Cc: mesa-dev@lists.freedesktop.org; Romanick, Ian D > Subject: Re: [Mesa-dev] [PATCH v3] i915: fixing driver crashes if too few > vertices are submitted > > > > -Original Message- > > From: mesa-dev [mailto:mesa-dev-boun.

Re: [Mesa-dev] [PATCH v3] i915: fixing driver crashes if too few vertices are submitted

2015-09-14 Thread Predut, Marius
edut, Marius > Cc: mesa-dev@lists.freedesktop.org; Romanick, Ian D > Subject: Re: [Mesa-dev] [PATCH v3] i915: fixing driver crashes if too few > vertices are submitted > > On 09/11/2015 10:55 AM, Ilia Mirkin wrote: > > On Fri, Sep 11, 2015 at 12:36 PM, Marius Predut >

Re: [Mesa-dev] [PATCH v3] i915: fixing driver crashes if too few vertices are submitted

2015-09-14 Thread Predut, Marius
> -Original Message- > From: mesa-dev [mailto:mesa-dev-boun...@lists.freedesktop.org] On Behalf Of > Ilia Mirkin > Sent: Saturday, September 12, 2015 8:40 AM > To: Ian Romanick > Cc: mesa-dev@lists.freedesktop.org; Romanick, Ian D > Subject: Re: [Mesa-dev] [PATCH v3

Re: [Mesa-dev] [PATCH v3] i915: fixing driver crashes if too few vertices are submitted

2015-09-14 Thread Predut, Marius
Re: [Mesa-dev] [PATCH v3] i915: fixing driver crashes if too few > vertices are submitted > > Ilia Mirkin writes: > > > On Fri, Sep 11, 2015 at 10:45 PM, Ian Romanick wrote: > >> On 09/11/2015 10:55 AM, Ilia Mirkin wrote: > >>> On Fri, Sep 11, 2015 at 12:36

Re: [Mesa-dev] [PATCH v3] i915: fixing driver crashes if too few vertices are submitted

2015-09-11 Thread Eirik Byrkjeflot Anonsen
Ilia Mirkin writes: > On Fri, Sep 11, 2015 at 10:45 PM, Ian Romanick wrote: >> On 09/11/2015 10:55 AM, Ilia Mirkin wrote: >>> On Fri, Sep 11, 2015 at 12:36 PM, Marius Predut >>> wrote: Comparison with a signed expression and unsigned value is converted to unsigned value, reason for m

Re: [Mesa-dev] [PATCH v3] i915: fixing driver crashes if too few vertices are submitted

2015-09-11 Thread Ilia Mirkin
On Fri, Sep 11, 2015 at 10:45 PM, Ian Romanick wrote: > On 09/11/2015 10:55 AM, Ilia Mirkin wrote: >> On Fri, Sep 11, 2015 at 12:36 PM, Marius Predut >> wrote: >>> Comparison with a signed expression and unsigned value >>> is converted to unsigned value, reason for minus value is interpreted >>>

Re: [Mesa-dev] [PATCH v3] i915: fixing driver crashes if too few vertices are submitted

2015-09-11 Thread Ian Romanick
On 09/11/2015 10:55 AM, Ilia Mirkin wrote: > On Fri, Sep 11, 2015 at 12:36 PM, Marius Predut > wrote: >> Comparison with a signed expression and unsigned value >> is converted to unsigned value, reason for minus value is interpreted >> as a big unsigned value. For this case the "for" loop >> is g

Re: [Mesa-dev] [PATCH v3] i915: fixing driver crashes if too few vertices are submitted

2015-09-11 Thread Ilia Mirkin
On Fri, Sep 11, 2015 at 12:36 PM, Marius Predut wrote: > Comparison with a signed expression and unsigned value > is converted to unsigned value, reason for minus value is interpreted > as a big unsigned value. For this case the "for" loop > is going into unexpected behavior. > > v1: Brian Paul: c

[Mesa-dev] [PATCH v3] i915: fixing driver crashes if too few vertices are submitted

2015-09-11 Thread Marius Predut
Comparison with a signed expression and unsigned value is converted to unsigned value, reason for minus value is interpreted as a big unsigned value. For this case the "for" loop is going into unexpected behavior. v1: Brian Paul: code style fix. v2: Ian Romanick: glDrawArrays(GL_QUADS, 0, (n * 4)