From patchwork Mon Nov 11 08:35:19 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Daniel Vetter X-Patchwork-Id: 3166671 Return-Path: X-Original-To: patchwork-intel-gfx@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork2.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.19.201]) by patchwork2.web.kernel.org (Postfix) with ESMTP id AA23DC045B for ; Mon, 11 Nov 2013 08:35:29 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id 6C1B1201BB for ; Mon, 11 Nov 2013 08:35:28 +0000 (UTC) Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) by mail.kernel.org (Postfix) with ESMTP id 4773D2017E for ; Mon, 11 Nov 2013 08:35:27 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 52986FAC94; Mon, 11 Nov 2013 00:35:22 -0800 (PST) X-Original-To: intel-gfx@lists.freedesktop.org Delivered-To: intel-gfx@lists.freedesktop.org Received: from mail-ea0-f179.google.com (mail-ea0-f179.google.com [209.85.215.179]) by gabe.freedesktop.org (Postfix) with ESMTP id AFEA1FAC77 for ; Mon, 11 Nov 2013 00:34:58 -0800 (PST) Received: by mail-ea0-f179.google.com with SMTP id r15so501810ead.38 for ; Mon, 11 Nov 2013 00:34:57 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ffwll.ch; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=DT7nvDjvfvw/fVY23c5akdtPvXpQRUp39NBE5oR0inY=; b=iL7MPRtpOxKj+pnK6FIQkAhZDsbnkQ4zMEwU6SECzCa9OyGPJtSHGEOGJuw66b12BU enK/2oVsyuWFSPvzxM3VkqDIM4Az9W7COb0cFgKApERmo2mjUY4GtozCSvcWWVlymGYY nR4ePrdxcOV1TzwTR4hb0tJ4RVqDgjURancYU= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=DT7nvDjvfvw/fVY23c5akdtPvXpQRUp39NBE5oR0inY=; b=hWVeFOuoW1JPikVsLLuCa9+jtK7LpvYR6EM/fHGKJl8S7tQQfPThRezV/rojpCUQsF yGrPiY2embtWSXVQTJkBlwMwOhpS9tDMJxD4iprTNqgMhNMQNxalSKbgO7/u0MCa2/48 W795lQzwhkObKXiMM/5nr/+4+L5cYlvRfGnxLEDZ+ayXxMNdOTgoCIWVYmEx0pkuT+Uk qx5W6FazVYRDkKGklQC5DtMussxXAHUtAVjs69PGu6fJ/71S48ZRGlrKuYsWBmX5fFUe L+edEqGM3sKh9RPkl7jJk6AOLxK7W9OzbTGxD+PlcY6cKiGE6mCIE3sEgUaV2TTXtaUr fa5g== X-Gm-Message-State: ALoCoQkl1gz0kZquO20RbHRqxAH9OEoXb7Eg1DXQvh/Byyrc0x/k2fHsLfi9DZ3X+fWBse9JFB58 X-Received: by 10.14.108.9 with SMTP id p9mr34066136eeg.8.1384158897773; Mon, 11 Nov 2013 00:34:57 -0800 (PST) Received: from phenom.ffwll.local (178-83-130-250.dynamic.hispeed.ch. [178.83.130.250]) by mx.google.com with ESMTPSA id a51sm21282522eeh.8.2013.11.11.00.34.56 for (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Mon, 11 Nov 2013 00:34:57 -0800 (PST) From: Daniel Vetter To: Intel Graphics Development Date: Mon, 11 Nov 2013 09:35:19 +0100 Message-Id: <1384158919-30592-4-git-send-email-daniel.vetter@ffwll.ch> X-Mailer: git-send-email 1.8.4.rc3 In-Reply-To: <1384158919-30592-1-git-send-email-daniel.vetter@ffwll.ch> References: <1384158919-30592-1-git-send-email-daniel.vetter@ffwll.ch> Cc: Daniel Vetter , DRI Development Subject: [Intel-gfx] [PATCH 3/3] drm/i915: Deprecated UMS support X-BeenThere: intel-gfx@lists.freedesktop.org X-Mailman-Version: 2.1.13 Precedence: list List-Id: Intel graphics driver community testing & development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Sender: intel-gfx-bounces@lists.freedesktop.org Errors-To: intel-gfx-bounces@lists.freedesktop.org X-Spam-Status: No, score=-4.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, RCVD_IN_DNSWL_MED,RP_MATCHES_RCVD,T_DKIM_INVALID,UNPARSEABLE_RELAY autolearn=unavailable version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP It's been 5 years since kms support was merged and roughly 4 years since UMS support was ripped out from userspace drivers. Thus far it's not been a big burden to keep the ums paths alive, and we've made some good progress in better separating it from the kms code by sprinkling DRIVER_MODESET checks all over the place. But now that the drm demidlayering is within reach this changes. I want to make the driver loading code more robust using devres.c and other cool tricks. But that doesn't work with ums due to the shadow-attach trick. Which means we either a) need to split out a complete ums codebase like radeon has b) kill it for good. The 2nd option is obviously much less work than the first, so I think it's time to test the waters and see how many people out there still use ums. I've decided that silently failing to initialize the driver (and not e.g. failing to load the module) is the right thing. That way we should only get reports from users that actually care about some ums features (like accelerated gl or support for secondary outputs). Everyone else will just fall back to the vesa X driver. For developers there's a small info level dmesg output. The plan is to drop this Kconfig option after 3.16 (so gives us 2 full releases) and then start killing code for real 2-3 releases afterwards. That should be more than enough time for users to pipe up. Of course if anyone does we need to revisit this plan and maybe go with option a) above. Also enable the KMS support by default in Kconfig and polish the help texts a bit. Cc: Dave Airlie Signed-off-by: Daniel Vetter --- drivers/gpu/drm/i915/Kconfig | 24 +++++++++++++++++++----- 1 file changed, 19 insertions(+), 5 deletions(-) diff --git a/drivers/gpu/drm/i915/Kconfig b/drivers/gpu/drm/i915/Kconfig index b0f61679c598..b0fa4c4055ee 100644 --- a/drivers/gpu/drm/i915/Kconfig +++ b/drivers/gpu/drm/i915/Kconfig @@ -37,12 +37,11 @@ config DRM_I915 config DRM_I915_KMS bool "Enable modesetting on intel by default" depends on DRM_I915 + default y help - Choose this option if you want kernel modesetting enabled by default, - and you have a new enough userspace to support this. Running old - userspaces with this enabled will cause pain. Note that this causes - the driver to bind to PCI devices, which precludes loading things - like intelfb. + Choose this option if you want kernel modesetting enabled by default. + + If in doubt, say "Y". config DRM_I915_FBDEV bool "Enable legacy fbdev support for the modesettting intel driver" @@ -57,9 +56,12 @@ config DRM_I915_FBDEV support. Note that this support also provide the linux console support on top of the intel modesetting driver. + If in doubt, say "Y". + config DRM_I915_PRELIMINARY_HW_SUPPORT bool "Enable preliminary support for prerelease Intel hardware by default" depends on DRM_I915 + default n help Choose this option if you have prerelease Intel hardware and want the i915 driver to support it by default. You can enable such support at @@ -67,3 +69,15 @@ config DRM_I915_PRELIMINARY_HW_SUPPORT option changes the default for that module option. If in doubt, say "N". + +config DRM_I915_UMS + bool "Enable userspace modesetting on Intel hardware (DEPRECATED)" + depends on DRM_I915 + default n + help + Choose this option if you still need userspace modesetting. + + Userspace modesetting is deprecated for quite some time now, so + enable this only if you have ancient versions of the DDX drivers. + + If in doubt, say "N".