Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757096Ab2BXPRq (ORCPT ); Fri, 24 Feb 2012 10:17:46 -0500 Received: from mx1.redhat.com ([209.132.183.28]:43444 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757047Ab2BXPRp (ORCPT ); Fri, 24 Feb 2012 10:17:45 -0500 Message-ID: <1330096655.26569.22.camel@atropine> Subject: Re: [PATCH] drm: Reduce the number of retries whilst reading EDIDs From: Adam Jackson To: Linus Torvalds Cc: Chris Wilson , Dave Airlie , linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, Eugeni Dodonov Date: Fri, 24 Feb 2012 10:17:35 -0500 In-Reply-To: References: <1330026771-19937-1-git-send-email-chris@chris-wilson.co.uk> Content-Type: multipart/signed; micalg="pgp-sha1"; protocol="application/pgp-signature"; boundary="=-LNXy8NS34izb/+DfYEse" Mime-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1741 Lines: 47 --=-LNXy8NS34izb/+DfYEse Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Thu, 2012-02-23 at 12:15 -0800, Linus Torvalds wrote: > Btw, clearly X does *not* cache the EDID results, at least not for > this case. So the explicit xrandr example is probably pretty close to > what wine does. Maybe the proper fix is to just make X.org force > caching when clients do this (because it's definitely X that does the > drm_mode_getconnector() thing - xrandr itself spends zero time on > this, it just does an X request and waits for the result). RANDR exposes two requests here for a reason. We used to only have RRGetScreenResources which always pulled data afresh. We added RRGetScreenResourcesCurrent to get the cached version. If you want xrandr(1) to use the latter say xrandr --current. Arguably xrandr should do things the other way around and require you to say --reprobe or something. Even then, the kernel should cache if it can. And any competent hardware has plug interrupts on DP and DVI, so we really should get this right. - ajax --=-LNXy8NS34izb/+DfYEse Content-Type: application/pgp-signature; name="signature.asc" Content-Description: This is a digitally signed message part Content-Transfer-Encoding: 7bit -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.11 (GNU/Linux) iEYEABECAAYFAk9Hqg8ACgkQW4otUKDs0NM/XACfXXyznC7tb5O01KyqOdRk5Xul 3eYAoMiARUruSD5lCytmP8AV7FApTasL =9dGB -----END PGP SIGNATURE----- --=-LNXy8NS34izb/+DfYEse-- -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/