From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756833AbaHHROJ (ORCPT ); Fri, 8 Aug 2014 13:14:09 -0400 Received: from bedivere.hansenpartnership.com ([66.63.167.143]:52265 "EHLO bedivere.hansenpartnership.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753049AbaHHROH (ORCPT ); Fri, 8 Aug 2014 13:14:07 -0400 Message-ID: <1407518045.1686.0.camel@HansenPartnership.com> Subject: Re: [PATCH 1/1] dpt_i2o: delete unnecessary null test on array From: James Bottomley To: Julia Lawall Cc: Adaptec OEM Raid Solutions , kernel-janitors@vger.kernel.org, linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org, joe@perches.com Date: Fri, 08 Aug 2014 10:14:05 -0700 In-Reply-To: References: <1407321551-27479-1-git-send-email-Julia.Lawall@lip6.fr> <1407321551-27479-2-git-send-email-Julia.Lawall@lip6.fr> <1407517179.2044.13.camel@jarvis.lan> Content-Type: text/plain; charset="ISO-8859-15" X-Mailer: Evolution 3.12.4 Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, 2014-08-08 at 19:03 +0200, Julia Lawall wrote: > > > diff --git a/drivers/scsi/dpt_i2o.c b/drivers/scsi/dpt_i2o.c > > > index 67283ef..62e276b 100644 > > > --- a/drivers/scsi/dpt_i2o.c > > > +++ b/drivers/scsi/dpt_i2o.c > > > @@ -1169,11 +1169,6 @@ static struct adpt_device* adpt_find_device(adpt_hba* pHba, u32 chan, u32 id, u6 > > > if(chan < 0 || chan >= MAX_CHANNEL) > > > return NULL; > > > > > > - if( pHba->channel[chan].device == NULL){ > > > - printk(KERN_DEBUG"Adaptec I2O RAID: Trying to find device before they are allocated\n"); > > > - return NULL; > > > - } > > > > dpt_i2o is always weirdly fun, but I think, based on the message, this > > check is supposed to be > > > > if( pHba->channel[chan].device[id] == NULL){ > > > > Since device is an array of device pointers which are allocated by > > parsing data. > > That seems to be already checked immediately below: > > d = pHba->channel[chan].device[id]; > if(!d || d->tid == 0) { > return NULL; Yes, I know, but no message is emitted. The message seems to be for a violation of the state machine which device[id] = NULL implies. James