Quoting Matthew Auld (2017-11-07 11:17:53) > On 7 November 2017 at 11:05, Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> wrote: > > If we only have 4k pages, we can't mix together different combinations > > of hugepages to see if the world burns. However, as the loops did > > nothing, we never set err to 0 and reported ENODEV aborting the test. > > Teach the test to skip instead. > > > > Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> > > Cc: Matthew Auld <matthew.william.auld@xxxxxxxxx> > > Cc: Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx> > > --- > > drivers/gpu/drm/i915/selftests/huge_pages.c | 5 ++++- > > 1 file changed, 4 insertions(+), 1 deletion(-) > > > > diff --git a/drivers/gpu/drm/i915/selftests/huge_pages.c b/drivers/gpu/drm/i915/selftests/huge_pages.c > > index 5cc8101bb2b1..a8d3328b8b64 100644 > > --- a/drivers/gpu/drm/i915/selftests/huge_pages.c > > +++ b/drivers/gpu/drm/i915/selftests/huge_pages.c > > @@ -1156,8 +1156,11 @@ static int igt_ppgtt_exhaust_huge(void *arg) > > struct drm_i915_gem_object *obj; > > unsigned int size_mask; > > unsigned int page_mask; > > - int n, i; > > int err = -ENODEV; > > + int n, i; > Keep the err at the end? The question is which looks less like a Christmas tree. Does the '=...' contribute to the width? -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
