On Mon, 2010-01-25 at 06:05 -0800, michal wrote: > Brian Paul wrote on 2010-01-22 17:56: > > michal wrote: > > > >> Brian Paul wrote on 2010-01-21 21:57: > >> > >>> michal wrote: > >>> > >>> > >>>> Hi, > >>>> > >>>> This simple feature branch adds support for two-dimensional constant > >>>> buffers in TGSI. > >>>> > >>>> An example shader would look like this: > >>>> > >>>> FRAG > >>>> > >>>> DCL IN[0], COLOR, LINEAR > >>>> DCL OUT[0], COLOR > >>>> DCL CONST[1][1..2] > >>>> > >>>> MAD OUT[0], IN[0], CONST[1][2], CONST[1][1] > >>>> > >>>> END > >>>> > >>>> For this to work, one needs to bind a buffer to slot nr 1 containing at > >>>> least 3 vectors. > >>>> > >>>> > >>> Just a terminology thing: this feature really implements arrays of > >>> constant buffers, not really two-dimensional buffers, right? > >>> > >>> > >>> > >> That's correct -- the access to constbuf data is two-dimensional, but > >> the constbufs themselves are an array of differently-sized constat buffers. > >> > >> > >>> In p_state.h we should probably rename PIPE_MAX_CONSTANT to > >>> PIPE_MAX_CONSTANT_BUFFERS to be clearer. > >>> > >>> Don't we need a new PIPE_CAP_MAX_CONSTANT_BUFFERS query? Maybe even a > >>> query per shader stage? > >>> > >>> > >>> > >> What about maximum size of a single constant buffer? I would think this > >> is a more crtical parameter than the number of constbuf slots the driver > >> support. > >> > > > > Yeah, I thought we already had a query for that, but we don't. > > > > I'd suggest: > > > > PIPE_CAP_MAX_CONST_BUFFERS > > PIPE_CAP_MAX_CONST_BUFFER_SIZE (in bytes) > > > > > > All, > > Thanks for your comments, I have commited my changes to the branch and > am awaiting for more comments.
Michal, Looks good to me. Go ahead and merge when you're ready. Keith ------------------------------------------------------------------------------ The Planet: dedicated and managed hosting, cloud storage, colocation Stay online with enterprise data centers and the best network in the business Choose flexible plans and management services without long-term contracts Personal 24x7 support from experience hosting pros just a phone call away. http://p.sf.net/sfu/theplanet-com _______________________________________________ Mesa3d-dev mailing list Mesa3d-dev@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/mesa3d-dev