24/06/2022 15:45, Srujana Challa: > Function rte_cryptodev_cb_fn prototype will be extended to > add new parameter qp_id, to return queue pair ID, which got > error interrupt to the application, so that application can > reset that particular queue pair. > > https://mails.dpdk.org/archives/dev/2022-June/245428.html > > Signed-off-by: Srujana Challa <scha...@marvell.com> > --- > +* cryptodev: The function pointer ``rte_cryptodev_cb_fn`` will be updated to > + have another parameter ``qp_id`` to return the queue_pair ID which got > error > + interrupt to the application so that application can reset that particular > + queue pair.
I think it is a wrong decision. Some events may not be related to queues. You cannot add all events infos in the callback. In ethdev, we are querying the event infos with additional functions, specific to the event type. Applied, but I will try to convince you to not do it.