Re: [PATCH 1/2 v2] target: drop spin_lock_assert() + irqs_disabled() combo checks

2018-03-23 Thread Bart Van Assche
On Fri, 2018-03-23 at 18:36 +0100, Sebastian Andrzej Siewior wrote: > There are a few functions which check for if the lock is held > (spin_lock_assert()) and the interrupts are disabled (irqs_disabled()). > > From looking at the code, each function is static, the caller is near by > > and does

[PATCH 1/2 v2] target: drop spin_lock_assert() + irqs_disabled() combo checks

2018-03-23 Thread Sebastian Andrzej Siewior
There are a few functions which check for if the lock is held (spin_lock_assert()) and the interrupts are disabled (irqs_disabled()). >From looking at the code, each function is static, the caller is near by and does spin_lock_irq|safe(). As Linus puts it: |It's not like this is some function