Commit c216f12b authored by Igor Druzhinin's avatar Igor Druzhinin Committed by Zhenyu Wang

drm/i915/gvt: fix high-order allocation failure on late load

If the module happens to be loaded later at runtime there is a chance
memory is already fragmented enough to fail allocation of firmware
blob storage and consequently GVT init. Since it doesn't seem to be
necessary to have the blob contiguous, use vmalloc() instead to avoid
the issue.
Reviewed-by: default avatarZhenyu Wang <zhenyuw@linux.intel.com>
Signed-off-by: default avatarIgor Druzhinin <igor.druzhinin@citrix.com>
Signed-off-by: default avatarZhenyu Wang <zhenyuw@linux.intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/1579723824-25711-1-git-send-email-igor.druzhinin@citrix.com
parent 5e822e44
...@@ -146,7 +146,7 @@ void intel_gvt_free_firmware(struct intel_gvt *gvt) ...@@ -146,7 +146,7 @@ void intel_gvt_free_firmware(struct intel_gvt *gvt)
clean_firmware_sysfs(gvt); clean_firmware_sysfs(gvt);
kfree(gvt->firmware.cfg_space); kfree(gvt->firmware.cfg_space);
kfree(gvt->firmware.mmio); vfree(gvt->firmware.mmio);
} }
static int verify_firmware(struct intel_gvt *gvt, static int verify_firmware(struct intel_gvt *gvt,
...@@ -229,7 +229,7 @@ int intel_gvt_load_firmware(struct intel_gvt *gvt) ...@@ -229,7 +229,7 @@ int intel_gvt_load_firmware(struct intel_gvt *gvt)
firmware->cfg_space = mem; firmware->cfg_space = mem;
mem = kmalloc(info->mmio_size, GFP_KERNEL); mem = vmalloc(info->mmio_size);
if (!mem) { if (!mem) {
kfree(path); kfree(path);
kfree(firmware->cfg_space); kfree(firmware->cfg_space);
......
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