[PATCH rfc 2/6] mlx5: move affinity hints assignments to generic code
Sagi Grimberg
sagi at grimberg.me
Thu Apr 6 01:29:12 PDT 2017
>> 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.
More information about the Linux-nvme
mailing list