From mboxrd@z Thu Jan 1 00:00:00 1970 From: Christoph Hellwig Subject: Re: smb2 work status Date: Sat, 16 Jul 2011 15:28:11 -0400 Message-ID: <20110716192811.GE26925@infradead.org> References: Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: linux-cifs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Jeff Layton , Christoph Hellwig , Steve French To: Pavel Shilovsky Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-cifs-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-ID: On Sat, Jul 16, 2011 at 01:29:22PM +0400, Pavel Shilovsky wrote: > Jeff, Christoph, can you comment on this, please? I really don't like all that ifdef mess. I'm not quite sure how to fix in in general, though. For the inode operations it's fairly simple: just have a different set for smb2, the only shared code is just the path_from_dentry call, so there's not much duplication. But for the rest I'm not too sure. Stubbing things out smarter would help, as would creating more helpers but I'm not sure that's going to help with everything. Also some of the big bulk commits earlier in the series add code that's pretty far from the normal Linux style, e.g. smb2pdu.c, it would be nice to at least make the new code in cifs look normal.