Commit aee2eeeb authored by Wambui Karuga's avatar Wambui Karuga Committed by Jani Nikula

drm/i915: conversion to new logging macros in i915/i915_vgpu.c

Replace the use of printk based logging macros with the struct
drm_device based macros in i915/i915_vgpu.c
Signed-off-by: default avatarWambui Karuga <wambui.karugax@gmail.com>
Signed-off-by: default avatarJani Nikula <jani.nikula@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/45e8bffff8cbffd72ed41901c3db9f7f6dbe79f3.1578560355.git.wambui.karugax@gmail.com
parent 77b20896
...@@ -77,7 +77,8 @@ void i915_detect_vgpu(struct drm_i915_private *dev_priv) ...@@ -77,7 +77,8 @@ void i915_detect_vgpu(struct drm_i915_private *dev_priv)
shared_area = pci_iomap_range(pdev, 0, VGT_PVINFO_PAGE, VGT_PVINFO_SIZE); shared_area = pci_iomap_range(pdev, 0, VGT_PVINFO_PAGE, VGT_PVINFO_SIZE);
if (!shared_area) { if (!shared_area) {
DRM_ERROR("failed to map MMIO bar to check for VGT\n"); drm_err(&dev_priv->drm,
"failed to map MMIO bar to check for VGT\n");
return; return;
} }
...@@ -87,7 +88,7 @@ void i915_detect_vgpu(struct drm_i915_private *dev_priv) ...@@ -87,7 +88,7 @@ void i915_detect_vgpu(struct drm_i915_private *dev_priv)
version_major = readw(shared_area + vgtif_offset(version_major)); version_major = readw(shared_area + vgtif_offset(version_major));
if (version_major < VGT_VERSION_MAJOR) { if (version_major < VGT_VERSION_MAJOR) {
DRM_INFO("VGT interface version mismatch!\n"); drm_info(&dev_priv->drm, "VGT interface version mismatch!\n");
goto out; goto out;
} }
...@@ -95,7 +96,7 @@ void i915_detect_vgpu(struct drm_i915_private *dev_priv) ...@@ -95,7 +96,7 @@ void i915_detect_vgpu(struct drm_i915_private *dev_priv)
dev_priv->vgpu.active = true; dev_priv->vgpu.active = true;
mutex_init(&dev_priv->vgpu.lock); mutex_init(&dev_priv->vgpu.lock);
DRM_INFO("Virtual GPU for Intel GVT-g detected.\n"); drm_info(&dev_priv->drm, "Virtual GPU for Intel GVT-g detected.\n");
out: out:
pci_iounmap(pdev, shared_area); pci_iounmap(pdev, shared_area);
...@@ -120,13 +121,15 @@ static struct _balloon_info_ bl_info; ...@@ -120,13 +121,15 @@ static struct _balloon_info_ bl_info;
static void vgt_deballoon_space(struct i915_ggtt *ggtt, static void vgt_deballoon_space(struct i915_ggtt *ggtt,
struct drm_mm_node *node) struct drm_mm_node *node)
{ {
struct drm_i915_private *dev_priv = ggtt->vm.i915;
if (!drm_mm_node_allocated(node)) if (!drm_mm_node_allocated(node))
return; return;
DRM_DEBUG_DRIVER("deballoon space: range [0x%llx - 0x%llx] %llu KiB.\n", drm_dbg(&dev_priv->drm,
node->start, "deballoon space: range [0x%llx - 0x%llx] %llu KiB.\n",
node->start + node->size, node->start,
node->size / 1024); node->start + node->size,
node->size / 1024);
ggtt->vm.reserved -= node->size; ggtt->vm.reserved -= node->size;
drm_mm_remove_node(node); drm_mm_remove_node(node);
...@@ -141,12 +144,13 @@ static void vgt_deballoon_space(struct i915_ggtt *ggtt, ...@@ -141,12 +144,13 @@ static void vgt_deballoon_space(struct i915_ggtt *ggtt,
*/ */
void intel_vgt_deballoon(struct i915_ggtt *ggtt) void intel_vgt_deballoon(struct i915_ggtt *ggtt)
{ {
struct drm_i915_private *dev_priv = ggtt->vm.i915;
int i; int i;
if (!intel_vgpu_active(ggtt->vm.i915)) if (!intel_vgpu_active(ggtt->vm.i915))
return; return;
DRM_DEBUG("VGT deballoon.\n"); drm_dbg(&dev_priv->drm, "VGT deballoon.\n");
for (i = 0; i < 4; i++) for (i = 0; i < 4; i++)
vgt_deballoon_space(ggtt, &bl_info.space[i]); vgt_deballoon_space(ggtt, &bl_info.space[i]);
...@@ -156,13 +160,15 @@ static int vgt_balloon_space(struct i915_ggtt *ggtt, ...@@ -156,13 +160,15 @@ static int vgt_balloon_space(struct i915_ggtt *ggtt,
struct drm_mm_node *node, struct drm_mm_node *node,
unsigned long start, unsigned long end) unsigned long start, unsigned long end)
{ {
struct drm_i915_private *dev_priv = ggtt->vm.i915;
unsigned long size = end - start; unsigned long size = end - start;
int ret; int ret;
if (start >= end) if (start >= end)
return -EINVAL; return -EINVAL;
DRM_INFO("balloon space: range [ 0x%lx - 0x%lx ] %lu KiB.\n", drm_info(&dev_priv->drm,
"balloon space: range [ 0x%lx - 0x%lx ] %lu KiB.\n",
start, end, size / 1024); start, end, size / 1024);
ret = i915_gem_gtt_reserve(&ggtt->vm, node, ret = i915_gem_gtt_reserve(&ggtt->vm, node,
size, start, I915_COLOR_UNEVICTABLE, size, start, I915_COLOR_UNEVICTABLE,
...@@ -219,7 +225,8 @@ static int vgt_balloon_space(struct i915_ggtt *ggtt, ...@@ -219,7 +225,8 @@ static int vgt_balloon_space(struct i915_ggtt *ggtt,
*/ */
int intel_vgt_balloon(struct i915_ggtt *ggtt) int intel_vgt_balloon(struct i915_ggtt *ggtt)
{ {
struct intel_uncore *uncore = &ggtt->vm.i915->uncore; struct drm_i915_private *dev_priv = ggtt->vm.i915;
struct intel_uncore *uncore = &dev_priv->uncore;
unsigned long ggtt_end = ggtt->vm.total; unsigned long ggtt_end = ggtt->vm.total;
unsigned long mappable_base, mappable_size, mappable_end; unsigned long mappable_base, mappable_size, mappable_end;
...@@ -241,16 +248,18 @@ int intel_vgt_balloon(struct i915_ggtt *ggtt) ...@@ -241,16 +248,18 @@ int intel_vgt_balloon(struct i915_ggtt *ggtt)
mappable_end = mappable_base + mappable_size; mappable_end = mappable_base + mappable_size;
unmappable_end = unmappable_base + unmappable_size; unmappable_end = unmappable_base + unmappable_size;
DRM_INFO("VGT ballooning configuration:\n"); drm_info(&dev_priv->drm, "VGT ballooning configuration:\n");
DRM_INFO("Mappable graphic memory: base 0x%lx size %ldKiB\n", drm_info(&dev_priv->drm,
"Mappable graphic memory: base 0x%lx size %ldKiB\n",
mappable_base, mappable_size / 1024); mappable_base, mappable_size / 1024);
DRM_INFO("Unmappable graphic memory: base 0x%lx size %ldKiB\n", drm_info(&dev_priv->drm,
"Unmappable graphic memory: base 0x%lx size %ldKiB\n",
unmappable_base, unmappable_size / 1024); unmappable_base, unmappable_size / 1024);
if (mappable_end > ggtt->mappable_end || if (mappable_end > ggtt->mappable_end ||
unmappable_base < ggtt->mappable_end || unmappable_base < ggtt->mappable_end ||
unmappable_end > ggtt_end) { unmappable_end > ggtt_end) {
DRM_ERROR("Invalid ballooning configuration!\n"); drm_err(&dev_priv->drm, "Invalid ballooning configuration!\n");
return -EINVAL; return -EINVAL;
} }
...@@ -287,7 +296,7 @@ int intel_vgt_balloon(struct i915_ggtt *ggtt) ...@@ -287,7 +296,7 @@ int intel_vgt_balloon(struct i915_ggtt *ggtt)
goto err_below_mappable; goto err_below_mappable;
} }
DRM_INFO("VGT balloon successfully\n"); drm_info(&dev_priv->drm, "VGT balloon successfully\n");
return 0; return 0;
err_below_mappable: err_below_mappable:
...@@ -297,6 +306,6 @@ int intel_vgt_balloon(struct i915_ggtt *ggtt) ...@@ -297,6 +306,6 @@ int intel_vgt_balloon(struct i915_ggtt *ggtt)
err_upon_mappable: err_upon_mappable:
vgt_deballoon_space(ggtt, &bl_info.space[2]); vgt_deballoon_space(ggtt, &bl_info.space[2]);
err: err:
DRM_ERROR("VGT balloon fail\n"); drm_err(&dev_priv->drm, "VGT balloon fail\n");
return ret; return ret;
} }
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