Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1545982
| From | Linus Torvalds <torvalds@linux-foundation.org> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: [RFC][PATCH] make global bitlock waitqueues per-node |
| Date | 2016-12-21 21:00 +0100 |
| Message-ID | <sQVdo-7Ef-9@gated-at.bofh.it> (permalink) |
| References | (5 earlier) <sQK8h-Qv-1@gated-at.bofh.it> <sQKBj-10r-9@gated-at.bofh.it> <sQTuV-6O0-3@gated-at.bofh.it> <sQTXX-6Xu-3@gated-at.bofh.it> <sQUr0-7n3-7@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
On Wed, Dec 21, 2016 at 11:01 AM, Nicholas Piggin <npiggin@gmail.com> wrote:
> Peter's patch is less code and in that regard a bit nicer. I tried
> going that way once, but I just thought it was a bit too sloppy to
> do nicely with wait bit APIs.
So I have to admit that when I read through your and PeterZ's patches
back-to-back, yours was easier to understand.
PeterZ's is smaller but kind of subtle. The whole "return zero from
lock_page_wait() and go around again" and the locking around that
isn't exactly clear. In contrast, yours has the obvious waitqueue
spinlock.
I'll think about it. And yes, it would be good to have more testing,
but at the same time xmas is imminent, and waiting around too much
isn't going to help either..
Linus
Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread
[RFC][PATCH] make global bitlock waitqueues per-node Dave Hansen <dave.hansen@linux.intel.com> - 2016-12-20 00:00 +0100
Re: [RFC][PATCH] make global bitlock waitqueues per-node Dave Hansen <dave.hansen@linux.intel.com> - 2016-12-20 01:30 +0100
Re: [RFC][PATCH] make global bitlock waitqueues per-node Nicholas Piggin <npiggin@gmail.com> - 2016-12-20 03:40 +0100
Re: [RFC][PATCH] make global bitlock waitqueues per-node Mel Gorman <mgorman@techsingularity.net> - 2016-12-20 14:00 +0100
Re: [RFC][PATCH] make global bitlock waitqueues per-node Nicholas Piggin <npiggin@gmail.com> - 2016-12-20 14:30 +0100
Re: [RFC][PATCH] make global bitlock waitqueues per-node Linus Torvalds <torvalds@linux-foundation.org> - 2016-12-20 18:40 +0100
Re: [RFC][PATCH] make global bitlock waitqueues per-node Linus Torvalds <torvalds@linux-foundation.org> - 2016-12-20 19:10 +0100
Re: [RFC][PATCH] make global bitlock waitqueues per-node Peter Zijlstra <peterz@infradead.org> - 2016-12-21 09:10 +0100
Re: [RFC][PATCH] make global bitlock waitqueues per-node Peter Zijlstra <peterz@infradead.org> - 2016-12-21 09:40 +0100
Re: [RFC][PATCH] make global bitlock waitqueues per-node Linus Torvalds <torvalds@linux-foundation.org> - 2016-12-21 19:10 +0100
Re: [RFC][PATCH] make global bitlock waitqueues per-node Nicholas Piggin <npiggin@gmail.com> - 2016-12-21 19:40 +0100
Re: [RFC][PATCH] make global bitlock waitqueues per-node Nicholas Piggin <npiggin@gmail.com> - 2016-12-21 20:10 +0100
Re: [RFC][PATCH] make global bitlock waitqueues per-node Linus Torvalds <torvalds@linux-foundation.org> - 2016-12-21 21:00 +0100
Re: [RFC][PATCH] make global bitlock waitqueues per-node Nicholas Piggin <npiggin@gmail.com> - 2016-12-22 03:10 +0100
Re: [RFC][PATCH] make global bitlock waitqueues per-node Hugh Dickins <hughd@google.com> - 2016-12-22 20:30 +0100
Re: [RFC][PATCH] make global bitlock waitqueues per-node Nicholas Piggin <npiggin@gmail.com> - 2016-12-21 11:30 +0100
Re: [RFC][PATCH] make global bitlock waitqueues per-node Nicholas Piggin <npiggin@gmail.com> - 2016-12-20 03:30 +0100
Re: [RFC][PATCH] make global bitlock waitqueues per-node Nicholas Piggin <npiggin@gmail.com> - 2016-12-21 13:40 +0100
Re: [RFC][PATCH] make global bitlock waitqueues per-node Linus Torvalds <torvalds@linux-foundation.org> - 2016-12-21 19:20 +0100
Re: [RFC][PATCH] make global bitlock waitqueues per-node Nicholas Piggin <npiggin@gmail.com> - 2016-12-21 19:50 +0100
csiph-web