On ti, 2017-05-16 at 07:53 +0000, Dong, Chuanxiao wrote: > > > > Thanks Joonas. If to fail with -EIO, how about for the other two checks: " > > if > > (!is_supported_device(dev_priv))" and " if (!i915.enable_execlists)"? > > Currently these two cases are failed with 0 instead of -EIO. Looks like > > should > > also fail with -EIO?
!is_supported_device check should probably be done in parameter sanitization phase, it should not result in EIO, because enable_gvt is not marked as an unsafe option. enable_execlists on the other hand is a an _unsafe option and should result in -EIO. Regards, Joonas -- Joonas Lahtinen Open Source Technology Center Intel Corporation _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
