Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752550AbdLLKPI (ORCPT ); Tue, 12 Dec 2017 05:15:08 -0500 Received: from mail-it0-f54.google.com ([209.85.214.54]:42378 "EHLO mail-it0-f54.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751547AbdLLKPG (ORCPT ); Tue, 12 Dec 2017 05:15:06 -0500 X-Google-Smtp-Source: ACJfBovG9H9RdnLRXRd7LJqceCKvrMukvdTqKfhQd20m541x22R4ZppGQ6WRyh+aFaTwJ2WRwf/izVmyzlAArIvsnFA= MIME-Version: 1.0 In-Reply-To: <1513043801-9164-1-git-send-email-baiyaowei@cmss.chinamobile.com> References: <1513043801-9164-1-git-send-email-baiyaowei@cmss.chinamobile.com> From: Ilya Dryomov Date: Tue, 12 Dec 2017 11:15:05 +0100 Message-ID: Subject: Re: [PATCH] fs/ceph: delete unused ceph_file_layout_is_valid function To: Yaowei Bai Cc: "Yan, Zheng" , Sage Weil , Ceph Development , linux-kernel@vger.kernel.org Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 384 Lines: 11 On Tue, Dec 12, 2017 at 2:56 AM, Yaowei Bai wrote: > There's no users of ceph_file_layout_is_valid, remove it. It'll get a user in the next kernel release -- we'll remove ad-hoc checks from ceph_calc_file_object_mapping() which is executed on every mapping and call ceph_file_layout_is_valid() once per layout instead. Thanks, Ilya