linux-kernel - Re: [PULL] drm-intel-fixes

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: Windows password security audit tool. GUI, reports in PDF.

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

Date:	Sat, 9 Aug 2014 10:44:24 +0200
From:	Daniel Vetter <daniel.vetter@...ll.ch>
To:	Linus Torvalds <torvalds@...ux-foundation.org>
Cc:	Dave Airlie <airlied@...il.com>,
	intel-gfx <intel-gfx@...ts.freedesktop.org>,
	DRI <dri-devel@...ts.freedesktop.org>,
	Jani Nikula <jani.nikula@...ux.intel.com>,
	LKML <linux-kernel@...r.kernel.org>,
	Rodrigo Vivi <rodrigo.vivi@...el.com>
Subject: Re: [PULL] drm-intel-fixes

On Sat, Aug 9, 2014 at 12:07 AM, Linus Torvalds
<torvalds@...ux-foundation.org> wrote:
> On Fri, Aug 8, 2014 at 1:52 PM, Linus Torvalds
> <torvalds@...ux-foundation.org> wrote:
>>
>> Got this while bisecting. I'm not sure it's related
>
> It's not.

Iirc it was an intermediate merge issue between two patch series. If
you still see this on your tip, please scream. The warning is just one
of our modeset sequence asserts firing.

> The actual bug was panel self refresh. It's still broken, and doesn't
> work. So enabling it by default was a big mistake (commit
> b6d547791fd3: "drm/i915: Enable PSR by default.")
>
> I've reverted that commit, you guys can try again with PSR for the
> next kernel release if somebody figures out how to get the damn thing
> *out* of panel self-refresh (because that's what I'm assuming is going
> on: the graphics may still work fine, but nothing updates on the panel
> any more).

A bit surprising since the only psr issue I was aware of was that we
failed to entire psr (and that is likely a fumble in the tracking of
cursor updates). Otoh PSR isn't enabled on baytrail because it's
hang-happy there and the testcase we have for all the corner-cases in
psr tracking turned out to be completely broken and no on fixed this
yet. Anyway thanks for digging this up and pushing the revert
yesterday - I was already in w/e mode ;-)

Cheers, Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/