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=-3.6 required=3.0 tests=BAYES_00,DKIM_INVALID, DKIM_SIGNED,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS autolearn=no 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 647DEC4727C for ; Tue, 29 Sep 2020 13:39:28 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id 8B6A52158C for ; Tue, 29 Sep 2020 13:39:27 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="signature verification failed" (2048-bit key) header.d=infradead.org header.i=@infradead.org header.b="Rs6r/RNx" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 8B6A52158C Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=infradead.org Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id 196ED90000C; Tue, 29 Sep 2020 09:39:27 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 1498B6B007E; Tue, 29 Sep 2020 09:39:27 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 036FB90000C; Tue, 29 Sep 2020 09:39:26 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0185.hostedemail.com [216.40.44.185]) by kanga.kvack.org (Postfix) with ESMTP id E23E36B007D for ; Tue, 29 Sep 2020 09:39:26 -0400 (EDT) Received: from smtpin10.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay04.hostedemail.com (Postfix) with ESMTP id 9F647585C for ; Tue, 29 Sep 2020 13:39:26 +0000 (UTC) X-FDA: 77316205932.10.vest67_2e0d0ee2718a Received: from filter.hostedemail.com (10.5.16.251.rfc1918.com [10.5.16.251]) by smtpin10.hostedemail.com (Postfix) with ESMTP id C3CBF16A4D0 for ; Tue, 29 Sep 2020 13:39:15 +0000 (UTC) X-HE-Tag: vest67_2e0d0ee2718a X-Filterd-Recvd-Size: 3044 Received: from casper.infradead.org (casper.infradead.org [90.155.50.34]) by imf01.hostedemail.com (Postfix) with ESMTP for ; Tue, 29 Sep 2020 13:39:15 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=casper.20170209; h=In-Reply-To:Content-Type:MIME-Version: References:Message-ID:Subject:Cc:To:From:Date:Sender:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description; bh=JVdM3afc/YXIjdB8I/oCcRIHSX+4wscc850PEE+wd4k=; b=Rs6r/RNxuUjCouIoyH1LgIFHig ZTsO/l01bEIOP/met2gMeh1XazUKKhGCBp+i68Dkbp7Ly9EChLkEQSuchYSoJrYUQQIjdRWJqPQ6l ZXsLtL9t5Bm96uttlO7747k2sz47KKux/V+2BjsvOGk7kXw3J6vBab1uwPhMufU90Dy6f9IaFQiBV 7nHPha5AX4B9vUKBvMY88fyAm1zzjbSNlRZBt45AVxJ/WBqpYAhV2AadCHo1r2aHTWJtoydUVOdd9 vfmHZpNf8857K3QZ+SW5hXeM7WmYdJ3X/P/+ljwT1SOzDQs4utR2GQt/1G9fkT5pDp+rrM4oOrXFh XgpoCeJQ==; Received: from willy by casper.infradead.org with local (Exim 4.92.3 #3 (Red Hat Linux)) id 1kNFqY-0001ow-7D; Tue, 29 Sep 2020 13:39:10 +0000 Date: Tue, 29 Sep 2020 14:39:10 +0100 From: Matthew Wilcox To: Jan Kara Cc: linux-mm@kvack.org, Andrew Morton , Hugh Dickins , William Kucharski , Johannes Weiner , Yang Shi , Dave Chinner , linux-kernel@vger.kernel.org Subject: Re: [PATCH v2 04/12] mm/filemap: Add mapping_seek_hole_data Message-ID: <20200929133910.GD20115@casper.infradead.org> References: <20200914130042.11442-1-willy@infradead.org> <20200914130042.11442-5-willy@infradead.org> <20200929084653.GC10896@quack2.suse.cz> <20200929124251.GB20115@casper.infradead.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20200929124251.GB20115@casper.infradead.org> X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: On Tue, Sep 29, 2020 at 01:42:51PM +0100, Matthew Wilcox wrote: > On Tue, Sep 29, 2020 at 10:46:53AM +0200, Jan Kara wrote: > > Also for shmem you've dropped the PageUptodate check which I'm not sure is > > safe? > > That was unintentional. I did run xfstests against this patch (just did > it again ... it passes), so I suspect it doesn't create a !Uptodate page. > I'll see if I can enhance the existing xfstests to catch this case. Ah. Diff'ing the output between before and after ... -Test skipped as fs doesn't support unwritten extents. +07.01 SEEK_HOLE expected 0 or 45056, got 0. succ +07.02 SEEK_HOLE expected 1 or 45056, got 1. succ +07.03 SEEK_DATA expected 40960 or 40960, got 40960. succ +07.04 SEEK_DATA expected 40960 or 40960, got 40960. succ so, er, the tests didn't report that I'd broken it because it was just an automatically skipped test. Not sure what to do about that; obviously we should skip tests that aren't applicable, but it'd be nice to see a warning that tmpfs used to support this and now doesn't.