Thanks! :P. Welcome back.
On 07/12/16 18:50, Joonas Lahtinen wrote:
On ma, 2016-06-20 at 08:17 -0400, Zhi Wang wrote:
The access right of kernel param "i915.enable_gvt" should be read-only as
it only applies during module load (and is not *runtime* writable).
Cc: Chris Wilson
R
Hi folks:
Here is the gvt-fixes pull for drm-intel-fixes. It contains:
- Make DRM_I915_GVT depend on X86 (Siva Mullati)
- Clean kernel doc in gtt.c (Randy Dunlap)
This pull has been tested by: dim apply-pull drm-intel-fixes < this_email.eml
Zhi.
The following changes since commit d46f329a3f604
state.
v6:
- Remove the reference of intel_gvt_device_info.(Christoph)
- Refine the save_mmio() function. (Christoph)
Cc: Christoph Hellwig
Cc: Jason Gunthorpe
Cc: Jani Nikula
Cc: Joonas Lahtinen
Cc: Vivi Rodrigo
Cc: Zhenyu Wang
Cc: Zhi Wang
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915
From: Zhi Wang
To support the new mdev interfaces and the re-factor patches from
Christoph, which moves the GVT-g code into a dedicated module, the GVT-g
MMIO tracking table needs to be separated from GVT-g.
v7:
- Keep the marcos of device generation in GVT-g. (Christoph, Jani)
v6:
- Move
The code of saving initial HW state snapshot has been moved into i915.
Let the GVT-g core logic use that snapshot.
Cc: Christoph Hellwig
Cc: Jason Gunthorpe
Cc: Jani Nikula
Cc: Joonas Lahtinen
Cc: Vivi Rodrigo
Cc: Zhenyu Wang
Cc: Zhi Wang
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915
Thanks. Let me fix that. :)
On 3/31/22 04:33, Christoph Hellwig wrote:
On Thu, Mar 31, 2022 at 08:04:04AM +, Wang, Zhi A wrote:
Hi Chris:
Thanks for the testing. Can you attach the dmesg? I tested mostly on my skylake
desktop with some 3D workload.
Sure, I should have done that from the
ed on. (Jani)
v2:
- Implement a mmio table instead of generating it by marco in i915. (Jani)
Zhi Wang (3):
i915/gvt: Separate the MMIO tracking table from GVT-g
i915/gvt: Save the initial HW state snapshot in i915
i915/gvt: Use the initial HW state snapshot saved in i915
drivers/gpu/drm
state.
---
v6:
- Remove the reference of intel_gvt_device_info.(Christoph)
- Refine the save_mmio() function. (Christoph)
Cc: Christoph Hellwig
Cc: Jason Gunthorpe
Cc: Jani Nikula
Cc: Joonas Lahtinen
Cc: Vivi Rodrigo
Cc: Zhenyu Wang
Cc: Zhi Wang
Signed-off-by: Zhi Wang
---
drivers/gpu/drm
The code of saving initial HW state snapshot has been moved into i915.
Let the GVT-g core logic use that snapshot.
Cc: Christoph Hellwig
Cc: Jason Gunthorpe
Cc: Jani Nikula
Cc: Joonas Lahtinen
Cc: Vivi Rodrigo
Cc: Zhenyu Wang
Cc: Zhi Wang
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915
From: Zhi Wang
To support the new mdev interfaces and the re-factor patches from
Christoph, which moves the GVT-g code into a dedicated module, the GVT-g
MMIO tracking table needs to be separated from GVT-g.
---
v8:
- Use SPDX header in the intel_gvt_mmio_table.c
- Reference the gvt.h with
when CONFIG_DRM_I915_WERROR is turned on. (Jani)
v2:
- Implement a mmio table instead of generating it by marco in i915. (Jani)
Zhi Wang (3):
i915/gvt: Separate the MMIO tracking table from GVT-g
i915/gvt: Save the initial HW state snapshot in i915
i915/gvt: Use the initial HW state snapshot
state.
v6:
- Remove the reference of intel_gvt_device_info.(Christoph)
- Refine the save_mmio() function. (Christoph)
Cc: Christoph Hellwig
Cc: Jason Gunthorpe
Cc: Jani Nikula
Cc: Joonas Lahtinen
Cc: Vivi Rodrigo
Cc: Zhenyu Wang
Cc: Zhi Wang
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915
From: Zhi Wang
To support the new mdev interfaces and the re-factor patches from
Christoph, which moves the GVT-g code into a dedicated module, the GVT-g
MMIO tracking table needs to be separated from GVT-g.
v9:
- Fix a problem might cause kernel panic.
v8:
- Use SPDX header in the
The code of saving initial HW state snapshot has been moved into i915.
Let the GVT-g core logic use that snapshot.
Cc: Christoph Hellwig
Cc: Jason Gunthorpe
Cc: Jani Nikula
Cc: Joonas Lahtinen
Cc: Vivi Rodrigo
Cc: Zhenyu Wang
Cc: Zhi Wang
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915
When CONFIG_DRM_I915_DEBUG_RUNTIME and CONFIG_DRM_I915_DEBUG_PM
are enabled, two more extra symols in i915 are required to be
exported.
Cc: Jani Nikula
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/intel_gvt.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/gpu/drm/i915
After the refactor of GVT-g, the reference of intel_gvt_match_device()
only happens in handlers.c. Make it static to let the compiler be
happy.
Cc: Jason Gunthorpe
Cc: Jani Nikula
Cc: Robert Beckett
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/gvt/handlers.c | 2 +-
1 file changed, 1
A compiling error was reported when CONFIG_DRM_I915_DEBUG_RUNTIME_PM=n.
Fix the problem by using the pre-defined macro.
Cc: Jason Gunthorpe
Cc: Jani Nikula
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/intel_gvt.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/gpu/drm/i915
From: Zhi Wang
Save the inital HW state snapshot in i915 so that the rest code of GVT-g
can be moved into a dedicated module while it can still get a clean
initial HW state saved at the correct time during the initialization of
i915. The futhrer vGPU created by GVT-g will use this HW state as
From: Zhi Wang
To support the new mdev interfaces and the re-factor patches from
Christoph, which moves the GVT-g code into a dedicated module, the GVT-g
MMIO snapshot still needs to be saved in i915 so that the inital clean HW
state can be used for the further vGPU. Seperate the tracked MMIO
From: Zhi Wang
The code of saving initial HW state snapshot has been moved into i915.
Let the GVT-g core logic use that snapshot.
Cc: Joonas Lahtinen
Cc: Jani Nikula
Cc: Rodrigo Vivi
Cc: Zhenyu Wang
Cc: Zhi Wang
Cc: Christoph Hellwig
Cc: Jason Gunthorpe
Signed-off-by: Zhi Wang
From: Zhi Wang
To support the new mdev interfaces and the re-factor patches from
Christoph, which moves the GVT-g code into a dedicated module, the GVT-g
initialization path has to be seperated into two phases:
a) Early initialization.
The early initialization of GVT requires to be done when
From: Zhi Wang
To support the early init of GVT-g, which will be put in i915, after the
GVT-g is moved into a dedicated module, we need to save the MMIO snapshot
in the early init of GVT-g, when the HW hasn't been touched.
Cc: Christoph Hellwig
Cc: Jason Gunthorpe
Cc: Jani Nikula
Cc: J
From: Zhi Wang
To support the early init of GVT-g, which will be put in i915, after the
GVT-g is moved into a dedicated module, we need to save the MMIO snapshot
in the early init of GVT-g, when the HW hasn't been touched.
v3:
- Fix errors when CONFIG_DRM_I915_WERROR is turned on. (Jani
From: Zhi Wang
To support the new mdev interfaces and the re-factor patches from
Christoph, which moves the GVT-g code into a dedicated module, the GVT-g
initialization path has to be seperated into two phases:
a) Early initialization.
The early initialization of GVT requires to be done when
From: Zhi Wang
To support the early init of GVT-g, which will be put in i915, after the
GVT-g is moved into a dedicated module, we need to save the MMIO snapshot
in the early init of GVT-g, when the HW hasn't been touched.
v3:
- Fix errors when CONFIG_DRM_I915_WERROR is turned on. (Jani
From: Zhi Wang
To support the new mdev interfaces and the re-factor patches from
Christoph, which moves the GVT-g code into a dedicated module, the GVT-g
initialization path has to be separated into two phases:
a) Early initialization.
The early initialization of GVT requires to be done when
From: Zhi Wang
To support the new mdev interfaces and the re-factor patches from
Christoph, which moves the GVT-g code into a dedicated module, the GVT-g
initialization path has to be separated into two phases:
a) Early initialization.
The early initialization of GVT requires to be done when
state.
Cc: Christoph Hellwig
Cc: Jason Gunthorpe
Cc: Jani Nikula
Cc: Joonas Lahtinen
Cc: Vivi Rodrigo
Cc: Zhenyu Wang
Cc: Zhi Wang
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/i915_drv.h | 2 +
drivers/gpu/drm/i915/intel_gvt.c | 110 ++-
2 files changed, 110
The code of saving initial HW state snapshot has been moved into i915.
Let the GVT-g core logic use that snapshot.
Cc: Christoph Hellwig
Cc: Jason Gunthorpe
Cc: Jani Nikula
Cc: Joonas Lahtinen
Cc: Vivi Rodrigo
Cc: Zhenyu Wang
Cc: Zhi Wang
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915
On 2/7/22 05:48, Jani Nikula wrote:
On Mon, 07 Feb 2022, Christoph Hellwig wrote:
On Mon, Feb 07, 2022 at 08:28:13AM +, Wang, Zhi A wrote:
1) About having the mmio_table.h, I would like to keep the stuff in a
dedicated header as putting them in intel_gvt.h might needs i915 guys
to maintai
state.
v6:
- Remove the reference of intel_gvt_device_info.(Christoph)
- Refine the save_mmio() function. (Christoph)
Cc: Christoph Hellwig
Cc: Jason Gunthorpe
Cc: Jani Nikula
Cc: Joonas Lahtinen
Cc: Vivi Rodrigo
Cc: Zhenyu Wang
Cc: Zhi Wang
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915
From: Zhi Wang
To support the new mdev interfaces and the re-factor patches from
Christoph, which moves the GVT-g code into a dedicated module, the GVT-g
initialization path has to be separated into two phases:
a) Early initialization.
The early initialization of GVT requires to be done when
The code of saving initial HW state snapshot has been moved into i915.
Let the GVT-g core logic use that snapshot.
Cc: Christoph Hellwig
Cc: Jason Gunthorpe
Cc: Jani Nikula
Cc: Joonas Lahtinen
Cc: Vivi Rodrigo
Cc: Zhenyu Wang
Cc: Zhi Wang
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915
-by: Zhi Wang
---
drivers/gpu/drm/i915/gvt/scheduler.c | 68 ++--
1 file changed, 57 insertions(+), 11 deletions(-)
diff --git a/drivers/gpu/drm/i915/gvt/scheduler.c
b/drivers/gpu/drm/i915/gvt/scheduler.c
index 1570eb8..fe7ee10 100644
--- a/drivers/gpu/drm/i915/gvt
Hi:
Here is the pull request of gvt-next for 4.17 with some new features and
optimizations.
Thanks,
Zhi.
--
The following changes since commit fadec6eefe232696c5c471b40df33e6db616e854:
drm/i915: Update DRIVER_DATE to 20180413 (2018-04-13 12:20:58 +0300)
are available in the git repository
Sorry it's gvt-next, so it should be for 4.18.
Thanks,
Zhi.
On 04/19/18 17:17, Zhi Wang wrote:
Hi:
Here is the pull request of gvt-next for 4.17 with some new features and
optimizations.
Thanks,
Zhi.
--
The following changes since commit
fadec6eefe232696c5c471b40df33e6db616e854:
Thanks, Let me discuss with Zhenyu about how to deal with this. It must
be the git rebase I've done which causes the commiter change without new
signoff-by.
Thanks,
Zhi.
On 04/19/18 17:34, Jani Nikula wrote:
On Thu, 19 Apr 2018, Zhi Wang wrote:
Hi:
Here is the pull request of gvt-nex
Weird. I try to apply the patches one by one on
drm-intel-next-2018-04-13. I didn't get any conflicts... Let me dig more..
On 04/19/18 17:50, Zhi Wang wrote:
Thanks, Let me discuss with Zhenyu about how to deal with this. It must
be the git rebase I've done which causes the commi
The following changes since commit fadec6eefe232696c5c471b40df33e6db616e854:
drm/i915: Update DRIVER_DATE to 20180413 (2018-04-13 12:20:58 +0300)
are available in the git repository at:
https://github.com/intel/gvt-linux.git tags/gvt-next-2018-04-23
for you to fetch changes up to 3eda0d22e
Hi Jani:
I picked out the patch which causes conflicts and may put it back in the
next back merge from drm-intel-next to gvt-next. So there shouldn't be
any conflict in this pull. Thanks for your efforts.
Thanks,
Zhi.
On 04/23/18 16:11, Zhi Wang wrote:
The following changes since c
Thanks! :)
On 04/23/18 18:25, Jani Nikula wrote:
On Mon, 23 Apr 2018, Zhi Wang wrote:
Hi Jani:
I picked out the patch which causes conflicts and may put it back in the
next back merge from drm-intel-next to gvt-next. So there shouldn't be
any conflict in this pull. Thanks for your ef
ption.
v2:
- refine description(Zhenyu)
Signed-off-by: Weinan Li
Signed-off-by: Zhenyu Wang
Signed-off-by: Zhi Wang
---
has a dependency with the following patch which is missing in git tag
drm-next-intel-2018-04-13:
---
commit b7268c5eed0ab4f052d614b4b0e3fe8a51c9d5a1
Author: Chris Wilson
Date
m/i915/gvt: set max priority for gvt context"
Zhao Yan (3):
drm/i915/gvt: let NOPID be the default value of force_to_nonpriv
registers
drm/i915/gvt: do not return error on handling force_to_nonpriv
registers
drm/i915/gvt: let force_to_nonpriv cmd handler only valid for
Hi Joonas:
Can you change GEN8_LR_CONTEXT_RENDER_SIZE = (19 * PAGE_SIZE)? Then we
don't need the hack in GVT-g. :P Actually it's 19 pages not 20 pages on BDW.
Thanks,
Zhi.
于 04/26/17 17:11, Joonas Lahtinen 写道:
Pre-calculate engine context size based on engine class and device
generation an
Hi Harsh:
Thanks for your help last time. You show us the size of context
image is actual 19 pages on BDW. Is it the same also on CHV?
Thanks,
Zhi.
于 04/26/17 17:52, Joonas Lahtinen 写道:
On ke, 2017-04-26 at 17:10 +0800, Zhi Wang wrote:
Hi Joonas:
Can you change
,
Zhi.
于 04/26/17 17:52, Joonas Lahtinen 写道:
On ke, 2017-04-26 at 17:10 +0800, Zhi Wang wrote:
Hi Joonas:
Can you change GEN8_LR_CONTEXT_RENDER_SIZE = (19 * PAGE_SIZE)?
Then we don't need the hack in GVT-g. :P Actually it's 19 pages not
20 pages on BDW.
The exception is only ma
KBL
halo is weird, not the same with other KBL SKUs.
Thanks,
Zhi.
于 04/27/17 00:20, Daniele Ceraolo Spurio 写道:
On 26/04/17 02:57, Zhi Wang wrote:
Uh...sorry for not mentioning that before:), and stolen memory is not my
business. :(
Actually we root-caused it.
This is how we found this case
Thanks for the reminder. :) So much appreciate. :) I just got two small
code changes which hasn't been tested by QA. So I guess you can send the
pull request first. Thanks again for the kindly reminding. :)
thanks,
Zhi.
On 09/19/18 02:34, Joonas Lahtinen wrote:
Just a reminder, if you are
Hi Min:
I remembered the IOTG guest kernel is using aliasing-ppgtt in the last
technical sharing (probably I didn't remember it correctly). Can you
confirm?
Also, thanks Joonas for the reminding. :)
thanks,
Zhi.
On 09/25/18 09:22, Joonas Lahtinen wrote:
+ Zhi and Zhenyu
For me the patch l
Sure. Acked-by: Zhi Wang
Meanwhile, I could send a patch to remove
gen8_preallocate_top_level_pdp() in i915_gem_gtt.c, which is only used
by vGPU with 32bit PPGTT (no one is going to use them after this patch
is landed)
Thanks,
Zhi.
On 09/26/18 04:02, Joonas Lahtinen wrote:
Quoting He
This is ideal and had been discussed in the beginning of upstream.
Mostly people didn't like this approach because it needs to modify i915
ppgtt code a lot and have to introduce a lot gvt-only code in i915 ppgtt.
The idea of the series of Joonas patch is to make PPGTT selection
per-platform. I
Hi Linus:
Thanks for letting us know that. We would fix this ASAP. The kvmgt.c
module is a part of GVT-g code. It's our fault that we didn't find this
mis-uses, not i915 or KVM guys. Wish they would feel better after seeing
this message.
Thanks,
Zhi.
On 08/23/18 00:44, Linus Torvalds wrote:
Hi Chris and Joonas:
Thanks for the discussion in the IRC for the suspend/resume request
APIs. I just a bit curious about the behavior of i915_wait_request()
after a caller suspend an i915 request. Will the caller of
i915_wait_request() be woken up at this time? or not? This behavior
decid
ean return value with int.
- Replace customized info/warn/debug macros with DRM macros.
- Document all non-static functions like i915.
- Remove empty and unused functions.
- Replace magic number with marcos.
- Set GVT-g in kernel config to "n" by default.
Signed-off-by: Zhi Wang
---
drivers
As the PVINFO page definition is used by both GVT-g guest (vGPU) and GVT-g
host (GVT-g kernel device model), factor it out for better code structure.
v3:
Take Joonas's comments:
- Use offsetof to calculate the member offset of PVINFO structure
Signed-off-by: Zhi Wang
---
drivers/gpu/drm
drm/i915: Introduce host graphics memory partition for GVT-g
Zhi Wang (8):
drm/i915: Factor out i915_pvinfo.h
drm/i915: Fold vGPU active check into inner functions
drm/i915: gvt: Introduce the basic architecture of GVT-g
drm/i915: Make ring buffer size of a LRC context configurable
drm/
v5:
- Let functions take struct drm_i915_private *. (Tvrtko)
- Fold vGPU related active check into the inner functions. (Kevin)
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/i915_gem_gtt.c | 11 ---
drivers/gpu/drm/i915/i915_vgpu.c| 13 +
drivers/gpu/drm/i915
errors and warnning reported by checkpatch.pl. (Joonas)
- Move the graphs into the header files. (Daniel)
Signed-off-by: Bing Niu
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/i915_vgpu.c | 23 +--
drivers/gpu/drm/i915/intel_gvt.h | 25 +
2 files changed
This patch introduces an option for configuring the ring buffer size
of a LRC context after the context creation.
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/i915_drv.h | 1 +
drivers/gpu/drm/i915/i915_gem_context.c | 1 +
drivers/gpu/drm/i915/intel_lrc.c| 3 ++-
3 files
LRC context should be configurable under this case.
v6:
- Directly save the addressing mode bits inside i915_gem_context. (Chris)
- Move the LRC context addressing mode bits into intel_lrc.h. (Chris)
v5:
- Change USES_FULL_48BIT(dev) to USES_FULL_48BIT(dev_priv) (Tvrtko)
Signed-off-by: Zhi Wang
tifier header, so it could be switched on/off
at runtime. (Chris)
v5:
- Only compile this feature when CONFIG_DRM_I915_GVT is enabled.(Tvrtko)
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/i915_drv.h | 2 ++
drivers/gpu/drm/i915/intel_lrc.c | 24
drivers/gpu/drm/i
.
v6:
- Make GVT code as dead code when !CONFIG_DRM_I915_GVT. (Chris)
v5:
- Only compile this feature when CONFIG_DRM_I915_GVT=y. (Tvrtko)
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/i915_drv.h | 1 +
drivers/gpu/drm/i915/intel_lrc.c | 15 +++
2 files changed, 16 insertions(+)
M_I915_GVT is enabled. (Tvrtko)
- Rebase the code into new repo.
- Add a comment about the ring buffer size. (Joonas)
v2:
Mostly based on Daniel's idea. Call the refactored core logic of GEM
context creation service and LRC context creation service to create the GVT
context.
Signed-o
v5:
- Let functions take struct drm_i915_private *. (Tvrtko)
- Fold vGPU related active check into the inner functions. (Kevin)
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/i915_gem_gtt.c | 11 ---
drivers/gpu/drm/i915/i915_vgpu.c| 13 +
drivers/gpu/drm/i915
drm/i915: Introduce host graphics memory partition for GVT-g
Zhi Wang (8):
drm/i915: Factor out i915_pvinfo.h
drm/i915: Fold vGPU active check into inner functions
drm/i915: gvt: Introduce the basic architecture of GVT-g
drm/i915: Make ring buffer size of a LRC context configurable
drm/
As the PVINFO page definition is used by both GVT-g guest (vGPU) and GVT-g
host (GVT-g kernel device model), factor it out for better code structure.
v3:
Take Joonas's comments:
- Use offsetof to calculate the member offset of PVINFO structure
Signed-off-by: Zhi Wang
---
drivers/gpu/drm
errors and warnning reported by checkpatch.pl. (Joonas)
- Move the graphs into the header files. (Daniel)
Signed-off-by: Bing Niu
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/i915_vgpu.c | 23 +--
drivers/gpu/drm/i915/intel_gvt.h | 25 +
2 files changed
This patch introduces an option for configuring the ring buffer size
of a LRC context after the context creation.
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/i915_drv.h | 1 +
drivers/gpu/drm/i915/i915_gem_context.c | 1 +
drivers/gpu/drm/i915/intel_lrc.c| 3 ++-
3 files
ean return value with int.
- Replace customized info/warn/debug macros with DRM macros.
- Document all non-static functions like i915.
- Remove empty and unused functions.
- Replace magic number with marcos.
- Set GVT-g in kernel config to "n" by default.
Signed-off-by: Zhi Wang
---
drivers
.
v6:
- Make GVT code as dead code when !CONFIG_DRM_I915_GVT. (Chris)
v5:
- Only compile this feature when CONFIG_DRM_I915_GVT=y. (Tvrtko)
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/i915_drv.h | 1 +
drivers/gpu/drm/i915/intel_lrc.c | 15 +++
2 files changed, 16 insertions(+)
tifier header, so it could be switched on/off
at runtime. (Chris)
v5:
- Only compile this feature when CONFIG_DRM_I915_GVT is enabled.(Tvrtko)
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/i915_drv.h | 2 ++
drivers/gpu/drm/i915/intel_lrc.c | 24
drivers/gpu/drm/i
M_I915_GVT is enabled. (Tvrtko)
- Rebase the code into new repo.
- Add a comment about the ring buffer size. (Joonas)
v2:
Mostly based on Daniel's idea. Call the refactored core logic of GEM
context creation service and LRC context creation service to create the GVT
context.
Signed-o
LRC context should be configurable under this case.
v6:
- Directly save the addressing mode bits inside i915_gem_context. (Chris)
- Move the LRC context addressing mode bits into intel_lrc.h. (Chris)
v5:
- Change USES_FULL_48BIT(dev) to USES_FULL_48BIT(dev_priv) (Tvrtko)
Signed-off-by: Zhi Wang
Bing Niu (1):
drm/i915: Introduce host graphics memory partition for GVT-g
Zhi Wang (10):
drm/i915: Factor out i915_pvinfo.h
drm/i915: Use offsetof() to calculate the offset of members in PVINFO
page
drm/i915: Fold vGPU active check into inner functions
drm/i915: Add teardo
To get the offset of the members in PVINFO page, offsetof() looks much
better than the tricky approach in current code.
v7:
- Move "offsetof()" modification into a dedicated patch. (Joonas)
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/i915_pvinfo.h | 2 +-
1 file changed, 1
unctions like i915.
- Remove empty and unused functions.
- Replace magic number with marcos.
- Set GVT-g in kernel config to "n" by default.
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/Kconfig | 22 +
drivers/gpu/drm/i915/Makefile| 5 ++
drivers/gpu/drm/
This function needs to be changed to have a proper goto teardown path.
Destructors/fini functions are only expected to be called after a
successful initialization, so calling it at random phase in init function
is bad. (Joonas)
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/i915_vgpu.c | 7
ssing mode bits into intel_lrc.h. (Chris)
v5:
- Change USES_FULL_48BIT(dev) to USES_FULL_48BIT(dev_priv) (Tvrtko)
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/i915_drv.h | 1 +
drivers/gpu/drm/i915/i915_gem_context.c | 2 ++
drivers/gpu/drm/i915/i915_reg.h | 12 +++
about the ring buffer size. (Joonas)
v2:
Mostly based on Daniel's idea. Call the refactored core logic of GEM
context creation service and LRC context creation service to create the GVT
context.
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/i915_gem_context.c | 65 +
v5:
- Let functions take struct drm_i915_private *. (Tvrtko)
- Fold vGPU related active check into the inner functions. (Kevin)
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/i915_gem_gtt.c | 11 ---
drivers/gpu/drm/i915/i915_vgpu.c| 13 +
drivers/gpu/drm/i915
:
- Address all coding-style comments from Joonas previously.
- Fix errors and warnning reported by checkpatch.pl. (Joonas)
- Move the graphs into the header files. (Daniel)
Signed-off-by: Bing Niu
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/i915_vgpu.c | 23 +--
drivers/gpu/drm
This patch introduces an option for configuring the ring buffer size
of a LRC context after the context creation.
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/i915_drv.h | 1 +
drivers/gpu/drm/i915/i915_gem_context.c | 1 +
drivers/gpu/drm/i915/intel_lrc.c| 3 ++-
3 files
.
v7:
- Fix typos in commit message. (Joonas)
v6:
- Make GVT code as dead code when !CONFIG_DRM_I915_GVT. (Chris)
v5:
- Only compile this feature when CONFIG_DRM_I915_GVT=y. (Tvrtko)
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/i915_drv.h | 1 +
drivers/gpu/drm/i915/intel_lrc.c
ris)
v5:
- Only compile this feature when CONFIG_DRM_I915_GVT is enabled.(Tvrtko)
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/i915_drv.h | 2 ++
drivers/gpu/drm/i915/i915_gem_context.c | 1 +
drivers/gpu/drm/i915/intel_lrc.c| 25 +
drivers/gpu/dr
structure (Joonas)
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/i915_pvinfo.h | 113 +
drivers/gpu/drm/i915/i915_vgpu.h | 86 +---
2 files changed, 114 insertions(+), 85 deletions(-)
create mode 100644 drivers/gpu/drm/i915/i915_pvin
esktop.org/archives/intel-gfx/2015-September/075397.html
Bing Niu (1):
drm/i915: Introduce host graphics memory partition for GVT-g
Zhi Wang (9):
drm/i915: Factor out i915_pvinfo.h
drm/i915: Use offsetof() to calculate the offset of members in PVINFO
page
drm/i915: Fold vGPU active c
structure (Joonas)
Reviewed-by: Joonas Lahtinen
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/i915_pvinfo.h | 113 +
drivers/gpu/drm/i915/i915_vgpu.h | 86 +---
2 files changed, 114 insertions(+), 85 deletions(-)
create mode 1006
To get the offset of the members in PVINFO page, offsetof() looks much
better than the tricky approach in current code.
v7:
- Move "offsetof()" modification into a dedicated patch. (Joonas)
Suggested-by: Joonas Lahtinen
Reviewed-by: Joonas Lahtinen
Signed-off-by: Zhi Wang
---
d
:
- Address all coding-style comments from Joonas previously.
- Fix errors and warnning reported by checkpatch.pl. (Joonas)
- Move the graphs into the header files. (Daniel)
Reviewed-by: Joonas Lahtinen
Cc: Chris Wilson
Cc: Kevin Tian
Cc: Daniel Vetter
Signed-off-by: Bing Niu
Signed-off-by: Zhi Wang
re logic of GEM
context creation service and LRC context creation service to create the GVT
context.
Reviewed-by: Joonas Lahtinen
Cc: Chris Wilson
Cc: Tvrtko Ursulin
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/i915_drv.h | 2 ++
drivers/gpu/drm/i915/i915_gem
This patch introduces an option for configuring the ring buffer size
of a LRC context after the context creation.
v8:
- Rename the data member in i915_gem_context. (Chris)
Reviewed-by: Joonas Lahtinen
Cc: Chris Wilson
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/i915_drv.h | 1
omized info/warn/debug macros with DRM macros.
- Document all non-static functions like i915.
- Remove empty and unused functions.
- Replace magic number with marcos.
- Set GVT-g in kernel config to "n" by default.
Reviewed-by: Joonas Lahtinen
Cc: Chris Wilson
Cc: Tvrtko Ursulin
Cc: Kevin T
mode bits inside i915_gem_context. (Chris)
- Move the LRC context addressing mode bits into intel_lrc.h. (Chris)
v5:
- Change USES_FULL_48BIT(dev) to USES_FULL_48BIT(dev_priv) (Tvrtko)
Reviewed-by: Joonas Lahtinen
Cc: Chris Wilson
Cc: Tvrtko Ursulin
Signed-off-by: Zhi Wang
---
drivers/gp
e notifier header, so it could be switched on/off
at runtime. (Chris)
v5:
- Only compile this feature when CONFIG_DRM_I915_GVT is enabled.(Tvrtko)
Reviewed-by: Joonas Lahtinen
Cc: Chris Wilson
Cc: Tvrtko Ursulin
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/i915_drv.h
n
Cc: Tvrtko Ursulin
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/i915_drv.h | 1 +
drivers/gpu/drm/i915/intel_lrc.c | 15 +++
2 files changed, 16 insertions(+)
diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h
index 08ba4e0..01f2f77 100644
--- a/dr
v5:
- Let functions take struct drm_i915_private *. (Tvrtko)
- Fold vGPU related active check into the inner functions. (Kevin)
Reviewed-by: Tvrtko Ursulin
Reviewed-by: Joonas Lahtinen
Suggested-by: Kevin Tian
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/i915_gem_gtt.c | 11
rtualization. Thanks!
Reference
-
[1] https://01.org/igvt-g
[2] http://lists.freedesktop.org/archives/intel-gfx/2014-September/053098.html
[3] http://lists.freedesktop.org/archives/intel-gfx/2015-September/075397.html
Bing Niu (1):
drm/i915: Introduce host graphics memory partition fo
: Zhi Wang
---
drivers/gpu/drm/i915/i915_drv.h | 1 +
drivers/gpu/drm/i915/i915_gem_context.c | 1 +
drivers/gpu/drm/i915/intel_lrc.c| 2 +-
3 files changed, 3 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h
index a461486
re logic of GEM
context creation service and LRC context creation service to create the GVT
context.
Reviewed-by: Joonas Lahtinen
Cc: Joonas Lahtinen
Cc: Chris Wilson
Cc: Tvrtko Ursulin
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/i915_drv.h | 2 ++
drivers/gpu/drm/i915/i915_gem
-off-by: Zhi Wang
---
drivers/gpu/drm/i915/i915_vgpu.c | 23 +--
drivers/gpu/drm/i915/intel_gvt.h | 36
2 files changed, 53 insertions(+), 6 deletions(-)
diff --git a/drivers/gpu/drm/i915/i915_vgpu.c b/drivers/gpu/drm/i915/i915_vgpu.c
index
1 - 100 of 419 matches
Mail list logo
