From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754430Ab0ELDW1 (ORCPT ); Tue, 11 May 2010 23:22:27 -0400 Received: from smtp-out.google.com ([74.125.121.35]:52308 "EHLO smtp-out.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754368Ab0ELDWY (ORCPT ); Tue, 11 May 2010 23:22:24 -0400 DomainKey-Signature: a=rsa-sha1; s=beta; d=google.com; c=nofws; q=dns; h=from:to:cc:subject:date:message-id:x-mailer:in-reply-to:references; b=dTxgHQ9hRC8MdCKVNu8uSaYw6FyuOwoXbCTcvuRv+deKs46UGJP3BC/x1te1/QoaF uyxkesUPl3NL0tT8ScRsQ== From: Michel Lespinasse To: Linus Torvalds , David Howells , Ingo Molnar , Thomas Gleixner Cc: LKML , Andrew Morton , Mike Waychison , Suleiman Souhlal , Ying Han , Michel Lespinasse Subject: [PATCH 12/12] Use down_read_unfair() for /sys//exe and /sys//maps files Date: Tue, 11 May 2010 20:21:02 -0700 Message-Id: <1273634462-2672-13-git-send-email-walken@google.com> X-Mailer: git-send-email 1.7.0.1 In-Reply-To: <1273634462-2672-1-git-send-email-walken@google.com> References: <1273634462-2672-1-git-send-email-walken@google.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org This helps in the following situation: - Thread A takes a page fault while reading or writing memory. do_page_fault() acquires the mmap_sem for read and blocks on disk (either reading the page from file, or hitting swap) for a long time. - Thread B does an mmap call and blocks trying to acquire the mmap_sem for write - Thread C is a monitoring process trying to read every /proc/pid/maps in the system. This requires acquiring the mmap_sem for read. Thread C blocks behind B, waiting for A to release the rwsem. If thread C could be allowed to run in parallel with A, it would probably get done long before thread A's disk access completes, thus not actually slowing down thread B. Test results with down_read_unfair_test (10 seconds): 2.6.33.3: threadA completes ~600 faults threadB completes ~300 mmap/munmap cycles threadC completes ~600 /proc/pid/maps reads 2.6.33.3 + down_read_unfair: threadA completes ~600 faults threadB completes ~300 mmap/munmap cycles threadC completes ~160000 /proc/pid/maps reads Signed-off-by: Michel Lespinasse --- fs/proc/base.c | 2 +- fs/proc/task_mmu.c | 2 +- fs/proc/task_nommu.c | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/fs/proc/base.c b/fs/proc/base.c index 8418fcc..9132488 100644 --- a/fs/proc/base.c +++ b/fs/proc/base.c @@ -1367,7 +1367,7 @@ struct file *get_mm_exe_file(struct mm_struct *mm) /* We need mmap_sem to protect against races with removal of * VM_EXECUTABLE vmas */ - down_read(&mm->mmap_sem); + down_read_unfair(&mm->mmap_sem); exe_file = mm->exe_file; if (exe_file) get_file(exe_file); diff --git a/fs/proc/task_mmu.c b/fs/proc/task_mmu.c index 0705534..09647ad 100644 --- a/fs/proc/task_mmu.c +++ b/fs/proc/task_mmu.c @@ -123,7 +123,7 @@ static void *m_start(struct seq_file *m, loff_t *pos) mm = mm_for_maps(priv->task); if (!mm) return NULL; - down_read(&mm->mmap_sem); + down_read_unfair(&mm->mmap_sem); tail_vma = get_gate_vma(priv->task); priv->tail_vma = tail_vma; diff --git a/fs/proc/task_nommu.c b/fs/proc/task_nommu.c index 46d4b5d..56ca830 100644 --- a/fs/proc/task_nommu.c +++ b/fs/proc/task_nommu.c @@ -194,7 +194,7 @@ static void *m_start(struct seq_file *m, loff_t *pos) priv->task = NULL; return NULL; } - down_read(&mm->mmap_sem); + down_read_unfair(&mm->mmap_sem); /* start from the Nth VMA */ for (p = rb_first(&mm->mm_rb); p; p = rb_next(p)) -- 1.7.0.1