[Intel-gfx] [PATCH i-g-t 3/4 v5] tests/drv_module_reload: Convert sh script to C version.

[Saarinen, Jani]

> -----Original Message-----
> From: Jani Nikula [mailto:jani.nikula at linux.intel.com]
> Sent: Thursday, November 3, 2016 6:37 PM
> To: Vlad, Marius C <marius.c.vlad at intel.com>; intel-gfx at lists.freedesktop.org
> Cc: Latvala, Petri <petri.latvala at intel.com>; chris at chris-wilson.co.uk; Saarinen,
> Jani <jani.saarinen at intel.com>
> Subject: Re: [PATCH i-g-t 3/4 v5] tests/drv_module_reload: Convert sh script to
> C version.
> 
> On Thu, 03 Nov 2016, Marius Vlad <marius.c.vlad at intel.com> wrote:
> > v5:
> > - reworked gem_info to gem_sanitychecks (Chris Wilson)
> > - remove subgroups/subtests for gem_exec_store and gem_sanitycheck
> > (Chris Wilson)
> >
> > v4:
> > - adjust test to make use of lib/igt_kmod
> > - replaced SW_FINISH with SET_CACHEING (Chris Wilson)
> >
> > v3:
> > - fix passing boolean value as flags to igt_kmod_unload().
> >
> > v2:
> > - embedded gem_alive and gem_exec_store into test (Chris Wilson)
> > - int main() to igt_main (Chris Wilson)
> > - moved tests/gem_alive -> tools/gem_info (Chris Wilson)
> > - added to intel-ci/fast-feedback.testlist (Petri Latvala)
> > - added hda_dynamic_debug() (Petri Latvala)
> > - renamed from tests/drv_module_reload_basic to
> > tests/drv_module_reload (all subtests are basic and have been added to
> > fast-feedback.testlist)
> >
> > Signed-off-by: Marius Vlad <marius.c.vlad at intel.com>
> 
> I'd like the merging of this patch be postponed until we have confidence that the
> snd_hda_intel module reload works in CI. It has been failing sporadically, so
> seeing just a couple of passing rounds after the latest fixes is not enough.
Ack, agree. 

> 
> BR,
> Jani.


Jani Saarinen
Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo