From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx1.redhat.com (mx3-rdu2.redhat.com [66.187.233.73]) by dpdk.org (Postfix) with ESMTP id 5A34E8E76 for ; Tue, 17 Apr 2018 10:24:36 +0200 (CEST) Received: from smtp.corp.redhat.com (int-mx05.intmail.prod.int.rdu2.redhat.com [10.11.54.5]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id E49014068046; Tue, 17 Apr 2018 08:24:35 +0000 (UTC) Received: from [10.36.112.51] (ovpn-112-51.ams2.redhat.com [10.36.112.51]) by smtp.corp.redhat.com (Postfix) with ESMTPS id A1EA07C2D; Tue, 17 Apr 2018 08:24:34 +0000 (UTC) To: Pavan Nikhilesh , jerin.jacob@caviumnetworks.com, santosh.shukla@caviumnetworks.com, erik.g.carrillo@intel.com Cc: dev@dpdk.org References: <20180318131216.11699-1-pbhagavatula@caviumnetworks.com> <20180406151324.9135-1-pbhagavatula@caviumnetworks.com> <7100b73d-de02-c70e-774f-2c254b63b7ec@redhat.com> <20180417082209.GA3161@ltp-pvn> From: Maxime Coquelin Message-ID: <1d6519ba-ebf5-cb53-567d-41f146a295c7@redhat.com> Date: Tue, 17 Apr 2018 10:24:32 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.7.0 MIME-Version: 1.0 In-Reply-To: <20180417082209.GA3161@ltp-pvn> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 8bit X-Scanned-By: MIMEDefang 2.79 on 10.11.54.5 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.11.55.5]); Tue, 17 Apr 2018 08:24:35 +0000 (UTC) X-Greylist: inspected by milter-greylist-4.5.16 (mx1.redhat.com [10.11.55.5]); Tue, 17 Apr 2018 08:24:35 +0000 (UTC) for IP:'10.11.54.5' DOMAIN:'int-mx05.intmail.prod.int.rdu2.redhat.com' HELO:'smtp.corp.redhat.com' FROM:'maxime.coquelin@redhat.com' RCPT:'' Subject: Re: [dpdk-dev] [PATCH v5 1/4] app/eventdev: add event timer adapter as a producer X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 17 Apr 2018 08:24:36 -0000 On 04/17/2018 10:22 AM, Pavan Nikhilesh wrote: > >> It seems your patch introduce build regression on RHEL 7.5: >> $ gcc --version >> gcc (GCC) 4.8.5 20150623 (Red Hat 4.8.5-28) >> >> $ make -j32 install T=x86_64-native-linuxapp-gcc DESTDIR=legacy_destdir >> >> == Build app/test-eventdev >> CC test_perf_common.o >> /tmp/dpdk_build/app/test-eventdev/test_perf_common.c: In function >> ‘perf_event_timer_producer’: >> /tmp/dpdk_build/app/test-eventdev/test_perf_common.c:99:3: error: missing >> initializer for field ‘priority’ of ‘struct ’ >> [-Werror=missing-field-initializers] >> .ev.sched_type = t->opt->sched_type_list[0], >> ^ >> In file included from >> /tmp/dpdk_build/app/test-eventdev/test_perf_common.h:14:0, >> from >> /tmp/dpdk_build/app/test-eventdev/test_perf_common.c:5: >> /tmp/dpdk_build/x86_64-native-linuxapp-gcc/include/rte_eventdev.h:1049:12: >> note: ‘priority’ declared here >> uint8_t priority; >> ^ >> /tmp/dpdk_build/app/test-eventdev/test_perf_common.c:100:3: error: missing >> initializer for field ‘priority’ of ‘struct ’ >> [-Werror=missing-field-initializers] >> .ev.priority = RTE_EVENT_DEV_PRIORITY_NORMAL, >> ^ >> In file included from >> /tmp/dpdk_build/app/test-eventdev/test_perf_common.h:14:0, >> from >> /tmp/dpdk_build/app/test-eventdev/test_perf_common.c:5: >> /tmp/dpdk_build/x86_64-native-linuxapp-gcc/include/rte_eventdev.h:1049:12: >> note: ‘priority’ declared here >> uint8_t priority; >> ^ >> /tmp/dpdk_build/app/test-eventdev/test_perf_common.c:101:3: error: missing >> initializer for field ‘impl_opaque’ of ‘struct ’ >> [-Werror=missing-field-initializers] >> .ev.event_type = RTE_EVENT_TYPE_TIMER, >> ^ >> In file included from >> /tmp/dpdk_build/app/test-eventdev/test_perf_common.h:14:0, >> from >> /tmp/dpdk_build/app/test-eventdev/test_perf_common.c:5: >> /tmp/dpdk_build/x86_64-native-linuxapp-gcc/include/rte_eventdev.h:1059:12: >> note: ‘impl_opaque’ declared here >> uint8_t impl_opaque; >> ^ >> /tmp/dpdk_build/app/test-eventdev/test_perf_common.c:102:3: error: missing >> initializer for field ‘impl_opaque’ of ‘struct ’ >> [-Werror=missing-field-initializers] >> .state = RTE_EVENT_TIMER_NOT_ARMED, >> ^ >> In file included from >> /tmp/dpdk_build/app/test-eventdev/test_perf_common.h:14:0, >> from >> /tmp/dpdk_build/app/test-eventdev/test_perf_common.c:5: >> /tmp/dpdk_build/x86_64-native-linuxapp-gcc/include/rte_eventdev.h:1059:12: >> note: ‘impl_opaque’ declared here >> uint8_t impl_opaque; >> ^ >> /tmp/dpdk_build/app/test-eventdev/test_perf_common.c: In function >> ‘perf_event_timer_producer_burst’: >> /tmp/dpdk_build/app/test-eventdev/test_perf_common.c:161:3: error: missing >> initializer for field ‘priority’ of ‘struct ’ >> [-Werror=missing-field-initializers] >> .ev.sched_type = t->opt->sched_type_list[0], >> ^ >> In file included from >> /tmp/dpdk_build/app/test-eventdev/test_perf_common.h:14:0, >> from >> /tmp/dpdk_build/app/test-eventdev/test_perf_common.c:5: >> /tmp/dpdk_build/x86_64-native-linuxapp-gcc/include/rte_eventdev.h:1049:12: >> note: ‘priority’ declared here >> uint8_t priority; >> ^ >> /tmp/dpdk_build/app/test-eventdev/test_perf_common.c:162:3: error: missing >> initializer for field ‘priority’ of ‘struct ’ >> [-Werror=missing-field-initializers] >> .ev.priority = RTE_EVENT_DEV_PRIORITY_NORMAL, >> ^ >> In file included from >> /tmp/dpdk_build/app/test-eventdev/test_perf_common.h:14:0, >> from >> /tmp/dpdk_build/app/test-eventdev/test_perf_common.c:5: >> /tmp/dpdk_build/x86_64-native-linuxapp-gcc/include/rte_eventdev.h:1049:12: >> note: ‘priority’ declared here >> uint8_t priority; >> ^ >> /tmp/dpdk_build/app/test-eventdev/test_perf_common.c:163:3: error: missing >> initializer for field ‘impl_opaque’ of ‘struct ’ >> [-Werror=missing-field-initializers] >> .ev.event_type = RTE_EVENT_TYPE_TIMER, >> ^ >> In file included from >> /tmp/dpdk_build/app/test-eventdev/test_perf_common.h:14:0, >> from >> /tmp/dpdk_build/app/test-eventdev/test_perf_common.c:5: >> /tmp/dpdk_build/x86_64-native-linuxapp-gcc/include/rte_eventdev.h:1059:12: >> note: ‘impl_opaque’ declared here >> uint8_t impl_opaque; >> ^ >> /tmp/dpdk_build/app/test-eventdev/test_perf_common.c:164:3: error: missing >> initializer for field ‘impl_opaque’ of ‘struct ’ >> [-Werror=missing-field-initializers] >> .state = RTE_EVENT_TIMER_NOT_ARMED, >> ^ >> In file included from >> /tmp/dpdk_build/app/test-eventdev/test_perf_common.h:14:0, >> from >> /tmp/dpdk_build/app/test-eventdev/test_perf_common.c:5: >> /tmp/dpdk_build/x86_64-native-linuxapp-gcc/include/rte_eventdev.h:1059:12: >> note: ‘impl_opaque’ declared here >> uint8_t impl_opaque; >> ^ >> cc1: all warnings being treated as errors >> make[5]: *** [test_perf_common.o] Error 1 >> make[4]: *** [test-eventdev] Error 2 >> make[3]: *** [app] Error 2 >> make[2]: *** [all] Error 2 >> make[1]: *** [pre_install] Error 2 >> make: *** [install] Error 2 > Hi Maxime, > > This is already fixed on the HEAD. > Please refer to > http://dpdk.org/dev/patchwork/patch/38189/ Thanks for the quick reply, I tested with net-next, so the patch didn't land into it yet. Regards, Maxime > >> >> Regards, >> Maxime > > Thanks, > Pavan. >