Re: [PATCH 1/1] cifs: potential missing check for posix_lock_file_wait

2015-05-20 Thread Steve French
merged into cifs-2.6.git for-next On Fri, Apr 3, 2015 at 7:11 AM, Jeff Layton wrote: > On Tue, 24 Mar 2015 20:18:49 -0400 > Chengyu Song wrote: > >> posix_lock_file_wait may fail under certain circumstances, and its result is >> usually checked/returned. But given the complexity of cifs, I'm

Re: [PATCH 1/1] cifs: potential missing check for posix_lock_file_wait

2015-05-20 Thread Steve French
merged into cifs-2.6.git for-next On Fri, Apr 3, 2015 at 7:11 AM, Jeff Layton jeff.lay...@primarydata.com wrote: On Tue, 24 Mar 2015 20:18:49 -0400 Chengyu Song cson...@gatech.edu wrote: posix_lock_file_wait may fail under certain circumstances, and its result is usually checked/returned.

Re: [PATCH 1/1] cifs: potential missing check for posix_lock_file_wait

2015-04-03 Thread Jeff Layton
On Tue, 24 Mar 2015 20:18:49 -0400 Chengyu Song wrote: > posix_lock_file_wait may fail under certain circumstances, and its result is > usually checked/returned. But given the complexity of cifs, I'm not sure if > the result is intentially left unchecked and always expected to succeed. > >

Re: [PATCH 1/1] cifs: potential missing check for posix_lock_file_wait

2015-04-03 Thread Jeff Layton
On Tue, 24 Mar 2015 20:18:49 -0400 Chengyu Song cson...@gatech.edu wrote: posix_lock_file_wait may fail under certain circumstances, and its result is usually checked/returned. But given the complexity of cifs, I'm not sure if the result is intentially left unchecked and always expected to

Re: [PATCH 1/1] cifs: potential missing check for posix_lock_file_wait

2015-03-25 Thread Chengyu Song
> On Mar 24, 2015, at 10:29 PM, Steve French wrote: > > On Tue, Mar 24, 2015 at 7:18 PM, Chengyu Song wrote: >> posix_lock_file_wait may fail under certain circumstances, and its result is >> usually checked/returned. But given the complexity of cifs, I'm not sure if >> the result is

Re: [PATCH 1/1] cifs: potential missing check for posix_lock_file_wait

2015-03-25 Thread Chengyu Song
On Mar 24, 2015, at 10:29 PM, Steve French smfre...@gmail.com wrote: On Tue, Mar 24, 2015 at 7:18 PM, Chengyu Song cson...@gatech.edu wrote: posix_lock_file_wait may fail under certain circumstances, and its result is usually checked/returned. But given the complexity of cifs, I'm not sure

Re: [PATCH 1/1] cifs: potential missing check for posix_lock_file_wait

2015-03-24 Thread Steve French
On Tue, Mar 24, 2015 at 7:18 PM, Chengyu Song wrote: > posix_lock_file_wait may fail under certain circumstances, and its result is > usually checked/returned. But given the complexity of cifs, I'm not sure if > the result is intentially left unchecked and always expected to succeed. > >

[PATCH 1/1] cifs: potential missing check for posix_lock_file_wait

2015-03-24 Thread Chengyu Song
posix_lock_file_wait may fail under certain circumstances, and its result is usually checked/returned. But given the complexity of cifs, I'm not sure if the result is intentially left unchecked and always expected to succeed. Signed-off-by: Chengyu Song --- fs/cifs/file.c | 4 ++-- 1 file

[PATCH 1/1] cifs: potential missing check for posix_lock_file_wait

2015-03-24 Thread Chengyu Song
posix_lock_file_wait may fail under certain circumstances, and its result is usually checked/returned. But given the complexity of cifs, I'm not sure if the result is intentially left unchecked and always expected to succeed. Signed-off-by: Chengyu Song cson...@gatech.edu --- fs/cifs/file.c | 4

Re: [PATCH 1/1] cifs: potential missing check for posix_lock_file_wait

2015-03-24 Thread Steve French
On Tue, Mar 24, 2015 at 7:18 PM, Chengyu Song cson...@gatech.edu wrote: posix_lock_file_wait may fail under certain circumstances, and its result is usually checked/returned. But given the complexity of cifs, I'm not sure if the result is intentially left unchecked and always expected to