On 9 November 2016 at 16:16, Robert Bragg <robert@xxxxxxxxxxxxx> wrote: > This updates the checking of disallowed loads to set a distinguishable > value before the load and explicitly check the load was a NOOP by > reading back the final value. > > Signed-off-by: Robert Bragg <robert@xxxxxxxxxxxxx> > --- > tests/gem_exec_parse.c | 20 ++++++++++++++++++-- > 1 file changed, 18 insertions(+), 2 deletions(-) > > diff --git a/tests/gem_exec_parse.c b/tests/gem_exec_parse.c > index d6a2885..f241ab7 100644 > --- a/tests/gem_exec_parse.c > +++ b/tests/gem_exec_parse.c > @@ -305,12 +305,17 @@ static void hsw_load_register_reg(void) > }; > int fd; > uint32_t handle; > + int parser_version; > + int bad_lrr_errno; > > /* Open again to get a non-master file descriptor */ > fd = drm_open_driver(DRIVER_INTEL); > > + parser_version = command_parser_version(fd); Ah, I was reading ahead, so move this to the fixture and re-use ? Reviewed-by: Matthew Auld <matthew.auld@xxxxxxxxx> _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
