Re: [linux-cifs-client] review 5, was Re: projected date for mount.cifs to support DFS junction points

2008-02-25 Thread Steve French
On Fri, Feb 15, 2008 at 4:11 PM, Christoph Hellwig <[EMAIL PROTECTED]> wrote: > If you like these kind of consolidation patches here's another one: Merged into cifs-2.6.git tree -- Thanks, Steve - To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message

Re: [linux-cifs-client] review 5, was Re: projected date for mount.cifs to support DFS junction points

2008-02-18 Thread Steve French
The patch looks fine - but since it does not set obj_type any more - I want to think about it a little more since it may be useful coming back from the open path (although the mode is probably good enough). jra added support to Samba for a new POSIX open/create/mkdir request (which we only use for

Re: Re[2]: [linux-cifs-client] review 5, was Re: projected date for mount.cifs to support DFS junction points

2008-02-16 Thread Christoph Hellwig
On Sat, Feb 16, 2008 at 11:51:52AM +0300, Q wrote: > At first glance cifs_get_inode_info_remote won't work cause it's old dfs > code not new one. But I caught what Christoph meant now, and will try to > rewrite it this way. Yes, this was supposed to be a refactoring of the existing code. By doing

Re[2]: [linux-cifs-client] review 5, was Re: projected date for mount.cifs to support DFS junction points

2008-02-16 Thread Q
-Original Message- From: "Steve French" <[EMAIL PROTECTED]> To: "Christoph Hellwig" <[EMAIL PROTECTED]> Date: Fri, 15 Feb 2008 15:02:19 -0600 Subject: Re: [linux-cifs-client] review 5, was Re: projected date for mount.cifs to support DFS junction po

Re: [linux-cifs-client] review 5, was Re: projected date for mount.cifs to support DFS junction points

2008-02-15 Thread Christoph Hellwig
If you like these kind of consolidation patches here's another one: Index: linux-2.6/fs/cifs/inode.c === --- linux-2.6.orig/fs/cifs/inode.c 2008-02-15 22:46:08.0 +0100 +++ linux-2.6/fs/cifs/inode.c 2008-02-15 23:09:28.

Re: [linux-cifs-client] review 5, was Re: projected date for mount.cifs to support DFS junction points

2008-02-15 Thread Steve French
On 2/15/08, Christoph Hellwig <[EMAIL PROTECTED]> wrote: > On Fri, Feb 15, 2008 at 07:37:35PM +0300, Q (Igor Mammedov) wrote: > > Here is what I've done the last weekend. > > Attached: > > fixed patch [5/5] > (0001-DFS-patch-that-connects-inode-with-dfs-handling-ops.patch). Not merged yet. >

Re: [linux-cifs-client] review 5, was Re: projected date for mount.cifs to support DFS junction points

2008-02-15 Thread Christoph Hellwig
On Fri, Feb 15, 2008 at 07:37:35PM +0300, Q (Igor Mammedov) wrote: > Sorry guys, but I have a lot of work for the last 3 weeks, > so I couldn't spare much time for a hobby and react quickly. No problem. I know this problem very well as almost all of my core kernel contributions are spare time as

Re: [linux-cifs-client] review 5, was Re: projected date for mount.cifs to support DFS junction points

2008-02-15 Thread Q (Igor Mammedov)
Sorry guys, but I have a lot of work for the last 3 weeks, so I couldn't spare much time for a hobby and react quickly. Here is what I've done the last weekend. Attached: fixed patch [5/5] (0001-DFS-patch-that-connects-inode-with-dfs-handling-ops.patch). fixed mixed case in struct member (0002