On Tue, Mar 04, 2025 at 12:33:56PM -0800, Koralahalli Channabasappa, Smita wrote: snip > > > +TRACE_EVENT(cxl_port_aer_uncorrectable_error, > > > + TP_PROTO(struct device *dev, u32 status, u32 fe, u32 *hl), > > > + TP_ARGS(dev, status, fe, hl), > > > + TP_STRUCT__entry( > > > + __string(devname, dev_name(dev)) > > > + __string(parent, dev_name(dev->parent)) > > > > Above devname, parent > > Ok I'm planning to keep as device and parent. Let me know if wording "host" > is preferred over "parent". Take a look at these in the same file that use memdev, 'host'. Maybe you want to be similar. TRACE_EVENT(cxl_aer_uncorrectable_error, TRACE_EVENT(cxl_aer_correctable_error, snip >