Re: [PATCH 2/2] fuse: Move clear_bit() up in request_end()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Jul 31, 2018 at 12:25 PM, Kirill Tkhai <ktkhai@xxxxxxxxxxxxx> wrote:
> Nobody can clear FR_BACKGROUND bit on processing
> request in parallel, so it's possible to do it
> out of fc->lock.

Moving such a cheap operation outside the splinlock won't make any
difference in real life.

And anyway the biggest problem with fc->lock is not contention (hold
for long period of time), I would guess, but cachline pingponging
(being acquired on different CPUs one after the other).

So there's definitely work to do regarding locking in fuse, but it
needs a bigger axe.

Thanks,
Miklos



[Index of Archives]     [Linux Ext4 Filesystem]     [Union Filesystem]     [Filesystem Testing]     [Ceph Users]     [Ecryptfs]     [AutoFS]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux Cachefs]     [Reiser Filesystem]     [Linux RAID]     [Samba]     [Device Mapper]     [CEPH Development]

  Powered by Linux