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
| ||
|
Date: Fri, 7 Feb 2014 15:05:59 +0100 From: Daniel Vetter <daniel.vetter@...ll.ch> To: Jiri Kosina <jkosina@...e.cz> Cc: Jani Nikula <jani.nikula@...el.com>, Dave Airlie <airlied@...ux.ie>, Linus Torvalds <torvalds@...ux-foundation.org>, Linux Kernel Mailing List <linux-kernel@...r.kernel.org>, DRI mailing list <dri-devel@...ts.freedesktop.org> Subject: Re: [git pull] drm next tree On Fri, Feb 7, 2014 at 2:40 PM, Jiri Kosina <jkosina@...e.cz> wrote: > On Fri, 7 Feb 2014, Jani Nikula wrote: > >> >> > git://people.freedesktop.org/~airlied/linux drm-next >> >> [ ... snip ... ] >> >> > Daniel Vetter (59): >> >> [ ... snip ... ] >> >> > drm/i915: dp aux irq support for g4x/vlv >> >> >> >> This commit causes all kinds of havoc on my ThinkPad x200s. It results in >> > [ ... snip ... ] >> > >> > Not having received any response, I suggest the following for 3.14 >> >> Daniel, it's your commit this bisects to... >> >> If we end up having to revert, I'd just touch the has_aux_irq part, for >> example: >> >> diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c >> index 5ede4e8..464c047 100644 >> --- a/drivers/gpu/drm/i915/intel_dp.c >> +++ b/drivers/gpu/drm/i915/intel_dp.c >> @@ -404,7 +404,7 @@ intel_dp_aux_ch(struct intel_dp *intel_dp, >> int i, ret, recv_bytes; >> uint32_t status; >> int try, precharge, clock = 0; >> - bool has_aux_irq = true; >> + bool has_aux_irq = INTEL_INFO(dev)->gen >= 5; >> uint32_t timeout; >> >> /* dp aux is extremely sensitive to irq latency, hence request the > > That, obviously, works for my system as well. Feel free to add > > Reported-and-tested-by: Jiri Kosina <jkosina@...e.cz> > > if you are going with this instead of the revert. Sorry for missing your report here. Before we disable this again for gen4 I want to make sure that it's the same irq misrouting issue which was already the cause for the gmbus irq mess. Can you please boot with pci=nomsi (so that i915 uses irq16)? If that's confirmed then I think we should add the same comment we've added in intel_i2c.c to this code so that no one dares to wake this dragon again. I'll do that when I commit the fix. Thanks, 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/
Powered by blists - more mailing lists