[Mesa-dev] [PATCH 09/11] gallium: add support for arb_cull_distance

Tobias Klausmann tobias.johannes.klausmann at mni.thm.de
Mon May 25 11:56:19 PDT 2015



On 25.05.2015 20:36, Roland Scheidegger wrote:
> This doesn't really do what the commit message claims - it just adds a
> cap bit, not actual support for arb_cull_distance (which was already
> there), so the log should be changed accordingly.

Yep, you are completely right here, will change it to better represent 
what was done here.

> Apart from what was already mentioned (that is, the cap bit isn't used
> by st/mesa which instead mistakenly uses glsl13),

will change it for v2

>   it would be nice if
> you could test softpipe/llvmpipe with this enabled, as it should already
> work (at least for llvmpipe, I'm not entirely sure for softpipe) if not
> there's some unaccounted difference somewhere how we'd thought of how
> this should work for dx10 vs. opengl).

Hey nice to know llvmpipe already supports this, i'll test it!

Tobias

>
> Roland
>
>
> Am 24.05.2015 um 19:58 schrieb Tobias Klausmann:
>> Add another pipe cap so we can savely enable of disable this extension
>>
>> Signed-off-by: Tobias Klausmann <tobias.johannes.klausmann at mni.thm.de>
>> ---
>>   src/gallium/auxiliary/cso_cache/cso_context.c    | 3 +++
>>   src/gallium/drivers/freedreno/freedreno_screen.c | 1 +
>>   src/gallium/drivers/i915/i915_screen.c           | 1 +
>>   src/gallium/drivers/ilo/ilo_screen.c             | 1 +
>>   src/gallium/drivers/llvmpipe/lp_screen.c         | 2 ++
>>   src/gallium/drivers/nouveau/nv30/nv30_screen.c   | 1 +
>>   src/gallium/drivers/nouveau/nv50/nv50_screen.c   | 1 +
>>   src/gallium/drivers/nouveau/nvc0/nvc0_screen.c   | 1 +
>>   src/gallium/drivers/r300/r300_screen.c           | 1 +
>>   src/gallium/drivers/r600/r600_pipe.c             | 1 +
>>   src/gallium/drivers/radeonsi/si_pipe.c           | 1 +
>>   src/gallium/drivers/softpipe/sp_screen.c         | 2 ++
>>   src/gallium/drivers/svga/svga_screen.c           | 1 +
>>   src/gallium/drivers/vc4/vc4_screen.c             | 1 +
>>   src/gallium/include/pipe/p_defines.h             | 1 +
>>   15 files changed, 19 insertions(+)
>>
>> diff --git a/src/gallium/auxiliary/cso_cache/cso_context.c b/src/gallium/auxiliary/cso_cache/cso_context.c
>> index 744b00c..7612b43 100644
>> --- a/src/gallium/auxiliary/cso_cache/cso_context.c
>> +++ b/src/gallium/auxiliary/cso_cache/cso_context.c
>> @@ -119,6 +119,9 @@ struct cso_context {
>>      struct pipe_clip_state clip;
>>      struct pipe_clip_state clip_saved;
>>   
>> +   struct pipe_clip_state cull;
>> +   struct pipe_clip_state cull_saved;
>> +
>>      struct pipe_framebuffer_state fb, fb_saved;
>>      struct pipe_viewport_state vp, vp_saved;
>>      struct pipe_blend_color blend_color;
>> diff --git a/src/gallium/drivers/freedreno/freedreno_screen.c b/src/gallium/drivers/freedreno/freedreno_screen.c
>> index c596d03..986a942 100644
>> --- a/src/gallium/drivers/freedreno/freedreno_screen.c
>> +++ b/src/gallium/drivers/freedreno/freedreno_screen.c
>> @@ -221,6 +221,7 @@ fd_screen_get_param(struct pipe_screen *pscreen, enum pipe_cap param)
>>   	case PIPE_CAP_MULTISAMPLE_Z_RESOLVE:
>>   	case PIPE_CAP_RESOURCE_FROM_USER_MEMORY:
>>   	case PIPE_CAP_DEVICE_RESET_STATUS_QUERY:
>> +	case PIPE_CAP_CULL_DISTANCE:
>>   		return 0;
>>   
>>   	case PIPE_CAP_MAX_VIEWPORTS:
>> diff --git a/src/gallium/drivers/i915/i915_screen.c b/src/gallium/drivers/i915/i915_screen.c
>> index 03fecd1..678347d 100644
>> --- a/src/gallium/drivers/i915/i915_screen.c
>> +++ b/src/gallium/drivers/i915/i915_screen.c
>> @@ -242,6 +242,7 @@ i915_get_param(struct pipe_screen *screen, enum pipe_cap cap)
>>      case PIPE_CAP_MULTISAMPLE_Z_RESOLVE:
>>      case PIPE_CAP_RESOURCE_FROM_USER_MEMORY:
>>      case PIPE_CAP_DEVICE_RESET_STATUS_QUERY:
>> +   case PIPE_CAP_CULL_DISTANCE:
>>         return 0;
>>   
>>      case PIPE_CAP_MAX_DUAL_SOURCE_RENDER_TARGETS:
>> diff --git a/src/gallium/drivers/ilo/ilo_screen.c b/src/gallium/drivers/ilo/ilo_screen.c
>> index b0fed73..f92d5de 100644
>> --- a/src/gallium/drivers/ilo/ilo_screen.c
>> +++ b/src/gallium/drivers/ilo/ilo_screen.c
>> @@ -459,6 +459,7 @@ ilo_get_param(struct pipe_screen *screen, enum pipe_cap param)
>>      case PIPE_CAP_MULTISAMPLE_Z_RESOLVE:
>>      case PIPE_CAP_RESOURCE_FROM_USER_MEMORY:
>>      case PIPE_CAP_DEVICE_RESET_STATUS_QUERY:
>> +   case PIPE_CAP_CULL_DISTANCE:
>>         return 0;
>>   
>>      case PIPE_CAP_VENDOR_ID:
>> diff --git a/src/gallium/drivers/llvmpipe/lp_screen.c b/src/gallium/drivers/llvmpipe/lp_screen.c
>> index 09ac9af..c90c405 100644
>> --- a/src/gallium/drivers/llvmpipe/lp_screen.c
>> +++ b/src/gallium/drivers/llvmpipe/lp_screen.c
>> @@ -293,6 +293,8 @@ llvmpipe_get_param(struct pipe_screen *screen, enum pipe_cap param)
>>      case PIPE_CAP_RESOURCE_FROM_USER_MEMORY:
>>      case PIPE_CAP_DEVICE_RESET_STATUS_QUERY:
>>         return 0;
>> +   case PIPE_CAP_CULL_DISTANCE:
>> +      return 0;
>>      }
>>      /* should only get here on unhandled cases */
>>      debug_printf("Unexpected PIPE_CAP %d query\n", param);
>> diff --git a/src/gallium/drivers/nouveau/nv30/nv30_screen.c b/src/gallium/drivers/nouveau/nv30/nv30_screen.c
>> index bb79ccc..fc33ddf 100644
>> --- a/src/gallium/drivers/nouveau/nv30/nv30_screen.c
>> +++ b/src/gallium/drivers/nouveau/nv30/nv30_screen.c
>> @@ -162,6 +162,7 @@ nv30_screen_get_param(struct pipe_screen *pscreen, enum pipe_cap param)
>>      case PIPE_CAP_MULTISAMPLE_Z_RESOLVE:
>>      case PIPE_CAP_RESOURCE_FROM_USER_MEMORY:
>>      case PIPE_CAP_DEVICE_RESET_STATUS_QUERY:
>> +   case PIPE_CAP_CULL_DISTANCE:
>>         return 0;
>>   
>>      case PIPE_CAP_VENDOR_ID:
>> diff --git a/src/gallium/drivers/nouveau/nv50/nv50_screen.c b/src/gallium/drivers/nouveau/nv50/nv50_screen.c
>> index f455a7f..d8efd75 100644
>> --- a/src/gallium/drivers/nouveau/nv50/nv50_screen.c
>> +++ b/src/gallium/drivers/nouveau/nv50/nv50_screen.c
>> @@ -210,6 +210,7 @@ nv50_screen_get_param(struct pipe_screen *pscreen, enum pipe_cap param)
>>      case PIPE_CAP_MULTISAMPLE_Z_RESOLVE: /* potentially supported on some hw */
>>      case PIPE_CAP_RESOURCE_FROM_USER_MEMORY:
>>      case PIPE_CAP_DEVICE_RESET_STATUS_QUERY:
>> +   case PIPE_CAP_CULL_DISTANCE:
>>         return 0;
>>   
>>      case PIPE_CAP_VENDOR_ID:
>> diff --git a/src/gallium/drivers/nouveau/nvc0/nvc0_screen.c b/src/gallium/drivers/nouveau/nvc0/nvc0_screen.c
>> index 1ca997a..c942dda 100644
>> --- a/src/gallium/drivers/nouveau/nvc0/nvc0_screen.c
>> +++ b/src/gallium/drivers/nouveau/nvc0/nvc0_screen.c
>> @@ -194,6 +194,7 @@ nvc0_screen_get_param(struct pipe_screen *pscreen, enum pipe_cap param)
>>      case PIPE_CAP_VERTEXID_NOBASE:
>>      case PIPE_CAP_RESOURCE_FROM_USER_MEMORY:
>>      case PIPE_CAP_DEVICE_RESET_STATUS_QUERY:
>> +   case PIPE_CAP_CULL_DISTANCE:
>>         return 0;
>>   
>>      case PIPE_CAP_VENDOR_ID:
>> diff --git a/src/gallium/drivers/r300/r300_screen.c b/src/gallium/drivers/r300/r300_screen.c
>> index 8e1d710..00bb6dd 100644
>> --- a/src/gallium/drivers/r300/r300_screen.c
>> +++ b/src/gallium/drivers/r300/r300_screen.c
>> @@ -191,6 +191,7 @@ static int r300_get_param(struct pipe_screen* pscreen, enum pipe_cap param)
>>           case PIPE_CAP_MULTISAMPLE_Z_RESOLVE:
>>           case PIPE_CAP_RESOURCE_FROM_USER_MEMORY:
>>           case PIPE_CAP_DEVICE_RESET_STATUS_QUERY:
>> +        case PIPE_CAP_CULL_DISTANCE:
>>               return 0;
>>   
>>           /* SWTCL-only features. */
>> diff --git a/src/gallium/drivers/r600/r600_pipe.c b/src/gallium/drivers/r600/r600_pipe.c
>> index 5a8eb06..87f556b 100644
>> --- a/src/gallium/drivers/r600/r600_pipe.c
>> +++ b/src/gallium/drivers/r600/r600_pipe.c
>> @@ -333,6 +333,7 @@ static int r600_get_param(struct pipe_screen* pscreen, enum pipe_cap param)
>>   	case PIPE_CAP_SAMPLER_VIEW_TARGET:
>>   	case PIPE_CAP_VERTEXID_NOBASE:
>>   	case PIPE_CAP_DEVICE_RESET_STATUS_QUERY:
>> +	case PIPE_CAP_CULL_DISTANCE:
>>   		return 0;
>>   
>>   	/* Stream output. */
>> diff --git a/src/gallium/drivers/radeonsi/si_pipe.c b/src/gallium/drivers/radeonsi/si_pipe.c
>> index 9d60ef1..7ec9758 100644
>> --- a/src/gallium/drivers/radeonsi/si_pipe.c
>> +++ b/src/gallium/drivers/radeonsi/si_pipe.c
>> @@ -294,6 +294,7 @@ static int si_get_param(struct pipe_screen* pscreen, enum pipe_cap param)
>>   	case PIPE_CAP_SAMPLER_VIEW_TARGET:
>>   	case PIPE_CAP_VERTEXID_NOBASE:
>>   	case PIPE_CAP_DEVICE_RESET_STATUS_QUERY:
>> +	case PIPE_CAP_CULL_DISTANCE:
>>   		return 0;
>>   
>>   	case PIPE_CAP_TEXTURE_BORDER_COLOR_QUIRK:
>> diff --git a/src/gallium/drivers/softpipe/sp_screen.c b/src/gallium/drivers/softpipe/sp_screen.c
>> index a688d31..c0d7d86 100644
>> --- a/src/gallium/drivers/softpipe/sp_screen.c
>> +++ b/src/gallium/drivers/softpipe/sp_screen.c
>> @@ -243,6 +243,8 @@ softpipe_get_param(struct pipe_screen *screen, enum pipe_cap param)
>>      case PIPE_CAP_RESOURCE_FROM_USER_MEMORY:
>>      case PIPE_CAP_DEVICE_RESET_STATUS_QUERY:
>>         return 0;
>> +   case PIPE_CAP_CULL_DISTANCE:
>> +      return 0;
>>      }
>>      /* should only get here on unhandled cases */
>>      debug_printf("Unexpected PIPE_CAP %d query\n", param);
>> diff --git a/src/gallium/drivers/svga/svga_screen.c b/src/gallium/drivers/svga/svga_screen.c
>> index 549a89a..6e5a3da 100644
>> --- a/src/gallium/drivers/svga/svga_screen.c
>> +++ b/src/gallium/drivers/svga/svga_screen.c
>> @@ -285,6 +285,7 @@ svga_get_param(struct pipe_screen *screen, enum pipe_cap param)
>>      case PIPE_CAP_VERTEXID_NOBASE:
>>      case PIPE_CAP_POLYGON_OFFSET_CLAMP:
>>      case PIPE_CAP_MULTISAMPLE_Z_RESOLVE:
>> +   case PIPE_CAP_CULL_DISTANCE:
>>         return 0;
>>      case PIPE_CAP_MIN_MAP_BUFFER_ALIGNMENT:
>>         return 64;
>> diff --git a/src/gallium/drivers/vc4/vc4_screen.c b/src/gallium/drivers/vc4/vc4_screen.c
>> index 134d644..565f0c4 100644
>> --- a/src/gallium/drivers/vc4/vc4_screen.c
>> +++ b/src/gallium/drivers/vc4/vc4_screen.c
>> @@ -176,6 +176,7 @@ vc4_screen_get_param(struct pipe_screen *pscreen, enum pipe_cap param)
>>           case PIPE_CAP_MULTISAMPLE_Z_RESOLVE:
>>           case PIPE_CAP_RESOURCE_FROM_USER_MEMORY:
>>           case PIPE_CAP_DEVICE_RESET_STATUS_QUERY:
>> +        case PIPE_CAP_CULL_DISTANCE:
>>                   return 0;
>>   
>>                   /* Stream output. */
>> diff --git a/src/gallium/include/pipe/p_defines.h b/src/gallium/include/pipe/p_defines.h
>> index 8fabf5e..50f9cfe 100644
>> --- a/src/gallium/include/pipe/p_defines.h
>> +++ b/src/gallium/include/pipe/p_defines.h
>> @@ -605,6 +605,7 @@ enum pipe_cap
>>      PIPE_CAP_MULTISAMPLE_Z_RESOLVE = 114,
>>      PIPE_CAP_RESOURCE_FROM_USER_MEMORY = 115,
>>      PIPE_CAP_DEVICE_RESET_STATUS_QUERY = 116,
>> +   PIPE_CAP_CULL_DISTANCE = 117,
>>   };
>>   
>>   #define PIPE_QUIRK_TEXTURE_BORDER_COLOR_SWIZZLE_NV50 (1 << 0)
>>



More information about the mesa-dev mailing list