Commit cab8300b authored by Mukesh Ojha's avatar Mukesh Ojha Committed by Bjorn Andersson
Browse files

remoteproc: Use unbounded workqueue for recovery work 



There could be a scenario when there is too much load on a core
(n number of tasks which is affined) or in a case when multiple
rproc subsystem is going for recovery, they queue their recovery
work to one core so even though subsystem are independent their
recovery will be delayed if one of the subsystem recovery work
is taking more time in completing.

If we make this queue unbounded, the recovery work could be picked
on any cpu. This patch is trying to address this.

Signed-off-by: default avatarMukesh Ojha <quic_mojha@quicinc.com>
Signed-off-by: default avatarBjorn Andersson <bjorn.andersson@linaro.org>
Link: https://lore.kernel.org/r/1650367554-15510-1-git-send-email-quic_mojha@quicinc.com
parent 60349fd5

drivers/remoteproc/remoteproc_core.c

+13 −2
Original line number Diff line number Diff line
@@ -59,6 +59,7 @@ static int rproc_release_carveout(struct rproc *rproc, 


/* Unique indices for remoteproc devices */

static DEFINE_IDA(rproc_dev_index);

static struct workqueue_struct *rproc_recovery_wq;



static const char * const rproc_crash_names[] = {

	[RPROC_MMUFAULT]	= "mmufault",
@@ -2763,8 +2764,7 @@ void rproc_report_crash(struct rproc *rproc, enum rproc_crash_type type) 
	dev_err(&rproc->dev, "crash detected in %s: type %s\n",

		rproc->name, rproc_crash_to_string(type));



	/* Have a worker handle the error; ensure system is not suspended */

	queue_work(system_freezable_wq, &rproc->crash_handler);

	queue_work(rproc_recovery_wq, &rproc->crash_handler);

}

EXPORT_SYMBOL(rproc_report_crash);
@@ -2813,6 +2813,13 @@ static void __exit rproc_exit_panic(void) 


static int __init remoteproc_init(void)

{

	rproc_recovery_wq = alloc_workqueue("rproc_recovery_wq",

						WQ_UNBOUND | WQ_FREEZABLE, 0);

	if (!rproc_recovery_wq) {

		pr_err("remoteproc: creation of rproc_recovery_wq failed\n");

		return -ENOMEM;

	}



	rproc_init_sysfs();

	rproc_init_debugfs();

	rproc_init_cdev();
@@ -2826,9 +2833,13 @@ static void __exit remoteproc_exit(void) 
{

	ida_destroy(&rproc_dev_index);



	if (!rproc_recovery_wq)

		return;



	rproc_exit_panic();

	rproc_exit_debugfs();

	rproc_exit_sysfs();

	destroy_workqueue(rproc_recovery_wq);

}

module_exit(remoteproc_exit);