Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757475Ab2HUM0U (ORCPT ); Tue, 21 Aug 2012 08:26:20 -0400 Received: from mailhub.sw.ru ([195.214.232.25]:33377 "EHLO relay.sw.ru" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757414Ab2HUM0R (ORCPT ); Tue, 21 Aug 2012 08:26:17 -0400 Message-ID: <50337E56.20808@parallels.com> Date: Tue, 21 Aug 2012 16:25:58 +0400 From: Pavel Emelyanov User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:10.0.1) Gecko/20120209 Thunderbird/10.0.1 MIME-Version: 1.0 To: "J. Bruce Fields" CC: "Aneesh Kumar K.V" , Cyrill Gorcunov , "linux-kernel@vger.kernel.org" , "linux-fsdevel@vger.kernel.org" , Al Viro , Alexey Dobriyan , Andrew Morton , James Bottomley , Matthew Helsley Subject: Re: [patch 4/8] fs, exportfs: Add export_encode_inode_fh helper References: <20120815092116.700948346@openvz.org> <20120815092409.591460800@openvz.org> <87fw7habo4.fsf@skywalker.in.ibm.com> <20120820163338.GN23596@moon> <20120820183225.GB4911@fieldses.org> <20120820190606.GE27443@moon> <20120820193204.GD5779@fieldses.org> <50335261.5090504@parallels.com> <87wr0sle4v.fsf@skywalker.in.ibm.com> <503367CB.9080609@parallels.com> <20120821120908.GB9483@fieldses.org> In-Reply-To: <20120821120908.GB9483@fieldses.org> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3429 Lines: 79 On 08/21/2012 04:09 PM, J. Bruce Fields wrote: > On Tue, Aug 21, 2012 at 02:49:47PM +0400, Pavel Emelyanov wrote: >> On 08/21/2012 02:42 PM, Aneesh Kumar K.V wrote: >>> Pavel Emelyanov writes: >>> >>>> On 08/20/2012 11:32 PM, J. Bruce Fields wrote: >>>>> On Mon, Aug 20, 2012 at 11:06:06PM +0400, Cyrill Gorcunov wrote: >>>>>> On Mon, Aug 20, 2012 at 02:32:25PM -0400, J. Bruce Fields wrote: >>>>>>> On Mon, Aug 20, 2012 at 08:33:38PM +0400, Cyrill Gorcunov wrote: >>>>>>>> On Mon, Aug 20, 2012 at 07:49:23PM +0530, Aneesh Kumar K.V wrote: >>>>>>>>> Cyrill Gorcunov writes: >>>>>>>>> >>>>>>>>>> To provide fsnotify object inodes being watched without >>>>>>>>>> binding to alphabetical path we need to encode them with >>>>>>>>>> exportfs help. This patch adds a helper which operates >>>>>>>>>> with plain inodes directly. >>>>>>>>> >>>>>>>>> doesn't name_to_handle_at() work for you ? It also allows to get a file >>>>>>>>> handle using file descriptor. >>>>>>>> >>>>>>>> Hi, sorry for dealy. Well, the last idea is to get rid of this helper, >>>>>>>> I've sent out an updated version where ino+dev is only printed. >>>>>>> >>>>>>> I don't understand how ino and dev are useful to you, though, if you're >>>>>>> still hoping to be able to look up inodes using this information later >>>>>>> on. >>>>>> >>>>>> Hi Bruce, I believe having ino+dev is better than nothing. Otherwise we >>>>>> simply have no clue which targets are bound to inotify mark. Sometime >>>>>> (!) we can try to generate fhandle in userspace from this ino+dev bundle >>>>>> and then open the target file. >>>>> >>>>> That's insufficient to generate a filehandle in general. >>>> >>>> Yes, sure, but for live migration having inode and device is enough and that's why. >>>> We can use two ways of having a filesystem on the target machine in the same >>>> state (from paths points of view) as it was on destination one: >>>> >>>> 1. copy file tree in a rsync manner >>>> 2. copy a virtual disk image file >>>> >>>> In the 1st case we can map inode number to path easily, since we iterate over a filesystem > > OK. Then you don't care about unlinked files? Yes. If it's unlinked file, we can emulate this on restore with opening any path, then unlinking it. The inode number will change, yes, but in many cases this is acceptable trade-off. > If the filesystem's frozen by the time you get here, I suppose you could > also just use paths? Sure, but where can we get the path from? This is what we're trying to resolve. >>>> anyway. I agree, that rsync is not perfect for migration but still. >>>> >>>> In the 2nd case we can generate filehandle out of an inode number only since we _do_ know >>>> that inode will not get reused. >>> >>> If you are going to to use open_by_handle, then that handle is not >>> sufficient right ? Or do you have open_by_inode ? as part of c/r ? >> >> Why? For e.g. ext4 you can construct a handle in userspace and open by it. > > If it's a real filehandle you want, in general you don't want to > construct it in userspace--depending on the filesystem it may require > filesystem-specific knowledge. I see. > --b. Thanks, Pavel -- 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/