Skip to content

Commit

Permalink
block: Don't disable interrupts in trigger_softirq()
Browse files Browse the repository at this point in the history
trigger_softirq() is always invoked as a SMP-function call which is
always invoked with disables interrupts.

Don't disable interrupt in trigger_softirq() because interrupts are
already disabled.

Reviewed-by: Ming Lei <[email protected]>
Signed-off-by: Sebastian Andrzej Siewior <[email protected]>
Signed-off-by: Jens Axboe <[email protected]>
  • Loading branch information
Sebastian Andrzej Siewior authored and axboe committed Nov 18, 2019
1 parent 708edaf commit de678bc
Showing 1 changed file with 0 additions and 4 deletions.
4 changes: 0 additions & 4 deletions block/blk-softirq.c
Original file line number Diff line number Diff line change
Expand Up @@ -42,17 +42,13 @@ static __latent_entropy void blk_done_softirq(struct softirq_action *h)
static void trigger_softirq(void *data)
{
struct request *rq = data;
unsigned long flags;
struct list_head *list;

local_irq_save(flags);
list = this_cpu_ptr(&blk_cpu_done);
list_add_tail(&rq->ipi_list, list);

if (list->next == &rq->ipi_list)
raise_softirq_irqoff(BLOCK_SOFTIRQ);

local_irq_restore(flags);
}

/*
Expand Down

0 comments on commit de678bc

Please sign in to comment.