Re: [01/15] drm/i915: Copy user requested buffers into the error state

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Mar 21, 2017 at 10:53:53AM -0700, Ben Widawsky wrote:
> On 17-03-21 16:23:05, Tahvanainen, Jari wrote:
> >See below [Jari]...
> >
> >-----Original Message-----
> >From: Ben Widawsky [mailto:ben@xxxxxxxxxxxx]
> >Sent: Tuesday, March 21, 2017 5:38 PM
> >To: Tahvanainen, Jari <jari.tahvanainen@xxxxxxxxx>
> >Cc: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>; intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> >Subject: Re: [01/15] drm/i915: Copy user requested buffers into the error state
> >
> >On 17-03-21 11:30:36, Tahvanainen, Jari wrote:
> >>Note that this is for all the patches in series, replied only on [1/15].
> >>
> >>See also https://bugs.freedesktop.org/show_bug.cgi?id=94001#c45
> >>
> >
> >Jari, did you test this patch specifically? It would involve introspection of the error state.
> >
> >[Jari]  like said I tested the patch series including this patch
> >	" Note that this is for all the patches in series, replied only on [1/15]"
> >             Tested-by " for https://patchwork.freedesktop.org/series/21377";
> >          	
> >	If this is not the way to do it then I need to stop.
> >          	And since being tester (not programmer) you need to tell more what do you mean with " would involve introspection of the error state".
> >	What should be outcome? What skill shall have for it, etc.? If I cannot do it then assumable tested-by is not the thing that I will do in future.
> 
> Well there is tested-by "this doesn't regress anything" and there is tested-by
> "this new feature works properly". I've no doubt you asserted the first, but my
> concern was around the second. For this patch specifically, it's a new feature
> and there is no igt test for it AFAIK.

There is/will be, gem_exec_capture; sent alongside the original
patch.
-Chris
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux