Code review comment for lp:~epics-core/epics-base/spinlockfix

Revision history for this message
mdavidsaver (mdavidsaver) wrote :

> As long as you're not going to be calling cantProceed() with interrupts and
> thread dispatch disabled that would be fine by me.

Hanging in an ISR isn't nice, but neither is entering the critical section w/o the lock. We (or rather our users) will lose either way.

Which way should we lose?

> ...I would just pass NULL into the cantProceed() call instead.

Ok.

« Back to merge proposal