Received: by 2002:a05:6a10:f347:0:0:0:0 with SMTP id d7csp1510666pxu; Thu, 17 Dec 2020 11:39:42 -0800 (PST) X-Google-Smtp-Source: ABdhPJycMkFcmTY+ymmv5jiyk6apcMgOHFWEwPHesPzBw4OKz7yKIxik/OdFRS6zbuz///YPjgGl X-Received: by 2002:a17:907:c15:: with SMTP id ga21mr609311ejc.472.1608233982665; Thu, 17 Dec 2020 11:39:42 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1608233982; cv=none; d=google.com; s=arc-20160816; b=G0W2pKffRNEapR6YJkjfhCT/sQxrsXUqujCVxVOTku4pwif9sfmM3F+LN5P6cVMotU WANyQJXhH47AdXayHxzeiVORloQMc4r4d4kvss5n85OerEROu8QPBiwv9LSpPLysKq/w 6OTd+dq3qsBjsDcHoOpPQFLhmUG2P3dwmu8GEMO1xdfC7JKs7UOuiR6xcqo15oWFP1uh INnkNd2BHCBi0ZmIoP8MmDLKK4n7rXgXT5Z253xZ6oZgrciylabf0BIpRpdAtlo7MH4P IIOMrfw8TKkHUgcT4UHb1LUagceDe4CbLBLYOH+OjiLBulNJm7knKIjJ0YBCM2/NNRWt oIgw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:dkim-signature:content-transfer-encoding :content-language:in-reply-to:mime-version:user-agent:date :message-id:from:references:cc:to:subject; bh=6rL5WMauA8+LXyGXmzjXvOTi9WOgrRuLBi9GxeFHQwQ=; b=fg6hKH/NSQxm3GPbnTqKn4cKAke/ndJARLSpSdp+6De5lBo8Wj6Fqzp91VaD/kJNvi 95UJSIqN78Njqd18+CWOITRaKgc6VUARVq4sP6QBsv9sEFhTqzbs4qREHVtAMDID/Nbh wZ4/0Vrm9hvZTcv7BnaQnI+ce0bHZ2lfjwIc7BlynkWwDO2sF4yBOCCy8IjF2/eqfTTd yQg9Nd9d1pz9/MrwcDN+B6avhCM4cbcWa791SruQvR59a40r8O2vISKy+aid8p67bBGx 5gvwPTldPAx8ing83lq5xA8qNNrqkFXx7WIhC7B9MUOGBQPoYDJPi/BeztoKOpRvAt77 3BCA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@nvidia.com header.s=n1 header.b=HLEYh7m+; 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=NONE dis=NONE) header.from=nvidia.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id a17si5105322edn.146.2020.12.17.11.39.19; Thu, 17 Dec 2020 11:39:42 -0800 (PST) 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=@nvidia.com header.s=n1 header.b=HLEYh7m+; 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=NONE dis=NONE) header.from=nvidia.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729685AbgLQThj (ORCPT + 99 others); Thu, 17 Dec 2020 14:37:39 -0500 Received: from hqnvemgate24.nvidia.com ([216.228.121.143]:3382 "EHLO hqnvemgate24.nvidia.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1729069AbgLQThj (ORCPT ); Thu, 17 Dec 2020 14:37:39 -0500 Received: from hqmail.nvidia.com (Not Verified[216.228.121.13]) by hqnvemgate24.nvidia.com (using TLS: TLSv1.2, AES256-SHA) id ; Thu, 17 Dec 2020 11:36:58 -0800 Received: from [10.2.61.104] (172.20.145.6) by HQMAIL107.nvidia.com (172.20.187.13) with Microsoft SMTP Server (TLS) id 15.0.1473.3; Thu, 17 Dec 2020 19:36:58 +0000 Subject: Re: [PATCH 18/25] btrfs: Use readahead_batch_length To: Matthew Wilcox CC: , , References: <20201216182335.27227-1-willy@infradead.org> <20201216182335.27227-19-willy@infradead.org> <20201217121246.GD15600@casper.infradead.org> <20201217134253.GE15600@casper.infradead.org> From: John Hubbard Message-ID: Date: Thu, 17 Dec 2020 11:36:58 -0800 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:84.0) Gecko/20100101 Thunderbird/84.0 MIME-Version: 1.0 In-Reply-To: <20201217134253.GE15600@casper.infradead.org> Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit X-Originating-IP: [172.20.145.6] X-ClientProxiedBy: HQMAIL111.nvidia.com (172.20.187.18) To HQMAIL107.nvidia.com (172.20.187.13) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nvidia.com; s=n1; t=1608233818; bh=6rL5WMauA8+LXyGXmzjXvOTi9WOgrRuLBi9GxeFHQwQ=; h=Subject:To:CC:References:From:Message-ID:Date:User-Agent: MIME-Version:In-Reply-To:Content-Type:Content-Language: Content-Transfer-Encoding:X-Originating-IP:X-ClientProxiedBy; b=HLEYh7m+YH14H3re5sfCCvg1KTEdsatWYftLMwpfepT3/eYkHw3P3Eq4rb0XgKmqZ LVFaBzKsGC26TSljIlKDxVNpKYsW1woSLuiA6iTXMp/QIa8NZi+ywUTtcm9BAkQC/s 8XHmXqz8Rb4h7gD6Kq8Cf1krkdXne6HSCxr38AlWn/OTts/nXkmfe4/abBzoNF7asE KW8knZ0bTpYRM00SzLYZl7SvfCIe4mc70RKN8pk3wfETmtcGHSqiPmQNx2gz1qG/Nc uQmsDgx4lszWHKwxqGa9s0O7yH6k6qWxr2PhzinE0v0c1HtFalpUHoD5Rv3N1zf37U wFuZozwfh4vBA== Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 12/17/20 5:42 AM, Matthew Wilcox wrote: > On Thu, Dec 17, 2020 at 12:12:46PM +0000, Matthew Wilcox wrote: >> ehh ... probably an off-by-one. Does subtracting 1 from contig_end fix it? >> I'll spool up a test VM shortly and try it out. > > Yes, this fixed it: > > - u64 contig_end = contig_start + readahead_batch_length(rac); > + u64 contig_end = contig_start + readahead_batch_length(rac) - 1; > Yes, confirmed on my end, too. thanks, -- John Hubbard NVIDIA