Re: [PATCH] USB: cdc-acm: remove unneeded spin_lock_irqsave/restore on write path

2013-06-07 Thread Greg Kroah-Hartman
On Fri, Jun 07, 2013 at 10:38:30PM +0400, Sergei Shtylyov wrote: > Hello. > > On 06/07/2013 10:32 PM, Greg Kroah-Hartman wrote: > > >From: Greg Kroah-Hartman > > > >When writing data we were: > > lock > > do some work > > unlock > > call function > > lock > >

Re: [PATCH] USB: cdc-acm: remove unneeded spin_lock_irqsave/restore on write path

2013-06-07 Thread Sergei Shtylyov
Hello. On 06/07/2013 10:32 PM, Greg Kroah-Hartman wrote: From: Greg Kroah-Hartman When writing data we were: lock do some work unlock call function lock do some work unlock return return

[PATCH] USB: cdc-acm: remove unneeded spin_lock_irqsave/restore on write path

2013-06-07 Thread Greg Kroah-Hartman
From: Greg Kroah-Hartman When writing data we were: lock do some work unlock call function lock do some work unlock return return It turns out, that "function" was only ever called in the

[PATCH] USB: cdc-acm: remove unneeded spin_lock_irqsave/restore on write path

2013-06-07 Thread Greg Kroah-Hartman
From: Greg Kroah-Hartman gre...@linuxfoundation.org When writing data we were: lock do some work unlock call function lock do some work unlock return return It turns out, that function was

Re: [PATCH] USB: cdc-acm: remove unneeded spin_lock_irqsave/restore on write path

2013-06-07 Thread Sergei Shtylyov
Hello. On 06/07/2013 10:32 PM, Greg Kroah-Hartman wrote: From: Greg Kroah-Hartman gre...@linuxfoundation.org When writing data we were: lock do some work unlock call function lock do some work unlock

Re: [PATCH] USB: cdc-acm: remove unneeded spin_lock_irqsave/restore on write path

2013-06-07 Thread Greg Kroah-Hartman
On Fri, Jun 07, 2013 at 10:38:30PM +0400, Sergei Shtylyov wrote: Hello. On 06/07/2013 10:32 PM, Greg Kroah-Hartman wrote: From: Greg Kroah-Hartman gre...@linuxfoundation.org When writing data we were: lock do some work unlock call function lock