From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-4.0 required=3.0 tests=DKIMWL_WL_MED,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_PASS,URIBL_BLOCKED autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 1405BC10F0E for ; Thu, 18 Apr 2019 16:59:03 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id D608F21479 for ; Thu, 18 Apr 2019 16:59:02 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=intel-com.20150623.gappssmtp.com header.i=@intel-com.20150623.gappssmtp.com header.b="lPDP9gRl" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2389736AbfDRQ65 (ORCPT ); Thu, 18 Apr 2019 12:58:57 -0400 Received: from mail-ot1-f67.google.com ([209.85.210.67]:33037 "EHLO mail-ot1-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S2389682AbfDRQ65 (ORCPT ); Thu, 18 Apr 2019 12:58:57 -0400 Received: by mail-ot1-f67.google.com with SMTP id j10so2314235otq.0 for ; Thu, 18 Apr 2019 09:58:56 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=intel-com.20150623.gappssmtp.com; s=20150623; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=RGS6YsOAVjQwa/YToZjt45UAijXd7Hd/Y2bsvCoDiv4=; b=lPDP9gRl2UM8hL2yHwEifePQ2gNrAFI7Lm7RsIYQ6i/ADklZsURlEsFOJzxucAfxEw lUaRhOTQcSS1bZvlJD7canHxoZsoBpryjPw2fpJ81Q8YMJvjGSFtilCNnJlWHWNgRHwo O3w01XwE6RAvU3KhewiLqPAOTt8H788oq3cOeLbii5wRr/l83ayuQCdbw4GPyO/Dx4si sb+3FQtXCv8psVj21FtUDDpIZi5Q5qEWf1Uyoj6ELOR3LBIKkjOjip1fGiW6pm5wXc/+ UObMfY8J4ms31f9IoSjhzGfRWVfMpHteNuGS9kH/FLe6yKdMaVg9sDii9o6qLqfDcYV1 4DMg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=RGS6YsOAVjQwa/YToZjt45UAijXd7Hd/Y2bsvCoDiv4=; b=WywoXwaZpbOXq8UPJgyYDa81mS7dems69rN//8yf3IoVUucNKh0XPUjO+KsZ+ED3YZ j7Xz4QY3u8qv441Vd7FtWi5PW0AlK160lBuoI+eUvNkVmf8sPZTuu+U26zwjKOHwUBRQ s5j6TJ0QY5TbuBcn6HntXuZq4IkcxgMYxh3UHcKh9nD8BTPilKbFik6Ls6u/CLspJn5c wLniJ+EdasRnJ2uDGbDV/r2pezbvpk/fNX0o7tM9907AC7pjhdT7Z3+aF7q7AgIkxUpi gsKd0w6txFgn+Jlh95YQJeKobMpSyD6UobB3mQNbWQD/z6QCj41y1Ram2zUIa7qQqvZj 5O+Q== X-Gm-Message-State: APjAAAWxtMGVIJv0eXc7gLJ0nEBXkc4beIu/M4fp+Iipp5J6ykEzJjzi 1voustm07EiMiz2kfhCnpfFohmfKWhP/ahZlohvwPjcF X-Google-Smtp-Source: APXvYqzG9AhEh+CfRsyU3+SnE2IKIkora7yoZ4lhLKbo3TwMDMW8FpW9W0RM8ttvNJnHFNXaiaMWi7ypdHHrK54vTYo= X-Received: by 2002:a05:6830:15d4:: with SMTP id j20mr4430515otr.367.1555606736360; Thu, 18 Apr 2019 09:58:56 -0700 (PDT) MIME-Version: 1.0 References: <20190416174900.GT2217@ZenIV.linux.org.uk> <20190416175340.21068-1-viro@ZenIV.linux.org.uk> <20190416175340.21068-49-viro@ZenIV.linux.org.uk> <20190418121620.GI28541@quack2.suse.cz> In-Reply-To: <20190418121620.GI28541@quack2.suse.cz> From: Dan Williams Date: Thu, 18 Apr 2019 09:58:45 -0700 Message-ID: Subject: Re: [RFC PATCH 49/62] dax: make use of ->free_inode() To: Jan Kara Cc: Al Viro , Linus Torvalds , Linux Kernel Mailing List , linux-fsdevel Content-Type: text/plain; charset="UTF-8" Sender: linux-fsdevel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-fsdevel@vger.kernel.org On Thu, Apr 18, 2019 at 5:16 AM Jan Kara wrote: > > On Tue 16-04-19 18:53:27, Al Viro wrote: > > From: Al Viro > > > > we might want to drop ->destroy_inode() there - it's used only for > > WARN_ON() now, and AFAICS that could be moved to ->evict_inode() > > if we had one... > > > > Signed-off-by: Al Viro > > --- > > drivers/dax/super.c | 7 ++----- > > 1 file changed, 2 insertions(+), 5 deletions(-) > > Added Dan to CC since I'm not sure how closely he follows fsdevel. The > patch looks good to me FWIW so feel free to add: > > Reviewed-by: Jan Kara Thanks Jan. Acked-by: Dan Williams Al, I assume you'll merge this with the other free_inode changes.