On Tue, May 27, 2014 at 11:17 PM, Imre Deak imre.d...@intel.com wrote:
On Tue, 2014-05-27 at 22:58 +0200, Daniel Vetter wrote:
Looks like work for Imre or someone else from the runtime pm gang.
Cc: Imre Deak imre.d...@intel.com
Signed-off-by: Daniel Vetter daniel.vet...@ffwll.ch
---
From: Sourab Gupta sourab.gu...@intel.com
This patch series replaces Blitter ring based flips with MMIO based flips.
This is useful for Media power well residency optimization. These may be
enabled on architectures where Render and Blitter engines reside in different
power wells.
The blitter ring
From: Sourab Gupta sourab.gu...@intel.com
This patch is for using mmio flips by default on VLV.
The module parameter controlling use of MMIO flips allows us to
control the default behaviour, which is set true for VLV and false
elsewhere.
Signed-off-by: Sourab Gupta sourab.gu...@intel.com
---
From: Sourab Gupta sourab.gu...@intel.com
Using MMIO based flips on Gen5+. The MMIO flips are useful for the Media power
well residency optimization. These maybe enabled on architectures where
Render and Blitter engines reside in different power wells.
The blitter ring is currently being used
On Wed, May 28, 2014 at 12:42:01PM +0530, sourab.gu...@intel.com wrote:
+static int intel_postpone_flip(struct drm_i915_gem_object *obj)
+{
+ int ret;
+
+ if (!obj-ring)
+ return 0;
+
+ if (i915_seqno_passed(obj-ring-get_seqno(obj-ring, true),
+
On Wed, May 28, 2014 at 12:42:01PM +0530, sourab.gu...@intel.com wrote:
+static void intel_do_mmio_flip(struct intel_crtc *intel_crtc)
+{
+ struct drm_device *dev = intel_crtc-base.dev;
+ struct drm_i915_private *dev_priv = dev-dev_private;
+ struct intel_framebuffer *intel_fb =
On Tue, May 27, 2014 at 10:24:14PM +0200, Daniel Vetter wrote:
On Tue, May 27, 2014 at 10:32:47PM +0300, Ville Syrjälä wrote:
On Fri, May 23, 2014 at 01:16:40PM -0700, Jesse Barnes wrote:
This is a bit like the CMN reset de-assert we do in DPIO_CTL, except
that it resets the whole common
On Wed, May 28, 2014 at 08:31:52AM +0100, Chris Wilson wrote:
On Wed, May 28, 2014 at 12:42:01PM +0530, sourab.gu...@intel.com wrote:
+static void intel_do_mmio_flip(struct intel_crtc *intel_crtc)
+{
+ struct drm_device *dev = intel_crtc-base.dev;
+ struct drm_i915_private *dev_priv =
On Tue, Feb 25, 2014 at 03:13:37PM +0200, ville.syrj...@linux.intel.com wrote:
From: Ville Syrjälä ville.syrj...@linux.intel.com
I tried to fix the C3 vs. vblank interrupt issues reportd in [1], but
it looks like the AGPBUSY# stuff doesn't help there for some reason. I
guess either the board
Digging out an ooold post of Daniel's...
On 04.03.2014 18:13, Daniel Vetter wrote:
On Tue, Feb 25, 2014 at 11:58:26AM +0900, Michel Dänzer wrote:
When the pre/post-modeset hooks were originally added, it worked like
this: the pre-modeset hook enabled the vblank interrupt, which updated
the
On Wed, 2014-05-28 at 07:30 +, Chris Wilson wrote:
On Wed, May 28, 2014 at 12:42:01PM +0530, sourab.gu...@intel.com wrote:
+static int intel_postpone_flip(struct drm_i915_gem_object *obj)
+{
+ int ret;
+
+ if (!obj-ring)
+ return 0;
+
+ if
On Wed, May 28, 2014 at 09:19:05AM +0100, Chris Wilson wrote:
On Tue, Feb 25, 2014 at 03:13:37PM +0200, ville.syrj...@linux.intel.com wrote:
From: Ville Syrjälä ville.syrj...@linux.intel.com
I tried to fix the C3 vs. vblank interrupt issues reportd in [1], but
it looks like the AGPBUSY#
On Wed, May 28, 2014 at 12:42:02PM +0530, sourab.gu...@intel.com wrote:
From: Sourab Gupta sourab.gu...@intel.com
This patch is for using mmio flips by default on VLV.
The module parameter controlling use of MMIO flips allows us to
control the default behaviour, which is set true for VLV and
On Wed, May 28, 2014 at 12:46:12PM +0300, Ville Syrjälä wrote:
On Wed, May 28, 2014 at 09:19:05AM +0100, Chris Wilson wrote:
The slow vblank delivery issue remains iirc though.
Yeah supposedly that's the case on gen3. This series does fix my 855
though. Unfortunately I've not come across
On Wed, May 28, 2014 at 06:12:54PM +0900, Michel Dänzer wrote:
Digging out an ooold post of Daniel's...
On 04.03.2014 18:13, Daniel Vetter wrote:
On Tue, Feb 25, 2014 at 11:58:26AM +0900, Michel Dänzer wrote:
When the pre/post-modeset hooks were originally added, it worked like
Functions that can't fail are such a bliss to work with, it'd be shame
to miss the occasion. The failure mode is the DSI connector not being
created, the rest of the initialization can carry on happily.
We weren't even checking that value anyway.
Suggested-by: Ville Syrjälä
On Tue, May 27, 2014 at 5:59 PM, Daniel Vetter daniel.vet...@ffwll.ch wrote:
After the split-out of crtc locks from the big mode_config.mutex
there's still two major areas it protects:
- Various connector probe states, like connector-status, EDID
properties, probed mode lists and similar
On Tue, May 27, 2014 at 09:33:09PM +0300, ville.syrj...@linux.intel.com wrote:
diff --git a/drivers/gpu/drm/i915/intel_display.c
b/drivers/gpu/drm/i915/intel_display.c
index 94ac51f..cb9dd8e 100644
--- a/drivers/gpu/drm/i915/intel_display.c
+++ b/drivers/gpu/drm/i915/intel_display.c
@@
On Wed, May 28, 2014 at 09:19:05AM +0100, Chris Wilson wrote:
On Tue, Feb 25, 2014 at 03:13:37PM +0200, ville.syrj...@linux.intel.com wrote:
From: Ville Syrjälä ville.syrj...@linux.intel.com
I tried to fix the C3 vs. vblank interrupt issues reportd in [1], but
it looks like the AGPBUSY#
On Tue, May 27, 2014 at 04:50:14PM -0700, Rodrigo Vivi wrote:
Some registers set during setup might not be persistent after suspend/resume.
This was causing bugs for some people that was unable to get PSR entry state
after resume cycle.
v2: Adding some comments and better commit message
On Wed, May 28, 2014 at 12:30:56PM +0100, Damien Lespiau wrote:
Functions that can't fail are such a bliss to work with, it'd be shame
to miss the occasion. The failure mode is the DSI connector not being
created, the rest of the initialization can carry on happily.
We weren't even checking
People from distros have reported that packaging the newer versions is
more difficult because we now depend on SWIG. We don't have to, it's
needed for the python dumper and is an optional build dependency.
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
configure.ac | 8 +++-
1
---
NEWS | 8
1 file changed, 8 insertions(+)
diff --git a/NEWS b/NEWS
index a03f20607dc6..c07269444e64 100644
--- a/NEWS
+++ b/NEWS
@@ -17,6 +17,14 @@ Release 1.7 (-XX-XX)
- Support for building libcairo based tests on Android. Set ANDROID_HAS_CAIRO=1
in the build enviroment
---
NEWS | 2 ++
1 file changed, 2 insertions(+)
diff --git a/NEWS b/NEWS
index c07269444e64..05a0c7d6cc2d 100644
--- a/NEWS
+++ b/NEWS
@@ -25,6 +25,8 @@ Release 1.7 (-XX-XX)
- Make swig an optional depency (Damien).
+- Helpers for runtime pm tests in igt_aux.
+
Release 1.6
This doesn't look possible but a little extra defense against the
improbable is worth it - an oops here could lockup the machine.
Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
---
drivers/gpu/drm/i915/intel_display.c |3 +++
1 file changed, 3 insertions(+)
diff --git
If the user tries to mmap through the GTT an object that is marked as
snooped, we report an error rather than allow the GPU to hang the
machine. The choice of EINVAL, however, was unfortunate as we turn that
into a WARN rather than a quiet SIGBUS.
Signed-off-by: Chris Wilson
Hello there,
I am not very good in linux stuff but, i would like to know if is there
any possibility to get this device work.
here is a result of lspci:
00:02.0 VGA compatible controller: Intel Corporation ValleyView Gen7
(rev 0a)
Or it is even planned?
Thanks
--
ANDRIAMILAMINA Monge
From: Ville Syrjälä ville.syrj...@linux.intel.com
Gen2 doesn't have the ring idle/stop bits in the SCPD/MI_MODE register,
so don't go spewing warnings about the state of those bits.
Signed-off-by: Ville Syrjälä ville.syrj...@linux.intel.com
---
drivers/gpu/drm/i915/intel_ringbuffer.c | 2 +-
1
On Wed, May 28, 2014 at 04:16:42PM +0100, Chris Wilson wrote:
This doesn't look possible but a little extra defense against the
improbable is worth it - an oops here could lockup the machine.
Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
---
drivers/gpu/drm/i915/intel_display.c |
On Wed, May 28, 2014 at 07:12:13PM +0300, ville.syrj...@linux.intel.com wrote:
From: Ville Syrjälä ville.syrj...@linux.intel.com
Gen2 doesn't have the ring idle/stop bits in the SCPD/MI_MODE register,
so don't go spewing warnings about the state of those bits.
Signed-off-by: Ville Syrjälä
This saves many ms per call on my BYT by eliminating Punit communication
from the hw readout paths.
Signed-off-by: Jesse Barnes jbar...@virtuousgeek.org
---
drivers/gpu/drm/i915/intel_pm.c | 8 +++-
1 file changed, 3 insertions(+), 5 deletions(-)
diff --git a/drivers/gpu/drm/i915/intel_pm.c
Mainly useful for catching all the callers in the previous patch.
Signed-off-by: Jesse Barnes jbar...@virtuousgeek.org
---
drivers/gpu/drm/i915/i915_drv.h | 4 ++--
drivers/gpu/drm/i915/intel_pm.c | 22 +++---
2 files changed, 13 insertions(+), 13 deletions(-)
diff --git
On Wed, 2014-05-28 at 09:50 -0700, Jesse Barnes wrote:
This saves many ms per call on my BYT by eliminating Punit communication
from the hw readout paths.
Signed-off-by: Jesse Barnes jbar...@virtuousgeek.org
---
drivers/gpu/drm/i915/intel_pm.c | 8 +++-
1 file changed, 3
On Wed, 28 May 2014 21:09:27 +0300
Imre Deak imre.d...@intel.com wrote:
On Wed, 2014-05-28 at 09:50 -0700, Jesse Barnes wrote:
This saves many ms per call on my BYT by eliminating Punit communication
from the hw readout paths.
Signed-off-by: Jesse Barnes jbar...@virtuousgeek.org
---
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
tests/kms_pipe_crc_basic.c | 9 +
1 file changed, 9 insertions(+)
diff --git a/tests/kms_pipe_crc_basic.c b/tests/kms_pipe_crc_basic.c
index c4abdf7..5727b61 100644
--- a/tests/kms_pipe_crc_basic.c
+++
Sometimes it's practical to be able to set the logging level to debug
from the command line arguments.
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
lib/igt_core.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/lib/igt_core.c b/lib/igt_core.c
index 6e553cf..9d59ab4 100644
We currenly have a bug on HSW where asking for the panel fitter CRC when the
power well is down returns 0x, but kms_pipe_crc_basic passes as it only
tests that the CRCs obtained for a FB are consistent over time. We consistently
read 0x, so the test was passing.
With this series
Let's make the test a bit more generic and have the number of CRCs we're
collecting a define so it can be changed easily.
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
tests/kms_pipe_crc_basic.c | 22 +++---
1 file changed, 11 insertions(+), 11 deletions(-)
diff
So we have a change to see something on the screen, while still being
relatively short (~1s per sub-test).
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
tests/kms_pipe_crc_basic.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/kms_pipe_crc_basic.c
Let's put the per-output test in its own function to get rid of 1 level
of indentation. We'll need it to cycle through 2 different framebuffers
to make sure we compute different CRCs if the fbs are different.
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
tests/kms_pipe_crc_basic.c |
Instead of just testing if the CRCs are stable, we also test 2 different
fbs to make sure that the CRC is actually changing.
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
tests/kms_pipe_crc_basic.c | 95 +-
1 file changed, 61
On Wed, May 28, 2014 at 05:15:13PM +0100, Chris Wilson wrote:
On Wed, May 28, 2014 at 07:12:13PM +0300, ville.syrj...@linux.intel.com wrote:
From: Ville Syrjälä ville.syrj...@linux.intel.com
Gen2 doesn't have the ring idle/stop bits in the SCPD/MI_MODE register,
so don't go spewing
On Wed, May 28, 2014 at 07:24:04PM +0100, Damien Lespiau wrote:
Sometimes it's practical to be able to set the logging level to debug
from the command line arguments.
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
The problem with this is that it won't work with simple igt tests
On Wed, May 28, 2014 at 07:24:02PM +0100, Damien Lespiau wrote:
So we have a change to see something on the screen, while still being
relatively short (~1s per sub-test).
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
igt_wait_for_keypress is geared at such use-cases ... Maybe we
From: Brad Volkin bradley.d.vol...@intel.com
This extends use of the command parser to VLV.
Note that the patch checks that the PPGTT bit is set appropriately when
PPGTT is enabled but ignores it when PPGTT is disabled. It would be
awkward to correctly invert the expected value to check that the
Jesse reportedly has a patch somewhere to (finally!) enable ppgtt on
vlv. Would we still need any part of this with ppgtt support on vlv?
-Daniel
On Wed, May 28, 2014 at 10:47 PM, bradley.d.vol...@intel.com wrote:
From: Brad Volkin bradley.d.vol...@intel.com
This extends use of the command
This gets us out of our init code and out to userspace quite a bit
faster, but does open us up to some bugs given the state of our init
time locking.
v2: switch to async_schedule (Chris)
check with lockdep, seems happy (Jesse)
move hotplug enable flag set to fbdev_initial_config (Jesse)
On Wed, May 28, 2014 at 02:39:03PM -0700, Jesse Barnes wrote:
This gets us out of our init code and out to userspace quite a bit
faster, but does open us up to some bugs given the state of our init
time locking.
v2: switch to async_schedule (Chris)
check with lockdep, seems happy
On Wed, May 28, 2014 at 02:29:34PM -0700, Daniel Vetter wrote:
Jesse reportedly has a patch somewhere to (finally!) enable ppgtt on
vlv. Would we still need any part of this with ppgtt support on vlv?
-Daniel
Of course, as soon as I accept that it'll never happen... :)
Off the top of my head,
On Wed, 28 May 2014 23:40:47 +0200
Daniel Vetter dan...@ffwll.ch wrote:
On Wed, May 28, 2014 at 02:39:03PM -0700, Jesse Barnes wrote:
This gets us out of our init code and out to userspace quite a bit
faster, but does open us up to some bugs given the state of our init
time locking.
On Wed, May 28, 2014 at 02:45:26PM -0700, Volkin, Bradley D wrote:
On Wed, May 28, 2014 at 02:29:34PM -0700, Daniel Vetter wrote:
Jesse reportedly has a patch somewhere to (finally!) enable ppgtt on
vlv. Would we still need any part of this with ppgtt support on vlv?
-Daniel
Of course,
Need testing and possibly disabling on earlier steppings, but looks ok
here on my B3.
Signed-off-by: Jesse Barnes jbar...@virtuousgeek.org
---
drivers/gpu/drm/i915/i915_drv.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i915/i915_drv.h
The always-on power well pixel path on haswell is routed such that it
bypasses the panel fitter when we use is. Which means the pfit CRC
source won't work in that configuration.
Add a new disallow-bypass flags to the pfit pipe config state and set
it when we want to use the pf CRC. Results in a
The PDPs seem to get screwed up otherwise, specifically PDP0. I am not
really clear why this is required, it just works with full PPGTT.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=78891
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=78934
Bugzilla:
The PDPs seem to get screwed up otherwise, specifically PDP0. I am not
really clear why this is required, it just works with full PPGTT.
v2: Only do it for gen8, to limit regression potential
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=78891
Bugzilla:
The PDPs seem to get screwed up otherwise, specifically PDP0. I am not
really clear why this is required, it just works with full PPGTT.
v2: Only do it for gen8, to limit regression potential
v3: Fix the bugzilla links
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=78891
Bugzilla:
On Thu, May 29, 2014 at 12:27:55AM +0200, Daniel Vetter wrote:
- if (IS_HASWELL(dev)
intel_crtc-config.pch_pfit.enabled)
+ if (IS_HASWELL(dev)
intel_crtc-config.pch_pfit.enabled ||
+ intel_crtc-config.pch_pfit.force_thru)
On 28.05.2014 20:19, Ville Syrjälä wrote:
On Wed, May 28, 2014 at 06:12:54PM +0900, Michel Dänzer wrote:
Digging out an ooold post of Daniel's...
On 04.03.2014 18:13, Daniel Vetter wrote:
On Tue, Feb 25, 2014 at 11:58:26AM +0900, Michel Dänzer wrote:
When the pre/post-modeset hooks were
On Wed, May 28, 2014 at 04:06:58PM -0700, Ben Widawsky wrote:
The PDPs seem to get screwed up otherwise, specifically PDP0. I am not
really clear why this is required, it just works with full PPGTT.
v2: Only do it for gen8, to limit regression potential
v3: Fix the bugzilla links
I should
59 matches
Mail list logo