Hi：
Thanks for the patch. This problem has been fixed on staging branch with:
a8e18bfc854141621a29600a6c13e5f40c6e9f19
drm/i915/gvt: Refine shadow batch buffer
Thanks,
Zhi.
-Original Message-
From: Christos Gkekas [mailto:chris.ge...@gmail.com]
Sent: Monday, October 16, 2017 1:18 A
== Series Details ==
Series: drm/i915/gvt: use ARRAY_SIZE
URL : https://patchwork.freedesktop.org/series/32003/
State : success
== Summary ==
Test kms_setmode:
Subgroup basic:
fail -> PASS (shard-hsw) fdo#99912
Test kms_plane:
Subgroup plane-panning-
== Series Details ==
Series: drm/i915/gvt: use ARRAY_SIZE
URL : https://patchwork.freedesktop.org/series/32003/
State : success
== Summary ==
Series 32003v1 drm/i915/gvt: use ARRAY_SIZE
https://patchwork.freedesktop.org/api/1.0/series/32003/revisions/1/mbox/
Test kms_pipe_crc_basic:
S
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 || report)@
type T;
T[] E;
position p;
@@
(
(sizeof(E)@p /sizeof(*E)
== Series Details ==
Series: drm/i915: Add in-flight request details to intel_engine_dump()
URL : https://patchwork.freedesktop.org/series/32001/
State : warning
== Summary ==
Test kms_plane:
Subgroup plane-panning-bottom-right-suspend-pipe-C-planes:
skip -> PASS
== Series Details ==
Series: drm/i915: Add in-flight request details to intel_engine_dump()
URL : https://patchwork.freedesktop.org/series/32001/
State : success
== Summary ==
Series 32001v1 drm/i915: Add in-flight request details to intel_engine_dump()
https://patchwork.freedesktop.org/api/1.
Quoting Oscar Mateo (2017-10-13 21:55:59)
> @@ -134,20 +147,20 @@ static int workaround_fail_count(int fd, uint32_t ctx)
> igt_debug("Addressvalmaskreadresult\n");
>
> out = gem_mmap__cpu(fd, obj[0].handle, 0, result_sz, PROT_READ);
> - for (int i
Quoting Michal Wajdeczko (2017-10-14 21:02:05)
> On Sat, 14 Oct 2017 12:12:50 +0200, Chris Wilson
> > diff --git a/drivers/gpu/drm/i915/Makefile
> > b/drivers/gpu/drm/i915/Makefile
> > index 52f0f8f101ec..e1bbb2565426 100644
> > --- a/drivers/gpu/drm/i915/Makefile
> > +++ b/drivers/gpu/drm/i915
In the intel_engine_cs dumper, we were showing the request details for
the request queue but not of those requests already passed to the hw
(just a summary of the seqno). If we show those details, we can then
eliminate the entirely redundant and forgotten debugfs/i915_gem_request
Signed-off-by: Ch
Quoting Chris Wilson (2017-10-15 15:37:25)
> If we fail to recover the HW state upon resume (i.e. our attempt to
> clear the wedged bit and reset during i915_gem_sanitize() fails), then
> skip the HW restart inside i915_gem_init_hw(). We will ultimately do the
> HW restart when successfully unwedgi
== Series Details ==
Series: drm/i915: Skip HW reinitialisation on resume if still wedged (rev2)
URL : https://patchwork.freedesktop.org/series/31987/
State : success
== Summary ==
Test kms_plane:
Subgroup plane-panning-bottom-right-suspend-pipe-C-planes:
skip ->
== Series Details ==
Series: drm/i915: Skip HW reinitialisation on resume if still wedged (rev2)
URL : https://patchwork.freedesktop.org/series/31987/
State : success
== Summary ==
Series 31987v2 drm/i915: Skip HW reinitialisation on resume if still wedged
https://patchwork.freedesktop.org/api
== Series Details ==
Series: drm/i915: Skip HW reinitialisation on resume if still wedged
URL : https://patchwork.freedesktop.org/series/31987/
State : success
== Summary ==
Series 31987v1 drm/i915: Skip HW reinitialisation on resume if still wedged
https://patchwork.freedesktop.org/api/1.0/se
If we fail to recover the HW state upon resume (i.e. our attempt to
clear the wedged bit and reset during i915_gem_sanitize() fails), then
skip the HW restart inside i915_gem_init_hw(). We will ultimately do the
HW restart when successfully unwedging and resetting the HW later,
but attempting to re
If we fail to recover the HW state upon resume (i.e. our attempt to
clear the wedged bit and reset during i915_gem_sanitize() fails), then
skip the HW restart inside i915_gem_init_hw(). We will ultimate do the
the HW restart when sucessfully unwedgeding and reseting the HW later,
but attempting to
== Series Details ==
Series: igt/core_suspend: Exercise igt_system_suspend_autoresume()
URL : https://patchwork.freedesktop.org/series/31986/
State : failure
== Summary ==
IGT patchset tested on top of latest successful build
58616272b23efce1e62a3ee0d37e13de6ffc012f igt/gem_eio: Check hang/eio
We have various basic suspend and debug modes, so iterate over them to
check they each work without test interference. This help work out the
cause of any suspend bugs.
Signed-off-by: Chris Wilson
---
tests/Makefile.sources | 1 +
tests/core_suspend.c | 60
Quoting Weinan Li (2017-10-15 04:55:25)
> Let GVT-g VM read the CSB and CSB write pointer from virtual HWSP, not all
> the host support this feature, need to check the BIT(3) of caps in PVINFO.
>
> v3 : Remove unnecessary comments.
> v4 : Separate VM enable patch with GVT-g implementation patch du
18 matches
Mail list logo
