From patchwork Mon Jan 29 19:59:54 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Niklas_S=C3=B6derlund?= X-Patchwork-Id: 13536331 Received: from mail-lj1-f179.google.com (mail-lj1-f179.google.com [209.85.208.179]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 8B07A60EFF for ; Mon, 29 Jan 2024 20:01:26 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.208.179 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1706558490; cv=none; b=D/dwEvTnnTCQVkTIbhYHnchXgZEi1dmeMvbYCTXHr9ypHoT1DTT4OynbhctsYCSoP+EbwrrgoTLL2ihQoY4sfoilWAccyL9b2BxXhmvMjBcsyZMyn8MsvXAiLEF6i+kBjEcQ4IVHpXgqGyHbnRsWqDzo3SjjlxnPSNl7qtTynTI= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1706558490; c=relaxed/simple; bh=l74wCMgNHIGILBM62GQEer3+RAnaoh90tTNWu1/Em6M=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version:Content-Type; b=Z0QTYK3Z6S6Q7Yy1+lZBu6vtMOwJJgWHCrv1J83N2wRTNsCXjxY8xdVNkBNsgZAEvW+80Vdq9uBugYZm/trctBALdc2qD/0ypGsZtlq7i98rda+r23Ma+E4eQl1Wp3Wmwiakf1umsoznWQF83etxzSFEGFIL+WGKsHvGtguBRsg= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=ragnatech.se; spf=pass smtp.mailfrom=ragnatech.se; dkim=pass (2048-bit key) header.d=ragnatech.se header.i=@ragnatech.se header.b=iRu4lOil; arc=none smtp.client-ip=209.85.208.179 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=ragnatech.se Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=ragnatech.se Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=ragnatech.se header.i=@ragnatech.se header.b="iRu4lOil" Received: by mail-lj1-f179.google.com with SMTP id 38308e7fff4ca-2d040a62a76so18697521fa.0 for ; Mon, 29 Jan 2024 12:01:26 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ragnatech.se; s=google; t=1706558485; x=1707163285; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=ZtKoU0dMc2/VwANL5DHhc0IZfKZvqlfdxMaMmK8zypg=; b=iRu4lOil1eWVnmr2chWiBkIF2vB37UDpaWBmdiua49u+IE2+QebO0W1b2NS6QKvQmc YHy2+JMcMnmJZwpIH1Z1CSaEefH3nZhb3niK13QGJcYNu5ybtIV8jvhM2khSxHPiN7lh Mjs81EGyTpNNSQR7m91hB9Aymax54qvAY6a7fBahsyv8yiSeipKOo0Km1G9NdJaI5DHs GGcuULmkqBuJRVDv8maYuDgx7P6SMMQHh3BJQuuGnlT6PspFuKlQaTa6Xu6zUYly3ajf 9FY13TmMBgj5uye+2MBquija0jqwjMoTULAU0JdSZRBJgkPK9vfBhoTRqoLsup+608uh c0Zg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1706558485; x=1707163285; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=ZtKoU0dMc2/VwANL5DHhc0IZfKZvqlfdxMaMmK8zypg=; b=gSlb4oYcrERSqsasGSxh5NGX15BM/2hpO+fRjs2Db9YKvYiLtGWOOM8r92LlzOm+XF 3rGfzHwfsvaA1dxYkIwncL6QYJkC6igfTEnG8llWy+V23SxdE2/o1SaSduJiPU7vSm2n AkkbYkGy269auF6U3hBswMB10xhKaNMhbg7zRdsA66yalSAtK8wV35GudGKSZh7TQgbZ ukU7WaR4vNZymqGbl2/5vFLOi8J82RR5xaBB/Yiiv2l+6pP5J9G9G75Yy82sVul2QgIH M6Pql8j91ozinmDJ46lwG0pYCBoiFSkgZ2TGSvNw21z7yxNcFL2Wqw75JJQFl1cDYk7o r80Q== X-Gm-Message-State: AOJu0YxT3DRvVedj48JqXjhmlnle3DZlK/SGXAB+/YJSVE1ImbdeozAA Cwjw6EGvc582tLYLjhABFeXusFHbDeZAz7o1Zm+tPFzlf48SYdk3x6Om8/7qwEog0jL0VqQ/5xB u X-Google-Smtp-Source: AGHT+IHwrrMwCYqB0UqECcVQOKDgs/9bTVL/0vnOmGCoVmeioWJ9/5gFmGemvoY77zEIlK8EEak7gA== X-Received: by 2002:a2e:2a82:0:b0:2cf:15b0:d14d with SMTP id q124-20020a2e2a82000000b002cf15b0d14dmr4582923ljq.41.1706558485028; Mon, 29 Jan 2024 12:01:25 -0800 (PST) Received: from sleipner.berto.se (p4fcc8c6a.dip0.t-ipconnect.de. [79.204.140.106]) by smtp.googlemail.com with ESMTPSA id t23-20020a056402241700b00558b0c513e0sm4085223eda.61.2024.01.29.12.01.24 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 29 Jan 2024 12:01:24 -0800 (PST) From: =?utf-8?q?Niklas_S=C3=B6derlund?= To: Sakari Ailus , Mauro Carvalho Chehab , linux-media@vger.kernel.org Cc: linux-renesas-soc@vger.kernel.org, =?utf-8?q?Niklas_S=C3=B6derlund?= Subject: [PATCH] media: v4l: async: Fix completion of chained subnotifiers Date: Mon, 29 Jan 2024 20:59:54 +0100 Message-ID: <20240129195954.1110643-1-niklas.soderlund+renesas@ragnatech.se> X-Mailer: git-send-email 2.43.0 Precedence: bulk X-Mailing-List: linux-media@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Allowing multiple connections between entities are very useful but the addition of this feature did not considerate nested subnotifiers. Consider the scenario, rcar-vin.ko rcar-isp.ko rcar-csi2.ko max96712.ko video0 ----> v4l-subdev0 -> v4l-subdev1 -> v4l-subdev2 video1 -´ Where each videoX or v4l-subdevX is controlled and register by a separate instance of the driver listed above it. And each driver instance registers a notifier (videoX) or a subnotifier (v4l-subdevX) trying to bind to the device pointed to. If the devices probe in any other except where the vidoeX ones are probed last only one of them will have their complete callback called, the one who last registered its notifier. Both of them will however have their bind() callback called as expected. This is due to v4l2_async_nf_try_complete() only walking the chain from the subnotifier to one root notifier and completing it while ignoring all other notifiers the subdevice might be part of. This works if there are only one subnotifier in the mix. For example if either v4l-subdev0 or v4l-subdev1 was not part of the pipeline above. This patch addresses the issue of nested subnotifiers by instead looking at all notifiers and try to complete all the ones that contain the subdevice which subnotifier was completed. Fixes: 28a1295795d8 ("media: v4l: async: Allow multiple connections between entities") Signed-off-by: Niklas Söderlund --- drivers/media/v4l2-core/v4l2-async.c | 68 ++++++++++++++++++++-------- 1 file changed, 49 insertions(+), 19 deletions(-) diff --git a/drivers/media/v4l2-core/v4l2-async.c b/drivers/media/v4l2-core/v4l2-async.c index 3ec323bd528b..8b603527923c 100644 --- a/drivers/media/v4l2-core/v4l2-async.c +++ b/drivers/media/v4l2-core/v4l2-async.c @@ -176,15 +176,16 @@ static LIST_HEAD(notifier_list); static DEFINE_MUTEX(list_lock); static struct v4l2_async_connection * -v4l2_async_find_match(struct v4l2_async_notifier *notifier, - struct v4l2_subdev *sd) +__v4l2_async_find_in_list(struct v4l2_async_notifier *notifier, + struct v4l2_subdev *sd, + struct list_head *list) { bool (*match)(struct v4l2_async_notifier *notifier, struct v4l2_subdev *sd, struct v4l2_async_match_desc *match); struct v4l2_async_connection *asc; - list_for_each_entry(asc, ¬ifier->waiting_list, asc_entry) { + list_for_each_entry(asc, list, asc_entry) { /* bus_type has been verified valid before */ switch (asc->match.type) { case V4L2_ASYNC_MATCH_TYPE_I2C: @@ -207,6 +208,20 @@ v4l2_async_find_match(struct v4l2_async_notifier *notifier, return NULL; } +static struct v4l2_async_connection * +v4l2_async_find_match(struct v4l2_async_notifier *notifier, + struct v4l2_subdev *sd) +{ + return __v4l2_async_find_in_list(notifier, sd, ¬ifier->waiting_list); +} + +static struct v4l2_async_connection * +v4l2_async_find_done(struct v4l2_async_notifier *notifier, + struct v4l2_subdev *sd) +{ + return __v4l2_async_find_in_list(notifier, sd, ¬ifier->done_list); +} + /* Compare two async match descriptors for equivalence */ static bool v4l2_async_match_equal(struct v4l2_async_match_desc *match1, struct v4l2_async_match_desc *match2) @@ -274,13 +289,14 @@ v4l2_async_nf_can_complete(struct v4l2_async_notifier *notifier) } /* - * Complete the master notifier if possible. This is done when all async + * Complete the master notifiers if possible. This is done when all async * sub-devices have been bound; v4l2_device is also available then. */ static int v4l2_async_nf_try_complete(struct v4l2_async_notifier *notifier) { - struct v4l2_async_notifier *__notifier = notifier; + struct v4l2_async_notifier *n; + int ret; /* Quick check whether there are still more sub-devices here. */ if (!list_empty(¬ifier->waiting_list)) @@ -290,24 +306,38 @@ v4l2_async_nf_try_complete(struct v4l2_async_notifier *notifier) dev_dbg(notifier_dev(notifier), "v4l2-async: trying to complete\n"); - /* Check the entire notifier tree; find the root notifier first. */ - while (notifier->parent) - notifier = notifier->parent; + /* + * Notifiers without a parent are either a subnotifier that have not + * yet been associated with it is a root notifier or a root notifier + * itself. If it is a root notifier try to complete it. + */ + if (!notifier->parent) { + /* This is root if it has v4l2_dev. */ + if (!notifier->v4l2_dev) { + dev_dbg(notifier_dev(notifier), + "v4l2-async: V4L2 device not available\n"); + return 0; + } - /* This is root if it has v4l2_dev. */ - if (!notifier->v4l2_dev) { - dev_dbg(notifier_dev(__notifier), - "v4l2-async: V4L2 device not available\n"); - return 0; - } + /* Is everything ready? */ + if (!v4l2_async_nf_can_complete(notifier)) + return 0; + + dev_dbg(notifier_dev(notifier), "v4l2-async: complete\n"); - /* Is everything ready? */ - if (!v4l2_async_nf_can_complete(notifier)) - return 0; + return v4l2_async_nf_call_complete(notifier); + } - dev_dbg(notifier_dev(__notifier), "v4l2-async: complete\n"); + /* Try to complete all notifiers containing the subdevices. */ + list_for_each_entry(n, ¬ifier_list, notifier_entry) { + if (v4l2_async_find_done(n, notifier->sd)) { + ret = v4l2_async_nf_try_complete(n); + if (ret) + return ret; + } + } - return v4l2_async_nf_call_complete(notifier); + return 0; } static int