[PATCH RESEND] Fix a dead loop in async_synchronize_full()

2012-07-16 Thread Li Zhong
resend it again with the email client fixed... in case it is needed This patch tries to fix a dead loop in async_synchronize_full(), which could be seen when preemption is disabled on a single cpu machine. void async_synchronize_full(void) { do { async_synchronize_cookie

Re: [PATCH RESEND] Fix a dead loop in async_synchronize_full()

2012-07-16 Thread Paul E. McKenney
On Mon, Jul 16, 2012 at 04:01:33PM -0700, Dan Williams wrote: > On Mon, Jul 16, 2012 at 11:32 AM, Christian Kujau > wrote: > > Is this still scheduled to go into 3.5? I'm asking because -rc7 has been > > released and does not contain this fix. W/o this fix, my powerpc system > > won't boot[0] :-\

Re: [PATCH RESEND] Fix a dead loop in async_synchronize_full()

2012-07-16 Thread Dan Williams
On Mon, Jul 16, 2012 at 11:32 AM, Christian Kujau wrote: > Is this still scheduled to go into 3.5? I'm asking because -rc7 has been > released and does not contain this fix. W/o this fix, my powerpc system > won't boot[0] :-\ I don't expect James is going to push my async changes for 3.5. So may

Re: [PATCH RESEND] Fix a dead loop in async_synchronize_full()

2012-07-16 Thread Christian Kujau
On Thu, 12 Jul 2012 at 17:56, Li Zhong wrote: > On Wed, 2012-07-11 at 15:50 -0700, Dan Williams wrote: > > On Wed, Jul 11, 2012 at 3:42 PM, Andrew Morton > > wrote: > > > The patch is fairly wordwrapped - please fix up your email client. > > > > > > More seriously, it does not apply to linux-next

Re: [PATCH RESEND] Fix a dead loop in async_synchronize_full()

2012-07-12 Thread Dan Williams
[ adding James ] On Thu, Jul 12, 2012 at 2:56 AM, Li Zhong wrote: > I have tested your pending patches, they fix the problem here. Thanks! James, if you get the chance please add: Tested-by: Li Zhong ...to the pending set, or I can just resend. Let me know. > But with ASYNC_DOMAIN_EXCLUSIV

Re: [PATCH RESEND] Fix a dead loop in async_synchronize_full()

2012-07-12 Thread Li Zhong
On Wed, 2012-07-11 at 15:50 -0700, Dan Williams wrote: > On Wed, Jul 11, 2012 at 3:42 PM, Andrew Morton > wrote: > > The patch is fairly wordwrapped - please fix up your email client. > > > > More seriously, it does not apply to linux-next due to some fairly > > significant changes which have been

Re: [PATCH RESEND] Fix a dead loop in async_synchronize_full()

2012-07-11 Thread Li Zhong
On Wed, 2012-07-11 at 15:42 -0700, Andrew Morton wrote: > On Mon, 09 Jul 2012 15:04:25 +0800 > Li Zhong wrote: > > > This patch tries to fix a dead loop in async_synchronize_full(), which > > could be seen when preemption is disabled on a single cpu machine. > > > > void async_synchronize_full

Re: [PATCH RESEND] Fix a dead loop in async_synchronize_full()

2012-07-11 Thread Dan Williams
On Wed, Jul 11, 2012 at 3:42 PM, Andrew Morton wrote: > The patch is fairly wordwrapped - please fix up your email client. > > More seriously, it does not apply to linux-next due to some fairly > significant changes which have been sitting in Dan's tree since May. > What's going on? > Those chang

Re: [PATCH RESEND] Fix a dead loop in async_synchronize_full()

2012-07-11 Thread Andrew Morton
On Mon, 09 Jul 2012 15:04:25 +0800 Li Zhong wrote: > This patch tries to fix a dead loop in async_synchronize_full(), which > could be seen when preemption is disabled on a single cpu machine. > > void async_synchronize_full(void) > { > do { > async_synchronize_cookie(n

[PATCH RESEND] Fix a dead loop in async_synchronize_full()

2012-07-09 Thread Li Zhong
This patch tries to fix a dead loop in async_synchronize_full(), which could be seen when preemption is disabled on a single cpu machine. void async_synchronize_full(void) { do { async_synchronize_cookie(next_cookie); } while (!list_empty(&async_running) || ! list