> From: Jerin Jacob [mailto:[email protected]] > Sent: Friday, June 30, 2017 10:12 AM > To: Van Haaren, Harry <[email protected]> > Cc: [email protected]; Richardson, Bruce <[email protected]>; > [email protected]; > Eads, Gage <[email protected]>; [email protected]; Vangati, Narender > <[email protected]>; Rao, Nikhil <[email protected]> > Subject: Re: [dpdk-dev] [PATCH 2/5] eventdev: introduce specialized enqueue > new op variant > > -----Original Message----- > > Date: Fri, 30 Jun 2017 08:40:06 +0000 > > From: "Van Haaren, Harry" <[email protected]> > > To: Jerin Jacob <[email protected]>, "[email protected]" > > <[email protected]> > > CC: "Richardson, Bruce" <[email protected]>, > > "[email protected]" <[email protected]>, "Eads, Gage" > > <[email protected]>, "[email protected]" <[email protected]>, > > "Vangati, Narender" <[email protected]>, "Rao, Nikhil" > > <[email protected]> > > Subject: RE: [dpdk-dev] [PATCH 2/5] eventdev: introduce specialized enqueue > > new op variant > > > > > From: Jerin Jacob [mailto:[email protected]] > > > Sent: Thursday, June 29, 2017 3:20 PM > > > To: [email protected] > > <snip> > > > diff --git a/drivers/event/octeontx/ssovf_evdev.c > b/drivers/event/octeontx/ssovf_evdev.c > > > index 8dc7b2ef8..0d0c6a186 100644 > > > --- a/drivers/event/octeontx/ssovf_evdev.c > > > +++ b/drivers/event/octeontx/ssovf_evdev.c > > > @@ -158,6 +158,7 @@ ssovf_fastpath_fns_set(struct rte_eventdev *dev) > > > dev->schedule = NULL; > > > dev->enqueue = ssows_enq; > > > dev->enqueue_burst = ssows_enq_burst; > > > + dev->enqueue_new_burst = ssows_enq_burst; > > > dev->dequeue = ssows_deq; > > > dev->dequeue_burst = ssows_deq_burst; > > > > > > diff --git a/drivers/event/sw/sw_evdev.c b/drivers/event/sw/sw_evdev.c > > > index fe2a61e2f..951ad1b33 100644 > > > --- a/drivers/event/sw/sw_evdev.c > > > +++ b/drivers/event/sw/sw_evdev.c > > > @@ -796,6 +796,7 @@ sw_probe(struct rte_vdev_device *vdev) > > > dev->dev_ops = &evdev_sw_ops; > > > dev->enqueue = sw_event_enqueue; > > > dev->enqueue_burst = sw_event_enqueue_burst; > > > + dev->enqueue_new_burst = sw_event_enqueue_burst; > > > dev->dequeue = sw_event_dequeue; > > > dev->dequeue_burst = sw_event_dequeue_burst; > > > dev->schedule = sw_event_schedule; > > > > > > I think it is possible to do this pointer-setting of new_burst() in > > eventdev.c, instead > of adding the new_burst() to each PMD individually? > > During rte_eventdev_configure(), if the dev->enqueue_new_burst() function > > is NULL, just > point it at the ordinary one; > > I thought so, But it will break in multi process use case as on probe() we are > updating the callbacks for secondary process. Doing it in probe() may be very > early as some PMD may update the callback anywhere on or before > rte_eventdev_start(). > > Thoughts?
Ah I forgot about the multi-proc init during my review. The current solution is more appropriate then, so this version Acked-by: Harry van Haaren <[email protected]>
