All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] nvc0: fix context destruction for partly implemented tesselation
@ 2015-05-18  0:56 Tobias Klausmann
  2015-05-18  1:49 ` [Nouveau] " Ilia Mirkin
  0 siblings, 1 reply; 3+ messages in thread
From: Tobias Klausmann @ 2015-05-18  0:56 UTC (permalink / raw)
  To: nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW

Backtrace:
...
0x00007ffff57fc392 in __assert_fail () from /lib64/libc.so.6
0x00007ffff0cf5bec in nvc0_shader_stage (pipe=<optimized out>) at ./nvc0/nvc0_context.h:204
nvc0_set_constant_buffer (pipe=0x631080, shader=<optimized out>, index=<optimized out>, cb=0x0)
  at nvc0/nvc0_state.c:771
0x00007ffff0a2cf63 in st_destroy_context (st=0x68a9f0) at state_tracker/st_context.c:382
...

Signed-off-by: Tobias Klausmann <tobias.johannes.klausmann@mni.thm.de>
---
 src/gallium/drivers/nouveau/nvc0/nvc0_context.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/gallium/drivers/nouveau/nvc0/nvc0_context.h b/src/gallium/drivers/nouveau/nvc0/nvc0_context.h
index 09d08e4..f910541 100644
--- a/src/gallium/drivers/nouveau/nvc0/nvc0_context.h
+++ b/src/gallium/drivers/nouveau/nvc0/nvc0_context.h
@@ -195,8 +195,8 @@ nvc0_shader_stage(unsigned pipe)
 {
    switch (pipe) {
    case PIPE_SHADER_VERTEX: return 0;
-/* case PIPE_SHADER_TESSELLATION_CONTROL: return 1; */
-/* case PIPE_SHADER_TESSELLATION_EVALUATION: return 2; */
+   case PIPE_SHADER_TESS_CTRL: return 1;
+   case PIPE_SHADER_TESS_EVAL: return 2;
    case PIPE_SHADER_GEOMETRY: return 3;
    case PIPE_SHADER_FRAGMENT: return 4;
    case PIPE_SHADER_COMPUTE: return 5;
-- 
2.4.1

_______________________________________________
Nouveau mailing list
Nouveau@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/nouveau

^ permalink raw reply related	[flat|nested] 3+ messages in thread
* Re: [Nouveau] [PATCH] nvc0: fix context destruction for partly implemented tesselation
  2015-05-18  0:56 [PATCH] nvc0: fix context destruction for partly implemented tesselation Tobias Klausmann
@ 2015-05-18  1:49 ` Ilia Mirkin
  2015-05-18  9:20   ` Marek Olšák
  0 siblings, 1 reply; 3+ messages in thread
From: Ilia Mirkin @ 2015-05-18  1:49 UTC (permalink / raw)
  To: Tobias Klausmann, Marek Olšák; +Cc: nouveau, mesa-dev

Hm, oops. But I guess my tess patchset will take care of this. I guess
it's a side-effect of adding to the gallium enums.

Marek, you ended up pushing those gallium patches out... I was going
to wait until the branchpoint was done. Should we just fix up the
drivers as necessary, or fix up st_destroy_context to not call it for
tess shaders when !tess? [But then we have to teach st/mesa about
tess]

On Sun, May 17, 2015 at 8:56 PM, Tobias Klausmann
<tobias.johannes.klausmann@mni.thm.de> wrote:
> Backtrace:
> ...
> 0x00007ffff57fc392 in __assert_fail () from /lib64/libc.so.6
> 0x00007ffff0cf5bec in nvc0_shader_stage (pipe=<optimized out>) at ./nvc0/nvc0_context.h:204
> nvc0_set_constant_buffer (pipe=0x631080, shader=<optimized out>, index=<optimized out>, cb=0x0)
>   at nvc0/nvc0_state.c:771
> 0x00007ffff0a2cf63 in st_destroy_context (st=0x68a9f0) at state_tracker/st_context.c:382
> ...
>
> Signed-off-by: Tobias Klausmann <tobias.johannes.klausmann@mni.thm.de>
> ---
>  src/gallium/drivers/nouveau/nvc0/nvc0_context.h | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/gallium/drivers/nouveau/nvc0/nvc0_context.h b/src/gallium/drivers/nouveau/nvc0/nvc0_context.h
> index 09d08e4..f910541 100644
> --- a/src/gallium/drivers/nouveau/nvc0/nvc0_context.h
> +++ b/src/gallium/drivers/nouveau/nvc0/nvc0_context.h
> @@ -195,8 +195,8 @@ nvc0_shader_stage(unsigned pipe)
>  {
>     switch (pipe) {
>     case PIPE_SHADER_VERTEX: return 0;
> -/* case PIPE_SHADER_TESSELLATION_CONTROL: return 1; */
> -/* case PIPE_SHADER_TESSELLATION_EVALUATION: return 2; */
> +   case PIPE_SHADER_TESS_CTRL: return 1;
> +   case PIPE_SHADER_TESS_EVAL: return 2;
>     case PIPE_SHADER_GEOMETRY: return 3;
>     case PIPE_SHADER_FRAGMENT: return 4;
>     case PIPE_SHADER_COMPUTE: return 5;
> --
> 2.4.1
>
> _______________________________________________
> Nouveau mailing list
> Nouveau@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/nouveau
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

^ permalink raw reply	[flat|nested] 3+ messages in thread
* Re: [Nouveau] [PATCH] nvc0: fix context destruction for partly implemented tesselation
  2015-05-18  1:49 ` [Nouveau] " Ilia Mirkin
