From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2992439AbcB0LqZ (ORCPT ); Sat, 27 Feb 2016 06:46:25 -0500 Received: from www.linutronix.de ([62.245.132.108]:35687 "EHLO Galois.linutronix.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756178AbcB0LqX (ORCPT ); Sat, 27 Feb 2016 06:46:23 -0500 Date: Sat, 27 Feb 2016 12:45:00 +0100 (CET) From: Thomas Gleixner To: Tejun Heo cc: Yang Shi , gregkh@linuxfoundation.org, lizefan@huawei.com, rostedt@goodmis.org, bigeasy@linutronix.de, linux-kernel@vger.kernel.org, linux-rt-users@vger.kernel.org, linaro-kernel@lists.linaro.org Subject: Re: [RFC V2 PATCH] kernfs: create raw version kernfs_path_len and kernfs_path In-Reply-To: <20160227114137.GZ3965@htj.duckdns.org> Message-ID: References: <1456523259-8588-1-git-send-email-yang.shi@linaro.org> <20160227111719.GV3965@htj.duckdns.org> <20160227114137.GZ3965@htj.duckdns.org> User-Agent: Alpine 2.11 (DEB 23 2013-08-11) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII X-Linutronix-Spam-Score: -1.0 X-Linutronix-Spam-Level: - X-Linutronix-Spam-Status: No , -1.0 points, 5.0 required, ALL_TRUSTED=-1,SHORTCIRCUIT=-0.0001 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sat, 27 Feb 2016, Tejun Heo wrote: > On Sat, Feb 27, 2016 at 12:37:23PM +0100, Thomas Gleixner wrote: > > On Sat, 27 Feb 2016, Tejun Heo wrote: > > > On Fri, Feb 26, 2016 at 01:47:39PM -0800, Yang Shi wrote: > > > > Call synchronize_sched() when kernfs_node is updated since tracepoints are > > > > protected by rcu_read_lock_sched. > > > > > > Adding synchronize_sched() to operations which can be triggered from > > > userland usually turns out to be problematic. If this can't be solved > > > any other way, I think the right thing to do is replacing the path > > > with cgroup id as Christoph suggested. > > > > Agreed. The path is not that important, right? > > It can be, but we can print out the ino and userland can match that up > with path if necessary. Wouldn't be cgroup id the better choice? Thanks, tglx