From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753507AbdADFyJ convert rfc822-to-8bit (ORCPT ); Wed, 4 Jan 2017 00:54:09 -0500 Received: from mailsec106.isp.belgacom.be ([195.238.20.102]:43262 "EHLO mailsec106.isp.belgacom.be" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752334AbdADFyF (ORCPT ); Wed, 4 Jan 2017 00:54:05 -0500 X-IronPort-Anti-Spam-Filtered: true X-IronPort-Anti-Spam-Result: =?us-ascii?q?A2BtAQCQjWxY/9MU7sNeGgEBAQECAQEBA?= =?us-ascii?q?QgBAQEBgzcBAQEBAR+Ba41XlEeTDoIPggiGIgKBTD8UAQIBAQEBAQEBYyiEaAE?= =?us-ascii?q?BAQMBIwRSBQsFBhgCAhgOAgJXBhMRiFcMrzKBazqEZ4VTAQsmgQuFOoRhhEaDB?= =?us-ascii?q?IJdBZp9ohSSPR84gSqGDT00iE0BAQE?= X-IPAS-Result: =?us-ascii?q?A2BtAQCQjWxY/9MU7sNeGgEBAQECAQEBAQgBAQEBgzcBAQE?= =?us-ascii?q?BAR+Ba41XlEeTDoIPggiGIgKBTD8UAQIBAQEBAQEBYyiEaAEBAQMBIwRSBQsFB?= =?us-ascii?q?hgCAhgOAgJXBhMRiFcMrzKBazqEZ4VTAQsmgQuFOoRhhEaDBIJdBZp9ohSSPR8?= =?us-ascii?q?4gSqGDT00iE0BAQE?= Date: Wed, 4 Jan 2017 06:53:31 +0100 (CET) From: Fabian Frederick Reply-To: Fabian Frederick To: Al Viro Cc: Andrew Morton , linux-kernel@vger.kernel.org, Jan Kara , linux-fsdevel@vger.kernel.org Message-ID: <736635494.1055379.1483509211858.open-xchange@webmail.nmp.proximus.be> In-Reply-To: <20170103222906.GC1555@ZenIV.linux.org.uk> References: <1483479039-5255-1-git-send-email-fabf@skynet.be> <20170103222906.GC1555@ZenIV.linux.org.uk> Subject: Re: [PATCH 3/6 linux-next] fs/affs: make affs exportable MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8BIT X-Priority: 3 Importance: Medium X-Mailer: Open-Xchange Mailer v7.2.2-Rev27 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org > On 03 January 2017 at 23:29 Al Viro wrote: > > > On Tue, Jan 03, 2017 at 10:30:39PM +0100, Fabian Frederick wrote: > > Add standard functions making AFFS work with NFS. > > > > Functions based on ext4 implementation. > > Tested on loop device. > > How the hell is that supposed to work with cold dcache?  You don't have > ->get_parent() there at all... > > There *IS* a reference to parent directory in those suckers - not the same > kind as in normal unix filesystems (".." is not a directory entry there - > it's all fake), but it's doable.  be32_to_cpu(AFFS_TAIL(sb, bh)->parent) > would be the inumber you need, where bh is the inode block of directory. > > So it can be done, but not in this form.  NAK for the time being... I worked on that function declared optional in Documentation/filesystems/nfs/Exporting but was unable to test it. Regards, Fabian