On Wed, Jul 16, 2014 at 9:54 PM, Linus Torvalds
torva...@linux-foundation.org wrote:
Sorry for the top post, I'm on the road..
In wondering if we couldn't just keep both the old an the new names and have
them both point at the same variable? Remove the description for the old
name, but keep
On Wed, Jul 16, 2014 at 06:19:13PM -0300, Paulo Zanoni wrote:
2014-07-14 14:34 GMT-03:00 Daniel Vetter dan...@ffwll.ch:
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
On Fri, Jun 20, 2014 at 11:57:33AM -0700, Jesse Barnes wrote:
To avoid more spew with the new warnings.
Signed-off-by: Jesse Barnes jbar...@virtuousgeek.org
Ok, I've pulled this all into dinq. I guess we'll see how it all works
out.
-Daniel
---
drivers/gpu/drm/i915/intel_display.c | 2 ++
On Wed, Jul 16, 2014 at 05:49:30PM -0300, Paulo Zanoni wrote:
From: Paulo Zanoni paulo.r.zan...@intel.com
By the time I wrote this patch, it allowed me to catch some problems.
But due to patch reordering - in order to prevent fake regression
reports - this patch may be merged after the fixes
On Wed, Jul 16, 2014 at 05:06:34PM -0300, Paulo Zanoni wrote:
From: Paulo Zanoni paulo.r.zan...@intel.com
Since we merged runtime PM support for DPMS, it is possible that these
functions will be called when the power wells are disabled but a mode
is set, resulting in failed assertion and
On Thu, Jul 17, 2014 at 02:21:14PM +0530, deepa...@linux.intel.com wrote:
From: Deepak S deepa...@linux.intel.com
This was fumbled while trying to use the cached min/min/rpe values in
the vlv debugfs code.
This is a regression from
commit 03af20458a57a50735b12c1e3c23abc7ff70c6fa
On Tue, Jul 01, 2014 at 11:17:41AM -0700, Ben Widawsky wrote:
The bound list is global (all objects which back the VMAs are stored
here). Recently the BUG() in the offset lookup was demoted to a WARN,
but the fault actually lies in the caller, here.
This bug has existed since the initial
On Fri, Jul 04, 2014 at 09:56:54AM -0700, Ben Widawsky wrote:
On Fri, Jul 04, 2014 at 08:57:08AM +0100, Chris Wilson wrote:
On Tue, Jul 01, 2014 at 11:17:43AM -0700, Ben Widawsky wrote:
Some of the original PPGTT patches in this area where unmerged, and this
left a lot of confusion in our
On (Thu) 17 Jul 2014 [09:35:20], Daniel Vetter wrote:
On Wed, Jul 16, 2014 at 9:54 PM, Linus Torvalds
torva...@linux-foundation.org wrote:
Sorry for the top post, I'm on the road..
In wondering if we couldn't just keep both the old an the new names and have
them both point at the same
On Thu, Jul 17, 2014 at 02:32:41PM +0530, Amit Shah wrote:
On (Thu) 17 Jul 2014 [09:35:20], Daniel Vetter wrote:
On Wed, Jul 16, 2014 at 9:54 PM, Linus Torvalds
torva...@linux-foundation.org wrote:
Sorry for the top post, I'm on the road..
In wondering if we couldn't just keep both
On (Thu) 17 Jul 2014 [11:11:15], Daniel Vetter wrote:
On Thu, Jul 17, 2014 at 02:32:41PM +0530, Amit Shah wrote:
On (Thu) 17 Jul 2014 [09:35:20], Daniel Vetter wrote:
On Wed, Jul 16, 2014 at 9:54 PM, Linus Torvalds
torva...@linux-foundation.org wrote:
Sorry for the top post, I'm on
On 2014/7/16 22:20, Konrad Rzeszutek Wilk wrote:
On Thu, Jul 03, 2014 at 11:27:40PM +0300, Michael S. Tsirkin wrote:
On Thu, Jul 03, 2014 at 12:09:28PM -0700, Jesse Barnes wrote:
On Thu, 3 Jul 2014 14:26:12 -0400
Konrad Rzeszutek Wilk konrad.w...@oracle.com wrote:
On Thu, Jul 03, 2014 at
On Tue, Jul 01, 2014 at 11:17:48AM -0700, Ben Widawsky wrote:
If a VM still have objects which are bound (exactly: have a node
reserved in the drm_mm), and we are in the middle of a reset, we have no
hope of the standard methods fixing the situation (ring idle won't
work). We must therefore
On Tue, Jul 01, 2014 at 11:17:47AM -0700, Ben Widawsky wrote:
The comment [which was mine] is wrong. The context object can never be
bound in a PPGTT because it is only capable of living in the Global GTT.
So, remove the comment, and reorder the unref. What's nice about the
latter is it keeps
On Tue, Jul 01, 2014 at 11:17:35AM -0700, Ben Widawsky wrote:
Here be all the patches to make full PPGTT relatively stable on Broadwell.
Most
of the work was actually to the generic PPGTT code, and not BDW specific.
There
are basically 3 fixes:
1. Make the error state not horrible, but
2014-07-17 5:38 GMT-03:00 Daniel Vetter dan...@ffwll.ch:
On Wed, Jul 16, 2014 at 05:06:34PM -0300, Paulo Zanoni wrote:
From: Paulo Zanoni paulo.r.zan...@intel.com
Since we merged runtime PM support for DPMS, it is possible that these
functions will be called when the power wells are disabled
On Thu, Jul 17, 2014 at 2:53 PM, Paulo Zanoni przan...@gmail.com wrote:
2014-07-17 5:38 GMT-03:00 Daniel Vetter dan...@ffwll.ch:
On Wed, Jul 16, 2014 at 05:06:34PM -0300, Paulo Zanoni wrote:
From: Paulo Zanoni paulo.r.zan...@intel.com
Since we merged runtime PM support for DPMS, it is
2014-07-17 10:23 GMT-03:00 Daniel Vetter dan...@ffwll.ch:
On Thu, Jul 17, 2014 at 2:53 PM, Paulo Zanoni przan...@gmail.com wrote:
2014-07-17 5:38 GMT-03:00 Daniel Vetter dan...@ffwll.ch:
On Wed, Jul 16, 2014 at 05:06:34PM -0300, Paulo Zanoni wrote:
From: Paulo Zanoni paulo.r.zan...@intel.com
2014-07-17 5:06 GMT-03:00 Daniel Vetter dan...@ffwll.ch:
On Wed, Jul 16, 2014 at 06:19:13PM -0300, Paulo Zanoni wrote:
2014-07-14 14:34 GMT-03:00 Daniel Vetter dan...@ffwll.ch:
On Mon, Jul 14, 2014 at 11:56:57AM -0300, Paulo Zanoni wrote:
2014-07-07 18:53 GMT-03:00 Jesse Barnes
From: Paulo Zanoni paulo.r.zan...@intel.com
Since we merged runtime PM support for DPMS, it is possible that these
assertions will be called when the power wells are disabled but a mode
is set, resulting in failed assertion and device suspended while
reading register WARNs.
To reproduce the bug:
On Wed, Jul 16, 2014 at 05:22:38PM +0100, daniele.ceraolospu...@intel.com wrote:
From: Daniele Ceraolo Spurio daniele.ceraolospu...@intel.com
The context used to execute a batchbuffer is becoming increasingly
important. Duplicating to avoid modifications to the original trace.
I am sure we
On Thu, Jul 17, 2014 at 3:42 PM, Paulo Zanoni przan...@gmail.com wrote:
On Wed, Jul 16, 2014 at 06:19:13PM -0300, Paulo Zanoni wrote:
2014-07-14 14:34 GMT-03:00 Daniel Vetter dan...@ffwll.ch:
On Mon, Jul 14, 2014 at 11:56:57AM -0300, Paulo Zanoni wrote:
2014-07-07 18:53 GMT-03:00 Jesse
On Thu, Jul 17, 2014 at 3:29 PM, Paulo Zanoni przan...@gmail.com wrote:
I can do this, but please notice that we already have
power-well-checking code in many of the other assertions on our
driver... And it will probably be just a matter of time since someone
starts using the assertions again
On Thu, Jul 17, 2014 at 11:16:39AM -0300, Paulo Zanoni wrote:
From: Paulo Zanoni paulo.r.zan...@intel.com
Since we merged runtime PM support for DPMS, it is possible that these
assertions will be called when the power wells are disabled but a mode
is set, resulting in failed assertion and
Hi,
I have an ASUS UX32A laptop, which have troubles booting up with an
external HDMI display. The BIOS boot screen appears on the external
display, and the internal one stays blank during the whole boot. After
login I can make it work again if I turn it off and on, e.g. with
Ubuntu's
That sounds great. Tiejun could you confirm that with windows driver guys too?
I believe windows driver can also assume specific CPU/PCH combos. I will
discuss this with native Windows driver guys. Preferably, the same code path
can be used for both native and virtualization cases to avoid
Otherwise some iteractions depending on the current number of active rings
could overflow.
Cc: Ben Widawsky benjamin.widaw...@intel.com
Signed-off-by: Rodrigo Vivi rodrigo.v...@intel.com
---
drivers/gpu/drm/i915/i915_drv.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git
just ignore this one..
On Thu, Jul 17, 2014 at 5:00 AM, Rodrigo Vivi rodrigo.v...@intel.com
wrote:
Otherwise some iteractions depending on the current number of active rings
could overflow.
Cc: Ben Widawsky benjamin.widaw...@intel.com
Signed-off-by: Rodrigo Vivi rodrigo.v...@intel.com
---
2014-07-17 13:58 GMT-03:00 Daniel Vetter dan...@ffwll.ch:
On Thu, Jul 17, 2014 at 3:29 PM, Paulo Zanoni przan...@gmail.com wrote:
I can do this, but please notice that we already have
power-well-checking code in many of the other assertions on our
driver... And it will probably be just a
semaphore _sync_seqno, _seqno and _mbox are smaller than number of rings.
This optimization is to remove the ring itself from the list and the logic to
do that
is at intel_ring_sync_index as below:
/*
* rcs - 0 = vcs, 1 = bcs, 2 = vecs, 3 = vcs2;
* vcs - 0 = bcs, 1 = vecs, 2 = vcs2, 3 = rcs;
From: Paulo Zanoni paulo.r.zan...@intel.com
Since we started using intel_runtime_pm_disable_interrupts() at normal
(non-runtime) suspend/resume, we had to remove a WARN from
ironlake_disable_display_irq to avoid a case where we were doing the
correct thing and the WARN was not really needed. The
Hi list,
you were very helpful to nail down the interrupt storm issue a year
ago, I hope we can track this down, too.
I'm experiencing an issue with 3.15.5 on my Lenovo T400:
When my machine boots, I can see some of the boot messages flying by
(quite fast because the machine has a SSD).
On a
On Thu, Jul 17, 2014 at 05:35:20AM -0700, Rodrigo Vivi wrote:
semaphore _sync_seqno, _seqno and _mbox are smaller than number of rings.
This optimization is to remove the ring itself from the list and the logic to
do that
is at intel_ring_sync_index as below:
/*
* rcs - 0 = vcs, 1 = bcs,
On Fri, Jul 11, 2014 at 10:20:08AM -0700, armin.c.re...@intel.com wrote:
From: Armin Reese armin.c.re...@intel.com
Signed-off-by: Armin Reese armin.c.re...@intel.com
---
drivers/gpu/drm/i915/i915_gem_gtt.c | 13 ++---
1 file changed, 6 insertions(+), 7 deletions(-)
diff --git
With the increasing number of rings,
we probably have more information to print than we were printing.
Cc: Ben Widawsky benjamin.widaw...@intel.com
Signed-off-by: Rodrigo Vivi rodrigo.v...@intel.com
---
drivers/gpu/drm/i915/i915_gpu_error.c | 18 ++
1 file changed, 6
On Fri, Jul 11, 2014 at 10:20:07AM -0700, armin.c.re...@intel.com wrote:
From: Armin Reese armin.c.re...@intel.com
Signed-off-by: Armin Reese armin.c.re...@intel.com
---
drivers/gpu/drm/i915/i915_gem.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git
On Thu, Jul 17, 2014 at 04:45:02PM -0700, Ben Widawsky wrote:
On Fri, Jul 11, 2014 at 10:20:07AM -0700, armin.c.re...@intel.com wrote:
From: Armin Reese armin.c.re...@intel.com
Signed-off-by: Armin Reese armin.c.re...@intel.com
---
drivers/gpu/drm/i915/i915_gem.c | 6 +++---
1 file
On Thu, Jul 17, 2014 at 09:39:55AM -0700, Rodrigo Vivi wrote:
With the increasing number of rings,
we probably have more information to print than we were printing.
After our discussion were you going to send a new patch?
[snip]
--
Ben Widawsky, Intel Open Source Technology Center
You mean you prefer the the loop with for (i = 0; i hweight(ring_mask); i++)
{
instead? I thought you were ok with either and I preferred this one just to be
on the safest side and let userspace parse it properly.
Or do you prefer that other version with double loop but with names of rings?
On Thu, Jul 17, 2014 at 05:36:52PM -0700, Vivi, Rodrigo wrote:
You mean you prefer the the loop with for (i = 0; i hweight(ring_mask);
i++) {
instead? I thought you were ok with either and I preferred this one just to
be on the safest side and let userspace parse it properly.
I can live
With the increasing number of rings,
we probably have more information to print than we were printing.
v2: Loop only over active rings and print info with ring names.
Cc: Ben Widawsky benjamin.widaw...@intel.com
Signed-off-by: Rodrigo Vivi rodrigo.v...@intel.com
---
On Thu, Jul 17, 2014 at 10:58:17AM -0700, Rodrigo Vivi wrote:
With the increasing number of rings,
we probably have more information to print than we were printing.
v2: Loop only over active rings and print info with ring names.
Cc: Ben Widawsky benjamin.widaw...@intel.com
Signed-off-by:
Oops, meant to reply-to-all
On Thu, Jul 17, 2014 at 07:44:25PM -0700, Ben Widawsky wrote:
On Thu, Jul 17, 2014 at 10:42:20AM +0200, Daniel Vetter wrote:
On Thu, Jul 17, 2014 at 02:21:14PM +0530, deepa...@linux.intel.com wrote:
From: Deepak S deepa...@linux.intel.com
This was fumbled
From: Sonika Jindal sonika.jin...@intel.com
Signed-off-by: Sonika Jindal sonika.jin...@intel.com
---
drivers/gpu/drm/i915/i915_debugfs.c |6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/i915/i915_debugfs.c
b/drivers/gpu/drm/i915/i915_debugfs.c
index
From: Sonika Jindal sonika.jin...@intel.com
Signed-off-by: Sonika Jindal sonika.jin...@intel.com
---
drivers/gpu/drm/i915/intel_hdmi.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i915/intel_hdmi.c
b/drivers/gpu/drm/i915/intel_hdmi.c
index
From: Sonika Jindal sonika.jin...@intel.com
Signed-off-by: Sonika Jindal sonika.jin...@intel.com
---
drivers/gpu/drm/i915/intel_display.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i915/intel_display.c
b/drivers/gpu/drm/i915/intel_display.c
index
From: Sonika Jindal sonika.jin...@intel.com
This series prepares future platform enabling by changing HAS_PCH_SPLIT to more
appropriate check since the code accessed may not have anything to do with
having PCH or not.
Sonika Jindal (7):
drm/i915: Allowing changing of wm latencies for other
From: Sonika Jindal sonika.jin...@intel.com
Signed-off-by: Sonika Jindal sonika.jin...@intel.com
---
drivers/gpu/drm/i915/i915_drv.h |6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h
index
From: Sonika Jindal sonika.jin...@intel.com
Signed-off-by: Sonika Jindal sonika.jin...@intel.com
---
drivers/gpu/drm/i915/intel_hdmi.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i915/intel_hdmi.c
b/drivers/gpu/drm/i915/intel_hdmi.c
index
From: Sonika Jindal sonika.jin...@intel.com
Signed-off-by: Sonika Jindal sonika.jin...@intel.com
---
drivers/gpu/drm/i915/intel_display.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i915/intel_display.c
b/drivers/gpu/drm/i915/intel_display.c
index
From: Sonika Jindal sonika.jin...@intel.com
Signed-off-by: Sonika Jindal sonika.jin...@intel.com
---
drivers/gpu/drm/i915/intel_display.c |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/i915/intel_display.c
b/drivers/gpu/drm/i915/intel_display.c
index
51 matches
Mail list logo