On Tue, 2017-10-03 at 12:52 +0100, Chris Wilson wrote:
> Quoting Joonas Lahtinen (2017-10-03 12:47:33)
> > Merge the remaining testing coverage delta from gem_bad_length and
> > check that creating an object of zero size fails with -EINVAL.
> >
> > Suggested-by: Daniele Ceraolo Spurio <daniele.ceraolospurio@xxxxxxxxx>
> > Signed-off-by: Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx>
> > Cc: Daniele Ceraolo Spurio <daniele.ceraolospurio@xxxxxxxxx>
> > Cc: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
> > ---
> > tests/gem_create.c | 14 +++++++++-----
> > 1 file changed, 9 insertions(+), 5 deletions(-)
> >
> > diff --git a/tests/gem_create.c b/tests/gem_create.c
> > index de7b8209..655f7a4e 100644
> > --- a/tests/gem_create.c
> > +++ b/tests/gem_create.c
> > @@ -70,17 +70,18 @@ struct local_i915_gem_create_v2 {
> > uint32_t pad;
> > #define I915_CREATE_PLACEMENT_STOLEN (1<<0)
> > uint32_t flags;
> > -} create;
> > +};
> >
> > #define LOCAL_IOCTL_I915_GEM_CREATE DRM_IOWR(DRM_COMMAND_BASE + DRM_I915_GEM_CREATE, struct local_i915_gem_create_v2)
> >
> > static void invalid_flag_test(int fd)
> > {
> > + struct local_i915_gem_create_v2 create;
> > int ret;
> >
> > gem_require_stolen_support(fd);
> >
> > - create.handle = 0;
> > + CLEAR(create);
> > create.size = PAGE_SIZE;
> > create.flags = ~I915_CREATE_PLACEMENT_STOLEN;
> > ret = drmIoctl(fd, LOCAL_IOCTL_I915_GEM_CREATE, &create);
> > @@ -95,10 +96,13 @@ static void invalid_flag_test(int fd)
>
> First chunk,
> Reviewed-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
>
> Second chunk, I think we should fix __gem_create() to follow the
> convention of returning the error code.
That's indeed better, I reviewed it. Please merge it first, I'll rebase
these two patches.
Regards, Joonas
--
Joonas Lahtinen
Open Source Technology Center
Intel Corporation
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
