Message-ID: <55BAE31D.6060004@hp.com>
Date: Thu, 30 Jul 2015 22:53:17 -0400
From: Waiman Long <waiman.long@hp.com>
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:10.0.12) Gecko/20130109 Thunderbird/10.0.12
MIME-Version: 1.0
To: "Eric W. Biederman" <ebiederm@xmission.com>
CC: Andrew Morton <akpm@linux-foundation.org>,
        Nicolas Dichtel <nicolas.dichtel@6wind.com>,
        Al Viro <viro@zeniv.linux.org.uk>,
        Alexey Dobriyan <adobriyan@gmail.com>, linux-kernel@vger.kernel.org,
        Scott J Norton <scott.norton@hp.com>,
        Douglas Hatch <doug.hatch@hp.com>
Subject: Re: [PATCH] proc: change proc_subdir_lock to a rwlock
References: <1438201199-2809-1-git-send-email-Waiman.Long@hp.com> <DDAFF52F-3277-4645-9DA9-62C388DA445C@xmission.com> <55BADA98.3000304@hp.com>
In-Reply-To: <55BADA98.3000304@hp.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1463
Lines: 34

On 07/30/2015 10:16 PM, Waiman Long wrote:
> On 07/29/2015 06:21 PM, Eric W. Biederman wrote:
>> Two quick questions.
>>
>> - What motivates this work?  Are you seeing lots of
>>    parallel reads on proc?
>
> The micro-benchmark that I used was artificial, but it was used to 
> reproduce an exit hanging problem that I saw in real application. In 
> fact, only allow one task to do a lookup seems too limiting to me.
>> - Why not rcu?  Additions and removal of proc generic
>>    files is very rare.  Conversion to rcu for reads should
>>    perform better and not take much more work.
>
> RCU is harder to verify its correctness, whereas rwlock is easier to 
> use and understand. If it is really a performance critical path where 
> every extra bit of performance counts, I will certainly think RCU may 
> be the right choice. However, in this particular case, I don't think 
> using RCU will give any noticeable performance gain compared with a 
> rwlock.

One more thing, RCU is typically used with linked list. It is not easy 
to use RCU with rbtree and may require major changes to the code.

Another alternative is to use seqlock + RCU, but it will still need more 
code changes than rwlock.

Cheers,
Longman
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/