On 21 August 2014 15:16, Ola Liljedahl <[email protected]> wrote: > This command line option would be interpreted by the application. But the > application needs a function for setting an interface (pktio instance) in > promiscuous mode. Yet another missing piece of API. >
What else required for existing pktio instance to operate on promiscuous mode other than sending a hint to driver. For example, The way I look at it per existing code, all I need to pass one promiscuous mode hint in case of dpdk, dpdk implementation api (which is this - rte_eth_promiscuous_enable();) will do the rest for me. Like to know more.. Thanks. > > On 21 August 2014 11:27, Venkatesh Vivekanandan > <[email protected]> wrote: >> >> >> >> >> On 21 August 2014 14:27, Santosh Shukla <[email protected]> wrote: >>> >>> On 21 August 2014 14:24, Ola Liljedahl <[email protected]> wrote: >>> > You are not always setting promiscuous mode on network interfaces in >>> > ODP/linux-dpdk? Promiscuous mode should only be enabled when the >>> > application >>> > requires it. The l2fwd (is this the DPDK port? "passthrough" would be a >>> > better name) application could have a command line option which >>> > requests >>> > promiscuous mode on the used network interfaces. >>> > >>> >>> Even better. Make sense to me. I'll spin the patch and do the changes >>> in l2fwd cli interface accordingly. >>> >> There is no way one can pass the command line parameters to the >> implementation currently, once that gets in then we can make it a user >> configurable parameter. For now, we don't have a choice other than keeping >> them enabled. >> >>> >>> > >>> > On 21 August 2014 08:57, Santosh Shukla <[email protected]> >>> > wrote: >>> >> >>> >> Noticed that enabling promiscous mode helps l2fwd application work >>> >> with >>> >> pktgen >>> >> or dpdk-pktgen >>> >> >>> >> Suggested-by: Venkatesh Vivekanandan >>> >> <[email protected]> >>> >> Signed-off-by: Santosh Shukla <[email protected]> >>> >> --- >>> >> platform/linux-dpdk/odp_packet_dpdk.c | 3 +++ >>> >> 1 file changed, 3 insertions(+) >>> >> >>> >> diff --git a/platform/linux-dpdk/odp_packet_dpdk.c >>> >> b/platform/linux-dpdk/odp_packet_dpdk.c >>> >> index d5c8e80..92edac5 100644 >>> >> --- a/platform/linux-dpdk/odp_packet_dpdk.c >>> >> +++ b/platform/linux-dpdk/odp_packet_dpdk.c >>> >> @@ -135,6 +135,9 @@ int setup_pkt_dpdk(pkt_dpdk_t * const pkt_dpdk, >>> >> const >>> >> char *netdev, >>> >> ODP_DBG("dpdk tx queue setup done\n"); >>> >> } >>> >> >>> >> + /* Enable promiscous mode */ >>> >> + rte_eth_promiscuous_enable(portid); >>> >> + >>> >> /* Start device */ >>> >> ret = rte_eth_dev_start(portid); >>> >> if (ret < 0) >>> >> -- >>> >> 1.7.9.5 >>> >> >>> >> >>> >> _______________________________________________ >>> >> lng-odp mailing list >>> >> [email protected] >>> >> http://lists.linaro.org/mailman/listinfo/lng-odp >>> > >>> > >>> >>> _______________________________________________ >>> lng-odp mailing list >>> [email protected] >>> http://lists.linaro.org/mailman/listinfo/lng-odp >> >> > _______________________________________________ lng-odp mailing list [email protected] http://lists.linaro.org/mailman/listinfo/lng-odp
