linux-kernel - Re: Linus, please revert 7dc8f11437: regression in 5.7-rc0, hangs while attempting to run X

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Tue, 07 Apr 2020 08:24:27 +0100
From:   Chris Wilson <chris@...is-wilson.co.uk>
To:     Linus Torvalds <torvalds@...ux-foundation.org>,
        Pavel Machek <pavel@....cz>, airlied@...hat.com,
        intel-gfx@...ts.freedesktop.org, jani.nikula@...ux.intel.com,
        joonas.lahtinen@...ux.intel.com,
        kernel list <linux-kernel@...r.kernel.org>,
        matthew.auld@...el.com, rodrigo.vivi@...el.com
Cc:     tglx@...utronix.de, mingo@...hat.com, bp@...en8.de, hpa@...or.com
Subject: Re: Linus, please revert 7dc8f11437: regression in 5.7-rc0, hangs while attempting to run X

Quoting Pavel Machek (2020-04-07 08:20:47)
> 
> On Fri 2020-04-03 15:00:31, Pavel Machek wrote:
> > Hi!
> > 
> > 7dc8f1143778a35b190f9413f228b3cf28f67f8d
> > 
> >     drm/i915/gem: Drop relocation slowpath
> >     
> >     Since the relocations are no longer performed under a global
> >     struct_mutex, or any other lock, that is also held by pagefault handlers,
> >     we can relax and allow our fast path to take a fault. As we no longer
> >     need to abort the fast path for lock avoidance, we no longer need the
> >     slow path handling at all.
> > 
> > causes regression on thinkpad x220: instead of starting X, I'm looking
> > at blinking cursor.
> > 
> > Reverting the patch on too of 919dce24701f7b3 fixes things for me.
> 
> I have received no feedback from patch authors, and I believe we don't
> want to break boot in -rc1 on Intel hardware... so the commit should
> be simply reverted.

Beyond the fix already submitted?
-Chris