ping - this can be merged now. On Tue, Dec 16, 2014 at 03:55:07PM +0000, Savolainen, Petri (NSN - FI/Espoo) wrote: > Reviewed-by: Petri Savolainen <[email protected]> > > > -----Original Message----- > > From: ext Stuart Haslam [mailto:[email protected]] > > Sent: Tuesday, December 16, 2014 4:55 PM > > To: [email protected] > > Cc: Savolainen, Petri (NSN - FI/Espoo); Stuart Haslam > > Subject: [PATCH] api: pktio: remove odp_pktio_get_input() and > > odp_pktio_set_input() > > > > odp_pktio_get_input() has been replaced by odp_packet_input() > > > > odp_pktio_set_input() has no valid use case outside of the > > implementation. > > > > Signed-off-by: Stuart Haslam <[email protected]> > > --- > > (This code contribution is provided under the terms of agreement LES-LTM- > > 21309) > > > > Note: depends on odp_packet_input() from the Packet API patches that > > are not yet in mainline. > > > > example/l2fwd/odp_l2fwd.c | 2 +- > > example/packet/odp_pktio.c | 2 +- > > platform/linux-generic/include/api/odp_packet_io.h | 19 ----------------- > > -- > > platform/linux-generic/odp_packet_io.c | 12 +----------- > > 4 files changed, 3 insertions(+), 32 deletions(-) > > > > diff --git a/example/l2fwd/odp_l2fwd.c b/example/l2fwd/odp_l2fwd.c > > index 9721ab7..3e11b60 100644 > > --- a/example/l2fwd/odp_l2fwd.c > > +++ b/example/l2fwd/odp_l2fwd.c > > @@ -229,7 +229,7 @@ static void *pktio_queue_thread(void *arg) > > continue; > > } > > > > - pktio_tmp = odp_pktio_get_input(pkt); > > + pktio_tmp = odp_packet_input(pkt); > > outq_def = odp_pktio_outq_getdef(dstpktio[pktio_tmp]); > > if (outq_def == ODP_QUEUE_INVALID) { > > EXAMPLE_ERR(" [%02i] Error: def output-Q query\n", > > diff --git a/example/packet/odp_pktio.c b/example/packet/odp_pktio.c > > index ea72e58..446aac5 100644 > > --- a/example/packet/odp_pktio.c > > +++ b/example/packet/odp_pktio.c > > @@ -209,7 +209,7 @@ static void *pktio_queue_thread(void *arg) > > continue; > > } > > > > - pktio_tmp = odp_pktio_get_input(pkt); > > + pktio_tmp = odp_packet_input(pkt); > > outq_def = odp_pktio_outq_getdef(pktio_tmp); > > > > if (outq_def == ODP_QUEUE_INVALID) { > > diff --git a/platform/linux-generic/include/api/odp_packet_io.h > > b/platform/linux-generic/include/api/odp_packet_io.h > > index 63c047c..a1ad754 100644 > > --- a/platform/linux-generic/include/api/odp_packet_io.h > > +++ b/platform/linux-generic/include/api/odp_packet_io.h > > @@ -111,25 +111,6 @@ int odp_pktio_inq_remdef(odp_pktio_t id); > > odp_queue_t odp_pktio_outq_getdef(odp_pktio_t id); > > > > /** > > - * Store packet input handle into packet > > - * > > - * @param pkt ODP packet buffer handle > > - * @param id ODP packet IO handle > > - * > > - * @return > > - */ > > -void odp_pktio_set_input(odp_packet_t pkt, odp_pktio_t id); > > - > > -/** > > - * Get stored packet input handle from packet > > - * > > - * @param pkt ODP packet buffer handle > > - * > > - * @return Packet IO handle > > - */ > > -odp_pktio_t odp_pktio_get_input(odp_packet_t pkt); > > - > > -/** > > * Configure the MTU for a packet IO interface. > > * > > * @param[in] id ODP packet IO handle. > > diff --git a/platform/linux-generic/odp_packet_io.c b/platform/linux- > > generic/odp_packet_io.c > > index aa37179..1b795d3 100644 > > --- a/platform/linux-generic/odp_packet_io.c > > +++ b/platform/linux-generic/odp_packet_io.c > > @@ -284,16 +284,6 @@ int odp_pktio_close(odp_pktio_t id) > > return 0; > > } > > > > -void odp_pktio_set_input(odp_packet_t pkt, odp_pktio_t pktio) > > -{ > > - odp_packet_hdr(pkt)->input = pktio; > > -} > > - > > -odp_pktio_t odp_pktio_get_input(odp_packet_t pkt) > > -{ > > - return odp_packet_hdr(pkt)->input; > > -} > > - > > int odp_pktio_recv(odp_pktio_t id, odp_packet_t pkt_table[], unsigned > > len) > > { > > pktio_entry_t *pktio_entry = get_pktio_entry(id); > > @@ -327,7 +317,7 @@ int odp_pktio_recv(odp_pktio_t id, odp_packet_t > > pkt_table[], unsigned len) > > return pkts; > > > > for (i = 0; i < pkts; ++i) > > - odp_pktio_set_input(pkt_table[i], id); > > + odp_packet_hdr(pkt_table[i])->input = id; > > > > return pkts; > > } > > -- > > 2.1.1 > > > >
_______________________________________________ lng-odp mailing list [email protected] http://lists.linaro.org/mailman/listinfo/lng-odp