@ 2015-05-18  9:20   ` Marek Olšák
  0 siblings, 0 replies; 3+ messages in thread
From: Marek Olšák @ 2015-05-18  9:20 UTC (permalink / raw)
  To: Ilia Mirkin; +Cc: nouveau, mesa-dev

We should fix st/mesa and move the code to cso_destroy_context, where
it belongs.

Marek

On Mon, May 18, 2015 at 3:49 AM, Ilia Mirkin <imirkin@alum.mit.edu> wrote:
> Hm, oops. But I guess my tess patchset will take care of this. I guess
> it's a side-effect of adding to the gallium enums.
>
> Marek, you ended up pushing those gallium patches out... I was going
> to wait until the branchpoint was done. Should we just fix up the
> drivers as necessary, or fix up st_destroy_context to not call it for
> tess shaders when !tess? [But then we have to teach st/mesa about
> tess]
>
> On Sun, May 17, 2015 at 8:56 PM, Tobias Klausmann
> <tobias.johannes.klausmann@mni.thm.de> wrote:
>> Backtrace:
>> ...
>> 0x00007ffff57fc392 in __assert_fail () from /lib64/libc.so.6
>> 0x00007ffff0cf5bec in nvc0_shader_stage (pipe=<optimized out>) at ./nvc0/nvc0_context.h:204
>> nvc0_set_constant_buffer (pipe=0x631080, shader=<optimized out>, index=<optimized out>, cb=0x0)
>>   at nvc0/nvc0_state.c:771
>> 0x00007ffff0a2cf63 in st_destroy_context (st=0x68a9f0) at state_tracker/st_context.c:382
>> ...
>>
>> Signed-off-by: Tobias Klausmann <tobias.johannes.klausmann@mni.thm.de>
>> ---
>>  src/gallium/drivers/nouveau/nvc0/nvc0_context.h | 4 ++--
>>  1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/src/gallium/drivers/nouveau/nvc0/nvc0_context.h b/src/gallium/drivers/nouveau/nvc0/nvc0_context.h
>> index 09d08e4..f910541 100644
>> --- a/src/gallium/drivers/nouveau/nvc0/nvc0_context.h
>> +++ b/src/gallium/drivers/nouveau/nvc0/nvc0_context.h
>> @@ -195,8 +195,8 @@ nvc0_shader_stage(unsigned pipe)
>>  {
>>     switch (pipe) {
>>     case PIPE_SHADER_VERTEX: return 0;
>> -/* case PIPE_SHADER_TESSELLATION_CONTROL: return 1; */
>> -/* case PIPE_SHADER_TESSELLATION_EVALUATION: return 2; */
>> +   case PIPE_SHADER_TESS_CTRL: return 1;
>> +   case PIPE_SHADER_TESS_EVAL: return 2;
>>     case PIPE_SHADER_GEOMETRY: return 3;
>>     case PIPE_SHADER_FRAGMENT: return 4;
>>     case PIPE_SHADER_COMPUTE: return 5;
>> --
>> 2.4.1
>>
>> _______________________________________________
>> Nouveau mailing list
>> Nouveau@lists.freedesktop.org
>> http://lists.freedesktop.org/mailman/listinfo/nouveau
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

^ permalink raw reply	[flat|nested] 3+ messages in thread
end of thread, other threads:[~2015-05-18  9:20 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-05-18  0:56 [PATCH] nvc0: fix context destruction for partly implemented tesselation Tobias Klausmann
2015-05-18  1:49 ` [Nouveau] " Ilia Mirkin
2015-05-18  9:20   ` Marek Olšák

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.