From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bryce Harrington Subject: [PATCH 8/8] xfree86: Also handle EAGAIN errors from drmSetInterfaceVersion(). Date: Fri, 15 Mar 2013 11:02:58 -0700 Message-ID: <1363370578-30443-9-git-send-email-bryce@canonical.com> References: <1363370578-30443-1-git-send-email-bryce@canonical.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from smtp.outflux.net (smtp.outflux.net [198.145.64.163]) by gabe.freedesktop.org (Postfix) with ESMTP id 1F928E6ABB for ; Fri, 15 Mar 2013 11:03:18 -0700 (PDT) In-Reply-To: <1363370578-30443-1-git-send-email-bryce@canonical.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org Errors-To: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org To: intel-gfx@lists.freedesktop.org, chris@chris-wilson.co.uk Cc: Bryce Harrington List-Id: intel-gfx@lists.freedesktop.org It has been suggested that the kernel may pass EAGAIN when the device is unavailable. This hasn't been seen in practice, and examination of the function definition in libdrm suggests EAGAIN is handled internally so would not be seen by the xserver when making this call. So, this patch is probably unneeded. But include support anyway. Signed-off-by: Bryce Harrington --- hw/xfree86/os-support/linux/lnx_platform.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/hw/xfree86/os-support/linux/lnx_platform.c b/hw/xfree86/os-support/linux/lnx_platform.c index 3386b67..b05719d 100644 --- a/hw/xfree86/os-support/linux/lnx_platform.c +++ b/hw/xfree86/os-support/linux/lnx_platform.c @@ -46,6 +46,9 @@ get_drm_info(struct OdevAttributes *attribs, char *path) } if (err == -EACCES) { if (tries % 500 == 0) LogMessage(X_INFO, "waiting on drm device...\n"); + } if (err == -EAGAIN) { + if (tries % 500 == 0) + LogMessage(X_INFO, "drm device busy...\n"); } else { break; } -- 1.7.9.5