[dpdk-dev] [PATCH v7 22/25] mlx5: work around spurious compilation errors
Bruce Richardson
bruce.richardson at intel.com
Mon Jun 27 14:12:15 CEST 2016
On Fri, Jun 24, 2016 at 03:18:01PM +0200, Nelio Laranjeiro wrote:
> From: Adrien Mazarguil <adrien.mazarguil at 6wind.com>
>
> Since commit "mlx5: resurrect Tx gather support", older GCC versions (such
> as 4.8.5) may complain about the following:
>
> mlx5_rxtx.c: In function `mlx5_tx_burst':
> mlx5_rxtx.c:705:25: error: `wqe' may be used uninitialized in this
> function [-Werror=maybe-uninitialized]
>
> mlx5_rxtx.c: In function `mlx5_tx_burst_inline':
> mlx5_rxtx.c:864:25: error: `wqe' may be used uninitialized in this
> function [-Werror=maybe-uninitialized]
>
> In both cases, this code cannot be reached when wqe is not initialized.
>
> Considering older GCC versions are still widely used, work around this
> issue by initializing wqe preemptively, even if it should not be necessary.
>
> Signed-off-by: Adrien Mazarguil <adrien.mazarguil at 6wind.com>
Since the previous patch is known to break the build on older compilers, I think
it would be better to merge this patch into the previous to prevent the break
from occurring in the first place. That will help with any future git-bisects
that people run.
If there is no objection, and no other issues appear requiring a new rev of
this patchset, I'll merge patches 21 and 22 on apply.
Regards,
/Bruce
More information about the dev
mailing list