From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756833AbcGZOug (ORCPT ); Tue, 26 Jul 2016 10:50:36 -0400 Received: from mx1.redhat.com ([209.132.183.28]:33941 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753946AbcGZOuf (ORCPT ); Tue, 26 Jul 2016 10:50:35 -0400 From: Lyude To: intel-gfx@lists.freedesktop.org, Maarten Lankhorst Cc: Lyude , Daniel Vetter , Jani Nikula , David Airlie , dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org Subject: [PATCH v3 0/6] Finally fix watermarks Date: Tue, 26 Jul 2016 10:50:11 -0400 Message-Id: <1469544617-20003-1-git-send-email-cpaul@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.30]); Tue, 26 Jul 2016 14:50:34 +0000 (UTC) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org So unfortunately, this patch series fixes most of the underruns on Skylake, but not all of them. Even with this patchset we're still apt to potentially hitting underruns since we don't update the ddb allocations atomically as well yet. I'm planning to do this eventually when I get the chance, but for now this patch series will improve the underrun situation on Skylake dramatically. Since the last patch series, I've brought back the patch "Always wait for pipes to update after a flush". While this patch doesn't entirely fix things either, it lessens the chance of hitting underruns when enabling/disabling pipes. Since we only flush when the ddb allocations change now, another vblank wait in there shouldn't be a huge deal. As well I've included the SAGV patch this time, since it's required for this patch series to work properly anyhow and it's been updated. Figured it'd be better to put everything in one place. Lyude (5): drm/i915/skl: Add support for the SAGV, fix underrun hangs drm/i915/skl: Only flush pipes when we change the ddb allocation drm/i915/skl: Fix extra whitespace in skl_flush_wm_values() drm/i915/skl: Update plane watermarks atomically during plane updates drm/i915/skl: Always wait for pipes to update after a flush Matt Roper (1): drm/i915/gen9: Only copy WM results for changed pipes to skl_hw drivers/gpu/drm/i915/i915_drv.h | 3 + drivers/gpu/drm/i915/i915_reg.h | 5 + drivers/gpu/drm/i915/intel_display.c | 5 + drivers/gpu/drm/i915/intel_drv.h | 2 + drivers/gpu/drm/i915/intel_pm.c | 245 +++++++++++++++++++++++++++++++---- drivers/gpu/drm/i915/intel_sprite.c | 2 + 6 files changed, 239 insertions(+), 23 deletions(-) -- 2.7.4