mirror of
https://github.com/torvalds/linux.git
synced 2026-09-23 22:14:03 +02:00
eth: bnxt: decrease indent in bnxt_request_irq()
bnxt_request_irq() has unnecessary level of indentation. Use continue instead. No need to re-fetch NUMA node for each IRQ, move to the function level. No functional changes. Reviewed-by: Breno Leitao <leitao@debian.org> Link: https://patch.msgid.link/20260803193135.2030368-4-kuba@kernel.org Signed-off-by: Jakub Kicinski <kuba@kernel.org>
This commit is contained in:
parent
4b5cb58a44
commit
0fd562c7b9
|
|
@ -11910,6 +11910,7 @@ static void bnxt_free_irq(struct bnxt *bp)
|
|||
|
||||
static int bnxt_request_irq(struct bnxt *bp)
|
||||
{
|
||||
const int numa_node = dev_to_node(&bp->pdev->dev);
|
||||
struct cpu_rmap *rmap = NULL;
|
||||
int i, j, rc = 0;
|
||||
unsigned long flags = 0;
|
||||
|
|
@ -11932,6 +11933,7 @@ static int bnxt_request_irq(struct bnxt *bp)
|
|||
for (i = 0, j = 0; i < bp->cp_nr_rings; i++) {
|
||||
int map_idx = bnxt_cp_num_to_irq_num(bp, i);
|
||||
struct bnxt_irq *irq = &bp->irq_tbl[map_idx];
|
||||
u16 tag;
|
||||
|
||||
if (IS_ENABLED(CONFIG_RFS_ACCEL) &&
|
||||
rmap && bp->bnapi[i]->rx_ring) {
|
||||
|
|
@ -11950,33 +11952,31 @@ static int bnxt_request_irq(struct bnxt *bp)
|
|||
netif_napi_set_irq_locked(&bp->bnapi[i]->napi, irq->vector);
|
||||
irq->requested = 1;
|
||||
|
||||
if (zalloc_cpumask_var(&irq->cpu_mask, GFP_KERNEL)) {
|
||||
int numa_node = dev_to_node(&bp->pdev->dev);
|
||||
u16 tag;
|
||||
if (!zalloc_cpumask_var(&irq->cpu_mask, GFP_KERNEL))
|
||||
continue;
|
||||
|
||||
irq->have_cpumask = 1;
|
||||
irq->msix_nr = map_idx;
|
||||
irq->ring_nr = i;
|
||||
cpumask_set_cpu(cpumask_local_spread(i, numa_node),
|
||||
irq->cpu_mask);
|
||||
rc = irq_update_affinity_hint(irq->vector, irq->cpu_mask);
|
||||
if (rc) {
|
||||
netdev_warn(bp->dev,
|
||||
"Update affinity hint failed, IRQ = %d\n",
|
||||
irq->vector);
|
||||
break;
|
||||
}
|
||||
|
||||
bnxt_register_irq_notifier(bp, irq);
|
||||
|
||||
/* Init ST table entry */
|
||||
if (pcie_tph_get_cpu_st(irq->bp->pdev, TPH_MEM_TYPE_VM,
|
||||
cpumask_first(irq->cpu_mask),
|
||||
&tag))
|
||||
continue;
|
||||
|
||||
pcie_tph_set_st_entry(irq->bp->pdev, irq->msix_nr, tag);
|
||||
irq->have_cpumask = 1;
|
||||
irq->msix_nr = map_idx;
|
||||
irq->ring_nr = i;
|
||||
cpumask_set_cpu(cpumask_local_spread(i, numa_node),
|
||||
irq->cpu_mask);
|
||||
rc = irq_update_affinity_hint(irq->vector, irq->cpu_mask);
|
||||
if (rc) {
|
||||
netdev_warn(bp->dev,
|
||||
"Update affinity hint failed, IRQ = %d\n",
|
||||
irq->vector);
|
||||
break;
|
||||
}
|
||||
|
||||
bnxt_register_irq_notifier(bp, irq);
|
||||
|
||||
/* Init ST table entry */
|
||||
if (pcie_tph_get_cpu_st(irq->bp->pdev, TPH_MEM_TYPE_VM,
|
||||
cpumask_first(irq->cpu_mask),
|
||||
&tag))
|
||||
continue;
|
||||
|
||||
pcie_tph_set_st_entry(irq->bp->pdev, irq->msix_nr, tag);
|
||||
}
|
||||
return rc;
|
||||
}
|
||||
|
|
|
|||
Loading…
Reference in New Issue
Block a user