Re: [PATCH] memcgroup: remove a useless VM_BUG_ON()

2008-02-18 Thread KAMEZAWA Hiroyuki
On Mon, 18 Feb 2008 15:34:48 +0800 Li Zefan <[EMAIL PROTECTED]> wrote: > Remove this VM_BUG_ON(), as Balbir stated: > > We used to have a for loop with !list_empty() as a termination condition > and VM_BUG_ON(!pc) is a spill over. With the new loop, VM_BUG_ON(!pc) does not > make sense. > >

Re: [PATCH] memcgroup: remove a useless VM_BUG_ON()

2008-02-18 Thread Balbir Singh
Li Zefan wrote: > Remove this VM_BUG_ON(), as Balbir stated: > > We used to have a for loop with !list_empty() as a termination condition > and VM_BUG_ON(!pc) is a spill over. With the new loop, VM_BUG_ON(!pc) does not > make sense. > > Signed-off-by: Li Zefan <[EMAIL PROTECTED]> > --- >

Re: [PATCH] memcgroup: remove a useless VM_BUG_ON()

2008-02-18 Thread Balbir Singh
Li Zefan wrote: Remove this VM_BUG_ON(), as Balbir stated: We used to have a for loop with !list_empty() as a termination condition and VM_BUG_ON(!pc) is a spill over. With the new loop, VM_BUG_ON(!pc) does not make sense. Signed-off-by: Li Zefan [EMAIL PROTECTED] --- mm/memcontrol.c |

Re: [PATCH] memcgroup: remove a useless VM_BUG_ON()

2008-02-18 Thread KAMEZAWA Hiroyuki
On Mon, 18 Feb 2008 15:34:48 +0800 Li Zefan [EMAIL PROTECTED] wrote: Remove this VM_BUG_ON(), as Balbir stated: We used to have a for loop with !list_empty() as a termination condition and VM_BUG_ON(!pc) is a spill over. With the new loop, VM_BUG_ON(!pc) does not make sense.

[PATCH] memcgroup: remove a useless VM_BUG_ON()

2008-02-17 Thread Li Zefan
Remove this VM_BUG_ON(), as Balbir stated: We used to have a for loop with !list_empty() as a termination condition and VM_BUG_ON(!pc) is a spill over. With the new loop, VM_BUG_ON(!pc) does not make sense. Signed-off-by: Li Zefan <[EMAIL PROTECTED]> --- mm/memcontrol.c |1 - 1 files

[PATCH] memcgroup: remove a useless VM_BUG_ON()

2008-02-17 Thread Li Zefan
Remove this VM_BUG_ON(), as Balbir stated: We used to have a for loop with !list_empty() as a termination condition and VM_BUG_ON(!pc) is a spill over. With the new loop, VM_BUG_ON(!pc) does not make sense. Signed-off-by: Li Zefan [EMAIL PROTECTED] --- mm/memcontrol.c |1 - 1 files changed,