Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753133Ab0HGKeL (ORCPT ); Sat, 7 Aug 2010 06:34:11 -0400 Received: from cdptpa-omtalb.mail.rr.com ([75.180.132.121]:49073 "EHLO cdptpa-omtalb.mail.rr.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751142Ab0HGKeH (ORCPT ); Sat, 7 Aug 2010 06:34:07 -0400 X-Authority-Analysis: v=1.1 cv=XXdbi/b60h1aHGKgZJQtFDgFv5mfg/3/ZtEWBleeTtY= c=1 sm=0 a=8V7XDGGG4kMA:10 a=kj9zAlcOel0A:10 a=ld/erqUjW76FpBUqCqkKeA==:17 a=hGzw-44bAAAA:8 a=2pZHIKDf-8NVIubnCcwA:9 a=gHFx3b6OxZgAjL3Nc89Vg--aUJ8A:4 a=CjuIK1q_8ugA:10 a=0kPLrQdw3YYA:10 a=dowx1zmaLagA:10 a=ld/erqUjW76FpBUqCqkKeA==:117 X-Cloudmark-Score: 0 X-Originating-IP: 71.70.153.3 Date: Sat, 7 Aug 2010 06:34:00 -0400 From: Jeff Layton To: Neil Brown Cc: Steve French , Jeremy Allison , utz lehmann , Linus Torvalds , Volker.Lendecke@sernet.de, David Howells , Jan Engelhardt , linux-cifs@vger.kernel.org, linux-nfs@vger.kernel.org, samba-technical@lists.samba.org, linux-kernel@vger.kernel.org, viro@zeniv.linux.org.uk, linux-fsde@jasper.es Subject: Re: [PATCH 02/18] xstat: Add a pair of system calls to make extended file stats available [ver #6] Message-ID: <20100807063400.6bd1776f@tlielax.poochiereds.net> In-Reply-To: <20100807133240.02883b83@notabene> References: <20100715021709.5544.64506.stgit@warthog.procyon.org.uk> <20100715021712.5544.44845.stgit@warthog.procyon.org.uk> <30448.1279800887@redhat.com> <1280524978.2452.9.camel@segv.aura.of.mankind> <20100801092529.5e6ba0e0@corrin.poochiereds.net> <20100805235218.GB31233@jeremy-laptop> <20100806133836.49757af9@notabene> <20100807093057.7683bedd@notabene> <20100807102901.6a0b53e7@notabene> <20100807133240.02883b83@notabene> X-Mailer: Claws Mail 3.7.6 (GTK+ 2.20.1; x86_64-redhat-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1682 Lines: 36 On Sat, 7 Aug 2010 13:32:40 +1000 Neil Brown wrote: > So we are left with an attribute that is needed for windows compatibility, > and so just needs to be understood by samba and wine. Some filesystems might > support it efficiently, others might require the use of generic > extended-attributes, still others might not support it at all (I guess you > store it in some 'tdb' and hope it works well enough). > > Core-linux doesn't really need to know about this - there just needs to be a > channel to pass it between samba/wine and the filesystem. xattr still seems > the best mechanism to pass this stuff around. Team-samba can negotiate with > fs developers to optimise/accelerate certain attributes, and linux-VFS > doesn't need to know or care (except maybe to provide generic non-blocking or > multiple-access interfaces). > IIUC, you're saying that we should basically just have samba stuff the current time into an xattr when it creates the file and leave the filesystems alone. If so, I disagree here. The problem with treating this as *just* an xattr is that it doesn't account for files that are created outside of samba but are then shared out by it. To handle this correctly, I believe it needs to be initialized by the kernel to the current time whenever an inode is created, even if samba doesn't create it. After that, it can be treated as just another xattr. -- Jeff Layton -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/