From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Steve French" Subject: Re: [PATCH 0/7] cifs: clean up socket creation, connection and sending (try #2) Date: Mon, 1 Dec 2008 19:43:14 -0600 Message-ID: <524f69650812011743r6d9a71f7oa89beeef9d13af45@mail.gmail.com> References: <1228070436-6063-1-git-send-email-jlayton@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Cc: linux-cifs-client@lists.samba.org, linux-fsdevel To: "Jeff Layton" Return-path: Received: from nf-out-0910.google.com ([64.233.182.190]:24404 "EHLO nf-out-0910.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751647AbYLBBnQ (ORCPT ); Mon, 1 Dec 2008 20:43:16 -0500 Received: by nf-out-0910.google.com with SMTP id d3so1452047nfc.21 for ; Mon, 01 Dec 2008 17:43:14 -0800 (PST) In-Reply-To: <1228070436-6063-1-git-send-email-jlayton@redhat.com> Content-Disposition: inline Sender: linux-fsdevel-owner@vger.kernel.org List-ID: After review of Jeff's 2 recent patch series, and discussion with Jeff on IRC (as a result Jeff respun and made minor modifications to a couple), I have just merged 10 of his 14 patches into cifs-2.6.git. See below: 3 min ago Jeff Layton cifs: make ipv6_connect take a TCP_Server_Info 3 min ago Jeff Layton cifs: make ipv4_connect take a TCP_Server_Info 14 min ago Jeff Layton cifs: don't declare smb_vol info on the stack 15 min ago Jeff Layton cifs: move allocation of new TCP_Server_Info into separ ... 7 min ago Jeff Layton cifs: account for IPv6 in ses->serverName and clean ... 21 min ago Jeff Layton cifs: make dnotify thread experimental code 5 hours ago Jeff Layton cifs: convert tcpSem to a mutex 5 hours ago Jeff Layton cifs: take module reference when starting cifsd 6 hours ago Jeff Layton cifs: display addr and prefixpath options in /proc ... 6 hours ago Jeff Layton cifs: remove unused SMB session pointer from struct ... On Sun, Nov 30, 2008 at 12:40 PM, Jeff Layton wrote: > This patchset is intended as a cleanup of the code that creates and > connects sockets. It's also intended to unify the smb_send and smb_send2 > functions, and simplify the sending code in the blocking I/O case. There > is a lot of duplicate code in those functions, and this should make it > easier to ensure that we fix bugs with it properly. This should also fix > the case where we get a partial send in smb_send. Previously, we did not > force a reconnect in that case. > > This is the second post of this patchset. The main differences from the > original set are: > > 1) fix some symbol naming style issues that HCH pointed out > > 2) move the address handling into cifs_get_tcp_server and out of > cifs_mount altogether (also pointed out by HCH) > > 3) the addition of a patch to reduce stack consumption in cifs_mount by > dynamically allocating the volume_info > > This patchset is also intended for 2.6.29 and depends on the 5 patch set > of cleanups that I sent last week. > > I've only given this set cursory testing. If it looks OK, I'll plan to > do some more with it. The last set was more heavily tested, and showed > no functional or performance regressions as best I could tell. Since the > differences between that set and this one are fairly superficial, I > don't expect much difference in performance here. > > Jeff Layton (7): > cifs: convert tcpSem to a mutex > cifs: move allocation of new TCP_Server_Info into separate function > cifs: don't declare smb_vol info on the stack > cifs: make ipv4_connect take a TCP_Server_Info arg > cifs: make ipv6_connect take a TCP_Server_Info arg > cifs: turn smb_send into a wrapper around smb_sendv > cifs: don't retry on blocking sends > > fs/cifs/cifsglob.h | 2 +- > fs/cifs/cifsproto.h | 4 +- > fs/cifs/connect.c | 609 ++++++++++++++++++++++++++------------------------- > fs/cifs/transport.c | 147 ++++--------- > 4 files changed, 359 insertions(+), 403 deletions(-) > > -- Thanks, Steve