Re: [PATCH 0/4] per anon_vma lock and turn anon_vma rwsem lock to rwlock_t

2013-11-05 Thread Yuanhan Liu
On Tue, Nov 05, 2013 at 11:10:43AM +0800, Yuanhan Liu wrote: > On Mon, Nov 04, 2013 at 05:44:00PM -0800, Tim Chen wrote: > > On Mon, 2013-11-04 at 11:59 +0800, Yuanhan Liu wrote: > > > On Fri, Nov 01, 2013 at 08:15:13PM -0700, Davidlohr Bueso wrote: > > > > On Fri, 2013-11-01 at 18:16 +0800,

Re: [PATCH 0/4] per anon_vma lock and turn anon_vma rwsem lock to rwlock_t

2013-11-05 Thread Yuanhan Liu
On Tue, Nov 05, 2013 at 11:10:43AM +0800, Yuanhan Liu wrote: On Mon, Nov 04, 2013 at 05:44:00PM -0800, Tim Chen wrote: On Mon, 2013-11-04 at 11:59 +0800, Yuanhan Liu wrote: On Fri, Nov 01, 2013 at 08:15:13PM -0700, Davidlohr Bueso wrote: On Fri, 2013-11-01 at 18:16 +0800, Yuanhan Liu

Re: [PATCH 0/4] per anon_vma lock and turn anon_vma rwsem lock to rwlock_t

2013-11-04 Thread Yuanhan Liu
On Mon, Nov 04, 2013 at 05:44:00PM -0800, Tim Chen wrote: > On Mon, 2013-11-04 at 11:59 +0800, Yuanhan Liu wrote: > > On Fri, Nov 01, 2013 at 08:15:13PM -0700, Davidlohr Bueso wrote: > > > On Fri, 2013-11-01 at 18:16 +0800, Yuanhan Liu wrote: > > > > On Fri, Nov 01, 2013 at 09:21:46AM +0100, Ingo

Re: [PATCH 0/4] per anon_vma lock and turn anon_vma rwsem lock to rwlock_t

2013-11-04 Thread Tim Chen
On Mon, 2013-11-04 at 17:44 -0800, Tim Chen wrote: > On Mon, 2013-11-04 at 11:59 +0800, Yuanhan Liu wrote: > > On Fri, Nov 01, 2013 at 08:15:13PM -0700, Davidlohr Bueso wrote: > > > On Fri, 2013-11-01 at 18:16 +0800, Yuanhan Liu wrote: > > > > On Fri, Nov 01, 2013 at 09:21:46AM +0100, Ingo Molnar

Re: [PATCH 0/4] per anon_vma lock and turn anon_vma rwsem lock to rwlock_t

2013-11-04 Thread Tim Chen
On Mon, 2013-11-04 at 11:59 +0800, Yuanhan Liu wrote: > On Fri, Nov 01, 2013 at 08:15:13PM -0700, Davidlohr Bueso wrote: > > On Fri, 2013-11-01 at 18:16 +0800, Yuanhan Liu wrote: > > > On Fri, Nov 01, 2013 at 09:21:46AM +0100, Ingo Molnar wrote: > > > > > > > > * Yuanhan Liu wrote: > > > > > >

Re: [PATCH 0/4] per anon_vma lock and turn anon_vma rwsem lock to rwlock_t

2013-11-04 Thread Tim Chen
On Mon, 2013-11-04 at 11:59 +0800, Yuanhan Liu wrote: On Fri, Nov 01, 2013 at 08:15:13PM -0700, Davidlohr Bueso wrote: On Fri, 2013-11-01 at 18:16 +0800, Yuanhan Liu wrote: On Fri, Nov 01, 2013 at 09:21:46AM +0100, Ingo Molnar wrote: * Yuanhan Liu yuanhan@linux.intel.com wrote:

Re: [PATCH 0/4] per anon_vma lock and turn anon_vma rwsem lock to rwlock_t

2013-11-04 Thread Tim Chen
On Mon, 2013-11-04 at 17:44 -0800, Tim Chen wrote: On Mon, 2013-11-04 at 11:59 +0800, Yuanhan Liu wrote: On Fri, Nov 01, 2013 at 08:15:13PM -0700, Davidlohr Bueso wrote: On Fri, 2013-11-01 at 18:16 +0800, Yuanhan Liu wrote: On Fri, Nov 01, 2013 at 09:21:46AM +0100, Ingo Molnar wrote:

