From patchwork Tue Oct 22 15:25:29 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Daniel Vetter X-Patchwork-Id: 11204685 Return-Path: Received: from mail.kernel.org (pdx-korg-mail-1.web.codeaurora.org [172.30.200.123]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id 58E1514E5 for ; Tue, 22 Oct 2019 15:25:41 +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 EC52E218AE for ; Tue, 22 Oct 2019 15:25:40 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org EC52E218AE 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 2D69B6E84A; Tue, 22 Oct 2019 15:25:40 +0000 (UTC) X-Original-To: dri-devel@lists.freedesktop.org Delivered-To: dri-devel@lists.freedesktop.org Received: from mail-wr1-x42d.google.com (mail-wr1-x42d.google.com [IPv6:2a00:1450:4864:20::42d]) by gabe.freedesktop.org (Postfix) with ESMTPS id E06FE6E84A for ; Tue, 22 Oct 2019 15:25:37 +0000 (UTC) Received: by mail-wr1-x42d.google.com with SMTP id s1so9807790wro.0 for ; Tue, 22 Oct 2019 08:25:37 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=+LH15woUHFV9Sv24z7gvZy7aG33Jvdl2MxKUB928QW4=; b=WjM5KIjOf1ID4+ITA4FdgjShMamn1gQehgcbyAZ62pMKqfG/XQeRR0ANsrasXvV6je Ta9ClkcuXR7cQlUOAO6uGAil8nMjlLd3iT5dEjgrex6xWxySb8eE8iMMte4LyOJD5Wsn ZWFz0mg0/2jBV74Xiq8tW/FWBt3RHsRPgzfrTyZvhoIgcpcv2oivBbETNIrXxi94UrEk OsQ0spfkx3s8dXRAdlnoR2Cf3+XcxfzjLnYoKf/r+J2H+HXLVE2/RjoUSJCh6CcG0Q4l o35LpHEMCD5HjTP9Nd8RrhhNWsdI6Wu1VhtJLrux84Jbm79WDLeE4rDNt8aRcqYIvtOk GlkA== X-Gm-Message-State: APjAAAUhONPKuy/Fm58j8RQPA6mS6SNUyrAPhPcupLSooS/VF1pPlvr2 IEuxcZeyypSzYNwzDuyr5Y3bdGhUPBs= X-Google-Smtp-Source: APXvYqx0WaHe3SLtBQfi5JGfGc06QNVYCwzB1rRY6lHBffbLS0ulrWWgcvHF9TKFPWICJo/POFsTzg== X-Received: by 2002:a5d:4285:: with SMTP id k5mr3956815wrq.344.1571757936201; Tue, 22 Oct 2019 08:25:36 -0700 (PDT) Received: from phenom.ffwll.local (212-51-149-96.fiber7.init7.net. [212.51.149.96]) by smtp.gmail.com with ESMTPSA id b5sm16574513wmj.18.2019.10.22.08.25.35 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 22 Oct 2019 08:25:35 -0700 (PDT) From: Daniel Vetter To: DRI Development Subject: [PATCH 1/2] drm/todo: Remove i915 device_link task Date: Tue, 22 Oct 2019 17:25:29 +0200 Message-Id: <20191022152530.22038-1-daniel.vetter@ffwll.ch> X-Mailer: git-send-email 2.23.0 MIME-Version: 1.0 X-Mailman-Original-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ffwll.ch; s=google; h=from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=+LH15woUHFV9Sv24z7gvZy7aG33Jvdl2MxKUB928QW4=; b=CdKUaCKZEWKtU7OjV7AG4Ag8uzoD9u4BoQ9eCzmXgxSmOtOJi/wc24N6Cf/JrDhuNl MozXKdfrXw81vEXnP1rg3FOi8zuT/Ds9BuPMcTQdOF0TT8184tsc+/BLhIzAJMGFlCKe ztlzzvC6tx0oDpOT06cSzs83r8N59MxOJ0dIc= X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Daniel Vetter Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" Done with commit aef9f33b7658a7489f71df5d6e6ecb47f2521e8a Author: Imre Deak Date: Tue Oct 23 17:43:10 2018 +0300 drm/i915: Ensure proper HDA suspend/resume ordering with a device link Cc: Imre Deak Signed-off-by: Daniel Vetter Reviewed-by: Sean Paul --- Documentation/gpu/todo.rst | 7 ------- 1 file changed, 7 deletions(-) diff --git a/Documentation/gpu/todo.rst b/Documentation/gpu/todo.rst index 23b3a67794ba..9ac102922712 100644 --- a/Documentation/gpu/todo.rst +++ b/Documentation/gpu/todo.rst @@ -438,13 +438,6 @@ See drivers/gpu/drm/amd/display/TODO for tasks. Contact: Harry Wentland, Alex Deucher -i915 ----- - -- Our early/late pm callbacks could be removed in favour of using - device_link_add to model the dependency between i915 and snd_had. See - https://dri.freedesktop.org/docs/drm/driver-api/device_link.html - Bootsplash ========== From patchwork Tue Oct 22 15:25:30 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Daniel Vetter X-Patchwork-Id: 11204687 Return-Path: Received: from mail.kernel.org (pdx-korg-mail-1.web.codeaurora.org [172.30.200.123]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id 9796F14E5 for ; Tue, 22 Oct 2019 15:25:43 +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 8002D2086D for ; Tue, 22 Oct 2019 15:25:43 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 8002D2086D 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 A2F336E84B; Tue, 22 Oct 2019 15:25:40 +0000 (UTC) X-Original-To: dri-devel@lists.freedesktop.org Delivered-To: dri-devel@lists.freedesktop.org Received: from mail-wm1-x341.google.com (mail-wm1-x341.google.com [IPv6:2a00:1450:4864:20::341]) by gabe.freedesktop.org (Postfix) with ESMTPS id 1CBB86E84A for ; Tue, 22 Oct 2019 15:25:39 +0000 (UTC) Received: by mail-wm1-x341.google.com with SMTP id r141so6988566wme.4 for ; Tue, 22 Oct 2019 08:25:39 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=R3wH4TJuoGPpaanDV8gVPsIn/nGO9dFs+nnjPBjU+HI=; b=n4+n8RemZnlLameK92DGBfnpq6JOc29iDj2HS+T5c7kvd2sjI5UNdCwECliUxNjB54 l/EneNplFhU9NkdEEYpWR9LSBpkVpWPWILiAjkncB43ecx8fcVAegWkkPnHtgoUykkfd tsI3OCd1ZIbpvXzGqDsID37/xS8blaqNAooo32G2iZPvOPzllN0fpvdV4bfd5h/qMJV3 V0yTku70p52E5AvxGesZjDKeWrm6oVaINhADQVAvmUfG00M3/I1F9JAxlpq+RmcTOd+K jCwL0Oc5ABKo1kOq8KndeYZeSW0a0iNLqeHqbCK3nXnR1r3iCt9Bmn1b66Yj4+FZOYIJ nd/Q== X-Gm-Message-State: APjAAAU6fw1VI+2ubZY5rqa5mb/zKp5P+y5K5ly6HP8WaHd+s4+F20KV Vr7JFxAM+9Qb5q16d018QWhQbLTikso= X-Google-Smtp-Source: APXvYqxq8v9N1Og7uR3R4OKKVZ/rCL5ct3GGlQ1TB/CeJuH2lq6cql2CiBx9xh1geaJthLey4b4hvA== X-Received: by 2002:a1c:5946:: with SMTP id n67mr3696663wmb.93.1571757937374; Tue, 22 Oct 2019 08:25:37 -0700 (PDT) Received: from phenom.ffwll.local (212-51-149-96.fiber7.init7.net. [212.51.149.96]) by smtp.gmail.com with ESMTPSA id b5sm16574513wmj.18.2019.10.22.08.25.36 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 22 Oct 2019 08:25:36 -0700 (PDT) From: Daniel Vetter To: DRI Development Subject: [PATCH 2/2] drm/todo: Add levels Date: Tue, 22 Oct 2019 17:25:30 +0200 Message-Id: <20191022152530.22038-2-daniel.vetter@ffwll.ch> X-Mailer: git-send-email 2.23.0 In-Reply-To: <20191022152530.22038-1-daniel.vetter@ffwll.ch> References: <20191022152530.22038-1-daniel.vetter@ffwll.ch> MIME-Version: 1.0 X-Mailman-Original-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 :mime-version:content-transfer-encoding; bh=R3wH4TJuoGPpaanDV8gVPsIn/nGO9dFs+nnjPBjU+HI=; b=KGFyAIomlcs2601n6lm1/SEAg6fs+4KYm5rn75oyDkp91Cw5NXp1cPaHcH0ICOQOdP Mxk1ul4UYT7SnTVgT+0qjIdm/GvNn3H5mSDxJ/oUhvsCSTaAjxMyUaFV2JcRUUYUHtXq xXHaG0wQqT8LqK9oqx+t6IrVXLCS9s2Z/djpU= X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Manasi Navare , Daniel Vetter , Sean Paul , Rodrigo Siqueira , Daniel Vetter Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" Should help new people pick suitable tasks. Cc: Rodrigo Siqueira Cc: Manasi Navare Cc: Sean Paul Signed-off-by: Daniel Vetter Reviewed-by: Sean Paul --- Documentation/gpu/todo.rst | 73 ++++++++++++++++++++++++++++++++++++++ 1 file changed, 73 insertions(+) diff --git a/Documentation/gpu/todo.rst b/Documentation/gpu/todo.rst index 9ac102922712..73c51b5a0997 100644 --- a/Documentation/gpu/todo.rst +++ b/Documentation/gpu/todo.rst @@ -7,6 +7,22 @@ TODO list This section contains a list of smaller janitorial tasks in the kernel DRM graphics subsystem useful as newbie projects. Or for slow rainy days. +Difficulty +---------- + +To make it easier task are categorized into different levels: + +Starter: Good tasks to get started with the DRM subsystem. + +Intermediate: Tasks which need some experience with working in the DRM +subsystem, or some specific GPU/display graphics knowledge. For debugging issue +it's good to have the relevant hardware (or a virtual driver set up) available +for testing. + +Advanced: Tricky tasks that need fairly good understanding of the DRM subsystem +and graphics topics. Generally need the relevant hardware for development and +testing. + Subsystem-wide refactorings =========================== @@ -20,6 +36,8 @@ implementations), and then remove it. Contact: Daniel Vetter, respective driver maintainers +Level: Intermediate + Convert existing KMS drivers to atomic modesetting -------------------------------------------------- @@ -38,6 +56,8 @@ do by directly using the new atomic helper driver callbacks. Contact: Daniel Vetter, respective driver maintainers +Level: Advanced + Clean up the clipped coordination confusion around planes --------------------------------------------------------- @@ -50,6 +70,8 @@ helpers. Contact: Ville Syrjälä, Daniel Vetter, driver maintainers +Level: Advanced + Convert early atomic drivers to async commit helpers ---------------------------------------------------- @@ -63,6 +85,8 @@ events for atomic commits correctly. But fixing these bugs is good anyway. Contact: Daniel Vetter, respective driver maintainers +Level: Advanced + Fallout from atomic KMS ----------------------- @@ -91,6 +115,8 @@ interfaces to fix these issues: Contact: Daniel Vetter +Level: Intermediate + Get rid of dev->struct_mutex from GEM drivers --------------------------------------------- @@ -114,6 +140,8 @@ fine-grained per-buffer object and per-context lockings scheme. Currently only t Contact: Daniel Vetter, respective driver maintainers +Level: Advanced + Convert instances of dev_info/dev_err/dev_warn to their DRM_DEV_* equivalent ---------------------------------------------------------------------------- @@ -129,6 +157,8 @@ are better. Contact: Sean Paul, Maintainer of the driver you plan to convert +Level: Starter + Convert drivers to use simple modeset suspend/resume ---------------------------------------------------- @@ -139,6 +169,8 @@ of the atomic suspend/resume code in older atomic modeset drivers. Contact: Maintainer of the driver you plan to convert +Level: Intermediate + Convert drivers to use drm_fb_helper_fbdev_setup/teardown() ----------------------------------------------------------- @@ -157,6 +189,8 @@ probably use drm_fb_helper_fbdev_teardown(). Contact: Maintainer of the driver you plan to convert +Level: Intermediate + Clean up mmap forwarding ------------------------ @@ -166,6 +200,8 @@ There's drm_gem_prime_mmap() for this now, but still needs to be rolled out. Contact: Daniel Vetter +Level: Intermediate + Generic fbdev defio support --------------------------- @@ -196,6 +232,8 @@ Might be good to also have some igt testcases for this. Contact: Daniel Vetter, Noralf Tronnes +Level: Advanced + idr_init_base() --------------- @@ -206,6 +244,8 @@ efficient. Contact: Daniel Vetter +Level: Starter + struct drm_gem_object_funcs --------------------------- @@ -216,6 +256,8 @@ We also need a 2nd version of the CMA define that doesn't require the vmapping to be present (different hook for prime importing). Plus this needs to be rolled out to all drivers using their own implementations, too. +Level: Intermediate + Use DRM_MODESET_LOCK_ALL_* helpers instead of boilerplate --------------------------------------------------------- @@ -231,6 +273,8 @@ As a reference, take a look at the conversions already completed in drm core. Contact: Sean Paul, respective driver maintainers +Level: Starter + Rename CMA helpers to DMA helpers --------------------------------- @@ -241,6 +285,9 @@ no one knows what that means) since underneath they just use dma_alloc_coherent. Contact: Laurent Pinchart, Daniel Vetter +Level: Intermediate (mostly because it is a huge tasks without good partial +milestones, not technically itself that challenging) + Convert direct mode.vrefresh accesses to use drm_mode_vrefresh() ---------------------------------------------------------------- @@ -259,6 +306,8 @@ drm_display_mode to avoid future use. Contact: Sean Paul +Level: Starter + Remove drm_display_mode.hsync ----------------------------- @@ -269,6 +318,8 @@ it to use drm_mode_hsync() instead. Contact: Sean Paul +Level: Starter + drm_fb_helper tasks ------------------- @@ -284,6 +335,8 @@ drm_fb_helper tasks removed: drm_fb_helper_single_add_all_connectors(), drm_fb_helper_add_one_connector() and drm_fb_helper_remove_one_connector(). +Level: Intermediate + connector register/unregister fixes ----------------------------------- @@ -296,6 +349,8 @@ connector register/unregister fixes drm_dp_aux_init, and moving the actual registering into a late_register callback as recommended in the kerneldoc. +Level: Intermediate + Core refactorings ================= @@ -338,6 +393,8 @@ This is a really varied tasks with lots of little bits and pieces: Contact: Daniel Vetter +Level: Advanced + Clean up the debugfs support ---------------------------- @@ -367,6 +424,8 @@ There's a bunch of issues with it: Contact: Daniel Vetter +Level: Intermediate + KMS cleanups ------------ @@ -382,6 +441,8 @@ Some of these date from the very introduction of KMS in 2008 ... end, for which we could add drm_*_cleanup_kfree(). And then there's the (for historical reasons) misnamed drm_primary_helper_destroy() function. +Level: Intermediate + Better Testing ============== @@ -390,6 +451,8 @@ Enable trinity for DRM And fix up the fallout. Should be really interesting ... +Level: Advanced + Make KMS tests in i-g-t generic ------------------------------- @@ -403,6 +466,8 @@ converting things over. For modeset tests we also first need a bit of infrastructure to use dumb buffers for untiled buffers, to be able to run all the non-i915 specific modeset tests. +Level: Advanced + Extend virtual test driver (VKMS) --------------------------------- @@ -412,6 +477,8 @@ fit the available time. Contact: Daniel Vetter +Level: See details + Backlight Refactoring --------------------- @@ -425,6 +492,8 @@ Plan to fix this: Contact: Daniel Vetter +Level: Intermediate + Driver Specific =============== @@ -453,6 +522,8 @@ for fbdev. Contact: Sam Ravnborg +Level: Advanced + Outside DRM =========== @@ -482,3 +553,5 @@ and Weston. - [2] https://gitlab.freedesktop.org/tzimmermann/linux/blob/fbconv/drivers/gpu/drm/drm_fbconv_helper.c Contact: Thomas Zimmermann + +Level: Advanced