Commit 2a20b857 authored by Wachowski, Karol's avatar Wachowski, Karol Committed by Jacek Lawrynowicz

accel/ivpu: Add diagnostic messages when VPU fails to boot or suspend

Make boot/suspend failure debugging easier by dumping FW logs and error
registers.
Signed-off-by: default avatarWachowski, Karol <karol.wachowski@intel.com>
Signed-off-by: default avatarJacek Lawrynowicz <jacek.lawrynowicz@linux.intel.com>
Reviewed-by: default avatarJeffrey Hugo <quic_jhugo@quicinc.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20240115134434.493839-5-jacek.lawrynowicz@linux.intel.com
parent 8047d36f
...@@ -17,6 +17,7 @@ ...@@ -17,6 +17,7 @@
#include "ivpu_debugfs.h" #include "ivpu_debugfs.h"
#include "ivpu_drv.h" #include "ivpu_drv.h"
#include "ivpu_fw.h" #include "ivpu_fw.h"
#include "ivpu_fw_log.h"
#include "ivpu_gem.h" #include "ivpu_gem.h"
#include "ivpu_hw.h" #include "ivpu_hw.h"
#include "ivpu_ipc.h" #include "ivpu_ipc.h"
...@@ -340,8 +341,6 @@ static int ivpu_wait_for_ready(struct ivpu_device *vdev) ...@@ -340,8 +341,6 @@ static int ivpu_wait_for_ready(struct ivpu_device *vdev)
if (!ret) if (!ret)
ivpu_dbg(vdev, PM, "VPU ready message received successfully\n"); ivpu_dbg(vdev, PM, "VPU ready message received successfully\n");
else
ivpu_hw_diagnose_failure(vdev);
return ret; return ret;
} }
...@@ -369,7 +368,9 @@ int ivpu_boot(struct ivpu_device *vdev) ...@@ -369,7 +368,9 @@ int ivpu_boot(struct ivpu_device *vdev)
ret = ivpu_wait_for_ready(vdev); ret = ivpu_wait_for_ready(vdev);
if (ret) { if (ret) {
ivpu_err(vdev, "Failed to boot the firmware: %d\n", ret); ivpu_err(vdev, "Failed to boot the firmware: %d\n", ret);
ivpu_hw_diagnose_failure(vdev);
ivpu_mmu_evtq_dump(vdev); ivpu_mmu_evtq_dump(vdev);
ivpu_fw_log_dump(vdev);
return ret; return ret;
} }
......
...@@ -13,6 +13,7 @@ ...@@ -13,6 +13,7 @@
#include "ivpu_drv.h" #include "ivpu_drv.h"
#include "ivpu_hw.h" #include "ivpu_hw.h"
#include "ivpu_fw.h" #include "ivpu_fw.h"
#include "ivpu_fw_log.h"
#include "ivpu_ipc.h" #include "ivpu_ipc.h"
#include "ivpu_job.h" #include "ivpu_job.h"
#include "ivpu_jsm_msg.h" #include "ivpu_jsm_msg.h"
...@@ -247,7 +248,8 @@ int ivpu_pm_runtime_suspend_cb(struct device *dev) ...@@ -247,7 +248,8 @@ int ivpu_pm_runtime_suspend_cb(struct device *dev)
ivpu_err(vdev, "Failed to set suspend VPU: %d\n", ret); ivpu_err(vdev, "Failed to set suspend VPU: %d\n", ret);
if (!hw_is_idle) { if (!hw_is_idle) {
ivpu_warn(vdev, "VPU failed to enter idle, force suspended.\n"); ivpu_err(vdev, "VPU failed to enter idle, force suspended.\n");
ivpu_fw_log_dump(vdev);
ivpu_pm_prepare_cold_boot(vdev); ivpu_pm_prepare_cold_boot(vdev);
} else { } else {
ivpu_pm_prepare_warm_boot(vdev); ivpu_pm_prepare_warm_boot(vdev);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment