Re: [PATCH 2/2] doc: update MLX5 LRO limitation

2022-11-22 Thread Thomas Monjalon
22/11/2022 06:17, Gregory Etelson: > Hello Thomas, > > > > .. note:: > > > > > > MAC addresses not already present in the bridge table of the > > associated > > > > If you would like to read the doc, I guess you'd prefer to find this info > > in the section dedicated to LRO, not in a random

RE: [PATCH 2/2] doc: update MLX5 LRO limitation

2022-11-21 Thread Gregory Etelson
Hello Thomas, > > .. note:: > > > > MAC addresses not already present in the bridge table of the > associated > > If you would like to read the doc, I guess you'd prefer to find this info > in the section dedicated to LRO, not in a random place. > I moved the patch location in v2 Regards,

Re: [PATCH 2/2] doc: update MLX5 LRO limitation

2022-11-21 Thread Thomas Monjalon
17/11/2022 15:39, Gregory Etelson: > Maximal LRO message size must be multiply of 256. > Otherwise, TCP payload may not fit into a single WQE. > > Cc: sta...@dpdk.org > Signed-off-by: Gregory Etelson > Acked-by: Matan Azrad Why the doc update is not in the same patch as the code change? > @@ -

[PATCH 2/2] doc: update MLX5 LRO limitation

2022-11-17 Thread Gregory Etelson
Maximal LRO message size must be multiply of 256. Otherwise, TCP payload may not fit into a single WQE. Cc: sta...@dpdk.org Signed-off-by: Gregory Etelson Acked-by: Matan Azrad --- doc/guides/nics/mlx5.rst | 3 +++ 1 file changed, 3 insertions(+) diff --git a/doc/guides/nics/mlx5.rst b/doc/gui