From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ben Widawsky Subject: Re: Memory corruption starting in i915 code, in 3.2-rc5 Date: Fri, 16 Dec 2011 14:04:23 -0800 Message-ID: <4EEBC067.2050006@bwidawsk.net> References: <4EE4F82E.9010801@palosanto.com> <86bore4z1l.fsf@sumi.keithp.com> <4EE520B0.1060105@palosanto.com> <4EE614E7.4010908@palosanto.com> <86pqftivy8.fsf@sumi.keithp.com> <4EE76BC7.4070000@palosanto.com> <86borcibjd.fsf@sumi.keithp.com> <20111213182650.GD4125@phenom.ffwll.local> <86ty50pbi2.fsf@sumi.keithp.com> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Return-path: Received: from cloud01.chad-versace.us (184-106-247-128.static.cloud-ips.com [184.106.247.128]) by gabe.freedesktop.org (Postfix) with ESMTP id 4B3729E789 for ; Fri, 16 Dec 2011 14:04:36 -0800 (PST) In-Reply-To: <86ty50pbi2.fsf@sumi.keithp.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org Errors-To: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org To: Keith Packard Cc: dri-devel@lists.freedesktop.org, =?ISO-8859-1?Q?Lasso?= , =?ISO-8859-1?Q?Alex_Villac=ED=ADs_?=@freedesktop.org List-Id: dri-devel@lists.freedesktop.org On 12/16/2011 11:22 AM, Keith Packard wrote: > On Tue, 13 Dec 2011 19:26:50 +0100, Daniel Vetter wrote: >> On Tue, Dec 13, 2011 at 10:14:46AM -0800, Keith Packard wrote: >>> On Tue, 13 Dec 2011 10:14:15 -0500, Alex Villac=ED=ADs Lasso wrote: >>> >>>> By using a bootable USB stick, I could check the logs, which >>>> showed many segfaults at /lib64/ld-2.14.90.so . >>> >>> Ouch! >>> >>> Please let me know if you find anything further; I'd like to get a >>> revert sent upstream in the next day or so. >> >> I think the revert is trtd. But if you revert it, please also >> revert/disable the ilk vt-d workaound or apply one of Ben's patches, >> because that one _does_ blow up, too. > = > Only if VT-d is enabled though, and that patch is now old enough that > reverting it may cause additional problems. > = > Ben's patches still appear to have problems -- they don't appear to > resolve the infinite recursion issue for unknown reasons. > = > I'm going to revert the patch which causes the reported regression, then > wait for Eric to finish up his request queue cleanups and revisit this > problem after that. If this is referring to Dave Airlie's comment that it blows up, I think he took that back as a bad backport on his part (maybe he can confirm). Aside from that, I never saw anyone report my patch as bad... or good. Ben