Re: [RESEND PATCH 2/2] staging/olpc_docn: reorder the lock sequence to avoid potential dead lock

2013-11-07 Thread Dan Carpenter
On Wed, Nov 06, 2013 at 09:22:36AM +0800, Gu Zheng wrote: > Hi Dan, > On 11/05/2013 07:02 PM, Dan Carpenter wrote: > > > On Tue, Nov 05, 2013 at 06:01:00PM +0800, Gu Zheng wrote: > >> The lock sequence of dcon_blank_fb(fb_info->lock ---> console_lock) is > >> against > >> with the one of

Re: [RESEND PATCH 2/2] staging/olpc_docn: reorder the lock sequence to avoid potential dead lock

2013-11-07 Thread Dan Carpenter
On Wed, Nov 06, 2013 at 09:22:36AM +0800, Gu Zheng wrote: Hi Dan, On 11/05/2013 07:02 PM, Dan Carpenter wrote: On Tue, Nov 05, 2013 at 06:01:00PM +0800, Gu Zheng wrote: The lock sequence of dcon_blank_fb(fb_info-lock --- console_lock) is against with the one of

Re: [RESEND PATCH 2/2] staging/olpc_docn: reorder the lock sequence to avoid potential dead lock

2013-11-05 Thread Gu Zheng
Hi Dan, On 11/05/2013 07:02 PM, Dan Carpenter wrote: > On Tue, Nov 05, 2013 at 06:01:00PM +0800, Gu Zheng wrote: >> The lock sequence of dcon_blank_fb(fb_info->lock ---> console_lock) is >> against >> with the one of console_callback(console_lock ---> fb_info->lock), it'll >> lead to a potential

Re: [RESEND PATCH 2/2] staging/olpc_docn: reorder the lock sequence to avoid potential dead lock

2013-11-05 Thread Dan Carpenter
On Tue, Nov 05, 2013 at 06:01:00PM +0800, Gu Zheng wrote: > The lock sequence of dcon_blank_fb(fb_info->lock ---> console_lock) is against > with the one of console_callback(console_lock ---> fb_info->lock), it'll > lead to a potential dead lock, so reorder the lock sequence of dcon_blank_fb > to

[RESEND PATCH 2/2] staging/olpc_docn: reorder the lock sequence to avoid potential dead lock

2013-11-05 Thread Gu Zheng
The lock sequence of dcon_blank_fb(fb_info->lock ---> console_lock) is against with the one of console_callback(console_lock ---> fb_info->lock), it'll lead to a potential dead lock, so reorder the lock sequence of dcon_blank_fb to avoid the potential dead lock. Signed-off-by: Gu Zheng ---

[RESEND PATCH 2/2] staging/olpc_docn: reorder the lock sequence to avoid potential dead lock

2013-11-05 Thread Gu Zheng
The lock sequence of dcon_blank_fb(fb_info-lock --- console_lock) is against with the one of console_callback(console_lock --- fb_info-lock), it'll lead to a potential dead lock, so reorder the lock sequence of dcon_blank_fb to avoid the potential dead lock. Signed-off-by: Gu Zheng

Re: [RESEND PATCH 2/2] staging/olpc_docn: reorder the lock sequence to avoid potential dead lock

2013-11-05 Thread Dan Carpenter
On Tue, Nov 05, 2013 at 06:01:00PM +0800, Gu Zheng wrote: The lock sequence of dcon_blank_fb(fb_info-lock --- console_lock) is against with the one of console_callback(console_lock --- fb_info-lock), it'll lead to a potential dead lock, so reorder the lock sequence of dcon_blank_fb to avoid

Re: [RESEND PATCH 2/2] staging/olpc_docn: reorder the lock sequence to avoid potential dead lock

2013-11-05 Thread Gu Zheng
Hi Dan, On 11/05/2013 07:02 PM, Dan Carpenter wrote: On Tue, Nov 05, 2013 at 06:01:00PM +0800, Gu Zheng wrote: The lock sequence of dcon_blank_fb(fb_info-lock --- console_lock) is against with the one of console_callback(console_lock --- fb_info-lock), it'll lead to a potential dead lock,