lkml.org 
[lkml]   [2015]   [Jun]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [RFC][PATCH] fs: optimize inotify/fsnotify code for unwatched files
On Fri, Jun 19, 2015 at 05:39:11PM -0700, Dave Hansen wrote:
> On 06/19/2015 04:33 PM, Andi Kleen wrote:
> >> > I *think* we can avoid taking the srcu_read_lock() for the
> >> > common case where there are no actual marks on the file
> >> > being modified *or* the vfsmount.
> > What is so expensive in it? Just the memory barrier in it?
>
> The profiling doesn't hit on the mfence directly, but I assume that the
> overhead is coming from there. The "mov 0x8(%rdi),%rcx" is identical
> before and after the barrier, but it appears much more expensive
> _after_. That makes no sense unless the barrier is the thing causing it.

OK, one thing to try is to simply delete the memory barrier. The
resulting code will be unsafe, but will probably run well enough to
get benchmark results. If it is the memory barrier, you should of
course get increased throughput.

Thanx, Paul

> Here's how the annotation mode of 'perf top' breaks it down:
>
> > │ ffffffff810fb480 <load0>:
> > │ nop
> > │ mov (%rdi),%rax
> > 0.58 │ push %rbp
> > │ incl %gs:0x7ef0f488(%rip)
> > 1.73 │ mov %rsp,%rbp
> > │ and $0x1,%eax
> > │ movslq %eax,%rdx
> > 0.58 │ mov 0x8(%rdi),%rcx
> > │ incq %gs:(%rcx,%rdx,8)
> > │ mfence
> > 69.94 │ add $0x2,%rdx
> > 7.51 │ mov 0x8(%rdi),%rcx
> > 4.05 │ incq %gs:(%rcx,%rdx,8)
> > 13.87 │ decl %gs:0x7ef0f45f(%rip)
> > │ pop %rbp
> > 1.73 │ ← retq
> >
>

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2015-06-20 04:41    [W:0.065 / U:0.500 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site