Re: [PATCH] Reduce cpuset.c write_lock_irq() to read_lock()

2007-05-23 Thread Paul Jackson
Paul M wrote: > cpuset.c:update_nodemask() uses a write_lock_irq() on tasklist_lock to > block concurrent forks; a read_lock() suffices and is less intrusive. Seems reasonable to me - thanks. > - write_lock_irq(_lock); /* block fork */ > + read_lock(_lock);

[PATCH] Reduce cpuset.c write_lock_irq() to read_lock()

2007-05-23 Thread Paul Menage
cpuset.c:update_nodemask() uses a write_lock_irq() on tasklist_lock to block concurrent forks; a read_lock() suffices and is less intrusive. Signed-off-by: Paul Menage<[EMAIL PROTECTED]> --- kernel/cpuset.c |6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) Index:

[PATCH] Reduce cpuset.c write_lock_irq() to read_lock()

2007-05-23 Thread Paul Menage
cpuset.c:update_nodemask() uses a write_lock_irq() on tasklist_lock to block concurrent forks; a read_lock() suffices and is less intrusive. Signed-off-by: Paul Menage[EMAIL PROTECTED] --- kernel/cpuset.c |6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) Index:

Re: [PATCH] Reduce cpuset.c write_lock_irq() to read_lock()

2007-05-23 Thread Paul Jackson
Paul M wrote: cpuset.c:update_nodemask() uses a write_lock_irq() on tasklist_lock to block concurrent forks; a read_lock() suffices and is less intrusive. Seems reasonable to me - thanks. - write_lock_irq(tasklist_lock); /* block fork */ +