From mboxrd@z Thu Jan 1 00:00:00 1970 From: Hannes Reinecke Subject: Re: [PATCHv2 1/1] SCSI: update hosts module to use idr index management Date: Wed, 7 Oct 2015 22:23:37 +0200 Message-ID: <56157F49.5030101@suse.com> References: <1a69690d615610c1dee725db7e6d81cf2db28266.1444085255.git.lduncan@suse.com> <20151006094040.GA24624@infradead.org> <56140188.6070107@suse.com> <1444151854.2245.18.camel@HansenPartnership.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1444151854.2245.18.camel@HansenPartnership.com> Sender: linux-kernel-owner@vger.kernel.org To: James Bottomley , Lee Duncan Cc: Christoph Hellwig , linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org, Tejun Heo , Johannes Thumshirn List-Id: linux-scsi@vger.kernel.org On 10/06/2015 07:17 PM, James Bottomley wrote: > On Tue, 2015-10-06 at 10:14 -0700, Lee Duncan wrote: >> On 10/06/2015 02:40 AM, Christoph Hellwig wrote: >>>> struct Scsi_Host *scsi_host_lookup(unsigned short hostnum) >>>> { >>>> - struct device *cdev; >>>> - struct Scsi_Host *shost = NULL; >>>> - >>>> - cdev = class_find_device(&shost_class, NULL, &hostnum, >>>> - __scsi_host_match); >>>> - if (cdev) { >>>> - shost = scsi_host_get(class_to_shost(cdev)); >>>> - put_device(cdev); >>>> - } >>>> + struct Scsi_Host *shost; >>>> + >>>> + spin_lock(&host_index_lock); >>>> + shost = idr_find(&host_index_idr, hostnum); >>>> + spin_unlock(&host_index_lock); >>>> + >>>> return shost; >>> >>> How does this actually grab a reference to the host? >> >> Good catch -- I should have noticed that. >> >> I will resubmit the patch. > > I'll wait to see what you produce, but I don't think, using a separate > idr array, you can close the race window between lookup and get. One of > the nice things about using the cdev iterator is that the get is part of > the lookup process. > Hmm. Should be possible if you free up the IDR in scsi_remove_host(), just after setting the state to SHOST_DEL. Cheers, Hannes