From patchwork Thu Jun 4 09:02:39 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Tomi Valkeinen X-Patchwork-Id: 6545661 Return-Path: X-Original-To: patchwork-dri-devel@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork1.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.136]) by patchwork1.web.kernel.org (Postfix) with ESMTP id 482CB9F326 for ; Thu, 4 Jun 2015 11:06:54 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id D0FAD206D6 for ; Thu, 4 Jun 2015 11:06:52 +0000 (UTC) Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) by mail.kernel.org (Postfix) with ESMTP id 2577B207B3 for ; Thu, 4 Jun 2015 11:06:51 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 17AFD6EB0A; Thu, 4 Jun 2015 04:06:37 -0700 (PDT) X-Original-To: dri-devel@lists.freedesktop.org Delivered-To: dri-devel@lists.freedesktop.org Received: from bear.ext.ti.com (bear.ext.ti.com [192.94.94.41]) by gabe.freedesktop.org (Postfix) with ESMTP id 54B9F6EAD8 for ; Thu, 4 Jun 2015 02:04:09 -0700 (PDT) Received: from dlelxv90.itg.ti.com ([172.17.2.17]) by bear.ext.ti.com (8.13.7/8.13.7) with ESMTP id t54946mB030654; Thu, 4 Jun 2015 04:04:06 -0500 Received: from DLEE70.ent.ti.com (dlee70.ent.ti.com [157.170.170.113]) by dlelxv90.itg.ti.com (8.14.3/8.13.8) with ESMTP id t54945bC011943; Thu, 4 Jun 2015 04:04:06 -0500 Received: from dlep33.itg.ti.com (157.170.170.75) by DLEE70.ent.ti.com (157.170.170.113) with Microsoft SMTP Server id 14.3.224.2; Thu, 4 Jun 2015 04:04:05 -0500 Received: from deskari.ti.com (ileax41-snat.itg.ti.com [10.172.224.153]) by dlep33.itg.ti.com (8.14.3/8.13.8) with ESMTP id t5493YwO014785; Thu, 4 Jun 2015 04:04:04 -0500 From: Tomi Valkeinen To: , Laurent Pinchart Subject: [PATCHv2 22/45] drm: omapdrm: Switch page flip to atomic helpers Date: Thu, 4 Jun 2015 12:02:39 +0300 Message-ID: <1433408582-9828-23-git-send-email-tomi.valkeinen@ti.com> X-Mailer: git-send-email 2.1.4 In-Reply-To: <1433408582-9828-1-git-send-email-tomi.valkeinen@ti.com> References: <1433408582-9828-1-git-send-email-tomi.valkeinen@ti.com> MIME-Version: 1.0 X-Mailman-Approved-At: Thu, 04 Jun 2015 04:06:34 -0700 Cc: Tomi Valkeinen X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" X-Spam-Status: No, score=-4.2 required=5.0 tests=BAYES_00, RCVD_IN_DNSWL_MED, T_RP_MATCHES_RCVD, 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 From: Laurent Pinchart The atomic page flip helper implements the page flip operation using asynchronous commits. As the legacy page flip was the last caller of omap_plane_mode_set(), remove the function. Signed-off-by: Laurent Pinchart Signed-off-by: Tomi Valkeinen --- drivers/gpu/drm/omapdrm/omap_crtc.c | 224 +++++++---------------------------- drivers/gpu/drm/omapdrm/omap_drv.h | 6 - drivers/gpu/drm/omapdrm/omap_plane.c | 23 ---- 3 files changed, 41 insertions(+), 212 deletions(-) diff --git a/drivers/gpu/drm/omapdrm/omap_crtc.c b/drivers/gpu/drm/omapdrm/omap_crtc.c index 5216fb07b534..a87ec26a883b 100644 --- a/drivers/gpu/drm/omapdrm/omap_crtc.c +++ b/drivers/gpu/drm/omapdrm/omap_crtc.c @@ -30,18 +30,10 @@ #define to_omap_crtc(x) container_of(x, struct omap_crtc, base) -enum omap_page_flip_state { - OMAP_PAGE_FLIP_IDLE, - OMAP_PAGE_FLIP_WAIT, - OMAP_PAGE_FLIP_QUEUED, - OMAP_PAGE_FLIP_CANCELLED, -}; - struct omap_crtc { struct drm_crtc base; const char *name; - int pipe; enum omap_channel channel; struct omap_overlay_manager_info info; struct drm_encoder *current_encoder; @@ -63,25 +55,9 @@ struct omap_crtc { /* list of framebuffers to unpin */ struct list_head pending_unpins; - /* - * flip_state flag indicates the current page flap state: IDLE if no - * page queue has been submitted, WAIT when waiting for GEM async - * completion, QUEUED when the page flip has been queued to the hardware - * or CANCELLED when the CRTC is turned off before the flip gets queued - * to the hardware. The flip event, if any, is stored in flip_event, and - * the framebuffer queued for page flip is stored in flip_fb. The - * flip_wait wait queue is used to wait for page flip completion. - * - * The flip_work work queue handles page flip requests without caring - * about what context the GEM async callback is called from. Possibly we - * should just make omap_gem always call the cb from the worker so we - * don't have to care about this. - */ - enum omap_page_flip_state flip_state; - struct drm_pending_vblank_event *flip_event; - struct drm_framebuffer *flip_fb; + /* pending event */ + struct drm_pending_vblank_event *event; wait_queue_head_t flip_wait; - struct work_struct flip_work; struct completion completion; @@ -284,39 +260,46 @@ static const struct dss_mgr_ops mgr_ops = { void omap_crtc_cancel_page_flip(struct drm_crtc *crtc, struct drm_file *file) { struct omap_crtc *omap_crtc = to_omap_crtc(crtc); + struct drm_pending_vblank_event *event; struct drm_device *dev = crtc->dev; unsigned long flags; + /* Destroy the pending vertical blanking event associated with the + * pending page flip, if any, and disable vertical blanking interrupts. + */ + spin_lock_irqsave(&dev->event_lock, flags); - /* Only complete events queued for our file handle. */ - if (omap_crtc->flip_event && - file == omap_crtc->flip_event->base.file_priv) { - drm_send_vblank_event(dev, omap_crtc->pipe, - omap_crtc->flip_event); - omap_crtc->flip_event = NULL; + event = omap_crtc->event; + omap_crtc->event = NULL; + + if (event && event->base.file_priv == file) { + event->base.destroy(&event->base); + drm_crtc_vblank_put(crtc); } spin_unlock_irqrestore(&dev->event_lock, flags); } -/* Must be called with dev->event_lock locked. */ -static void omap_crtc_complete_page_flip(struct drm_crtc *crtc, - enum omap_page_flip_state state) +static void omap_crtc_complete_page_flip(struct drm_crtc *crtc) { struct omap_crtc *omap_crtc = to_omap_crtc(crtc); + struct drm_pending_vblank_event *event; struct drm_device *dev = crtc->dev; + unsigned long flags; - if (omap_crtc->flip_event) { - drm_send_vblank_event(dev, omap_crtc->pipe, - omap_crtc->flip_event); - omap_crtc->flip_event = NULL; - } + spin_lock_irqsave(&dev->event_lock, flags); - omap_crtc->flip_state = state; + event = omap_crtc->event; + omap_crtc->event = NULL; - if (state == OMAP_PAGE_FLIP_IDLE) + if (event) { + drm_crtc_send_vblank_event(crtc, event); wake_up(&omap_crtc->flip_wait); + drm_crtc_vblank_put(crtc); + } + + spin_unlock_irqrestore(&dev->event_lock, flags); } static bool omap_crtc_page_flip_pending(struct drm_crtc *crtc) @@ -327,7 +310,7 @@ static bool omap_crtc_page_flip_pending(struct drm_crtc *crtc) bool pending; spin_lock_irqsave(&dev->event_lock, flags); - pending = omap_crtc->flip_state != OMAP_PAGE_FLIP_IDLE; + pending = omap_crtc->event != NULL; spin_unlock_irqrestore(&dev->event_lock, flags); return pending; @@ -336,28 +319,6 @@ static bool omap_crtc_page_flip_pending(struct drm_crtc *crtc) static void omap_crtc_wait_page_flip(struct drm_crtc *crtc) { struct omap_crtc *omap_crtc = to_omap_crtc(crtc); - struct drm_device *dev = crtc->dev; - bool cancelled = false; - unsigned long flags; - - /* - * If we're still waiting for the GEM async operation to complete just - * cancel the page flip, as we're holding the CRTC mutex preventing the - * page flip work handler from queueing the page flip. - * - * We can't release the reference to the frame buffer here as the async - * operation doesn't keep its own reference to the buffer. We'll just - * let the page flip work queue handle that. - */ - spin_lock_irqsave(&dev->event_lock, flags); - if (omap_crtc->flip_state == OMAP_PAGE_FLIP_WAIT) { - omap_crtc_complete_page_flip(crtc, OMAP_PAGE_FLIP_CANCELLED); - cancelled = true; - } - spin_unlock_irqrestore(&dev->event_lock, flags); - - if (cancelled) - return; if (wait_event_timeout(omap_crtc->flip_wait, !omap_crtc_page_flip_pending(crtc), @@ -366,9 +327,7 @@ static void omap_crtc_wait_page_flip(struct drm_crtc *crtc) dev_warn(crtc->dev->dev, "page flip timeout!\n"); - spin_lock_irqsave(&dev->event_lock, flags); - omap_crtc_complete_page_flip(crtc, OMAP_PAGE_FLIP_IDLE); - spin_unlock_irqrestore(&dev->event_lock, flags); + omap_crtc_complete_page_flip(crtc); } static void omap_crtc_error_irq(struct omap_drm_irq *irq, uint32_t irqstatus) @@ -390,7 +349,6 @@ static void omap_crtc_vblank_irq(struct omap_drm_irq *irq, uint32_t irqstatus) struct omap_crtc *omap_crtc = container_of(irq, struct omap_crtc, vblank_irq); struct drm_device *dev = omap_crtc->base.dev; - unsigned long flags; if (dispc_mgr_go_busy(omap_crtc->channel)) return; @@ -399,9 +357,7 @@ static void omap_crtc_vblank_irq(struct omap_drm_irq *irq, uint32_t irqstatus) __omap_irq_unregister(dev, &omap_crtc->vblank_irq); /* wakeup userspace */ - spin_lock_irqsave(&dev->event_lock, flags); - omap_crtc_complete_page_flip(&omap_crtc->base, OMAP_PAGE_FLIP_IDLE); - spin_unlock_irqrestore(&dev->event_lock, flags); + omap_crtc_complete_page_flip(&omap_crtc->base); complete(&omap_crtc->completion); } @@ -594,124 +550,28 @@ static void omap_crtc_mode_set_nofb(struct drm_crtc *crtc) static void omap_crtc_atomic_begin(struct drm_crtc *crtc) { - dispc_runtime_get(); -} - -static void omap_crtc_atomic_flush(struct drm_crtc *crtc) -{ - omap_crtc_flush(crtc); - - dispc_runtime_put(); -} - -static void page_flip_worker(struct work_struct *work) -{ - struct omap_crtc *omap_crtc = - container_of(work, struct omap_crtc, flip_work); - struct drm_crtc *crtc = &omap_crtc->base; - struct drm_display_mode *mode = &crtc->mode; - struct drm_device *dev = crtc->dev; - struct drm_framebuffer *fb; - struct drm_gem_object *bo; - unsigned long flags; - bool queue_flip; - - drm_modeset_lock(&crtc->mutex, NULL); - - spin_lock_irqsave(&dev->event_lock, flags); - - /* - * The page flip could have been cancelled while waiting for the GEM - * async operation to complete. Don't queue the flip in that case. - */ - if (omap_crtc->flip_state == OMAP_PAGE_FLIP_WAIT) { - omap_crtc->flip_state = OMAP_PAGE_FLIP_QUEUED; - queue_flip = true; - } else { - omap_crtc->flip_state = OMAP_PAGE_FLIP_IDLE; - queue_flip = false; - } - - fb = omap_crtc->flip_fb; - omap_crtc->flip_fb = NULL; - - spin_unlock_irqrestore(&dev->event_lock, flags); - - if (queue_flip) { - omap_plane_mode_set(crtc->primary, crtc, fb, - 0, 0, mode->hdisplay, mode->vdisplay, - crtc->x, crtc->y, mode->hdisplay, - mode->vdisplay); - omap_crtc_flush(crtc); - } - - drm_modeset_unlock(&crtc->mutex); - - bo = omap_framebuffer_bo(fb, 0); - drm_gem_object_unreference_unlocked(bo); - drm_framebuffer_unreference(fb); -} - -static void page_flip_cb(void *arg) -{ - struct drm_crtc *crtc = arg; + struct drm_pending_vblank_event *event = crtc->state->event; struct omap_crtc *omap_crtc = to_omap_crtc(crtc); - struct omap_drm_private *priv = crtc->dev->dev_private; - - /* avoid assumptions about what ctxt we are called from: */ - queue_work(priv->wq, &omap_crtc->flip_work); -} - -static int omap_crtc_page_flip(struct drm_crtc *crtc, - struct drm_framebuffer *fb, - struct drm_pending_vblank_event *event, - uint32_t page_flip_flags) -{ struct drm_device *dev = crtc->dev; - struct omap_crtc *omap_crtc = to_omap_crtc(crtc); - struct drm_plane *primary = crtc->primary; - struct drm_gem_object *bo; unsigned long flags; - DBG("%d -> %d (event=%p)", primary->fb ? primary->fb->base.id : -1, - fb->base.id, event); + dispc_runtime_get(); - spin_lock_irqsave(&dev->event_lock, flags); + if (event) { + WARN_ON(omap_crtc->event); + WARN_ON(drm_crtc_vblank_get(crtc) != 0); - if (omap_crtc->flip_state != OMAP_PAGE_FLIP_IDLE) { + spin_lock_irqsave(&dev->event_lock, flags); + omap_crtc->event = event; spin_unlock_irqrestore(&dev->event_lock, flags); - dev_err(dev->dev, "already a pending flip\n"); - return -EBUSY; } +} - /* - * Store a reference to the framebuffer queued for page flip in the CRTC - * private structure. We can't rely on crtc->primary->fb in the page - * flip worker, as a racing CRTC disable (due for instance to an - * explicit framebuffer deletion from userspace) would set that field to - * NULL before the worker gets a change to run. - */ - drm_framebuffer_reference(fb); - omap_crtc->flip_fb = fb; - omap_crtc->flip_event = event; - omap_crtc->flip_state = OMAP_PAGE_FLIP_WAIT; - - drm_atomic_set_fb_for_plane(primary->state, fb); - primary->fb = fb; - - spin_unlock_irqrestore(&dev->event_lock, flags); - - /* - * Hold a reference temporarily until the crtc is updated - * and takes the reference to the bo. This avoids it - * getting freed from under us: - */ - bo = omap_framebuffer_bo(fb, 0); - drm_gem_object_reference(bo); - - omap_gem_op_async(bo, OMAP_GEM_READ, page_flip_cb, crtc); +static void omap_crtc_atomic_flush(struct drm_crtc *crtc) +{ + omap_crtc_flush(crtc); - return 0; + dispc_runtime_put(); } static int omap_crtc_set_property(struct drm_crtc *crtc, @@ -729,7 +589,7 @@ static const struct drm_crtc_funcs omap_crtc_funcs = { .reset = drm_atomic_helper_crtc_reset, .set_config = drm_atomic_helper_set_config, .destroy = omap_crtc_destroy, - .page_flip = omap_crtc_page_flip, + .page_flip = drm_atomic_helper_page_flip, .set_property = omap_crtc_set_property, .atomic_duplicate_state = drm_atomic_helper_crtc_duplicate_state, .atomic_destroy_state = drm_atomic_helper_crtc_destroy_state, @@ -782,7 +642,6 @@ struct drm_crtc *omap_crtc_init(struct drm_device *dev, crtc = &omap_crtc->base; - INIT_WORK(&omap_crtc->flip_work, page_flip_worker); init_waitqueue_head(&omap_crtc->flip_wait); INIT_LIST_HEAD(&omap_crtc->pending_unpins); @@ -791,7 +650,6 @@ struct drm_crtc *omap_crtc_init(struct drm_device *dev, omap_crtc->channel = channel; omap_crtc->name = channel_names[channel]; - omap_crtc->pipe = id; omap_crtc->vblank_irq.irqmask = pipe2vbl(crtc); omap_crtc->vblank_irq.irq = omap_crtc_vblank_irq; diff --git a/drivers/gpu/drm/omapdrm/omap_drv.h b/drivers/gpu/drm/omapdrm/omap_drv.h index 21c3929b06e7..0231f81dfb9b 100644 --- a/drivers/gpu/drm/omapdrm/omap_drv.h +++ b/drivers/gpu/drm/omapdrm/omap_drv.h @@ -153,12 +153,6 @@ struct drm_crtc *omap_crtc_init(struct drm_device *dev, struct drm_plane *omap_plane_init(struct drm_device *dev, int id, enum drm_plane_type type); int omap_plane_set_enable(struct drm_plane *plane, bool enable); -int omap_plane_mode_set(struct drm_plane *plane, - struct drm_crtc *crtc, struct drm_framebuffer *fb, - int crtc_x, int crtc_y, - unsigned int crtc_w, unsigned int crtc_h, - unsigned int src_x, unsigned int src_y, - unsigned int src_w, unsigned int src_h); void omap_plane_install_properties(struct drm_plane *plane, struct drm_mode_object *obj); int omap_plane_set_property(struct drm_plane *plane, diff --git a/drivers/gpu/drm/omapdrm/omap_plane.c b/drivers/gpu/drm/omapdrm/omap_plane.c index fcc5d9603292..117fa37534b6 100644 --- a/drivers/gpu/drm/omapdrm/omap_plane.c +++ b/drivers/gpu/drm/omapdrm/omap_plane.c @@ -144,29 +144,6 @@ static int omap_plane_setup(struct omap_plane *omap_plane) return ret; } -int omap_plane_mode_set(struct drm_plane *plane, - struct drm_crtc *crtc, struct drm_framebuffer *fb, - int crtc_x, int crtc_y, - unsigned int crtc_w, unsigned int crtc_h, - unsigned int src_x, unsigned int src_y, - unsigned int src_w, unsigned int src_h) -{ - struct omap_plane *omap_plane = to_omap_plane(plane); - struct omap_drm_window *win = &omap_plane->win; - - win->crtc_x = crtc_x; - win->crtc_y = crtc_y; - win->crtc_w = crtc_w; - win->crtc_h = crtc_h; - - win->src_x = src_x; - win->src_y = src_y; - win->src_w = src_w; - win->src_h = src_h; - - return omap_plane_setup(omap_plane); -} - int omap_plane_set_enable(struct drm_plane *plane, bool enable) { struct omap_plane *omap_plane = to_omap_plane(plane);