From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 5A2D8C433EF for ; Fri, 1 Oct 2021 22:17:23 +0000 (UTC) Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 1DCEE6136A for ; Fri, 1 Oct 2021 22:17:23 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org 1DCEE6136A Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=linux.intel.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=lists.freedesktop.org Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id A0C286E593; Fri, 1 Oct 2021 22:17:22 +0000 (UTC) Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by gabe.freedesktop.org (Postfix) with ESMTPS id B4DE16E593 for ; Fri, 1 Oct 2021 22:17:20 +0000 (UTC) X-IronPort-AV: E=McAfee;i="6200,9189,10124"; a="248167060" X-IronPort-AV: E=Sophos;i="5.85,340,1624345200"; d="scan'208";a="248167060" Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by fmsmga101.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 01 Oct 2021 15:17:08 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.85,340,1624345200"; d="scan'208";a="619441943" Received: from stinkbox.fi.intel.com (HELO stinkbox) ([10.237.72.171]) by fmsmga001.fm.intel.com with SMTP; 01 Oct 2021 15:17:06 -0700 Received: by stinkbox (sSMTP sendmail emulation); Sat, 02 Oct 2021 01:17:05 +0300 Date: Sat, 2 Oct 2021 01:17:05 +0300 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= To: Matt Roper Cc: intel-gfx@lists.freedesktop.org, stable@vger.kernel.org, Karthik B S Message-ID: References: <20210930190943.17547-1-ville.syrjala@linux.intel.com> <20211001210815.GG3389343@mdroper-desk1.amr.corp.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: X-Patchwork-Hint: comment Subject: Re: [Intel-gfx] [PATCH 1/2] drm/i915: Extend the async flip VT-d w/a to skl/bxt X-BeenThere: intel-gfx@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Intel graphics driver community testing & development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" On Sat, Oct 02, 2021 at 01:01:31AM +0300, Ville Syrjälä wrote: > On Fri, Oct 01, 2021 at 02:08:15PM -0700, Matt Roper wrote: > > On Thu, Sep 30, 2021 at 10:09:42PM +0300, Ville Syrjala wrote: > > > From: Ville Syrjälä > > > > > > Looks like skl/bxt/derivatives also need the plane stride > > > stretch w/a when using async flips and VT-d is enabled, or > > > else we get corruption on screen. To my surprise this was > > > even documented in bspec, but only as a note on the > > > CHICHKEN_PIPESL register description rather than on the > > > w/a list. > > > > > > So very much the same thing as on HSW/BDW, except the bits > > > moved yet again. > > > > Bspec 7522 doesn't say anything about this requirement being tied to > > VT-d on these platforms. Should we drop the intel_vtd_active() > > condition to be safe? > > I think it's just an oversight in bspec. I read through the hsd and > IIRC it did specify that it's VT-d only. Also real life confirms > it. No problems whatsoever when VT-d is disabled. BTW I was hopeful this would fix shard-skl but no such luck. Well, in fact it does fix the crc error, indicating the patch does work. Unfortunately those systems have yet another undiagnosed async flip problem. From the ci report on this series I can see that the machine was only capable of ~1.2 async flips per frame during the crc test. I guess technically anything >1 counts as "some async flips did happen" but it really should not be that low (I put the arbitrary limit in the test at two flips per frame). My cfl can do IIRC 50-150 per frame, depending on the phase of the moon and whatnot. -- Ville Syrjälä Intel