On 10/21/22 12:03, Guilherme G. Piccoli wrote: ... >> if (need_release_sem) >> up(...); >> >> That's nice and compact. It's also logically easy to follow because you >> can see how the need_release_sem gets set only after the >> down_interruptible(). It's also nice to have both sites share the >> 'need_release_sem' naming for grepping. >> > > ...but, this is a very good suggestion, and will eliminate the need for > two delayed_works, right? Yes, that too.
