Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1662721 > unrolled thread

[4.4.70 REGRESSION] Nouveau hangs up at boot

Started byTakashi Iwai <tiwai@suse.de>
First post2017-06-09 22:30 +0200
Last post2017-06-15 08:40 +0200
Articles 6 — 4 participants

Back to article view | Back to linux.kernel


Contents

  [4.4.70 REGRESSION] Nouveau hangs up at boot Takashi Iwai <tiwai@suse.de> - 2017-06-09 22:30 +0200
    Re: [4.4.70 REGRESSION] Nouveau hangs up at boot Ben Skeggs <bskeggs@redhat.com> - 2017-06-13 00:50 +0200
      Re: [4.4.70 REGRESSION] Nouveau hangs up at boot Takashi Iwai <tiwai@suse.de> - 2017-06-13 08:10 +0200
        Re: [4.4.70 REGRESSION] Nouveau hangs up at boot Takashi Iwai <tiwai@suse.de> - 2017-06-13 15:40 +0200
          Re: [4.4.70 REGRESSION] Nouveau hangs up at boot "Luigi Baldoni" <aloisio@gmx.com> - 2017-06-13 16:50 +0200
          Re: [4.4.70 REGRESSION] Nouveau hangs up at boot Greg Kroah-Hartman <gregkh@linuxfoundation.org> - 2017-06-15 08:40 +0200

#1662721 — [4.4.70 REGRESSION] Nouveau hangs up at boot

FromTakashi Iwai <tiwai@suse.de>
Date2017-06-09 22:30 +0200
Subject[4.4.70 REGRESSION] Nouveau hangs up at boot
Message-ID<tQyHD-67N-3@gated-at.bofh.it>
Hi,

we've received a bug report about 4.4.70 kernel showing the hang up at
boot.  And, this turned out to be a regression in nouveau driver:
  https://bugzilla.suse.com/show_bug.cgi?id=1043467

I provided a test kernel reverting the last five commits about
nouveau below, and it was confirmed to work.  But still not figured
out which one actually breaks.

e4add1cf6b4154804350c3385c6d447cff3570de
    drm/nouveau/tmr: handle races with hw when updating the next alarm time
        commit 1b0f84380b10ee97f7d2dd191294de9017e94d1d upstream.

9d78e40f5f41ad1db1849f8d15acbda99d0871b4
    drm/nouveau/tmr: avoid processing completed alarms when adding a new one
        commit 330bdf62fe6a6c5b99a647f7bf7157107c9348b3 upstream.

5e07724c28f4e06fe42dd5b58bb6f9dd56510567
    drm/nouveau/tmr: fix corruption of the pending list when rescheduling an alarm
        commit 9fc64667ee48c9a25e7dca1a6bcb6906fec5bcc5 upstream.

27f82df2f02688c51d2c1d9f624cc0c5b8a62661
    drm/nouveau/tmr: ack interrupt before processing alarms
        commit 3733bd8b407211739e72d051e5f30ad82a52c4bc upstream.

3819271d8a5f4c6e0c8f71c339e44e2efbe40710
    drm/nouveau/therm: remove ineffective workarounds for alarm bugs
        commit e4311ee51d1e2676001b2d8fcefd92bdd79aad85 upstream.


Ben, is this a known problem?  Or is there any fixup?
The kernel back trace found in the bugzilla report shows the issue in
nvkm_timer_alarm_trigger(), at least.


thanks,

Takashi

[toc] | [next] | [standalone]


#1664279

FromBen Skeggs <bskeggs@redhat.com>
Date2017-06-13 00:50 +0200
Message-ID<tRGjL-8du-1@gated-at.bofh.it>
In reply to#1662721

[Multipart message — attachments visible in raw view] — view raw

