From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Mon, 15 Jan 2024 20:52:07 +0100 From: Mauro Carvalho Chehab To: Jani Nikula Subject: Re: [igt-dev] [PATCH i-g-t] xe_live_ktest: Use xe_live_test kernel module Message-ID: <20240115205207.5b491e3e@maurocar-mobl2> In-Reply-To: <87mst7rri8.fsf@intel.com> References: <20231205224926.2188996-1-lucas.demarchi@intel.com> <20231207164152.57be3d07@maurocar-mobl2> <3kunqgqyf7tobiwhcpvjp7rcvidrgnz2jq3qqoxjbl7kaftdq7@7hdweo57htff> <20240112093252.0a4bf850@maurocar-mobl2> <57epajumgzrllmyufx7233yjja2c3fbs2zgfflkhcspfvwak47@h64l4uzne54h> <11e1b3f4-b0ba-4095-96c6-f2bf18f27fd2@linux.intel.com> <87mst7rri8.fsf@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: igt-dev@lists.freedesktop.org, Lucas De Marchi , intel-xe@lists.freedesktop.org Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" List-ID: On Mon, 15 Jan 2024 09:21:35 +0200 Jani Nikula wrote: > On Mon, 15 Jan 2024, Mauro Carvalho Chehab wrote: > > On 1/12/24 15:29, Lucas De Marchi wrote: > >>> On such case, you should document kunit_debugfs at Documentation/ABI. > >> > >> debugfs is not part of ABI. > > > > That's arguable. Check Documentation/ABI and you'll see several debugfs > > nodes documented there. > > I guess you could argue documenting your debugfs under Documentation/ABI > makes it ABI, but I'd avoid doing that. Usually we use debugfs precisely > to avoid the burden of ABI and all the baggage that brings. Placing things at debugfs "precisely to avoid the burden of ABI and all the baggage that brings" sounds a bad practice to me, but you can ask others at LKML and see if this is a acceptable. My understanding is that, if userspace tools rely on what's there at debugfs, and a change break such tools, there is a high chance that such change will end being reverted - at best, it will surely provide some flame wars. I remember we had some troubles related to trace logic in the past, due to something similar: even not originally meant to provide a stable ABI, and being under debugfs, there were some changes breaking userspace which caused lots of discussions. Can't remember anymore if the changes were reverted or not, but at the end ftrace ABI was considered as stable. I have an userspace program which depends on it (rasdaemon). Also, having it documented or not doesn't mean it is not ABI. See, for instance, this article: https://lwn.net/Articles/309298/. Regards, Mauro