From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id CDF68C43334 for ; Mon, 18 Jul 2022 18:54:15 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233749AbiGRSyO (ORCPT ); Mon, 18 Jul 2022 14:54:14 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:49196 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S235916AbiGRSxx (ORCPT ); Mon, 18 Jul 2022 14:53:53 -0400 Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 32ABC112F; Mon, 18 Jul 2022 11:53:51 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1658170432; x=1689706432; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=uTLGXZcNL+L1yAPxsxYvJrrXWc9UE0EXaIFLw+I10gc=; b=JWegjbxQoKE3n6yGf8RWnZSgl8NI/nBKknRJEI1fkq2AKR3BV+f3v3C3 Q1P0999bEEO8hpnb3ARxCQP9tTg7xIC/feOvcSad1HYA8mzEcwiRHHFD+ D7gRMbvW6v6jxbSZTyQr5wP8yscjYFYhYNCZNJT/rPAuDlPTizghg2bYc y2RXSnjlK0IeHgJC9EbC8iQdstBr2YBKBj1V+XCLGph+kGtmfVlRZtJaR B3LkSRHJZCiynfTyGsvgt8ka7g1OUjupTA1YLuoSm7o9s/1Zzys+eyV/S XFop+32VH9Z/FjBU+zJHcBZC8YNkJxHLWejfOiOwuXa3i/p+OS43R0cei A==; X-IronPort-AV: E=McAfee;i="6400,9594,10412"; a="273125585" X-IronPort-AV: E=Sophos;i="5.92,281,1650956400"; d="scan'208";a="273125585" Received: from orsmga004.jf.intel.com ([10.7.209.38]) by orsmga101.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 18 Jul 2022 11:53:51 -0700 X-IronPort-AV: E=Sophos;i="5.92,281,1650956400"; d="scan'208";a="723961155" Received: from smile.fi.intel.com ([10.237.72.54]) by orsmga004-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 18 Jul 2022 11:53:43 -0700 Received: from andy by smile.fi.intel.com with local (Exim 4.96) (envelope-from ) id 1oDVsB-001OOr-1i; Mon, 18 Jul 2022 21:53:39 +0300 Date: Mon, 18 Jul 2022 21:53:39 +0300 From: Andy Shevchenko To: "Russell King (Oracle)" Cc: Vladimir Oltean , Andrew Lunn , Heiner Kallweit , Alexandre Belloni , Alvin __ipraga , Claudiu Manoil , Daniel Scally , "David S. Miller" , DENG Qingfang , Eric Dumazet , Florian Fainelli , George McCollister , Greg Kroah-Hartman , Hauke Mehrtens , Heikki Krogerus , Jakub Kicinski , Kurt Kanzenbach , Landen Chao , Linus Walleij , linux-acpi@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, Matthias Brugger , netdev@vger.kernel.org, Paolo Abeni , "Rafael J. Wysocki" , Sakari Ailus , Sean Wang , UNGLinuxDriver@microchip.com, Vivien Didelot , Woojung Huh , Marek =?iso-8859-1?Q?Beh=FAn?= Subject: Re: [PATCH net-next 2/6] software node: allow named software node to be created Message-ID: References: <20220715201715.foea4rifegmnti46@skbuf> <20220715204841.pwhvnue2atrkc2fx@skbuf> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org On Mon, Jul 18, 2022 at 09:43:42PM +0300, Andy Shevchenko wrote: > On Mon, Jul 18, 2022 at 02:27:02PM +0100, Russell King (Oracle) wrote: > > On Mon, Jul 18, 2022 at 03:29:52PM +0300, Andy Shevchenko wrote: > > > On Fri, Jul 15, 2022 at 11:48:41PM +0300, Vladimir Oltean wrote: > > > > So won't kobject_init_and_add() fail on namespace collision? Is it the > > > > problem that it's going to fail, or that it's not trivial to statically > > > > determine whether it'll fail? > > > > > > > > Sorry, but I don't see something actionable about this. > > > > > > I'm talking about validation before a runtime. But if you think that is fine, > > > let's fail it at runtime, okay, and consume more backtraces in the future. > > > > Is there any sane way to do validation of this namespace before > > runtime? > > For statically compiled, I think we can do it (to some extent). > Currently only three drivers, if I'm not mistaken, define software nodes with > names. It's easy to check that their node names are unique. > > When you allow such an API then we might have tracebacks (from sysfs) bout name > collisions. Not that is something new to kernel (we have seen many of a kind), > but I prefer, if possible, to validate this before sysfs issues a traceback. > > > The problem in this instance is we need a node named "fixed-link" that > > is attached to the parent node as that is defined in the binding doc, > > and we're creating swnodes to provide software generated nodes for > > this binding. > > And how you guarantee that it will be only a single one with unique pathname? > > For example, you have two DSA cards (or whatever it's called) in the SMP system, > it mean that there is non-zero probability of coexisting swnodes for them. > > > There could be several such nodes scattered around, but in this > > instance they are very short-lived before they are destroyed, they > > don't even need to be published to userspace (and its probably a waste > > of CPU cycles for them to be published there.) > > > > So, for this specific case, is this the best approach, or is there > > some better way to achieve what we need here? > > Honestly, I don't know. > > The "workaround" (but it looks to me rather a hack) is to create unique swnode > and make fixed-link as a child of it. > > Or entire concept of the root swnodes (when name is provided) should be > reconsidered, so somehow we will have a uniqueness so that the entire > path(s) behind it will be caller-dependent. But this I also don't like. > > Maybe Heikki, Sakari, Rafael can share their thoughts... > > Just for my learning, why PHY uses "fixed-link" instead of relying on a > (firmware) graph? It might be the actual solution to your problem. > > How graphs are used with swnodes, you may look into IPU3 (Intel Camera) > glue driver to support devices before MIPI standardisation of the > respective properties. Forgot to say (yes, it maybe obvious) that this API will be exported, anyone can use it and trap into the similar issue, because, for example, of testing in environment with a single instance of the caller. -- With Best Regards, Andy Shevchenko