* [PATCH] tests/gem_exec_parse: Test for OACONTROL tracking
@ 2014-03-27 18:44 bradley.d.volkin
2014-04-08 21:28 ` Volkin, Bradley D
0 siblings, 1 reply; 3+ messages in thread
From: bradley.d.volkin @ 2014-03-27 18:44 UTC (permalink / raw)
To: intel-gfx
From: Brad Volkin <bradley.d.volkin@intel.com>
Signed-off-by: Brad Volkin <bradley.d.volkin@intel.com>
---
tests/gem_exec_parse.c | 48 ++++++++++++++++++++++++++++++++++++++++++++++++
1 file changed, 48 insertions(+)
diff --git a/tests/gem_exec_parse.c b/tests/gem_exec_parse.c
index 34d097d..853eb57 100644
--- a/tests/gem_exec_parse.c
+++ b/tests/gem_exec_parse.c
@@ -204,6 +204,8 @@ int fd;
#define PIPE_CONTROL_QW_WRITE (1<<14)
#define PIPE_CONTROL_LRI_POST_OP (1<<23)
+#define OACONTROL 0x2360
+
igt_main
{
igt_fixture {
@@ -337,6 +339,52 @@ igt_main
0));
}
+ igt_subtest("oacontrol-tracking") {
+ uint32_t lri_ok[] = {
+ MI_LOAD_REGISTER_IMM,
+ OACONTROL,
+ 0x31337000,
+ MI_LOAD_REGISTER_IMM,
+ OACONTROL,
+ 0x0,
+ MI_BATCH_BUFFER_END,
+ 0
+ };
+ uint32_t lri_bad[] = {
+ MI_LOAD_REGISTER_IMM,
+ OACONTROL,
+ 0x31337000,
+ MI_BATCH_BUFFER_END,
+ };
+ uint32_t lri_extra_bad[] = {
+ MI_LOAD_REGISTER_IMM,
+ OACONTROL,
+ 0x31337000,
+ MI_LOAD_REGISTER_IMM,
+ OACONTROL,
+ 0x0,
+ MI_LOAD_REGISTER_IMM,
+ OACONTROL,
+ 0x31337000,
+ MI_BATCH_BUFFER_END,
+ };
+ igt_assert(
+ exec_batch(fd, handle,
+ lri_ok, sizeof(lri_ok),
+ I915_EXEC_RENDER,
+ 0));
+ igt_assert(
+ exec_batch(fd, handle,
+ lri_bad, sizeof(lri_bad),
+ I915_EXEC_RENDER,
+ -EINVAL));
+ igt_assert(
+ exec_batch(fd, handle,
+ lri_extra_bad, sizeof(lri_extra_bad),
+ I915_EXEC_RENDER,
+ -EINVAL));
+ }
+
igt_fixture {
gem_close(fd, handle);
--
1.8.3.2
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH] tests/gem_exec_parse: Test for OACONTROL tracking
2014-03-27 18:44 [PATCH] tests/gem_exec_parse: Test for OACONTROL tracking bradley.d.volkin
@ 2014-04-08 21:28 ` Volkin, Bradley D
2014-04-09 12:25 ` Daniel Vetter
0 siblings, 1 reply; 3+ messages in thread
From: Volkin, Bradley D @ 2014-04-08 21:28 UTC (permalink / raw)
To: intel-gfx@lists.freedesktop.org, daniel
Hi Daniel, we've merged the kernel change for this but not the test. I'm
assuming we still want the test case.
Brad
On Thu, Mar 27, 2014 at 11:44:45AM -0700, Volkin, Bradley D wrote:
> From: Brad Volkin <bradley.d.volkin@intel.com>
>
> Signed-off-by: Brad Volkin <bradley.d.volkin@intel.com>
> ---
> tests/gem_exec_parse.c | 48 ++++++++++++++++++++++++++++++++++++++++++++++++
> 1 file changed, 48 insertions(+)
>
> diff --git a/tests/gem_exec_parse.c b/tests/gem_exec_parse.c
> index 34d097d..853eb57 100644
> --- a/tests/gem_exec_parse.c
> +++ b/tests/gem_exec_parse.c
> @@ -204,6 +204,8 @@ int fd;
> #define PIPE_CONTROL_QW_WRITE (1<<14)
> #define PIPE_CONTROL_LRI_POST_OP (1<<23)
>
> +#define OACONTROL 0x2360
> +
> igt_main
> {
> igt_fixture {
> @@ -337,6 +339,52 @@ igt_main
> 0));
> }
>
> + igt_subtest("oacontrol-tracking") {
> + uint32_t lri_ok[] = {
> + MI_LOAD_REGISTER_IMM,
> + OACONTROL,
> + 0x31337000,
> + MI_LOAD_REGISTER_IMM,
> + OACONTROL,
> + 0x0,
> + MI_BATCH_BUFFER_END,
> + 0
> + };
> + uint32_t lri_bad[] = {
> + MI_LOAD_REGISTER_IMM,
> + OACONTROL,
> + 0x31337000,
> + MI_BATCH_BUFFER_END,
> + };
> + uint32_t lri_extra_bad[] = {
> + MI_LOAD_REGISTER_IMM,
> + OACONTROL,
> + 0x31337000,
> + MI_LOAD_REGISTER_IMM,
> + OACONTROL,
> + 0x0,
> + MI_LOAD_REGISTER_IMM,
> + OACONTROL,
> + 0x31337000,
> + MI_BATCH_BUFFER_END,
> + };
> + igt_assert(
> + exec_batch(fd, handle,
> + lri_ok, sizeof(lri_ok),
> + I915_EXEC_RENDER,
> + 0));
> + igt_assert(
> + exec_batch(fd, handle,
> + lri_bad, sizeof(lri_bad),
> + I915_EXEC_RENDER,
> + -EINVAL));
> + igt_assert(
> + exec_batch(fd, handle,
> + lri_extra_bad, sizeof(lri_extra_bad),
> + I915_EXEC_RENDER,
> + -EINVAL));
> + }
> +
> igt_fixture {
> gem_close(fd, handle);
>
> --
> 1.8.3.2
>
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH] tests/gem_exec_parse: Test for OACONTROL tracking
2014-04-08 21:28 ` Volkin, Bradley D
@ 2014-04-09 12:25 ` Daniel Vetter
0 siblings, 0 replies; 3+ messages in thread
From: Daniel Vetter @ 2014-04-09 12:25 UTC (permalink / raw)
To: Volkin, Bradley D; +Cc: intel-gfx@lists.freedesktop.org
On Tue, Apr 08, 2014 at 02:28:58PM -0700, Volkin, Bradley D wrote:
> Hi Daniel, we've merged the kernel change for this but not the test. I'm
> assuming we still want the test case.
Sorry this fell through the cracks - pretty much everyone on our team has
igt access so we just push igt patches and tests after a bit of soaking
time on intel-gfx.
It might be good to designate a point person for all things igt on the vpg
side (Tim Gore maybe) who has push access and could take care of such
things. Adding Tim.
-Daniel
>
> Brad
>
> On Thu, Mar 27, 2014 at 11:44:45AM -0700, Volkin, Bradley D wrote:
> > From: Brad Volkin <bradley.d.volkin@intel.com>
> >
> > Signed-off-by: Brad Volkin <bradley.d.volkin@intel.com>
> > ---
> > tests/gem_exec_parse.c | 48 ++++++++++++++++++++++++++++++++++++++++++++++++
> > 1 file changed, 48 insertions(+)
> >
> > diff --git a/tests/gem_exec_parse.c b/tests/gem_exec_parse.c
> > index 34d097d..853eb57 100644
> > --- a/tests/gem_exec_parse.c
> > +++ b/tests/gem_exec_parse.c
> > @@ -204,6 +204,8 @@ int fd;
> > #define PIPE_CONTROL_QW_WRITE (1<<14)
> > #define PIPE_CONTROL_LRI_POST_OP (1<<23)
> >
> > +#define OACONTROL 0x2360
> > +
> > igt_main
> > {
> > igt_fixture {
> > @@ -337,6 +339,52 @@ igt_main
> > 0));
> > }
> >
> > + igt_subtest("oacontrol-tracking") {
> > + uint32_t lri_ok[] = {
> > + MI_LOAD_REGISTER_IMM,
> > + OACONTROL,
> > + 0x31337000,
> > + MI_LOAD_REGISTER_IMM,
> > + OACONTROL,
> > + 0x0,
> > + MI_BATCH_BUFFER_END,
> > + 0
> > + };
> > + uint32_t lri_bad[] = {
> > + MI_LOAD_REGISTER_IMM,
> > + OACONTROL,
> > + 0x31337000,
> > + MI_BATCH_BUFFER_END,
> > + };
> > + uint32_t lri_extra_bad[] = {
> > + MI_LOAD_REGISTER_IMM,
> > + OACONTROL,
> > + 0x31337000,
> > + MI_LOAD_REGISTER_IMM,
> > + OACONTROL,
> > + 0x0,
> > + MI_LOAD_REGISTER_IMM,
> > + OACONTROL,
> > + 0x31337000,
> > + MI_BATCH_BUFFER_END,
> > + };
> > + igt_assert(
> > + exec_batch(fd, handle,
> > + lri_ok, sizeof(lri_ok),
> > + I915_EXEC_RENDER,
> > + 0));
> > + igt_assert(
> > + exec_batch(fd, handle,
> > + lri_bad, sizeof(lri_bad),
> > + I915_EXEC_RENDER,
> > + -EINVAL));
> > + igt_assert(
> > + exec_batch(fd, handle,
> > + lri_extra_bad, sizeof(lri_extra_bad),
> > + I915_EXEC_RENDER,
> > + -EINVAL));
> > + }
> > +
> > igt_fixture {
> > gem_close(fd, handle);
> >
> > --
> > 1.8.3.2
> >
--
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2014-04-09 12:25 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-03-27 18:44 [PATCH] tests/gem_exec_parse: Test for OACONTROL tracking bradley.d.volkin
2014-04-08 21:28 ` Volkin, Bradley D
2014-04-09 12:25 ` Daniel Vetter
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox