Hello
I would like to have more information about the Intel_GPU_TOP tool
I have not been able to find much information, but I did found that it is part
of the intel-gpu-tools.
The system is using Skylake H i7-6700TE
I have the following questions
1) Is there a glossary of terms used by
On Thu, 2016-03-17 at 15:55 +, Chris Wilson wrote:
> On Thu, Mar 17, 2016 at 05:46:11PM +0200, Imre Deak wrote:
> > Atm, in case failure injection forces an error the subsequent
> > "*ERROR* failed to init modeset" error message will make automated
> > tests (CI) report this event as a breakage
On Tue, Mar 15, 2016 at 10:14:05AM -0500, Nathan Schulte wrote:
> Adds an (unsafe; auto-kernel-tainting) boolean module parameter to the i915
> drm driver: "enable_dp_mst", which is enabled by default. Disabling the
> parameter forces newly connected DisplayPort sinks to report as not
> supporting
Regards
Shashank
On 3/17/2016 9:31 PM, Ville Syrjälä wrote:
On Thu, Mar 17, 2016 at 09:15:39PM +0530, Sharma, Shashank wrote:
Regards
Shashank
On 3/17/2016 6:34 PM, Ville Syrjälä wrote:
On Thu, Mar 17, 2016 at 01:29:25PM +0530, Shashank Sharma wrote:
This patch restricts usage of live status
On Fri, 2016-03-18 at 10:59 +0200, Joonas Lahtinen wrote:
> On pe, 2016-03-18 at 00:18 +0200, Imre Deak wrote:
> > On Thu, 2016-03-17 at 22:14 +, Chris Wilson wrote:
> > >
> > > On Fri, Mar 18, 2016 at 12:09:30AM +0200, Imre Deak wrote:
> > > >
> > > > On Thu, 2016-03-17 at 21:48 +, Chris
This series picks up earlier work by Deepak and Yogesh, and splits it up
to smaller chunks for easier review and merging. This way we can make
progress and get some prep work merged.
BXT isn't included yet.
BR,
Jani.
Deepak M (1):
drm/i915/dsi: add support for sequence block v3 gpio for VLV
J
From: Tvrtko Ursulin
A mix of Coccinelle, sed and manual editing.
v2: Rename engine->buffer as well.
v3: Rebased.
Signed-off-by: Tvrtko Ursulin
Cc: Chris Wilson
---
drivers/gpu/drm/i915/i915_debugfs.c| 14 +-
drivers/gpu/drm/i915/i915_drv.h| 4 +-
drivers/gpu/drm/i915/
On Thu, Mar 17, 2016 at 01:29:25PM +0530, Shashank Sharma wrote:
> This patch restricts usage of live status check for HDMI detection.
> While testing certain (monitor + cable) combinations with various
> intel platforms, it seems that live status register is not reliable
> on some older devices.
Ander Conselvan De Oliveira writes:
> [ text/plain ]
> On Fri, 2016-03-04 at 14:59 +0200, Ville Syrjälä wrote:
>> On Thu, Mar 03, 2016 at 11:27:06AM +0200, Ander Conselvan De Oliveira wrote:
>> > (Cc'ing Sivakumar, as he might have some ideas on this)
>> >
>> > On Fri, 2016-02-26 at 12:54 +0200,
>> I tested on:
>> i5-4570 (Haswell) and B85 Chipset
>> Fedora 24 (Developmental Branch) x86_64
>> Cinnamon 2.8.7
>> xf86-video-intel commit 74cd4d09ae051d3a1f4cb9fe29e656b044f03ece
xf86-video-intel configure: --enable-kms-only --with-default-dri=3
_
On Thu, 2016-03-17 at 22:14 +, Chris Wilson wrote:
> On Fri, Mar 18, 2016 at 12:09:30AM +0200, Imre Deak wrote:
> > On Thu, 2016-03-17 at 21:48 +, Chris Wilson wrote:
> > > I would also like this to be the preferred
> > > DRM_ERROR reporting mechanism i.e. anytime we emit an ERROR we
> > >
On Fri, Mar 18, 2016 at 06:13:14PM +0900, Shu Yokoyama wrote:
> The short-curcuit introduced by commit 963b5bb (later modified by commit
> aacc344) breaks XShmGetImage().
> Images captured by ffmpeg's x11grab or x11vnc become choppy, while I use
> Cinnamon Desktop Environment.
The idea there (wi
Refer to Global GTT consistently as GGTT, thus rename dev_priv->gtt
to dev_priv->ggtt and struct i915_gtt to struct i915_ggtt.
Fix a couple of whitespace problems while at it.
v2:
- Fix a typo in commit message.
Reviewed-by: Chris Wilson
Signed-off-by: Joonas Lahtinen
---
drivers/gpu/drm/i915
Refer to Global GTT consistencly as GGTT, thus rename dev_priv->gtt
to dev_priv->ggtt and struct i915_gtt to struct i915_ggtt.
Fix a couple of whitespace problems while at it.
Signed-off-by: Joonas Lahtinen
---
drivers/gpu/drm/i915/i915_debugfs.c| 8 +-
drivers/gpu/drm/i915/i915_dma.c
On Wed, Mar 16, 2016 at 01:38:49PM +0200, Imre Deak wrote:
> This is v3 of [1]. It includes now the fault injection patches Chris
> requested and addresses comments from Daniel and Joonas. Since IGT has
> by now an updated drv_module_reload_basic too using load failure
> injection, CI hopefully cat
On 16/03/16 15:40, Ville Syrjälä wrote:
On Wed, Mar 16, 2016 at 03:35:13PM +, Tvrtko Ursulin wrote:
On 16/03/16 15:01, Patchwork wrote:
== Series Details ==
Series: series starting with [1/5] drm/i915: Rename local struct
intel_engine_cs variables
URL : https://patchwork.freedesktop.o
The convenience of saving a few characters by using a consistent
interface to obtain our drm_i915_private struct from intel_guc.
Signed-off-by: Chris Wilson
---
drivers/gpu/drm/i915/i915_drv.h| 4 +++-
drivers/gpu/drm/i915/i915_guc_submission.c | 23 ++-
2 files
On Thu, Mar 17, 2016 at 06:08:05PM +0200, Imre Deak wrote:
> On Thu, 2016-03-17 at 15:55 +, Chris Wilson wrote:
> > 80cols rules still apply to messages :)
> >
> > "Device initialization failed (%d). "
> > "Please file a bug at https://bugs.freedesktop.org/enter_bug.cgi "
> > "against DRI/DRM/
From: Tvrtko Ursulin
Signed-off-by: Tvrtko Ursulin
---
drivers/gpu/drm/i915/i915_gem.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/gpu/drm/i915/i915_gem.c b/drivers/gpu/drm/i915/i915_gem.c
index 31652c1da761..9c4d9c186d91 100644
--- a/drivers/gpu/drm/i915/i915_gem.c
+++ b/drive
On Wed, Mar 16, 2016 at 05:15:31PM +0200, Imre Deak wrote:
> Matt has a similar fix already on the list, Matt/Ville what's the plan
> with that? Could we have a stop-gap solution?
>
> --Imre
Yeah, I wrote two separate patches that would both be fixes for that
issue. One of them moved the PM refe
On Thu, 2016-03-17 at 19:41 +, Chris Wilson wrote:
> On Thu, Mar 17, 2016 at 06:08:05PM +0200, Imre Deak wrote:
> > On Thu, 2016-03-17 at 15:55 +, Chris Wilson wrote:
> > > 80cols rules still apply to messages :)
> > >
> > > "Device initialization failed (%d). "
> > > "Please file a bug at
On 17/03/16 11:27, Jani Nikula wrote:
On Thu, 17 Mar 2016, Tvrtko Ursulin wrote:
[ text/plain ]
On 11/03/16 15:31, Tvrtko Ursulin wrote:
From: Tvrtko Ursulin
Makes sure that the diff line adding the BUG is not immediately
preceded by the diff line removing the BUG. Or in other words,
avoid
From: Yogesh Mohan Marimuthu
Add support for CHV gpio programming in DSI gpio elements.
[Rewritten by Jani, based on earlier work by Yogesh and Deepak.]
Signed-off-by: Yogesh Mohan Marimuthu
Signed-off-by: Deepak M
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/i915/intel_dsi_panel_vbt.c |
On Wed, 16 Mar 2016, Ville Syrjälä wrote:
> [ text/plain ]
> On Tue, Mar 15, 2016 at 09:51:15PM +0200, Jani Nikula wrote:
>> Makes it neater to add the same for DSI transcoder. No functional
>> changes.
>>
>> Signed-off-by: Jani Nikula
>> ---
>> drivers/gpu/drm/i915/intel_display.c | 83
>> +++
On 3/18/2016 2:52 PM, Chris Wilson wrote:
On Fri, Mar 18, 2016 at 02:31:23PM +0530, Goel, Akash wrote:
On 3/18/2016 2:06 PM, Chris Wilson wrote:
On Fri, Mar 18, 2016 at 02:07:40PM +0530, akash.g...@intel.com wrote:
+/* emit_store_qword
+ * populate batch buffer with MI_STORE_DWORD_IMM comm
On Thu, Mar 17, 2016 at 07:56:33PM +0200, Jani Nikula wrote:
> On Thu, 17 Mar 2016, Lyude wrote:
> > Since we've fixed up drm_dp_dpcd_read() to allow for retries when things
> > timeout, there's no use for having this function anymore. Good riddens.
> >
> > Signed-off-by: Lyude
> > ---
> > drive
On Wed, 16 Mar 2016, Michel Thierry wrote:
> [ text/plain ]
> On 2/29/2016 2:35 PM, Jani Nikula wrote:
>> On Fri, 26 Feb 2016, Rodrigo Vivi wrote:
>>> Hi Jani and Daniel,
>>>
>>> I believe I forgot to cc:stable on this one and this is missing on
>>> most branches out there including Linus 4.5-r5.
On Wed, Mar 16, 2016 at 07:56:58PM +0200, Imre Deak wrote:
> On Fri, 2016-03-04 at 21:43 +0200, ville.syrj...@linux.intel.com wrote:
> > From: Ville Syrjälä
> >
> > Currently we set the initial GPU frequency to min_freq_softlimit
> > on gen9, and to efficient_freq on VLV/CHV. On all the other pla
From: Ankitprasad Sharma
Extend the drm_i915_gem_create structure to add support for
creating Stolen memory backed objects. Added a new flag through
which user can specify the preference to allocate the object from
stolen memory, which if set, an attempt will be made to allocate
the object from s
On Fri, Mar 18, 2016 at 11:10:50AM +, Tvrtko Ursulin wrote:
>
>
> On 18/03/16 10:46, Matthew Auld wrote:
> >No functional change, just makes the code easier to follow.
> >
> >v2:
> > - Remove local fence_size variable
> >(Tvrtko Ursulin)
> > - Remove redundant NULL ggtt_view check
> >
Signed-off-by: Lionel Landwerlin
---
lib/igt_kms.c | 1 +
lib/igt_kms.h | 1 +
2 files changed, 2 insertions(+)
diff --git a/lib/igt_kms.c b/lib/igt_kms.c
index f9a7bb0..515ffd0 100644
--- a/lib/igt_kms.c
+++ b/lib/igt_kms.c
@@ -1213,6 +1213,7 @@ void igt_display_init(igt_display_t *display, int
Drivers, especially i915.ko, can fail during the initial migration of a
dma-buf for CPU access. However, the error code from the driver was not
being propagated back to ioctl and so userspace was blissfully ignorant
of the failure. Rendering corruption ensues.
Whilst fixing the ioctl to return the
On Thu, 17 Mar 2016 08:57:23 +0100,
Tomi Sarvela wrote:
>
> On Wednesday 16 March 2016 16:40:24 Takashi Iwai wrote:
> > On Wed, 16 Mar 2016 16:37:06 +0100, Daniel Vetter wrote:
> > > On Wed, Mar 16, 2016 at 02:37:24PM +0200, Tomi Sarvela wrote:
> > > >
> > > > SND is recurring problem. I'd like t
On Wed, 2016-03-16 at 12:00 +, Chris Wilson wrote:
> On Wed, Mar 16, 2016 at 01:39:08PM +0200, Imre Deak wrote:
> > Add support for forcing an error at selected places in the driver.
> > As an
> > example add 4 options to fail during driver loading.
> >
> > Requested by Chris.
> >
> > v2:
> >
Atm, in case failure injection forces an error a subsequent error
messages will make automated tests (CI) report this event as a breakage,
even though the event is expected. To fix this print the error message
with debug log level in this case.
While at it print the error message for any init fail
On Tue, Mar 15, 2016 at 09:51:13PM +0200, Jani Nikula wrote:
> Prep work for DSI transcoders. No functional changes.
>
> Signed-off-by: Jani Nikula
> ---
> drivers/gpu/drm/i915/intel_display.c | 27 +++
> 1 file changed, 23 insertions(+), 4 deletions(-)
>
> diff --git a/
No functional change, just makes the code easier to follow.
Signed-off-by: Matthew Auld
---
drivers/gpu/drm/i915/i915_gem.c | 41 +++--
1 file changed, 11 insertions(+), 30 deletions(-)
diff --git a/drivers/gpu/drm/i915/i915_gem.c b/drivers/gpu/drm/i915/i915_
On Mon, 2016-03-14 at 09:27 +0100, Maarten Lankhorst wrote:
> Signed-off-by: Maarten Lankhorst
Reviewed-by: Ander Conselvan de Oliveira
> ---
> drivers/gpu/drm/i915/intel_display.c | 6 +++---
> drivers/gpu/drm/i915/intel_dpll_mgr.c | 4
> 2 files changed, 3 insertions(+), 7 deletions(-)
On Thu, 17 Mar 2016, Tvrtko Ursulin wrote:
> [ text/plain ]
>
>
> On 11/03/16 15:31, Tvrtko Ursulin wrote:
>> From: Tvrtko Ursulin
>>
>> Makes sure that the diff line adding the BUG is not immediately
>> preceded by the diff line removing the BUG. Or in other words,
>> avoids false positives when
From: Ankitprasad Sharma
This patch adds support for extending the pread/pwrite functionality
for objects not backed by shmem. The access will be made through
gtt interface. This will cover objects backed by stolen memory as well
as other non-shmem backed objects.
v2: Drop locks around slow_user
On Fri, 18 Mar 2016 09:12:53 +0100,
Tomi Sarvela wrote:
>
> On Friday 18 March 2016 09:00:13 Takashi Iwai wrote:
> > On Fri, 18 Mar 2016 08:36:27 +0100,
> >
> > Do you have the commit of the last working kernel?
> >
> > > The unit tests that hang quite reliably on first run are
> > > igt@gem_rin
BXT isn't as limited as BYT and CHT regarding DSI pipes and ports.
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/i915/intel_dsi.c | 9 +++--
1 file changed, 7 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/i915/intel_dsi.c b/drivers/gpu/drm/i915/intel_dsi.c
index dd6f7bc4f444..
On Thu, 2016-03-17 at 21:48 +, Chris Wilson wrote:
> On Thu, Mar 17, 2016 at 11:10:42PM +0200, Imre Deak wrote:
> > Well, I can follow-up on an export patch for that, but getting that
> > merged may take a while, so could we go with the current version
> > until
> > that?
>
> What's the curren
Matt has a similar fix already on the list, Matt/Ville what's the plan
with that? Could we have a stop-gap solution?
--Imre
On Wed, 2016-03-16 at 17:11 +0200, Gabriel Feceoru wrote:
> On HSW/BDW there are random dmesg-warn errors while running kms
> tests, like:
>
> [90587.867077] Call Trace:
>
On Thu, 17 Mar 2016, Shashank Sharma wrote:
> [ text/plain ]
> This patch restricts usage of live status check for HDMI detection.
> While testing certain (monitor + cable) combinations with various
> intel platforms, it seems that live status register is not reliable
> on some older devices. So
On Fri, Mar 18, 2016 at 08:51:29AM +0530, Sharma, Shashank wrote:
> Regards
> Shashank
>
> On 3/17/2016 10:17 PM, Ville Syrjälä wrote:
> > On Thu, Mar 17, 2016 at 10:03:04PM +0530, Sharma, Shashank wrote:
> >> Hey Chris,
> >> I added comments for both Ville and you, please help me to understand th
No functional change, just makes the code easier to follow.
v2:
- Remove local fence_size variable
(Tvrtko Ursulin)
- Remove redundant NULL ggtt_view check
- Reuse size variable
v3:
(Chris Wilson)
- Keep start, end computation separate
- Simplify alignment
Signed-off-by: Matth
On Thu, Mar 17, 2016 at 05:10:36PM +, Tvrtko Ursulin wrote:
> diff --git a/drivers/gpu/drm/i915/intel_ringbuffer.h
> b/drivers/gpu/drm/i915/intel_ringbuffer.h
> index 3e40f7bf2147..ce7337b0aa14 100644
> --- a/drivers/gpu/drm/i915/intel_ringbuffer.h
> +++ b/drivers/gpu/drm/i915/intel_ringbuffer
drivers/gpu/drm/i915/intel_dpll_mgr.c:1200:32: warning: Using plain integer as
NULL pointer
Fixes: 304b65cbdc8d ("drm/i915: Move SKL/KLB pll selection logic to
intel_dpll_mgr.c")
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/i915/intel_dpll_mgr.c | 2 +-
1 file changed, 1 insertion(+), 1 dele
On Thu, 17 Mar 2016, "Sharma, Shashank" wrote:
> +intel-gfx
Please repost the patch to intel-gfx, just adding cc: makes the patch
painful to review and apply, and CI won't pick it up.
BR,
Jani.
>
> Regards
> Shashank
> -Original Message-
> From: Sharma, Shashank
> Sent: Thursday, March
On Thu, Mar 17, 2016 at 12:59:46PM +, Tvrtko Ursulin wrote:
> From: Tvrtko Ursulin
>
> By reading the CSB (slow MMIO accesses) into a temporary local
> buffer we can decrease the duration of holding the execlist
> lock.
>
> Main advantage is that during heavy batch buffer submission we
> red
On Fri, Mar 18, 2016 at 08:07:54PM +, Arun Siluvery wrote:
> From: Tomas Elf
>
> Initialize hangcheck struct during driver load. Since we do the same after
> recovering from a reset, this is extracted into a helper function.
>
> Cc: Mika Kuoppala
> Signed-off-by: Tomas Elf
> Signed-off-by:
In full gpu reset we prime all engines and reset domains corresponding to
each engine. Per engine reset is just a special case of this process
wherein only a single engine is reset. This change is aimed to modify
relevant functions to achieve this. There are some other steps we carry out
in case of
On 17/03/16 09:37, Chris Wilson wrote:
Although the long term future of get_user_pages_locked() itself is
doubtful, the kernel currently recommends:
/* get_user_pages should be phased out in favor of
* get_user_pages_locked|unlocked or get_user_pages_fast. Nothing
* should use get_user_page
On Wed, Mar 16, 2016 at 06:06:58PM +0200, Jani Nikula wrote:
> drivers/gpu/drm/i915/intel_dpll_mgr.c:1200:32: warning: Using plain integer
> as NULL pointer
>
> Fixes: 304b65cbdc8d ("drm/i915: Move SKL/KLB pll selection logic to
> intel_dpll_mgr.c")
> Signed-off-by: Jani Nikula
Reviewed-by: Vi
On 3/11/2016 11:48 AM, akash.g...@intel.com wrote:
From: Akash Goel
This patch provides the testcase to exercise the TRTT hardware.
Some platforms have an additional address translation hardware support in
form of Tiled Resource Translation Table (TR-TT) which provides an extra level
of abstra
Mika Kuoppala writes:
> [ text/plain ]
> In full gpu reset we prime all engines and reset domains corresponding to
> each engine. Per engine reset is just a special case of this process
> wherein only a single engine is reset. This change is aimed to modify
> relevant functions to achieve this. T
On Wed, Mar 16, 2016 at 05:26:24PM +0200, Jani Nikula wrote:
> On Wed, 16 Mar 2016, Ville Syrjälä wrote:
> > [ text/plain ]
> > On Tue, Mar 15, 2016 at 09:51:15PM +0200, Jani Nikula wrote:
> >> Makes it neater to add the same for DSI transcoder. No functional
> >> changes.
> >>
> >> Signed-off-by
On Fri, Mar 18, 2016 at 10:56:55AM +, Tvrtko Ursulin wrote:
>
> On 18/03/16 08:42, Joonas Lahtinen wrote:
> >From: Chris Wilson
> >
> >Throughout the code base, we use u32 for offsets into the global GTT. If
> >we ever see any hardware with a larger GGTT, then we run the real risk
> >of silen
On Wed, Mar 16, 2016 at 03:44:37PM -0400, Lyude Paul wrote:
> On Wed, 2016-03-16 at 21:39 +0200, Ville Syrjälä wrote:
> > On Wed, Mar 16, 2016 at 03:18:04PM -0400, Lyude wrote:
> > >
> > > After unplugging a DP MST display from the system, we have to go through
> > > and destroy all of the DRM con
On Wed, Mar 16, 2016 at 03:40:52PM +, Tvrtko Ursulin wrote:
> From: Tvrtko Ursulin
>
> Signed-off-by: Tvrtko Ursulin
This function is horrible, this is slightly better but not much.
Reviewed-by: Chris Wilson
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
___
On Fri, Mar 18, 2016 at 03:08:56PM -0300, Tiago Vignatti wrote:
> This patch adds ioctl-errors subtest to be used for exercising prime sync
> ioctl
> errors.
>
> The subtest constantly interrupts via signals a function doing concurrent blit
> to stress out the right usage of prime_sync_*, making
Signed-off-by: Lionel Landwerlin
---
lib/igt_debugfs.c | 17 +
lib/igt_debugfs.h | 1 +
2 files changed, 18 insertions(+)
diff --git a/lib/igt_debugfs.c b/lib/igt_debugfs.c
index c291ef3..a32ed78 100644
--- a/lib/igt_debugfs.c
+++ b/lib/igt_debugfs.c
@@ -252,6 +252,23 @@ bool ig
From: Ankitprasad Sharma
Propagating correct error codes to userspace by using ERR_PTR and
PTR_ERR macros for stolen memory based object allocation. We generally
return -ENOMEM to the user whenever there is a failure in object
allocation. This patch helps user to identify the correct reason for t
== Series Details ==
Series: drm/i915/gen9: add WaClearFlowControlGpgpuContextSave (rev3)
URL : https://patchwork.freedesktop.org/series/4272/
State : failure
== Summary ==
Series 4272v3 drm/i915/gen9: add WaClearFlowControlGpgpuContextSave
http://patchwork.freedesktop.org/api/1.0/series/4272/
On Fri, Mar 18, 2016 at 05:05:41PM +0200, Jani Nikula wrote:
> Makes it neater to add the same for DSI transcoder. No functional
> changes.
>
> v2: rename to hsw_get_transcoder_state and add a comment about grabbing
> power reference (Ville)
>
> Signed-off-by: Jani Nikula
Reviewed-by: Ville
On Wed, Mar 16, 2016 at 07:17:51PM +0200, Imre Deak wrote:
> On Fri, 2016-03-04 at 21:43 +0200, ville.syrj...@linux.intel.com wrote:
> > From: Ville Syrjälä
> >
> > Extract the GPLL reference frequency from CCK and use it in the
> > GPU freq<->opcode conversions on VLV/CHV. This eliminates all th
From: Chris Wilson
This utility function is a companion to i915_gem_object_get_page() that
uses the same cached iterator for the scatterlist to perform fast
sequential lookup of the dma address associated with any page within the
object.
Signed-off-by: Chris Wilson
Signed-off-by: Ankitprasad Sh
On Thu, 2016-03-17 at 14:23 +0200, Imre Deak wrote:
> On Thu, 2016-03-17 at 11:32 +, Patchwork wrote:
> > == Series Details ==
> >
> > Series: Split driver init step to phases (rev3)
> > URL : https://patchwork.freedesktop.org/series/4509/
> > State : warning
> >
> > == Summary ==
> >
> >
From: Ankitprasad Sharma
This patch adds support for clearing buffer objects via CPU/GTT. This
is particularly useful for clearing out the non shmem backed objects.
Currently intend to use this only for buffers allocated from stolen
region.
v2: Added kernel doc for i915_gem_clear_object(), corre
On Wed, Mar 16, 2016 at 12:43:32PM +0200, Jani Nikula wrote:
> Hide knowledge about VBT child devices in intel_bios.c.
>
> v2: Move port check to intel_bios.c (Sivakumar)
>
> Signed-off-by: Jani Nikula
> ---
> drivers/gpu/drm/i915/i915_drv.h | 2 +-
> drivers/gpu/drm/i915/intel_bios.c | 39
On 03/18/2016 03:11 PM, Daniel Vetter wrote:
On Fri, Mar 18, 2016 at 03:08:56PM -0300, Tiago Vignatti wrote:
This patch adds ioctl-errors subtest to be used for exercising prime sync ioctl
errors.
The subtest constantly interrupts via signals a function doing concurrent blit
to stress out the r
== Series Details ==
Series: drm/i915: Modify reset func to handle per engine resets (rev3)
URL : https://patchwork.freedesktop.org/series/4510/
State : failure
== Summary ==
Series 4510v3 drm/i915: Modify reset func to handle per engine resets
http://patchwork.freedesktop.org/api/1.0/series/4
On HSW/BDW there are random dmesg-warn errors while running kms tests, like:
[90587.867077] Call Trace:
[90587.867085] [] dump_stack+0x67/0x99
[90587.867094] [] warn_slowpath_common+0x81/0xc0
[90587.867099] [] warn_slowpath_fmt+0x47/0x50
[90587.867150] [] hsw_write32+0x280/0x290 [i915]
[90587.
From: Chris Wilson
If we run out of stolen memory when trying to allocate an object, see if
we can reap enough purgeable objects to free up enough contiguous free
space for the allocation. This is in principle very much like evicting
objects to free up enough contiguous space in the vma when bind
On 17/03/16 13:14, Chris Wilson wrote:
On Thu, Mar 17, 2016 at 12:59:46PM +, Tvrtko Ursulin wrote:
From: Tvrtko Ursulin
By reading the CSB (slow MMIO accesses) into a temporary local
buffer we can decrease the duration of holding the execlist
lock.
Main advantage is that during heavy bat
On Tue, Mar 15, 2016 at 09:51:11PM +0200, Jani Nikula wrote:
> Make the code easier to read and update. No functional changes.
>
> Signed-off-by: Jani Nikula
The logic looks intact
Reviewed-by: Ville Syrjälä
> ---
> drivers/gpu/drm/i915/intel_dsi.c | 37 +++--
>
From: Yogesh Mohan Marimuthu
Add support for CHV gpio programming in DSI gpio elements.
XXX: I'd like to have a gpio table for chv as well as others.
[Rewritten by Jani, based on earlier work by Yogesh and Deepak.]
Signed-off-by: Yogesh Mohan Marimuthu
Signed-off-by: Deepak M
Signed-off-by:
Hi,
If we don't do any of the fence business for !i915_is_gtt, then will
this not change the following code:
if (alignment == 0)
alignment = flags & PIN_MAPPABLE ? fence_alignment : unfenced_alignment;
Or am I missing something?
Regards,
Matt
___
v2: Rename CTM_MATRIX property to CTM
v3: Add support for atomic commits
Signed-off-by: Lionel Landwerlin
---
lib/igt_kms.c | 87 +--
lib/igt_kms.h | 20 +-
2 files changed, 104 insertions(+), 3 deletions(-)
diff --git a/lib/i
On Wed, 16 Mar 2016 16:37:06 +0100,
Daniel Vetter wrote:
>
> On Wed, Mar 16, 2016 at 02:37:24PM +0200, Tomi Sarvela wrote:
> > On Wednesday 16 March 2016 10:48:43 Imre Deak wrote:
> > > Tomi, noticed two things that maybe infrastructure related, see below:
> > >
> > > > Test drv_module_reload_bas
Use less pointers with the probing code, making it much less confusing
to read.
Cc: Mika Kuoppala
Reviewed-by: Chris Wilson
Signed-off-by: Joonas Lahtinen
---
drivers/gpu/drm/i915/i915_gem_gtt.c | 100
drivers/gpu/drm/i915/i915_gem_gtt.h | 5 +-
2 files c
From: Ankitprasad Sharma
Skip gem_stolen and pread/pwrite on stolen backed objects if Intel
RST device is present.
Signed-off-by: Ankitprasad Sharma
---
lib/ioctl_wrappers.c | 14 +-
tests/gem_pread.c| 3 +++
tests/gem_pwrite.c | 2 ++
3 files changed, 18 insertions(+), 1 d
On Thu, Mar 17, 2016 at 12:23:10PM +0200, Mika Kahola wrote:
> It was noticed on bug #94087 that module parameter
> i915.edp_vswing=2 that should override the VBT setting
> to use default voltage swing (400 mV) was not applied
> for Broadwell.
>
> This patch provides a fix for this by checking if
Useful for creating common setup code in igt_fixture which is only
needed by a subset of tests. And since I'm a good citizen it comes
with a library testcase/example included.
v2: Make testcase nastier to ensure a subtest group SKIPS when it's
parent is skipping already. I accidentally got this ri
On Fri, 2016-03-04 at 21:43 +0200, ville.syrj...@linux.intel.com wrote:
> From: Ville Syrjälä
>
> Currently we set the initial GPU frequency to min_freq_softlimit
> on gen9, and to efficient_freq on VLV/CHV. On all the other platforms
> we set it to idle_freq. Let's use idle_freq across the board
Move the opregion unregistration earlier to match its corresponding
registration order.
Signed-off-by: Imre Deak
---
drivers/gpu/drm/i915/i915_dma.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/i915/i915_dma.c b/drivers/gpu/drm/i915/i915_dma.c
index 7ab8a
Looks tidy again, and doesn't break igt assumptions.
Signed-off-by: Daniel Vetter
---
tests/gem_concurrent_all.c | 67 +-
1 file changed, 30 insertions(+), 37 deletions(-)
diff --git a/tests/gem_concurrent_all.c b/tests/gem_concurrent_all.c
index 67c9
On Fri, 18 Mar 2016 08:36:27 +0100,
Tomi Sarvela wrote:
>
> On Thursday 17 March 2016 18:00:52 Takashi Iwai wrote:
> ...
>
> > > CI_DRM_1147 git://anongit.freedesktop.org/drm-intel
> > > dbbc6d276864d7b7a3a1edb04f0511153f9c3852
> > >
> > > Note that drm-intel-nightly history changes, so the exac
On Wed, Mar 16, 2016 at 4:33 PM Clint Taylor
wrote:
> On 03/16/2016 12:27 AM, Daniel Vetter wrote:
> > On Tue, Mar 15, 2016 at 02:34:05PM -0700, clinton.a.tay...@intel.com
> wrote:
> >> From: Clint Taylor
> >>
> >> WARNING: Using ChromeOS with an eDP panel and a 4K@60 DP monitor
> connected
> >>
This is v3 of [1]. It includes now the fault injection patches Chris
requested and addresses comments from Daniel and Joonas. Since IGT has
by now an updated drv_module_reload_basic too using load failure
injection, CI hopefully catches any problems introduced by this
patchset.
CC: Chris Wilson
C
... meanwhile ...
$ modinfo hdmi_audio
filename:
/lib/modules/4.5.0-301.fc22.x86_64/kernel/sound/hdmi_audio/hdmi_audio.ko.xz
version:0.01.003
license:GPL v2
description:Intel HDMI Audio driver
author: Vaibhav Agarwal
author: Ramesh Babu K V
author:
On Fri, Mar 18, 2016 at 08:02:39PM +, Chris Wilson wrote:
> Drivers, especially i915.ko, can fail during the initial migration of a
> dma-buf for CPU access. However, the error code from the driver was not
> being propagated back to ioctl and so userspace was blissfully ignorant
> of the failur
This lets you add e.g. review tags to a commit on the command line:
$ dim commit-add-tag "Reviewed-by: Ville Syrjälä
"
This could be used as a building block for further scripting.
Signed-off-by: Jani Nikula
---
dim | 19 ++-
1 file changed, 10 insertions(+), 9 deletions(-)
d
From: Tvrtko Ursulin
By reading the CSB (slow MMIO accesses) into a temporary local
buffer we can decrease the duration of holding the execlist
lock.
Main advantage is that during heavy batch buffer submission we
reduce the execlist lock contention, which should decrease the
latency and CPU usag
On Mon, 2016-03-14 at 09:27 +0100, Maarten Lankhorst wrote:
> This makes it easier to verify correct dpll setup with only a single crtc.
> It is also useful to detect double dpll enable/disable.
>
> Changes since v1:
> - Rebase on top of Ander's dpll rework.
> - Change debugfs active to a mask.
>
On Wed, 2016-03-16 at 12:44 +, Chris Wilson wrote:
> On Wed, Mar 16, 2016 at 02:12:35PM +0200, Imre Deak wrote:
> > On Wed, 2016-03-16 at 12:00 +, Chris Wilson wrote:
> > > On Wed, Mar 16, 2016 at 01:39:08PM +0200, Imre Deak wrote:
> > > > Add support for forcing an error at selected places
This init step accesses the device, but doesn't have any device
specific side effect. It also sets up some platform specific
attributes that may be required early, so move it earlier.
Signed-off-by: Imre Deak
---
drivers/gpu/drm/i915/i915_dma.c | 4 ++--
1 file changed, 2 insertions(+), 2 deleti
On 18/03/16 08:42, Joonas Lahtinen wrote:
Reference variable value from pointer, not assumed pointer destination.
Since:
commit c44ef60e437019b8ca1dab8b4d2e8761fd4ce1e9
Author: Mika Kuoppala
Date: Thu Jun 25 18:35:05 2015 +0300
drm/i915/gtt: Allow >= 4GB sizes for vm.
Cc: Mika Kuoppa
On Thu, Mar 17, 2016 at 01:34:59PM +, Chris Wilson wrote:
> vmaps are temporary kernel mappings that may be of long duration.
> Reusing a vmap on an object is preferrable for a driver as the cost of
> setting up the vmap can otherwise dominate the operation on the object.
> However, the vmap ad
1 - 100 of 324 matches
Mail list logo
