* Joonas Lahtinen wrote:
> + Dave as a heads-up
>
> On Thu, 2017-11-23 at 07:17 +0100, Ingo Molnar wrote:
> > * Matthew Auld wrote:
> >
> > > We duplicate the stolen discovery code in early-quirks and in i915,
> > > however if we just export the region as a resource from early-quirks we
> > >
On Wed, Nov 22, 2017 at 11:50:48PM -0800, Nick Desaulniers wrote:
> Fixes the following enum conversion warning:
>
> drivers/gpu/drm/i915/intel_ddi.c:1481:30: error: implicit conversion
> from enumeration type 'enum port' to different enumeration type 'enum
> intel_dpll_id' [-Werror,-Wenum-convers
On Wed, Nov 22, 2017 at 02:56:10PM +0200, Ville Syrjälä wrote:
> On Wed, Nov 22, 2017 at 01:35:39PM +0100, Daniel Vetter wrote:
> > On Wed, Nov 22, 2017 at 1:25 PM, Daniel Vetter
> > wrote:
> > > On Mon, Nov 13, 2017 at 3:36 PM, Maarten Lankhorst
> > > wrote:
> > >> Op 13-11-17 om 14:36 schreef
Make sure the HW is idle before we start sampling the GPU for busyness.
If we do not rest for long enough between tests, we may carry the
sampling over.
Signed-off-by: Chris Wilson
Cc: Tvrtko Ursulin
---
tests/perf_pmu.c | 18 ++
1 file changed, 18 insertions(+)
diff --git a/te
gem_quiescent_gpu() is supposed to ensure that the HW is idle, and in
the process kick the GPU into rc6, so we should not need a long delay
afterwards to ensure that we are indeed in rc6. We do however need a
small delay in order to be sure that rc6 cycle counter has started and
stopped.
v2: Apply
The stragglers for easy consumption and discussion.
-Chris
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
Program the MI_WAIT_FOR_EVENT without reference to DERRMR by knowing its
state is ~0u when not in use, and is only in use when userspace requires
it. By not touching intel_regsiter_access we completely eliminate the
risk that we leak the forcewake ref, which can cause later rc6 to fail.
At the sam
v18->v19:
1) include "Handle-orphan-dmabuf_objs.patch".
2) rebase to the latest staging branch.
v17->v18:
1) unmap vgpu's opregion when destroying vgpu.
2) update comments for VFIO_DEVICE_GET_GFX_DMABUF. (Alex)
v16->v17:
1) modify VFIO_DEVICE_GET_GFX_DMABUF interface. (Alex)
2) add comments for x
Add VFIO_DEVICE_QUERY_GFX_PLANE ioctl command to let user query and get
a plane and its information. So far, two types of buffers are supported:
buffers based on dma-buf and buffers based on region.
This ioctl can be invoked with:
1) Either DMABUF or REGION flag. Vendor driver returns a plane_info
This patch is to introduce the framebuffer decoder which can decode guest
OS's framebuffer information, including primary, cursor and sprite plane.
v16:
- rebase to 4.14.0-rc6.
v14:
- refine pixel format table. (Zhenyu)
v9:
- move drm format change to a separate patch. (Xiaoguang)
v8:
- fix a b
This patch introduces a guest's framebuffer sharing mechanism based on
dma-buf subsystem. With this sharing mechanism, guest's framebuffer can
be shared between guest VM and host.
v17:
- modify VFIO_DEVICE_GET_GFX_DMABUF interface. (Alex)
v16:
- add x_hot and y_hot. (Gerd)
- add flag validation f
dmabuf_obj's destruction relys on GEM release operation, which is managed
in i915 driver. And there is a time window between vgpu's destruction and
its dmabuf_objs' destruction. This patch is to free the orphan dmabuf_objs
correctly after the vgpu passes away.
Signed-off-by: Tina Zhang
Cc: Zhenyu
The RGB 64-bit 16:16:16:16 float pixel format is needed by some Apps in
windows. The float format in each component is 1:5:10 MSb-sign:exponent:
fraction.
This patch is to introduce the format to drm, so that the windows guest's
framebuffer in this kind of format can be recognized and used by linu
The RGB 64-bit 16:16:16:16 float pixel format is needed by windows 10
guest VM. This patch is to add this pixel format support to gvt device
model. Without this patch, some Apps, e.g. "DXGIGammaVM.exe", will crash
and make guest screen black.
Signed-off-by: Tina Zhang
---
drivers/gpu/drm/i915/gv
When printing the execlist ports, we first print the ELSP header then
follow it with the pretty-printed request. Since switching to
drm_printer and show the output via printk, it automatically appends a
newline to each call (unlike the old seq_printf output). To avoid the
unwanted line break, const
Simple va_args equivalent to the existing drm_printf() for use with the
drm_printer.
v2: Fixup kerneldoc to match final parameter names.
v3: Turn it into a kerneldoc comment
Signed-off-by: Chris Wilson
Cc: Rob Clark
Reviewed-by: Daniel Vetter
Cc: Dave Airlie
Acked-by: Dave Airlie
---
driver
Now that we have a common engine state pretty printer, we can use that
instead of the adhoc information printed when we miss a breadcrumb.
v2: Rearrange intel_engine_disarm_breadcrumbs() to avoid calling
intel_engine_dump() under the rb spinlock (Mika) and to pretty-print the
error state early so
Comparing the state tested by intel_engine_is_idle() and printed by
intel_engine_dump(), the only bit not shown is whether or not the device
is wedged. Add that little bit of information to the pretty printer so
that if the engine fails to idle we can see why.
Signed-off-by: Chris Wilson
Cc: Mika
Pass in a format string (and args) to specify the header to be emitted
along with the engine state when pretty-printing. This allows the header
to be emitted inside the drm_printer stream, so sharing the same prefix
and output characteristics (e.g. debug level and filtering).
Signed-off-by: Chris
Since a global reset affects the engine, include that along side the
per-engine reset counter when pretty printing the engine state in
intel_engine_dump().
Signed-off-by: Chris Wilson
Cc: Mika Kuoppala
Reviewed-by: Mika Kuoppala
---
drivers/gpu/drm/i915/intel_engine_cs.c | 5 +++--
1 file chan
On 23/11/2017 07:40, Chris Wilson wrote:
Quoting Tvrtko Ursulin (2017-11-23 07:36:35)
On 22/11/2017 18:26, Chris Wilson wrote:
Even for static CPU configurations, the hotplug CPU framework is still
used to determine the CPU topology, and is still being used by the perf
event register to check
When turning off the engines, and the pmu sampling, clear the previous
value as the current measurement should be 0.
v2: Use a for-loop
Signed-off-by: Chris Wilson
Cc: Tvrtko Ursulin
---
drivers/gpu/drm/i915/i915_pmu.c | 10 ++
1 file changed, 10 insertions(+)
diff --git a/drivers/gpu
== Series Details ==
Series: drm/i915: Avoid enum conversion warning
URL : https://patchwork.freedesktop.org/series/34269/
State : failure
== Summary ==
commit ffa17d69cf53bd5af4d7e41900c3ba40f6656e31
Author: Chris Wilson
Date: Thu Nov 23 07:47:39 2017 +
drm-tip: 2017y-11m-23d-07h-
== Series Details ==
Series: drm/i915/gvt: Dma-buf support for GVT-g
URL : https://patchwork.freedesktop.org/series/34272/
State : failure
== Summary ==
commit ffa17d69cf53bd5af4d7e41900c3ba40f6656e31
Author: Chris Wilson
Date: Thu Nov 23 07:47:39 2017 +
drm-tip: 2017y-11m-23d-07h-
This patch is from "[PATCH v19 0/6] drm/i915/gvt: Dma-buf support for GVT-g":
https://lists.freedesktop.org/archives/intel-gvt-dev/2017-November/002508.html
The RGB 64-bit 16:16:16:16 float pixel format is needed by some Apps in
windows. The float format in each component is 1:5:10 MSb-sign:expone
The RGB 64-bit 16:16:16:16 float pixel format is needed by some Apps in
windows. The float format in each component is 1:5:10 MSb-sign:exponent:
fraction.
This patch is to introduce the format to drm, so that the windows guest's
framebuffer in this kind of format can be recognized and used by linu
On Thu, 09 Nov 2017, Rodrigo Vivi wrote:
> On Thu, Nov 09, 2017 at 06:51:43PM +, Anusha Srivatsa wrote:
>> There is a new version of DMC available for KBL.
>>
>> The release notes mentions:
>> 1. Fix for the issue where DC_STATE was getting enabled even
>> when disabled by driver causing data
== Series Details ==
Series: series starting with [CI,1/6] drm/printer: Add drm_vprintf()
URL : https://patchwork.freedesktop.org/series/34278/
State : success
== Summary ==
Series 34278v1 series starting with [CI,1/6] drm/printer: Add drm_vprintf()
https://patchwork.freedesktop.org/api/1.0/se
From: Tvrtko Ursulin
As Chris has discover 100ms is not long enough to cover the sampling
error in general, fix the semaphore subtest as well to measure for
500ms.
Signed-off-by: Tvrtko Ursulin
Cc: Chris Wilson
---
tests/perf_pmu.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff -
Quoting Chris Wilson (2017-11-23 08:40:46)
> Simple va_args equivalent to the existing drm_printf() for use with the
> drm_printer.
>
> v2: Fixup kerneldoc to match final parameter names.
> v3: Turn it into a kerneldoc comment
>
> Signed-off-by: Chris Wilson
> Cc: Rob Clark
> Reviewed-by: Danie
Quoting Tvrtko Ursulin (2017-11-23 09:24:01)
> From: Tvrtko Ursulin
>
> As Chris has discover 100ms is not long enough to cover the sampling
> error in general, fix the semaphore subtest as well to measure for
> 500ms.
>
> Signed-off-by: Tvrtko Ursulin
> Cc: Chris Wilson
And one value for all
From: Tvrtko Ursulin
When turning off the engines, and the pmu sampling, clear the previous
value as the current measurement should be 0.
v2: Use a for-loop
v3:
* Move clearing to timer self-dis-arm to avoid race with parking.
* Clear frequency samples as well.
Signed-off-by: Tvrtko Ursulin
From: Tvrtko Ursulin
We have agreed during the engine classes discussion that fields marked as
non-ABI are better left out altogether from uapi headers.
Signed-off-by: Tvrtko Ursulin
---
drivers/gpu/drm/i915/intel_ringbuffer.h | 2 +-
include/uapi/drm/i915_drm.h | 3 +--
2 files ch
Hi Dave, some early fixes for v4.15 and stable.
BR,
Jani.
The following changes since commit e8c49fa96838101435b9e4884d49b30da7a4e0c6:
drm/i915: Reorder context-close to avoid calling i915_vma_close() under RCU
(2017-11-09 16:18:37 +0200)
are available in the git repository at:
git://ano
Op 23-11-17 om 09:11 schreef Daniel Vetter:
> On Wed, Nov 22, 2017 at 02:56:10PM +0200, Ville Syrjälä wrote:
>> On Wed, Nov 22, 2017 at 01:35:39PM +0100, Daniel Vetter wrote:
>>> On Wed, Nov 22, 2017 at 1:25 PM, Daniel Vetter
>>> wrote:
On Mon, Nov 13, 2017 at 3:36 PM, Maarten Lankhorst
Quoting Tvrtko Ursulin (2017-11-23 09:27:40)
> From: Tvrtko Ursulin
>
> When turning off the engines, and the pmu sampling, clear the previous
> value as the current measurement should be 0.
>
> v2: Use a for-loop
> v3:
> * Move clearing to timer self-dis-arm to avoid race with parking.
> * Cl
Quoting Tvrtko Ursulin (2017-11-23 09:32:29)
> From: Tvrtko Ursulin
>
> We have agreed during the engine classes discussion that fields marked as
> non-ABI are better left out altogether from uapi headers.
>
> Signed-off-by: Tvrtko Ursulin
> ---
> drivers/gpu/drm/i915/intel_ringbuffer.h | 2 +-
Hi Ville,
On Monday, 6 November 2017 20:04:38 EET Ville Syrjälä wrote:
> On Thu, Nov 02, 2017 at 03:19:30PM +0200, Ville Syrjälä wrote:
> > On Thu, Nov 02, 2017 at 11:12:09AM +0100, Daniel Vetter wrote:
> >> On Wed, Nov 01, 2017 at 08:29:18PM +0200, Ville Syrjala wrote:
> >>> From: Ville Syrjälä
Hi Ville,
Thank you for the patch.
On Wednesday, 1 November 2017 20:29:19 EET Ville Syrjala wrote:
> From: Ville Syrjälä
>
> drm_plane_helper_check_state() is supposed to do things the atomic way,
> so it should not be inspecting crtc->enabled. Rather we should be
> looking at crtc_state->enabl
Hi Ville,
Thank you for the patch.
On Wednesday, 1 November 2017 20:29:20 EET Ville Syrjala wrote:
> From: Ville Syrjälä
>
> drm_plane_helper_check_update() isn't a transitional helper, so let's
> rename it to drm_atomic_helper_check_plane_state() and move it into
> drm_atomic_helper.c.
>
> Cc
Hi Ville,
Thank you for the patch.
On Wednesday, 1 November 2017 20:29:16 EET Ville Syrjala wrote:
> From: Ville Syrjälä
>
> Throw away the bugs crtc coords vs. fb size check. Crtc coords don't
> define the viewport inside the fb, that's a job for the src coords,
> which have been checked by th
Hi Ville,
Thank you for the patch.
On Wednesday, 1 November 2017 20:29:17 EET Ville Syrjala wrote:
> From: Ville Syrjälä
>
> Atomic drivers have no reason to use drm_plane_helper_check_update()
> instead of drm_plane_helper_check_state(). So let's switch over.
>
> Cc: VMware Graphics
> Cc: Si
Hi Ville,
On Monday, 20 November 2017 21:36:46 EET Ville Syrjälä wrote:
> On Mon, Nov 20, 2017 at 09:32:56AM -0800, Sinclair Yeh wrote:
> > On Mon, Nov 20, 2017 at 08:34:50AM +0100, Daniel Vetter wrote:
> >> On Fri, Nov 10, 2017 at 11:42:59PM +0200, Ville Syrjälä wrote:
> >>> On Fri, Nov 10, 2017
Signed-off-by: Maarten Lankhorst
---
tests/kms_color.c | 26 +-
1 file changed, 13 insertions(+), 13 deletions(-)
diff --git a/tests/kms_color.c b/tests/kms_color.c
index dcda12de3a39..3b0a88802c93 100644
--- a/tests/kms_color.c
+++ b/tests/kms_color.c
@@ -855,7 +855,7 @@
There's no need to test this more than once.
Signed-off-by: Maarten Lankhorst
---
tests/kms_flip.c | 10 +-
tests/kms_vblank.c | 18 ++
2 files changed, 15 insertions(+), 13 deletions(-)
diff --git a/tests/kms_flip.c b/tests/kms_flip.c
index fb37859af8d2..da55263bb307
With the removal of mmio flips, blt and rcs flips are no longer
different from doing busy testing in kms_busy.
Signed-off-by: Maarten Lankhorst
---
tests/kms_flip.c | 37 +
1 file changed, 1 insertion(+), 36 deletions(-)
diff --git a/tests/kms_flip.c b/tests/
== Series Details ==
Series: drm: Introduce RGB 64-bit 16:16:16:16 float format
URL : https://patchwork.freedesktop.org/series/34283/
State : success
== Summary ==
Series 34283v1 drm: Introduce RGB 64-bit 16:16:16:16 float format
https://patchwork.freedesktop.org/api/1.0/series/34283/revisions
From: Tvrtko Ursulin
We have agreed during the engine classes discussion that fields marked as
non-ABI are better left out altogether from uapi headers.
v2: Use a local define for maintanability. (Chris Wilson)
Signed-off-by: Tvrtko Ursulin
---
drivers/gpu/drm/i915/intel_ringbuffer.h | 1 +
i
Quoting Tvrtko Ursulin (2017-11-23 10:07:01)
> From: Tvrtko Ursulin
>
> We have agreed during the engine classes discussion that fields marked as
> non-ABI are better left out altogether from uapi headers.
>
> v2: Use a local define for maintanability. (Chris Wilson)
>
> Signed-off-by: Tvrtko U
On 11/23/2017 3:07 PM, Chris Wilson wrote:
Quoting Tvrtko Ursulin (2017-11-23 09:27:40)
From: Tvrtko Ursulin
When turning off the engines, and the pmu sampling, clear the previous
value as the current measurement should be 0.
v2: Use a for-loop
v3:
* Move clearing to timer self-dis-arm to
== Series Details ==
Series: drm/i915/pmu: Clear the previous sample value when parking (rev3)
URL : https://patchwork.freedesktop.org/series/34262/
State : success
== Summary ==
Series 34262v3 drm/i915/pmu: Clear the previous sample value when parking
https://patchwork.freedesktop.org/api/1.0
On 22 November 2017 at 21:43, Chris Wilson wrote:
> Quoting Matthew Auld (2017-11-22 21:19:16)
>> The name mappable_end seems to suggest the end of the mappable region,
>
> This seems to be arguing for using _end :)
>
>> but is actually just the size, so make that obvious to the reader.
>
> We did
From: Tvrtko Ursulin
When turning off the engines, and the pmu sampling, clear the previous
value as the current measurement should be 0.
v2: Use a for-loop
v3:
* Move clearing to timer self-dis-arm to avoid race with parking.
* Clear frequency samples as well.
v4:
* Init frequency to idle_fr
On Thu, 2017-11-23 at 09:26 +, Chris Wilson wrote:
> Quoting Chris Wilson (2017-11-23 08:40:46)
> > Simple va_args equivalent to the existing drm_printf() for use with the
> > drm_printer.
> >
> > v2: Fixup kerneldoc to match final parameter names.
> > v3: Turn it into a kerneldoc comment
> >
We added crtc_id to the atomic ioctl, but forgot to add it for vblank
and page flip events. Commit bd386e518056 ("drm: Reorganize
drm_pending_event to support future event types [v2]") added it to
the vblank event, but page flip event was still missing.
Correct this and add a test for making sure
== Series Details ==
Series: drm/i915/pmu: Drop I915_ENGINE_SAMPLE_MAX from uapi headers (rev2)
URL : https://patchwork.freedesktop.org/series/34286/
State : success
== Summary ==
Series 34286v2 drm/i915/pmu: Drop I915_ENGINE_SAMPLE_MAX from uapi headers
https://patchwork.freedesktop.org/api/1
== Series Details ==
Series: drm/i915/pmu: Clear the previous sample value when parking (rev4)
URL : https://patchwork.freedesktop.org/series/34262/
State : success
== Summary ==
Series 34262v4 drm/i915/pmu: Clear the previous sample value when parking
https://patchwork.freedesktop.org/api/1.0
== Series Details ==
Series: drm/vblank: Pass crtc_id to page_flip_ioctl.
URL : https://patchwork.freedesktop.org/series/34292/
State : success
== Summary ==
Series 34292v1 drm/vblank: Pass crtc_id to page_flip_ioctl.
https://patchwork.freedesktop.org/api/1.0/series/34292/revisions/1/mbox/
fi
== Series Details ==
Series: drm/i915/gvt: Dma-buf support for GVT-g
URL : https://patchwork.freedesktop.org/series/34272/
State : failure
== Summary ==
commit 606231c4d58188daa3a9a02ac6b95f42c4fcf267
Author: Joonas Lahtinen
Date: Thu Nov 23 12:37:47 2017 +0200
drm-tip: 2017y-11m-23d-1
== Series Details ==
Series: drm/i915: Avoid enum conversion warning
URL : https://patchwork.freedesktop.org/series/34269/
State : failure
== Summary ==
commit 606231c4d58188daa3a9a02ac6b95f42c4fcf267
Author: Joonas Lahtinen
Date: Thu Nov 23 12:37:47 2017 +0200
drm-tip: 2017y-11m-23d-1
On Thu, Nov 23, 2017 at 08:58:47AM +, Patchwork wrote:
> == Series Details ==
>
> Series: drm/i915: Avoid enum conversion warning
> URL : https://patchwork.freedesktop.org/series/34269/
> State : failure
>
> == Summary ==
This change does not apply cleanly on top of the commit below:
> co
Since the shrinker is registered and unregistered during
i915_driver_register and i915_driver_unregister, respectively, rename
the init/cleanup functions to match.
Signed-off-by: Chris Wilson
Cc: Joonas Lahtinen
---
drivers/gpu/drm/i915/i915_drv.c | 4 ++--
drivers/gpu/drm/i915/i915_dr
Switch over from the non-descript dev_priv locals to i915.
Signed-off-by: Chris Wilson
Cc: Joonas Lahtinen
---
drivers/gpu/drm/i915/i915_drv.h | 8 +-
drivers/gpu/drm/i915/i915_gem_shrinker.c | 134 +++
2 files changed, 71 insertions(+), 71 deletions(-)
d
== Series Details ==
Series: series starting with [CI,1/6] drm/printer: Add drm_vprintf()
URL : https://patchwork.freedesktop.org/series/34278/
State : success
== Summary ==
Test kms_flip:
Subgroup plain-flip-fb-recreate-interruptible:
pass -> FAIL (shard-hs
== Series Details ==
Series: series starting with [1/5] tools/intel_watermark: Silence gcc7
snprintf() warnings
URL : https://patchwork.freedesktop.org/series/34187/
State : success
== Summary ==
IGT patchset tested on top of latest successful build
8cf978bb55c312fb585364451c5aaf9f4ef9781a te
Quoting Tvrtko Ursulin (2017-11-23 10:26:54)
> From: Tvrtko Ursulin
>
> When turning off the engines, and the pmu sampling, clear the previous
> value as the current measurement should be 0.
>
> v2: Use a for-loop
> v3:
> * Move clearing to timer self-dis-arm to avoid race with parking.
> * Cl
I'm juggling too many things, and drm-misc maintenance is one that I
keep dropping on the floor. Admit reality and remove myself as
maintainer. This still leaves us with a nice team of three who are
actually doing the drm-misc work, while I focus on drm-intel.
Cc: Daniel Vetter
Cc: Gustavo Padova
== Series Details ==
Series: series starting with [1/2] drm/i915: Rename shrinker init/cleanup to
match driver initialisation phase
URL : https://patchwork.freedesktop.org/series/34295/
State : success
== Summary ==
Series 34295v1 series starting with [1/2] drm/i915: Rename shrinker
init/cle
This was implemented correctly only on the atomic ioctl before, but
it should really be working on all 3 ioctl's involved, so ensure we
always set crtc_id correctly with a testcase.
Signed-off-by: Maarten Lankhorst
---
tests/kms_vblank.c | 60 ++---
From: Chris Wilson
Even for static CPU configurations, the hotplug CPU framework is still
used to determine the CPU topology, and is still being used by the perf
event register to check for valid CPUs.
Fixes: b46a33e271ed ("drm/i915/pmu: Expose a PMU interface for perf queries")
Signed-off-by: C
From: Tvrtko Ursulin
We do two things, both of which are purely to simplify and clarify the
implementation:
1.
Simplify the CPU online callback so it is more obvious that the purpose
there is to set a single CPU mask bit for the first CPU which comes
online. Using cpumask_weight for this reads
On 23/11/2017 10:22, Patchwork wrote:
== Series Details ==
Series: drm/i915/pmu: Clear the previous sample value when parking (rev3)
URL : https://patchwork.freedesktop.org/series/34262/
State : success
== Summary ==
Series 34262v3 drm/i915/pmu: Clear the previous sample value when parking
On 23/11/2017 10:41, Patchwork wrote:
== Series Details ==
Series: drm/i915/pmu: Drop I915_ENGINE_SAMPLE_MAX from uapi headers (rev2)
URL : https://patchwork.freedesktop.org/series/34286/
State : success
== Summary ==
Series 34286v2 drm/i915/pmu: Drop I915_ENGINE_SAMPLE_MAX from uapi header
== Series Details ==
Series: MAINTAINERS: Remove Jani as drm-misc co-maintainer
URL : https://patchwork.freedesktop.org/series/34296/
State : success
== Summary ==
Series 34296v1 MAINTAINERS: Remove Jani as drm-misc co-maintainer
https://patchwork.freedesktop.org/api/1.0/series/34296/revisions
Quoting Tvrtko Ursulin (2017-11-23 12:34:32)
> From: Tvrtko Ursulin
>
> We do two things, both of which are purely to simplify and clarify the
> implementation:
>
> 1.
>
> Simplify the CPU online callback so it is more obvious that the purpose
> there is to set a single CPU mask bit for the fir
== Series Details ==
Series: drm: Introduce RGB 64-bit 16:16:16:16 float format
URL : https://patchwork.freedesktop.org/series/34283/
State : success
== Summary ==
Test drv_selftest:
Subgroup mock_sanitycheck:
dmesg-warn -> PASS (shard-snb) fdo#103717
Quoting Matthew Auld (2017-11-22 21:19:17)
> Signed-off-by: Matthew Auld
> Cc: Joonas Lahtinen
> Cc: Chris Wilson
> Cc: Paulo Zanoni
> ---
> drivers/gpu/drm/i915/i915_drv.h| 5 ++---
> drivers/gpu/drm/i915/i915_gem_stolen.c | 19 +++
> drivers/gpu/drm/i915/intel_fbc.c
Quoting Matthew Auld (2017-11-22 21:19:19)
> Signed-off-by: Matthew Auld
> Cc: Joonas Lahtinen
> Cc: Chris Wilson
> Cc: Paulo Zanoni
> ---
> drivers/gpu/drm/i915/gvt/gvt.h| 2 +-
> drivers/gpu/drm/i915/i915_drv.c | 2 +-
> drivers/gpu/drm/i915/i915_gem.c
== Series Details ==
Series: drm/i915/pmu: Drop I915_ENGINE_SAMPLE_MAX from uapi headers (rev2)
URL : https://patchwork.freedesktop.org/series/34286/
State : success
== Summary ==
Test kms_frontbuffer_tracking:
Subgroup fbc-1p-offscren-pri-shrfb-draw-render:
pass
Quoting Matthew Auld (2017-11-22 21:19:20)
> diff --git a/include/drm/intel-gtt.h b/include/drm/intel-gtt.h
> index e901de0b6dd8..71505a5e376d 100644
> --- a/include/drm/intel-gtt.h
> +++ b/include/drm/intel-gtt.h
> @@ -5,9 +5,9 @@
> #define_DRM_INTEL_GTT_H
>
> void intel_gtt_get(u64 *g
For igt_write_huge make sure the higher gtt offsets don't feel left out,
which is especially true when dealing with the 48b PPGTT, where we
timeout long before we are able exhaust the address space.
Suggested-by: Chris Wilson
Signed-off-by: Matthew Auld
Cc: Chris Wilson
---
drivers/gpu/drm/i91
Rather than repeat the test for each engine, which takes a long time,
let's try alternating between the engines in some randomized
order.
v2: fix gen2 blunder
fix !order blunder
more cunning permutation construction!
Suggested-by: Chris Wilson
Signed-off-by: Matthew Auld
Cc: Chris Wilso
Quoting Matthew Auld (2017-11-22 21:19:18)
> Signed-off-by: Matthew Auld
> Cc: Joonas Lahtinen
> Cc: Chris Wilson
> Cc: Paulo Zanoni
> ---
> @@ -6464,9 +6463,8 @@ static bool bxt_check_bios_rc6_setup(struct
> drm_i915_private *dev_priv)
> * for this check.
> */
> rc6_
Quoting Matthew Auld (2017-11-23 13:22:57)
> Rather than repeat the test for each engine, which takes a long time,
> let's try alternating between the engines in some randomized
> order.
>
> v2: fix gen2 blunder
> fix !order blunder
> more cunning permutation construction!
>
> Suggested-b
Quoting Matthew Auld (2017-11-23 13:22:58)
> For igt_write_huge make sure the higher gtt offsets don't feel left out,
> which is especially true when dealing with the 48b PPGTT, where we
> timeout long before we are able exhaust the address space.
>
> Suggested-by: Chris Wilson
> Signed-off-by: M
On 23/11/2017 12:50, Chris Wilson wrote:
Quoting Tvrtko Ursulin (2017-11-23 12:34:32)
From: Tvrtko Ursulin
We do two things, both of which are purely to simplify and clarify the
implementation:
1.
Simplify the CPU online callback so it is more obvious that the purpose
there is to set a sing
On 23/11/2017 08:22, Chris Wilson wrote:
gem_quiescent_gpu() is supposed to ensure that the HW is idle, and in
the process kick the GPU into rc6, so we should not need a long delay
afterwards to ensure that we are indeed in rc6. We do however need a
small delay in order to be sure that rc6 cycle
On 23/11/2017 08:22, Chris Wilson wrote:
Make sure the HW is idle before we start sampling the GPU for busyness.
If we do not rest for long enough between tests, we may carry the
sampling over.
I'd rather not have this since as I said yesterday each opened PMU event
is supposed to record the
== Series Details ==
Series: series starting with [1/2] drm/i915/pmu: Remove conditional HOTPLUG_CPU
registration
URL : https://patchwork.freedesktop.org/series/34300/
State : success
== Summary ==
Series 34300v1 series starting with [1/2] drm/i915/pmu: Remove conditional
HOTPLUG_CPU registr
Chris Wilson writes:
> The legacy context switch for ringbuffer submission is multistaged,
> where each of those stages may fail. However, we were updating global
> state after some stages, and so we had to force the incomplete request
> to be submitted because we could not unwind. Save the globa
Rather than repeat the test for each engine, which takes a long time,
let's try alternating between the engines in some randomized
order.
v2: fix gen2 blunder
fix !order blunder
more cunning permutation construction!
Suggested-by: Chris Wilson
Signed-off-by: Matthew Auld
Cc: Chris Wilso
For igt_write_huge make sure the higher gtt offsets don't feel left out,
which is especially true when dealing with the 48b PPGTT, where we
timeout long before we are able exhaust the address space.
v2: just use IGT_TIMEOUT
Suggested-by: Chris Wilson
Signed-off-by: Matthew Auld
Cc: Chris Wilson
== Series Details ==
Series: drm/i915/pmu: Clear the previous sample value when parking (rev4)
URL : https://patchwork.freedesktop.org/series/34262/
State : warning
== Summary ==
Test kms_plane_lowres:
Subgroup pipe-b-tiling-none:
pass -> SKIP (shard-hsw) fd
== Series Details ==
Series: series starting with [1/2] drm/i915/selftests: rein in igt_write_huge
URL : https://patchwork.freedesktop.org/series/34302/
State : success
== Summary ==
Series 34302v1 series starting with [1/2] drm/i915/selftests: rein in
igt_write_huge
https://patchwork.freedes
== Series Details ==
Series: drm/i915: Avoid enum conversion warning
URL : https://patchwork.freedesktop.org/series/34269/
State : failure
== Summary ==
Applying: drm/i915: Avoid enum conversion warning
Using index info to reconstruct a base tree...
M drivers/gpu/drm/i915/intel_ddi.c
Fal
== Series Details ==
Series: drm/vblank: Pass crtc_id to page_flip_ioctl.
URL : https://patchwork.freedesktop.org/series/34292/
State : warning
== Summary ==
Test kms_cursor_legacy:
Subgroup basic-busy-flip-before-cursor-atomic:
pass -> SKIP (shard-hsw)
Test
== Series Details ==
Series: drm/i915/gvt: Dma-buf support for GVT-g
URL : https://patchwork.freedesktop.org/series/34272/
State : failure
== Summary ==
Applying: drm/i915/gvt: Add framebuffer decoder support
Applying: drm: Introduce RGB 64-bit 16:16:16:16 float format
Applying: drm/i915/gvt:
On Wed, Nov 22, 2017 at 12:54:49PM -0800, Kees Cook wrote:
> Hi,
>
> While doing Clang test builds, this was reported:
>
> drivers/gpu/drm/i915/intel_ddi.c:1481:30: warning: implicit conversion
> from enumeration type 'enum port' to different enumeration type 'enum
> intel_dpll_id' [-Wenum-conver
== Series Details ==
Series: series starting with [1/2] drm/i915/selftests: rein in igt_write_huge
URL : https://patchwork.freedesktop.org/series/34304/
State : success
== Summary ==
Series 34304v1 series starting with [1/2] drm/i915/selftests: rein in
igt_write_huge
https://patchwork.freedes
On Wed, Nov 22, 2017 at 07:39:01PM +0100, Maarten Lankhorst wrote:
> ips_enabled was used as a variable of whether IPS can be enabled or not,
> but should be used to test whether IPS is actually enabled.
>
> Changes since v1:
> - Call needs_modeset on new crtc state. (Ville)
> - IPS can be enabled
1 - 100 of 215 matches
Mail list logo
