From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wi0-f175.google.com (mail-wi0-f175.google.com [209.85.212.175]) by dpdk.org (Postfix) with ESMTP id 5965BC35C for ; Wed, 15 Apr 2015 22:50:15 +0200 (CEST) Received: by wiun10 with SMTP id n10so74316819wiu.1 for ; Wed, 15 Apr 2015 13:50:15 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=k1xMRz9v6TJ65LNCqVE/rVbjjL8drLuojHaj2qPtPOs=; b=lW7z1YN+pOwr81LRMX7bYKN8Bijiijeaj5Dqm/ODSyiI5aTg2m20/KoLykjkjwh2+e OidxAleaD2MhRTB1yoNBAbzFeVzAvb7ZN8+b9ImW01OvrnhNvAS76pv3vxJcFH0FX1Fa ZZ8+W9b54Jj+EINBZ1/bl2UmstMa1GxuoSsVszs1sacAkCzalryhW3atCxE1y5Hoht1Z EGB5MNBLQmzfGyvunq4ioIs/10PyFncWRyQ/yUvYP4nBm4ZVjYNb+RW8BTNevP97M3pj U7R1vu0h8Pq6/fm6srsePCwdCH6VinUTTrYLBt8Ou5v5F8F3Gjp+NjkJl2wqtFa2ysWn S9dg== X-Gm-Message-State: ALoCoQkl1EgHtAnS4ZJxUFjD2QhNU1vy+jXyk79WD8Jv/ceS305Di7DQrMB3Z3p5pqKi+5ERxJAs X-Received: by 10.194.6.37 with SMTP id x5mr54901931wjx.73.1429131015212; Wed, 15 Apr 2015 13:50:15 -0700 (PDT) Received: from localhost.localdomain (136-92-190-109.dsl.ovh.fr. [109.190.92.136]) by mx.google.com with ESMTPSA id pv2sm7480174wjc.33.2015.04.15.13.50.13 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Wed, 15 Apr 2015 13:50:14 -0700 (PDT) From: Thomas Monjalon To: dev@dpdk.org Date: Wed, 15 Apr 2015 22:49:16 +0200 Message-Id: <1429130956-17828-2-git-send-email-thomas.monjalon@6wind.com> X-Mailer: git-send-email 2.2.2 In-Reply-To: <1429130956-17828-1-git-send-email-thomas.monjalon@6wind.com> References: <1958525.YbKd0lDtje@xps13> <1429130956-17828-1-git-send-email-thomas.monjalon@6wind.com> Subject: [dpdk-dev] [PATCH v2 2/2] use simple zero initializers X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 15 Apr 2015 20:50:15 -0000 To initialize a structure with zeros, one field was explicitly set to avoid "missing initializer" bug with old GCC (e.g. 4.4). This warning is now disabled (commit ) for old versions of GCC, so the workarounds may be removed. These initializers should not be needed for static variables but they are still used to workaround an ICC bug (see commit b2595c4aa92d). There is one remaining exception where {0} initializer doesn't work cleanly, even with recent GCC: lib/librte_pmd_ixgbe/ixgbe_rxtx_vec.c:735:9: error: missing braces around initializer [-Werror=missing-braces] struct rte_mbuf mb_def = {0}; /* zeroed mbuf */ Tested with GCC 4.4.7 (CentOS), 4.7.2 (Debian) and 4.9.2 (Arch). Signed-off-by: Thomas Monjalon --- changes in v2: - new patch app/test/test_ring_perf.c | 2 +- lib/librte_pmd_e1000/em_ethdev.c | 2 +- lib/librte_pmd_e1000/igb_ethdev.c | 4 ++-- lib/librte_pmd_e1000/igb_rxtx.c | 6 ++---- lib/librte_pmd_enic/enic_clsf.c | 2 +- lib/librte_pmd_i40e/i40e_rxtx.c | 2 +- lib/librte_pmd_ixgbe/ixgbe_rxtx.c | 8 +++----- lib/librte_pmd_ixgbe/ixgbe_rxtx_vec.c | 3 +-- lib/librte_pmd_mlx4/mlx4.c | 2 +- 9 files changed, 13 insertions(+), 18 deletions(-) diff --git a/app/test/test_ring_perf.c b/app/test/test_ring_perf.c index 44dda4d..8c47ccb 100644 --- a/app/test/test_ring_perf.c +++ b/app/test/test_ring_perf.c @@ -253,7 +253,7 @@ static void run_on_core_pair(struct lcore_pair *cores, lcore_function_t f1, lcore_function_t f2) { - struct thread_params param1 = {.size = 0}, param2 = {.size = 0}; + struct thread_params param1 = {0}, param2 = {0}; unsigned i; for (i = 0; i < sizeof(bulk_sizes)/sizeof(bulk_sizes[0]); i++) { lcore_count = 0; diff --git a/lib/librte_pmd_e1000/em_ethdev.c b/lib/librte_pmd_e1000/em_ethdev.c index 12ecf5f..82e0b7a 100644 --- a/lib/librte_pmd_e1000/em_ethdev.c +++ b/lib/librte_pmd_e1000/em_ethdev.c @@ -130,7 +130,7 @@ static struct rte_pci_id pci_id_em_map[] = { #define RTE_PCI_DEV_ID_DECL_EM(vend, dev) {RTE_PCI_DEVICE(vend, dev)}, #include "rte_pci_dev_ids.h" -{.device_id = 0}, +{0}, }; static const struct eth_dev_ops eth_em_ops = { diff --git a/lib/librte_pmd_e1000/igb_ethdev.c b/lib/librte_pmd_e1000/igb_ethdev.c index 1ea2d38..e2b7cf3 100644 --- a/lib/librte_pmd_e1000/igb_ethdev.c +++ b/lib/librte_pmd_e1000/igb_ethdev.c @@ -221,7 +221,7 @@ static struct rte_pci_id pci_id_igb_map[] = { #define RTE_PCI_DEV_ID_DECL_IGB(vend, dev) {RTE_PCI_DEVICE(vend, dev)}, #include "rte_pci_dev_ids.h" -{.device_id = 0}, +{0}, }; /* @@ -232,7 +232,7 @@ static struct rte_pci_id pci_id_igbvf_map[] = { #define RTE_PCI_DEV_ID_DECL_IGBVF(vend, dev) {RTE_PCI_DEVICE(vend, dev)}, #include "rte_pci_dev_ids.h" -{.device_id = 0}, +{0}, }; static const struct eth_dev_ops eth_igb_ops = { diff --git a/lib/librte_pmd_e1000/igb_rxtx.c b/lib/librte_pmd_e1000/igb_rxtx.c index 946b39d..084e45a 100644 --- a/lib/librte_pmd_e1000/igb_rxtx.c +++ b/lib/librte_pmd_e1000/igb_rxtx.c @@ -1164,8 +1164,7 @@ igb_reset_tx_queue_stat(struct igb_tx_queue *txq) static void igb_reset_tx_queue(struct igb_tx_queue *txq, struct rte_eth_dev *dev) { - static const union e1000_adv_tx_desc zeroed_desc = { .read = { - .buffer_addr = 0}}; + static const union e1000_adv_tx_desc zeroed_desc = {{0}}; struct igb_tx_entry *txe = txq->sw_ring; uint16_t i, prev; struct e1000_hw *hw; @@ -1330,8 +1329,7 @@ eth_igb_rx_queue_release(void *rxq) static void igb_reset_rx_queue(struct igb_rx_queue *rxq) { - static const union e1000_adv_rx_desc zeroed_desc = { .read = { - .pkt_addr = 0}}; + static const union e1000_adv_rx_desc zeroed_desc = {{0}}; unsigned i; /* Zero out HW ring memory */ diff --git a/lib/librte_pmd_enic/enic_clsf.c b/lib/librte_pmd_enic/enic_clsf.c index b61d625..a069194 100644 --- a/lib/librte_pmd_enic/enic_clsf.c +++ b/lib/librte_pmd_enic/enic_clsf.c @@ -96,7 +96,7 @@ int enic_fdir_add_fltr(struct enic *enic, struct rte_fdir_filter *params, u16 queue, u8 drop) { struct enic_fdir_node *key; - struct filter fltr = {.type = 0}; + struct filter fltr = {0}; int32_t pos; u8 do_free = 0; u16 old_fltr_id = 0; diff --git a/lib/librte_pmd_i40e/i40e_rxtx.c b/lib/librte_pmd_i40e/i40e_rxtx.c index 9c7be6f..abe68f4 100644 --- a/lib/librte_pmd_i40e/i40e_rxtx.c +++ b/lib/librte_pmd_i40e/i40e_rxtx.c @@ -1228,7 +1228,7 @@ i40e_xmit_pkts(void *tx_queue, struct rte_mbuf **tx_pkts, uint16_t nb_pkts) uint16_t tx_last; uint16_t slen; uint64_t buf_dma_addr; - union i40e_tx_offload tx_offload = { .data = 0 }; + union i40e_tx_offload tx_offload = {0}; txq = tx_queue; sw_ring = txq->sw_ring; diff --git a/lib/librte_pmd_ixgbe/ixgbe_rxtx.c b/lib/librte_pmd_ixgbe/ixgbe_rxtx.c index 6475c44..19286c9 100644 --- a/lib/librte_pmd_ixgbe/ixgbe_rxtx.c +++ b/lib/librte_pmd_ixgbe/ixgbe_rxtx.c @@ -579,7 +579,7 @@ ixgbe_xmit_pkts(void *tx_queue, struct rte_mbuf **tx_pkts, uint64_t tx_ol_req; uint32_t ctx = 0; uint32_t new_ctx; - union ixgbe_tx_offload tx_offload = { .data = 0 }; + union ixgbe_tx_offload tx_offload = {0}; txq = tx_queue; sw_ring = txq->sw_ring; @@ -2052,8 +2052,7 @@ ixgbe_dev_tx_queue_release(void *txq) static void ixgbe_reset_tx_queue(struct ixgbe_tx_queue *txq) { - static const union ixgbe_adv_tx_desc zeroed_desc = { .read = { - .buffer_addr = 0}}; + static const union ixgbe_adv_tx_desc zeroed_desc = {{0}}; struct ixgbe_tx_entry *txe = txq->sw_ring; uint16_t prev, i; @@ -2445,8 +2444,7 @@ check_rx_burst_bulk_alloc_preconditions(__rte_unused struct ixgbe_rx_queue *rxq) static void ixgbe_reset_rx_queue(struct ixgbe_hw *hw, struct ixgbe_rx_queue *rxq) { - static const union ixgbe_adv_rx_desc zeroed_desc = { .read = { - .pkt_addr = 0}}; + static const union ixgbe_adv_rx_desc zeroed_desc = {{0}}; unsigned i; uint16_t len = rxq->nb_rx_desc; diff --git a/lib/librte_pmd_ixgbe/ixgbe_rxtx_vec.c b/lib/librte_pmd_ixgbe/ixgbe_rxtx_vec.c index 7ac6b61..abd10f6 100644 --- a/lib/librte_pmd_ixgbe/ixgbe_rxtx_vec.c +++ b/lib/librte_pmd_ixgbe/ixgbe_rxtx_vec.c @@ -691,8 +691,7 @@ ixgbe_tx_free_swring(struct ixgbe_tx_queue *txq) static void ixgbe_reset_tx_queue(struct ixgbe_tx_queue *txq) { - static const union ixgbe_adv_tx_desc zeroed_desc = { .read = { - .buffer_addr = 0} }; + static const union ixgbe_adv_tx_desc zeroed_desc = {{0}}; struct ixgbe_tx_entry_v *txe = (struct ixgbe_tx_entry_v *)txq->sw_ring; uint16_t i; diff --git a/lib/librte_pmd_mlx4/mlx4.c b/lib/librte_pmd_mlx4/mlx4.c index e096071..024282a 100644 --- a/lib/librte_pmd_mlx4/mlx4.c +++ b/lib/librte_pmd_mlx4/mlx4.c @@ -3497,7 +3497,7 @@ static void mlx4_stats_get(struct rte_eth_dev *dev, struct rte_eth_stats *stats) { struct priv *priv = dev->data->dev_private; - struct rte_eth_stats tmp = { .ipackets = 0 }; + struct rte_eth_stats tmp = {0}; unsigned int i; unsigned int idx; -- 2.2.2