On Fri, 2009-09-25 at 02:02 -0700, Christoph Bumiller wrote: > Module: Mesa > Branch: master > Commit: 513cadf5afad18516f7299ade246f59d520753d0 > URL: > http://cgit.freedesktop.org/mesa/mesa/commit/?id=513cadf5afad18516f7299ade246f59d520753d0 > > Author: Christoph Bumiller <e0425...@student.tuwien.ac.at> > Date: Thu Sep 24 17:37:08 2009 +0200 > > nv50: actually enable view volume clipping > > Until now, only primitives wholly outside the view volume > were not drawn. > This was only visibile when using a viewport smaller than > the window size, naturally. > > --- > > src/gallium/drivers/nv50/nv50_state_validate.c | 11 ++++++++++- > 1 files changed, 10 insertions(+), 1 deletions(-) > > diff --git a/src/gallium/drivers/nv50/nv50_state_validate.c > b/src/gallium/drivers/nv50/nv50_state_validate.c > index 5a3559e..4ed7697 100644 > --- a/src/gallium/drivers/nv50/nv50_state_validate.c > +++ b/src/gallium/drivers/nv50/nv50_state_validate.c > @@ -312,7 +312,7 @@ scissor_uptodate: > goto viewport_uptodate; > nv50->state.viewport_bypass = bypass; > > - so = so_new(12, 0); > + so = so_new(14, 0); > if (!bypass) { > so_method(so, tesla, NV50TCL_VIEWPORT_TRANSLATE(0), 3); > so_data (so, fui(nv50->viewport.translate[0])); > @@ -325,12 +325,21 @@ scissor_uptodate: > > so_method(so, tesla, NV50TCL_VIEWPORT_TRANSFORM_EN, 1); > so_data (so, 1); > + /* 0x0000 = remove whole primitive only (xyz) > + * 0x1018 = remove whole primitive only (xy), clamp z > + * 0x1080 = clip primitive (xyz) > + * 0x1098 = clip primitive (xy), clamp z > + */ > + so_method(so, tesla, NV50TCL_VIEW_VOLUME_CLIP_CTRL, 1); > + so_data (so, 0x1080); > /* no idea what 0f90 does */ > so_method(so, tesla, 0x0f90, 1); > so_data (so, 0); > } else { > so_method(so, tesla, NV50TCL_VIEWPORT_TRANSFORM_EN, 1); > so_data (so, 0); > + so_method(so, tesla, NV50TCL_VIEW_VOLUME_CLIP_CTRL, 1); > + so_data (so, 0x0000); > so_method(so, tesla, 0x0f90, 1); > so_data (so, 1); > } >
Chris, Do you notice any performance difference after doing this? I suspect that the fastpath for a lot of hardware is to actually avoid geometry-based clipping and do it with the rasterizer. If this forces geometry clipping on when not required, it may bottleneck the hardware in vertex processing. I'm only guessing, but something to keep in mind or investigate one day. Keith ------------------------------------------------------------------------------ Come build with us! The BlackBerry® Developer Conference in SF, CA is the only developer event you need to attend this year. Jumpstart your developing skills, take BlackBerry mobile applications to market and stay ahead of the curve. Join us from November 9-12, 2009. Register now! http://p.sf.net/sfu/devconf _______________________________________________ Mesa3d-dev mailing list Mesa3d-dev@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/mesa3d-dev
