Re: [PATCH RFC] mailbox: move controller timer to per-channel timers

2017-05-31 Thread Jassi Brar
On Tue, May 30, 2017 at 9:59 PM, Alexey Klimov wrote: > > I was able to come up with new version with only HR-timer spinlock in timer > callback. That one seems a little bit better. > Yes, this is what I meant by simply using a lock. However the lock around the last_tx_done() for all channels see

Re: [PATCH RFC] mailbox: move controller timer to per-channel timers

2017-05-30 Thread Alexey Klimov
On Thu, May 25, 2017 at 6:43 PM, Alexey Klimov wrote: > Hi Jassi, > > sorry for delay again. > > On Tue, Apr 11, 2017 at 06:30:08PM +0530, Jassi Brar wrote: > > On 11 April 2017 at 18:04, Alexey Klimov wrote: > > > On Fri, Apr 07, 2017 at 08:39:35PM +0530, Jassi Brar wrote: > > >> On Thu, Apr 6

Re: [PATCH RFC] mailbox: move controller timer to per-channel timers

2017-05-25 Thread Alexey Klimov
Hi Jassi, sorry for delay again. On Tue, Apr 11, 2017 at 06:30:08PM +0530, Jassi Brar wrote: > On 11 April 2017 at 18:04, Alexey Klimov wrote: > > On Fri, Apr 07, 2017 at 08:39:35PM +0530, Jassi Brar wrote: > >> On Thu, Apr 6, 2017 at 11:01 PM, Alexey Klimov > >> wrote: > >> > When mailbox con

Re: [PATCH RFC] mailbox: move controller timer to per-channel timers

2017-04-11 Thread Jassi Brar
On 11 April 2017 at 18:04, Alexey Klimov wrote: > On Fri, Apr 07, 2017 at 08:39:35PM +0530, Jassi Brar wrote: >> On Thu, Apr 6, 2017 at 11:01 PM, Alexey Klimov wrote: >> > When mailbox controller provides two or more channels and >> > they are actively used by mailbox client(s) it's very easy >>

Re: [PATCH RFC] mailbox: move controller timer to per-channel timers

2017-04-11 Thread Alexey Klimov
On Fri, Apr 07, 2017 at 08:39:35PM +0530, Jassi Brar wrote: > On Thu, Apr 6, 2017 at 11:01 PM, Alexey Klimov wrote: > > When mailbox controller provides two or more channels and > > they are actively used by mailbox client(s) it's very easy > > to trigger the warning in hrtimer_forward(): > > > >

Re: [PATCH RFC] mailbox: move controller timer to per-channel timers

2017-04-07 Thread Jassi Brar
On Thu, Apr 6, 2017 at 11:01 PM, Alexey Klimov wrote: > When mailbox controller provides two or more channels and > they are actively used by mailbox client(s) it's very easy > to trigger the warning in hrtimer_forward(): > > [ 247.853060] WARNING: CPU: 6 PID: 0 at kernel/time/hrtimer.c:805 > hr

[PATCH RFC] mailbox: move controller timer to per-channel timers

2017-04-06 Thread Alexey Klimov
When mailbox controller provides two or more channels and they are actively used by mailbox client(s) it's very easy to trigger the warning in hrtimer_forward(): [ 247.853060] WARNING: CPU: 6 PID: 0 at kernel/time/hrtimer.c:805 hrtimer_forward+0x88/0xd8 [ 247.853549] Modules linked in: [ 247.8