From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chris Wilson Subject: Re: force wake reference counting (another try) Date: Tue, 12 Apr 2011 09:02:21 +0100 Message-ID: <1bdc18$k6h5tu@fmsmga002.fm.intel.com> References: <1302570079-17032-1-git-send-email-ben@bwidawsk.net> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by gabe.freedesktop.org (Postfix) with ESMTP id 308139E7CE for ; Tue, 12 Apr 2011 01:02:24 -0700 (PDT) In-Reply-To: <1302570079-17032-1-git-send-email-ben@bwidawsk.net> 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: Ben Widawsky , intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org On Mon, 11 Apr 2011 18:01:15 -0700, Ben Widawsky wrote: > So once again, I expect this patch to potentially generate a lot of > warnings, but I consider all of those warnings to be serious bugs for > SNB. > > If anyone has clever ideas on how to handle this outside of what I've > already mentioned, please let me know. > > I expect ongoing patches to fix these issues as they come up. Continuing the general discussion from IRC, we really do need to clarify which locks we expect to be holding when reading different sets of registers. Along with similar documentation over which parts of our structures are covered by struct_mutex, mode_config.lock and the ensemble of spinlocks. This task is not limited to just our driver, but we need to review the core as well as looking at how to improve the locking overall. (The clear example that something sucks is that probing outputs causes a rendering stall (fortunately less noticeable on recent hardware where hotplug is prevalent and the polling itself is much quicker, but still present.) The only concern I have with the extra warnings are if we leave false positives in the code we submit upstream. Those warnings will quickly become regression reports and angry users. Alas, no clever ideas. -Chris -- Chris Wilson, Intel Open Source Technology Centre