When a user tries to allocate too many or too big vGPUs and runs out of graphics memory, the resulting error message is not actionable and looks like an internal error.
Change the error message to clearly point out what actions a user can take to resolve this situation.
Cc: Thomas Prescher thomas.prescher@cyberus-technology.de Cc: Zhenyu Wang zhenyuw@linux.intel.com Signed-off-by: Julian Stecklina julian.stecklina@cyberus-technology.de --- drivers/gpu/drm/i915/gvt/aperture_gm.c | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/i915/gvt/aperture_gm.c b/drivers/gpu/drm/i915/gvt/aperture_gm.c index 0d6d598713082..5c5c8e871dae2 100644 --- a/drivers/gpu/drm/i915/gvt/aperture_gm.c +++ b/drivers/gpu/drm/i915/gvt/aperture_gm.c @@ -69,9 +69,12 @@ static int alloc_gm(struct intel_vgpu *vgpu, bool high_gm) start, end, flags); mmio_hw_access_post(gt); mutex_unlock(>->ggtt->vm.mutex); - if (ret) - gvt_err("fail to alloc %s gm space from host\n", - high_gm ? "high" : "low"); + if (ret) { + gvt_err("vgpu%d: failed to allocate %s gm space from host\n", + vgpu->id, high_gm ? "high" : "low"); + gvt_err("vgpu%d: destroying vGPUs, decreasing vGPU memory size or increasing GPU aperture size may resolve this\n", + vgpu->id); + }
return ret; }