On Fri, 7 Jun 2024 18:25:35 +0300
Jani Nikula wrote:
Reviewed-by: Zhi Wang
> All callers of calc_index() pass 0 for the end parameter. Remove it.
>
> Cc: Zhenyu Wang
> Cc: Zhi Wang
> Cc: intel-gvt-...@lists.freedesktop.org
> Signed-off-by: Jani Nikula
> ---
>
On Fri, 7 Jun 2024 18:25:38 +0300
Jani Nikula wrote:
> Do not rely on having dev_priv local variable, pass it to the macro.
>
> Cc: Zhenyu Wang
> Cc: Zhi Wang
> Cc: intel-gvt-...@lists.freedesktop.org
> Signed-off-by: Jani Nikula
> ---
> drivers/gpu/drm/i915/gvt/
On Fri, 7 Jun 2024 18:25:37 +0300
Jani Nikula wrote:
> Range is a bit odd name for what really is stride. Rename. Switch to
> u32 while at it.
>
> Cc: Zhenyu Wang
> Cc: Zhi Wang
> Cc: intel-gvt-...@lists.freedesktop.org
> Signed-off-by: Jani Nikula
> ---
>
; doesn't work for plane C for CHV (but that's okay for gvt). This
> doesn't support planes beyond C either. But all that is unrelated to
> the change at hand.
>
> Cc: Zhenyu Wang
> Cc: Zhi Wang
> Cc: intel-gvt-...@lists.freedesktop.org
> Signed-off-by: Jani Nik
On Thu, 30 May 2024 15:43:52 +0300
Jani Nikula wrote:
> drm_edid_block_valid() is no longer used outside of drm_edid.c. Make
> it static.
>
> Signed-off-by: Jani Nikula
>
> ---
>
> Cc: Zhenyu Wang
> Cc: Zhi Wang
> Cc: intel-gvt-...@lists.f
the TODO comment about EDID extension block support.
>
> Signed-off-by: Jani Nikula
>
> ---
>
> Cc: Zhenyu Wang
> Cc: Zhi Wang
> Cc: intel-gvt-...@lists.freedesktop.org
> Cc: intel-gfx@lists.freedesktop.org
> Cc: dri-de...@lists.freedesktop.org
t; for users of I2C_ALGOBIT bitbanging interface, now that the approved
> verbiage exists in the specification.
>
> Compile tested, no functionality changes intended
>
For GVT part,
Acked-by: Zhi Wang
Thanks,
Zhi.
> [1]:
> https://lore.kernel.org/all/20240322132619.6389-1-wsa+ren
On Fri, 16 Feb 2024 10:51:20 +0200
Jani Nikula wrote:
> On Fri, 16 Feb 2024, "Jiri Slaby (SUSE)" wrote:
> > this series removes unused i915 structure members as found by
> > clang-struct (and manually checked by me).
>
> Thanks Jiri, good stuff!
>
> Acked-by: Jani Nikula
>
> However, you may
_gvt_irq_info *up_irq_info = NULL;
> u32 set_bits = 0;
> u32 clear_bits = 0;
Thanks for the patch!
Reviewed-by: Zhi Wang
t; drivers/gpu/drm/i915/intel_gvt.c | 2 +-
> 3 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index d516295978a4..805d33a107aa 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -10436,7 +10436,7 @@ M:Zhi Wang
> L: intel
Wang
Cc: intel-gvt-...@lists.freedesktop.org
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/gvt/cmd_parser.c | 6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/gpu/drm/i915/gvt/cmd_parser.c
b/drivers/gpu/drm/i915/gvt/cmd_parser.c
index 3c4ae1da0d41..05f9348b7a9d
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
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
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.
v9:
- Fix a problem might cause kernel panic.
v8:
- Use SPDX header in the
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
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
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
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
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
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
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
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
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.
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
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
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
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
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
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
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 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 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 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
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
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
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
-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
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
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
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
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 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:
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
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
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
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
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
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
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
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:
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
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
Hi guys:
Here are the latest gvt-next-fixes pull. It contains vGPU reset
enhancement, which refines vGPU reset flow and the support of virtual
aperture read/write when x-no-mmap=on is set in KVM, which is required
by a test case from Redhat and also another fix for virtual OpRegion.
Thank
ACTHD mmio read handler
Zhenyu Wang (1):
drm/i915/gvt: properly check per_ctx bb valid state
Zhi Wang (1):
drm/i915/gvt: Refine MMIO_RING_F()
drivers/gpu/drm/i915/gvt/cmd_parser.c | 3 ++
drivers/gpu/drm/i915/gvt/execlist.c | 3 +-
drivers/gpu/drm/i915/gvt/handler
Thanks, applied!
On 10/16/17 10:32, Jérémy Lefaure wrote:
Using the ARRAY_SIZE macro improves the readability of the code. Also,
it's useless to use a variable to store this constant calculated at
compile time.
Found with Coccinelle with the following semantic patch:
@r depends on (org || repor
Thanks, applied! :)
On 10/16/17 06:32, Christos Gkekas wrote:
Delete variables 'gma_bottom' that are set but never used.
Signed-off-by: Christos Gkekas
---
drivers/gpu/drm/i915/gvt/cmd_parser.c | 6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/gpu/drm/i915/gv
Hi:
Here is a fix of one bug for 4.14-rc5 which fixes a GPU hang after
resuing one vGPU across different guest OSes.
Thanks.
---
The following changes since commit ea850f64c2722278f150dc11de2141baeb24211c:
drm/i915/bios: parse DDI ports also for CHV for HDMI DDC pin and DP
AUX channel (2
Thanks for the patch! :)
2017-10-01 22:30 GMT+03:00 Jérémy Lefaure :
> Hi everyone,
> Using ARRAY_SIZE improves the code readability. I used coccinelle (I
> made a change to the array_size.cocci file [1]) to find several places
> where ARRAY_SIZE could be used instead of other macros or sizeof
>
Signed-off-by: Zhi Wang
Cc: Ben Widawsky
Cc: Rodrigo Vivi
Cc: Chris Wilson
Reviewed-by: Joonas Lahtinen
---
drivers/gpu/drm/i915/i915_gem_gtt.c | 52 ++---
1 file changed, 26 insertions(+), 26 deletions(-)
diff --git a/drivers/gpu/drm/i915/i915_gem_gtt.c
b
(Chris)
Cc: Ben Widawsky
Cc: Rodrigo Vivi
Cc: Joonas Lahtinen
Suggested-by: Chris Wilson
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/selftests/i915_gem_gtt.c | 372 ++
1 file changed, 372 insertions(+)
diff --git a/drivers/gpu/drm/i915/selftests/i915_gem_gtt.c
/i915: Introduce private PAT management")
Cc: Chris Wilson
Cc: Ben Widawsky
Cc: Rodrigo Vivi
Cc: Joonas Lahtinen
Signed-off-by: Zhi Wang
Reviewed-by: Joonas Lahtinen
---
drivers/gpu/drm/i915/i915_gem_gtt.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gp
Cc: Joonas Lahtinen
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/selftests/i915_gem_gtt.c | 28 +++
1 file changed, 11 insertions(+), 17 deletions(-)
diff --git a/drivers/gpu/drm/i915/selftests/i915_gem_gtt.c
b/drivers/gpu/drm/i915/selftests/i915_gem_gtt.c
index
Signed-off-by: Zhi Wang
Cc: Ben Widawsky
Cc: Rodrigo Vivi
Cc: Chris Wilson
Reviewed-by: Joonas Lahtinen
---
drivers/gpu/drm/i915/i915_gem_gtt.c | 52 ++---
1 file changed, 26 insertions(+), 26 deletions(-)
diff --git a/drivers/gpu/drm/i915/i915_gem_gtt.c
b
/i915: Introduce private PAT management")
Cc: Chris Wilson
Cc: Ben Widawsky
Cc: Rodrigo Vivi
Cc: Joonas Lahtinen
Signed-off-by: Zhi Wang
Reviewed-by: Joonas Lahtinen
---
drivers/gpu/drm/i915/i915_gem_gtt.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gp
struct_mutext needs to be held before call these two APIs.
Cc: Chris Wilson
Cc: Ben Widawsky
Cc: Rodrigo Vivi
Cc: Joonas Lahtinen
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/i915_gem_gtt.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/gpu/drm/i915/i915_gem_gtt.c
b
Wilson
Cc: Ben Widawsky
Cc: Rodrigo Vivi
Cc: Joonas Lahtinen
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/i915_gem_gtt.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i915/i915_gem_gtt.c
b/drivers/gpu/drm/i915/i915_gem_gtt.c
index 729ebaf..37cd086 10
struct_mutext needs to be held before call these two APIs.
Cc: Chris Wilson
Cc: Ben Widawsky
Cc: Rodrigo Vivi
Cc: Joonas Lahtinen
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/i915_gem_gtt.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/gpu/drm/i915/i915_gem_gtt.c
b
Cc: Chris Wilson
Cc: Ben Widawsky
Cc: Rodrigo Vivi
Cc: Joonas Lahtinen
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/i915_gem_gtt.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i915/i915_gem_gtt.c
b/drivers/gpu/drm/i915/i915_gem_gtt.c
index 729ebaf
Remove the "INDEX" suffix from PPAT marcos as they are bits actually, not
indexes.
Suggested-by: Chris Wilson
Signed-off-by: Zhi Wang
Cc: Ben Widawsky
Cc: Rodrigo Vivi
Cc: Joonas Lahtinen
Reviewed-by: Chris Wilson
---
drivers/gpu/drm/i915/gvt/gtt.c | 2 +-
drivers/gp
Cc: Chris Wilson
Cc: Ben Widawsky
Cc: Rodrigo Vivi
Cc: Joonas Lahtinen
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/i915_gem_gtt.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i915/i915_gem_gtt.c
b/drivers/gpu/drm/i915/i915_gem_gtt.c
index 729ebaf
struct_mutext needs to be held before call these two APIs.
Cc: Chris Wilson
Cc: Ben Widawsky
Cc: Rodrigo Vivi
Cc: Joonas Lahtinen
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/i915_gem_gtt.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/gpu/drm/i915/i915_gem_gtt.c
b
atforms which don't have PPAT.
v3:
- Introduce dirty bitmap for PPAT registers. (Chris)
- Change the name of the pointer "dev_priv" to "i915". (Chris)
- intel_ppat_{get, put} returns/takes a const intel_ppat_entry *. (Chris)
v2:
- API re-design. (Chris)
Signed-off-by:
Only PPAT entries 0/2/3/4 are using. Remove extra PPAT entry allocation
during initialization.
v8:
- Move ppat_index() into i915_gem_gtt.c. (Chris)
- Change the name of ppat_bits_to_index to ppat_index.
Suggested-by: Joonas Lahtinen
Signed-off-by: Zhi Wang
Cc: Ben Widawsky
Cc: Rodrigo Vivi
atforms which don't have PPAT.
v3:
- Introduce dirty bitmap for PPAT registers. (Chris)
- Change the name of the pointer "dev_priv" to "i915". (Chris)
- intel_ppat_{get, put} returns/takes a const intel_ppat_entry *. (Chris)
v2:
- API re-design. (Chris)
Signed-off-by:
test case. (Chris)
v9:
- Refine generate_new_value(). (Chris)
- Refine failure output. (Chris)
- Refine test flow of "perfect match". (Chris)
- Introduce another negative test case after "partial match". (Chris)
v8:
- Remove noisy output. (Chris)
- Add negative test case. (Chris)
Remove the "INDEX" suffix from PPAT marcos as they are bits actually, not
indexes.
Suggested-by: Chris Wilson
Signed-off-by: Zhi Wang
Cc: Ben Widawsky
Cc: Rodrigo Vivi
Cc: Joonas Lahtinen
Reviewed-by: Chris Wilson
---
drivers/gpu/drm/i915/gvt/gtt.c | 2 +-
drivers/gp
test case. (Chris)
v9:
- Refine generate_new_value(). (Chris)
- Refine failure output. (Chris)
- Refine test flow of "perfect match". (Chris)
- Introduce another negative test case after "partial match". (Chris)
v8:
- Remove noisy output. (Chris)
- Add negative test case. (Chris)
Only PPAT entries 0/2/3/4 are using. Remove extra PPAT entry allocation
during initialization.
v8:
- Move ppat_index() into i915_gem_gtt.c. (Chris)
- Change the name of ppat_bits_to_index to ppat_index.
Suggested-by: Joonas Lahtinen
Signed-off-by: Zhi Wang
Cc: Ben Widawsky
Cc: Rodrigo Vivi
Remove the "INDEX" suffix from PPAT marcos as they are bits actually, not
indexes.
Suggested-by: Chris Wilson
Signed-off-by: Zhi Wang
Cc: Ben Widawsky
Cc: Rodrigo Vivi
Cc: Joonas Lahtinen
Reviewed-by: Chris Wilson
---
drivers/gpu/drm/i915/gvt/gtt.c | 2 +-
drivers/gp
rty bitmap for PPAT registers. (Chris)
- Change the name of the pointer "dev_priv" to "i915". (Chris)
- intel_ppat_{get, put} returns/takes a const intel_ppat_entry *. (Chris)
v2:
- API re-design. (Chris)
Signed-off-by: Zhi Wang
Cc: Ben Widawsky
Cc: Rodrigo Vivi
Cc: Chris Wilso
On 09/11/17 16:59, Joonas Lahtinen wrote:
On Mon, 2017-09-11 at 12:26 +0800, Zhi Wang wrote:
The private PAT management is to support PPAT entry manipulation. Two
APIs are introduced for dynamically managing PPAT entries: intel_ppat_get
and intel_ppat_put.
intel_ppat_get will search for an
"reserved" means reserve something from somewhere. Actually they are
buffers used by command scanner. Rename it to ring_scan_buffer.
v2:
- Remove the usage of an extra variable. (Zhenyu)
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/gvt/cmd_parser.c | 16
drive
Move vGPU workload cache initialization/de-initialization into
intel_vgpu_{setup, clean}_submission() since they are not specific to
execlist stuffs.
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/gvt/execlist.c | 15 +--
drivers/gpu/drm/i915/gvt/scheduler.c | 24
To move workload related functions into scheduler.c, an expected way is
to collect all the init/clean functions related to vGPU workload
submission into fewer functions.
Rename intel_vgpu_{init, clean}_gvt_context() for above usage in future.
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915
-by: Zhi Wang
---
drivers/gpu/drm/i915/gvt/execlist.c | 22 +++---
drivers/gpu/drm/i915/gvt/execlist.h | 8
drivers/gpu/drm/i915/gvt/handlers.c | 2 +-
3 files changed, 16 insertions(+), 16 deletions(-)
diff --git a/drivers/gpu/drm/i915/gvt/execlist.c
b/drivers/gpu/drm
chance to get the buffer of suitable size in the first
submission.
Tested on my SKL NUC.
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/gvt/execlist.c | 21 -
drivers/gpu/drm/i915/gvt/gvt.h | 1 -
2 files changed, 22 deletions(-)
diff --git a/drivers/gpu/drm/i915/gvt
The pointer points to the original memory can never take the return value
of krealloc().
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/gvt/cmd_parser.c | 10 ++
1 file changed, 6 insertions(+), 4 deletions(-)
diff --git a/drivers/gpu/drm/i915/gvt/cmd_parser.c
b/drivers/gpu/drm/i915
Move tlb_handle_pending into intel_vgpu_submssion since it belongs to a
part of vGPU submission stuffs
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/gvt/gvt.h | 2 +-
drivers/gpu/drm/i915/gvt/handlers.c | 2 +-
drivers/gpu/drm/i915/gvt/render.c| 3 ++-
drivers/gpu/drm/i915/gvt
Move ring scan buffers into intel_vgpu_submission since they belongs to
a part of vGPU submission stuffs.
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/gvt/cmd_parser.c | 11 ++-
drivers/gpu/drm/i915/gvt/execlist.c | 23 +--
drivers/gpu/drm/i915/gvt/gvt.h
Introduce intel_vgpu_submission to hold all members related to submission
in struct intel_vgpu before.
Signed-off-by: Zhi Wang
---
drivers/gpu/drm/i915/gvt/execlist.c | 30 +---
drivers/gpu/drm/i915/gvt/gvt.h | 17 +
drivers/gpu/drm/i915/gvt/handlers.c | 2
Only PPAT entries 0/2/3/4 are using. Remove extra PPAT entry allocation
during initialization.
v8:
- Move ppat_index() into i915_gem_gtt.c. (Chris)
- Change the name of ppat_bits_to_index to ppat_index.
Suggested-by: Joonas Lahtinen
Signed-off-by: Zhi Wang
Cc: Ben Widawsky
Cc: Rodrigo Vivi
test case. (Chris)
v9:
- Refine generate_new_value(). (Chris)
- Refine failure output. (Chris)
- Refine test flow of "perfect match". (Chris)
- Introduce another negative test case after "partial match". (Chris)
v8:
- Remove noisy output. (Chris)
- Add negative test case. (Chris)
Factor out setup_private_pat() for introducing the following patches.
Signed-off-by: Zhi Wang
Cc: Ben Widawsky
Cc: Rodrigo Vivi
Cc: Chris Wilson
Cc: Joonas Lahtinen
Reviewed-by: Chris Wilson
Reviewed-by: Ben Widawsky
---
drivers/gpu/drm/i915/i915_gem_gtt.c | 20
1
1 - 100 of 413 matches
Mail list logo