When not using GuC submission, the ring buffer size for GVT context is
512KB which is the max size. When switching to GuC submission, the ring
buffer size is required to be less than 16KB. So use the GVT context
default ring buffer size if GuC submission is enabled.
Signed-off-by: Chuanxiao Dong
: kevin.t...@intel.com
Cc: joonas.lahti...@linux.intel.com
Cc: ch...@chris-wilson.co.uk
Signed-off-by: Chuanxiao Dong <chuanxiao.d...@intel.com>
---
drivers/gpu/drm/i915/i915_guc_submission.c | 4
drivers/gpu/drm/i915/intel_lrc.c | 27 ++-
drivers/gpu/dr
...@intel.com
Cc: joonas.lahti...@linux.intel.com
Cc: ch...@chris-wilson.co.uk
Signed-off-by: Chuanxiao Dong <chuanxiao.d...@intel.com>
---
drivers/gpu/drm/i915/i915_guc_submission.c | 4
drivers/gpu/drm/i915/intel_gvt.h | 12
drivers/gpu/drm/i915/intel_lrc.c
)
Cc: ch...@chris-wilson.co.uk
Signed-off-by: Chuanxiao Dong <chuanxiao.d...@intel.com>
---
drivers/gpu/drm/i915/i915_guc_submission.c | 7 ++-
drivers/gpu/drm/i915/intel_gvt.h | 11 +++
drivers/gpu/drm/i915/intel_lrc.c | 25 -
3 files c
GVT requires force-single-submission and notification when i915
using execlist submit, and these should be extended to GuC when
i915 using GuC submit. Below two patches are used to implement this
Chuanxiao Dong (2):
drm/i915/scheduler: add gvt force-single-submission for guc
drm/i915
GVT needs single submission and cannot allow merge. So when GuC submitting
a GVT request, the next one should be submitted to guc later until the
previous one is completed. This is following the usage when using execlist
mode submission.
Cc: ch...@chris-wilson.co.uk
Signed-off-by: Chuanxiao Dong
GVT requires force-single-submission and notification when i915
using execlist submit, and these should be extended to GuC when
i915 using GuC submit. Below two patches are used to implement this
Chuanxiao Dong (2):
drm/i915/scheduler: add gvt force-single-submission for guc
drm/i915
...@intel.com
Cc: joonas.lahti...@linux.intel.com
Cc: ch...@chris-wilson.co.uk
Signed-off-by: Chuanxiao Dong <chuanxiao.d...@intel.com>
---
drivers/gpu/drm/i915/i915_guc_submission.c | 4
drivers/gpu/drm/i915/intel_gvt.h | 13 +
drivers/gpu/drm/i915/intel_lrc.c
intel_gvt_notify_context_status to intel_gvt.h (Joonas)
Cc: xiao.zh...@intel.com
Cc: kevin.t...@intel.com
Cc: joonas.lahti...@linux.intel.com
Cc: ch...@chris-wilson.co.uk
Signed-off-by: Chuanxiao Dong <chuanxiao.d...@intel.com>
---
drivers/gpu/drm/i915/i915_guc_submission.c | 4
drivers/gpu/drm/i915/intel
GVT request needs a manual mmio load/restore. Before GuC submit
a request, send notification to gvt for mmio loading. And after
the GuC finished this GVT request, notify gvt again for mmio
restore. This follows the usage when using execlists submission.
Signed-off-by: Chuanxiao Dong <chuanxia
of execlists_context_status_change
for better understanding (ZhengXiao)
Cc: xiao.zh...@intel.com
Signed-off-by: Chuanxiao Dong <chuanxiao.d...@intel.com>
---
drivers/gpu/drm/i915/i915_guc_submission.c | 12
drivers/gpu/drm/i915/intel_lrc.c | 21 +++--
d
of execlists_context_status_change
for better understanding (ZhengXiao)
v3: remove the comment as it is obvious and not friendly to
the caller (Kevin)
Cc: xiao.zh...@intel.com
Cc: kevin.t...@intel.com
Signed-off-by: Chuanxiao Dong <chuanxiao.d...@intel.com>
---
drivers/gpu/dr
...@intel.com
Cc: joonas.lahti...@linux.intel.com
Cc: ch...@chris-wilson.co.uk
Signed-off-by: Chuanxiao Dong <chuanxiao.d...@intel.com>
---
drivers/gpu/drm/i915/i915_guc_submission.c | 4
drivers/gpu/drm/i915/intel_gvt.h | 12
drivers/gpu/drm/i915/intel_lrc.c
: keep the original code implementation (Chris)
Cc: ch...@chris-wilson.co.uk
Signed-off-by: Chuanxiao Dong <chuanxiao.d...@intel.com>
---
drivers/gpu/drm/i915/i915_gem_context.h| 13 +
drivers/gpu/drm/i915/i915_guc_submission.c | 6 +-
drivers/gpu/drm/i915/intel
GVT requires force-single-submission and notification when i915
using execlist submit, and these should be extended to GuC when
i915 using GuC submit. Below two patches are used to implement this
Chuanxiao Dong (2):
drm/i915/scheduler: add gvt force-single-submission for guc
drm/i915
GVT needs single submission and cannot allow merge. So when GuC submitting
a GVT request, the next one should be submitted to guc later until the
previous one is completed. This is following the usage when using execlist
mode submission.
Cc: ch...@chris-wilson.co.uk
Signed-off-by: Chuanxiao Dong
...@intel.com
Cc: joonas.lahti...@linux.intel.com
Cc: ch...@chris-wilson.co.uk
Signed-off-by: Chuanxiao Dong <chuanxiao.d...@intel.com>
---
drivers/gpu/drm/i915/i915_guc_submission.c | 4
1 file changed, 4 insertions(+)
diff --git a/drivers/gpu/drm/i915/i915_guc_submission.c
b/drivers/gpu/dr
to intel_gvt_notify_context_status(Chris)
move intel_gvt_notify_context_status to intel_gvt.h(Joonas)
Cc: xiao.zh...@intel.com
Cc: kevin.t...@intel.com
Cc: joonas.lahti...@linux.intel.com
Cc: ch...@chris-wilson.co.uk
Signed-off-by: Chuanxiao Dong <chuanxiao.d...@intel.com>
---
drivers/gpu/dr
refactor gvt force-single-submission to proper files
v1:
make force-single-submission specific to gvt (Chris)
keep the original code implementation (Chris)
Cc: ch...@chris-wilson.co.uk
Signed-off-by: Chuanxiao Dong <chuanxiao.d...@intel.com>
---
drivers/gpu/drm/i915/i915_gem_context.
The GVT-g needs execlists to be enabled otherwise gvt should be
disabled. Add a check for enable_execlists before enabling gvt.
v2: use DRM_INFO in response to the user action
Signed-off-by: Chuanxiao Dong <chuanxiao.d...@intel.com>
---
drivers/gpu/drm/i915/intel_gvt.c | 5 +
1 file c
The GVT-g needs execlists to be enabled otherwise gvt should be
disabled. Add a check for enable_execlists before enabling gvt.
Signed-off-by: Chuanxiao Dong <chuanxiao.d...@intel.com>
---
drivers/gpu/drm/i915/intel_gvt.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/drivers/g
The ppgtt should be get directly from i915_address_space *vm instead of
vma->vm.
v2:
- add one more fix for bxt. (Chris)
Fixes: 4a234c5fae16 ("drm/i915: pass the vma to insert_entries")
Bugzilla:https://bugs.freedesktop.org/show_bug.cgi?id=101713
Signed-off-by: Chuanxiao Dong
: a1ef70e14453 ("drm/i915: Add support for per engine reset recovery")
Cc: Michel Thierry <michel.thie...@intel.com>
Signed-off-by: Chuanxiao Dong <chuanxiao.d...@intel.com>
---
drivers/gpu/drm/i915/i915_drv.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git
The ppgtt should be get directly from i915_address_space *vm instead of
vma->vm as in alias ppgtt case the vma->vm is not same with vm.
Fixes: 4a234c5fae16 ("drm/i915: pass the vma to insert_entries")
Bugzilla:https://bugs.freedesktop.org/show_bug.cgi?id=101713
Signed-off-by
initialised is fixup by the GVT shadow context as true to avoid the init
from the host because it cannot take the settings from the host. Add a
check to let host driver only overwrite it when the init callback is NULL
Cc: Chris Wilson <ch...@chris-wilson.co.uk>
Signed-off-by: Chuanxia
Currently GVT-g cannot work properly when host GuC submission
is enabled, so disable GVT in this case.
v2: update the user message (Joonas)
Cc: Zhenyu Wang <zhen...@linux.intel.com>
Signed-off-by: Chuanxiao Dong <chuanxiao.d...@intel.com>
---
drivers/gpu/drm/i915/intel_gvt.c | 5 +
: Chris Wilson <ch...@chris-wilson.co.uk>
Signed-off-by: Chuanxiao Dong <chuanxiao.d...@intel.com>
---
drivers/gpu/drm/i915/i915_drv.c | 2 ++
drivers/gpu/drm/i915/intel_gvt.c | 36 ++--
drivers/gpu/drm/i915/intel_gvt.h | 5 +
3 files changed,
co.uk>
Cc: Joonas Lahtinen <joonas.lahti...@linux.intel.com>
Cc: Chris Wilson <ch...@chris-wilson.co.uk>
Cc: Zhenyu Wang <zhen...@linux.intel.com>
Signed-off-by: Chuanxiao Dong <chuanxiao.d...@intel.com>
---
drivers/gpu/drm/i915/intel_gvt.c | 5 +
1 file changed, 5
nas.lahti...@linux.intel.com>
Cc: Joonas Lahtinen <joonas.lahti...@linux.intel.com>
Signed-off-by: Chuanxiao Dong <chuanxiao.d...@intel.com>
---
drivers/gpu/drm/i915/intel_gvt.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/i915/intel_gvt.
In this series, options sanitize API is added for GVT-g, and GVT-g init
will fail with -EIO if detected incompatible i915 parameters.
Chuanxiao Dong (3):
drm/i915/gvt: Add gvt options sanitize function
drm/i915/gvt: Return -EIO if host enable_execlists not enabled when
loading GVT-g
drm
In this series, options sanitize API is added for GVT-g, and GVT-g init
will fail with -EIO if detected incompatible i915 parameters.
Chuanxiao Dong (3):
drm/i915/gvt: Add gvt options sanitize function
drm/i915/gvt: Return -EIO if host enable_execlists not enabled when
loading GVT-g
drm
x.intel.com>
Cc: Chris Wilson <ch...@chris-wilson.co.uk>
Cc: Zhenyu Wang <zhen...@linux.intel.com>
Signed-off-by: Chuanxiao Dong <chuanxiao.d...@intel.com>
---
drivers/gpu/drm/i915/intel_gvt.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/drivers/gpu/drm/i91
The intel_gvt_sanitize_options will sanitize the GVT-g related
options before doing GVT-g init.
Suggested-by: Joonas Lahtinen <joonas.lahti...@linux.intel.com>
Cc: Joonas Lahtinen <joonas.lahti...@linux.intel.com>
Signed-off-by: Chuanxiao Dong <chuanxiao.d...@intel.com>
---
dr
GVT-g relies on the enable_execlists parameter in i915. If this option
is not enabled for GVT-g, should return -EIO to make i915 driver loading
failed.
Suggested-by: Joonas Lahtinen <joonas.lahti...@linux.intel.com>
Cc: Joonas Lahtinen <joonas.lahti...@linux.intel.com>
Signed-off-b
Currently GVT-g cannot work properly when host GuC submission
is enabled, so disable GVT in this case.
Cc: Zhenyu Wang <zhen...@linux.intel.com>
Signed-off-by: Chuanxiao Dong <chuanxiao.d...@intel.com>
---
drivers/gpu/drm/i915/intel_gvt.c | 5 +
1 file changed, 5 insertions(+)
35 matches
Mail list logo