Received: by 2002:a25:ef43:0:0:0:0:0 with SMTP id w3csp1129020ybm; Wed, 27 May 2020 17:17:27 -0700 (PDT) X-Google-Smtp-Source: ABdhPJy1PaI3qGjYp75VeaDrJNjcCrBIUV9esbP3OigSN3GSCpt59SrcWLWgD/nhF9QfwfOxkdmg X-Received: by 2002:a17:906:160a:: with SMTP id m10mr818178ejd.170.1590625047057; Wed, 27 May 2020 17:17:27 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1590625047; cv=none; d=google.com; s=arc-20160816; b=GYjfX8fq4vcYSDxR7P2s8Nfc2WsrcImyW27HuxER7lA4twsg1EKfKS4aVDfI9ofsJm +8Lisy4lyAwLmI7sfXq/HB7p9o6JQZW9NhyOEk5RXbDAK30FUhE+0+gAKVENXyaLHBqw CpIg7Dkf0TwmbBG78sV5CUUnBbz430MH2Z3ZB9b9KDpqEtYpk7xW2gMiAnNDSz+6uOg/ xB3NVboY9izJ/yGGZCnd9x5Ln93kUF+lygjeFz/kBxGMHRtsLlfMKSSfjdlepbQuJvTf G5nCbLOHHmM3S+AxS4RKcepAm0lZ2K3jGe+nZ5u54zvKm7/2M3GyVcDxHqfvclbzd/vO EpJw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding :content-language:in-reply-to:mime-version:user-agent:date :message-id:from:references:cc:to:subject:dkim-signature; bh=P8ZX6NswqYpmp7AerrBDt1jBaXqAyhSsaEu8p+L3oCE=; b=UWktLPT4RN5MqMb3BYk+G1kMEj1RBW5bZMEdC9jjSGjJuynfW/4zU6TuCuvbRqpFNq 62BbvVMO098bx626HQmmj5mQcV7CNoBZX3D51bv1l1bV99hTIvAa2jXHya2Bz0C10ecO Hj8/oUWJvbe3GPhLFdZoPOb0nwqvyrQDPKTeEaukrMDGXv7BiflN55YMVvDTjH7fucai C1wX3mPtVM+3Ct9P6x2Ujaeu6M1oQW+Bowb2pDt2PZj4sPAD1gaXfQ4mF9JddMOK6PWl yvBgpa5Q9FEmk5lOuJ2m4cFgemTmPSw9LM6kUCL/YRU1J6CdL/V/JrRIb8TuKu5nmB3V Q01A== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=iKLqcWmV; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id se14si2769511ejb.103.2020.05.27.17.17.02; Wed, 27 May 2020 17:17:27 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=iKLqcWmV; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1725850AbgE1AM4 (ORCPT + 99 others); Wed, 27 May 2020 20:12:56 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:50620 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725681AbgE1AM4 (ORCPT ); Wed, 27 May 2020 20:12:56 -0400 Received: from mail-pf1-x443.google.com (mail-pf1-x443.google.com [IPv6:2607:f8b0:4864:20::443]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 6912EC05BD1E; Wed, 27 May 2020 17:12:56 -0700 (PDT) Received: by mail-pf1-x443.google.com with SMTP id z26so12599173pfk.12; Wed, 27 May 2020 17:12:56 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=subject:to:cc:references:from:message-id:date:user-agent :mime-version:in-reply-to:content-language:content-transfer-encoding; bh=P8ZX6NswqYpmp7AerrBDt1jBaXqAyhSsaEu8p+L3oCE=; b=iKLqcWmV5z5p/JypITIwKEFX2LyCZV9tu37lGMrCJvoOrB+JBVptoFiBh5FgHaUjF0 wOqAigXgQ4PhC/G9wWNnNPI4VG1aZsCSJZVY6Fo4s/xWgeXEeJbWGQuOn1XiZSDo2vTt sIVJq4JN0uOo25JAznQVs7c8LVC0xmI++cZK+XSr5f5GeVvIKF+PJRtP1bFAud4HVWQy 3CAKwCM//8w28wWFoNKwy+qOQG5rnMDB9cjTm54fLySgkhnBeTGHJqFBHj+0Nd4qLgOe RfWLucbAEZOnfsQRqdxWAUaPLzN5s0ggQNbKaMWdliVyILqQk1ocXrtPe+i86S5gTbF5 j7fg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:cc:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=P8ZX6NswqYpmp7AerrBDt1jBaXqAyhSsaEu8p+L3oCE=; b=nfRMEhBtbNZfiRmBIHFN7K/s+X4xdJmFSwqBDdvYa9rQNtMeBF9gDKSYPwkGPzfzAB PIbJqpqeoMn6kdJiQ6cwgfziJ4IU/bXt5inISavF443IiS8YJ5xbOs8FdToH8neMHIZf fXzHSjNQQadepUm7vFVcnG4G3BtQnY7042xAfFD9w+eCr0Yp9oCPU7IGkGWP5xoJ9Y3c MKi6F2g01e6pxdoYcliyB50uEnh2VRDcYXe3d4DuCW7xcuHwNfg8rrtYtV5NZVu2GhxK oKCs78CWFF1cMQb+a2CTKlLydJxdgJpvS+jHwaeyjDp2ZjYI9c5khSyX1fuJUzu245St jl2Q== X-Gm-Message-State: AOAM532OViw58fl/o9Q31NKXirZoHz5r1s7dkVmQdToEkJcFP84cf9Xd 8AJ4E8cpkJOmtGm6GiWsW1hHUz6VI2E= X-Received: by 2002:a05:6a00:134c:: with SMTP id k12mr275880pfu.313.1590624775625; Wed, 27 May 2020 17:12:55 -0700 (PDT) Received: from ?IPv6:::1? ([2404:7a87:83e0:f800:295a:ef64:e071:39ab]) by smtp.gmail.com with ESMTPSA id f136sm2915747pfa.59.2020.05.27.17.12.53 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Wed, 27 May 2020 17:12:54 -0700 (PDT) Subject: Re: [PATCH] exfat: optimize dir-cache To: Sungjong Seo , 'Namjae Jeon' , "'Kohada.Tetsuhiro@dc.MitsubishiElectric.co.jp'" Cc: "'Mori.Takahiro@ab.MitsubishiElectric.co.jp'" , "'Motai.Hirotaka@aj.MitsubishiElectric.co.jp'" , 'Namjae Jeon' , linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org References: <20200520075641.32441-1-kohada.tetsuhiro@dc.mitsubishielectric.co.jp> <055a01d63306$82b13440$88139cc0$@samsung.com> <000701d63432$ace24f10$06a6ed30$@samsung.com> From: Tetsuhiro Kohada Message-ID: <22dfcd8a-4416-e2a7-b8a7-0375660ba465@gmail.com> Date: Thu, 28 May 2020 09:12:52 +0900 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:68.0) Gecko/20100101 Thunderbird/68.8.1 MIME-Version: 1.0 In-Reply-To: <000701d63432$ace24f10$06a6ed30$@samsung.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit X-Antivirus: Avast (VPS 200527-0, 2020/05/27), Outbound message X-Antivirus-Status: Clean Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org >>> > In order to prevent illegal accesses to bh and dentries, it would >>> be better to check validation for num and bh. >>> >>> There is no new error checking for same reason as above. >>> >>> I'll try to add error checking to this v2 patch. >>> Or is it better to add error checking in another patch? >> The latter:) >> Thanks! > > Yes, the latter looks better. I will do so. I will post additional patches for error checking, after this patch is merged into tree. OK?