Re: [PATCH] staging: unisys: visorbus: visorchannel: Refactor locking code to be statically deterministic.

2016-10-19 Thread Chuckleberryfinn
On Wed, Oct 19, 2016 at 05:00:53PM +, Sell, Timothy C wrote: > On Wednesday, October 19, 2016 7:31 AM, Cathal Mullaney wrote: > > This patch makes locking in visorchannel_signalempty statically > > deterministic. > > As a result this patch fixes the sparse warning: > > Context imbalance in

RE: [PATCH] staging: unisys: visorbus: visorchannel: Refactor locking code to be statically deterministic.

2016-10-19 Thread Sell, Timothy C
On Wednesday, October 19, 2016 7:31 AM, Cathal Mullaney wrote: > This patch makes locking in visorchannel_signalempty statically deterministic. > As a result this patch fixes the sparse warning: > Context imbalance in 'visorchannel_signalempty' - different lock contexts for > basic block. > > The

[PATCH] staging: unisys: visorbus: visorchannel: Refactor locking code to be statically deterministic.

2016-10-19 Thread Cathal Mullaney
This patch makes locking in visorchannel_signalempty statically deterministic. As a result this patch fixes the sparse warning: Context imbalance in 'visorchannel_signalempty' - different lock contexts for basic block. The logic of the locking code doesn't change but the layout of the original