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 X-Spam-Level: X-Spam-Status: No, score=-13.6 required=3.0 tests=BAYES_00,DKIM_INVALID, DKIM_SIGNED,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, MAILING_LIST_MULTI,MENTIONS_GIT_HOSTING,SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 1B4B0C47092 for ; Tue, 1 Jun 2021 07:28:37 +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 9162D6139A for ; Tue, 1 Jun 2021 07:28:36 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 9162D6139A Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=ffwll.ch Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=dri-devel-bounces@lists.freedesktop.org Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id C17E76E841; Tue, 1 Jun 2021 07:28:34 +0000 (UTC) Received: from mail-oi1-x22b.google.com (mail-oi1-x22b.google.com [IPv6:2607:f8b0:4864:20::22b]) by gabe.freedesktop.org (Postfix) with ESMTPS id 1470F6E243 for ; Tue, 1 Jun 2021 07:28:33 +0000 (UTC) Received: by mail-oi1-x22b.google.com with SMTP id v22so14619019oic.2 for ; Tue, 01 Jun 2021 00:28:32 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ffwll.ch; s=google; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc:content-transfer-encoding; bh=7YWxZ4LYPadaEaLleuE6m94kneRRJG7Efk68kZp5Wpg=; b=JOavtG1PeB2XpJQM7EqfiqZBnFczKs7ZMLjCzGuXWiP4G73KYOxA5pnuBuBHxMRzk2 eMkGgPvdwbwCK2oywXXElt87UMKnXRclo9pQRykfSUpqTzU5AjfnZ2xb0sZ5ieeisnWm MaNLy+Jjqd67rbHTh2yHMMvWDXBN/AAsLoaF4= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc:content-transfer-encoding; bh=7YWxZ4LYPadaEaLleuE6m94kneRRJG7Efk68kZp5Wpg=; b=YJs7J+v+DCysg3Ypju77M4NO3TY1RV84EFXMBFe9eUd9Fr3nlIQEoJTpUf9esFvb4Z gB4xfp5nvx4JHFcmhWSq+T2tFbzGrrt5jurNNCDh3OkDJ0K9i4kWVgVfWTIp6AgOKTtC rGX7Tmfi9qgXkzl0En5h3Yn45ey7PLVH5ZKzwkxFCciU+bXWbFhOvLyXT9iEKc3b/z25 3EWJJ4FtLdXgaVlHTG5Jel9kwJIY8NkpamMuzPiNSWOFpyoacxuS+PlBDnQTLICFNbyq XEV4PSceP5dHRnNGmLdFiLFMVskolIiVnXrca0m4M7mB8Ke/l7EFPuPAi6W4RJ4ePMQx 0B3w== X-Gm-Message-State: AOAM530Z4RrNMZ46aWCmFs+4KEGeliWPUXJdrEIn5LSni0B8oPKciFUX ZeizpakEhUGhGmgQrKhwYD7VnbbYnJQ/MwpERhCCeQ== X-Google-Smtp-Source: ABdhPJxnqsP3l7lVMLKZMPPIb17LjqzUQ/8nUsAihHsoC77yCwzqB1NVHuNnS1wDlUEVeIkUMyUMZd+YQhkN8HUjqtM= X-Received: by 2002:a05:6808:1142:: with SMTP id u2mr2072784oiu.101.1622532512233; Tue, 01 Jun 2021 00:28:32 -0700 (PDT) MIME-Version: 1.0 References: <20210511083139.54002-1-zbigniew.kempczynski@intel.com> In-Reply-To: From: Daniel Vetter Date: Tue, 1 Jun 2021 09:28:21 +0200 Message-ID: Subject: Re: [Intel-gfx] [PATCH] drm/i915: Add relocation exceptions for two other platforms To: Dave Airlie Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Dave Airlie , =?UTF-8?Q?Zbigniew_Kempczy=C5=84ski?= , Intel Graphics Development , dri-devel , Daniel Vetter Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" On Tue, Jun 1, 2021 at 9:19 AM Dave Airlie wrote: > On Thu, 27 May 2021 at 20:04, Daniel Vetter wrote: > > On Wed, May 26, 2021 at 10:35:49AM +1000, Dave Airlie wrote: > > > On Wed, 12 May 2021 at 03:05, Daniel Vetter wrote: > > > > On Tue, May 11, 2021 at 10:31:39AM +0200, Zbigniew Kempczy=C5=84ski= wrote: > > > > > We have established previously we stop using relocations starting > > > > > from gen12 platforms with Tigerlake as an exception. Unfortunatel= y > > > > > we need extend transition period and support relocations for two > > > > > other igfx platforms - Rocketlake and Alderlake. > > > > > > > > > > Signed-off-by: Zbigniew Kempczy=C5=84ski > > > > > Cc: Dave Airlie > > > > > Cc: Daniel Vetter > > > > > Cc: Jason Ekstrand > > > > > > > > So the annoying thing here is that now media-driver is fixed: > > > > > > > > https://github.com/intel/media-driver/commit/144020c37770083974bedf= 59902b70b8f444c799 > > > > > > > > Which means igt is really the only thing left. > > > > > > > > Dave, is this still ok for an acked exception, or is this now leani= ng > > > > towards "just fix igt"? > > > > > > Oh that isn't great is it, I had thought it was the media-driver, > > > keeping a big uAPI like this open just for the test code seems a bit > > > silly. I get the tests are testing more than just relocs, but it's a > > > pretty big interface to leave lying around if we can avoid it. > > > > So since we need to do the work anyway for DG1+ what about some interim > > hack? Currently ADL still has the require_force_probe flag set. We coul= d > > re-enable relocations just for ADL, only while this flag is set. This > > gives us a bunch of wiggle room, unblocks everything else (CI is a bit = on > > fire right now due to this and practially unuseable on ADL gem areay) a= nd > > we'll still make sure that when ADL is all done we wont expose relocati= ons > > just for igt. > > > > Thoughts? > > I can handle that as a compromise, though I'd not want to block ADL > getting out of alpha over some IGT that needs porting. Uh given that EHL/JSL are still in require_force_probe =3D true mode because of a random igt (and shipping since a while) ... I guess worst case if it's not sorted in a few months latest we can undo that again. -Daniel --=20 Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch