From patchwork Mon Jun 15 10:46:18 2009 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jeff Layton X-Patchwork-Id: 30295 Received: from lists.samba.org (mail.samba.org [66.70.73.150]) by demeter.kernel.org (8.14.2/8.14.2) with ESMTP id n5FAlfpP000705 for ; Mon, 15 Jun 2009 10:47:41 GMT Received: from dp.samba.org (localhost [127.0.0.1]) by lists.samba.org (Postfix) with ESMTP id 684B2163C2D for ; Mon, 15 Jun 2009 10:47:07 +0000 (GMT) X-Spam-Checker-Version: SpamAssassin 3.1.7 (2006-10-05) on dp.samba.org X-Spam-Level: X-Spam-Status: No, score=-3.8 required=3.8 tests=AWL,BAYES_00, FORGED_RCVD_HELO,SPF_HELO_PASS,SPF_PASS autolearn=ham version=3.1.7 X-Original-To: linux-cifs-client@lists.samba.org Delivered-To: linux-cifs-client@lists.samba.org Received: from mx2.redhat.com (mx2.redhat.com [66.187.237.31]) by lists.samba.org (Postfix) with ESMTP id 56BF1163BA2 for ; Mon, 15 Jun 2009 10:46:00 +0000 (GMT) Received: from int-mx2.corp.redhat.com (int-mx2.corp.redhat.com [172.16.27.26]) by mx2.redhat.com (8.13.8/8.13.8) with ESMTP id n5FAkNdM018250; Mon, 15 Jun 2009 06:46:23 -0400 Received: from ns3.rdu.redhat.com (ns3.rdu.redhat.com [10.11.255.199]) by int-mx2.corp.redhat.com (8.13.1/8.13.1) with ESMTP id n5FAkMYh002499; Mon, 15 Jun 2009 06:46:22 -0400 Received: from tlielax.poochiereds.net (vpn-12-236.rdu.redhat.com [10.11.12.236]) by ns3.rdu.redhat.com (8.13.8/8.13.8) with ESMTP id n5FAkJST030914; Mon, 15 Jun 2009 06:46:20 -0400 Date: Mon, 15 Jun 2009 06:46:18 -0400 From: Jeff Layton To: Stephen Rothwell Message-ID: <20090615064618.120a2337@tlielax.poochiereds.net> In-Reply-To: <20090615133222.7405743c.sfr@canb.auug.org.au> References: <20090615133222.7405743c.sfr@canb.auug.org.au> Mime-Version: 1.0 X-Scanned-By: MIMEDefang 2.58 on 172.16.27.26 Cc: Steve French , linux-next@vger.kernel.org, linux-cifs-client@lists.samba.org, linux-kernel@vger.kernel.org Subject: [linux-cifs-client] Re: linux-next: cifs tree build failure X-BeenThere: linux-cifs-client@lists.samba.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: The Linux CIFS VFS client List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: linux-cifs-client-bounces+patchwork-cifs-client=patchwork.kernel.org@lists.samba.org Errors-To: linux-cifs-client-bounces+patchwork-cifs-client=patchwork.kernel.org@lists.samba.org On Mon, 15 Jun 2009 13:32:22 +1000 Stephen Rothwell wrote: > Hi Steve, > > Today's linux-next build (powerpc ppc64_defconfig) failed like this: > > fs/cifs/cifsfs.c: In function 'cifs_show_options': > fs/cifs/cifsfs.c:357: error: implicit declaration of function 'cifs_show_address' > > Caused by commit 8616e0fc1e27295316f9821a883f0e9fa6f8200f ("cifs: remove > unneeded NULL checks from cifs_show_options"). > > I have used the cifs tree from next-20090612 for today. My fault. That function gets introduced in a later patch and I had a bogus call left in here from when I reorganized the series. This patch should fix it and a typo that got silently fixed in a patch that hasn't been committed yet. This patch should fix both. Steve F., let me know if you want me to respin the series on top of this patch. Note that there's still a harmless compiler warning that gets fixed in a later patch in the series. Thanks, From c5949e669f4a85c16b3529777f5d550bddea9dd9 Mon Sep 17 00:00:00 2001 From: Jeff Layton Date: Mon, 15 Jun 2009 06:40:23 -0400 Subject: [PATCH] cifs: fix problems with earlier patches cifs_show_address hasn't been introduced yet, and fix a typo that was silently fixed by a later patch in the series. Signed-off-by: Jeff Layton --- fs/cifs/cifsfs.c | 2 -- fs/cifs/netmisc.c | 2 +- 2 files changed, 1 insertions(+), 3 deletions(-) diff --git a/fs/cifs/cifsfs.c b/fs/cifs/cifsfs.c index 8b31570..4b8ec25 100644 --- a/fs/cifs/cifsfs.c +++ b/fs/cifs/cifsfs.c @@ -354,8 +354,6 @@ cifs_show_options(struct seq_file *s, struct vfsmount *m) if (tcon->ses->domainName) seq_printf(s, ",domain=%s", tcon->ses->domainName); - cifs_show_address(s, tcon->ses->server); - seq_printf(s, ",uid=%d", cifs_sb->mnt_uid); if (cifs_sb->mnt_cifs_flags & CIFS_MOUNT_OVERR_UID) seq_printf(s, ",forceuid"); diff --git a/fs/cifs/netmisc.c b/fs/cifs/netmisc.c index 00e6e35..f9a54da 100644 --- a/fs/cifs/netmisc.c +++ b/fs/cifs/netmisc.c @@ -166,7 +166,7 @@ int cifs_convert_address(char *src, void *dst) { struct sockaddr_in *s4 = (struct sockaddr_in *) dst; - struct sockaddr_in6 *s6 = (Struct sockaddr_in6 *) dst; + struct sockaddr_in6 *s6 = (struct sockaddr_in6 *) dst; if (cifs_inet_pton(AF_INET, src, &s4->sin_addr.s_addr)) { s4->sin_family = AF_INET; -- 1.6.0.6