Date:   Thu, 9 Feb 2023 23:07:36 -0800
From:   Ira Weiny <ira.weiny@intel.com>
To:     Ira Weiny <ira.weiny@intel.com>,
        Alison Schofield <alison.schofield@intel.com>,
        Vishal Verma <vishal.l.verma@intel.com>,
        "Ben Widawsky" <bwidawsk@kernel.org>,
        Dan Williams <dan.j.williams@intel.com>
CC:     <linux-cxl@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
        Ira Weiny <ira.weiny@intel.com>,
        Steven Rostedt <rostedt@goodmis.org>,
        Dave Jiang <dave.jiang@intel.com>
Subject: Re: [PATCH 0/2] cxl: Enhance trace point information
Message-ID: <63e5ed38d77d9_138fbc2947a@iweiny-mobl.notmuch>
References: <20230208-cxl-event-names-v1-0-73f0ff3a3870@intel.com>
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <20230208-cxl-event-names-v1-0-73f0ff3a3870@intel.com>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?wfsUpyM16ijg7SH2xIb6kXgmaC4oFMWGgr1cCF/axOhQPBPaTrP0HWaN/HFt?=
 =?us-ascii?Q?EWO741NJ6h1xNIN9PWIhXFUpveZz5rl70nvJXeONzV5bcxl8uhXLVVFu9PUh?=
 =?us-ascii?Q?PxlUoa+cgCei2DhbODsPv+v78P25bsL/dZGaQD3gwsIJeaWXRd4pXCVy/foo?=
 =?us-ascii?Q?ArL/EDn7EV66qLE8RC0AqHd81wNRcwL//BaHzET1iWEi9+F014WHUOBfHZs+?=
 =?us-ascii?Q?Ylpac1XFTOB2SEKqSLuHW3wwHRe9ILujK6ALeFx02AJ4fnpeLwgjE6FF1tWP?=
 =?us-ascii?Q?pPHvrdhdZDE9OOU1R07XxLA30RTLz450k+x983SZRnq/e2iuBmtVvygxD4EW?=
 =?us-ascii?Q?n7+RzNnNlZmGPz0qpYwLgCFbDqnvDPrLN40Wi404j+v9xf8HO05Xz8EiD7Oo?=
 =?us-ascii?Q?VOvNElWjBVTO0BRs+GSvRtxaH7GjmEFRw6U8qGGekB35AQgj2N8DzJdvmYjR?=
 =?us-ascii?Q?QbHJwVx2H1roWQ6nrYNhF0JPugGQjjg4vl8kBH/HAKkRlUKxB1Da8O9IVgs+?=
 =?us-ascii?Q?f4adJgIZtJAXvb5Sz+R7ZYlFI/xDTF0ERE7Z7xoXdixDt2OTdHX2RxixkdnP?=
 =?us-ascii?Q?a91dcfE/m18H0pOJMR10ZzKAlvcLvx7Q50243zidq8SxfpNWAEhsZx4Fo52Z?=
 =?us-ascii?Q?EiVm4Rc3s/DLmmbSzhccG+7XqFnCU8xac7AXC55y47h/8MFwmbI/J36QkK7B?=
 =?us-ascii?Q?88ogYa4YOjofVNikOowapAVBs4tSmN2qKa8drx0E8+56EzKJEltv5z/uWCg8?=
 =?us-ascii?Q?KxfDyB/taUPz1rseaW7kzAzayMe4xx5Rkuh9eTv0g0lqAcmlly+ZkzpM2RCy?=
 =?us-ascii?Q?uVnSuCfN8vzbWmxzp4L1pZPYkuv4voG2TnU9CXJlqaNkkkNntKWqA3bekKkD?=
 =?us-ascii?Q?5sAMJz3fH1v89csyd7NAPu+Jqpo+xFlptGAMX8Alq+pNSy87EIXvDTdWbUSP?=
 =?us-ascii?Q?5WOjPmjt2sWrN3roR+IojN8rgnocEDjoTFwGAi0c3fbtVp7T4udmTTaswliH?=
 =?us-ascii?Q?mY9znhnc+KvkLkxw1dqaBp4bgz4Gx8mRonMIxnPQ3rYXbcHSSqSsjLSbcjBA?=
 =?us-ascii?Q?d6NA91NVOlq6Wavh5AdAIvPI/JIi+jOyf1FPvpqgkS5Dqv6WYHGvCXRdcJ45?=
 =?us-ascii?Q?1DxiHrGpH1tZTPXK0elxM1cowbs4Ow0hgwFlJ4pskV9uwqz0aZYOyhCvyOu8?=
 =?us-ascii?Q?cKFlRHlHEcsMMOD3EkQHNtgEOGeuO+yPMAowY6drxgRf5ToPm22wWAhv8nu8?=
 =?us-ascii?Q?ihLVCXF3fYf8pGVO6KL3QBTxh6+e6TqUnblNSUMLb99jYjxYdyt8ok7UsCbh?=
 =?us-ascii?Q?MLziH647C2csPLTDiDMwfYYUrvhH5l5SVZShBqQDwexWwmyLCAyfI/nEVMQh?=
 =?us-ascii?Q?Zz96OvlvflzqWFDJoMen7JeB2SmlQh+xFZLSYn50A3H8pWUWE99MXFYlsZse?=
 =?us-ascii?Q?P9gv3UsQM1cOz8sSnzAc20KgbErWePJKtt1Avfet4w4NiZ6AcJ+UUDz3PYYF?=
 =?us-ascii?Q?LrACokGHqwrtBjR51Vk3Z2eqAT6elU458MBAYzQh22kH1Y3rpgwOL0VUHN34?=
 =?us-ascii?Q?D+sanNOONTw45oRDe5ifsgn+JykQT9Rj4wQxLt7J?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 11c9160d-4d34-43df-1148-08db0b357f00
X-MS-Exchange-CrossTenant-AuthSource: SA1PR11MB6733.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 10 Feb 2023 07:07:39.8220
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: FoxH2UaiY1zgJTpjDgXnvWcXvnkJD815BcxVmO3xfWzDZPocZOAw56uk54hf+ekj7hYvOJj6x1/0448eaJD7SQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MN0PR11MB6134
X-OriginatorOrg: intel.com
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Ira Weiny wrote:
> Trace point definitions are reporting different linux devices for
> the same CXL hardware device.  Clean this up and add parent device
> information to all the trace points.

I've found a couple of small changes that I wanted to make but got side
tracked and forgot when I sent this.

Also Dan reminded me we would like to see the device serial number in
these traces as well.

v2 coming shortly.

Ira

> 
> Signed-off-by: Ira Weiny <ira.weiny@intel.com>
> ---
> Ira Weiny (2):
>       cxl/trace: Standardize device information output
>       cxl/trace: Add host output to trace points
> 
>  drivers/cxl/core/mbox.c  | 14 ++++----
>  drivers/cxl/core/pci.c   |  8 ++---
>  drivers/cxl/core/trace.h | 85 +++++++++++++++++++++++++++---------------------
>  3 files changed, 57 insertions(+), 50 deletions(-)
> ---
> base-commit: dbe9f7d1e155b97a42f7da81e22acc98fe0a9072
> change-id: 20230208-cxl-event-names-9372fb72a607
> 
> Best regards,
> -- 
> Ira Weiny <ira.weiny@intel.com>
>