On 06/10/2017 06:25 AM, Takashi Iwai wrote:
> Hi,
> 
> we've received a bug report about 4.4.70 kernel showing the hang up at
> boot.  And, this turned out to be a regression in nouveau driver:
>   https://bugzilla.suse.com/show_bug.cgi?id=1043467
> 
> I provided a test kernel reverting the last five commits about
> nouveau below, and it was confirmed to work.  But still not figured
> out which one actually breaks.
> 
> e4add1cf6b4154804350c3385c6d447cff3570de
>     drm/nouveau/tmr: handle races with hw when updating the next alarm time
>         commit 1b0f84380b10ee97f7d2dd191294de9017e94d1d upstream.
> 
> 9d78e40f5f41ad1db1849f8d15acbda99d0871b4
>     drm/nouveau/tmr: avoid processing completed alarms when adding a new one
>         commit 330bdf62fe6a6c5b99a647f7bf7157107c9348b3 upstream.
> 
> 5e07724c28f4e06fe42dd5b58bb6f9dd56510567
>     drm/nouveau/tmr: fix corruption of the pending list when rescheduling an alarm
>         commit 9fc64667ee48c9a25e7dca1a6bcb6906fec5bcc5 upstream.
> 
> 27f82df2f02688c51d2c1d9f624cc0c5b8a62661
>     drm/nouveau/tmr: ack interrupt before processing alarms
>         commit 3733bd8b407211739e72d051e5f30ad82a52c4bc upstream.
> 
> 3819271d8a5f4c6e0c8f71c339e44e2efbe40710
>     drm/nouveau/therm: remove ineffective workarounds for alarm bugs
>         commit e4311ee51d1e2676001b2d8fcefd92bdd79aad85 upstream.
> 
> 
> Ben, is this a known problem?  Or is there any fixup?
> The kernel back trace found in the bugzilla report shows the issue in
> nvkm_timer_alarm_trigger(), at least.
> 
A fix (b4e382ca7586a63b6c1e5221ce0863ff867c2df6) has been submitted already.

Sorry for the trouble!
Ben.

> 
> thanks,
> 
> Takashi
> 

[toc] | [prev] | [next] | [standalone]


#1664470

FromTakashi Iwai <tiwai@suse.de>
Date2017-06-13 08:10 +0200
Message-ID<tRNbz-4k7-17@gated-at.bofh.it>
In reply to#1664279
On Tue, 13 Jun 2017 00:40:26 +0200,
Ben Skeggs wrote:
> 
> On 06/10/2017 06:25 AM, Takashi Iwai wrote:
> > Hi,
> > 
> > we've received a bug report about 4.4.70 kernel showing the hang up at
> > boot.  And, this turned out to be a regression in nouveau driver:
> >   https://bugzilla.suse.com/show_bug.cgi?id=1043467
> > 
> > I provided a test kernel reverting the last five commits about
> > nouveau below, and it was confirmed to work.  But still not figured
> > out which one actually breaks.
> > 
> > e4add1cf6b4154804350c3385c6d447cff3570de
> >     drm/nouveau/tmr: handle races with hw when updating the next alarm time
> >         commit 1b0f84380b10ee97f7d2dd191294de9017e94d1d upstream.
> > 
> > 9d78e40f5f41ad1db1849f8d15acbda99d0871b4
> >     drm/nouveau/tmr: avoid processing completed alarms when adding a new one
> >         commit 330bdf62fe6a6c5b99a647f7bf7157107c9348b3 upstream.
> > 
> > 5e07724c28f4e06fe42dd5b58bb6f9dd56510567
> >     drm/nouveau/tmr: fix corruption of the pending list when rescheduling an alarm
> >         commit 9fc64667ee48c9a25e7dca1a6bcb6906fec5bcc5 upstream.
> > 
> > 27f82df2f02688c51d2c1d9f624cc0c5b8a62661
> >     drm/nouveau/tmr: ack interrupt before processing alarms
> >         commit 3733bd8b407211739e72d051e5f30ad82a52c4bc upstream.
> > 
> > 3819271d8a5f4c6e0c8f71c339e44e2efbe40710
> >     drm/nouveau/therm: remove ineffective workarounds for alarm bugs
> >         commit e4311ee51d1e2676001b2d8fcefd92bdd79aad85 upstream.
> > 
> > 
> > Ben, is this a known problem?  Or is there any fixup?
> > The kernel back trace found in the bugzilla report shows the issue in
> > nvkm_timer_alarm_trigger(), at least.
> > 
> A fix (b4e382ca7586a63b6c1e5221ce0863ff867c2df6) has been submitted already.
> 
> Sorry for the trouble!
> Ben.

Hrm, the commit doesn't apply to 4.4.x kernel properly.

Could you cook up a 4.4.x fix?  Then I'll prepare a test kernel
package for Luigi, so that he can test quickly.


thanks,

Takashi

[toc] | [prev] | [next] | [standalone]


#1664852