Re: [PATCH 0/4] per anon_vma lock and turn anon_vma rwsem lock to rwlock_t

2013-11-04 Thread Yuanhan Liu
On Mon, Nov 04, 2013 at 05:44:00PM -0800, Tim Chen wrote: On Mon, 2013-11-04 at 11:59 +0800, Yuanhan Liu wrote: On Fri, Nov 01, 2013 at 08:15:13PM -0700, Davidlohr Bueso wrote: On Fri, 2013-11-01 at 18:16 +0800, Yuanhan Liu wrote: On Fri, Nov 01, 2013 at 09:21:46AM +0100, Ingo Molnar

Re: [PATCH 0/4] per anon_vma lock and turn anon_vma rwsem lock to rwlock_t

2013-11-03 Thread Yuanhan Liu
On Fri, Nov 01, 2013 at 08:15:13PM -0700, Davidlohr Bueso wrote: > On Fri, 2013-11-01 at 18:16 +0800, Yuanhan Liu wrote: > > On Fri, Nov 01, 2013 at 09:21:46AM +0100, Ingo Molnar wrote: > > > > > > * Yuanhan Liu wrote: > > > > > > > > Btw., another _really_ interesting comparison would be

Re: [PATCH 0/4] per anon_vma lock and turn anon_vma rwsem lock to rwlock_t

2013-11-03 Thread Yuanhan Liu
On Fri, Nov 01, 2013 at 08:15:13PM -0700, Davidlohr Bueso wrote: On Fri, 2013-11-01 at 18:16 +0800, Yuanhan Liu wrote: On Fri, Nov 01, 2013 at 09:21:46AM +0100, Ingo Molnar wrote: * Yuanhan Liu yuanhan@linux.intel.com wrote: Btw., another _really_ interesting comparison

Re: [PATCH 0/4] per anon_vma lock and turn anon_vma rwsem lock to rwlock_t

2013-11-01 Thread Davidlohr Bueso
On Fri, 2013-11-01 at 11:55 -0700, Linus Torvalds wrote: > On Fri, Nov 1, 2013 at 11:47 AM, Michel Lespinasse wrote: > > > > Should copy Andrea on this. I talked with him during KS, and there are > > no current in-tree users who are doing such sleeping; however there > > are prospective users for

Re: [PATCH 0/4] per anon_vma lock and turn anon_vma rwsem lock to rwlock_t

2013-11-01 Thread Davidlohr Bueso
On Fri, 2013-11-01 at 18:16 +0800, Yuanhan Liu wrote: > On Fri, Nov 01, 2013 at 09:21:46AM +0100, Ingo Molnar wrote: > > > > * Yuanhan Liu wrote: > > > > > > Btw., another _really_ interesting comparison would be against > > > > the latest rwsem patches. Mind doing such a comparison? > > > >

Re: [PATCH 0/4] per anon_vma lock and turn anon_vma rwsem lock to rwlock_t

2013-11-01 Thread KOSAKI Motohiro
(11/1/13 3:54 AM), Yuanhan Liu wrote: > Patch 1 turns locking the anon_vma's root to locking itself to let it be > a per anon_vma lock, which would reduce contentions. > > In the same time, lock range becomes quite small then, which is bascially > a call of anon_vma_interval_tree_insert(). Patch

Re: [PATCH 0/4] per anon_vma lock and turn anon_vma rwsem lock to rwlock_t

2013-11-01 Thread Linus Torvalds
On Fri, Nov 1, 2013 at 11:47 AM, Michel Lespinasse wrote: > > Should copy Andrea on this. I talked with him during KS, and there are > no current in-tree users who are doing such sleeping; however there > are prospective users for networking (RDMA) or GPU stuff who want to > use this to let

Re: [PATCH 0/4] per anon_vma lock and turn anon_vma rwsem lock to rwlock_t

2013-11-01 Thread Michel Lespinasse
On Fri, Nov 1, 2013 at 11:09 AM, Linus Torvalds wrote: > On Fri, Nov 1, 2013 at 10:49 AM, Davidlohr Bueso wrote: >> >> Andrea's last input from this kind of conversion is that it cannot be >> done (at least yet): https://lkml.org/lkml/2013/9/30/53 > > No, none of the invalidate_page users really

Re: [PATCH 0/4] per anon_vma lock and turn anon_vma rwsem lock to rwlock_t

