From patchwork Thu May 19 18:00:09 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Michael Rodin X-Patchwork-Id: 12855874 X-Patchwork-Delegate: kieran@bingham.xyz Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 2A3D4C433EF for ; Thu, 19 May 2022 18:02:06 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S243549AbiESSCF (ORCPT ); Thu, 19 May 2022 14:02:05 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:48900 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S243559AbiESSCE (ORCPT ); Thu, 19 May 2022 14:02:04 -0400 Received: from smtp1.de.adit-jv.com (smtp1.de.adit-jv.com [93.241.18.167]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 0AF0A5BE7B; Thu, 19 May 2022 11:02:02 -0700 (PDT) Received: from hi2exch02.adit-jv.com (hi2exch02.adit-jv.com [10.72.92.28]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp1.de.adit-jv.com (Postfix) with ESMTPS id BD7463C00BA; Thu, 19 May 2022 20:02:00 +0200 (CEST) Received: from vmlxhi-121.localdomain (10.72.92.132) by hi2exch02.adit-jv.com (10.72.92.28) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2308.27; Thu, 19 May 2022 20:02:00 +0200 From: Michael Rodin To: Mauro Carvalho Chehab , =?utf-8?q?Niklas_S=C3=B6derl?= =?utf-8?q?und?= , , , CC: Michael Rodin , , Subject: [PATCH 3/3] rcar-vin: handle transfer errors from subdevices and stop streaming if required Date: Thu, 19 May 2022 20:00:09 +0200 Message-ID: <1652983210-1194-4-git-send-email-mrodin@de.adit-jv.com> X-Mailer: git-send-email 2.7.4 In-Reply-To: <1652983210-1194-1-git-send-email-mrodin@de.adit-jv.com> References: <1652983210-1194-1-git-send-email-mrodin@de.adit-jv.com> MIME-Version: 1.0 X-Originating-IP: [10.72.92.132] X-ClientProxiedBy: hi2exch02.adit-jv.com (10.72.92.28) To hi2exch02.adit-jv.com (10.72.92.28) Precedence: bulk List-ID: X-Mailing-List: linux-renesas-soc@vger.kernel.org When a subdevice sends a transfer error event during streaming and we can not capture new frames, then we know for sure that this is an unrecoverable failure and not just a temporary glitch. In this case we can not ignore the transfer error any more and have to notify userspace. In response to the transfer error event userspace can try to restart streaming and hope that it works again. This patch is based on the patch [1] from Niklas Söderlund, however it adds more logic to check whether the VIN hardware module is actually affected by the transfer errors reported by the usptream device. For this it takes some ideas from the imx driver where EOF interrupts are monitored by the eof_timeout_timer added by commit 4a34ec8e470c ("[media] media: imx: Add CSI subdev driver"). [1] https://lore.kernel.org/linux-renesas-soc/20211108160220.767586-4-niklas.soderlund+renesas@ragnatech.se/ Signed-off-by: Michael Rodin --- drivers/media/platform/renesas/rcar-vin/rcar-dma.c | 34 ++++++++++++++++++++++ .../media/platform/renesas/rcar-vin/rcar-v4l2.c | 18 +++++++++++- drivers/media/platform/renesas/rcar-vin/rcar-vin.h | 7 +++++ 3 files changed, 58 insertions(+), 1 deletion(-) diff --git a/drivers/media/platform/renesas/rcar-vin/rcar-dma.c b/drivers/media/platform/renesas/rcar-vin/rcar-dma.c index 2272f1c..596a367 100644 --- a/drivers/media/platform/renesas/rcar-vin/rcar-dma.c +++ b/drivers/media/platform/renesas/rcar-vin/rcar-dma.c @@ -13,6 +13,7 @@ #include #include #include +#include #include @@ -1060,6 +1061,9 @@ static irqreturn_t rvin_irq(int irq, void *data) vin_dbg(vin, "Dropping frame %u\n", vin->sequence); } + cancel_delayed_work(&vin->frame_timeout); + schedule_delayed_work(&vin->frame_timeout, msecs_to_jiffies(FRAME_TIMEOUT_MS)); + vin->sequence++; /* Prepare for next frame */ @@ -1283,6 +1287,7 @@ int rvin_start_streaming(struct rvin_dev *vin) spin_lock_irqsave(&vin->qlock, flags); vin->sequence = 0; + vin->xfer_error = false; ret = rvin_capture_start(vin); if (ret) @@ -1290,6 +1295,10 @@ int rvin_start_streaming(struct rvin_dev *vin) spin_unlock_irqrestore(&vin->qlock, flags); + /* We start the frame watchdog only after we have successfully started streaming */ + if (!ret) + schedule_delayed_work(&vin->frame_timeout, msecs_to_jiffies(FRAME_TIMEOUT_MS)); + return ret; } @@ -1332,6 +1341,12 @@ void rvin_stop_streaming(struct rvin_dev *vin) } vin->state = STOPPING; + /* + * Since we are now stopping and don't expect more frames to be captured, make sure that + * there is no pending work for error handling. + */ + cancel_delayed_work_sync(&vin->frame_timeout); + vin->xfer_error = false; /* Wait until only scratch buffer is used, max 3 interrupts. */ retries = 0; @@ -1424,6 +1439,23 @@ void rvin_dma_unregister(struct rvin_dev *vin) v4l2_device_unregister(&vin->v4l2_dev); } +static void rvin_frame_timeout(struct work_struct *work) +{ + struct delayed_work *dwork = to_delayed_work(work); + struct rvin_dev *vin = container_of(dwork, struct rvin_dev, frame_timeout); + struct v4l2_event event = { + .type = V4L2_EVENT_XFER_ERROR, + }; + + vin_dbg(vin, "Frame timeout!\n"); + + if (!vin->xfer_error) + return; + vin_err(vin, "Unrecoverable transfer error detected, stopping streaming\n"); + vb2_queue_error(&vin->queue); + v4l2_event_queue(&vin->vdev, &event); +} + int rvin_dma_register(struct rvin_dev *vin, int irq) { struct vb2_queue *q = &vin->queue; @@ -1470,6 +1502,8 @@ int rvin_dma_register(struct rvin_dev *vin, int irq) goto error; } + INIT_DELAYED_WORK(&vin->frame_timeout, rvin_frame_timeout); + return 0; error: rvin_dma_unregister(vin); diff --git a/drivers/media/platform/renesas/rcar-vin/rcar-v4l2.c b/drivers/media/platform/renesas/rcar-vin/rcar-v4l2.c index 2e2aa9d..bd7f6fe2 100644 --- a/drivers/media/platform/renesas/rcar-vin/rcar-v4l2.c +++ b/drivers/media/platform/renesas/rcar-vin/rcar-v4l2.c @@ -648,6 +648,8 @@ static int rvin_subscribe_event(struct v4l2_fh *fh, switch (sub->type) { case V4L2_EVENT_SOURCE_CHANGE: return v4l2_event_subscribe(fh, sub, 4, NULL); + case V4L2_EVENT_XFER_ERROR: + return v4l2_event_subscribe(fh, sub, 1, NULL); } return v4l2_ctrl_subscribe_event(fh, sub); } @@ -1000,9 +1002,23 @@ void rvin_v4l2_unregister(struct rvin_dev *vin) static void rvin_notify_video_device(struct rvin_dev *vin, unsigned int notification, void *arg) { + const struct v4l2_event *event; + switch (notification) { case V4L2_DEVICE_NOTIFY_EVENT: - v4l2_event_queue(&vin->vdev, arg); + event = arg; + + switch (event->type) { + case V4L2_EVENT_XFER_ERROR: + if (vin->state != STOPPED && vin->state != STOPPING) { + vin_dbg(vin, "Subdevice signaled transfer error.\n"); + vin->xfer_error = true; + } + break; + default: + break; + } + break; default: break; diff --git a/drivers/media/platform/renesas/rcar-vin/rcar-vin.h b/drivers/media/platform/renesas/rcar-vin/rcar-vin.h index 1f94589..4726a69 100644 --- a/drivers/media/platform/renesas/rcar-vin/rcar-vin.h +++ b/drivers/media/platform/renesas/rcar-vin/rcar-vin.h @@ -31,6 +31,9 @@ /* Max number on VIN instances that can be in a system */ #define RCAR_VIN_NUM 32 +/* maximum time we wait before signalling an error to userspace */ +#define FRAME_TIMEOUT_MS 1000 + struct rvin_group; enum model_id { @@ -207,6 +210,8 @@ struct rvin_info { * @std: active video standard of the video source * * @alpha: Alpha component to fill in for supported pixel formats + * @xfer_error: Indicates if any transfer errors occurred in the current streaming session. + * @frame_timeout: Watchdog for monitoring regular capturing of frames in rvin_irq. */ struct rvin_dev { struct device *dev; @@ -251,6 +256,8 @@ struct rvin_dev { v4l2_std_id std; unsigned int alpha; + bool xfer_error; + struct delayed_work frame_timeout; }; #define vin_to_source(vin) ((vin)->parallel.subdev)