Re: [dpdk-dev] [PATCH v2] ether: use a default for max Rx frame size in configure()

2017-04-07 Thread Andriy Berestovskyy
On 07.04.2017 10:34, Thomas Monjalon wrote: We can set the right default value if the app input is 0, as a special case. For any other value, we must try to set it or return an error. Right, I will resend the patch. Andriy

Re: [dpdk-dev] [PATCH v2] ether: use a default for max Rx frame size in configure()

2017-04-07 Thread Thomas Monjalon
2017-04-07 10:09, Andriy Berestovskyy: > Hi Thomas, > > On 06.04.2017 22:48, Thomas Monjalon wrote: > > Anyway, why not fixing it in the reverse way: returning error for > > out of range of non-jumbo frames? > > I guess we need to fix most of the examples then, since most of them > just pass 0 f

Re: [dpdk-dev] [PATCH v2] ether: use a default for max Rx frame size in configure()

2017-04-07 Thread Bruce Richardson
On Mon, Mar 27, 2017 at 10:38:13AM +0200, Andriy Berestovskyy wrote: > Hey Qiming, > > On 27.03.2017 08:15, Yang, Qiming wrote: > > I don't think this is a bug. Return errors when configure an invalid > > max_rx_pkt_len is suitable for this generic API. > > It is not a bug, it is an inconsistenc

Re: [dpdk-dev] [PATCH v2] ether: use a default for max Rx frame size in configure()

2017-04-07 Thread Andriy Berestovskyy
Hi Thomas, On 06.04.2017 22:48, Thomas Monjalon wrote: Anyway, why not fixing it in the reverse way: returning error for out of range of non-jumbo frames? I guess we need to fix most of the examples then, since most of them just pass 0 for normal frames. And there is no default for jumbo fram

Re: [dpdk-dev] [PATCH v2] ether: use a default for max Rx frame size in configure()

2017-04-06 Thread Thomas Monjalon
2017-03-24 12:52, Andriy Berestovskyy: > At the moment rte_eth_dev_configure() behaves inconsistent: > - for normal frames: out of range max_rx_pkt_len uses a default > - for jumbo frames: out of range max_rx_pkt_len gives an error > > This patch fixes this inconsistency by using a default value

Re: [dpdk-dev] [PATCH v2] ether: use a default for max Rx frame size in configure()

2017-03-27 Thread Andriy Berestovskyy
Hey Qiming, On 27.03.2017 08:15, Yang, Qiming wrote: I don't think this is a bug. Return errors when configure an invalid max_rx_pkt_len is suitable for this generic API. It is not a bug, it is an inconsistency. At the moment we can set max_rx_pkt_len for normal frames and if it is out of ra

Re: [dpdk-dev] [PATCH v2] ether: use a default for max Rx frame size in configure()

2017-03-26 Thread Yang, Qiming
pdk.org > Subject: [dpdk-dev] [PATCH v2] ether: use a default for max Rx frame size in > configure() > > At the moment behaves inconsistent: > - for normal frames: out of range max_rx_pkt_len uses a default > - for jumbo frames: out of range max_rx_pkt_len gives an error >

[dpdk-dev] [PATCH v2] ether: use a default for max Rx frame size in configure()

2017-03-24 Thread Andriy Berestovskyy
At the moment rte_eth_dev_configure() behaves inconsistent: - for normal frames: out of range max_rx_pkt_len uses a default - for jumbo frames: out of range max_rx_pkt_len gives an error This patch fixes this inconsistency by using a default value for max_rx_pkt_len both for normal and jumbo fra