From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933915AbcBRH4a (ORCPT ); Thu, 18 Feb 2016 02:56:30 -0500 Received: from mail-wm0-f49.google.com ([74.125.82.49]:35544 "EHLO mail-wm0-f49.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933511AbcBRH42 (ORCPT ); Thu, 18 Feb 2016 02:56:28 -0500 Subject: Re: [PATCH] ovl: fix working on distributed fs as lower layer To: Konstantin Khlebnikov , linux-fsdevel@vger.kernel.org, linux-unionfs@vger.kernel.org, Miklos Szeredi References: <145424653619.10827.10119853770850679426.stgit@zurg> Cc: linux-kernel@vger.kernel.org, Alexander Viro From: Nikolay Borisov Message-ID: <56C57928.2010606@kyup.com> Date: Thu, 18 Feb 2016 09:56:24 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.1.0 MIME-Version: 1.0 In-Reply-To: <145424653619.10827.10119853770850679426.stgit@zurg> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 01/31/2016 03:22 PM, Konstantin Khlebnikov wrote: > This adds missing .d_select_inode into alternative dentry_operations. > > Signed-off-by: Konstantin Khlebnikov > Fixes: 7c03b5d45b8e ("ovl: allow distributed fs as lower layer") > Fixes: 4bacc9c9234c ("overlayfs: Make f_path always point to the overlay and f_inode to the underlay") > --- > fs/overlayfs/super.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/fs/overlayfs/super.c b/fs/overlayfs/super.c > index e6ae59c7119c..1a354840f262 100644 > --- a/fs/overlayfs/super.c > +++ b/fs/overlayfs/super.c > @@ -343,6 +343,7 @@ static const struct dentry_operations ovl_dentry_operations = { > > static const struct dentry_operations ovl_reval_dentry_operations = { > .d_release = ovl_dentry_release, > + .d_select_inode = ovl_d_select_inode, > .d_revalidate = ovl_dentry_revalidate, > .d_weak_revalidate = ovl_dentry_weak_revalidate, > }; I wish I had seen this patch earlier than https://marc.info/?l=linux-unionfs&m=145494313009959 Reviewed-by: Nikolay Borisov Tested-by: Nikolay Borisov