From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mgamail.intel.com (mgamail.intel.com [192.198.163.9]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 83424303A3D; Wed, 3 Dec 2025 17:10:55 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=192.198.163.9 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1764781857; cv=none; b=SSWdMD60g6sfA4/XBYhTrPm7kaHq7jeDwtLiB2PBoDh5GESy9TPpNha6EkzaJu4EsRdaRKdbxXQdxk3toEQb+h3Yttn/6PRCbTwThA6i1ji2InsSkwgUazqKnLm1oP6n3nZJNLVZNovEhZ+ZZPuXa0q+r5pSwtgH9zGfGZo2GoE= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1764781857; c=relaxed/simple; bh=lFO3wtNsh5dGyFwM7eIT6bOnEPizuYPsL5CIYF1uh10=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=hLWRd4Li4sI0PMgJ2I/e6orCdJ+6vaPy40FBYYsMBOFjHSNqOJxooxxpIc+CARYAd8LGa8Kn9Ef75f97m9+oVUX9ZMnyMbav+rYi8l7YTCvc5K/ff7VO1QrpXix5Y8lzNde2AXposV+Kh5fjXxEqsBz8Lrc5iT8JXXi0vCLq+oc= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.intel.com; spf=pass smtp.mailfrom=linux.intel.com; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b=G39M7PyL; arc=none smtp.client-ip=192.198.163.9 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.intel.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linux.intel.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b="G39M7PyL" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1764781855; x=1796317855; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=lFO3wtNsh5dGyFwM7eIT6bOnEPizuYPsL5CIYF1uh10=; b=G39M7PyLxPy0KZaIASz11aFZgbgjxc7Q6KBSYT+OqhCYkloZ4PoHURQB +23CBsqwZbMtMWSqUkbFiXBNrtPi7NEeRgjmQ4TY9dPx0lnXrfdUhS4ay WbASZNGhAKrsUDQqE9CNOoohZGl887hQFmIxFx2xcmZ3t+I4k/c3PRuVI B1aJoOkkiYiX4SQU233bfzvMNJTxUbtRzKcP6TfBu8tY13hYNCrYgk313 o2GTq0lobjWLzFziS5sql4/du4osnsoToUVDZxJRWsam7E4FfK/Yqjs+h 8DQatX63zjuD3fZerBSozG50i5jSXBCaQvQSF0xQICSiIggId4GgH7eTj A==; X-CSE-ConnectionGUID: 1UkAHXSGR1yoszFEjwb8bQ== X-CSE-MsgGUID: kQAOOhLcQ4GdbryTPWZiCw== X-IronPort-AV: E=McAfee;i="6800,10657,11631"; a="77464932" X-IronPort-AV: E=Sophos;i="6.20,246,1758610800"; d="scan'208";a="77464932" Received: from fmviesa010.fm.intel.com ([10.60.135.150]) by fmvoesa103.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 03 Dec 2025 09:10:54 -0800 X-CSE-ConnectionGUID: 0hnk3xmfRvmma7MQ7PxR0w== X-CSE-MsgGUID: uskhqYyCQvWejXnOJSSuLw== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.20,246,1758610800"; d="scan'208";a="195548113" Received: from dhhellew-desk2.ger.corp.intel.com (HELO localhost) ([10.245.245.81]) by fmviesa010-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 03 Dec 2025 09:10:45 -0800 Date: Wed, 3 Dec 2025 19:10:43 +0200 From: Andy Shevchenko To: "Yury Norov (NVIDIA)" Cc: Steven Rostedt , Masami Hiramatsu , Mathieu Desnoyers , Christophe Leroy , Randy Dunlap , Ingo Molnar , Jani Nikula , Joonas Lahtinen , David Laight , Petr Pavlu , Andi Shyti , Rodrigo Vivi , Tvrtko Ursulin , Daniel Gomez , Greg Kroah-Hartman , "Rafael J. Wysocki" , Danilo Krummrich , Andrew Morton , linux-kernel@vger.kernel.org, intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org, linux-modules@vger.kernel.org, linux-trace-kernel@vger.kernel.org Subject: Re: [PATCH v2 4/4] tracing: move tracing declarations from kernel.h to a dedicated header Message-ID: References: <20251203162329.280182-1-yury.norov@gmail.com> <20251203162329.280182-5-yury.norov@gmail.com> Precedence: bulk X-Mailing-List: linux-trace-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20251203162329.280182-5-yury.norov@gmail.com> Organization: Intel Finland Oy - BIC 0357606-4 - c/o Alberga Business Park, 6 krs, Bertel Jungin Aukio 5, 02600 Espoo On Wed, Dec 03, 2025 at 11:23:25AM -0500, Yury Norov (NVIDIA) wrote: > Tracing is a half of the kernel.h in terms of LOCs, although it's > a self-consistent part. It is intended for quick debugging purposes > and isn't used by the normal tracing utilities. > > Move it to a separate header. If someone needs to just throw a > trace_printk() in their driver, they will not have to pull all > the heavy tracing machinery. > > This is a pure move, except for removing a few 'extern's. ... > #include > #include > #include > +#include > #include And kill this one? We really don't care about the mess in the kernel.h right now, and your header will bring that anyway. Also consider killing other inclusions that has been moved from kernel.h to tracing.h. > #include > #include ... > +#ifndef _LINUX_TRACING_H > +#define _LINUX_TRACING_H > + > +#include > +#include + stddef.h // you use NULL + string.h // use of strlen() > +#include -- With Best Regards, Andy Shevchenko