Messages in this thread | | | From | Linus Torvalds <> | Date | Sat, 14 Apr 2012 10:08:54 -0700 | Subject | Re: [PATCH RFC 0/7] rcu: v2 Inlinable preemptible rcu_read_lock() and rcu_read_unlock() |
| |
On Sat, Apr 14, 2012 at 9:58 AM, Linus Torvalds <torvalds@linux-foundation.org> wrote: > > I'll see if it boots and what it does to my profiles and > microbenchmark, though.
Ok, I can't see any performance difference in the numbers - my benchmark noise is *much* bigger than anything this would show.
The profile looks fine, and obviously __rcu_read_lock() is entirely gone. The top user (avc_has_perm_flags()) looks fine. I note that you might want to look at the placement of the percpu data - I think it probably makes sense to put the RCU data close to 'current' etc to get as much cacheline sharing as possible, and it doesn't seem to be right now, but it looks reasonable.
But on the whole, I can't claim that it looks noticeable ;*(
Linus
| |