Re: rwsem: down_read_unfair() proposal

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Michel Lespinasse
Date: Wednesday, May 5, 2010 - 3:36 am

On Wed, May 05, 2010 at 11:03:40AM +0100, David Howells wrote:

Reader/Writer fairness is not disabled in the general case - it only is
for a few specific readers such as /proc/<pid>/maps. In particular, the
do_page_fault path, which holds a read lock on mmap_sem for potentially long
(~disk latency) periods of times, still uses a fair down_read() call.
In comparison, the /proc/<pid>/maps path which we made unfair does not
normally hold the mmap_sem for very long (it does not end up hitting disk);
so it's been working out well for us in practice.

-- 
Michel "Walken" Lespinasse
A program is never fully debugged until the last user dies.
--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
rwsem: down_read_unfair() proposal, Michel Lespinasse, (Tue May 4, 8:20 pm)
Re: rwsem: down_read_unfair() proposal, David Howells, (Wed May 5, 3:03 am)
Re: rwsem: down_read_unfair() proposal, David Howells, (Wed May 5, 3:06 am)
Re: rwsem: down_read_unfair() proposal, Michel Lespinasse, (Wed May 5, 3:36 am)
Re: rwsem: down_read_unfair() proposal, Michel Lespinasse, (Wed May 5, 3:48 am)
Re: rwsem: down_read_unfair() proposal, David Howells, (Wed May 5, 4:09 am)
Re: rwsem: down_read_unfair() proposal, Mike Waychison, (Thu May 6, 4:26 pm)