linux-kernel - Re: [RFC][PATCH] fs: optimize inotify/fsnotify code for unwatched files

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Fri, 19 Jun 2015 19:21:35 -0700
From:	"Paul E. McKenney" <paulmck@...ux.vnet.ibm.com>
To:	Dave Hansen <dave@...1.net>
Cc:	Andi Kleen <ak@...ux.intel.com>, dave.hansen@...ux.intel.com,
	akpm@...ux-foundation.org, jack@...e.cz, viro@...iv.linux.org.uk,
	eparis@...hat.com, john@...nmccutchan.com, rlove@...ve.org,
	tim.c.chen@...ux.intel.com, linux-kernel@...r.kernel.org
Subject: Re: [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/