[PATCH] Update two optimisations that check for oplocks to the lease area

Jeremy Allison jra at samba.org
Fri May 5 05:34:46 UTC 2017


On Thu, May 04, 2017 at 01:35:36PM +0200, Ralph Böhme via samba-technical wrote:
> On Thu, May 04, 2017 at 04:25:04AM -0700, Jeremy Allison wrote:
> > On Thu, May 04, 2017 at 12:26:48PM +0200, Ralph Böhme wrote:
> > > Hi!
> > > 
> > > This is the one I talked about in my talk.
> > > 
> > > I took another look and found another plase where we need this.
> > > 
> > > Both combined eat 12%CPU in perf, not including syscalls.
> > > 
> > > Please review carefully and push if ok. Thanks!
> > 
> > Reviewed carefully :-) and pushed ! Thanks Ralph,
> > nice work.
> 
> woohoo! Thanks!

Oh not so fast, sorry :-(. I think there's a logic
bug in the num_read_oplocks() case. Still investigating..

Jeremy.



More information about the samba-technical mailing list