2013-11-01 Thread Linus Torvalds
On Fri, Nov 1, 2013 at 10:49 AM, Davidlohr Bueso wrote: > > Andrea's last input from this kind of conversion is that it cannot be > done (at least yet): https://lkml.org/lkml/2013/9/30/53 No, none of the invalidate_page users really need to sleep. If doing this makes some people not do stupid

Re: [PATCH 0/4] per anon_vma lock and turn anon_vma rwsem lock to rwlock_t

2013-11-01 Thread Davidlohr Bueso
On Fri, 2013-11-01 at 09:01 +0100, Ingo Molnar wrote: > * Yuanhan Liu wrote: > > > Patch 1 turns locking the anon_vma's root to locking itself to let it be > > a per anon_vma lock, which would reduce contentions. > > > > In the same time, lock range becomes quite small then, which is bascially

Re: [PATCH 0/4] per anon_vma lock and turn anon_vma rwsem lock to rwlock_t

2013-11-01 Thread Yuanhan Liu
On Fri, Nov 01, 2013 at 09:21:46AM +0100, Ingo Molnar wrote: > > * Yuanhan Liu wrote: > > > > Btw., another _really_ interesting comparison would be against > > > the latest rwsem patches. Mind doing such a comparison? > > > > Sure. Where can I get it? Are they on some git tree? > > I've

Re: [PATCH 0/4] per anon_vma lock and turn anon_vma rwsem lock to rwlock_t

2013-11-01 Thread Ingo Molnar
* Yuanhan Liu wrote: > > Btw., another _really_ interesting comparison would be against > > the latest rwsem patches. Mind doing such a comparison? > > Sure. Where can I get it? Are they on some git tree? I've Cc:-ed Tim Chen who might be able to point you to the latest version. The last

Re: [PATCH 0/4] per anon_vma lock and turn anon_vma rwsem lock to rwlock_t

2013-11-01 Thread Yuanhan Liu
On Fri, Nov 01, 2013 at 09:01:36AM +0100, Ingo Molnar wrote: > > * Yuanhan Liu wrote: > > > Patch 1 turns locking the anon_vma's root to locking itself to let it be > > a per anon_vma lock, which would reduce contentions. > > > > In the same time, lock range becomes quite small then, which is

Re: [PATCH 0/4] per anon_vma lock and turn anon_vma rwsem lock to rwlock_t

2013-11-01 Thread Ingo Molnar
* Yuanhan Liu wrote: > Patch 1 turns locking the anon_vma's root to locking itself to let it be > a per anon_vma lock, which would reduce contentions. > > In the same time, lock range becomes quite small then, which is bascially > a call of anon_vma_interval_tree_insert(). Patch 2 turn rwsem

[PATCH 0/4] per anon_vma lock and turn anon_vma rwsem lock to rwlock_t

2013-11-01 Thread Yuanhan Liu
Patch 1 turns locking the anon_vma's root to locking itself to let it be a per anon_vma lock, which would reduce contentions. In the same time, lock range becomes quite small then, which is bascially a call of anon_vma_interval_tree_insert(). Patch 2 turn rwsem to rwlock_t. It's a patch made from

[PATCH 0/4] per anon_vma lock and turn anon_vma rwsem lock to rwlock_t

2013-11-01 Thread Yuanhan Liu
Patch 1 turns locking the anon_vma's root to locking itself to let it be a per anon_vma lock, which would reduce contentions. In the same time, lock range becomes quite small then, which is bascially a call of anon_vma_interval_tree_insert(). Patch 2 turn rwsem to rwlock_t. It's a patch made from

Re: [PATCH 0/4] per anon_vma lock and turn anon_vma rwsem lock to rwlock_t

2013-11-01 Thread Ingo Molnar
* Yuanhan Liu yuanhan@linux.intel.com wrote: Patch 1 turns locking the anon_vma's root to locking itself to let it be a per anon_vma lock, which would reduce contentions. In the same time, lock range becomes quite small then, which is bascially a call of

Re: [PATCH 0/4] per anon_vma lock and turn anon_vma rwsem lock to rwlock_t

2013-11-01 Thread Yuanhan Liu
On Fri, Nov 01, 2013 at 09:01:36AM +0100, Ingo Molnar wrote: * Yuanhan Liu yuanhan@linux.intel.com wrote: Patch 1 turns locking the anon_vma's root to locking itself to let it be a per anon_vma lock, which would reduce contentions. In the same time, lock range becomes quite small

