On Fri, Jun 22, 2018 at 09:41:26AM +0100, Chris Wilson wrote:
> Quoting Hang Yuan (2018-06-22 09:18:02)
> > On Fri, Jun 22, 2018 at 08:44:50AM +0100, Chris Wilson wrote:
> > > Quoting hang.yuan@xxxxxxxxxxxxxxx (2018-06-22 08:32:57)
> > > > From: Hang Yuan <hang.yuan@xxxxxxxxxxxxxxx>
> > > >
> > > > This helps kvmgt included in initramfs and got loaded after i915.
> > > >
> > > > Signed-off-by: Hang Yuan <hang.yuan@xxxxxxxxxxxxxxx>
> > > > ---
> > > > drivers/gpu/drm/i915/i915_pci.c | 2 ++
> > > > 1 file changed, 2 insertions(+)
> > > >
> > > > diff --git a/drivers/gpu/drm/i915/i915_pci.c b/drivers/gpu/drm/i915/i915_pci.c
> > > > index 062e91b..a0fb51f 100644
> > > > --- a/drivers/gpu/drm/i915/i915_pci.c
> > > > +++ b/drivers/gpu/drm/i915/i915_pci.c
> > > > @@ -773,3 +773,5 @@ MODULE_AUTHOR("Intel Corporation");
> > > >
> > > > MODULE_DESCRIPTION(DRIVER_DESC);
> > > > MODULE_LICENSE("GPL and additional rights");
> > > > +
> > > > +MODULE_SOFTDEP("post: kvmgt");
> > >
> > > The dependency is added by gvt, so what's it doing here?
> > > -Chris
> > It wants to indicate the soft dependency between kvmgt and i915. So kvmgt can be loaded after i915 module in initramfs. Do you think it's ok?
>
> #if IS_ENABLED(CONFIG_DRM_I915_GVT_KVMGT)
> /* not in Xen. Try KVMGT */
> intel_gvt_host.mpt = try_then_request_module(
> symbol_get(kvmgt_mpt), "kvmgt");
> intel_gvt_host.hypervisor_type = INTEL_GVT_HYPERVISOR_KVM;
> #endif
>
> Does not look all too soft, conditional and a pre-dependency not post.
> Also only required if gvt + gvt-kvmgt is compiled in. Not part of the
> base i915 module.
> -Chris
Thanks for the comments. I will re-consider the implementation.
Regards,
Henry
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
