On 04/17, David Marchand wrote: >On Wed, Apr 17, 2019 at 3:55 PM Xiaolong Ye <xiaolong...@intel.com> wrote: > >> This patchset provides some fixes to af_xdp pmd, at first, I just added >> a simple error handling when Tx queue allocation fails, then David >> suggested a better way to do it and pointed out the inconsistent issue >> of reserve/submit ops (for Tx queue) and peek/release ops (for Rx >> queue), the third patch addressed this. >> >> v4 changes: >> >> - remove unnecessary mtu valid check in eth_dev_mtu_set >> - add Reported-by and Reviewed-by tags of David >> >> v3 changes: >> >> - address David's review comments >> - add one patch to specify the mtu range >> - add one fix patch for typo >> >> v2 changes: >> >> - adopt David's suggestion to refactor the code >> >> Xiaolong Ye (4): >> net/af_xdp: enqueue buf ring when allocate Tx queue fails >> net/af_xdp: specify minimal and maximal MTU >> net/af_xdp: make reserve/submit peek/release consistent >> net/af_xdp: fix typos in Rx function >> >> drivers/net/af_xdp/rte_eth_af_xdp.c | 87 +++++++++++++++-------------- >> 1 file changed, 46 insertions(+), 41 deletions(-) >> >> -- >> 2.17.1 >> >> >LGTM. >Thanks Xiaolong.
Thanks a lot for your guidance. :) > > >-- >David Marchand
