Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757342Ab0FUBDw (ORCPT ); Sun, 20 Jun 2010 21:03:52 -0400 Received: from mail-px0-f174.google.com ([209.85.212.174]:44155 "EHLO mail-px0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755143Ab0FUBDu (ORCPT ); Sun, 20 Jun 2010 21:03:50 -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=CtzG7wEBPfWf3HN3wM2GTFUIw+BCqAzR7BWObgLc2K6FZnNFZ0P8Zze0IYff/dgRMA n7SSnQjb26SOX6Sh2zD5m8gc/VU4uIIQZhk7VGNMLpIxrqYMxxWq4u8IQjgUYBDDr02j zz1JLyxAo1rW4/jJuyii1OVWjF21OHqs3NqmY= Message-ID: <4C1EBA7D.4060508@gmail.com> Date: Sun, 20 Jun 2010 18:03:57 -0700 From: "Justin P. Mattock" User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.9) Gecko/20100615 Lightning/1.0b2pre Thunderbird/3.0.4 MIME-Version: 1.0 To: Jeff Layton CC: linux-cifs-client@lists.samba.org, linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 3/6 v2]cifs Fix warnings variables set but not used References: <1276980427-24728-1-git-send-email-justinmattock@gmail.com> <20100620171713.782e9104@tlielax.poochiereds.net> <4C1E9752.8040508@gmail.com> <20100620202811.2d655c95@corrin.poochiereds.net> In-Reply-To: <20100620202811.2d655c95@corrin.poochiereds.net> 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 Content-Length: 1675 Lines: 49 On 06/20/2010 05:28 PM, Jeff Layton wrote: > On Sun, 20 Jun 2010 15:33:54 -0700 > "Justin P. Mattock" wrote: > >> >>>> /* since paths are not looked up by component - the parent >>>> directories are presumed to be good here */ >>>> renew_parental_timestamps(direntry); >>>> diff --git a/fs/cifs/file.c b/fs/cifs/file.c >>>> index 75541af..73e540b 100644 >>>> --- a/fs/cifs/file.c >>>> +++ b/fs/cifs/file.c >>>> @@ -1312,7 +1312,7 @@ static int cifs_partialpagewrite(struct page *page, unsigned from, unsigned to) >>>> int rc = -EFAULT; >>>> int bytes_written = 0; >>>> struct cifs_sb_info *cifs_sb; >>>> - struct cifsTconInfo *pTcon; >>>> + struct cifsTconInfo *pTcon __attribute__((unused)); >>> >>> This variable can just be removed (along with the spot where it gets >>> set later). >>> >>>> struct inode *inode; >>>> struct cifsFileInfo *open_file; >>>> >>> >>> >> >> one thing I want to make sure of before sending this out, is with file.c >> if I remove *pTcon another warning shows up: >> >> CC [M] fs/cifs/file.o >> fs/cifs/file.c: In function 'cifs_partialpagewrite': >> fs/cifs/file.c:1314:23: warning: variable 'cifs_sb' set but not used >> >> with the original patch I had removed this as well, should I do this or >> is this not a good idea? > > If it's not used, remove it... > alright, I'll send those two, then go from there. Justin P. Mttock -- 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/