Re: [PATCH] mm: remove global locks from mm/highmem.c

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Christoph Hellwig <hch@...>, Peter Zijlstra <a.p.zijlstra@...>, <linux-kernel@...>, <linux-mm@...>, Linus Torvalds <torvalds@...>, Andrew Morton <akpm@...>
Date: Sunday, January 28, 2007 - 11:48 am

* Christoph Hellwig <hch@infradead.org> wrote:


i'm sorry, but do you realize that files_lock is a global lock, 
triggered by /every single/ file close?

   " files_lock is a global lock and we touch it for every single
     sys_close() system call that the system does. "

You really dont need to be a rocket scientist to see that it's a 
globally bouncing cacheline that has a major effect on certain 
VFS-intense workloads. Peter has worked hard to eliminate its effects 
without having to couple this to an intrusive rewrite of the TTY layer.

( really, i personally find your dismissive style apalling and i think 
  such a reception of a nice patchset must be humiliating to Peter. I
  certainly try to avoid to be involved with any VFS internals, due to
  this unwelcoming tone of discussion. Had you been around when i
  started contributing to the Linux kernel i'd probably not be hacking
  the kernel today. You are a good hacker but the simultaneous
  collateral damage you are causing is significantly reducing the net
  benefit. )


( You are messing up the reply headers, everyone is listed in the 'To:'
  field for any reply to your mail, instead of being added to the Cc:
  list. )

	Ingo
-
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH] mm: remove global locks from mm/highmem.c, Peter Zijlstra, (Sun Jan 28, 10:11 am)
Re: [PATCH] mm: remove global locks from mm/highmem.c, Andrew Morton, (Sun Jan 28, 6:29 pm)
Re: [PATCH] mm: remove global locks from mm/highmem.c, Ingo Molnar, (Mon Jan 29, 3:08 pm)
Re: [PATCH] mm: remove global locks from mm/highmem.c, Nick Piggin, (Mon Jan 29, 10:02 pm)
Re: [PATCH] mm: remove global locks from mm/highmem.c, Ingo Molnar, (Mon Jan 29, 4:06 pm)
Re: [PATCH] mm: remove global locks from mm/highmem.c, Hugh Dickins, (Mon Jan 29, 3:19 pm)
Re: [PATCH] mm: remove global locks from mm/highmem.c, Ingo Molnar, (Mon Jan 29, 3:53 pm)
Re: [PATCH] mm: remove global locks from mm/highmem.c, Peter Zijlstra, (Mon Jan 29, 5:44 am)
Re: [PATCH] mm: remove global locks from mm/highmem.c, Martin J. Bligh, (Mon Jan 29, 9:31 pm)
Re: [PATCH] mm: remove global locks from mm/highmem.c, Andrew Morton, (Mon Jan 29, 9:41 pm)
Re: [PATCH] mm: remove global locks from mm/highmem.c, Martin J. Bligh, (Mon Jan 29, 9:49 pm)
Re: [PATCH] mm: remove global locks from mm/highmem.c, Andrew Morton, (Mon Jan 29, 10:15 pm)
Re: [PATCH] mm: remove global locks from mm/highmem.c, David Chinner, (Tue Jan 30, 8:44 pm)
Re: [PATCH] mm: remove global locks from mm/highmem.c, Andrew Morton, (Tue Jan 30, 9:11 pm)
Re: [PATCH] mm: remove global locks from mm/highmem.c, David Chinner, (Tue Jan 30, 11:22 pm)
Re: [PATCH] mm: remove global locks from mm/highmem.c, Christoph Hellwig, (Fri Feb 2, 8:05 am)
Re: [PATCH] mm: remove global locks from mm/highmem.c, David Chinner, (Fri Feb 2, 7:14 pm)
Re: [PATCH] mm: remove global locks from mm/highmem.c, Matt Mackall, (Fri Feb 2, 3:24 pm)
Re: [PATCH] mm: remove global locks from mm/highmem.c, David Chinner, (Fri Feb 2, 7:16 pm)
Re: [PATCH] mm: remove global locks from mm/highmem.c, Nick Piggin, (Sun Jan 28, 10:52 pm)
Re: [PATCH] mm: remove global locks from mm/highmem.c, Christoph Hellwig, (Sun Jan 28, 10:49 am)
Re: [PATCH] mm: remove global locks from mm/highmem.c, Ingo Molnar, (Sun Jan 28, 11:17 am)
Re: [PATCH] mm: remove global locks from mm/highmem.c, Christoph Hellwig, (Sun Jan 28, 11:28 am)
Re: [PATCH] mm: remove global locks from mm/highmem.c, Ingo Molnar, (Sun Jan 28, 11:48 am)
Re: [PATCH] mm: remove global locks from mm/highmem.c, Christoph Hellwig, (Sun Jan 28, 11:54 am)
Re: [PATCH] mm: remove global locks from mm/highmem.c, Ingo Molnar, (Sun Jan 28, 2:19 pm)