From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932858Ab3BKWrB (ORCPT ); Mon, 11 Feb 2013 17:47:01 -0500 Received: from cmexedge2.ext.emulex.com ([138.239.224.100]:21403 "EHLO CMEXEDGE2.ext.emulex.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932776Ab3BKWq7 (ORCPT ); Mon, 11 Feb 2013 17:46:59 -0500 Message-ID: <511974E0.4040703@emulex.com> Date: Mon, 11 Feb 2013 17:46:56 -0500 From: James Smart Reply-To: User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:16.0) Gecko/20121026 Thunderbird/16.0.2 MIME-Version: 1.0 To: Tejun Heo CC: , , , , , , , , Subject: Re: [PATCH 47/62] scsi/lpfc: convert to idr_alloc() References: <1359854463-2538-1-git-send-email-tj@kernel.org> <1359854463-2538-48-git-send-email-tj@kernel.org> In-Reply-To: <1359854463-2538-48-git-send-email-tj@kernel.org> Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Acked-by: James Smart -- james s On 2/2/2013 8:20 PM, Tejun Heo wrote: > Convert to the much saner new idr interface. > > Only compile tested. > > Signed-off-by: Tejun Heo > Cc: James Smart > Cc: linux-scsi@vger.kernel.org > --- > This patch depends on an earlier idr changes and I think it would be > best to route these together through -mm. Please holler if there's > any objection. Thanks. > > drivers/scsi/lpfc/lpfc_init.c | 12 ++++-------- > 1 file changed, 4 insertions(+), 8 deletions(-) > > diff --git a/drivers/scsi/lpfc/lpfc_init.c b/drivers/scsi/lpfc/lpfc_init.c > index 89ad558..7de4ef14 100644 > --- a/drivers/scsi/lpfc/lpfc_init.c > +++ b/drivers/scsi/lpfc/lpfc_init.c > @@ -3165,14 +3165,10 @@ destroy_port(struct lpfc_vport *vport) > int > lpfc_get_instance(void) > { > - int instance = 0; > - > - /* Assign an unused number */ > - if (!idr_pre_get(&lpfc_hba_index, GFP_KERNEL)) > - return -1; > - if (idr_get_new(&lpfc_hba_index, NULL, &instance)) > - return -1; > - return instance; > + int ret; > + > + ret = idr_alloc(&lpfc_hba_index, NULL, 0, 0, GFP_KERNEL); > + return ret < 0 ? -1 : ret; > } > > /**