- 25 Dec, 2008 10 commits
-
-
Volker Lendecke authored
Streamline SendReceiveBlockingLock: Use "goto out:" in an error condition Signed-off-by:
Volker Lendecke <vl@samba.org> Signed-off-by:
Steve French <sfrench@us.ibm.com>
-
Steve French authored
Signed-off-by:
Volker Lendecke <vl@samba.org> Signed-off-by:
Steve French <sfrench@us.ibm.com>
-
Volker Lendecke authored
Slightly streamline SendReceive[2] Remove an else branch by naming the error condition what it is Signed-off-by:
Volker Lendecke <vl@samba.org> Signed-off-by:
Steve French <sfrench@us.ibm.com>
-
Volker Lendecke authored
Check the return value of cifs_sign_smb[2] Signed-off-by:
Volker Lendecke <vl@samba.org> Signed-off-by:
Steve French <sfrench@us.ibm.com>
-
Volker Lendecke authored
This is no functional change, because in the "if" branch we do an early "return 0;". Signed-off-by:
Volker Lendecke <vl@samba.org> Signed-off-by:
Steve French <sfrench@us.ibm.com>
-
Volker Lendecke authored
Simplify allocate_mid() slightly: Remove some unnecessary "else" branches Signed-off-by:
Volker Lendecke <vl@samba.org> Acked-by:
Jeff Layton <jlayton@redhat.com> Signed-off-by:
Steve French <sfrench@us.ibm.com>
-
Volker Lendecke authored
inbuf->smb_buf_length does not change in in wait_for_free_request() or in allocate_mid(), so we can check it early. Signed-off-by:
Volker Lendecke <vl@samba.org> Acked-by:
Jeff Layton <jlayton@redhat.com> Signed-off-by:
Steve French <sfrench@us.ibm.com>
-
Jeff Layton authored
cifs: fix wait_for_response to time out sleeping processes correctly The current scheme that CIFS uses to sleep and wait for a response is not quite what we want. After sending a request, wait_for_response puts the task to sleep with wait_event(). One of the conditions for wait_event is a timeout (using time_after()). The problem with this is that there is no guarantee that the process will ever be woken back up. If the server stops sending data, then cifs_demultiplex_thread will leave its response queue sleeping. I think the only thing that saves us here is the fact that cifs_dnotify_thread periodically (every 15s) wakes up sleeping processes on all response_q's that have calls in flight. This makes for unnecessary wakeups of some processes. It also means large variability in the timeouts since they're all woken up at once. Instead of this, put the tasks to sleep with wait_event_timeout. This makes them wake up on their own if they time out. With this change, cifs_dnotify_thread should no longer be needed. I've been testing this in conjunction with some other patches that I'm working on. It doesn't seem to affect performance at all with with heavy I/O. Identical iozone -ac runs complete in almost exactly the same time (<1% difference in times). Thanks to Wasrshi Nimara for initially pointing this out. Wasrshi, it would be nice to know whether this patch also helps your testcase. Signed-off-by:
Jeff Layton <jlayton@redhat.com> Cc: Wasrshi Nimara <warshinimara@gmail.com> Signed-off-by:
Steve French <sfrench@us.ibm.com>
-
Jeff Layton authored
Mutexes are preferred for single-holder semaphores... Signed-off-by:
Jeff Layton <jlayton@redhat.com> Signed-off-by:
Steve French <sfrench@us.ibm.com>
-
Jeff Layton authored
Signed-off-by:
Jeff Layton <jlayton@redhat.com> Signed-off-by:
Steve French <sfrench@us.ibm.com>
-
- 30 Oct, 2008 1 commit
-
-
Steve French authored
smb_send2 exit logic was strange, and with the previous change could cause us to fail large smb writes when all of the smb was not sent as one chunk. Acked-by:
Jeff Layton <jlayton@redhat.com> Signed-off-by:
Steve French <sfrench@us.ibm.com>
-
- 28 Oct, 2008 1 commit
-
-
Steve French authored
CIFS in some heavy stress conditions cifs could get EAGAIN repeatedly in smb_send2 which led to repeated retries and eventually failure of large writes which could lead to data corruption. There are three changes that were suggested by various network developers: 1) convert cifs from non-blocking to blocking tcp sendmsg (we left in the retry on failure) 2) change cifs to not set sendbuf and rcvbuf size for the socket (let tcp autotune the buffer sizes since that works much better in the TCP stack now) 3) if we have a partial frame sent in smb_send2, mark the tcp session as invalid (close the socket and reconnect) so we do not corrupt the remaining part of the SMB with the beginning of the next SMB. This does not appear to hurt performance measurably and has been run in various scenarios, but it definately removes a corruption that we were seeing in some high stress test cases. Acked-by:
Shirish Pargaonkar <shirishp@us.ibm.com> Signed-off-by:
Steve French <sfrench@us.ibm.com>
-
- 22 Sep, 2008 1 commit
-
-
Pekka Enberg authored
GFP_KERNEL and GFP_NOFS are mutually exclusive. If you combine them, you end up with plain GFP_KERNEL which can deadlock in cases where you really want GFP_NOFS. Cc: Steve French <sfrench@samba.org> Signed-off-by:
Pekka Enberg <penberg@cs.helsinki.fi> Signed-off-by:
Steve French <sfrench@us.ibm.com>
-
- 29 Jul, 2008 1 commit
-
-
Shirish Pargaonkar authored
There are cases in which, on a full socket which requires retry on sending data by the app (cifs in this case), that we were not retrying since we did not reinitialize a counter. This fixes the retry logic to retry up to 15 seconds on stuck sockets. Signed-off-by:
Shirish Pargaonkar <shirishp@us.ibm.com> Signed-off-by:
Steve French <sfrench@us.ibm.com>
-
- 15 Apr, 2008 1 commit
-
-
Steve French authored
If a tcon is being freed in call tconInfoFree, clean up any entries that may exist in global oplock queue as the tcon structure hanging off of those entries will be invalid and can cause oops while accesing any elements in the tcon structure. Signed-off-by:
Shirish Pargaonkar <shirishp@us.ibm.com> Signed-off-by:
Steve French <sfrench@us.ibm.com>
-
- 12 Feb, 2008 1 commit
-
-
Steve French authored
Christoph had noticed too many ifdefs in the CIFS code making it hard to read. This patch removes about a quarter of them from the C files in cifs by improving a few key ifdefs in the .h files. Signed-off-by:
Steve French <sfrench@us.ibm.com>
-
- 07 Feb, 2008 1 commit
-
-
Steve French authored
Signed-off-by:
Steve French <sfrench@us.ibm.com>
-
- 13 Nov, 2007 1 commit
-
-
Steve French authored
request In SendReceive() function in transport.c - it memcpy's message payload into a buffer passed via out_buf param. The function assumes that all buffers are of size (CIFSMaxBufSize + MAX_CIFS_HDR_SIZE) , unfortunately it is also called with smaller (MAX_CIFS_SMALL_BUFFER_SIZE) buffers. There are eight callers (SMB worker functions) which are primarily affected by this change: TreeDisconnect, uLogoff, Close, findClose, SetFileSize, SetFileTimes, Lock and PosixLock CC: Dave Kleikamp <shaggy@austin.ibm.com> CC: Przemyslaw Wegrzyn <czajnik@czajsoft.pl> Acked-by:
Jeff Layton <jlayton@redhat.com> Signed-off-by:
Steve French <sfrench@us.ibm.com>
-
- 18 Oct, 2007 1 commit
-
-
Steve French authored
Also returns more accurate errors to mount for the cases of account expired and password expired Acked-by:
Jeff Layton <jlayton@redhat.com> Signed-off-by:
Steve French <sfrench@us.ibm.com>
-
- 30 Aug, 2007 1 commit
-
-
Steve French authored
Signed-off-by:
Steve French <sfrench@us.ibm.com>
-
- 12 Jul, 2007 1 commit
-
-
Steve French authored
This should be the last big batch of whitespace/formatting fixes. checkpatch warnings for the cifs directory are down about 90% and many of the remaining ones are harder to remove or make the code harder to read. Signed-off-by:
Steve French <sfrench@us.ibm.com>
-
- 09 Jul, 2007 1 commit
-
-
Steve French authored
Signed-off-by:
Yehuda Sadeh Weinraub <Yehuda.Sadeh@expand.com> Signed-off-by:
Steve French <sfrench@us.ibm.com>
-
- 06 Jul, 2007 1 commit
-
-
Steve French authored
checkpatch.pl redux Signed-off-by:
Steve French <sfrench@us.ibm.com>
-
- 05 Mar, 2007 1 commit
-
-
Steve French authored
Noticed by Shaggy. Signed-off-by:
Shaggy <shaggy@us.ibm.com> Signed-off-by:
Steve French <sfrench@us.ibm.com>
-
- 07 Dec, 2006 3 commits
-
-
Christoph Lameter authored
Replace all uses of kmem_cache_t with struct kmem_cache. The patch was generated using the following script: #!/bin/sh # # Replace one string by another in all the kernel sources. # set -e for file in `find * -name "*.c" -o -name "*.h"|xargs grep -l $1`; do quilt add $file sed -e "1,\$s/$1/$2/g" $file >/tmp/$$ mv /tmp/$$ $file quilt refresh done The script was run like this sh replace kmem_cache_t "struct kmem_cache" Signed-off-by:
Christoph Lameter <clameter@sgi.com> Signed-off-by:
Andrew Morton <akpm@osdl.org> Signed-off-by:
Linus Torvalds <torvalds@osdl.org>
-
Christoph Lameter authored
SLAB_KERNEL is an alias of GFP_KERNEL. Signed-off-by:
Christoph Lameter <clameter@sgi.com> Signed-off-by:
Andrew Morton <akpm@osdl.org> Signed-off-by:
Linus Torvalds <torvalds@osdl.org>
-
Christoph Lameter authored
SLAB_NOFS is an alias of GFP_NOFS. Signed-off-by:
Christoph Lameter <clameter@sgi.com> Signed-off-by:
Andrew Morton <akpm@osdl.org> Signed-off-by:
Linus Torvalds <torvalds@osdl.org>
-
- 11 Aug, 2006 4 commits
-
-
Jeremy Allison authored
Allow Windows blocking locks to be cancelled via a CANCEL_LOCK call. TODO - restrict this to servers that support NT_STATUS codes (Win9x will probably not support this call). Signed-off-by:
Jeremy Allison <jra@samba.org> Signed-off-by:
Steve French <sfrench@us.ibm.com> (cherry picked from 570d4d2d895569825d0d017d4e76b51138f68864 commit)
-
Steve French authored
Although harmless, we were sometimes treating midState like it contained flags but they are exclusive states, and this makes that more clear. Signed-off-by:
Jeremy Allison <jra@samba.org> Signed-off-by:
Steve French <sfrench@us.ibm.com> (cherry picked from 586c057c3a68dd6ae0f3ba94fbf76798b1558074 commit)
-
Steve French authored
Signed-off-by:
Jeremy Allison <jra@samba.org> Signed-off-by:
Steve French <sfrench@us.ibm.com> (cherry picked from b33a3f55e54fd210fc043eafcf83728b03bc9e02 commit)
-
Steve French authored
request and do not time out slow requests to a server that is still responding well to other threads Suggested by jra of Samba team Signed-off-by:
Steve French <sfrench@us.ibm.com> (cherry picked from 89b57148115479eef074b8d3f86c4c86c96ac969 commit)
-
- 31 May, 2006 1 commit
-
-
Steve French authored
-
- 26 Feb, 2006 1 commit
-
-
Steve French authored
small req buffer pool less often. Signed-off-by:
Steve French <sfrench@us.ibm.com>
-
- 23 Jan, 2006 1 commit
-
-
Steve French authored
Could cause memory leak in some readpaths depending on what junk followed it in the stack. Signed-off-by:
Steve French <sfrench@us.ibm.com>
-
- 12 Jan, 2006 1 commit
-
-
Steve French authored
Signed-off-by:
Steve French <sfrench@us.ibm.com>
-
- 12 Dec, 2005 1 commit
-
-
Steve French authored
Signed-off-by:
Steve French <sfrench@us.ibm.com>
-
- 02 Dec, 2005 1 commit
-
-
Steve French authored
memcpy. Part 1 Signed-off-by:
Steve French <sfrench@us.ibm.com>
-
- 29 Nov, 2005 1 commit
-
-
Steve French authored
the request queue. Also periodically wakeup response_q so threads can check if stuck requests have timed out. Workaround Windows server illegal smb length on transact2 findfirst response. Signed-off-by:
Steve French <sfrench@us.ibm.com>
-
- 16 Nov, 2005 1 commit
-
-
Steve French authored
Signed-off-by:
Dave Kleikamp <shaggy@austin.ibm.com> Signed-off-by:
Steve French <sfrench@us.ibm.com>
-
- 11 Oct, 2005 1 commit
-
-
Steve French authored
New cifs_writepages routine was not updated bytes written in cifs stats. Also added ability to clear /proc/fs/cifs/Stats by writing (0 or 1) to it. Signed-off-by:
Steve French <sfrench@us.ibm.com>
-