From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756615AbaGWHbx (ORCPT ); Wed, 23 Jul 2014 03:31:53 -0400 Received: from mail-ig0-f180.google.com ([209.85.213.180]:37684 "EHLO mail-ig0-f180.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751432AbaGWHbw convert rfc822-to-8bit (ORCPT ); Wed, 23 Jul 2014 03:31:52 -0400 MIME-Version: 1.0 X-Originating-IP: [84.73.67.144] In-Reply-To: <53CF63C2.7070407@vodafone.de> References: <20140709093124.11354.3774.stgit@patser> <20140709122953.11354.46381.stgit@patser> <53CE2421.5040906@amd.com> <20140722114607.GL15237@phenom.ffwll.local> <20140722115737.GN15237@phenom.ffwll.local> <53CE56ED.4040109@vodafone.de> <20140722132652.GO15237@phenom.ffwll.local> <53CE6AFA.1060807@vodafone.de> <53CE84AA.9030703@amd.com> <53CE8A57.2000803@vodafone.de> <53CF58FB.8070609@canonical.com> <53CF5B9F.1050800@amd.com> <53CF5EFE.6070307@canonical.com> <53CF63C2.7070407@vodafone.de> Date: Wed, 23 Jul 2014 09:31:51 +0200 Message-ID: Subject: Re: [Nouveau] [PATCH 09/17] drm/radeon: use common fence implementation for fences From: Daniel Vetter To: =?UTF-8?Q?Christian_K=C3=B6nig?= Cc: Maarten Lankhorst , =?UTF-8?Q?Christian_K=C3=B6nig?= , Thomas Hellstrom , nouveau , LKML , dri-devel , Ben Skeggs , "Deucher, Alexander" Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Jul 23, 2014 at 9:26 AM, Christian König wrote: > It's not a locking problem I'm talking about here. Radeons lockup handling > kicks in when anything calls into the driver from the outside, if you have a > fence wait function that's called from the outside but doesn't handle > lockups you essentially rely on somebody else calling another radeon > function for the lockup to be resolved. So you don't have a timer in radeon that periodically checks whether progress is still being made? That's the approach we're using in i915, together with some tricks to kick any stuck waiters so that we can reliably step in and grab locks for the reset. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [Nouveau] [PATCH 09/17] drm/radeon: use common fence implementation for fences Date: Wed, 23 Jul 2014 09:31:51 +0200 Message-ID: References: <20140709093124.11354.3774.stgit@patser> <20140709122953.11354.46381.stgit@patser> <53CE2421.5040906@amd.com> <20140722114607.GL15237@phenom.ffwll.local> <20140722115737.GN15237@phenom.ffwll.local> <53CE56ED.4040109@vodafone.de> <20140722132652.GO15237@phenom.ffwll.local> <53CE6AFA.1060807@vodafone.de> <53CE84AA.9030703@amd.com> <53CE8A57.2000803@vodafone.de> <53CF58FB.8070609@canonical.com> <53CF5B9F.1050800@amd.com> <53CF5EFE.6070307@canonical.com> <53CF63C2.7070407@vodafone.de> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <53CF63C2.7070407@vodafone.de> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: =?UTF-8?Q?Christian_K=C3=B6nig?= Cc: Thomas Hellstrom , nouveau , LKML , dri-devel , "Deucher, Alexander" , Ben Skeggs , =?UTF-8?Q?Christian_K=C3=B6nig?= List-Id: nouveau.vger.kernel.org T24gV2VkLCBKdWwgMjMsIDIwMTQgYXQgOToyNiBBTSwgQ2hyaXN0aWFuIEvDtm5pZwo8ZGVhdGhz aW1wbGVAdm9kYWZvbmUuZGU+IHdyb3RlOgo+IEl0J3Mgbm90IGEgbG9ja2luZyBwcm9ibGVtIEkn bSB0YWxraW5nIGFib3V0IGhlcmUuIFJhZGVvbnMgbG9ja3VwIGhhbmRsaW5nCj4ga2lja3MgaW4g d2hlbiBhbnl0aGluZyBjYWxscyBpbnRvIHRoZSBkcml2ZXIgZnJvbSB0aGUgb3V0c2lkZSwgaWYg eW91IGhhdmUgYQo+IGZlbmNlIHdhaXQgZnVuY3Rpb24gdGhhdCdzIGNhbGxlZCBmcm9tIHRoZSBv dXRzaWRlIGJ1dCBkb2Vzbid0IGhhbmRsZQo+IGxvY2t1cHMgeW91IGVzc2VudGlhbGx5IHJlbHkg b24gc29tZWJvZHkgZWxzZSBjYWxsaW5nIGFub3RoZXIgcmFkZW9uCj4gZnVuY3Rpb24gZm9yIHRo ZSBsb2NrdXAgdG8gYmUgcmVzb2x2ZWQuCgpTbyB5b3UgZG9uJ3QgaGF2ZSBhIHRpbWVyIGluIHJh ZGVvbiB0aGF0IHBlcmlvZGljYWxseSBjaGVja3Mgd2hldGhlcgpwcm9ncmVzcyBpcyBzdGlsbCBi ZWluZyBtYWRlPyBUaGF0J3MgdGhlIGFwcHJvYWNoIHdlJ3JlIHVzaW5nIGluIGk5MTUsCnRvZ2V0 aGVyIHdpdGggc29tZSB0cmlja3MgdG8ga2ljayBhbnkgc3R1Y2sgd2FpdGVycyBzbyB0aGF0IHdl IGNhbgpyZWxpYWJseSBzdGVwIGluIGFuZCBncmFiIGxvY2tzIGZvciB0aGUgcmVzZXQuCi1EYW5p ZWwKLS0gCkRhbmllbCBWZXR0ZXIKU29mdHdhcmUgRW5naW5lZXIsIEludGVsIENvcnBvcmF0aW9u Cis0MSAoMCkgNzkgMzY1IDU3IDQ4IC0gaHR0cDovL2Jsb2cuZmZ3bGwuY2gKX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRldmVsIG1haWxpbmcgbGlz dApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHA6Ly9saXN0cy5mcmVlZGVza3Rv cC5vcmcvbWFpbG1hbi9saXN0aW5mby9kcmktZGV2ZWwK