On Mon, Aug 15, 2016 at 03:25:43PM +0300, Mika Kuoppala wrote: > Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> writes: > > > On Mon, Aug 15, 2016 at 02:48:04PM +0300, Mika Kuoppala wrote: > >> From: Jesse Barnes <jbarnes@xxxxxxxxxxxxxxxx> > >> > >> Add i915_gem_context_create2_ioctl for passing flags > >> (e.g. SVM) when creating a context. > >> > >> v2: check the pad on create_context > >> v3: rebase > >> v4: i915_dma is no more. create_gvt needs flags > >> > >> Cc: Daniel Vetter <daniel.vetter@xxxxxxxx> > >> Cc: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> > >> Cc: Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx> > >> Signed-off-by: Jesse Barnes <jbarnes@xxxxxxxxxxxxxxxx> (v1) > >> Signed-off-by: Mika Kuoppala <mika.kuoppala@xxxxxxxxx> > > > > Considering we can use deferred ppgtt creation and have setparam do we > > need a new create ioctl just to set a flag? > > So like this: > > - create ctx with the default create ioctl > - set cxt param it for svm capable. > - first submit deferred creates > > And we use the setparam point for returning > error if svm context are not there. (and a call to set svm on a context after first use is illegal) That's the outline I had in my head. I am not sure if the result is cleaner - I just hope it is ;) -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
