Re: [dpdk-dev] [PATCH v2 03/10] app/eventdev: fix maybe-uninitialized warnings for LTO build

[Jerin Jacob <jerinjacobk@gmail.com>]

On Tue, Sep 17, 2019 at 1:28 PM Andrzej Ostruszka <amo@semihalf.com> wrote:
>
> During LTO build compiler reports some 'false positive' warnings about
> variables being possibly used uninitialized.  This patch silences these
> warnings.
>
> Signed-off-by: Andrzej Ostruszka <aostruszka@marvell.com>

# Please fix check-git-log.sh errors.
Missing 'Fixes' tag:
        app/eventdev: fix maybe-uninitialized warnings for LTO build

# Please add the compiler 'false positive' warning output in git commit.

# Since it is a slow path change, IMO there is no harm in
introducing this change to make compiler happy.

With the above fix:
Reviewed-by: Jerin Jacob <jerinj@marvell.com>


> ---
>  app/test-eventdev/test_perf_common.c     | 2 +-
>  app/test-eventdev/test_pipeline_common.c | 4 ++--
>  2 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/app/test-eventdev/test_perf_common.c b/app/test-eventdev/test_perf_common.c
> index aa925a7ef..a974685cb 100644
> --- a/app/test-eventdev/test_perf_common.c
> +++ b/app/test-eventdev/test_perf_common.c
> @@ -439,7 +439,7 @@ perf_event_timer_adapter_setup(struct test_perf *t)
>
>                 if (!(adapter_info.caps &
>                                 RTE_EVENT_TIMER_ADAPTER_CAP_INTERNAL_PORT)) {
> -                       uint32_t service_id;
> +                       uint32_t service_id = -1U;
>
>                         rte_event_timer_adapter_service_id_get(wl,
>                                         &service_id);
> diff --git a/app/test-eventdev/test_pipeline_common.c b/app/test-eventdev/test_pipeline_common.c
> index 16c49b860..813d0cf44 100644
> --- a/app/test-eventdev/test_pipeline_common.c
> +++ b/app/test-eventdev/test_pipeline_common.c
> @@ -306,7 +306,7 @@ pipeline_event_rx_adapter_setup(struct evt_options *opt, uint8_t stride,
>                 }
>
>                 if (!(cap & RTE_EVENT_ETH_RX_ADAPTER_CAP_INTERNAL_PORT)) {
> -                       uint32_t service_id;
> +                       uint32_t service_id = -1U;
>
>                         rte_event_eth_rx_adapter_service_id_get(prod,
>                                         &service_id);
> @@ -358,7 +358,7 @@ pipeline_event_tx_adapter_setup(struct evt_options *opt,
>                 }
>
>                 if (!(cap & RTE_EVENT_ETH_TX_ADAPTER_CAP_INTERNAL_PORT)) {
> -                       uint32_t service_id;
> +                       uint32_t service_id = -1U;
>
>                         rte_event_eth_tx_adapter_service_id_get(consm,
>                                         &service_id);
> --
> 2.17.1
>