Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759176Ab0FPQAZ (ORCPT ); Wed, 16 Jun 2010 12:00:25 -0400 Received: from mail-pw0-f46.google.com ([209.85.160.46]:57912 "EHLO mail-pw0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1759080Ab0FPQAX (ORCPT ); Wed, 16 Jun 2010 12:00:23 -0400 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=message-id:date:from:user-agent:mime-version:to:cc:subject :references:in-reply-to:content-type:content-transfer-encoding; b=iBNHwgZKyUtKdG1ApEOkV4QdOu6znlTs98BOt1uPUV8muzIwqGNbS6oC+ayhUR+LJ7 rXbfp5UYdiZui6jjWCBKWovcC+xBfFpkBf4R4sSIFBN4iqVFTBH9WRFGS9tV1+J4wGN6 C3jVU2f2CBP1y59+4/IaqyJhTNfjJmPei+3P0= Message-ID: <4C18F524.4000902@gmail.com> Date: Wed, 16 Jun 2010 09:00:36 -0700 From: "Justin P. Mattock" User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.4pre) Gecko/20091114 Lightning/1.0pre Thunderbird/3.0b4 MIME-Version: 1.0 To: James Bottomley CC: linux-kernel@vger.kernel.org, linux-wireless@vger.kernel.org, linux-pci@vger.kernel.org, linux-scsi@vger.kernel.org Subject: Re: [PATCH 5/5]scsi:hosts.c Fix warning: variable 'rval' set but not used References: <1276666434-11227-1-git-send-email-justinmattock@gmail.com> <1276666434-11227-6-git-send-email-justinmattock@gmail.com> <1276702443.2847.153.camel@mulgrave.site> In-Reply-To: <1276702443.2847.153.camel@mulgrave.site> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3093 Lines: 94 On 06/16/2010 08:34 AM, James Bottomley wrote: > On Tue, 2010-06-15 at 22:33 -0700, Justin P. Mattock wrote: >> The below patch fixes a warning message generated by gcc 4.6.0 >> CC drivers/scsi/hosts.o >> drivers/scsi/hosts.c: In function 'scsi_host_alloc': >> drivers/scsi/hosts.c:328:6: warning: variable 'rval' set but not used >> >> Signed-off-by: Justin P. Mattock >> >> --- >> drivers/scsi/hosts.c | 2 -- >> 1 files changed, 0 insertions(+), 2 deletions(-) >> >> diff --git a/drivers/scsi/hosts.c b/drivers/scsi/hosts.c >> index 6660fa9..00fd6a4 100644 >> --- a/drivers/scsi/hosts.c >> +++ b/drivers/scsi/hosts.c >> @@ -325,7 +325,6 @@ struct Scsi_Host *scsi_host_alloc(struct scsi_host_template *sht, int privsize) >> { >> struct Scsi_Host *shost; >> gfp_t gfp_mask = GFP_KERNEL; >> - int rval; >> >> if (sht->unchecked_isa_dma&& privsize) >> gfp_mask |= __GFP_DMA; >> @@ -420,7 +419,6 @@ struct Scsi_Host *scsi_host_alloc(struct scsi_host_template *sht, int privsize) >> shost->ehandler = kthread_run(scsi_error_handler, shost, >> "scsi_eh_%d", shost->host_no); >> if (IS_ERR(shost->ehandler)) { >> - rval = PTR_ERR(shost->ehandler); >> goto fail_kfree; >> } > > For future reference, this is less stylistically acceptable C: you've > reduced the if clause to a single statement, so the braces need > removing. > > However, I don't think we should be ignoring the fact that the eh thread > failed to spawn, so I think some type of printed warning (giving the > error code) would be a much more appropriate replacement. > > James > > > o.k. I'll give a try at that.. as a test I did this(below) seemed to compile clean, but not sure if this is what you're asking for though: From 8a4d6e793e0f92d180a6f48c53bbf00d2751ad01 Mon Sep 17 00:00:00 2001 From: Justin P. Mattock Date: Wed, 16 Jun 2010 08:58:13 -0700 Subject: [PATCH] test Signed-off-by: Justin P. Mattock --- drivers/scsi/hosts.c | 3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/drivers/scsi/hosts.c b/drivers/scsi/hosts.c index 6660fa9..8d98a46 100644 --- a/drivers/scsi/hosts.c +++ b/drivers/scsi/hosts.c @@ -325,7 +325,6 @@ struct Scsi_Host *scsi_host_alloc(struct scsi_host_template *sht, int privsize) { struct Scsi_Host *shost; gfp_t gfp_mask = GFP_KERNEL; - int rval; if (sht->unchecked_isa_dma && privsize) gfp_mask |= __GFP_DMA; @@ -420,7 +419,7 @@ struct Scsi_Host *scsi_host_alloc(struct scsi_host_template *sht, int privsize) shost->ehandler = kthread_run(scsi_error_handler, shost, "scsi_eh_%d", shost->host_no); if (IS_ERR(shost->ehandler)) { - rval = PTR_ERR(shost->ehandler); + printk(KERN_WARNING "test.....\n"); goto fail_kfree; } -- 1.7.1.rc1.21.gf3bd6 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/