FromTakashi Iwai <tiwai@suse.de>
Date2017-06-13 15:40 +0200
Message-ID<tRUd4-6S-17@gated-at.bofh.it>
In reply to#1664470
On Tue, 13 Jun 2017 08:08:17 +0200,
Takashi Iwai wrote:
> 
> On Tue, 13 Jun 2017 00:40:26 +0200,
> Ben Skeggs wrote:
> > 
> > On 06/10/2017 06:25 AM, Takashi Iwai wrote:
> > > Hi,
> > > 
> > > we've received a bug report about 4.4.70 kernel showing the hang up at
> > > boot.  And, this turned out to be a regression in nouveau driver:
> > >   https://bugzilla.suse.com/show_bug.cgi?id=1043467
> > > 
> > > I provided a test kernel reverting the last five commits about
> > > nouveau below, and it was confirmed to work.  But still not figured
> > > out which one actually breaks.
> > > 
> > > e4add1cf6b4154804350c3385c6d447cff3570de
> > >     drm/nouveau/tmr: handle races with hw when updating the next alarm time
> > >         commit 1b0f84380b10ee97f7d2dd191294de9017e94d1d upstream.
> > > 
> > > 9d78e40f5f41ad1db1849f8d15acbda99d0871b4
> > >     drm/nouveau/tmr: avoid processing completed alarms when adding a new one
> > >         commit 330bdf62fe6a6c5b99a647f7bf7157107c9348b3 upstream.
> > > 
> > > 5e07724c28f4e06fe42dd5b58bb6f9dd56510567
> > >     drm/nouveau/tmr: fix corruption of the pending list when rescheduling an alarm
> > >         commit 9fc64667ee48c9a25e7dca1a6bcb6906fec5bcc5 upstream.
> > > 
> > > 27f82df2f02688c51d2c1d9f624cc0c5b8a62661
> > >     drm/nouveau/tmr: ack interrupt before processing alarms
> > >         commit 3733bd8b407211739e72d051e5f30ad82a52c4bc upstream.
> > > 
> > > 3819271d8a5f4c6e0c8f71c339e44e2efbe40710
> > >     drm/nouveau/therm: remove ineffective workarounds for alarm bugs
> > >         commit e4311ee51d1e2676001b2d8fcefd92bdd79aad85 upstream.
> > > 
> > > 
> > > Ben, is this a known problem?  Or is there any fixup?
> > > The kernel back trace found in the bugzilla report shows the issue in
> > > nvkm_timer_alarm_trigger(), at least.
> > > 
> > A fix (b4e382ca7586a63b6c1e5221ce0863ff867c2df6) has been submitted already.
> > 
> > Sorry for the trouble!
> > Ben.
> 
> Hrm, the commit doesn't apply to 4.4.x kernel properly.

My bad, it *does* apply.  I must have looked at a wrong commit, sorry
for the noise!

> Could you cook up a 4.4.x fix?  Then I'll prepare a test kernel
> package for Luigi, so that he can test quickly.

Luigi, a new test kernel is being built in OBS home:tiwai:bnc1043467-2
repo.  Please give it a try.


thanks,

Takashi

[toc] | [prev] | [next] | [standalone]


#1664942

From"Luigi Baldoni" <aloisio@gmx.com>
Date2017-06-13 16:50 +0200
Message-ID<tRViO-Jw-29@gated-at.bofh.it>
In reply to#1664852
Sent: Tuesday, June 13, 2017 at 3:32 PM
From: "Takashi Iwai" <tiwai@suse.de>
> Subject: Re: [4.4.70 REGRESSION] Nouveau hangs up at boot
>
> On Tue, 13 Jun 2017 08:08:17 +0200,
> Takashi Iwai wrote:
> > 
> > On Tue, 13 Jun 2017 00:40:26 +0200,
> > Ben Skeggs wrote:
> > > 
> > > On 06/10/2017 06:25 AM, Takashi Iwai wrote:
> > > > Hi,
> > > > 
> > > > we've received a bug report about 4.4.70 kernel showing the hang up at
> > > > boot.  And, this turned out to be a regression in nouveau driver:
> > > >   https://bugzilla.suse.com/show_bug.cgi?id=1043467
> > > > 
> > > > I provided a test kernel reverting the last five commits about
> > > > nouveau below, and it was confirmed to work.  But still not figured
> > > > out which one actually breaks.
> > > > 
> > > > e4add1cf6b4154804350c3385c6d447cff3570de
> > > >     drm/nouveau/tmr: handle races with hw when updating the next alarm time
> > > >         commit 1b0f84380b10ee97f7d2dd191294de9017e94d1d upstream.
> > > > 
> > > > 9d78e40f5f41ad1db1849f8d15acbda99d0871b4
> > > >     drm/nouveau/tmr: avoid processing completed alarms when adding a new one
> > > >         commit 330bdf62fe6a6c5b99a647f7bf7157107c9348b3 upstream.
> > > > 
> > > > 5e07724c28f4e06fe42dd5b58bb6f9dd56510567
> > > >     drm/nouveau/tmr: fix corruption of the pending list when rescheduling an alarm
> > > >         commit 9fc64667ee48c9a25e7dca1a6bcb6906fec5bcc5 upstream.
> > > > 
> > > > 27f82df2f02688c51d2c1d9f624cc0c5b8a62661
> > > >     drm/nouveau/tmr: ack interrupt before processing alarms
> > > >         commit 3733bd8b407211739e72d051e5f30ad82a52c4bc upstream.
> > > > 
> > > > 3819271d8a5f4c6e0c8f71c339e44e2efbe40710
> > > >     drm/nouveau/therm: remove ineffective workarounds for alarm bugs
> > > >         commit e4311ee51d1e2676001b2d8fcefd92bdd79aad85 upstream.
> > > > 
> > > > 
> > > > Ben, is this a known problem?  Or is there any fixup?
> > > > The kernel back trace found in the bugzilla report shows the issue in
> > > > nvkm_timer_alarm_trigger(), at least.
> > > > 
> > > A fix (b4e382ca7586a63b6c1e5221ce0863ff867c2df6) has been submitted already.
> > > 
> > > Sorry for the trouble!
> > > Ben.
> > 
> > Hrm, the commit doesn't apply to 4.4.x kernel properly.
> 
> My bad, it *does* apply.  I must have looked at a wrong commit, sorry
> for the noise!
> 
> > Could you cook up a 4.4.x fix?  Then I'll prepare a test kernel
> > package for Luigi, so that he can test quickly.
> 
> Luigi, a new test kernel is being built in OBS home:tiwai:bnc1043467-2
> repo.  Please give it a try.

