Re: [workqueue/driver-core PATCH 1/5] workqueue: Provide queue_work_near to queue work near a given NUMA node

From: Tejun Heo
Date: Wed Oct 10 2018 - 11:30:20 EST


Hello,

> +static int workqueue_select_cpu_near(int node)
> +{
...
> + /* Use "random" otherwise know as "first" online CPU of node */
> + cpu = cpumask_any_and(cpumask_of_node(node), cpu_online_mask);

Maybe we wanna round-robin within the node?

> +bool queue_work_near(int node, struct workqueue_struct *wq,
> + struct work_struct *work)

Can we name it queue_work_node() to stay consistent with numa-aware
memory interface?

Thanks.

--
tejun