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 gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 711BDCCFA05 for ; Thu, 6 Nov 2025 14:58:06 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id E7B0910E916; Thu, 6 Nov 2025 14:58:05 +0000 (UTC) Authentication-Results: gabe.freedesktop.org; dkim=pass (2048-bit key; unprotected) header.d=kernel.org header.i=@kernel.org header.b="S0cxmtmR"; dkim-atps=neutral Received: from sea.source.kernel.org (sea.source.kernel.org [172.234.252.31]) by gabe.freedesktop.org (Postfix) with ESMTPS id B7BAC10E908; Thu, 6 Nov 2025 14:58:04 +0000 (UTC) Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by sea.source.kernel.org (Postfix) with ESMTP id 33DF44165A; Thu, 6 Nov 2025 14:58:04 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id A6199C4CEF7; Thu, 6 Nov 2025 14:58:03 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1762441084; bh=3MzzI7eda1HcBKQQLuDs92NFebxA65nvy9xo/gSXejE=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=S0cxmtmRiyLuJnpNgUqFMrYRYHP/Sn3IgwSPXA8zVNJ27uJVauSMUd3DujH71CqqQ FT3LlbX89aDaiTBvzk0ucsearN8iQvLDJgZTogLtm+FBd8IYX5rCKzq4ogBK+t57eN 0/JFB7NXK0SKQs3mkT+qRI4K5VsW/vx0SxLbc3wkUor4L3aBB/ASw4HbPgFa94ssjB Q3RsecprggSAaQMRQJju9/TzV/dUo6iOfjP+8JJQnluWV4LzUqcanJ7oibMmd5fhxN CTn/6n/TfbDuTjs8i2OUjKFRf4x504KwZwRe4fXlpH241YkgORAn0vKMM1ZBVrkkvg CYEtvZSJUynWA== Date: Thu, 6 Nov 2025 06:58:02 -0800 From: Jakub Kicinski To: Rodrigo Vivi Cc: , , "Zack McKevitt" , Lukas Wunner , Lijo Lazar , Hawking Zhang , Aravind Iddamsetty Subject: Re: [PATCH 1/2] drm/ras: Introduce the DRM RAS infrastructure over generic netlink Message-ID: <20251106065802.76d8ac77@kernel.org> In-Reply-To: References: <20250929214415.326414-4-rodrigo.vivi@intel.com> <20250929214415.326414-5-rodrigo.vivi@intel.com> <20251030183254.10d64ee1@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-BeenThere: intel-xe@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Intel Xe graphics driver List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-xe-bounces@lists.freedesktop.org Sender: "Intel-xe" On Thu, 6 Nov 2025 08:30:37 -0500 Rodrigo Vivi wrote: > > If you're using Fedora or another good distro ynl CLI is packaged (for > > Fedora in kernel-tools). The in-tree syntax is a bit verbose. > > I didn't know this tool was getting package with the kernel-tools > I thought it was only helping for debug during the development. > > Now I'm even wondering if we really need to code a user-space tool > for this drm-ras, or simply recommending the kernel-tools/ynl as > the official consumer of this API. Right, depends on the intended use of the API. In many cases, especially for configuration interfaces we no longer write separate CLI tools. But for certain things typing in the JSON gets a bit tedious, and other cases need some sort of summarization if the kernel output is too verbose. So YMMV. > > Separate handling of -EMSGSIZE and returning skb->len is not necessary > > as of a few releases ago. Just return ret; core will do the right thing > > if ret == -EMSGSIZE and skb->len != 0 > > Any good modern example that I could get the right inspiration from? It's a moving target but: net/core/netdev-genl.c net/psp/psp_nl.c