On Tue, Oct 18, 2016 at 02:50:16PM +0200, Jan Blunck wrote: > >From my understanding this is a massive API breakage. This forces all > existing users of the virtual PMDs to change with zero benefit. Even > if that isn't enough it also makes it impossible to switch between > releases by recompiling. > > Can we please revert these changes and work on some aliasing support > for the PMDs to fix it long term?
+1. Aliasing is also something I would suggest before making such renames. --yliu > > Thanks, > Jan > > > On Fri, Sep 16, 2016 at 11:58 AM, Thomas Monjalon > <thomas.monjalon at 6wind.com> wrote: > > 2016-08-24 22:37, Mcnamara, John: > >> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Pablo de Lara > >> > > >> > ... > >> > > >> > - $RTE_TARGET/app/testpmd -c '0xf' -n 4 --vdev > >> > 'eth_pcap0,rx_pcap=/path/to/ file_rx.pcap,tx_pcap=/path/to/file_tx.pcap' > >> > -- --port-topology=chained > >> > + $RTE_TARGET/app/testpmd -c '0xf' -n 4 --vdev > >> > 'net_pcap0,rx_pcap=/path/to/ file_rx.pcap,tx_pcap=/path/to/file_tx.pcap' > >> > -- --port-topology=chained > >> > >> > >> I know that this is an existing issue but there shouldn't be a space in > >> "/path/to/ file". Perhaps you could fix that (in a number of places) as > >> part > >> of this patch. You could probably leave out the "/path/to/" part > >> altogether as > >> it may be clearer, see below. > >> > >> Also, could you wrap the long code lines in the sections that you change at > >> 80 chars using "\" to keep them on the page in the PDF docs, like: > >> > >> $RTE_TARGET/app/testpmd -c '0xf' -n 4 \ > >> --vdev > >> 'net_pcap0,rx_pcap=/path/to/file_rx.pcap,tx_pcap=/path/to/file_tx.pcap' \ > >> -- --port-topology=chained > >> > >> Or without the path part: > >> > >> $RTE_TARGET/app/testpmd -c '0xf' -n 4 \ > >> --vdev 'net_pcap0,rx_pcap=file_rx.pcap,tx_pcap=file_tx.pcap' \ > >> -- --port-topology=chained > > > > Applied with above comments fixed and release notes updated, thanks.