From mboxrd@z Thu Jan 1 00:00:00 1970 From: naresh.kumar.kachhi@intel.com Subject: [PATCH v2 0/3] disable rings before sync flush Date: Wed, 12 Mar 2014 16:39:39 +0530 Message-ID: <1394622582-21762-1-git-send-email-naresh.kumar.kachhi@intel.com> References: <20140312081205.GC23307@nuc-i3427.alporthouse.com> 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 5FA56FAB93 for ; Wed, 12 Mar 2014 04:11:18 -0700 (PDT) In-Reply-To: <20140312081205.GC23307@nuc-i3427.alporthouse.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: intel-gfx-bounces@lists.freedesktop.org Errors-To: intel-gfx-bounces@lists.freedesktop.org To: intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org From: Naresh Kumar Kachhi initial version: stop rings before sync flush v1: address the comments from Chris Wilson on http://www.spinics.net/lists/intel-gfx/msg41025.html. Rather than adding stop and start ring, we are moving ring disable before hw_status_page_setup (sync flush). v2: Based on Chris's suggestion. Added one more patch to wait until the rings become idle after disabling. In case of timeout we log a message. Naresh Kumar Kachhi (3): drm/i915: disable rings before HW status page setup drm/i915: wait for rings to become idle once disabled drm/i915: warn if ring is active before sync flush drivers/gpu/drm/i915/i915_reg.h | 2 ++ drivers/gpu/drm/i915/intel_ringbuffer.c | 18 +++++++++++++----- drivers/gpu/drm/i915/intel_ringbuffer.h | 2 ++ 3 files changed, 17 insertions(+), 5 deletions(-) -- 1.8.5.3