Re: [prepatch] 2.4 waitqueues

2000-12-30 Thread Andrew Morton
Linus Torvalds wrote: > > On Wed, 27 Dec 2000, Andrew Morton wrote: > > > > - Introduces a kernel-wide macro `SMP_KERNEL'. This is designed to > > be used as a `compiled ifdef' in place of `#ifdef CONFIG_SMP'. There > > are a few examples in __wake_up_common(). > > Please don't do this,

Re: [prepatch] 2.4 waitqueues

2000-12-30 Thread Andrew Morton
Linus Torvalds wrote: On Wed, 27 Dec 2000, Andrew Morton wrote: - Introduces a kernel-wide macro `SMP_KERNEL'. This is designed to be used as a `compiled ifdef' in place of `#ifdef CONFIG_SMP'. There are a few examples in __wake_up_common(). Please don't do this, OK. So

Re: [prepatch] 2.4 waitqueues

2000-12-28 Thread Linus Torvalds
On Wed, 27 Dec 2000, Andrew Morton wrote: > > - Introduces a kernel-wide macro `SMP_KERNEL'. This is designed to > be used as a `compiled ifdef' in place of `#ifdef CONFIG_SMP'. There > are a few examples in __wake_up_common(). Please don't do this, it screws up the config option

Re: [prepatch] 2.4 waitqueues

2000-12-28 Thread Linus Torvalds
On Wed, 27 Dec 2000, Andrew Morton wrote: - Introduces a kernel-wide macro `SMP_KERNEL'. This is designed to be used as a `compiled ifdef' in place of `#ifdef CONFIG_SMP'. There are a few examples in __wake_up_common(). Please don't do this, it screws up the config option

Re: [prepatch] 2.4 waitqueues

2000-12-27 Thread george anzinger
Andrew Morton wrote: > > It's been quiet around here lately... > > This is a rework of the 2.4 wakeup code based on the discussions Andrea > and I had last week. There were two basic problems: > > - If two tasks are on a waitqueue in exclusive mode and one gets > woken, it will put itself

Re: [prepatch] 2.4 waitqueues

2000-12-27 Thread Andrea Arcangeli
On Wed, Dec 27, 2000 at 10:45:29PM +1100, Andrew Morton wrote: > Andrea Arcangeli wrote: > > > > > > Not a big deal but still I'd prefer the CONFIG_SMP #ifdef though, it looks even > > more obvious that it's a compile check and at least in your usage I cannot see > > a relevant readability

Re: [prepatch] 2.4 waitqueues

2000-12-27 Thread Andrew Morton
Andrea Arcangeli wrote: > > > Not a big deal but still I'd prefer the CONFIG_SMP #ifdef though, it looks even > more obvious that it's a compile check and at least in your usage I cannot see > a relevant readability advantage. And my own feeling is not having to rely on > more things to produce

Re: [prepatch] 2.4 waitqueues

2000-12-27 Thread Andrew Morton
Andrea Arcangeli wrote: Not a big deal but still I'd prefer the CONFIG_SMP #ifdef though, it looks even more obvious that it's a compile check and at least in your usage I cannot see a relevant readability advantage. And my own feeling is not having to rely on more things to produce the

Re: [prepatch] 2.4 waitqueues

2000-12-27 Thread Andrea Arcangeli
On Wed, Dec 27, 2000 at 10:45:29PM +1100, Andrew Morton wrote: Andrea Arcangeli wrote: Not a big deal but still I'd prefer the CONFIG_SMP #ifdef though, it looks even more obvious that it's a compile check and at least in your usage I cannot see a relevant readability advantage. And

Re: [prepatch] 2.4 waitqueues

2000-12-27 Thread george anzinger
Andrew Morton wrote: It's been quiet around here lately... This is a rework of the 2.4 wakeup code based on the discussions Andrea and I had last week. There were two basic problems: - If two tasks are on a waitqueue in exclusive mode and one gets woken, it will put itself back into

Re: [prepatch] 2.4 waitqueues

2000-12-26 Thread Andrea Arcangeli
On Wed, Dec 27, 2000 at 01:57:12PM +1100, Andrew Morton wrote: > Oh, it's all still there, but it's now all in the header file: > > #ifdef DEBUG > #define foo() printk(stuff) > #else > #define foo() > #endif I intentionally didn't focused on such part of your patch because I understood from the

Re: [prepatch] 2.4 waitqueues

2000-12-26 Thread Andrew Morton
Andrea Arcangeli wrote: > > On Wed, Dec 27, 2000 at 11:29:06AM +1100, Andrew Morton wrote: > > - Got rid of all the debugging ifdefs - these have been folded into > > wait.h > > Why? Such debugging code is just disabled so it doesn't get compiled in, but if > somebody wants he can enable it

Re: [prepatch] 2.4 waitqueues

2000-12-26 Thread Andrea Arcangeli
On Wed, Dec 27, 2000 at 11:29:06AM +1100, Andrew Morton wrote: > - Got rid of all the debugging ifdefs - these have been folded into > wait.h Why? Such debugging code is just disabled so it doesn't get compiled in, but if somebody wants he can enable it changing the #define in the sources to

[prepatch] 2.4 waitqueues

2000-12-26 Thread Andrew Morton
It's been quiet around here lately... This is a rework of the 2.4 wakeup code based on the discussions Andrea and I had last week. There were two basic problems: - If two tasks are on a waitqueue in exclusive mode and one gets woken, it will put itself back into TASK_[UN]INTERRUPTIBLE state

[prepatch] 2.4 waitqueues

2000-12-26 Thread Andrew Morton
It's been quiet around here lately... This is a rework of the 2.4 wakeup code based on the discussions Andrea and I had last week. There were two basic problems: - If two tasks are on a waitqueue in exclusive mode and one gets woken, it will put itself back into TASK_[UN]INTERRUPTIBLE state

Re: [prepatch] 2.4 waitqueues

2000-12-26 Thread Andrea Arcangeli
On Wed, Dec 27, 2000 at 11:29:06AM +1100, Andrew Morton wrote: - Got rid of all the debugging ifdefs - these have been folded into wait.h Why? Such debugging code is just disabled so it doesn't get compiled in, but if somebody wants he can enable it changing the #define in the sources to

Re: [prepatch] 2.4 waitqueues

2000-12-26 Thread Andrea Arcangeli
On Wed, Dec 27, 2000 at 01:57:12PM +1100, Andrew Morton wrote: Oh, it's all still there, but it's now all in the header file: #ifdef DEBUG #define foo() printk(stuff) #else #define foo() #endif I intentionally didn't focused on such part of your patch because I understood from the