4.4.71-2.ge1e822f-default works for me.

Regards

[toc] | [prev] | [next] | [standalone]


#1666485

FromGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Date2017-06-15 08:40 +0200
Message-ID<tSwBH-7qD-5@gated-at.bofh.it>
In reply to#1664852
On Tue, Jun 13, 2017 at 03:32:22PM +0200, Takashi Iwai wrote:
> On Tue, 13 Jun 2017 08:08:17 +0200,
> Takashi Iwai wrote:
> > 
> > On Tue, 13 Jun 2017 00:40:26 +0200,
> > Ben Skeggs wrote:
> > > 
> > > On 06/10/2017 06:25 AM, Takashi Iwai wrote:
> > > > Hi,
> > > > 
> > > > we've received a bug report about 4.4.70 kernel showing the hang up at
> > > > boot.  And, this turned out to be a regression in nouveau driver:
> > > >   https://bugzilla.suse.com/show_bug.cgi?id=1043467
> > > > 
> > > > I provided a test kernel reverting the last five commits about
> > > > nouveau below, and it was confirmed to work.  But still not figured
> > > > out which one actually breaks.
> > > > 
> > > > e4add1cf6b4154804350c3385c6d447cff3570de
> > > >     drm/nouveau/tmr: handle races with hw when updating the next alarm time
> > > >         commit 1b0f84380b10ee97f7d2dd191294de9017e94d1d upstream.
> > > > 
> > > > 9d78e40f5f41ad1db1849f8d15acbda99d0871b4
> > > >     drm/nouveau/tmr: avoid processing completed alarms when adding a new one
> > > >         commit 330bdf62fe6a6c5b99a647f7bf7157107c9348b3 upstream.
> > > > 
> > > > 5e07724c28f4e06fe42dd5b58bb6f9dd56510567
> > > >     drm/nouveau/tmr: fix corruption of the pending list when rescheduling an alarm
> > > >         commit 9fc64667ee48c9a25e7dca1a6bcb6906fec5bcc5 upstream.
> > > > 
> > > > 27f82df2f02688c51d2c1d9f624cc0c5b8a62661
> > > >     drm/nouveau/tmr: ack interrupt before processing alarms
> > > >         commit 3733bd8b407211739e72d051e5f30ad82a52c4bc upstream.
> > > > 
> > > > 3819271d8a5f4c6e0c8f71c339e44e2efbe40710
> > > >     drm/nouveau/therm: remove ineffective workarounds for alarm bugs
> > > >         commit e4311ee51d1e2676001b2d8fcefd92bdd79aad85 upstream.
> > > > 
> > > > 
> > > > Ben, is this a known problem?  Or is there any fixup?
> > > > The kernel back trace found in the bugzilla report shows the issue in
> > > > nvkm_timer_alarm_trigger(), at least.
> > > > 
> > > A fix (b4e382ca7586a63b6c1e5221ce0863ff867c2df6) has been submitted already.
> > > 
> > > Sorry for the trouble!
> > > Ben.
> > 
> > Hrm, the commit doesn't apply to 4.4.x kernel properly.
> 
> My bad, it *does* apply.  I must have looked at a wrong commit, sorry
> for the noise!

Great, that means this is fixed in 4.4.72.

thanks,

greg k-h

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web