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=-0.8 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=no 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 AF053C3A59C for ; Thu, 15 Aug 2019 14:04:40 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 8A19020644 for ; Thu, 15 Aug 2019 14:04:40 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1732705AbfHOOEj (ORCPT ); Thu, 15 Aug 2019 10:04:39 -0400 Received: from mail-ot1-f65.google.com ([209.85.210.65]:44711 "EHLO mail-ot1-f65.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1732671AbfHOOEj (ORCPT ); Thu, 15 Aug 2019 10:04:39 -0400 Received: by mail-ot1-f65.google.com with SMTP id w4so5992131ote.11 for ; Thu, 15 Aug 2019 07:04:38 -0700 (PDT) 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; bh=/zM3G1xWX3WogoYIxIH0si8CwpIiqkYLZ2hiihxpOqQ=; b=CnYqMHQfB1q0aqvaDikCI8K3JB5IspmFdP1qmVUFGAQUkJTzWyNFmu3Z0wHXPODkUZ EFtnAT8S+m2B6FUjIboRWkoqUDxlTi64bpOTnZitFI9Lh8ZS1osi3+W5gmgTPE0dtuKk HzJKj908JmsWE1OMTm8lgbP7X25hxtsucdm5OBIfKBVjX0IQCM/5rb4PSjbTSq1M6Yq9 xwaQlE9dLLTN30jk0Aq3zFHVG/NzIK+3wMLBC069Fpu55TPy1+4Jc2g9h7rw+DrlXTae fXQxaXij6yxLL2HWNHNTDIY1c83NomIIGErlb32RdAY5Wm4dCAcEfKhJpnTYwMsxAaPV tJUQ== X-Gm-Message-State: APjAAAUgSdy5rT8l4ryvYL5+CNnEjQfGj8Mp0gUvYT61hHxIMMlxXRM3 n3oso0+XTfQ14ONyKWWHJn+tk8itzJrxwKl3eAZ0Jw== X-Google-Smtp-Source: APXvYqzchRXgP4x789+GjQl+9hOyJZGL6tLjLzEJ8LcaNO1Yx6DCAmdlcRIEhnpH8KVHbuj5x4BhAACAIvjM27sfOC8= X-Received: by 2002:a5d:8195:: with SMTP id u21mr5821290ion.260.1565877878036; Thu, 15 Aug 2019 07:04:38 -0700 (PDT) MIME-Version: 1.0 References: <20190814213118.28473-1-kherbst@redhat.com> <20190814213118.28473-2-kherbst@redhat.com> <5e05532328324d01bc554c573f6298f8@AUSX13MPC101.AMER.DELL.COM> In-Reply-To: <5e05532328324d01bc554c573f6298f8@AUSX13MPC101.AMER.DELL.COM> From: Karol Herbst Date: Thu, 15 Aug 2019 16:04:27 +0200 Message-ID: Subject: Re: [Nouveau] [PATCH 1/7] Revert "ACPI / OSI: Add OEM _OSI string to enable dGPU direct output" To: Mario.Limonciello@dell.com Cc: Dave Airlie , LKML , Linux ACPI Mailing List , dri-devel , nouveau , "Rafael J . Wysocki" , Alex Hung , Ben Skeggs , David Airlie Content-Type: text/plain; charset="UTF-8" Sender: linux-acpi-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-acpi@vger.kernel.org On Thu, Aug 15, 2019 at 3:56 PM wrote: > > > -----Original Message----- > > From: linux-acpi-owner@vger.kernel.org On > > Behalf Of Dave Airlie > > Sent: Wednesday, August 14, 2019 5:48 PM > > To: Karol Herbst > > Cc: LKML; Linux ACPI; dri-devel; nouveau; Rafael J . Wysocki; Alex Hung; Ben > > Skeggs; Dave Airlie > > Subject: Re: [Nouveau] [PATCH 1/7] Revert "ACPI / OSI: Add OEM _OSI string to > > enable dGPU direct output" > > > > On Thu, 15 Aug 2019 at 07:31, Karol Herbst wrote: > > > > > > This reverts commit 28586a51eea666d5531bcaef2f68e4abbd87242c. > > > > > > The original commit message didn't even make sense. AMD _does_ support it and > > > it works with Nouveau as well. > > > > > > Also what was the issue being solved here? No references to any bugs and not > > > even explaining any issue at all isn't the way we do things. > > > > > > And even if it means a muxed design, then the fix is to make it work inside the > > > driver, not adding some hacky workaround through ACPI tricks. > > > > > > And what out of tree drivers do or do not support we don't care one bit anyway. > > > > > > > I think the reverts should be merged via Rafael's tree as the original > > patches went in via there, and we should get them in asap. > > > > Acked-by: Dave Airlie > > Dave. > > There are definitely going to be regressions on machines in the field with the > in tree drivers by reverting this. I think we should have an answer for all of those > before this revert is accepted. > > Regarding systems with Intel+NVIDIA, we'll have to work with partners to collect > some information on the impact of reverting this. > > When this is used on a system with Intel+AMD the ASL configures AMD GPU to use > "Hybrid Graphics" when on Windows and "Power Express" and "Switchable Graphics" > when on Linux. and what's exactly the difference between those? And what's the actual issue here? We already have the PRIME offloading in place and if that's not enough, we should work on extending it, not adding some ACPI based workarounds, because that's exactly how that looks like. Also, was this discussed with anybody involved in the drm subsystem? > > I feel we need a knob and/or DMI detection to affect the changes that the ASL > normally performs. Why do we have to do that on a firmware level at all?