Re: [Intel-gfx] [PATCH] tools/null_state_gen: generate null render state

2014-05-14 Thread Damien Lespiau
On Tue, May 06, 2014 at 04:39:01PM +0300, Mika Kuoppala wrote: > Generate valid (null) render state for each gen. Output > it as a c source file with batch and relocations. > > Signed-off-by: Mika Kuoppala With the GT3 URB allocation restriction added, this series is Acked-by: Damien Lespiau

Re: [Intel-gfx] [PATCH] tools/null_state_gen: generate null render state

2014-05-09 Thread Damien Lespiau
On Tue, May 06, 2014 at 02:47:40PM +0100, Chris Wilson wrote: > Why does this work? It is neither the most minimal batch, nor the > maximal. Which state is truly required? It looks like cargo-culted > Chinese. I'll have to echo this. It's really not obvious why this is needed. If you look at the r

Re: [Intel-gfx] [PATCH] tools/null_state_gen: generate null render state

2014-05-09 Thread Damien Lespiau
On Tue, May 06, 2014 at 04:39:01PM +0300, Mika Kuoppala wrote: > diff --git a/tools/null_state_gen/intel_renderstate_gen8.c > b/tools/null_state_gen/intel_renderstate_gen8.c > new file mode 100644 > index 000..7e22b24 > --- /dev/null > +++ b/tools/null_state_gen/intel_renderstate_gen8.c [...]

Re: [Intel-gfx] [PATCH] tools/null_state_gen: generate null render state

2014-05-08 Thread Mika Kuoppala
Damien Lespiau writes: > On Tue, May 06, 2014 at 04:39:01PM +0300, Mika Kuoppala wrote: >> diff --git a/tools/null_state_gen/Makefile.am >> b/tools/null_state_gen/Makefile.am >> new file mode 100644 >> index 000..40d2237 >> --- /dev/null >> +++ b/tools/null_state_gen/Makefile.am >> @@ -0,0 +

Re: [Intel-gfx] [PATCH] tools/null_state_gen: generate null render state

2014-05-08 Thread Damien Lespiau
On Tue, May 06, 2014 at 04:39:01PM +0300, Mika Kuoppala wrote: > diff --git a/tools/null_state_gen/Makefile.am > b/tools/null_state_gen/Makefile.am > new file mode 100644 > index 000..40d2237 > --- /dev/null > +++ b/tools/null_state_gen/Makefile.am > @@ -0,0 +1,16 @@ > +bin_PROGRAMS = intel_nu

Re: [Intel-gfx] [PATCH] tools/null_state_gen: generate null render state

2014-05-08 Thread Damien Lespiau
On Tue, May 06, 2014 at 04:39:01PM +0300, Mika Kuoppala wrote: > Generate valid (null) render state for each gen. Output > it as a c source file with batch and relocations. > > Signed-off-by: Mika Kuoppala > --- > configure.ac |1 + > lib/gen6_render.h

Re: [Intel-gfx] [PATCH] tools/null_state_gen: generate null render state

2014-05-06 Thread Mika Kuoppala
Chris Wilson writes: > Why does this work? It is neither the most minimal batch, nor the > maximal. Which state is truly required? It looks like cargo-culted > Chinese. These are just stripped down version of rendercopy for each gen. What I would guess would be the key to understanding the issu

Re: [Intel-gfx] [PATCH] tools/null_state_gen: generate null render state

2014-05-06 Thread Chris Wilson
Why does this work? It is neither the most minimal batch, nor the maximal. Which state is truly required? It looks like cargo-culted Chinese. -Chris -- Chris Wilson, Intel Open Source Technology Centre ___ Intel-gfx mailing list Intel-gfx@lists.freedesk