static int mlx5e_get_cpu(struct mlx5e_priv *priv, int ix) { - return cpumask_first(priv->mdev->priv.irq_info[ix].mask); + return cpumask_first(pci_irq_get_affinity(priv->mdev->pdev, + MLX5_EQ_VEC_COMP_BASE + ix));This looks ok for now, but if we look at the callers we'd probably want to make direct use of pci_irq_get_node and pci_irq_get_affinity for the uses directly in mlx5e_open_channel as well as the stored away ->cpu field. But maybe that should be left for another patch after this one.
Its small enough to fold it in.
+ struct irq_affinity irqdesc = { .pre_vectors = MLX5_EQ_VEC_COMP_BASE, };I usually move assignments inside structures onto a separate line to make it more readable, e.g. struct irq_affinity irqdesc = { .pre_vectors = MLX5_EQ_VEC_COMP_BASE, };
Will do.
