From patchwork Sat Nov 17 01:50:22 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Lyude Paul X-Patchwork-Id: 10687315 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id C3EBD109C for ; Sat, 17 Nov 2018 01:50:40 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id A60462C840 for ; Sat, 17 Nov 2018 01:50:40 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 998A02C892; Sat, 17 Nov 2018 01:50:40 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-5.2 required=2.0 tests=BAYES_00,MAILING_LIST_MULTI, RCVD_IN_DNSWL_MED autolearn=ham version=3.3.1 Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher DHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id 23A5D2C840 for ; Sat, 17 Nov 2018 01:50:39 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 3C82D6E732; Sat, 17 Nov 2018 01:50:38 +0000 (UTC) X-Original-To: dri-devel@lists.freedesktop.org Delivered-To: dri-devel@lists.freedesktop.org Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by gabe.freedesktop.org (Postfix) with ESMTPS id BB2EA6E89B; Sat, 17 Nov 2018 01:50:33 +0000 (UTC) Received: from smtp.corp.redhat.com (int-mx03.intmail.prod.int.phx2.redhat.com [10.5.11.13]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 361B258E35; Sat, 17 Nov 2018 01:50:33 +0000 (UTC) Received: from malachite.bss.redhat.com (dhcp-10-20-1-11.bss.redhat.com [10.20.1.11]) by smtp.corp.redhat.com (Postfix) with ESMTP id 4DF98614C6; Sat, 17 Nov 2018 01:50:32 +0000 (UTC) From: Lyude Paul To: nouveau@lists.freedesktop.org, dri-devel@lists.freedesktop.org Subject: [PATCH 1/2] drm/dp: Add ->pre/post_transfer() hooks for drm_dp_aux Date: Fri, 16 Nov 2018 20:50:22 -0500 Message-Id: <20181117015024.5771-2-lyude@redhat.com> In-Reply-To: <20181117015024.5771-1-lyude@redhat.com> References: <20181117015024.5771-1-lyude@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.13 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.39]); Sat, 17 Nov 2018 01:50:33 +0000 (UTC) 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: Maxime Ripard , Sean Paul , David Airlie , linux-kernel@vger.kernel.org Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" X-Virus-Scanned: ClamAV using ClamSMTP Many DRM drivers unfortunately need to be able to access the DP AUX channel during their suspend/resume callbacks. This leads to an annoying catch-22: drivers which try to ensure that the DP AUX channel is initialized and ready may need to runtime-resume the device housing the channel, which would lead to a deadlock between runtime power management and drm_dp_aux->hw_mutex. So: add a simple set of optional hooks that drivers can implement in order to perform such setup before hw_mutex is locked, then clean up afterwards. We additionally add the drm_dp_aux_get() and drm_dp_aux_put() functions so that users of the AUX channel that need to prepare the AUX channel ahead of time to avoid other kinds of locking version can do so. We'll need this if we ever want to have a universal dp_mst_status debugfs node, since dumping the MST topology without having the AUX channel prepared beforehand would lead to lock inversion. Signed-off-by: Lyude Paul --- drivers/gpu/drm/drm_dp_helper.c | 5 ++ include/drm/drm_dp_helper.h | 91 +++++++++++++++++++++++++++++++++ 2 files changed, 96 insertions(+) diff --git a/drivers/gpu/drm/drm_dp_helper.c b/drivers/gpu/drm/drm_dp_helper.c index 6d483487f2b4..fb1912a2f246 100644 --- a/drivers/gpu/drm/drm_dp_helper.c +++ b/drivers/gpu/drm/drm_dp_helper.c @@ -224,6 +224,10 @@ static int drm_dp_dpcd_access(struct drm_dp_aux *aux, u8 request, msg.buffer = buffer; msg.size = size; + ret = drm_dp_aux_get(aux); + if (ret) + return ret; + mutex_lock(&aux->hw_mutex); /* @@ -265,6 +269,7 @@ static int drm_dp_dpcd_access(struct drm_dp_aux *aux, u8 request, unlock: mutex_unlock(&aux->hw_mutex); + drm_dp_aux_put(aux); return ret; } diff --git a/include/drm/drm_dp_helper.h b/include/drm/drm_dp_helper.h index 3314e91f6eb3..b0208bc666d1 100644 --- a/include/drm/drm_dp_helper.h +++ b/include/drm/drm_dp_helper.h @@ -1243,6 +1243,46 @@ struct drm_dp_aux { struct mutex hw_mutex; struct work_struct crc_work; u8 crc_count; + /** + * @pre_transfer: + * + * An optional callback for drivers that if implemented, will be + * called before locking @hw_mutex and beginning a DP AUX transaction. + * + * Drivers can use this to perform any initialization that might be + * required before the DP AUX channel is ready to be used, such as + * waking up the device housing the AUX channel. + * + * This callback may be called more then once for a single + * transaction. + * + * See also: + * drm_dp_aux_get() + * drm_dp_aux_put() + * + * Returns: + * + * 0 on success, negative error code on failure. + */ + int (*pre_transfer)(struct drm_dp_aux *aux); + /** + * @post_transfer: + * + * An optional callback for drivers that if implemented, will be + * called after having performed a DP AUX transaction. + * + * Drivers can use this to undo any initialization that was performed + * by @pre_transfer, such as putting the device housing the DP AUX + * channel back to sleep. + * + * This callback may be called more then once for a single + * transaction. + * + * See also: + * drm_dp_aux_get() + * drm_dp_aux_put() + */ + void (*post_transfer)(struct drm_dp_aux *aux); ssize_t (*transfer)(struct drm_dp_aux *aux, struct drm_dp_aux_msg *msg); /** @@ -1259,6 +1299,57 @@ struct drm_dp_aux { struct drm_dp_aux_cec cec; }; +/** + * drm_dp_aux_get() - Prepare a DP AUX channel for a transaction + * @aux: DisplayPort AUX channel to initialize + * + * If implemented by the driver, this function will invoke the + * &drm_dp_aux.pre_transfer callback for the given @aux device. This function + * can be used to setup the DP AUX channel before going under lock, in order + * to avoid lock inversion between the DP AUX channel setup and + * &drm_dp_aux.hw_mutex. This function is implicitly called by + * drm_dp_dpcd_read(), drm_dp_dpcd_readb(), drm_dp_dpcd_write(), and + * drm_dp_dpcd_writeb(). + * + * Each call to drm_dp_aux_get() must have a matching drm_dp_aux_put() call to + * cleanup any resources that were required for the DP AUX transaction. + * + * See also: + * drm_dp_aux_put() + * + * Returns: + * 0 on success, negative error code on failure + */ +static inline int drm_dp_aux_get(struct drm_dp_aux *aux) +{ + if (aux->pre_transfer) + return aux->pre_transfer(aux); + else + return 0; +} + +/** + * drm_dp_aux_put() - Cleanup after performing a transaction on a DP AUX + * channel + * @aux: DisplayPort AUX channel to cleanup + * + * If implemented by the driver, this function will invoke the + * &drm_dp_aux.post_transfer callback for the given @aux device. This function + * is implicitly called by drm_dp_dpcd_read(), drm_dp_dpcd_readb(), + * drm_dp_dpcd_write(), and drm_dp_dpcd_writeb(). + * + * Each call to drm_dp_aux_get() must have a matching drm_dp_aux_put() call to + * cleanup any resources that were required for the DP AUX transaction. + * + * See also: + * drm_dp_aux_get() + */ +static inline void drm_dp_aux_put(struct drm_dp_aux *aux) +{ + if (aux->post_transfer) + aux->post_transfer(aux); +} + ssize_t drm_dp_dpcd_read(struct drm_dp_aux *aux, unsigned int offset, void *buffer, size_t size); ssize_t drm_dp_dpcd_write(struct drm_dp_aux *aux, unsigned int offset,