Re: [PATCH 0/4] per anon_vma lock and turn anon_vma rwsem lock to rwlock_t

2013-11-01 Thread Ingo Molnar
* Yuanhan Liu yuanhan@linux.intel.com wrote: Btw., another _really_ interesting comparison would be against the latest rwsem patches. Mind doing such a comparison? Sure. Where can I get it? Are they on some git tree? I've Cc:-ed Tim Chen who might be able to point you to the latest

Re: [PATCH 0/4] per anon_vma lock and turn anon_vma rwsem lock to rwlock_t

2013-11-01 Thread Yuanhan Liu
On Fri, Nov 01, 2013 at 09:21:46AM +0100, Ingo Molnar wrote: * Yuanhan Liu yuanhan@linux.intel.com wrote: Btw., another _really_ interesting comparison would be against the latest rwsem patches. Mind doing such a comparison? Sure. Where can I get it? Are they on some git tree?

Re: [PATCH 0/4] per anon_vma lock and turn anon_vma rwsem lock to rwlock_t

2013-11-01 Thread Davidlohr Bueso
On Fri, 2013-11-01 at 09:01 +0100, Ingo Molnar wrote: * Yuanhan Liu yuanhan@linux.intel.com wrote: Patch 1 turns locking the anon_vma's root to locking itself to let it be a per anon_vma lock, which would reduce contentions. In the same time, lock range becomes quite small then,

Re: [PATCH 0/4] per anon_vma lock and turn anon_vma rwsem lock to rwlock_t

2013-11-01 Thread Linus Torvalds
On Fri, Nov 1, 2013 at 10:49 AM, Davidlohr Bueso davidl...@hp.com wrote: Andrea's last input from this kind of conversion is that it cannot be done (at least yet): https://lkml.org/lkml/2013/9/30/53 No, none of the invalidate_page users really need to sleep. If doing this makes some people not

Re: [PATCH 0/4] per anon_vma lock and turn anon_vma rwsem lock to rwlock_t

2013-11-01 Thread Michel Lespinasse
On Fri, Nov 1, 2013 at 11:09 AM, Linus Torvalds torva...@linux-foundation.org wrote: On Fri, Nov 1, 2013 at 10:49 AM, Davidlohr Bueso davidl...@hp.com wrote: Andrea's last input from this kind of conversion is that it cannot be done (at least yet): https://lkml.org/lkml/2013/9/30/53 No, none

Re: [PATCH 0/4] per anon_vma lock and turn anon_vma rwsem lock to rwlock_t

2013-11-01 Thread Linus Torvalds
On Fri, Nov 1, 2013 at 11:47 AM, Michel Lespinasse wal...@google.com wrote: Should copy Andrea on this. I talked with him during KS, and there are no current in-tree users who are doing such sleeping; however there are prospective users for networking (RDMA) or GPU stuff who want to use this

Re: [PATCH 0/4] per anon_vma lock and turn anon_vma rwsem lock to rwlock_t

2013-11-01 Thread KOSAKI Motohiro
(11/1/13 3:54 AM), Yuanhan Liu wrote: Patch 1 turns locking the anon_vma's root to locking itself to let it be a per anon_vma lock, which would reduce contentions. In the same time, lock range becomes quite small then, which is bascially a call of anon_vma_interval_tree_insert(). Patch 2

Re: [PATCH 0/4] per anon_vma lock and turn anon_vma rwsem lock to rwlock_t

2013-11-01 Thread Davidlohr Bueso
On Fri, 2013-11-01 at 18:16 +0800, Yuanhan Liu wrote: On Fri, Nov 01, 2013 at 09:21:46AM +0100, Ingo Molnar wrote: * Yuanhan Liu yuanhan@linux.intel.com wrote: Btw., another _really_ interesting comparison would be against the latest rwsem patches. Mind doing such a

Re: [PATCH 0/4] per anon_vma lock and turn anon_vma rwsem lock to rwlock_t

2013-11-01 Thread Davidlohr Bueso
On Fri, 2013-11-01 at 11:55 -0700, Linus Torvalds wrote: On Fri, Nov 1, 2013 at 11:47 AM, Michel Lespinasse wal...@google.com wrote: Should copy Andrea on this. I talked with him during KS, and there are no current in-tree users who are doing such sleeping; however there are prospective