Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 18 Jul 2021 00:36:56 GMT
From:      "Bjoern A. Zeeb" <bz@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-branches@FreeBSD.org
Subject:   git: f2bd70c674f0 - stable/13 - LinuxKPI: firmware, implement deferred loading for "nowait"
Message-ID:  <202107180036.16I0auIV049786@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch stable/13 has been updated by bz:

URL: https://cgit.FreeBSD.org/src/commit/?id=f2bd70c674f0409dc4f1dc5b3ca1142a1456adf6

commit f2bd70c674f0409dc4f1dc5b3ca1142a1456adf6
Author:     Bjoern A. Zeeb <bz@FreeBSD.org>
AuthorDate: 2021-06-20 13:49:46 +0000
Commit:     Bjoern A. Zeeb <bz@FreeBSD.org>
CommitDate: 2021-07-18 00:35:04 +0000

    LinuxKPI: firmware, implement deferred loading for "nowait"
    
    Change linuxkpi_request_firmware_nowait() to deferred firmware loading
    scheduling a task.  This changes behaviour in some cases that we
    return from loading the driver before the driver is finished
    initialising if the driver does not deal with it (wait).
    This brings the behaviour one would expect from when this function is
    called and I implemented it to see if it would help a specific case.
    
    Sponsored by:   The FreeBSD Foundation
    Reviewed by:    hselasky, imp (earlier version)
    Differential Revision: https://reviews.freebsd.org/D30830
    
    (cherry picked from commit 399da52fff81a33b1f2c0cee3e8574bc3c63166f)
---
 sys/compat/linuxkpi/common/src/linux_firmware.c | 74 +++++++++++++++++++------
 1 file changed, 58 insertions(+), 16 deletions(-)

diff --git a/sys/compat/linuxkpi/common/src/linux_firmware.c b/sys/compat/linuxkpi/common/src/linux_firmware.c
index 289779c5246c..d779b509105e 100644
--- a/sys/compat/linuxkpi/common/src/linux_firmware.c
+++ b/sys/compat/linuxkpi/common/src/linux_firmware.c
@@ -30,9 +30,13 @@
  * $FreeBSD$
  */
 
+#include <sys/param.h>
+#include <sys/kernel.h>
 #include <sys/types.h>
 #include <sys/malloc.h>
 #include <sys/firmware.h>
+#include <sys/queue.h>
+#include <sys/taskqueue.h>
 
 #include <linux/types.h>
 #include <linux/device.h>
@@ -42,6 +46,16 @@
 
 MALLOC_DEFINE(M_LKPI_FW, "lkpifw", "LinuxKPI firmware");
 
+struct lkpi_fw_task {
+	/* Task and arguments for the "nowait" callback. */
+	struct task		fw_task;
+	gfp_t			gfp;
+	const char		*fw_name;
+	struct device		*dev;
+	void			*drv;
+	void(*cont)(const struct linuxkpi_firmware *, void *);
+};
+
 static int
 _linuxkpi_request_firmware(const char *fw_name, const struct linuxkpi_firmware **fw,
     struct device *dev, gfp_t gfp __unused, bool enoentok, bool warn)
@@ -52,8 +66,10 @@ _linuxkpi_request_firmware(const char *fw_name, const struct linuxkpi_firmware *
 	char *p;
 	uint32_t flags;
 
-	if (fw_name == NULL || fw == NULL || dev == NULL)
+	if (fw_name == NULL || fw == NULL || dev == NULL) {
+		*fw = NULL;
 		return (-EINVAL);
+	}
 
 	/* Set independent on "warn". To debug, bootverbose is avail. */
 	flags = FIRMWARE_GET_NOWARN;
@@ -129,28 +145,54 @@ _linuxkpi_request_firmware(const char *fw_name, const struct linuxkpi_firmware *
 	return (0);
 }
 
-int
-linuxkpi_request_firmware_nowait(struct module *mod __unused, bool _t __unused,
-    const char *fw_name, struct device *dev, gfp_t gfp, void *drv,
-    void(*cont)(const struct linuxkpi_firmware *, void *))
+static void
+lkpi_fw_task(void *ctx, int pending)
 {
-	const struct linuxkpi_firmware *lfw;
+	struct lkpi_fw_task *lfwt;
+	const struct linuxkpi_firmware *fw;
 	int error;
 
+	KASSERT(ctx != NULL && pending == 1, ("%s: lfwt %p, pending %d\n",
+	    __func__, ctx, pending));
+
+	lfwt = ctx;
+	if (lfwt->cont == NULL)
+		goto out;
+
+	error = _linuxkpi_request_firmware(lfwt->fw_name, &fw, lfwt->dev,
+	    lfwt->gfp, true, true);
+
 	/*
 	 * Linux seems to run the callback if it cannot find the firmware.
-	 * The fact that this is "_nowait()" and has a callback seems to
-	 * imply that this is run in a deferred conext which we currently
-	 * do not do.  Should it become necessary (a driver actually requiring
-	 * it) we would need to implement it here.
+	 * We call it in all cases as it is the only feedback to the requester.
 	 */
-	error = _linuxkpi_request_firmware(fw_name, &lfw, dev, gfp, true, true);
-	if (error == -ENOENT)
-		error = 0;
-	if (error == 0)
-		cont(lfw, drv);
+	lfwt->cont(fw, lfwt->drv);
+	/* Do not assume fw is still valid! */
 
-	return (error);
+out:
+	free(lfwt, M_LKPI_FW);
+}
+
+int
+linuxkpi_request_firmware_nowait(struct module *mod __unused, bool _t __unused,
+    const char *fw_name, struct device *dev, gfp_t gfp, void *drv,
+    void(*cont)(const struct linuxkpi_firmware *, void *))
+{
+	struct lkpi_fw_task *lfwt;
+	int error;
+
+	lfwt = malloc(sizeof(*lfwt), M_LKPI_FW, M_WAITOK | M_ZERO);
+	lfwt->gfp = gfp;
+	lfwt->fw_name = fw_name;
+	lfwt->dev = dev;
+	lfwt->drv = drv;
+	lfwt->cont = cont;
+	TASK_INIT(&lfwt->fw_task, 0, lkpi_fw_task, lfwt);
+	error = taskqueue_enqueue(taskqueue_thread, &lfwt->fw_task);
+
+	if (error)
+		return (-error);
+	return (0);
 }
 
 int



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202107180036.16I0auIV049786>