On Wed, Jun 11, 2014 at 10:46:48AM +0530, Vandana Kannan wrote:
Added a property to enable user space to set aspect ratio.
This patch contains declaration of the property and code to create the
property.
v2: Thierry's review comments.
- Made aspect ratio enum generic instead of
On Sun, Jul 13, 2014 at 10:32:41PM +0200, Paul Bolle wrote:
Paul Bolle schreef op wo 02-07-2014 om 10:53 [+0200]:
On Tue, 2014-07-01 at 12:17 +0300, Jani Nikula wrote:
This does not ring any bells to me (but that doesn't prove anything). A
bisect result would be awesome.
The bisect
On Mon, Jul 14, 2014 at 04:22:44AM +, Sun, Daisy wrote:
3) The function will be called when flip happened, this should cover
most of the cases. One exception is background media process without any
display output, it's relatively rare. Please let me know if you have
concern on other
On Mon, Jul 14, 2014 at 11:04:39AM +1000, Dave Airlie wrote:
From: Dave Airlie airl...@redhat.com
This reverts commit 38aecea0ccbb909d635619cba22f1891e589b434.
This breaks Haswell Thinkpad + Lenovo dock in SST mode with a HDMI monitor
attached.
Before this we can 1920x1200 mode, after
On Mon, Jul 14, 2014 at 8:59 AM, Daniel Vetter dan...@ffwll.ch wrote:
On Mon, Jul 14, 2014 at 04:22:44AM +, Sun, Daisy wrote:
3) The function will be called when flip happened, this should cover
most of the cases. One exception is background media process without any
display output, it's
On Mon, 2014-07-14 at 08:56 +0200, Daniel Vetter wrote:
Please test the below patch, thanks.
-Daniel
diff --git a/drivers/gpu/drm/i915/intel_display.c
b/drivers/gpu/drm/i915/intel_display.c
index 82e7d57f0a8a..f0be855ddf45 100644
--- a/drivers/gpu/drm/i915/intel_display.c
+++
On Mon, Jul 14, 2014 at 09:13:40AM +0200, Paul Bolle wrote:
On Mon, 2014-07-14 at 08:56 +0200, Daniel Vetter wrote:
Please test the below patch, thanks.
-Daniel
diff --git a/drivers/gpu/drm/i915/intel_display.c
b/drivers/gpu/drm/i915/intel_display.c
index 82e7d57f0a8a..f0be855ddf45
On Saturday 28 June 2014 11:26 AM, deepa...@linux.intel.com wrote:
From: Deepak S deepa...@linux.intel.com
Drop WaGsvBringDownFreq on CHV.
When in RC6 requesting the min freq should be fine to bring the
voltage down.
Signed-off-by: Deepak S deepa...@linux.intel.com
---
On Friday 20 June 2014 08:03 PM, deepa...@linux.intel.com wrote:
From: Deepak S deepa...@linux.intel.com
We might be leaving the GPU Frequency (and thus vnn) high during the suspend.
Force gt to move to lowest freq while suspending.
v2: Fixed typo in commit message (Deepak)
v3: Force gt to
On Fri, Jul 11, 2014 at 3:45 PM, Thomas Gleixner t...@linutronix.de wrote:
No point in converting timespecs back and forth.
Signed-off-by: Thomas Gleixner t...@linutronix.de
Cc: Daniel Vetter daniel.vet...@ffwll.ch
I guess simplest to merge together with the new ktime_get_raw_ns.
Acked-by: me
On Sun, 2014-07-06 at 18:29 +0530, sourab gupta wrote:
On Fri, 2014-06-20 at 10:02 +, Gupta, Sourab wrote:
From: Sourab Gupta sourab.gu...@intel.com
This patch series introduces a new gem create ioctl for user specified
placement.
Despite being a unified memory architecture
From: Tvrtko Ursulin tvrtko.ursu...@intel.com
Userptr v23 was not thread safe against memory map operations and object
creation from separate threads. MMU notifier callback would get triggered
on a partially constructed object causing a NULL pointer dereference.
This test excercises that path a
On Mon, Jun 30, 2014 at 09:53:41AM -0700, Rodrigo Vivi wrote:
+ for_each_ring(useless, dev_priv, i) {
+ u16 signal_offset =
+ (GEN8_SIGNAL_OFFSET(ring, i) PAGE_MASK) / 4;
+ u32 *tmp = error-semaphore_obj-pages[0];
+
+
-Original Message-
From: Intel-gfx [mailto:intel-gfx-boun...@lists.freedesktop.org] On Behalf
Of Damien Lespiau
Sent: Friday, July 11, 2014 6:34 PM
To: intel-gfx@lists.freedesktop.org
Subject: [Intel-gfx] [PATCH 1/2] drm/i915: Provide a config option to select a
target platform
On Mon, Jul 14, 2014 at 11:03:26AM +0100, Tvrtko Ursulin wrote:
From: Tvrtko Ursulin tvrtko.ursu...@intel.com
Userptr v23 was not thread safe against memory map operations and object
creation from separate threads. MMU notifier callback would get triggered
on a partially constructed object
On Mon, Jul 14, 2014 at 11:18:30AM +0100, Barbalho, Rafael wrote:
-Original Message-
From: Intel-gfx [mailto:intel-gfx-boun...@lists.freedesktop.org] On Behalf
Of Damien Lespiau
Sent: Friday, July 11, 2014 6:34 PM
To: intel-gfx@lists.freedesktop.org
Subject: [Intel-gfx]
On 07/14/2014 11:34 AM, Chris Wilson wrote:
On Mon, Jul 14, 2014 at 11:03:26AM +0100, Tvrtko Ursulin wrote:
From: Tvrtko Ursulin tvrtko.ursu...@intel.com
Userptr v23 was not thread safe against memory map operations and object
creation from separate threads. MMU notifier callback would get
There's a pile of ideas around to add generally useful options like
--debug to all igt tests. Or unify the runtime behaviour between
simple and subtest tests a bit more.
The first step to get there is to add argc/argv to igt_simple_init so
that we can get at the argument list.
Cc: Tim Gore
You don't have any cancellation points in the loop. (mmap may or may not
be, it is not required to be.)
But rather than use a global, just pass a pointer to a local struct.
It doesn't need both a cancellation point and a flag. Should I just
add pthread_testcancel in the loop and not have
On 07/14/2014 02:07 PM, Chris Wilson wrote:
You don't have any cancellation points in the loop. (mmap may or may not
be, it is not required to be.)
But rather than use a global, just pass a pointer to a local struct.
It doesn't need both a cancellation point and a flag. Should I just
add
From: Tvrtko Ursulin tvrtko.ursu...@intel.com
Userptr v23 was not thread safe against memory map operations and object
creation from separate threads. MMU notifier callback would get triggered
on a partially constructed object causing a NULL pointer dereference.
This test excercises that path a
On Mon, Jul 14, 2014 at 02:13:22PM +0100, Tvrtko Ursulin wrote:
On 07/14/2014 02:07 PM, Chris Wilson wrote:
You don't have any cancellation points in the loop. (mmap may or may not
be, it is not required to be.)
But rather than use a global, just pass a pointer to a local struct.
It
On 7/12/2014 5:28 PM, Daniel Vetter wrote:
On Sat, Jul 12, 2014 at 1:47 PM, Shobhit Kumar shobhit.ku...@intel.com wrote:
Call to vlv_crtc_clock_get is not needed for DSI and was causing dpio
read WARN dumps as well. Absence of -get_config was casuing othet WARN
dumps as well. With this the last
2014-07-07 18:53 GMT-03:00 Jesse Barnes jbar...@virtuousgeek.org:
On Mon, 7 Jul 2014 18:48:47 -0300
Paulo Zanoni przan...@gmail.com wrote:
(documenting what we discussed on IRC)
2014-06-20 13:29 GMT-03:00 Jesse Barnes jbar...@virtuousgeek.org:
This was always the case on our suspend path,
2014-06-20 13:29 GMT-03:00 Jesse Barnes jbar...@virtuousgeek.org:
Now that we use the runtime IRQ enable/disable functions in our suspend
path, we can simply check the pm._irqs_disabled flag everywhere. So
rename it to catch the users, and add an inline for it to make the
checks clear
2014-06-20 13:29 GMT-03:00 Jesse Barnes jbar...@virtuousgeek.org:
We're taking the right power well refs now, so this shouldn't be needed.
Reported-by: Imre Deak imre.d...@intel.com
Signed-off-by: Jesse Barnes jbar...@virtuousgeek.org
I submitted the exact same patch back in April, and got a
On Tue, Dec 17, 2013 at 09:33:27AM +0200, Jani Nikula wrote:
On Mon, 16 Dec 2013, Chris Wilson ch...@chris-wilson.co.uk wrote:
Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
Reviewed-by: Jani Nikula jani.nik...@intel.com
Queued for -next, thanks for the patch.
-Daniel
---
On Tue, 15 Jul 2014 13:03:55 +0530
Deepak S deepa...@linux.intel.com wrote:
On Saturday 28 June 2014 11:26 AM, deepa...@linux.intel.com wrote:
From: Deepak S deepa...@linux.intel.com
Drop WaGsvBringDownFreq on CHV.
When in RC6 requesting the min freq should be fine to bring the
On Tue, 15 Jul 2014 13:05:48 +0530
Deepak S deepa...@linux.intel.com wrote:
On Friday 20 June 2014 08:03 PM, deepa...@linux.intel.com wrote:
From: Deepak S deepa...@linux.intel.com
We might be leaving the GPU Frequency (and thus vnn) high during the
suspend.
Force gt to move to
2014-07-14 12:06 GMT-03:00 Paulo Zanoni przan...@gmail.com:
2014-06-20 13:29 GMT-03:00 Jesse Barnes jbar...@virtuousgeek.org:
Now that we use the runtime IRQ enable/disable functions in our suspend
path, we can simply check the pm._irqs_disabled flag everywhere. So
rename it to catch the
2014-06-20 13:29 GMT-03:00 Jesse Barnes jbar...@virtuousgeek.org:
Before we've installed the handler, we can set this and avoid confusing
init code that then thinks IRQs are enabled and spews complaints
everywhere.
But then at some point the DRM layer will call our IRQ init callbacks,
which
On Mon, Jul 14, 2014 at 4:36 PM, Kumar, Shobhit shobhit.ku...@intel.com wrote:
/* XXX: read flags, set to adjusted_mode */
+ pipe_config-quirks = 1;
Nack. First you need to use one of the symbolic quirk definitions
(there's a bunch of them). Second this needs a comment why
On Mon, 14 Jul 2014 12:19:54 -0300
Paulo Zanoni przan...@gmail.com wrote:
2014-07-14 12:06 GMT-03:00 Paulo Zanoni przan...@gmail.com:
2014-06-20 13:29 GMT-03:00 Jesse Barnes jbar...@virtuousgeek.org:
Now that we use the runtime IRQ enable/disable functions in our suspend
path, we can
On Thu, Apr 24, 2014 at 05:05:21PM +0300, Ville Syrjälä wrote:
On Thu, Apr 24, 2014 at 10:50:56AM -0300, Paulo Zanoni wrote:
From: Paulo Zanoni paulo.r.zan...@intel.com
We already call intel_dp_power_get, which will get a power domain, and
^^
Commit
Daniel Vetter schreef op ma 14-07-2014 om 09:18 [+0200]:
On Mon, Jul 14, 2014 at 09:13:40AM +0200, Paul Bolle wrote:
On Mon, 2014-07-14 at 08:56 +0200, Daniel Vetter wrote:
diff --git a/drivers/gpu/drm/i915/intel_display.c
b/drivers/gpu/drm/i915/intel_display.c
index
Can we find a way to fix those for good? My preference goes to not
having that single/multi split but always have a test being a list of
one or more subtests.
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
tests/Makefile.sources | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
This patch got a conflict on latest -collector. Is it still needed?
If yes, could you please rebase?
I was also going to tell about the Make the pysical object coherent...
but I noticed you already rebased.
Thanks,
Rodrigo.
On Wed, May 14, 2014 at 1:46 AM, Chris Wilson
On Mon, Jul 14, 2014 at 12:23:11PM -0300, Paulo Zanoni wrote:
2014-06-20 13:29 GMT-03:00 Jesse Barnes jbar...@virtuousgeek.org:
Before we've installed the handler, we can set this and avoid confusing
init code that then thinks IRQs are enabled and spews complaints
everywhere.
But then at
On Mon, Jul 14, 2014 at 08:17:15AM -0700, Jesse Barnes wrote:
On Tue, 15 Jul 2014 13:03:55 +0530
Deepak S deepa...@linux.intel.com wrote:
On Saturday 28 June 2014 11:26 AM, deepa...@linux.intel.com wrote:
From: Deepak S deepa...@linux.intel.com
Drop WaGsvBringDownFreq on CHV.
On Mon, Jul 14, 2014 at 11:56:57AM -0300, Paulo Zanoni wrote:
2014-07-07 18:53 GMT-03:00 Jesse Barnes jbar...@virtuousgeek.org:
On Mon, 7 Jul 2014 18:48:47 -0300
Paulo Zanoni przan...@gmail.com wrote:
(documenting what we discussed on IRC)
2014-06-20 13:29 GMT-03:00 Jesse Barnes
commit 98ec77397a5c68ce753dc283aaa6f4742328bcdd
Author: Ville Syrjälä ville.syrj...@linux.intel.com
Date: Wed Apr 30 17:43:01 2014 +0300
drm/i915: Make primary_enabled match the actual hardware state
introduced more accurate tracking of the primary plane and some
checks. It missed the
On Mon, Jul 14, 2014 at 06:10:29PM +0100, Damien Lespiau wrote:
Can we find a way to fix those for good? My preference goes to not
having that single/multi split but always have a test being a list of
one or more subtests.
We're working on this ... We = Thomasme and Tim Gore a bit.
-Daniel
2014-07-14 14:26 GMT-03:00 Daniel Vetter dan...@ffwll.ch:
On Mon, Jul 14, 2014 at 12:23:11PM -0300, Paulo Zanoni wrote:
2014-06-20 13:29 GMT-03:00 Jesse Barnes jbar...@virtuousgeek.org:
Before we've installed the handler, we can set this and avoid confusing
init code that then thinks IRQs
On Mon, 2014-07-14 at 19:39 +0200, Daniel Vetter wrote:
commit 98ec77397a5c68ce753dc283aaa6f4742328bcdd
Author: Ville Syrjälä ville.syrj...@linux.intel.com
Date: Wed Apr 30 17:43:01 2014 +0300
drm/i915: Make primary_enabled match the actual hardware state
introduced more accurate
On Mon, Jul 14, 2014 at 11:04:22AM +0100, Damien Lespiau wrote:
On Mon, Jun 30, 2014 at 09:53:41AM -0700, Rodrigo Vivi wrote:
+ for_each_ring(useless, dev_priv, i) {
+ u16 signal_offset =
+ (GEN8_SIGNAL_OFFSET(ring, i) PAGE_MASK) / 4;
+ u32 *tmp =
When getting this patch for -collector I noticed it conflicts with mgag200
and break its compilation.
On Wed, Jun 11, 2014 at 3:55 AM, Chris Wilson ch...@chris-wilson.co.uk
wrote:
i915.ko has a custom fbdev initialisation routine that aims to preserve
the current mode set by the BIOS, unless
This patch set adds the foundational support for Displayport compliance
testing in the
i915 driver. It implements the framework for automated test support that
preclude the
need (most) for operator input during testing. Tests for AUX transactions,
EDID reads
and basic link training have also
Add the skeleton framework for supporting automation for Displayport compliance
testing. This patch
adds the necessary framework for the source device to appropriately responded
to test automation
requests from a sink device.
Signed-off-by: Todd Previte tprev...@gmail.com
---
Add a simple function to pull the preferred mode out of an EDID block. This
function
is designed for use during Displayport compliance testing.
Signed-off-by: Todd Previte tprev...@gmail.com
---
drivers/gpu/drm/i915/intel_dp.c | 25 +
1 file changed, 25 insertions(+)
Adds the failsafe mode (640x480@60hz) as a constant and a function that
retrieves it. These are
designed for use in Displayport compliance testing only and should not be used
outside that
context.
Signed-off-by: Todd Previte tprev...@gmail.com
---
drivers/gpu/drm/i915/intel_dp.c | 12
This function computes the EDID checksum for a block of EDID data. This function
is necessary for Displayport compliance testing as it does not not require the
complete EDID checking functionality provided by the DRM layer functions.
Signed-off-by: Todd Previte tprev...@gmail.com
---
Add a wrapper around intel_crtc_set_config() to allow it to be called from the
DP compliance test functions. This is necessary to perform the internal mode
set operations required for compliance testing.
Signed-off-by: Todd Previte tprev...@gmail.com
---
drivers/gpu/drm/i915/intel_display.c | 6
These counters are used for Displayort complinace testing to detect error
conditions
when executing certain compliance tests. Currently these are used in the EDID
tests
to determine if the video mode needs to be set to the preferred mode or the
failsafe
mode.
Signed-off-by: Todd Previte
Link training for Displayport can fail in many ways and at multiple different
points
during the training process. Previously, errors were logged but no additional
action
was taken based on them. Consequently, training attempts could continue even
after
errors have occured that would prevent
The Displayport Link Layer Compliance Testing Specification 1.2 rev 1.1
specifies that
repeated AUX transactions after a failure (NACK, DEFER or no response) must have
a minimum delay of 400us before the resend can occur. Tests 4.2.1.1 and 4.2.1.2
are two
tests that require this specifically.
Implements an updated version of the automated testing function that handles
Displayport compliance for EDID operations.
Signed-off-by: Todd Previte tprev...@gmail.com
---
drivers/gpu/drm/i915/intel_dp.c | 77 -
1 file changed, 76 insertions(+), 1
Cleans up a couple of unused variables and an extraneous debug log
message that was unintentionally left behind.
Signed-off-by: Todd Previte tprev...@gmail.com
---
drivers/gpu/drm/i915/intel_dp.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git
Move the DPCD read to the top and check for an interrupt from the sink to catch
Displayport automated testing requests necessary to support Displayport
compliance
testing. The checks for active connectors and link status are moved below the
check for the interrupt.
Adds a check at the top to
Implements basic link training functionality for Displayport automated
compliance
testing.
Signed-off-by: Todd Previte tprev...@gmail.com
---
drivers/gpu/drm/i915/intel_dp.c | 47 +
1 file changed, 47 insertions(+)
diff --git
From: Deepak S deepa...@intel.com
We need do forcewake before Disabling RC6, This is what the BIOS
expects while going into suspend.
v2: updated commit message. (Daniel)
Reviewer: Paulo Zanoni paulo.r.zan...@intel.com
Cc: Paulo Zanoni paulo.r.zan...@intel.com
Signed-off-by: Deepak S
This is another drm-intel-collector updated notice:
http://cgit.freedesktop.org/~vivijim/drm-intel/log/?h=drm-intel-collector
Here goes the update list in order for better reviewers assignment:
Patch drm/i915: Bring UP Power Wells before disabling RC6. - Reviewer:
Paulo Zanoni
From: Deepak S deepa...@linux.intel.com
We might be leaving the PGU Frequency (and thus vnn) high during the suspend.
Flusing the delayed work queue should take care of this.
Signed-off-by: Deepak S deepa...@linux.intel.com
Signed-off-by: Rodrigo Vivi rodrigo.v...@intel.com
---
From: Vandana Kannan vandana.kan...@intel.com
Added a property to enable user space to set aspect ratio for HDMI displays.
If there is no user specified value, then PAR_NONE/Automatic option is set
by default. User can select aspect ratio 4:3 or 16:9. The aspect ratio
selected by user would come
From: Ville Syrjälä ville.syrj...@linux.intel.com
If the object is already UC leave it as UC instead of automagically
promoting it to WT in i915_gem_object_pin_to_display_plane() when
the hardware is WT capable.
Supposedly the user wanted UC for a reason, so let's respect that.
Signed-off-by:
From: Ben Widawsky benjamin.widaw...@intel.com
v2: fix conflict on rebase.
Cc: Kenneth Graunke kenn...@whitecape.org
Signed-off-by: Ben Widawsky b...@bwidawsk.net
Signed-off-by: Rodrigo Vivi rodrigo.v...@intel.com
---
drivers/gpu/drm/i915/i915_gem_context.c | 10 ++
1 file changed, 6
From: Chris Wilson ch...@chris-wilson.co.uk
If we try to execute on a known ring, but it has failed to be
initialised correctly, report that the GPU is hung rather than the
command invalid. This leaves us reporting EINVAL only if the user
requests execution on a ring that is not supported by the
From: Chris Wilson ch...@chris-wilson.co.uk
In the move over to use BIOS connector configs, we lost the ability to
force a specific set of connectors on or off. Try to remedy that by
dropping back to the old behavior if we detect a hard coded connector
config that tries to enable a connector
From: Chris Wilson ch...@chris-wilson.co.uk
We duplicated the legacy physical HWS setup routine for no good reason.
Combine it with the more recent virtual HWS setup for simplicity.
Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
Signed-off-by: Rodrigo Vivi rodrigo.v...@intel.com
---
From: Jesse Barnes jbar...@virtuousgeek.org
sanitize_enable_ppgtt is the function that checks all the conditions,
honoring a forced ppgtt status or doing auto-detect as necessary. Just
make sure it returns the right value in all cases and use that in the
macros instead of the confusing
From: Jesse Barnes jbar...@virtuousgeek.org
Put hw limitations first, disabling ppgtt if necessary right away.
After that, check user passed args or auto-detect and do the right
thing, falling back to aliasing PPGTT if the user tries to enable full
PPGTT but it isn't available.
v2: simplify
On Mon, 14 Jul 2014 14:47:07 -0300
Paulo Zanoni przan...@gmail.com wrote:
2014-07-14 14:26 GMT-03:00 Daniel Vetter dan...@ffwll.ch:
On Mon, Jul 14, 2014 at 12:23:11PM -0300, Paulo Zanoni wrote:
2014-06-20 13:29 GMT-03:00 Jesse Barnes jbar...@virtuousgeek.org:
Before we've installed the
Without the () gtkdoc won't recognize them as function references.
Signed-off-by: Daniel Vetter daniel.vet...@ffwll.ch
---
lib/igt_core.h | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/lib/igt_core.h b/lib/igt_core.h
index 8438d0063d81..a30d0d141acf 100644
---
On Mon, Jul 14, 2014 at 11:04:39AM +1000, Dave Airlie wrote:
From: Dave Airlie airl...@redhat.com
This reverts commit 38aecea0ccbb909d635619cba22f1891e589b434.
This breaks Haswell Thinkpad + Lenovo dock in SST mode with a HDMI monitor
attached.
Before this we can 1920x1200 mode, after
On Monday 14 July 2014 08:47 PM, Jesse Barnes wrote:
On Tue, 15 Jul 2014 13:03:55 +0530
Deepak S deepa...@linux.intel.com wrote:
On Saturday 28 June 2014 11:26 AM, deepa...@linux.intel.com wrote:
From: Deepak S deepa...@linux.intel.com
Drop WaGsvBringDownFreq on CHV.
When in RC6 requesting
On Monday 14 July 2014 08:47 PM, Jesse Barnes wrote:
On Tue, 15 Jul 2014 13:05:48 +0530
Deepak S deepa...@linux.intel.com wrote:
On Friday 20 June 2014 08:03 PM, deepa...@linux.intel.com wrote:
From: Deepak S deepa...@linux.intel.com
We might be leaving the GPU Frequency (and thus vnn) high
75 matches
Mail list logo
