[Mesa-dev] [PATCH 00/19] gallium: basic tessellation support

Roland Scheidegger sroland at vmware.com
Mon May 4 04:21:59 PDT 2015


Hmm someting like TESSEVAL is fine by me but I'd prefer something a bit
more descriptive than TESSC/TESSE...

Roland

Am 03.05.2015 um 17:59 schrieb Marek Olšák:
> Renaming everything would be slightly annoying. I personally prefer
> shorter names anyway (TESSC/TESSE, PIPE_SHADER_TESSEVAL, etc.). I
> don't think longer names would make any difference in readability.
> 
> Marek
> 
> On Sun, May 3, 2015 at 5:06 PM, Ilia Mirkin <imirkin at alum.mit.edu> wrote:
>> On Sun, May 3, 2015 at 6:18 AM, Glenn Kennard <glenn.kennard at gmail.com> wrote:
>>> On Sat, 02 May 2015 22:16:24 +0200, Ilia Mirkin <imirkin at alum.mit.edu>
>>> wrote:
>>>
>>>> This series adds tokens and updates some helper gallium functions to
>>>> know about tessellation. This provides no actual support for
>>>> tessellation in either core or drivers, however this will make it
>>>> possible to work on the core and driver pieces without crazy
>>>> interdependencies, as well as be landed separately and without
>>>> (direct) dependency.
>>>>
>>>> Most of these patches have existed for about a year already, and have
>>>> been part of my and Marek's trees enabling tessellation in the nvc0
>>>> and radeonsi drivers. I've taken this opportunity to fix up and fold
>>>> some of them though.
>>>>
>>>> This should be pretty safe to land, since even if I messed something
>>>> up, having this in-tree will make it easier for others to identify and
>>>> fix any issues collaboratively.
>>>>
>>>> Ilia Mirkin (11):
>>>>   gallium: add tessellation shader types
>>>>   gallium: add new PATCHES primitive type
>>>>   gallium: add new semantics for tessellation
>>>>   gallium: add interfaces for controlling tess program state
>>>>   gallium: add tessellation shader properties
>>>>   gallium: add patch_vertices to draw info
>>>>   gallium: add set_tess_state to configure default tessellation
>>>>     parameters
>>>>   tgsi/scan: allow scanning tessellation shaders
>>>>   tgsi/sanity: set implicit in/out array sizes based on patch sizes
>>>>   tgsi/ureg: allow ureg_dst to have dimension indices
>>>>   tgsi/dump: fix declaration printing of tessellation inputs/outputs
>>>>
>>>> Marek Olšák (8):
>>>>   gallium: bump shader input and output limits
>>>>   trace: implement new tessellation functions
>>>>   gallium/util: print patch_vertices in util_dump_draw_info
>>>>   gallium/u_blitter: disable tessellation for all operations
>>>>   gallium/cso: add support for tessellation shaders
>>>>   gallium/cso: set NULL shaders at context destruction
>>>>   gallium: disable tessellation shaders for meta ops
>>>>   tgsi/ureg: use correct limit for max input count
>>>>
>>>>  src/gallium/auxiliary/cso_cache/cso_context.c | 100
>>>> ++++++++++++++++++++++++++
>>>>  src/gallium/auxiliary/cso_cache/cso_context.h |  12 ++++
>>>>  src/gallium/auxiliary/hud/hud_context.c       |   6 ++
>>>>  src/gallium/auxiliary/postprocess/pp_run.c    |   6 ++
>>>>  src/gallium/auxiliary/tgsi/tgsi_dump.c        |  20 +++++-
>>>>  src/gallium/auxiliary/tgsi/tgsi_info.c        |   4 ++
>>>>  src/gallium/auxiliary/tgsi/tgsi_sanity.c      |  36 ++++++++--
>>>>  src/gallium/auxiliary/tgsi/tgsi_scan.c        |   6 +-
>>>>  src/gallium/auxiliary/tgsi/tgsi_strings.c     |  19 ++++-
>>>>  src/gallium/auxiliary/tgsi/tgsi_strings.h     |   2 +-
>>>>  src/gallium/auxiliary/tgsi/tgsi_ureg.c        |  26 ++++++-
>>>>  src/gallium/auxiliary/tgsi/tgsi_ureg.h        |  59 +++++++++++++--
>>>>  src/gallium/auxiliary/util/u_blit.c           |   6 ++
>>>>  src/gallium/auxiliary/util/u_blitter.c        |  27 +++++++
>>>>  src/gallium/auxiliary/util/u_blitter.h        |  16 ++++-
>>>>  src/gallium/auxiliary/util/u_dump_state.c     |   2 +
>>>>  src/gallium/docs/source/context.rst           |   5 ++
>>>>  src/gallium/docs/source/tgsi.rst              |  70 ++++++++++++++++++
>>>>  src/gallium/drivers/trace/tr_context.c        |  26 +++++++
>>>>  src/gallium/drivers/trace/tr_dump_state.c     |   2 +
>>>>  src/gallium/include/pipe/p_context.h          |  14 ++++
>>>>  src/gallium/include/pipe/p_defines.h          |  16 ++++-
>>>>  src/gallium/include/pipe/p_shader_tokens.h    |  18 ++++-
>>>>  src/gallium/include/pipe/p_state.h            |   6 +-
>>>>  src/mesa/state_tracker/st_cb_bitmap.c         |   8 ++-
>>>>  src/mesa/state_tracker/st_cb_clear.c          |   6 ++
>>>>  src/mesa/state_tracker/st_cb_drawpixels.c     |   8 ++-
>>>>  src/mesa/state_tracker/st_cb_drawtex.c        |   6 ++
>>>>  28 files changed, 501 insertions(+), 31 deletions(-)
>>>>
>>>
>>> Some minor nits for patches 1, 6 and 7, see separate mails
>>>
>>> Patches 2-5, 8-19 are
>>> Reviewed-by: Glenn Kennard <glenn.kennard at gmail.com>
>>
>> Thanks for taking a look! It's easy enough to make the naming changes
>> you proposed (just sed -i all the patches, and git am them), but I'd
>> rather not do so repeatedly. Marek, thoughts on Glenn's naming
>> suggestions?
>>
>>   -ilia
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://urldefense.proofpoint.com/v2/url?u=http-3A__lists.freedesktop.org_mailman_listinfo_mesa-2Ddev&d=AwIGaQ&c=Sqcl0Ez6M0X8aeM67LKIiDJAXVeAw-YihVMNtXt-uEs&r=Vjtt0vs_iqoI31UfJxBl7yv9I2FeiaeAYgMTLKRBc_I&m=8VMLUcMKVTLj9XdVgfTyVboRFm9LpgwGUi-E6MmAh9w&s=U9zQzxFoXxlphK53hX8mzRlMS_GGSEj4RA1EvjfDNcI&e= 
> 



More information about the mesa-dev mailing list