提交 1c66a12a 编写于 作者: M Matt Roper 提交者: Joonas Lahtinen

drm/i915: Handle each GT on init/release and suspend/resume

In preparation for enabling a second GT, there are a number of GT/uncore
operations that happen during initialization or suspend flows that need
to be performed on each GT, not just the primary,

Cc: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
Signed-off-by: NMatt Roper <matthew.d.roper@intel.com>
Reviewed-by: NDaniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20220906234934.3655440-10-matthew.d.roper@intel.comSigned-off-by: NJoonas Lahtinen <joonas.lahtinen@linux.intel.com>
上级 cfb0fa42
...@@ -310,8 +310,13 @@ static void intel_detect_preproduction_hw(struct drm_i915_private *dev_priv) ...@@ -310,8 +310,13 @@ static void intel_detect_preproduction_hw(struct drm_i915_private *dev_priv)
static void sanitize_gpu(struct drm_i915_private *i915) static void sanitize_gpu(struct drm_i915_private *i915)
{ {
if (!INTEL_INFO(i915)->gpu_reset_clobbers_display) if (!INTEL_INFO(i915)->gpu_reset_clobbers_display) {
__intel_gt_reset(to_gt(i915), ALL_ENGINES); struct intel_gt *gt;
unsigned int i;
for_each_gt(gt, i915, i)
__intel_gt_reset(gt, ALL_ENGINES);
}
} }
/** /**
...@@ -715,6 +720,8 @@ static void i915_driver_hw_remove(struct drm_i915_private *dev_priv) ...@@ -715,6 +720,8 @@ static void i915_driver_hw_remove(struct drm_i915_private *dev_priv)
static void i915_driver_register(struct drm_i915_private *dev_priv) static void i915_driver_register(struct drm_i915_private *dev_priv)
{ {
struct drm_device *dev = &dev_priv->drm; struct drm_device *dev = &dev_priv->drm;
struct intel_gt *gt;
unsigned int i;
i915_gem_driver_register(dev_priv); i915_gem_driver_register(dev_priv);
i915_pmu_register(dev_priv); i915_pmu_register(dev_priv);
...@@ -734,7 +741,8 @@ static void i915_driver_register(struct drm_i915_private *dev_priv) ...@@ -734,7 +741,8 @@ static void i915_driver_register(struct drm_i915_private *dev_priv)
/* Depends on sysfs having been initialized */ /* Depends on sysfs having been initialized */
i915_perf_register(dev_priv); i915_perf_register(dev_priv);
intel_gt_driver_register(to_gt(dev_priv)); for_each_gt(gt, dev_priv, i)
intel_gt_driver_register(gt);
intel_display_driver_register(dev_priv); intel_display_driver_register(dev_priv);
...@@ -753,6 +761,9 @@ static void i915_driver_register(struct drm_i915_private *dev_priv) ...@@ -753,6 +761,9 @@ static void i915_driver_register(struct drm_i915_private *dev_priv)
*/ */
static void i915_driver_unregister(struct drm_i915_private *dev_priv) static void i915_driver_unregister(struct drm_i915_private *dev_priv)
{ {
struct intel_gt *gt;
unsigned int i;
i915_switcheroo_unregister(dev_priv); i915_switcheroo_unregister(dev_priv);
intel_unregister_dsm_handler(); intel_unregister_dsm_handler();
...@@ -762,7 +773,8 @@ static void i915_driver_unregister(struct drm_i915_private *dev_priv) ...@@ -762,7 +773,8 @@ static void i915_driver_unregister(struct drm_i915_private *dev_priv)
intel_display_driver_unregister(dev_priv); intel_display_driver_unregister(dev_priv);
intel_gt_driver_unregister(to_gt(dev_priv)); for_each_gt(gt, dev_priv, i)
intel_gt_driver_unregister(gt);
i915_perf_unregister(dev_priv); i915_perf_unregister(dev_priv);
i915_pmu_unregister(dev_priv); i915_pmu_unregister(dev_priv);
...@@ -784,6 +796,8 @@ static void i915_welcome_messages(struct drm_i915_private *dev_priv) ...@@ -784,6 +796,8 @@ static void i915_welcome_messages(struct drm_i915_private *dev_priv)
{ {
if (drm_debug_enabled(DRM_UT_DRIVER)) { if (drm_debug_enabled(DRM_UT_DRIVER)) {
struct drm_printer p = drm_debug_printer("i915 device info:"); struct drm_printer p = drm_debug_printer("i915 device info:");
struct intel_gt *gt;
unsigned int i;
drm_printf(&p, "pciid=0x%04x rev=0x%02x platform=%s (subplatform=0x%x) gen=%i\n", drm_printf(&p, "pciid=0x%04x rev=0x%02x platform=%s (subplatform=0x%x) gen=%i\n",
INTEL_DEVID(dev_priv), INTEL_DEVID(dev_priv),
...@@ -796,7 +810,8 @@ static void i915_welcome_messages(struct drm_i915_private *dev_priv) ...@@ -796,7 +810,8 @@ static void i915_welcome_messages(struct drm_i915_private *dev_priv)
intel_device_info_print_static(INTEL_INFO(dev_priv), &p); intel_device_info_print_static(INTEL_INFO(dev_priv), &p);
intel_device_info_print_runtime(RUNTIME_INFO(dev_priv), &p); intel_device_info_print_runtime(RUNTIME_INFO(dev_priv), &p);
i915_print_iommu_status(dev_priv, &p); i915_print_iommu_status(dev_priv, &p);
intel_gt_info_print(&to_gt(dev_priv)->info, &p); for_each_gt(gt, dev_priv, i)
intel_gt_info_print(&gt->info, &p);
} }
if (IS_ENABLED(CONFIG_DRM_I915_DEBUG)) if (IS_ENABLED(CONFIG_DRM_I915_DEBUG))
...@@ -1206,13 +1221,15 @@ static int i915_drm_suspend_late(struct drm_device *dev, bool hibernation) ...@@ -1206,13 +1221,15 @@ static int i915_drm_suspend_late(struct drm_device *dev, bool hibernation)
struct drm_i915_private *dev_priv = to_i915(dev); struct drm_i915_private *dev_priv = to_i915(dev);
struct pci_dev *pdev = to_pci_dev(dev_priv->drm.dev); struct pci_dev *pdev = to_pci_dev(dev_priv->drm.dev);
struct intel_runtime_pm *rpm = &dev_priv->runtime_pm; struct intel_runtime_pm *rpm = &dev_priv->runtime_pm;
int ret; struct intel_gt *gt;
int ret, i;
disable_rpm_wakeref_asserts(rpm); disable_rpm_wakeref_asserts(rpm);
i915_gem_suspend_late(dev_priv); i915_gem_suspend_late(dev_priv);
intel_uncore_suspend(&dev_priv->uncore); for_each_gt(gt, dev_priv, i)
intel_uncore_suspend(gt->uncore);
intel_power_domains_suspend(dev_priv, intel_power_domains_suspend(dev_priv,
get_suspend_mode(dev_priv, hibernation)); get_suspend_mode(dev_priv, hibernation));
...@@ -1352,7 +1369,8 @@ static int i915_drm_resume_early(struct drm_device *dev) ...@@ -1352,7 +1369,8 @@ static int i915_drm_resume_early(struct drm_device *dev)
{ {
struct drm_i915_private *dev_priv = to_i915(dev); struct drm_i915_private *dev_priv = to_i915(dev);
struct pci_dev *pdev = to_pci_dev(dev_priv->drm.dev); struct pci_dev *pdev = to_pci_dev(dev_priv->drm.dev);
int ret; struct intel_gt *gt;
int ret, i;
/* /*
* We have a resume ordering issue with the snd-hda driver also * We have a resume ordering issue with the snd-hda driver also
...@@ -1408,9 +1426,10 @@ static int i915_drm_resume_early(struct drm_device *dev) ...@@ -1408,9 +1426,10 @@ static int i915_drm_resume_early(struct drm_device *dev)
drm_err(&dev_priv->drm, drm_err(&dev_priv->drm,
"Resume prepare failed: %d, continuing anyway\n", ret); "Resume prepare failed: %d, continuing anyway\n", ret);
intel_uncore_resume_early(&dev_priv->uncore); for_each_gt(gt, dev_priv, i) {
intel_uncore_resume_early(gt->uncore);
intel_gt_check_and_clear_faults(to_gt(dev_priv)); intel_gt_check_and_clear_faults(gt);
}
intel_display_power_resume_early(dev_priv); intel_display_power_resume_early(dev_priv);
...@@ -1591,7 +1610,8 @@ static int intel_runtime_suspend(struct device *kdev) ...@@ -1591,7 +1610,8 @@ static int intel_runtime_suspend(struct device *kdev)
struct drm_i915_private *dev_priv = kdev_to_i915(kdev); struct drm_i915_private *dev_priv = kdev_to_i915(kdev);
struct intel_runtime_pm *rpm = &dev_priv->runtime_pm; struct intel_runtime_pm *rpm = &dev_priv->runtime_pm;
struct pci_dev *pdev = to_pci_dev(dev_priv->drm.dev); struct pci_dev *pdev = to_pci_dev(dev_priv->drm.dev);
int ret; struct intel_gt *gt;
int ret, i;
if (drm_WARN_ON_ONCE(&dev_priv->drm, !HAS_RUNTIME_PM(dev_priv))) if (drm_WARN_ON_ONCE(&dev_priv->drm, !HAS_RUNTIME_PM(dev_priv)))
return -ENODEV; return -ENODEV;
...@@ -1606,11 +1626,13 @@ static int intel_runtime_suspend(struct device *kdev) ...@@ -1606,11 +1626,13 @@ static int intel_runtime_suspend(struct device *kdev)
*/ */
i915_gem_runtime_suspend(dev_priv); i915_gem_runtime_suspend(dev_priv);
intel_gt_runtime_suspend(to_gt(dev_priv)); for_each_gt(gt, dev_priv, i)
intel_gt_runtime_suspend(gt);
intel_runtime_pm_disable_interrupts(dev_priv); intel_runtime_pm_disable_interrupts(dev_priv);
intel_uncore_suspend(&dev_priv->uncore); for_each_gt(gt, dev_priv, i)
intel_uncore_suspend(gt->uncore);
intel_display_power_suspend(dev_priv); intel_display_power_suspend(dev_priv);
...@@ -1681,7 +1703,8 @@ static int intel_runtime_resume(struct device *kdev) ...@@ -1681,7 +1703,8 @@ static int intel_runtime_resume(struct device *kdev)
struct drm_i915_private *dev_priv = kdev_to_i915(kdev); struct drm_i915_private *dev_priv = kdev_to_i915(kdev);
struct intel_runtime_pm *rpm = &dev_priv->runtime_pm; struct intel_runtime_pm *rpm = &dev_priv->runtime_pm;
struct pci_dev *pdev = to_pci_dev(dev_priv->drm.dev); struct pci_dev *pdev = to_pci_dev(dev_priv->drm.dev);
int ret; struct intel_gt *gt;
int ret, i;
if (drm_WARN_ON_ONCE(&dev_priv->drm, !HAS_RUNTIME_PM(dev_priv))) if (drm_WARN_ON_ONCE(&dev_priv->drm, !HAS_RUNTIME_PM(dev_priv)))
return -ENODEV; return -ENODEV;
...@@ -1702,7 +1725,8 @@ static int intel_runtime_resume(struct device *kdev) ...@@ -1702,7 +1725,8 @@ static int intel_runtime_resume(struct device *kdev)
ret = vlv_resume_prepare(dev_priv, true); ret = vlv_resume_prepare(dev_priv, true);
intel_uncore_runtime_resume(&dev_priv->uncore); for_each_gt(gt, dev_priv, i)
intel_uncore_runtime_resume(gt->uncore);
intel_runtime_pm_enable_interrupts(dev_priv); intel_runtime_pm_enable_interrupts(dev_priv);
...@@ -1710,7 +1734,8 @@ static int intel_runtime_resume(struct device *kdev) ...@@ -1710,7 +1734,8 @@ static int intel_runtime_resume(struct device *kdev)
* No point of rolling back things in case of an error, as the best * No point of rolling back things in case of an error, as the best
* we can do is to hope that things will still work (and disable RPM). * we can do is to hope that things will still work (and disable RPM).
*/ */
intel_gt_runtime_resume(to_gt(dev_priv)); for_each_gt(gt, dev_priv, i)
intel_gt_runtime_resume(gt);
/* /*
* On VLV/CHV display interrupts are part of the display * On VLV/CHV display interrupts are part of the display
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册