From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ben Hutchings Subject: Re: [PATCH 1/2] i915: Remove pipe A force quirk for 855GM and 845G Date: Tue, 22 Mar 2011 03:05:45 +0000 Message-ID: <1300763145.26693.374.camel@localhost> References: <1300662425.26693.301.camel@localhost> <849307$c2qo8g@azsmga001.ch.intel.com> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============2138921550==" Return-path: Received: from shadbolt.e.decadent.org.uk (shadbolt.e.decadent.org.uk [88.96.1.126]) by gabe.freedesktop.org (Postfix) with ESMTP id 2ECF99E7CE for ; Mon, 21 Mar 2011 20:05:56 -0700 (PDT) In-Reply-To: <849307$c2qo8g@azsmga001.ch.intel.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: Chris Wilson , Jesse Barnes Cc: 619019@bugs.debian.org, dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org --===============2138921550== Content-Type: multipart/signed; micalg="pgp-sha512"; protocol="application/pgp-signature"; boundary="=-iqWtdgBtZlM8Z6qlX1K9" --=-iqWtdgBtZlM8Z6qlX1K9 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Mon, 2011-03-21 at 07:38 +0000, Chris Wilson wrote: > On Sun, 20 Mar 2011 23:07:04 +0000, Ben Hutchings w= rote: > > Applying this quirk to the 855GM in all systems causes regressions > > (Debian bugs #493096, #619019). Instead, apply the quirk to specific > > models as listed in the old X driver. > >=20 > > I don't see any explanation for this quirk being applied to the 845G, > > except perhaps that VT switching used to hang if pipe A was turned > > off. However, that seems to be a problem only when using UMS. So > > remove the quirk for the 845G as well. >=20 > The quirk should only be required for 830M due to the numerous instances > where a unit on the second pipe is actually wired into the clock on the > first pipe. (And so it is easiest to keep the first pipe active at all > times.) When you say 'wired into', is this part of the chip design or something done on the board? Jesse, why did you add the quirk for other chips? > I'd prefer the quirk table to disappear and simply be replaced by > IS_830M(). However, that requires testing and so should only be done > piecemeal. And leaves some doubt as to why the other machines were in the > quirk table in the first place. The commit messages referring to VT switching suggest that the problems related to disabling part A may actually have been related to handover to the console driver before KMS. > Can you please repost each of these removals as a separate patch and lets > try and get a tested-by for each one? (Make sure the tester includes the > model name for his machine so we can double check the veracity of the > change.) I already have 4 regression reports for the addition of the quirk for 855GM: http://bugs.debian.org/618665 http://bugs.debian.org/618997 http://bugs.debian.org/619019 http://bugs.debian.org/619192 and one on an unidentified (as yet) chip: http://bugs.debian.org/619199 So I can just send a patch to revert 855GM. The odd thing about these reports is that the regression is reported to occur before the system has ever been suspended, and to be fixed (or mitigated) by suspending and resuming. I don't understand why the quirk even comes into play during boot. Ben. --=20 Ben Hutchings Once a job is fouled up, anything done to improve it makes it worse. --=-iqWtdgBtZlM8Z6qlX1K9 Content-Type: application/pgp-signature; name="signature.asc" Content-Description: This is a digitally signed message part -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.11 (GNU/Linux) iQIVAwUATYgSBOe/yOyVhhEJAQq76g/8C+BQSEaV7aUeM7S/cI+A0xZkh0kLEcum HK+UI+/UkbVja+T+y26b10LpEOcYB1kvIwrhKJEivDREP3DpZWwCffnNOAIOziyl sIk7/M0YJRACXi5JpglvZKUWDJovWdCmUHp2BeP7SMVmJseZPYKtToGE9H6EfzSY qjJ+/98dQF6nZfFC+mQJrveLFirXrkeE+/BPcatOke63NlBxOYFkeK6P6L1XADy+ dstso3/Y8Y3xWEFBAcILaj7JzfKGei3Habd8xkTgna/kBGc9TBp+Qt53NribdIsQ Bmg8Kuir4Hpour/zyE59cSfd7i+TQAtCatv4v2FdnnlYYdqOeeYCLmt1mjWztaPT DDvkY2kM25B3EVQeLzEQLsr20IL3/oLaFEc//Ya6DEoZaJAPHscequA7U7fstZMp zWkTV3YDCmZ2OVTHl/il9sfy/6ydcSKB55abIakgwptuc+tlZvLiwa4u+DFZG7uA GPJNDQ2y5nUZZIydyaz2iW1wcBH9ZaFzAm7DNtNUunVUH/VrDQFnyV1cT4idwaC4 wr7w66GNJ/58M/ShDlfKeZ6S5EaxQaRoBoUO2rCoTwrOqdNW2Z9+XzZf7OduiSoT vplNkxPp0SoQoNTt0yMuumhDidnGJ0LoFMrpjYYf2RsyZh3mrNnjEbXY1FRnOY6f C+VeVD1d970= =2Kk2 -----END PGP SIGNATURE----- --=-iqWtdgBtZlM8Z6qlX1K9-- --===============2138921550== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/dri-devel --===============2138921550==--