Re: [patch v6 5/7] genirq/affinity: Remove the leftovers of the original set support

From: Ming Lei
Date: Sun Feb 17 2019 - 08:39:17 EST


On Sat, Feb 16, 2019 at 06:13:11PM +0100, Thomas Gleixner wrote:
> Now that the NVME driver is converted over to the calc_set() callback, the
> workarounds of the original set support can be removed.
>
> Signed-off-by: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
> ---
> kernel/irq/affinity.c | 20 ++++----------------
> 1 file changed, 4 insertions(+), 16 deletions(-)
>
> Index: b/kernel/irq/affinity.c
> ===================================================================
> --- a/kernel/irq/affinity.c
> +++ b/kernel/irq/affinity.c
> @@ -264,20 +264,13 @@ irq_create_affinity_masks(unsigned int n
>
> /*
> * Simple invocations do not provide a calc_sets() callback. Install
> - * the generic one. The check for affd->nr_sets is a temporary
> - * workaround and will be removed after the NVME driver is converted
> - * over.
> + * the generic one.
> */
> - if (!affd->nr_sets && !affd->calc_sets)
> + if (!affd->calc_sets)
> affd->calc_sets = default_calc_sets;
>
> - /*
> - * If the device driver provided a calc_sets() callback let it
> - * recalculate the number of sets and their size. The check will go
> - * away once the NVME driver is converted over.
> - */
> - if (affd->calc_sets)
> - affd->calc_sets(affd, affvecs);
> + /* Recalculate the sets */
> + affd->calc_sets(affd, affvecs);
>
> if (WARN_ON_ONCE(affd->nr_sets > IRQ_AFFINITY_MAX_SETS))
> return NULL;
> @@ -344,11 +337,6 @@ unsigned int irq_calc_affinity_vectors(u
>
> if (affd->calc_sets) {
> set_vecs = maxvec - resv;
> - } else if (affd->nr_sets) {
> - unsigned int i;
> -
> - for (i = 0, set_vecs = 0; i < affd->nr_sets; i++)
> - set_vecs += affd->set_size[i];
> } else {
> get_online_cpus();
> set_vecs = cpumask_weight(cpu_possible_mask);
>
>

Reviewed-by: Ming Lei <ming.lei@xxxxxxxxxx>

Thanks,
Ming