On 11/17/2016 2:24 PM, Jan Blunck wrote: > Lets clear the eth_dev->data when allocating a new rte_eth_dev so that > drivers only need to set non-zero values. > > Signed-off-by: Jan Blunck <jblunck at infradead.org> > --- > drivers/net/mlx4/mlx4.c | 1 - > drivers/net/mlx5/mlx5.c | 1 - > lib/librte_ether/rte_ethdev.c | 2 +-
+ drivers/net/bonding/rte_eth_bond_api.c ? - eth_dev->data->dev_link.link_status = ETH_LINK_DOWN; - ... - eth_dev->data->dev_started = 0; - eth_dev->data->promiscuous = 0; - eth_dev->data->scattered_rx = 0; - eth_dev->data->all_multicast = 0; - + drivers/net/mpipe/mpipe_tilegx.c ? - eth_dev->data->dev_flags = 0; <...> > @@ -212,6 +212,7 @@ rte_eth_dev_allocate(const char *name) > > eth_dev = &rte_eth_devices[port_id]; > eth_dev->data = &rte_eth_dev_data[port_id]; > + memset(eth_dev->data, 0, sizeof(*eth_dev->data)); > snprintf(eth_dev->data->name, sizeof(eth_dev->data->name), "%s", name); > eth_dev->data->port_id = port_id; > eth_dev->attached = DEV_ATTACHED; > @@ -259,7 +260,6 @@ rte_eth_dev_pci_probe(struct rte_pci_driver *pci_drv, > } > eth_dev->pci_dev = pci_dev; > eth_dev->driver = eth_drv; > - eth_dev->data->rx_mbuf_alloc_failed = 0; > > /* init user callbacks */ > TAILQ_INIT(&(eth_dev->link_intr_cbs)); >
