Re: [Dri-devel] fix for i810 issues of late..

2004-02-05 Thread Keith Whitwell
Keith Whitwell wrote: Dave Airlie wrote: I think I'll give up for today and go at it again later, perhaps the i810 just can't do this in hardware and I need to fallback or perhaps the number of vertices being output is out by one .. who knows :-) Okay I found it and checked in the fixes for

Re: [Dri-devel] fix for i810 issues of late..

2004-02-04 Thread Dave Airlie
This is GL_LINE_LOOP? That makes sense. Interesting - this should only be a heuristic to choose the fastest path, not have any correctness value. Correctness should be ensured by i810_validate_render, from t_dd_dmatmp.h. Can you try find out what's going wrong with that? okay I've

Re: [Dri-devel] fix for i810 issues of late..

2004-02-04 Thread Dave Airlie
I see the code uses LINE_STRIPS to LINE_LOOP but it seems to be in-correct, well it locks the card, I've no idea if it is correct or not!! Hi Keith, To my untrained eye it looks like the test at the start of the render_line_loop_verts is incorrect, but maybe I am just covering up the problem,

Re: [Dri-devel] fix for i810 issues of late..

2004-02-04 Thread Dave Airlie
Hi Keith, To my untrained eye it looks like the test at the start of the render_line_loop_verts is incorrect, but maybe I am just covering up the problem, with this change blender and tri work without my previous patch... but maybe I'm just moving the issue :-) patch below.. apologies to

Re: [Dri-devel] fix for i810 issues of late..

2004-02-04 Thread Dave Airlie
I think I'll give up for today and go at it again later, perhaps the i810 just can't do this in hardware and I need to fallback or perhaps the number of vertices being output is out by one .. who knows :-) Okay I found it and checked in the fixes for it, the macro expansion was catching the

Re: [Dri-devel] fix for i810 issues of late..

2004-02-03 Thread Keith Whitwell
Dave Airlie wrote: Since Keith's tnl_dd_dmatmp.h patches on the 11 Dec last year, the poor i810 has been seriously fubar, I've just gotten time to look at it now (well I've been staring at it on and off for weeks with a blank) The fix is up at http://freedesktop.org/~airlied/i810_render_fix.diff

[Dri-devel] fix for i810 issues of late..

2004-02-02 Thread Dave Airlie
Since Keith's tnl_dd_dmatmp.h patches on the 11 Dec last year, the poor i810 has been seriously fubar, I've just gotten time to look at it now (well I've been staring at it on and off for weeks with a blank) The fix is up at http://freedesktop.org/~airlied/i810_render_fix.diff I've taken it