On 2/2/2017 10:34 AM, Nelio Laranjeiro wrote:
> First two bytes of the Ethernet header was written twice at the same place.

Is this patch just prevents re-writing 2 bytes of buffer, or changes the
buffer content as well?

If buffer content also updated, I think it would be nice to mention in
the commit log.

And if buffer content is not changed, will it be fair to say this patch
is refactor patch instead of fix?

> 
> Fixes: b8fe952ec5b6 ("net/mlx5: prepare Tx vectorization")
> 
> Signed-off-by: Yongseok Koh <ys...@mellanox.com>
> Signed-off-by: Nelio Laranjeiro <nelio.laranje...@6wind.com>
> ---
<...>

Reply via email to