On 9/18/2018 7:33 PM, Thomas Monjalon wrote:
18/09/2018 15:31, Hemant Agrawal:
Avoid return after the jumbo buffer config in dev config API
Fixes: 9658ac3a4ef6 ("net/dpaa: set the correct frame size in device MTU")
Cc: sta...@dpdk.org
Signed-off-by: Hemant Agrawal <hemant.agra...@nxp.com>
Thanks for adding some comments in this series.
About this fix, would it be easier to understand if explaining
what is the bug first?
yes. I can be more specific here.
1. the dev->data->mtu was not getting updated earlier for the jumbo
buffer config.
2. we don't expect to return error, if this config fails. A debug err
log is ok. DPAA1 - supports jumbo by default, enable/disable may give
errors.