Re: [PATCH] cdc-acm: some enhancement on acm delayed write

2014-04-11 Thread Oliver Neukum
On Fri, 2014-04-11 at 06:51 +0800, Xiao Jin wrote: > Hi, Oliver, > > On 04/10/2014 04:02 PM, Oliver Neukum wrote: > > On Wed, 2014-04-09 at 22:57 +0800, Xiao Jin wrote: > >> Thanks all for the review. We meet with the problems when developing > >> product. I would like to explain my understanding.

Re: [PATCH] cdc-acm: some enhancement on acm delayed write

2014-04-11 Thread Johan Hovold
On Tue, Apr 08, 2014 at 12:22:22PM +0200, Oliver Neukum wrote: > On Tue, 2014-04-08 at 09:33 +0200, Johan Hovold wrote: > > On Tue, Apr 08, 2014 at 11:05:20AM +0800, Xiao Jin wrote: > > > We find two problems on acm tty write delayed mechanism. > > > > Then you should split this into two patches.

Re: [PATCH] cdc-acm: some enhancement on acm delayed write

2014-04-11 Thread Johan Hovold
[ +CC: Jiri and Peter ] On Thu, Apr 10, 2014 at 10:02:03AM +0200, Oliver Neukum wrote: > On Wed, 2014-04-09 at 22:57 +0800, Xiao Jin wrote: > > Thanks all for the review. We meet with the problems when developing > > product. I would like to explain my understanding. > > > > On 04/08/2014 11:05

Re: [PATCH] cdc-acm: some enhancement on acm delayed write

2014-04-10 Thread Xiao Jin
Hi, Oliver, On 04/10/2014 04:02 PM, Oliver Neukum wrote: On Wed, 2014-04-09 at 22:57 +0800, Xiao Jin wrote: Thanks all for the review. We meet with the problems when developing product. I would like to explain my understanding. On 04/08/2014 11:05 AM, Xiao Jin wrote: We find two problems on

Re: [PATCH] cdc-acm: some enhancement on acm delayed write

2014-04-10 Thread Oliver Neukum
On Wed, 2014-04-09 at 22:57 +0800, Xiao Jin wrote: > Thanks all for the review. We meet with the problems when developing > product. I would like to explain my understanding. > > On 04/08/2014 11:05 AM, Xiao Jin wrote: > > > > We find two problems on acm tty write delayed mechanism. > > (1) When

Re: [PATCH] cdc-acm: some enhancement on acm delayed write

2014-04-09 Thread David Cohen
On Wed, Apr 09, 2014 at 10:57:10PM +0800, Xiao Jin wrote: > Thanks all for the review. We meet with the problems when developing > product. I would like to explain my understanding. > > On 04/08/2014 11:05 AM, Xiao Jin wrote: > > > >We find two problems on acm tty write delayed mechanism. > >(1) W

Re: [PATCH] cdc-acm: some enhancement on acm delayed write

2014-04-09 Thread Xiao Jin
Thanks all for the review. We meet with the problems when developing product. I would like to explain my understanding. On 04/08/2014 11:05 AM, Xiao Jin wrote: We find two problems on acm tty write delayed mechanism. (1) When acm resume, the delayed wb will be started. But now only one write c

Re: [PATCH] cdc-acm: some enhancement on acm delayed write

2014-04-08 Thread Johan Hovold
> > > If I see this correctly, then ASYNCB_INITIALIZED is cleared in > > > tty_port_close() we is called from acm_tty_close(). Thus it should > > > be enough to make sure that the device is resumed at the beginning > > > of acm_tty_close() and acm_resume() will do the job automatically. > > > What

Re: [PATCH] cdc-acm: some enhancement on acm delayed write

2014-04-08 Thread Oliver Neukum
On Tue, 2014-04-08 at 15:17 +0200, Johan Hovold wrote: > [ +CC: Alan ] > > On Tue, Apr 08, 2014 at 12:33:31PM +0200, Oliver Neukum wrote: > > On Tue, 2014-04-08 at 09:33 +0200, Johan Hovold wrote: > > > On Tue, Apr 08, 2014 at 11:05:20AM +0800, Xiao Jin wrote: > > > > > > (2) acm tty port ASYNCB_

Re: [PATCH] cdc-acm: some enhancement on acm delayed write

2014-04-08 Thread Johan Hovold
[ +CC: Alan ] On Tue, Apr 08, 2014 at 12:33:31PM +0200, Oliver Neukum wrote: > On Tue, 2014-04-08 at 09:33 +0200, Johan Hovold wrote: > > On Tue, Apr 08, 2014 at 11:05:20AM +0800, Xiao Jin wrote: > > > > (2) acm tty port ASYNCB_INITIALIZED flag will be cleared when > > > close. If acm resume call

Re: [PATCH] cdc-acm: some enhancement on acm delayed write

2014-04-08 Thread Johan Hovold
[ +CC: Oliver ] On Tue, Apr 08, 2014 at 12:22:29PM +0100, One Thousand Gnomes wrote: > > (2) acm tty port ASYNCB_INITIALIZED flag will be cleared when > > close. If acm resume callback run after ASYNCB_INITIALIZED flag > > cleared, there will have no chance for delayed write to start. > > That lea

Re: [PATCH] cdc-acm: some enhancement on acm delayed write

2014-04-08 Thread One Thousand Gnomes
> (2) acm tty port ASYNCB_INITIALIZED flag will be cleared when > close. If acm resume callback run after ASYNCB_INITIALIZED flag > cleared, there will have no chance for delayed write to start. > That lead to acm_wb.use can't be cleared. If user space open > acm tty again and try to setd, tty will

Re: [PATCH] cdc-acm: some enhancement on acm delayed write

2014-04-08 Thread Oliver Neukum
On Tue, 2014-04-08 at 09:33 +0200, Johan Hovold wrote: > On Tue, Apr 08, 2014 at 11:05:20AM +0800, Xiao Jin wrote: > > (2) acm tty port ASYNCB_INITIALIZED flag will be cleared when > > close. If acm resume callback run after ASYNCB_INITIALIZED flag > > cleared, there will have no chance for delaye

Re: [PATCH] cdc-acm: some enhancement on acm delayed write

2014-04-08 Thread Oliver Neukum
On Tue, 2014-04-08 at 09:33 +0200, Johan Hovold wrote: > On Tue, Apr 08, 2014 at 11:05:20AM +0800, Xiao Jin wrote: > > We find two problems on acm tty write delayed mechanism. > > Then you should split this into two patches. > > > (1) When acm resume, the delayed wb will be started. But now > > o

Re: [PATCH] cdc-acm: some enhancement on acm delayed write

2014-04-08 Thread Johan Hovold
On Tue, Apr 08, 2014 at 11:05:20AM +0800, Xiao Jin wrote: > We find two problems on acm tty write delayed mechanism. Then you should split this into two patches. > (1) When acm resume, the delayed wb will be started. But now > only one write can be saved during acm suspend. More acm write > may b