Message ID | 1444989651-6236-3-git-send-email-tomaszx.kulasek@intel.com (mailing list archive) |
---|---|
State | Superseded, archived |
Headers |
Return-Path: <dev-bounces@dpdk.org> X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [IPv6:::1]) by dpdk.org (Postfix) with ESMTP id DA6928E95; Fri, 16 Oct 2015 12:05:21 +0200 (CEST) Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by dpdk.org (Postfix) with ESMTP id 4365B8E92 for <dev@dpdk.org>; Fri, 16 Oct 2015 12:05:20 +0200 (CEST) Received: from orsmga002.jf.intel.com ([10.7.209.21]) by orsmga101.jf.intel.com with ESMTP; 16 Oct 2015 03:05:19 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.17,688,1437462000"; d="scan'208";a="828137429" Received: from unknown (HELO Sent) ([10.217.248.129]) by orsmga002.jf.intel.com with SMTP; 16 Oct 2015 03:05:17 -0700 Received: by Sent (sSMTP sendmail emulation); Fri, 16 Oct 2015 12:03:51 +0200 From: Tomasz Kulasek <tomaszx.kulasek@intel.com> To: dev@dpdk.org Date: Fri, 16 Oct 2015 12:00:44 +0200 Message-Id: <1444989651-6236-3-git-send-email-tomaszx.kulasek@intel.com> X-Mailer: git-send-email 2.1.4 In-Reply-To: <1444989651-6236-1-git-send-email-tomaszx.kulasek@intel.com> References: <1443621905-7872-1-git-send-email-tomaszx.kulasek@intel.com> <1444989651-6236-1-git-send-email-tomaszx.kulasek@intel.com> Subject: [dpdk-dev] [PATCH v6 2/9] null: fix segfault when null_pmd added to bonding X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK <dev.dpdk.org> List-Unsubscribe: <http://dpdk.org/ml/options/dev>, <mailto:dev-request@dpdk.org?subject=unsubscribe> List-Archive: <http://dpdk.org/ml/archives/dev/> List-Post: <mailto:dev@dpdk.org> List-Help: <mailto:dev-request@dpdk.org?subject=help> List-Subscribe: <http://dpdk.org/ml/listinfo/dev>, <mailto:dev-request@dpdk.org?subject=subscribe> Errors-To: dev-bounces@dpdk.org Sender: "dev" <dev-bounces@dpdk.org> |
Commit Message
Tomasz Kulasek
Oct. 16, 2015, 10 a.m. UTC
This patch initializes eth_dev->link_intr_cbs queue used when null pmd is
added to the bonding.
v5 changes:
- removed unnecessary malloc for eth_driver (rte_null_pmd)
Signed-off-by: Tomasz Kulasek <tomaszx.kulasek@intel.com>
---
drivers/net/null/rte_eth_null.c | 2 ++
1 file changed, 2 insertions(+)
Comments
Hi, There is no change in v6 for this patch which was acked by Tetsuya. So why not keep the Acked-by below your Signed-off-by? It seems patches 2, 3, 4 and 5 were Acked by Tetsuya. Other acks I'm missing? 2015-10-16 12:00, Tomasz Kulasek: > This patch initializes eth_dev->link_intr_cbs queue used when null pmd is > added to the bonding. > > v5 changes: > - removed unnecessary malloc for eth_driver (rte_null_pmd) > > Signed-off-by: Tomasz Kulasek <tomaszx.kulasek@intel.com>
> -----Original Message----- > From: Thomas Monjalon [mailto:thomas.monjalon@6wind.com] > Sent: Tuesday, October 27, 2015 17:59 > To: Kulasek, TomaszX > Cc: dev@dpdk.org; Tetsuya Mukawa > Subject: Re: [dpdk-dev] [PATCH v6 2/9] null: fix segfault when null_pmd > added to bonding > > Hi, > There is no change in v6 for this patch which was acked by Tetsuya. > So why not keep the Acked-by below your Signed-off-by? > > It seems patches 2, 3, 4 and 5 were Acked by Tetsuya. > Other acks I'm missing? > Hi, Patches 4 and 5 were changed due to the Tetsuya's suggestions and already reviewed. There are not big changes, but I'm not sure if it should be reacked by Tetsuya, or I can just copy ack? Tomasz
On 2015/10/28 22:43, Kulasek, TomaszX wrote: >> -----Original Message----- >> From: Thomas Monjalon [mailto:thomas.monjalon@6wind.com] >> Sent: Tuesday, October 27, 2015 17:59 >> To: Kulasek, TomaszX >> Cc: dev@dpdk.org; Tetsuya Mukawa >> Subject: Re: [dpdk-dev] [PATCH v6 2/9] null: fix segfault when null_pmd >> added to bonding >> >> Hi, >> There is no change in v6 for this patch which was acked by Tetsuya. >> So why not keep the Acked-by below your Signed-off-by? >> >> It seems patches 2, 3, 4 and 5 were Acked by Tetsuya. >> Other acks I'm missing? >> > Hi, > > Patches 4 and 5 were changed due to the Tetsuya's suggestions and already reviewed. There are not big changes, but I'm not sure if it should be reacked by Tetsuya, or I can just copy ack? > > Tomasz Hi Tomasz, Could you please check Bernard's patch series? - [PATCH v5 00/28] remove pci driver from vdevs He tries to remove pci_drv from PMD. It seems your below patch will be affected by his patch. - null: fix segfault when null_pmd added to bonding I just wonder your bonding RSS patch works without below fixing. drivers/net/null/rte_eth_null.c : + pci_dev->driver = &rte_null_pmd.pci_drv; Anyway, Could you please check him patches? Thanks, Tetsuya
> -----Original Message----- > From: Tetsuya Mukawa [mailto:mukawa@igel.co.jp] > Sent: Thursday, October 29, 2015 08:19 > To: Kulasek, TomaszX > Cc: Thomas Monjalon; dev@dpdk.org > Subject: Re: [dpdk-dev] [PATCH v6 2/9] null: fix segfault when null_pmd > added to bonding > > On 2015/10/28 22:43, Kulasek, TomaszX wrote: > >> -----Original Message----- > >> From: Thomas Monjalon [mailto:thomas.monjalon@6wind.com] > >> Sent: Tuesday, October 27, 2015 17:59 > >> To: Kulasek, TomaszX > >> Cc: dev@dpdk.org; Tetsuya Mukawa > >> Subject: Re: [dpdk-dev] [PATCH v6 2/9] null: fix segfault when > >> null_pmd added to bonding > >> > >> Hi, > >> There is no change in v6 for this patch which was acked by Tetsuya. > >> So why not keep the Acked-by below your Signed-off-by? > >> > >> It seems patches 2, 3, 4 and 5 were Acked by Tetsuya. > >> Other acks I'm missing? > >> > > Hi, > > > > Patches 4 and 5 were changed due to the Tetsuya's suggestions and > already reviewed. There are not big changes, but I'm not sure if it > should be reacked by Tetsuya, or I can just copy ack? > > > > Tomasz > > Hi Tomasz, > > Could you please check Bernard's patch series? > - [PATCH v5 00/28] remove pci driver from vdevs He tries to remove > pci_drv from PMD. > > It seems your below patch will be affected by his patch. > - null: fix segfault when null_pmd added to bonding > > I just wonder your bonding RSS patch works without below fixing. > drivers/net/null/rte_eth_null.c : > + pci_dev->driver = &rte_null_pmd.pci_drv; > > Anyway, Could you please check him patches? > > Thanks, > Tetsuya Hi Tetsuya, I've just checked Bernard's patches and it seems that it can be safely removed from here. It's not used in RSS implementation, so have no impact on it. I'm sending patchset v7 with copied your acks, as Thomas suggested, if you have no other objections. Tomasz
diff --git a/drivers/net/null/rte_eth_null.c b/drivers/net/null/rte_eth_null.c index e244595..c748101 100644 --- a/drivers/net/null/rte_eth_null.c +++ b/drivers/net/null/rte_eth_null.c @@ -432,6 +432,7 @@ eth_dev_null_create(const char *name, internals->numa_node = numa_node; pci_dev->numa_node = numa_node; + pci_dev->driver = &rte_null_pmd.pci_drv; data->dev_private = internals; data->port_id = eth_dev->data->port_id; @@ -445,6 +446,7 @@ eth_dev_null_create(const char *name, eth_dev->dev_ops = &ops; eth_dev->pci_dev = pci_dev; eth_dev->driver = &rte_null_pmd; + TAILQ_INIT(ð_dev->link_intr_cbs); /* finally assign rx and tx ops */ if (packet_copy) {