Re: [PATCH 1/2] lightnvm: warn if irqs are disabled in lock laddr
On 02/01/2016 11:34 AM, Javier González wrote: > Add a warning if irqs are disabled when locking a new address in rrpc. > The typical path to a new request does not disable irqs, but this is not > guaranteed in the future. > > Signed-off-by: Javier González > --- > drivers/lightnvm/rrpc.h | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/lightnvm/rrpc.h b/drivers/lightnvm/rrpc.h > index dfca5c4..c27283a 100644 > --- a/drivers/lightnvm/rrpc.h > +++ b/drivers/lightnvm/rrpc.h > @@ -184,6 +184,8 @@ static int __rrpc_lock_laddr(struct rrpc *rrpc, sector_t > laddr, > sector_t laddr_end = laddr + pages - 1; > struct rrpc_inflight_rq *rtmp; > > + WARN_ON(irqs_disabled()); > + > spin_lock_irq(>inflights.lock); > list_for_each_entry(rtmp, >inflights.reqs, list) { > if (unlikely(request_intersects(rtmp, laddr, laddr_end))) { > Thanks, applied for next -rc.
Re: [PATCH 1/2] lightnvm: warn if irqs are disabled in lock laddr
On 02/01/2016 11:34 AM, Javier González wrote: > Add a warning if irqs are disabled when locking a new address in rrpc. > The typical path to a new request does not disable irqs, but this is not > guaranteed in the future. > > Signed-off-by: Javier González> --- > drivers/lightnvm/rrpc.h | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/lightnvm/rrpc.h b/drivers/lightnvm/rrpc.h > index dfca5c4..c27283a 100644 > --- a/drivers/lightnvm/rrpc.h > +++ b/drivers/lightnvm/rrpc.h > @@ -184,6 +184,8 @@ static int __rrpc_lock_laddr(struct rrpc *rrpc, sector_t > laddr, > sector_t laddr_end = laddr + pages - 1; > struct rrpc_inflight_rq *rtmp; > > + WARN_ON(irqs_disabled()); > + > spin_lock_irq(>inflights.lock); > list_for_each_entry(rtmp, >inflights.reqs, list) { > if (unlikely(request_intersects(rtmp, laddr, laddr_end))) { > Thanks, applied for next -rc.
[PATCH 1/2] lightnvm: warn if irqs are disabled in lock laddr
Add a warning if irqs are disabled when locking a new address in rrpc. The typical path to a new request does not disable irqs, but this is not guaranteed in the future. Signed-off-by: Javier González --- drivers/lightnvm/rrpc.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/lightnvm/rrpc.h b/drivers/lightnvm/rrpc.h index dfca5c4..c27283a 100644 --- a/drivers/lightnvm/rrpc.h +++ b/drivers/lightnvm/rrpc.h @@ -184,6 +184,8 @@ static int __rrpc_lock_laddr(struct rrpc *rrpc, sector_t laddr, sector_t laddr_end = laddr + pages - 1; struct rrpc_inflight_rq *rtmp; + WARN_ON(irqs_disabled()); + spin_lock_irq(>inflights.lock); list_for_each_entry(rtmp, >inflights.reqs, list) { if (unlikely(request_intersects(rtmp, laddr, laddr_end))) { -- 2.1.4
[PATCH 1/2] lightnvm: warn if irqs are disabled in lock laddr
Add a warning if irqs are disabled when locking a new address in rrpc. The typical path to a new request does not disable irqs, but this is not guaranteed in the future. Signed-off-by: Javier González--- drivers/lightnvm/rrpc.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/lightnvm/rrpc.h b/drivers/lightnvm/rrpc.h index dfca5c4..c27283a 100644 --- a/drivers/lightnvm/rrpc.h +++ b/drivers/lightnvm/rrpc.h @@ -184,6 +184,8 @@ static int __rrpc_lock_laddr(struct rrpc *rrpc, sector_t laddr, sector_t laddr_end = laddr + pages - 1; struct rrpc_inflight_rq *rtmp; + WARN_ON(irqs_disabled()); + spin_lock_irq(>inflights.lock); list_for_each_entry(rtmp, >inflights.reqs, list) { if (unlikely(request_intersects(rtmp, laddr, laddr_end))) { -- 2.1.4