From mboxrd@z Thu Jan  1 00:00:00 1970
From: "Burakov, Anatoly" <anatoly.burakov@intel.com>
Subject: Re: [PATCH dpdk 2/5] pci: Initialize common rte
	driver	pointer
Date: Mon, 24 Apr 2017 09:28:23 +0000
Message-ID: <C6ECDF3AB251BE4894318F4E45123697822102EE@IRSMSX109.ger.corp.intel.com>
References: <20170420072402.38106-1-aik@ozlabs.ru>
 <20170420072402.38106-3-aik@ozlabs.ru>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
Cc: "JPF@zurich.ibm.com" <JPF@zurich.ibm.com>, Gowrishankar Muthukrishnan
 <gowrishankar.m@in.ibm.com>
To: Alexey Kardashevskiy <aik@ozlabs.ru>, "dev@dpdk.org" <dev@dpdk.org>
Return-path: <dev-bounces@dpdk.org>
Received: from mga09.intel.com (mga09.intel.com [134.134.136.24])
 by dpdk.org (Postfix) with ESMTP id A5C45298F
 for <dev@dpdk.org>; Mon, 24 Apr 2017 11:28:27 +0200 (CEST)
In-Reply-To: <20170420072402.38106-3-aik@ozlabs.ru>
Content-Language: en-US
List-Id: DPDK patches and discussions <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>

> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Alexey
> Kardashevskiy
> Sent: Thursday, April 20, 2017 8:24 AM
> To: dev@dpdk.org
> Cc: Alexey Kardashevskiy <aik@ozlabs.ru>; JPF@zurich.ibm.com;
> Gowrishankar Muthukrishnan <gowrishankar.m@in.ibm.com>
> Subject: [dpdk-dev] [PATCH dpdk 2/5] pci: Initialize common rte driver
> pointer
>=20
> The existing code initializes a PCI driver pointer but not the common one=
.
> As the result, ring_dma_zone_reserve() in drivers/net/bnx2x/bnx2x_rxtx.c
> crashed as dev->device->driver=3D=3DNULL.
>=20
> This adds missing initialization.
>=20
> Signed-off-by: Alexey Kardashevskiy <aik@ozlabs.ru>

This and 3rd patch - could you please split them so that there are no unrel=
ated patches in the patchset? I.e. a patchset for VFIO changes, a separate =
patch for PCI, and a patch for bnx2x driver, and then just make a note in t=
he annotation that they are interdependent.

Thanks,
Anatoly