> > +DECLARE_EVENT_CLASS(ib_mad_agent_template, > > + TP_PROTO(struct ib_mad_agent_private *agent), > > + TP_ARGS(agent), > > + > > + TP_STRUCT__entry( > > + __string(dev_name, agent->agent.device->name) > > I'm trying to get rid of device->name, this should be > dev_name(&device->dev) > > Though, should it use the stable device index instead? Both good points. My excuse is me basing them off my old work and not having correlated the upstream discussions with these patches... I'll respin with the device index as that is most unique. I still think there needs to be 'super easy' device index to name decoding... But I'm not really up on what has been done so forgive me if it is there. Ira
