Hi Thomas, <snip>
> Subject: Re: [dpdk-dev] [PATCH v7 2/2] net/ixgbe: add callback to user app > on VF to PF mbox msg > > 2016-10-07 17:46, Bernard Iremonger: > > +struct rte_pmd_ixgbe_mb_event_param { > > + uint16_t vfid; /**< Virtual Function number */ > > + uint16_t msg_type; /**< message type */ > > + uint16_t retval; /**< return value */ > > + void *userdata; /**< pointer to user data */ > > Generally speaking, the user data is a pointer passed by the application when > registering the callback and must be untouched. > It should be the name of the parameter that you are overriding with this > structure. > So this "userdata" pointer could probably be better defined. > > By the way, it is far from trivial to understand how to write the callback. > I think it deserves more explanations. > I will clarify the description of the struct rte_pmd_ixgbe_mb_event_param, and rename the member userdate to msg in a v8 patchset. Regards, Bernard.