Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753073AbdGIVae (ORCPT ); Sun, 9 Jul 2017 17:30:34 -0400 Received: from mx1.redhat.com ([209.132.183.28]:37746 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752596AbdGIVab (ORCPT ); Sun, 9 Jul 2017 17:30:31 -0400 DMARC-Filter: OpenDMARC Filter v1.3.2 mx1.redhat.com B466881226 Authentication-Results: ext-mx01.extmail.prod.ext.phx2.redhat.com; dmarc=none (p=none dis=none) header.from=redhat.com Authentication-Results: ext-mx01.extmail.prod.ext.phx2.redhat.com; spf=pass smtp.mailfrom=dledford@redhat.com DKIM-Filter: OpenDKIM Filter v2.11.0 mx1.redhat.com B466881226 Message-ID: <1499635828.68875.4.camel@redhat.com> Subject: Re: [PATCH v10 00/15] Replace PCI pool by DMA pool API From: Doug Ledford To: Romain Perier , Dan Williams , Sean Hefty , Hal Rosenstock , jeffrey.t.kirsher@intel.com, "David S. Miller" , stas.yakovlev@gmail.com, "James E.J. Bottomley" , "Martin K. Petersen" , Felipe Balbi , Greg Kroah-Hartman Cc: linux-rdma@vger.kernel.org, netdev@vger.kernel.org, linux-usb@vger.kernel.org, linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org Date: Sun, 09 Jul 2017 17:30:28 -0400 In-Reply-To: <20170706081310.30964-1-romain.perier@collabora.com> References: <20170706081310.30964-1-romain.perier@collabora.com> Organization: Red Hat, Inc. Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Content-Transfer-Encoding: 7bit X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.25]); Sun, 09 Jul 2017 21:30:31 +0000 (UTC) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 4981 Lines: 125 On Thu, 2017-07-06 at 10:12 +0200, Romain Perier wrote: > The current PCI pool API are simple macro functions direct expanded > to > the appropriate dma pool functions. The prototypes are almost the > same > and semantically, they are very similar. I propose to use the DMA > pool > API directly and get rid of the old API. > > This set of patches, replaces the old API by the dma pool API > and remove the defines. Is someone planning on merging this series? If not, I'll send through the patches I've personally tested (3, 5, and 6). > Changes in v10: > - Rebased series onto next-20170706 > - I have fixed and improved patch "scsi: megaraid: Replace PCI pool > old API" > > Changes in v9: > - Rebased series onto next-20170522 > - I have fixed and improved the patch for lpfc driver > > Changes in v8: > - Rebased series onto next-20170428 > > Changes in v7: > - Rebased series onto next-20170416 > - Added Acked-by, Tested-by and Reviwed-by tags > > Changes in v6: > - Fixed an issue reported by kbuild test robot about changes in > DAC960 > - Removed patches 15/19,16/19,17/19,18/19. They have been merged by > Greg > - Added Acked-by Tags > > Changes in v5: > - Re-worded the cover letter (remove sentence about checkpatch.pl) > - Rebased series onto next-20170308 > - Fix typos in commit message > - Added Acked-by Tags > > Changes in v4: > - Rebased series onto next-20170301 > - Removed patch 20/20: checks done by checkpath.pl, no longer > required. > Thanks to Peter and Joe for their feedbacks. > - Added Reviewed-by tags > > Changes in v3: > - Rebased series onto next-20170224 > - Fix checkpath.pl reports for patch 11/20 and patch 12/20 > - Remove prefix RFC > Changes in v2: > - Introduced patch 18/20 > - Fixed cosmetic changes: spaces before brace, live over 80 > characters > - Removed some of the check for NULL pointers before calling > dma_pool_destroy > - Improved the regexp in checkpatch for pci_pool, thanks to Joe > Perches > - Added Tested-by and Acked-by tags > > Romain Perier (15): > block: DAC960: Replace PCI pool old API > dmaengine: pch_dma: Replace PCI pool old API > IB/mthca: Replace PCI pool old API > net: e100: Replace PCI pool old API > mlx4: Replace PCI pool old API > mlx5: Replace PCI pool old API > wireless: ipw2200: Replace PCI pool old API > scsi: be2iscsi: Replace PCI pool old API > scsi: csiostor: Replace PCI pool old API > scsi: lpfc: Replace PCI pool old API > scsi: megaraid: Replace PCI pool old API > scsi: mpt3sas: Replace PCI pool old API > scsi: mvsas: Replace PCI pool old API > scsi: pmcraid: Replace PCI pool old API > PCI: Remove PCI pool macro functions > > drivers/block/DAC960.c | 38 ++++++----- > drivers/block/DAC960.h | 4 +- > drivers/dma/pch_dma.c | 12 ++-- > drivers/infiniband/hw/mthca/mthca_av.c | 10 +-- > drivers/infiniband/hw/mthca/mthca_cmd.c | 8 +-- > drivers/infiniband/hw/mthca/mthca_dev.h | 4 +- > drivers/net/ethernet/intel/e100.c | 12 ++-- > drivers/net/ethernet/mellanox/mlx4/cmd.c | 10 +-- > drivers/net/ethernet/mellanox/mlx4/mlx4.h | 2 +- > drivers/net/ethernet/mellanox/mlx5/core/cmd.c | 11 ++-- > drivers/net/wireless/intel/ipw2x00/ipw2200.c | 13 ++-- > drivers/scsi/be2iscsi/be_iscsi.c | 6 +- > drivers/scsi/be2iscsi/be_main.c | 6 +- > drivers/scsi/be2iscsi/be_main.h | 2 +- > drivers/scsi/csiostor/csio_hw.h | 2 +- > drivers/scsi/csiostor/csio_init.c | 11 ++-- > drivers/scsi/csiostor/csio_scsi.c | 6 +- > drivers/scsi/lpfc/lpfc.h | 16 ++--- > drivers/scsi/lpfc/lpfc_init.c | 16 ++--- > drivers/scsi/lpfc/lpfc_mem.c | 90 +++++++++++++-- > ------------ > drivers/scsi/lpfc/lpfc_nvme.c | 6 +- > drivers/scsi/lpfc/lpfc_nvmet.c | 4 +- > drivers/scsi/lpfc/lpfc_scsi.c | 12 ++-- > drivers/scsi/lpfc/lpfc_sli.c | 6 +- > drivers/scsi/megaraid/megaraid_mbox.c | 30 ++++----- > drivers/scsi/megaraid/megaraid_mm.c | 29 ++++----- > drivers/scsi/megaraid/megaraid_sas_base.c | 27 ++++---- > drivers/scsi/megaraid/megaraid_sas_fusion.c | 46 +++++++------- > drivers/scsi/mpt3sas/mpt3sas_base.c | 73 ++++++++++------- > ----- > drivers/scsi/mvsas/mv_init.c | 6 +- > drivers/scsi/mvsas/mv_sas.c | 6 +- > drivers/scsi/pmcraid.c | 10 +-- > drivers/scsi/pmcraid.h | 2 +- > include/linux/mlx5/driver.h | 2 +- > include/linux/pci.h | 9 --- > 35 files changed, 269 insertions(+), 278 deletions(-) > -- Doug Ledford GPG KeyID: B826A3330E572FDD Key fingerprint = AE6B 1BDA 122B 23B4 265B 1274 B826 A333 0E57 2FDD