* [PULL] gvt-fixes for 4.17
@ 2018-04-19 7:39 Zhenyu Wang
2018-04-19 9:17 ` [PULL] gvt-next " Zhi Wang
2018-06-06 7:49 ` [PULL] gvt-fixes for 4.17 Zhenyu Wang
0 siblings, 2 replies; 18+ messages in thread
From: Zhenyu Wang @ 2018-04-19 7:39 UTC (permalink / raw)
To: Joonas Lahtinen, Jani Nikula, Vivi, Rodrigo
Cc: intel-gfx, intel-gvt-dev, Lv, Zhiyuan, Yuan, Hang
[-- Attachment #1.1: Type: text/plain, Size: 1756 bytes --]
Hi,
Here's current gvt fixes for 4.17 with several kernel warning
and other misc fixes as detailed below.
p.s: I'll be on vacation from next week till May 2, Zhi will cover for me.
Thanks
--
The following changes since commit 60cc43fc888428bb2f18f08997432d426a243338:
Linux 4.17-rc1 (2018-04-15 18:24:20 -0700)
are available in the Git repository at:
https://github.com/intel/gvt-linux.git tags/gvt-fixes-2018-04-19
for you to fetch changes up to 39b4cbadb9a95bf3f13ea102d6ec841940916ee2:
drm/i915/kvmgt: Check the pfn got from vfio_pin_pages (2018-04-17 10:46:49 +0800)
----------------------------------------------------------------
gvt-fixes-2018-04-19
- cmd parser error path mem leak fix (Colin)
- fix dp aux header validation (Changbin)
- sanity check on pfn after vfio pin page (Changbin)
- fix msi eventfd put (Xiong)
----------------------------------------------------------------
Changbin Du (2):
drm/i915/gvt: Fix the validation on size field of dp aux header
drm/i915/kvmgt: Check the pfn got from vfio_pin_pages
Colin Ian King (1):
drm/i915/gvt: fix memory leak of a cmd_entry struct on error exit path
Xiong Zhang (1):
drm/i915/gvt: Dereference msi eventfd_ctx when it isn't used anymore
Zhenyu Wang (1):
Back merge 'drm-intel-fixes' into gvt-fixes
drivers/gpu/drm/i915/gvt/cmd_parser.c | 1 +
drivers/gpu/drm/i915/gvt/display.h | 2 +-
drivers/gpu/drm/i915/gvt/handlers.c | 13 +++++++++----
drivers/gpu/drm/i915/gvt/kvmgt.c | 34 +++++++++++++++++++++++++++++++++-
4 files changed, 44 insertions(+), 6 deletions(-)
--
Open Source Technology Center, Intel ltd.
$gpg --keyserver wwwkeys.pgp.net --recv-keys 4D781827
[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 bytes --]
[-- Attachment #2: Type: text/plain, Size: 160 bytes --]
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 18+ messages in thread
* [PULL] gvt-next for 4.17
2018-04-19 7:39 [PULL] gvt-fixes for 4.17 Zhenyu Wang
@ 2018-04-19 9:17 ` Zhi Wang
2018-04-19 9:21 ` Zhi Wang
2018-04-19 9:34 ` Jani Nikula
2018-06-06 7:49 ` [PULL] gvt-fixes for 4.17 Zhenyu Wang
1 sibling, 2 replies; 18+ messages in thread
From: Zhi Wang @ 2018-04-19 9:17 UTC (permalink / raw)
To: Zhenyu Wang, Joonas Lahtinen, Jani Nikula, Vivi, Rodrigo
Cc: intel-gfx, intel-gvt-dev, Lv, Zhiyuan, Yuan, Hang
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 at:
https://github.com/intel/gvt-linux.git tags/gvt-next-2018-04-19
for you to fetch changes up to c0fb4098fc47dcaeb47085c08d8fafa42fa8e471:
drm/i915/gvt: Mark expected switch fall-through in
handle_g2v_notification (2018-04-19 16:35:55 +0800)
----------------------------------------------------------------
- Minor condition check improvment (Gustavo A. R. Silva)
- Reverting GVT context priority hack (Weinan Li)
- Non-priviliged batch buffer scan (Yan Zhao)
- Scheduling optimizations (Zhipeng Gong)
----------------------------------------------------------------
Gustavo A. R. Silva (2):
drm/i915/gvt/scheduler: Remove unnecessary NULL checks in sr_oa_regs
drm/i915/gvt: Mark expected switch fall-through in
handle_g2v_notification
Weinan Li (1):
Revert "drm/i915/gvt: set max priority for gvt context"
Zhao Yan (1):
drm/i915/gvt: scan non-privileged batch buffer for debug purpose
Zhipeng Gong (2):
drm/i915/gvt: Use real time to do timer check
drm/i915/gvt: Update time slice more frequently
drivers/gpu/drm/i915/gvt/cmd_parser.c | 55 +++++++++++++++++++-------
drivers/gpu/drm/i915/gvt/debugfs.c | 67
++++++++++++++++++++++++++++++++
drivers/gpu/drm/i915/gvt/gvt.h | 1 +
drivers/gpu/drm/i915/gvt/handlers.c | 1 +
drivers/gpu/drm/i915/gvt/sched_policy.c | 31 ++++++++-------
drivers/gpu/drm/i915/gvt/scheduler.c | 69
+++++++++++++++++++++------------
drivers/gpu/drm/i915/gvt/scheduler.h | 1 +
drivers/gpu/drm/i915/gvt/trace.h | 24 +++++++++---
8 files changed, 193 insertions(+), 56 deletions(-)
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PULL] gvt-next for 4.17
2018-04-19 9:17 ` [PULL] gvt-next " Zhi Wang
@ 2018-04-19 9:21 ` Zhi Wang
2018-04-19 9:34 ` Jani Nikula
1 sibling, 0 replies; 18+ messages in thread
From: Zhi Wang @ 2018-04-19 9:21 UTC (permalink / raw)
To: Zhenyu Wang, Joonas Lahtinen, Jani Nikula, Vivi, Rodrigo
Cc: intel-gfx, intel-gvt-dev, Lv, Zhiyuan, Yuan, Hang
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:
>
> 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-19
>
> for you to fetch changes up to c0fb4098fc47dcaeb47085c08d8fafa42fa8e471:
>
> drm/i915/gvt: Mark expected switch fall-through in
> handle_g2v_notification (2018-04-19 16:35:55 +0800)
>
> ----------------------------------------------------------------
> - Minor condition check improvment (Gustavo A. R. Silva)
> - Reverting GVT context priority hack (Weinan Li)
> - Non-priviliged batch buffer scan (Yan Zhao)
> - Scheduling optimizations (Zhipeng Gong)
>
> ----------------------------------------------------------------
> Gustavo A. R. Silva (2):
> drm/i915/gvt/scheduler: Remove unnecessary NULL checks in sr_oa_regs
> drm/i915/gvt: Mark expected switch fall-through in
> handle_g2v_notification
>
> Weinan Li (1):
> Revert "drm/i915/gvt: set max priority for gvt context"
>
> Zhao Yan (1):
> drm/i915/gvt: scan non-privileged batch buffer for debug purpose
>
> Zhipeng Gong (2):
> drm/i915/gvt: Use real time to do timer check
> drm/i915/gvt: Update time slice more frequently
>
> drivers/gpu/drm/i915/gvt/cmd_parser.c | 55 +++++++++++++++++++-------
> drivers/gpu/drm/i915/gvt/debugfs.c | 67
> ++++++++++++++++++++++++++++++++
> drivers/gpu/drm/i915/gvt/gvt.h | 1 +
> drivers/gpu/drm/i915/gvt/handlers.c | 1 +
> drivers/gpu/drm/i915/gvt/sched_policy.c | 31 ++++++++-------
> drivers/gpu/drm/i915/gvt/scheduler.c | 69
> +++++++++++++++++++++------------
> drivers/gpu/drm/i915/gvt/scheduler.h | 1 +
> drivers/gpu/drm/i915/gvt/trace.h | 24 +++++++++---
> 8 files changed, 193 insertions(+), 56 deletions(-)
> _______________________________________________
> intel-gvt-dev mailing list
> intel-gvt-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gvt-dev
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PULL] gvt-next for 4.17
2018-04-19 9:17 ` [PULL] gvt-next " Zhi Wang
2018-04-19 9:21 ` Zhi Wang
@ 2018-04-19 9:34 ` Jani Nikula
2018-04-19 9:41 ` Zhenyu Wang
2018-04-19 9:50 ` Zhi Wang
1 sibling, 2 replies; 18+ messages in thread
From: Jani Nikula @ 2018-04-19 9:34 UTC (permalink / raw)
To: Zhi Wang, Zhenyu Wang, Joonas Lahtinen, Vivi, Rodrigo
Cc: intel-gfx, intel-gvt-dev, Lv, Zhiyuan, Yuan, Hang
On Thu, 19 Apr 2018, Zhi Wang <zhi.a.wang@intel.com> 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:
>
> 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-19
>
> for you to fetch changes up to c0fb4098fc47dcaeb47085c08d8fafa42fa8e471:
>
> drm/i915/gvt: Mark expected switch fall-through in
> handle_g2v_notification (2018-04-19 16:35:55 +0800)
>
> ----------------------------------------------------------------
> - Minor condition check improvment (Gustavo A. R. Silva)
> - Reverting GVT context priority hack (Weinan Li)
> - Non-priviliged batch buffer scan (Yan Zhao)
> - Scheduling optimizations (Zhipeng Gong)
>
> ----------------------------------------------------------------
> Gustavo A. R. Silva (2):
> drm/i915/gvt/scheduler: Remove unnecessary NULL checks in sr_oa_regs
> drm/i915/gvt: Mark expected switch fall-through in
> handle_g2v_notification
>
> Weinan Li (1):
> Revert "drm/i915/gvt: set max priority for gvt context"
This reverts a commit in v4.15. Why is it in a -next pull and not a
-fixes pull?
It also conflicts, please advise how to resolve:
diff --cc drivers/gpu/drm/i915/gvt/scheduler.c
index f3d21849b0cb,080fb5027d9c..000000000000
--- a/drivers/gpu/drm/i915/gvt/scheduler.c
+++ b/drivers/gpu/drm/i915/gvt/scheduler.c
@@@ -1134,9 -1156,6 +1156,12 @@@ int intel_vgpu_setup_submission(struct
if (IS_ERR(s->shadow_ctx))
return PTR_ERR(s->shadow_ctx);
++<<<<<<< HEAD
+ if (HAS_LOGICAL_RING_PREEMPTION(vgpu->gvt->dev_priv))
+ s->shadow_ctx->sched.priority = INT_MAX;
+
++=======
++>>>>>>> c2f6410ef67740ebcbf5d92dffc2679d4a0e288c
bitmap_zero(s->shadow_ctx_desc_updated, I915_NUM_ENGINES);
s->workloads = kmem_cache_create_usercopy("gvt-g_vgpu_workload",
Finally, it's committed by Zhi Wang <zhi.a.wang@intel.com> but without
his Signed-off-by.
BR,
Jani.
>
> Zhao Yan (1):
> drm/i915/gvt: scan non-privileged batch buffer for debug purpose
>
> Zhipeng Gong (2):
> drm/i915/gvt: Use real time to do timer check
> drm/i915/gvt: Update time slice more frequently
>
> drivers/gpu/drm/i915/gvt/cmd_parser.c | 55 +++++++++++++++++++-------
> drivers/gpu/drm/i915/gvt/debugfs.c | 67
> ++++++++++++++++++++++++++++++++
> drivers/gpu/drm/i915/gvt/gvt.h | 1 +
> drivers/gpu/drm/i915/gvt/handlers.c | 1 +
> drivers/gpu/drm/i915/gvt/sched_policy.c | 31 ++++++++-------
> drivers/gpu/drm/i915/gvt/scheduler.c | 69
> +++++++++++++++++++++------------
> drivers/gpu/drm/i915/gvt/scheduler.h | 1 +
> drivers/gpu/drm/i915/gvt/trace.h | 24 +++++++++---
> 8 files changed, 193 insertions(+), 56 deletions(-)
--
Jani Nikula, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PULL] gvt-next for 4.17
2018-04-19 9:34 ` Jani Nikula
@ 2018-04-19 9:41 ` Zhenyu Wang
2018-04-19 9:50 ` Zhi Wang
1 sibling, 0 replies; 18+ messages in thread
From: Zhenyu Wang @ 2018-04-19 9:41 UTC (permalink / raw)
To: Jani Nikula
Cc: intel-gfx, Yuan, Hang, Lv, Zhiyuan, Vivi, Rodrigo, intel-gvt-dev
[-- Attachment #1.1: Type: text/plain, Size: 3702 bytes --]
On 2018.04.19 12:34:16 +0300, Jani Nikula wrote:
> On Thu, 19 Apr 2018, Zhi Wang <zhi.a.wang@intel.com> 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:
> >
> > 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-19
> >
> > for you to fetch changes up to c0fb4098fc47dcaeb47085c08d8fafa42fa8e471:
> >
> > drm/i915/gvt: Mark expected switch fall-through in
> > handle_g2v_notification (2018-04-19 16:35:55 +0800)
> >
> > ----------------------------------------------------------------
> > - Minor condition check improvment (Gustavo A. R. Silva)
> > - Reverting GVT context priority hack (Weinan Li)
> > - Non-priviliged batch buffer scan (Yan Zhao)
> > - Scheduling optimizations (Zhipeng Gong)
> >
> > ----------------------------------------------------------------
> > Gustavo A. R. Silva (2):
> > drm/i915/gvt/scheduler: Remove unnecessary NULL checks in sr_oa_regs
> > drm/i915/gvt: Mark expected switch fall-through in
> > handle_g2v_notification
> >
> > Weinan Li (1):
> > Revert "drm/i915/gvt: set max priority for gvt context"
>
> This reverts a commit in v4.15. Why is it in a -next pull and not a
> -fixes pull?
This one was originally queued for 4.17 in gvt-next-fixes, but declined as
Joonas thought this is new feature as enabling vGPU priority scheduling,
https://lists.freedesktop.org/archives/intel-gfx/2018-March/160431.html
>
> It also conflicts, please advise how to resolve:
>
> diff --cc drivers/gpu/drm/i915/gvt/scheduler.c
> index f3d21849b0cb,080fb5027d9c..000000000000
> --- a/drivers/gpu/drm/i915/gvt/scheduler.c
> +++ b/drivers/gpu/drm/i915/gvt/scheduler.c
> @@@ -1134,9 -1156,6 +1156,12 @@@ int intel_vgpu_setup_submission(struct
> if (IS_ERR(s->shadow_ctx))
> return PTR_ERR(s->shadow_ctx);
>
> ++<<<<<<< HEAD
> + if (HAS_LOGICAL_RING_PREEMPTION(vgpu->gvt->dev_priv))
> + s->shadow_ctx->sched.priority = INT_MAX;
> +
> ++=======
> ++>>>>>>> c2f6410ef67740ebcbf5d92dffc2679d4a0e288c
> bitmap_zero(s->shadow_ctx_desc_updated, I915_NUM_ENGINES);
>
> s->workloads = kmem_cache_create_usercopy("gvt-g_vgpu_workload",
>
> Finally, it's committed by Zhi Wang <zhi.a.wang@intel.com> but without
> his Signed-off-by.
>
>
> BR,
> Jani.
>
>
> >
> > Zhao Yan (1):
> > drm/i915/gvt: scan non-privileged batch buffer for debug purpose
> >
> > Zhipeng Gong (2):
> > drm/i915/gvt: Use real time to do timer check
> > drm/i915/gvt: Update time slice more frequently
> >
> > drivers/gpu/drm/i915/gvt/cmd_parser.c | 55 +++++++++++++++++++-------
> > drivers/gpu/drm/i915/gvt/debugfs.c | 67
> > ++++++++++++++++++++++++++++++++
> > drivers/gpu/drm/i915/gvt/gvt.h | 1 +
> > drivers/gpu/drm/i915/gvt/handlers.c | 1 +
> > drivers/gpu/drm/i915/gvt/sched_policy.c | 31 ++++++++-------
> > drivers/gpu/drm/i915/gvt/scheduler.c | 69
> > +++++++++++++++++++++------------
> > drivers/gpu/drm/i915/gvt/scheduler.h | 1 +
> > drivers/gpu/drm/i915/gvt/trace.h | 24 +++++++++---
> > 8 files changed, 193 insertions(+), 56 deletions(-)
>
> --
> Jani Nikula, Intel Open Source Technology Center
--
Open Source Technology Center, Intel ltd.
$gpg --keyserver wwwkeys.pgp.net --recv-keys 4D781827
[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 bytes --]
[-- Attachment #2: Type: text/plain, Size: 160 bytes --]
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PULL] gvt-next for 4.17
2018-04-19 9:34 ` Jani Nikula
2018-04-19 9:41 ` Zhenyu Wang
@ 2018-04-19 9:50 ` Zhi Wang
2018-04-19 10:16 ` Zhi Wang
1 sibling, 1 reply; 18+ messages in thread
From: Zhi Wang @ 2018-04-19 9:50 UTC (permalink / raw)
To: Jani Nikula, Zhenyu Wang, Joonas Lahtinen, Vivi, Rodrigo
Cc: intel-gfx, intel-gvt-dev, Lv, Zhiyuan, Yuan, Hang
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 <zhi.a.wang@intel.com> 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:
>>
>> 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-19
>>
>> for you to fetch changes up to c0fb4098fc47dcaeb47085c08d8fafa42fa8e471:
>>
>> drm/i915/gvt: Mark expected switch fall-through in
>> handle_g2v_notification (2018-04-19 16:35:55 +0800)
>>
>> ----------------------------------------------------------------
>> - Minor condition check improvment (Gustavo A. R. Silva)
>> - Reverting GVT context priority hack (Weinan Li)
>> - Non-priviliged batch buffer scan (Yan Zhao)
>> - Scheduling optimizations (Zhipeng Gong)
>>
>> ----------------------------------------------------------------
>> Gustavo A. R. Silva (2):
>> drm/i915/gvt/scheduler: Remove unnecessary NULL checks in sr_oa_regs
>> drm/i915/gvt: Mark expected switch fall-through in
>> handle_g2v_notification
>>
>> Weinan Li (1):
>> Revert "drm/i915/gvt: set max priority for gvt context"
>
> This reverts a commit in v4.15. Why is it in a -next pull and not a
> -fixes pull?
>
> It also conflicts, please advise how to resolve:
>
> diff --cc drivers/gpu/drm/i915/gvt/scheduler.c
> index f3d21849b0cb,080fb5027d9c..000000000000
> --- a/drivers/gpu/drm/i915/gvt/scheduler.c
> +++ b/drivers/gpu/drm/i915/gvt/scheduler.c
> @@@ -1134,9 -1156,6 +1156,12 @@@ int intel_vgpu_setup_submission(struct
> if (IS_ERR(s->shadow_ctx))
> return PTR_ERR(s->shadow_ctx);
>
> ++<<<<<<< HEAD
> + if (HAS_LOGICAL_RING_PREEMPTION(vgpu->gvt->dev_priv))
> + s->shadow_ctx->sched.priority = INT_MAX;
> +
> ++=======
> ++>>>>>>> c2f6410ef67740ebcbf5d92dffc2679d4a0e288c
> bitmap_zero(s->shadow_ctx_desc_updated, I915_NUM_ENGINES);
>
> s->workloads = kmem_cache_create_usercopy("gvt-g_vgpu_workload",
>
> Finally, it's committed by Zhi Wang <zhi.a.wang@intel.com> but without
> his Signed-off-by.
>
>
> BR,
> Jani.
>
>
>>
>> Zhao Yan (1):
>> drm/i915/gvt: scan non-privileged batch buffer for debug purpose
>>
>> Zhipeng Gong (2):
>> drm/i915/gvt: Use real time to do timer check
>> drm/i915/gvt: Update time slice more frequently
>>
>> drivers/gpu/drm/i915/gvt/cmd_parser.c | 55 +++++++++++++++++++-------
>> drivers/gpu/drm/i915/gvt/debugfs.c | 67
>> ++++++++++++++++++++++++++++++++
>> drivers/gpu/drm/i915/gvt/gvt.h | 1 +
>> drivers/gpu/drm/i915/gvt/handlers.c | 1 +
>> drivers/gpu/drm/i915/gvt/sched_policy.c | 31 ++++++++-------
>> drivers/gpu/drm/i915/gvt/scheduler.c | 69
>> +++++++++++++++++++++------------
>> drivers/gpu/drm/i915/gvt/scheduler.h | 1 +
>> drivers/gpu/drm/i915/gvt/trace.h | 24 +++++++++---
>> 8 files changed, 193 insertions(+), 56 deletions(-)
>
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PULL] gvt-next for 4.17
2018-04-19 9:50 ` Zhi Wang
@ 2018-04-19 10:16 ` Zhi Wang
2018-04-23 8:11 ` [PULL] gvt-next for 4.18 Zhi Wang
0 siblings, 1 reply; 18+ messages in thread
From: Zhi Wang @ 2018-04-19 10:16 UTC (permalink / raw)
To: Jani Nikula, Zhenyu Wang, Joonas Lahtinen, Vivi, Rodrigo
Cc: intel-gfx, intel-gvt-dev, Lv, Zhiyuan, Yuan, Hang
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 commiter change without new
> signoff-by.
>
> Thanks,
> Zhi.
>
> On 04/19/18 17:34, Jani Nikula wrote:
>> On Thu, 19 Apr 2018, Zhi Wang <zhi.a.wang@intel.com> 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:
>>>
>>> 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-19
>>>
>>> for you to fetch changes up to c0fb4098fc47dcaeb47085c08d8fafa42fa8e471:
>>>
>>> drm/i915/gvt: Mark expected switch fall-through in
>>> handle_g2v_notification (2018-04-19 16:35:55 +0800)
>>>
>>> ----------------------------------------------------------------
>>> - Minor condition check improvment (Gustavo A. R. Silva)
>>> - Reverting GVT context priority hack (Weinan Li)
>>> - Non-priviliged batch buffer scan (Yan Zhao)
>>> - Scheduling optimizations (Zhipeng Gong)
>>>
>>> ----------------------------------------------------------------
>>> Gustavo A. R. Silva (2):
>>> drm/i915/gvt/scheduler: Remove unnecessary NULL checks in
>>> sr_oa_regs
>>> drm/i915/gvt: Mark expected switch fall-through in
>>> handle_g2v_notification
>>>
>>> Weinan Li (1):
>>> Revert "drm/i915/gvt: set max priority for gvt context"
>>
>> This reverts a commit in v4.15. Why is it in a -next pull and not a
>> -fixes pull?
>>
>> It also conflicts, please advise how to resolve:
>>
>> diff --cc drivers/gpu/drm/i915/gvt/scheduler.c
>> index f3d21849b0cb,080fb5027d9c..000000000000
>> --- a/drivers/gpu/drm/i915/gvt/scheduler.c
>> +++ b/drivers/gpu/drm/i915/gvt/scheduler.c
>> @@@ -1134,9 -1156,6 +1156,12 @@@ int intel_vgpu_setup_submission(struct
>> if (IS_ERR(s->shadow_ctx))
>> return PTR_ERR(s->shadow_ctx);
>> ++<<<<<<< HEAD
>> + if (HAS_LOGICAL_RING_PREEMPTION(vgpu->gvt->dev_priv))
>> + s->shadow_ctx->sched.priority = INT_MAX;
>> +
>> ++=======
>> ++>>>>>>> c2f6410ef67740ebcbf5d92dffc2679d4a0e288c
>> bitmap_zero(s->shadow_ctx_desc_updated, I915_NUM_ENGINES);
>> s->workloads = kmem_cache_create_usercopy("gvt-g_vgpu_workload",
>>
>> Finally, it's committed by Zhi Wang <zhi.a.wang@intel.com> but without
>> his Signed-off-by.
>>
>>
>> BR,
>> Jani.
>>
>>
>>>
>>> Zhao Yan (1):
>>> drm/i915/gvt: scan non-privileged batch buffer for debug purpose
>>>
>>> Zhipeng Gong (2):
>>> drm/i915/gvt: Use real time to do timer check
>>> drm/i915/gvt: Update time slice more frequently
>>>
>>> drivers/gpu/drm/i915/gvt/cmd_parser.c | 55
>>> +++++++++++++++++++-------
>>> drivers/gpu/drm/i915/gvt/debugfs.c | 67
>>> ++++++++++++++++++++++++++++++++
>>> drivers/gpu/drm/i915/gvt/gvt.h | 1 +
>>> drivers/gpu/drm/i915/gvt/handlers.c | 1 +
>>> drivers/gpu/drm/i915/gvt/sched_policy.c | 31 ++++++++-------
>>> drivers/gpu/drm/i915/gvt/scheduler.c | 69
>>> +++++++++++++++++++++------------
>>> drivers/gpu/drm/i915/gvt/scheduler.h | 1 +
>>> drivers/gpu/drm/i915/gvt/trace.h | 24 +++++++++---
>>> 8 files changed, 193 insertions(+), 56 deletions(-)
>>
> _______________________________________________
> intel-gvt-dev mailing list
> intel-gvt-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gvt-dev
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 18+ messages in thread
* [PULL] gvt-next for 4.18
2018-04-19 10:16 ` Zhi Wang
@ 2018-04-23 8:11 ` Zhi Wang
2018-04-23 8:13 ` Zhi Wang
0 siblings, 1 reply; 18+ messages in thread
From: Zhi Wang @ 2018-04-23 8:11 UTC (permalink / raw)
To: Jani Nikula, Zhenyu Wang, Joonas Lahtinen, Vivi, Rodrigo
Cc: intel-gfx, intel-gvt-dev, Lv, Zhiyuan, Yuan, Hang
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 3eda0d22ead04f81ea59c9584bcbf5b496745e92:
drm/i915/gvt: Mark expected switch fall-through in
handle_g2v_notification (2018-04-23 13:09:36 +0800)
----------------------------------------------------------------
- Minor condition check improvment (Gustavo A. R. Silva)
- Non-priviliged batch buffer scan (Yan Zhao)
- Scheduling optimizations (Zhipeng Gong)
----------------------------------------------------------------
Gustavo A. R. Silva (2):
drm/i915/gvt/scheduler: Remove unnecessary NULL checks in sr_oa_regs
drm/i915/gvt: Mark expected switch fall-through in
handle_g2v_notification
Zhao Yan (1):
drm/i915/gvt: scan non-privileged batch buffer for debug purpose
Zhipeng Gong (2):
drm/i915/gvt: Use real time to do timer check
drm/i915/gvt: Update time slice more frequently
drivers/gpu/drm/i915/gvt/cmd_parser.c | 55 ++++++++++++++++++++-------
drivers/gpu/drm/i915/gvt/debugfs.c | 67
+++++++++++++++++++++++++++++++++
drivers/gpu/drm/i915/gvt/gvt.h | 1 +
drivers/gpu/drm/i915/gvt/handlers.c | 1 +
drivers/gpu/drm/i915/gvt/sched_policy.c | 31 ++++++++-------
drivers/gpu/drm/i915/gvt/scheduler.c | 66
+++++++++++++++++++++-----------
drivers/gpu/drm/i915/gvt/scheduler.h | 1 +
drivers/gpu/drm/i915/gvt/trace.h | 24 +++++++++---
8 files changed, 193 insertions(+), 53 deletions(-)
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PULL] gvt-next for 4.18
2018-04-23 8:11 ` [PULL] gvt-next for 4.18 Zhi Wang
@ 2018-04-23 8:13 ` Zhi Wang
2018-04-23 10:25 ` Jani Nikula
2018-04-24 10:17 ` Zhi Wang
0 siblings, 2 replies; 18+ messages in thread
From: Zhi Wang @ 2018-04-23 8:13 UTC (permalink / raw)
To: Jani Nikula, Zhenyu Wang, Joonas Lahtinen, Vivi, Rodrigo
Cc: intel-gfx, intel-gvt-dev, Lv, Zhiyuan, Yuan, Hang
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 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 3eda0d22ead04f81ea59c9584bcbf5b496745e92:
>
> drm/i915/gvt: Mark expected switch fall-through in
> handle_g2v_notification (2018-04-23 13:09:36 +0800)
>
> ----------------------------------------------------------------
> - Minor condition check improvment (Gustavo A. R. Silva)
> - Non-priviliged batch buffer scan (Yan Zhao)
> - Scheduling optimizations (Zhipeng Gong)
>
> ----------------------------------------------------------------
> Gustavo A. R. Silva (2):
> drm/i915/gvt/scheduler: Remove unnecessary NULL checks in sr_oa_regs
> drm/i915/gvt: Mark expected switch fall-through in
> handle_g2v_notification
>
> Zhao Yan (1):
> drm/i915/gvt: scan non-privileged batch buffer for debug purpose
>
> Zhipeng Gong (2):
> drm/i915/gvt: Use real time to do timer check
> drm/i915/gvt: Update time slice more frequently
>
> drivers/gpu/drm/i915/gvt/cmd_parser.c | 55 ++++++++++++++++++++-------
> drivers/gpu/drm/i915/gvt/debugfs.c | 67
> +++++++++++++++++++++++++++++++++
> drivers/gpu/drm/i915/gvt/gvt.h | 1 +
> drivers/gpu/drm/i915/gvt/handlers.c | 1 +
> drivers/gpu/drm/i915/gvt/sched_policy.c | 31 ++++++++-------
> drivers/gpu/drm/i915/gvt/scheduler.c | 66
> +++++++++++++++++++++-----------
> drivers/gpu/drm/i915/gvt/scheduler.h | 1 +
> drivers/gpu/drm/i915/gvt/trace.h | 24 +++++++++---
> 8 files changed, 193 insertions(+), 53 deletions(-)
>
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PULL] gvt-next for 4.18
2018-04-23 8:13 ` Zhi Wang
@ 2018-04-23 10:25 ` Jani Nikula
2018-04-23 16:01 ` Zhi Wang
2018-04-24 10:17 ` Zhi Wang
1 sibling, 1 reply; 18+ messages in thread
From: Jani Nikula @ 2018-04-23 10:25 UTC (permalink / raw)
To: Zhi Wang, Zhenyu Wang, Joonas Lahtinen, Vivi, Rodrigo
Cc: intel-gfx, intel-gvt-dev, Lv, Zhiyuan, Yuan, Hang
On Mon, 23 Apr 2018, Zhi Wang <zhi.a.wang@intel.com> 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 efforts.
Thanks, pulled to drm-intel-next-queued.
Just as a reminder, if you have more stuff for v4.18 the deadline is in
about three weeks.
BR,
Jani.
>
> Thanks,
> Zhi.
>
> On 04/23/18 16:11, Zhi Wang wrote:
>> 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 3eda0d22ead04f81ea59c9584bcbf5b496745e92:
>>
>> drm/i915/gvt: Mark expected switch fall-through in
>> handle_g2v_notification (2018-04-23 13:09:36 +0800)
>>
>> ----------------------------------------------------------------
>> - Minor condition check improvment (Gustavo A. R. Silva)
>> - Non-priviliged batch buffer scan (Yan Zhao)
>> - Scheduling optimizations (Zhipeng Gong)
>>
>> ----------------------------------------------------------------
>> Gustavo A. R. Silva (2):
>> drm/i915/gvt/scheduler: Remove unnecessary NULL checks in sr_oa_regs
>> drm/i915/gvt: Mark expected switch fall-through in
>> handle_g2v_notification
>>
>> Zhao Yan (1):
>> drm/i915/gvt: scan non-privileged batch buffer for debug purpose
>>
>> Zhipeng Gong (2):
>> drm/i915/gvt: Use real time to do timer check
>> drm/i915/gvt: Update time slice more frequently
>>
>> drivers/gpu/drm/i915/gvt/cmd_parser.c | 55 ++++++++++++++++++++-------
>> drivers/gpu/drm/i915/gvt/debugfs.c | 67
>> +++++++++++++++++++++++++++++++++
>> drivers/gpu/drm/i915/gvt/gvt.h | 1 +
>> drivers/gpu/drm/i915/gvt/handlers.c | 1 +
>> drivers/gpu/drm/i915/gvt/sched_policy.c | 31 ++++++++-------
>> drivers/gpu/drm/i915/gvt/scheduler.c | 66
>> +++++++++++++++++++++-----------
>> drivers/gpu/drm/i915/gvt/scheduler.h | 1 +
>> drivers/gpu/drm/i915/gvt/trace.h | 24 +++++++++---
>> 8 files changed, 193 insertions(+), 53 deletions(-)
>>
--
Jani Nikula, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PULL] gvt-next for 4.18
2018-04-23 10:25 ` Jani Nikula
@ 2018-04-23 16:01 ` Zhi Wang
0 siblings, 0 replies; 18+ messages in thread
From: Zhi Wang @ 2018-04-23 16:01 UTC (permalink / raw)
To: Jani Nikula, Zhenyu Wang, Joonas Lahtinen, Vivi, Rodrigo
Cc: intel-gfx, intel-gvt-dev, Lv, Zhiyuan, Yuan, Hang
Thanks! :)
On 04/23/18 18:25, Jani Nikula wrote:
> On Mon, 23 Apr 2018, Zhi Wang <zhi.a.wang@intel.com> 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 efforts.
>
> Thanks, pulled to drm-intel-next-queued.
>
> Just as a reminder, if you have more stuff for v4.18 the deadline is in
> about three weeks.
>
> BR,
> Jani.
>
>
>>
>> Thanks,
>> Zhi.
>>
>> On 04/23/18 16:11, Zhi Wang wrote:
>>> 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 3eda0d22ead04f81ea59c9584bcbf5b496745e92:
>>>
>>> drm/i915/gvt: Mark expected switch fall-through in
>>> handle_g2v_notification (2018-04-23 13:09:36 +0800)
>>>
>>> ----------------------------------------------------------------
>>> - Minor condition check improvment (Gustavo A. R. Silva)
>>> - Non-priviliged batch buffer scan (Yan Zhao)
>>> - Scheduling optimizations (Zhipeng Gong)
>>>
>>> ----------------------------------------------------------------
>>> Gustavo A. R. Silva (2):
>>> drm/i915/gvt/scheduler: Remove unnecessary NULL checks in sr_oa_regs
>>> drm/i915/gvt: Mark expected switch fall-through in
>>> handle_g2v_notification
>>>
>>> Zhao Yan (1):
>>> drm/i915/gvt: scan non-privileged batch buffer for debug purpose
>>>
>>> Zhipeng Gong (2):
>>> drm/i915/gvt: Use real time to do timer check
>>> drm/i915/gvt: Update time slice more frequently
>>>
>>> drivers/gpu/drm/i915/gvt/cmd_parser.c | 55 ++++++++++++++++++++-------
>>> drivers/gpu/drm/i915/gvt/debugfs.c | 67
>>> +++++++++++++++++++++++++++++++++
>>> drivers/gpu/drm/i915/gvt/gvt.h | 1 +
>>> drivers/gpu/drm/i915/gvt/handlers.c | 1 +
>>> drivers/gpu/drm/i915/gvt/sched_policy.c | 31 ++++++++-------
>>> drivers/gpu/drm/i915/gvt/scheduler.c | 66
>>> +++++++++++++++++++++-----------
>>> drivers/gpu/drm/i915/gvt/scheduler.h | 1 +
>>> drivers/gpu/drm/i915/gvt/trace.h | 24 +++++++++---
>>> 8 files changed, 193 insertions(+), 53 deletions(-)
>>>
>
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PULL] gvt-next for 4.18
2018-04-23 8:13 ` Zhi Wang
2018-04-23 10:25 ` Jani Nikula
@ 2018-04-24 10:17 ` Zhi Wang
2018-05-13 22:17 ` Zhi Wang
1 sibling, 1 reply; 18+ messages in thread
From: Zhi Wang @ 2018-04-24 10:17 UTC (permalink / raw)
To: Jani Nikula, Zhenyu Wang, Joonas Lahtinen, Vivi, Rodrigo
Cc: intel-gfx, intel-gvt-dev, Lv, Zhiyuan, Yuan, Hang
The patch which causes conflict:
---
From 2e8ea2777af45fe651dbd14b8b8e000e94467c24 Mon Sep 17 00:00:00 2001
From: Weinan Li <weinan.z.li@intel.com>
Date: Wed, 21 Mar 2018 15:40:32 +0800
Subject: [PATCH] Revert "drm/i915/gvt: set max priority for gvt context"
This reverts commit 11474e9091cf2002e948647fd9f63a7f027e488a.
There are issues which will block the host preemption before, instead of
disabling it use one workaround "setting max priority for gvt context"
to avoid the gvt context be preempted by the host. Now the issues have been
cleared, so revert this patch to enable host preemption.
v2:
- refine description(Zhenyu)
Signed-off-by: Weinan Li <weinan.z.li@intel.com>
Signed-off-by: Zhenyu Wang <zhenyuw@linux.intel.com>
Signed-off-by: Zhi Wang <zhi.a.wang@intel.com>
---
has a dependency with the following patch which is missing in git tag
drm-next-intel-2018-04-13:
---
commit b7268c5eed0ab4f052d614b4b0e3fe8a51c9d5a1
Author: Chris Wilson <chris@chris-wilson.co.uk>
Date: Wed Apr 18 19:40:52 2018 +0100
drm/i915: Pack params to engine->schedule() into a struct
Today we only want to pass along the priority to
engine->schedule(), but
in the future we want to have much more control over the various
aspects
of the GPU during a context's execution, for example controlling the
frequency allowed. As we need an ever growing number of parameters for
scheduling, move those into a struct for convenience.
v2: Move the anonymous struct into its own function for legibility and
ye olde gcc.
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Link:
https://patchwork.freedesktop.org/patch/msgid/20180418184052.7129-3-chris@chris-wilson.co.uk
---
I found it in drm-tip. Looks I have to send another pull request when
the patch above is in a new drm-intel-next tag.
Thanks for everyone's efforts.
Thanks,
Zhi.
On 04/23/18 16:13, 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 efforts.
>
> Thanks,
> Zhi.
>
> On 04/23/18 16:11, Zhi Wang wrote:
>> 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 3eda0d22ead04f81ea59c9584bcbf5b496745e92:
>>
>> drm/i915/gvt: Mark expected switch fall-through in
>> handle_g2v_notification (2018-04-23 13:09:36 +0800)
>>
>> ----------------------------------------------------------------
>> - Minor condition check improvment (Gustavo A. R. Silva)
>> - Non-priviliged batch buffer scan (Yan Zhao)
>> - Scheduling optimizations (Zhipeng Gong)
>>
>> ----------------------------------------------------------------
>> Gustavo A. R. Silva (2):
>> drm/i915/gvt/scheduler: Remove unnecessary NULL checks in
>> sr_oa_regs
>> drm/i915/gvt: Mark expected switch fall-through in
>> handle_g2v_notification
>>
>> Zhao Yan (1):
>> drm/i915/gvt: scan non-privileged batch buffer for debug purpose
>>
>> Zhipeng Gong (2):
>> drm/i915/gvt: Use real time to do timer check
>> drm/i915/gvt: Update time slice more frequently
>>
>> drivers/gpu/drm/i915/gvt/cmd_parser.c | 55
>> ++++++++++++++++++++-------
>> drivers/gpu/drm/i915/gvt/debugfs.c | 67
>> +++++++++++++++++++++++++++++++++
>> drivers/gpu/drm/i915/gvt/gvt.h | 1 +
>> drivers/gpu/drm/i915/gvt/handlers.c | 1 +
>> drivers/gpu/drm/i915/gvt/sched_policy.c | 31 ++++++++-------
>> drivers/gpu/drm/i915/gvt/scheduler.c | 66
>> +++++++++++++++++++++-----------
>> drivers/gpu/drm/i915/gvt/scheduler.h | 1 +
>> drivers/gpu/drm/i915/gvt/trace.h | 24 +++++++++---
>> 8 files changed, 193 insertions(+), 53 deletions(-)
>>
> _______________________________________________
> intel-gvt-dev mailing list
> intel-gvt-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gvt-dev
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 18+ messages in thread
* [PULL] gvt-next for 4.18
2018-04-24 10:17 ` Zhi Wang
@ 2018-05-13 22:17 ` Zhi Wang
2018-05-14 8:35 ` Jani Nikula
0 siblings, 1 reply; 18+ messages in thread
From: Zhi Wang @ 2018-05-13 22:17 UTC (permalink / raw)
To: Jani Nikula, Zhenyu Wang, Joonas Lahtinen, Vivi, Rodrigo
Cc: intel-gfx, intel-gvt-dev, Lv, Zhiyuan, Yuan, Hang
The following changes since commit 0c79f9cb77eae28d48a4f9fc1b3341aacbbd260c:
drm/i915/gen9: Add WaClearHIZ_WM_CHICKEN3 for bxt and glk (2018-05-13
10:29:44 +0100)
are available in the git repository at:
https://github.com/intel/gvt-linux.git tags/gvt-next-2018-05-14
for you to fetch changes up to 41e403d04e7050c8d88682939febcdbe117d4c82:
Revert "drm/i915/gvt: set max priority for gvt context" (2018-05-14
05:26:09 +0800)
----------------------------------------------------------------
- Improve the emulation of virtual non-priv register. (Yan)
- Reverse the hack of host of preemption of GVT-g. (Weinan)
- Improve untracked warning message.(Changbin)
----------------------------------------------------------------
Changbin Du (1):
drm/i915/gvt: Remove disable_warn_untrack and print untracked
mmio with debug level
Weinan Li (1):
Revert "drm/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 LRI cmd
Zhi Wang (1):
Merge branch 'drm-intel-next-queued' into gvt-next
drivers/gpu/drm/i915/gvt/cmd_parser.c | 26 +++++++++++++++++++++-----
drivers/gpu/drm/i915/gvt/gvt.h | 1 -
drivers/gpu/drm/i915/gvt/handlers.c | 35
++++++++++++++++++-----------------
drivers/gpu/drm/i915/gvt/mmio.c | 2 --
drivers/gpu/drm/i915/gvt/scheduler.c | 3 ---
5 files changed, 39 insertions(+), 28 deletions(-)
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PULL] gvt-next for 4.18
2018-05-13 22:17 ` Zhi Wang
@ 2018-05-14 8:35 ` Jani Nikula
2018-05-14 12:13 ` Wang, Zhi A
0 siblings, 1 reply; 18+ messages in thread
From: Jani Nikula @ 2018-05-14 8:35 UTC (permalink / raw)
To: Zhi Wang, Zhenyu Wang, Joonas Lahtinen, Vivi, Rodrigo
Cc: intel-gfx, intel-gvt-dev, Lv, Zhiyuan, Yuan, Hang
On Mon, 14 May 2018, Zhi Wang <zhi.a.wang@intel.com> wrote:
> The following changes since commit 0c79f9cb77eae28d48a4f9fc1b3341aacbbd260c:
>
> drm/i915/gen9: Add WaClearHIZ_WM_CHICKEN3 for bxt and glk (2018-05-13
> 10:29:44 +0100)
>
> are available in the git repository at:
>
> https://github.com/intel/gvt-linux.git tags/gvt-next-2018-05-14
>
> for you to fetch changes up to 41e403d04e7050c8d88682939febcdbe117d4c82:
>
> Revert "drm/i915/gvt: set max priority for gvt context" (2018-05-14
> 05:26:09 +0800)
Pulled, thanks.
Please try to avoid rebases; it looks like you're now the committer for
each commit but without Signed-off-by.
BR,
Jani.
>
> ----------------------------------------------------------------
> - Improve the emulation of virtual non-priv register. (Yan)
> - Reverse the hack of host of preemption of GVT-g. (Weinan)
> - Improve untracked warning message.(Changbin)
>
> ----------------------------------------------------------------
> Changbin Du (1):
> drm/i915/gvt: Remove disable_warn_untrack and print untracked
> mmio with debug level
>
> Weinan Li (1):
> Revert "drm/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 LRI cmd
>
> Zhi Wang (1):
> Merge branch 'drm-intel-next-queued' into gvt-next
>
> drivers/gpu/drm/i915/gvt/cmd_parser.c | 26 +++++++++++++++++++++-----
> drivers/gpu/drm/i915/gvt/gvt.h | 1 -
> drivers/gpu/drm/i915/gvt/handlers.c | 35
> ++++++++++++++++++-----------------
> drivers/gpu/drm/i915/gvt/mmio.c | 2 --
> drivers/gpu/drm/i915/gvt/scheduler.c | 3 ---
> 5 files changed, 39 insertions(+), 28 deletions(-)
>
--
Jani Nikula, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PULL] gvt-next for 4.18
2018-05-14 8:35 ` Jani Nikula
@ 2018-05-14 12:13 ` Wang, Zhi A
0 siblings, 0 replies; 18+ messages in thread
From: Wang, Zhi A @ 2018-05-14 12:13 UTC (permalink / raw)
To: Nikula, Jani, Zhenyu Wang, Joonas Lahtinen, Vivi, Rodrigo
Cc: intel-gfx, intel-gvt-dev, Lv, Zhiyuan, Yuan, Hang
Thanks for the reminder! :)
Thanks,
Zhi.
-----Original Message-----
From: Nikula, Jani
Sent: Monday, May 14, 2018 11:36 AM
To: Wang, Zhi A <zhi.a.wang@intel.com>; Zhenyu Wang <zhenyuw@linux.intel.com>; Joonas Lahtinen <joonas.lahtinen@linux.intel.com>; Vivi, Rodrigo <rodrigo.vivi@intel.com>
Cc: intel-gfx <intel-gfx@lists.freedesktop.org>; intel-gvt-dev <intel-gvt-dev@lists.freedesktop.org>; Lv, Zhiyuan <zhiyuan.lv@intel.com>; Yuan, Hang <hang.yuan@intel.com>
Subject: Re: [PULL] gvt-next for 4.18
On Mon, 14 May 2018, Zhi Wang <zhi.a.wang@intel.com> wrote:
> The following changes since commit 0c79f9cb77eae28d48a4f9fc1b3341aacbbd260c:
>
> drm/i915/gen9: Add WaClearHIZ_WM_CHICKEN3 for bxt and glk
> (2018-05-13
> 10:29:44 +0100)
>
> are available in the git repository at:
>
> https://github.com/intel/gvt-linux.git tags/gvt-next-2018-05-14
>
> for you to fetch changes up to 41e403d04e7050c8d88682939febcdbe117d4c82:
>
> Revert "drm/i915/gvt: set max priority for gvt context" (2018-05-14
> 05:26:09 +0800)
Pulled, thanks.
Please try to avoid rebases; it looks like you're now the committer for each commit but without Signed-off-by.
BR,
Jani.
>
> ----------------------------------------------------------------
> - Improve the emulation of virtual non-priv register. (Yan)
> - Reverse the hack of host of preemption of GVT-g. (Weinan)
> - Improve untracked warning message.(Changbin)
>
> ----------------------------------------------------------------
> Changbin Du (1):
> drm/i915/gvt: Remove disable_warn_untrack and print untracked
> mmio with debug level
>
> Weinan Li (1):
> Revert "drm/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
> LRI cmd
>
> Zhi Wang (1):
> Merge branch 'drm-intel-next-queued' into gvt-next
>
> drivers/gpu/drm/i915/gvt/cmd_parser.c | 26 +++++++++++++++++++++-----
> drivers/gpu/drm/i915/gvt/gvt.h | 1 -
> drivers/gpu/drm/i915/gvt/handlers.c | 35
> ++++++++++++++++++-----------------
> drivers/gpu/drm/i915/gvt/mmio.c | 2 --
> drivers/gpu/drm/i915/gvt/scheduler.c | 3 ---
> 5 files changed, 39 insertions(+), 28 deletions(-)
>
--
Jani Nikula, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PULL] gvt-fixes for 4.17
2018-04-19 7:39 [PULL] gvt-fixes for 4.17 Zhenyu Wang
2018-04-19 9:17 ` [PULL] gvt-next " Zhi Wang
@ 2018-06-06 7:49 ` Zhenyu Wang
2018-06-06 13:11 ` Joonas Lahtinen
1 sibling, 1 reply; 18+ messages in thread
From: Zhenyu Wang @ 2018-06-06 7:49 UTC (permalink / raw)
To: Joonas Lahtinen
Cc: Jani Nikula, intel-gfx, Yuan, Hang, Lv, Zhiyuan, Vivi, Rodrigo,
intel-gvt-dev
[-- Attachment #1.1: Type: text/plain, Size: 2419 bytes --]
On 2018.04.19 15:39:48 +0800, Zhenyu Wang wrote:
>
> Hi,
>
> Here's current gvt fixes for 4.17 with several kernel warning
> and other misc fixes as detailed below.
>
> p.s: I'll be on vacation from next week till May 2, Zhi will cover for me.
>
> Thanks
> --
Looks this one got missed for merge...just found when trying to apply new
change that caused conflict..Pls help to merge and will request a backmerge
to apply -next change against those.
thanks
> The following changes since commit 60cc43fc888428bb2f18f08997432d426a243338:
>
> Linux 4.17-rc1 (2018-04-15 18:24:20 -0700)
>
> are available in the Git repository at:
>
> https://github.com/intel/gvt-linux.git tags/gvt-fixes-2018-04-19
>
> for you to fetch changes up to 39b4cbadb9a95bf3f13ea102d6ec841940916ee2:
>
> drm/i915/kvmgt: Check the pfn got from vfio_pin_pages (2018-04-17 10:46:49 +0800)
>
> ----------------------------------------------------------------
> gvt-fixes-2018-04-19
>
> - cmd parser error path mem leak fix (Colin)
> - fix dp aux header validation (Changbin)
> - sanity check on pfn after vfio pin page (Changbin)
> - fix msi eventfd put (Xiong)
>
> ----------------------------------------------------------------
> Changbin Du (2):
> drm/i915/gvt: Fix the validation on size field of dp aux header
> drm/i915/kvmgt: Check the pfn got from vfio_pin_pages
>
> Colin Ian King (1):
> drm/i915/gvt: fix memory leak of a cmd_entry struct on error exit path
>
> Xiong Zhang (1):
> drm/i915/gvt: Dereference msi eventfd_ctx when it isn't used anymore
>
> Zhenyu Wang (1):
> Back merge 'drm-intel-fixes' into gvt-fixes
>
> drivers/gpu/drm/i915/gvt/cmd_parser.c | 1 +
> drivers/gpu/drm/i915/gvt/display.h | 2 +-
> drivers/gpu/drm/i915/gvt/handlers.c | 13 +++++++++----
> drivers/gpu/drm/i915/gvt/kvmgt.c | 34 +++++++++++++++++++++++++++++++++-
> 4 files changed, 44 insertions(+), 6 deletions(-)
>
>
> --
> Open Source Technology Center, Intel ltd.
>
> $gpg --keyserver wwwkeys.pgp.net --recv-keys 4D781827
> _______________________________________________
> intel-gvt-dev mailing list
> intel-gvt-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gvt-dev
--
Open Source Technology Center, Intel ltd.
$gpg --keyserver wwwkeys.pgp.net --recv-keys 4D781827
[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 bytes --]
[-- Attachment #2: Type: text/plain, Size: 160 bytes --]
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PULL] gvt-fixes for 4.17
2018-06-06 7:49 ` [PULL] gvt-fixes for 4.17 Zhenyu Wang
@ 2018-06-06 13:11 ` Joonas Lahtinen
2018-06-08 12:04 ` Jani Nikula
0 siblings, 1 reply; 18+ messages in thread
From: Joonas Lahtinen @ 2018-06-06 13:11 UTC (permalink / raw)
To: Zhenyu Wang
Cc: Jani Nikula, intel-gfx, Yuan, Hang, Lv, Zhiyuan, Vivi, Rodrigo,
intel-gvt-dev
Quoting Zhenyu Wang (2018-06-06 10:49:54)
> On 2018.04.19 15:39:48 +0800, Zhenyu Wang wrote:
> >
> > Hi,
> >
> > Here's current gvt fixes for 4.17 with several kernel warning
> > and other misc fixes as detailed below.
> >
> > p.s: I'll be on vacation from next week till May 2, Zhi will cover for me.
> >
> > Thanks
> > --
>
> Looks this one got missed for merge...just found when trying to apply new
> change that caused conflict..Pls help to merge and will request a backmerge
> to apply -next change against those.
As discussed in IRC, the gvt-next PR was as --in-reply-to for this
message, so I indeed confused it for being discussion about the gvt-next
between Jani and Zhi, and missed it :(
Regards, Joonas
>
> thanks
>
> > The following changes since commit 60cc43fc888428bb2f18f08997432d426a243338:
> >
> > Linux 4.17-rc1 (2018-04-15 18:24:20 -0700)
> >
> > are available in the Git repository at:
> >
> > https://github.com/intel/gvt-linux.git tags/gvt-fixes-2018-04-19
> >
> > for you to fetch changes up to 39b4cbadb9a95bf3f13ea102d6ec841940916ee2:
> >
> > drm/i915/kvmgt: Check the pfn got from vfio_pin_pages (2018-04-17 10:46:49 +0800)
> >
> > ----------------------------------------------------------------
> > gvt-fixes-2018-04-19
> >
> > - cmd parser error path mem leak fix (Colin)
> > - fix dp aux header validation (Changbin)
> > - sanity check on pfn after vfio pin page (Changbin)
> > - fix msi eventfd put (Xiong)
> >
> > ----------------------------------------------------------------
> > Changbin Du (2):
> > drm/i915/gvt: Fix the validation on size field of dp aux header
> > drm/i915/kvmgt: Check the pfn got from vfio_pin_pages
> >
> > Colin Ian King (1):
> > drm/i915/gvt: fix memory leak of a cmd_entry struct on error exit path
> >
> > Xiong Zhang (1):
> > drm/i915/gvt: Dereference msi eventfd_ctx when it isn't used anymore
> >
> > Zhenyu Wang (1):
> > Back merge 'drm-intel-fixes' into gvt-fixes
> >
> > drivers/gpu/drm/i915/gvt/cmd_parser.c | 1 +
> > drivers/gpu/drm/i915/gvt/display.h | 2 +-
> > drivers/gpu/drm/i915/gvt/handlers.c | 13 +++++++++----
> > drivers/gpu/drm/i915/gvt/kvmgt.c | 34 +++++++++++++++++++++++++++++++++-
> > 4 files changed, 44 insertions(+), 6 deletions(-)
> >
> >
> > --
> > Open Source Technology Center, Intel ltd.
> >
> > $gpg --keyserver wwwkeys.pgp.net --recv-keys 4D781827
>
>
>
> > _______________________________________________
> > intel-gvt-dev mailing list
> > intel-gvt-dev@lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/intel-gvt-dev
>
>
> --
> Open Source Technology Center, Intel ltd.
>
> $gpg --keyserver wwwkeys.pgp.net --recv-keys 4D781827
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PULL] gvt-fixes for 4.17
2018-06-06 13:11 ` Joonas Lahtinen
@ 2018-06-08 12:04 ` Jani Nikula
0 siblings, 0 replies; 18+ messages in thread
From: Jani Nikula @ 2018-06-08 12:04 UTC (permalink / raw)
To: Joonas Lahtinen, Zhenyu Wang
Cc: intel-gfx, Yuan, Hang, Lv, Zhiyuan, Vivi, Rodrigo, intel-gvt-dev
On Wed, 06 Jun 2018, Joonas Lahtinen <joonas.lahtinen@linux.intel.com> wrote:
> Quoting Zhenyu Wang (2018-06-06 10:49:54)
>> On 2018.04.19 15:39:48 +0800, Zhenyu Wang wrote:
>> >
>> > Hi,
>> >
>> > Here's current gvt fixes for 4.17 with several kernel warning
>> > and other misc fixes as detailed below.
>> >
>> > p.s: I'll be on vacation from next week till May 2, Zhi will cover for me.
>> >
>> > Thanks
>> > --
>>
>> Looks this one got missed for merge...just found when trying to apply new
>> change that caused conflict..Pls help to merge and will request a backmerge
>> to apply -next change against those.
>
> As discussed in IRC, the gvt-next PR was as --in-reply-to for this
> message, so I indeed confused it for being discussion about the gvt-next
> between Jani and Zhi, and missed it :(
So this missed v4.17. I've now pulled this into drm-intel-next-fixes,
and it'll get merged upstream for v4.18. If you need the commits
backported, you'll need to make stable backport requests to the stable
team after the commits have been merged to Linus' master.
BR,
Jani.
--
Jani Nikula, Intel Open Source Graphics Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 18+ messages in thread
end of thread, other threads:[~2018-06-08 12:04 UTC | newest]
Thread overview: 18+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-04-19 7:39 [PULL] gvt-fixes for 4.17 Zhenyu Wang
2018-04-19 9:17 ` [PULL] gvt-next " Zhi Wang
2018-04-19 9:21 ` Zhi Wang
2018-04-19 9:34 ` Jani Nikula
2018-04-19 9:41 ` Zhenyu Wang
2018-04-19 9:50 ` Zhi Wang
2018-04-19 10:16 ` Zhi Wang
2018-04-23 8:11 ` [PULL] gvt-next for 4.18 Zhi Wang
2018-04-23 8:13 ` Zhi Wang
2018-04-23 10:25 ` Jani Nikula
2018-04-23 16:01 ` Zhi Wang
2018-04-24 10:17 ` Zhi Wang
2018-05-13 22:17 ` Zhi Wang
2018-05-14 8:35 ` Jani Nikula
2018-05-14 12:13 ` Wang, Zhi A
2018-06-06 7:49 ` [PULL] gvt-fixes for 4.17 Zhenyu Wang
2018-06-06 13:11 ` Joonas Lahtinen
2018-06-08 12:04 ` Jani Nikula
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.