On Tue, 9 Mar 2010 09:18:45 +0900 Daisuke Nishimura <nishimura@xxxxxxxxxxxxxxxxx> wrote: > On Mon, 8 Mar 2010 17:31:00 +0900, KAMEZAWA Hiroyuki <kamezawa.hiroyu@xxxxxxxxxxxxxx> wrote: > > On Mon, 8 Mar 2010 17:07:11 +0900 > > Daisuke Nishimura <nishimura@xxxxxxxxxxxxxxxxx> wrote: > > Hmm...accepatable ? (sounds it's in error-range) > > > > BTW, why local_irq_disable() ? > > local_irq_save()/restore() isn't better ? > > > I don't have any strong reason. All of lock_page_cgroup() is *now* called w/o irq disabled, > so I used just disable()/enable() instead of save()/restore(). My point is, this will be used under treelock soon. Thanks, -Kame -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxxx For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>