* [PATCH 1/2] drm/i915: fix the SDE irq dmesg warnings properly
@ 2015-11-25 14:47 Jani Nikula
2015-11-25 14:47 ` [PATCH 2/2] Revert "drm/i915: shut up gen8+ SDE irq dmesg noise" Jani Nikula
` (2 more replies)
0 siblings, 3 replies; 9+ messages in thread
From: Jani Nikula @ 2015-11-25 14:47 UTC (permalink / raw)
To: intel-gfx; +Cc: jani.nikula, Daniel Vetter
We had the "The master control interrupt lied (SDE)!" check and error
message in place for a long time without any problems, until
commit aaf5ec2e51ab1d9c5e962b4728a1107ed3ff7a3e
Author: Sonika Jindal <sonika.jindal@intel.com>
Date: Wed Jul 8 17:07:47 2015 +0530
drm/i915: Handle HPD when it has actually occurred
caused the errors to start happening. This was bisected and reported,
but the error message was silenced in
commit 97e5ed1111dcc5300a0f59a55248cd243937a8ab
Author: Daniel Vetter <daniel.vetter@ffwll.ch>
Date: Fri Oct 23 10:56:12 2015 +0200
drm/i915: shut up gen8+ SDE irq dmesg noise
shooting the messenger while the debugging for why Sonika's commit
triggered the errors was still in progress.
It looks like we need to read and acknowledge the PCH_PORT_HOTPLUG
register even though the hotplug trigger indicates there isn't a hotplug
irq to handle.
Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
Cc: Sonika Jindal <sonika.jindal@intel.com>
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=92084
Fixes: aaf5ec2e51ab ("drm/i915: Handle HPD when it has actually occurred")
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
drivers/gpu/drm/i915/i915_irq.c | 16 ++++++++++++----
1 file changed, 12 insertions(+), 4 deletions(-)
diff --git a/drivers/gpu/drm/i915/i915_irq.c b/drivers/gpu/drm/i915/i915_irq.c
index c8ba94968aaf..982951d3153a 100644
--- a/drivers/gpu/drm/i915/i915_irq.c
+++ b/drivers/gpu/drm/i915/i915_irq.c
@@ -1825,7 +1825,17 @@ static void ibx_hpd_irq_handler(struct drm_device *dev, u32 hotplug_trigger,
u32 dig_hotplug_reg, pin_mask = 0, long_mask = 0;
dig_hotplug_reg = I915_READ(PCH_PORT_HOTPLUG);
+ if (!hotplug_trigger) {
+ u32 mask = PORTA_HOTPLUG_STATUS_MASK |
+ PORTD_HOTPLUG_STATUS_MASK |
+ PORTC_HOTPLUG_STATUS_MASK |
+ PORTB_HOTPLUG_STATUS_MASK;
+ dig_hotplug_reg &= ~mask;
+ }
+
I915_WRITE(PCH_PORT_HOTPLUG, dig_hotplug_reg);
+ if (!hotplug_trigger)
+ return;
intel_get_hpd_pins(&pin_mask, &long_mask, hotplug_trigger,
dig_hotplug_reg, hpd,
@@ -1840,8 +1850,7 @@ static void ibx_irq_handler(struct drm_device *dev, u32 pch_iir)
int pipe;
u32 hotplug_trigger = pch_iir & SDE_HOTPLUG_MASK;
- if (hotplug_trigger)
- ibx_hpd_irq_handler(dev, hotplug_trigger, hpd_ibx);
+ ibx_hpd_irq_handler(dev, hotplug_trigger, hpd_ibx);
if (pch_iir & SDE_AUDIO_POWER_MASK) {
int port = ffs((pch_iir & SDE_AUDIO_POWER_MASK) >>
@@ -1934,8 +1943,7 @@ static void cpt_irq_handler(struct drm_device *dev, u32 pch_iir)
int pipe;
u32 hotplug_trigger = pch_iir & SDE_HOTPLUG_MASK_CPT;
- if (hotplug_trigger)
- ibx_hpd_irq_handler(dev, hotplug_trigger, hpd_cpt);
+ ibx_hpd_irq_handler(dev, hotplug_trigger, hpd_cpt);
if (pch_iir & SDE_AUDIO_POWER_MASK_CPT) {
int port = ffs((pch_iir & SDE_AUDIO_POWER_MASK_CPT) >>
--
2.1.4
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH 2/2] Revert "drm/i915: shut up gen8+ SDE irq dmesg noise"
2015-11-25 14:47 [PATCH 1/2] drm/i915: fix the SDE irq dmesg warnings properly Jani Nikula
@ 2015-11-25 14:47 ` Jani Nikula
2015-11-25 14:49 ` [PATCH 1/2] drm/i915: fix the SDE irq dmesg warnings properly Ville Syrjälä
2015-12-13 12:49 ` Chris Wilson
2 siblings, 0 replies; 9+ messages in thread
From: Jani Nikula @ 2015-11-25 14:47 UTC (permalink / raw)
To: intel-gfx; +Cc: jani.nikula, Daniel Vetter
This reverts
commit 97e5ed1111dcc5300a0f59a55248cd243937a8ab
Author: Daniel Vetter <daniel.vetter@ffwll.ch>
Date: Fri Oct 23 10:56:12 2015 +0200
drm/i915: shut up gen8+ SDE irq dmesg noise
With the proper fix ("drm/i915: fix the SDE irq dmesg warnings
properly") reliably in place, bring back the error message.
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
drivers/gpu/drm/i915/i915_irq.c | 10 +++-------
1 file changed, 3 insertions(+), 7 deletions(-)
diff --git a/drivers/gpu/drm/i915/i915_irq.c b/drivers/gpu/drm/i915/i915_irq.c
index 982951d3153a..b327b37ed3b1 100644
--- a/drivers/gpu/drm/i915/i915_irq.c
+++ b/drivers/gpu/drm/i915/i915_irq.c
@@ -2359,13 +2359,9 @@ static irqreturn_t gen8_irq_handler(int irq, void *arg)
spt_irq_handler(dev, pch_iir);
else
cpt_irq_handler(dev, pch_iir);
- } else {
- /*
- * Like on previous PCH there seems to be something
- * fishy going on with forwarding PCH interrupts.
- */
- DRM_DEBUG_DRIVER("The master control interrupt lied (SDE)!\n");
- }
+ } else
+ DRM_ERROR("The master control interrupt lied (SDE)!\n");
+
}
I915_WRITE_FW(GEN8_MASTER_IRQ, GEN8_MASTER_IRQ_CONTROL);
--
2.1.4
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH 1/2] drm/i915: fix the SDE irq dmesg warnings properly
2015-11-25 14:47 [PATCH 1/2] drm/i915: fix the SDE irq dmesg warnings properly Jani Nikula
2015-11-25 14:47 ` [PATCH 2/2] Revert "drm/i915: shut up gen8+ SDE irq dmesg noise" Jani Nikula
@ 2015-11-25 14:49 ` Ville Syrjälä
2015-11-26 14:31 ` Jani Nikula
2015-12-13 12:49 ` Chris Wilson
2 siblings, 1 reply; 9+ messages in thread
From: Ville Syrjälä @ 2015-11-25 14:49 UTC (permalink / raw)
To: Jani Nikula; +Cc: intel-gfx, Daniel Vetter
On Wed, Nov 25, 2015 at 04:47:22PM +0200, Jani Nikula wrote:
> We had the "The master control interrupt lied (SDE)!" check and error
> message in place for a long time without any problems, until
>
> commit aaf5ec2e51ab1d9c5e962b4728a1107ed3ff7a3e
> Author: Sonika Jindal <sonika.jindal@intel.com>
> Date: Wed Jul 8 17:07:47 2015 +0530
>
> drm/i915: Handle HPD when it has actually occurred
>
> caused the errors to start happening. This was bisected and reported,
> but the error message was silenced in
>
> commit 97e5ed1111dcc5300a0f59a55248cd243937a8ab
> Author: Daniel Vetter <daniel.vetter@ffwll.ch>
> Date: Fri Oct 23 10:56:12 2015 +0200
>
> drm/i915: shut up gen8+ SDE irq dmesg noise
>
> shooting the messenger while the debugging for why Sonika's commit
> triggered the errors was still in progress.
>
> It looks like we need to read and acknowledge the PCH_PORT_HOTPLUG
> register even though the hotplug trigger indicates there isn't a hotplug
> irq to handle.
>
> Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
> Cc: Sonika Jindal <sonika.jindal@intel.com>
> Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=92084
> Fixes: aaf5ec2e51ab ("drm/i915: Handle HPD when it has actually occurred")
> Signed-off-by: Jani Nikula <jani.nikula@intel.com>
> ---
> drivers/gpu/drm/i915/i915_irq.c | 16 ++++++++++++----
> 1 file changed, 12 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/i915_irq.c b/drivers/gpu/drm/i915/i915_irq.c
> index c8ba94968aaf..982951d3153a 100644
> --- a/drivers/gpu/drm/i915/i915_irq.c
> +++ b/drivers/gpu/drm/i915/i915_irq.c
> @@ -1825,7 +1825,17 @@ static void ibx_hpd_irq_handler(struct drm_device *dev, u32 hotplug_trigger,
> u32 dig_hotplug_reg, pin_mask = 0, long_mask = 0;
>
> dig_hotplug_reg = I915_READ(PCH_PORT_HOTPLUG);
> + if (!hotplug_trigger) {
> + u32 mask = PORTA_HOTPLUG_STATUS_MASK |
> + PORTD_HOTPLUG_STATUS_MASK |
> + PORTC_HOTPLUG_STATUS_MASK |
> + PORTB_HOTPLUG_STATUS_MASK;
> + dig_hotplug_reg &= ~mask;
> + }
> +
> I915_WRITE(PCH_PORT_HOTPLUG, dig_hotplug_reg);
> + if (!hotplug_trigger)
> + return;
I would add some kind of comment around these parts to explain that
somehow the PCH doesn't seem to really ack the interrupt to the CPU
unless we touch the hotplug register.
Otherwise (for both patches)
Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Also not sure if SKL might need something similar as well...
>
> intel_get_hpd_pins(&pin_mask, &long_mask, hotplug_trigger,
> dig_hotplug_reg, hpd,
> @@ -1840,8 +1850,7 @@ static void ibx_irq_handler(struct drm_device *dev, u32 pch_iir)
> int pipe;
> u32 hotplug_trigger = pch_iir & SDE_HOTPLUG_MASK;
>
> - if (hotplug_trigger)
> - ibx_hpd_irq_handler(dev, hotplug_trigger, hpd_ibx);
> + ibx_hpd_irq_handler(dev, hotplug_trigger, hpd_ibx);
>
> if (pch_iir & SDE_AUDIO_POWER_MASK) {
> int port = ffs((pch_iir & SDE_AUDIO_POWER_MASK) >>
> @@ -1934,8 +1943,7 @@ static void cpt_irq_handler(struct drm_device *dev, u32 pch_iir)
> int pipe;
> u32 hotplug_trigger = pch_iir & SDE_HOTPLUG_MASK_CPT;
>
> - if (hotplug_trigger)
> - ibx_hpd_irq_handler(dev, hotplug_trigger, hpd_cpt);
> + ibx_hpd_irq_handler(dev, hotplug_trigger, hpd_cpt);
>
> if (pch_iir & SDE_AUDIO_POWER_MASK_CPT) {
> int port = ffs((pch_iir & SDE_AUDIO_POWER_MASK_CPT) >>
> --
> 2.1.4
--
Ville Syrjälä
Intel OTC
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 1/2] drm/i915: fix the SDE irq dmesg warnings properly
2015-11-25 14:49 ` [PATCH 1/2] drm/i915: fix the SDE irq dmesg warnings properly Ville Syrjälä
@ 2015-11-26 14:31 ` Jani Nikula
0 siblings, 0 replies; 9+ messages in thread
From: Jani Nikula @ 2015-11-26 14:31 UTC (permalink / raw)
To: Ville Syrjälä; +Cc: intel-gfx, Daniel Vetter
On Wed, 25 Nov 2015, Ville Syrjälä <ville.syrjala@linux.intel.com> wrote:
> On Wed, Nov 25, 2015 at 04:47:22PM +0200, Jani Nikula wrote:
>> We had the "The master control interrupt lied (SDE)!" check and error
>> message in place for a long time without any problems, until
>>
>> commit aaf5ec2e51ab1d9c5e962b4728a1107ed3ff7a3e
>> Author: Sonika Jindal <sonika.jindal@intel.com>
>> Date: Wed Jul 8 17:07:47 2015 +0530
>>
>> drm/i915: Handle HPD when it has actually occurred
>>
>> caused the errors to start happening. This was bisected and reported,
>> but the error message was silenced in
>>
>> commit 97e5ed1111dcc5300a0f59a55248cd243937a8ab
>> Author: Daniel Vetter <daniel.vetter@ffwll.ch>
>> Date: Fri Oct 23 10:56:12 2015 +0200
>>
>> drm/i915: shut up gen8+ SDE irq dmesg noise
>>
>> shooting the messenger while the debugging for why Sonika's commit
>> triggered the errors was still in progress.
>>
>> It looks like we need to read and acknowledge the PCH_PORT_HOTPLUG
>> register even though the hotplug trigger indicates there isn't a hotplug
>> irq to handle.
>>
>> Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
>> Cc: Sonika Jindal <sonika.jindal@intel.com>
>> Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
>> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=92084
>> Fixes: aaf5ec2e51ab ("drm/i915: Handle HPD when it has actually occurred")
>> Signed-off-by: Jani Nikula <jani.nikula@intel.com>
>> ---
>> drivers/gpu/drm/i915/i915_irq.c | 16 ++++++++++++----
>> 1 file changed, 12 insertions(+), 4 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/i915/i915_irq.c b/drivers/gpu/drm/i915/i915_irq.c
>> index c8ba94968aaf..982951d3153a 100644
>> --- a/drivers/gpu/drm/i915/i915_irq.c
>> +++ b/drivers/gpu/drm/i915/i915_irq.c
>> @@ -1825,7 +1825,17 @@ static void ibx_hpd_irq_handler(struct drm_device *dev, u32 hotplug_trigger,
>> u32 dig_hotplug_reg, pin_mask = 0, long_mask = 0;
>>
>> dig_hotplug_reg = I915_READ(PCH_PORT_HOTPLUG);
>> + if (!hotplug_trigger) {
>> + u32 mask = PORTA_HOTPLUG_STATUS_MASK |
>> + PORTD_HOTPLUG_STATUS_MASK |
>> + PORTC_HOTPLUG_STATUS_MASK |
>> + PORTB_HOTPLUG_STATUS_MASK;
>> + dig_hotplug_reg &= ~mask;
>> + }
>> +
>> I915_WRITE(PCH_PORT_HOTPLUG, dig_hotplug_reg);
>> + if (!hotplug_trigger)
>> + return;
>
> I would add some kind of comment around these parts to explain that
> somehow the PCH doesn't seem to really ack the interrupt to the CPU
> unless we touch the hotplug register.
>
> Otherwise (for both patches)
> Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Pushed to drm-intel-next-queued, with a comment added and the commit
message amended slightly, thanks for the review.
BR,
Jani.
>
> Also not sure if SKL might need something similar as well...
>
>>
>> intel_get_hpd_pins(&pin_mask, &long_mask, hotplug_trigger,
>> dig_hotplug_reg, hpd,
>> @@ -1840,8 +1850,7 @@ static void ibx_irq_handler(struct drm_device *dev, u32 pch_iir)
>> int pipe;
>> u32 hotplug_trigger = pch_iir & SDE_HOTPLUG_MASK;
>>
>> - if (hotplug_trigger)
>> - ibx_hpd_irq_handler(dev, hotplug_trigger, hpd_ibx);
>> + ibx_hpd_irq_handler(dev, hotplug_trigger, hpd_ibx);
>>
>> if (pch_iir & SDE_AUDIO_POWER_MASK) {
>> int port = ffs((pch_iir & SDE_AUDIO_POWER_MASK) >>
>> @@ -1934,8 +1943,7 @@ static void cpt_irq_handler(struct drm_device *dev, u32 pch_iir)
>> int pipe;
>> u32 hotplug_trigger = pch_iir & SDE_HOTPLUG_MASK_CPT;
>>
>> - if (hotplug_trigger)
>> - ibx_hpd_irq_handler(dev, hotplug_trigger, hpd_cpt);
>> + ibx_hpd_irq_handler(dev, hotplug_trigger, hpd_cpt);
>>
>> if (pch_iir & SDE_AUDIO_POWER_MASK_CPT) {
>> int port = ffs((pch_iir & SDE_AUDIO_POWER_MASK_CPT) >>
>> --
>> 2.1.4
--
Jani Nikula, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 1/2] drm/i915: fix the SDE irq dmesg warnings properly
2015-11-25 14:47 [PATCH 1/2] drm/i915: fix the SDE irq dmesg warnings properly Jani Nikula
2015-11-25 14:47 ` [PATCH 2/2] Revert "drm/i915: shut up gen8+ SDE irq dmesg noise" Jani Nikula
2015-11-25 14:49 ` [PATCH 1/2] drm/i915: fix the SDE irq dmesg warnings properly Ville Syrjälä
@ 2015-12-13 12:49 ` Chris Wilson
2015-12-14 14:54 ` Ville Syrjälä
2 siblings, 1 reply; 9+ messages in thread
From: Chris Wilson @ 2015-12-13 12:49 UTC (permalink / raw)
To: Jani Nikula; +Cc: Daniel Vetter, intel-gfx
On Wed, Nov 25, 2015 at 04:47:22PM +0200, Jani Nikula wrote:
> We had the "The master control interrupt lied (SDE)!" check and error
> message in place for a long time without any problems, until
>
> commit aaf5ec2e51ab1d9c5e962b4728a1107ed3ff7a3e
> Author: Sonika Jindal <sonika.jindal@intel.com>
> Date: Wed Jul 8 17:07:47 2015 +0530
>
> drm/i915: Handle HPD when it has actually occurred
>
> caused the errors to start happening. This was bisected and reported,
> but the error message was silenced in
>
> commit 97e5ed1111dcc5300a0f59a55248cd243937a8ab
> Author: Daniel Vetter <daniel.vetter@ffwll.ch>
> Date: Fri Oct 23 10:56:12 2015 +0200
>
> drm/i915: shut up gen8+ SDE irq dmesg noise
>
> shooting the messenger while the debugging for why Sonika's commit
> triggered the errors was still in progress.
>
> It looks like we need to read and acknowledge the PCH_PORT_HOTPLUG
> register even though the hotplug trigger indicates there isn't a hotplug
> irq to handle.
>
> Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
> Cc: Sonika Jindal <sonika.jindal@intel.com>
> Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=92084
> Fixes: aaf5ec2e51ab ("drm/i915: Handle HPD when it has actually occurred")
> Signed-off-by: Jani Nikula <jani.nikula@intel.com>
And to you I say:
[ 4.249930] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
[ 4.251037] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
[ 4.251403] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
[ 4.252141] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
[ 4.252509] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
[ 4.253247] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
[ 4.253616] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
[ 4.253985] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
[ 4.254724] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
[ 4.255460] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
[ 4.256198] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
[ 4.256567] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
[ 4.257305] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
[ 4.258043] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
[ 4.258781] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
[ 4.259519] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
[ 4.259889] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
[ 4.260626] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
[ 4.261365] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
[ 4.261734] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
[ 4.262103] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
[ 4.262163] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
[ 4.263261] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
[ 4.263999] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
[ 4.264737] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
[ 4.265106] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
[ 4.266214] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
i7-5557U nuc currently connected to HDMI.
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 1/2] drm/i915: fix the SDE irq dmesg warnings properly
2015-12-13 12:49 ` Chris Wilson
@ 2015-12-14 14:54 ` Ville Syrjälä
2015-12-15 15:26 ` Chris Wilson
0 siblings, 1 reply; 9+ messages in thread
From: Ville Syrjälä @ 2015-12-14 14:54 UTC (permalink / raw)
To: Chris Wilson, Jani Nikula, intel-gfx, Daniel Vetter
On Sun, Dec 13, 2015 at 12:49:45PM +0000, Chris Wilson wrote:
> On Wed, Nov 25, 2015 at 04:47:22PM +0200, Jani Nikula wrote:
> > We had the "The master control interrupt lied (SDE)!" check and error
> > message in place for a long time without any problems, until
> >
> > commit aaf5ec2e51ab1d9c5e962b4728a1107ed3ff7a3e
> > Author: Sonika Jindal <sonika.jindal@intel.com>
> > Date: Wed Jul 8 17:07:47 2015 +0530
> >
> > drm/i915: Handle HPD when it has actually occurred
> >
> > caused the errors to start happening. This was bisected and reported,
> > but the error message was silenced in
> >
> > commit 97e5ed1111dcc5300a0f59a55248cd243937a8ab
> > Author: Daniel Vetter <daniel.vetter@ffwll.ch>
> > Date: Fri Oct 23 10:56:12 2015 +0200
> >
> > drm/i915: shut up gen8+ SDE irq dmesg noise
> >
> > shooting the messenger while the debugging for why Sonika's commit
> > triggered the errors was still in progress.
> >
> > It looks like we need to read and acknowledge the PCH_PORT_HOTPLUG
> > register even though the hotplug trigger indicates there isn't a hotplug
> > irq to handle.
> >
> > Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
> > Cc: Sonika Jindal <sonika.jindal@intel.com>
> > Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
> > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=92084
> > Fixes: aaf5ec2e51ab ("drm/i915: Handle HPD when it has actually occurred")
> > Signed-off-by: Jani Nikula <jani.nikula@intel.com>
>
> And to you I say:
>
> [ 4.249930] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
> [ 4.251037] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
> [ 4.251403] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
> [ 4.252141] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
> [ 4.252509] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
> [ 4.253247] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
> [ 4.253616] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
> [ 4.253985] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
> [ 4.254724] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
> [ 4.255460] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
> [ 4.256198] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
> [ 4.256567] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
> [ 4.257305] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
> [ 4.258043] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
> [ 4.258781] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
> [ 4.259519] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
> [ 4.259889] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
> [ 4.260626] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
> [ 4.261365] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
> [ 4.261734] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
> [ 4.262103] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
> [ 4.262163] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
> [ 4.263261] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
> [ 4.263999] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
> [ 4.264737] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
> [ 4.265106] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
> [ 4.266214] [drm:gen8_irq_handler [i915]] *ERROR* The master control interrupt lied (SDE)!
>
> i7-5557U nuc currently connected to HDMI.
Sigh. Do those correcpond to AUX attempts by any chance? IIRC that was where
Jani saw the problem on his BDW.
Oh and maybe you can try Jani's debug patch
https://bugs.freedesktop.org/show_bug.cgi?id=92084#c20
to show us what the hotplug register says during these fails?
--
Ville Syrjälä
Intel OTC
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 1/2] drm/i915: fix the SDE irq dmesg warnings properly
2015-12-14 14:54 ` Ville Syrjälä
@ 2015-12-15 15:26 ` Chris Wilson
2016-01-06 1:47 ` Josh Boyer
0 siblings, 1 reply; 9+ messages in thread
From: Chris Wilson @ 2015-12-15 15:26 UTC (permalink / raw)
To: Ville Syrjälä; +Cc: Jani Nikula, Daniel Vetter, intel-gfx
On Mon, Dec 14, 2015 at 04:54:02PM +0200, Ville Syrjälä wrote:
> On Sun, Dec 13, 2015 at 12:49:45PM +0000, Chris Wilson wrote:
> > i7-5557U nuc currently connected to HDMI.
>
> Sigh. Do those correcpond to AUX attempts by any chance? IIRC that was where
> Jani saw the problem on his BDW.
>
> Oh and maybe you can try Jani's debug patch
> https://bugs.freedesktop.org/show_bug.cgi?id=92084#c20
> to show us what the hotplug register says during these fails?
Sure, just only happens when plugged in, so likely be a while before I
allow it back into the warmth.
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 1/2] drm/i915: fix the SDE irq dmesg warnings properly
2015-12-15 15:26 ` Chris Wilson
@ 2016-01-06 1:47 ` Josh Boyer
2016-01-07 9:45 ` Jani Nikula
0 siblings, 1 reply; 9+ messages in thread
From: Josh Boyer @ 2016-01-06 1:47 UTC (permalink / raw)
To: Chris Wilson, Ville Syrjälä,
Jani Nikula, Intel Graphics Development, Daniel Vetter
On Tue, Dec 15, 2015 at 10:26 AM, Chris Wilson <chris@chris-wilson.co.uk> wrote:
> On Mon, Dec 14, 2015 at 04:54:02PM +0200, Ville Syrjälä wrote:
>> On Sun, Dec 13, 2015 at 12:49:45PM +0000, Chris Wilson wrote:
>> > i7-5557U nuc currently connected to HDMI.
>>
>> Sigh. Do those correcpond to AUX attempts by any chance? IIRC that was where
>> Jani saw the problem on his BDW.
>>
>> Oh and maybe you can try Jani's debug patch
>> https://bugs.freedesktop.org/show_bug.cgi?id=92084#c20
>> to show us what the hotplug register says during these fails?
>
> Sure, just only happens when plugged in, so likely be a while before I
> allow it back into the warmth.
Has there been any further progress on this issue? I'm still seeing
this with 4.3.3 and we're looking to rebase Fedora to 4.3.y or 4.4
soon. As far as I'm aware this remains unfixed upstream.
josh
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 1/2] drm/i915: fix the SDE irq dmesg warnings properly
2016-01-06 1:47 ` Josh Boyer
@ 2016-01-07 9:45 ` Jani Nikula
0 siblings, 0 replies; 9+ messages in thread
From: Jani Nikula @ 2016-01-07 9:45 UTC (permalink / raw)
To: Josh Boyer, Chris Wilson, Ville Syrjälä,
Intel Graphics Development, Daniel Vetter
On Wed, 06 Jan 2016, Josh Boyer <jwboyer@fedoraproject.org> wrote:
> On Tue, Dec 15, 2015 at 10:26 AM, Chris Wilson <chris@chris-wilson.co.uk> wrote:
>> On Mon, Dec 14, 2015 at 04:54:02PM +0200, Ville Syrjälä wrote:
>>> On Sun, Dec 13, 2015 at 12:49:45PM +0000, Chris Wilson wrote:
>>> > i7-5557U nuc currently connected to HDMI.
>>>
>>> Sigh. Do those correcpond to AUX attempts by any chance? IIRC that was where
>>> Jani saw the problem on his BDW.
>>>
>>> Oh and maybe you can try Jani's debug patch
>>> https://bugs.freedesktop.org/show_bug.cgi?id=92084#c20
>>> to show us what the hotplug register says during these fails?
>>
>> Sure, just only happens when plugged in, so likely be a while before I
>> allow it back into the warmth.
>
> Has there been any further progress on this issue? I'm still seeing
> this with 4.3.3 and we're looking to rebase Fedora to 4.3.y or 4.4
> soon. As far as I'm aware this remains unfixed upstream.
Back and forth,
http://patchwork.freedesktop.org/patch/msgid/1452155350-14658-1-git-send-email-jani.nikula@intel.com
BR,
Jani.
>
> josh
--
Jani Nikula, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2016-01-07 9:45 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-11-25 14:47 [PATCH 1/2] drm/i915: fix the SDE irq dmesg warnings properly Jani Nikula
2015-11-25 14:47 ` [PATCH 2/2] Revert "drm/i915: shut up gen8+ SDE irq dmesg noise" Jani Nikula
2015-11-25 14:49 ` [PATCH 1/2] drm/i915: fix the SDE irq dmesg warnings properly Ville Syrjälä
2015-11-26 14:31 ` Jani Nikula
2015-12-13 12:49 ` Chris Wilson
2015-12-14 14:54 ` Ville Syrjälä
2015-12-15 15:26 ` Chris Wilson
2016-01-06 1:47 ` Josh Boyer
2016-01-07 9:45 ` Jani Nikula
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.