Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Mon, 17 Apr 2023 05:10:21 +0100
From:   Matthew Wilcox <willy@infradead.org>
To:     kernel test robot <oliver.sang@intel.com>
Cc:     Hannes Reinecke <hare@suse.de>, oe-lkp@lists.linux.dev,
        lkp@intel.com, linux-fsdevel@vger.kernel.org,
        Pankaj Raghav <p.raghav@samsung.com>,
        linux-kernel@vger.kernel.org, mcgrof@kernel.org
Subject: Re: [PATCH] mm/filemap: allocate folios according to the blocksize
Message-ID: <ZDzGrbXBUuITlxwI@casper.infradead.org>
References: <20230414134908.103932-1-hare@suse.de>
 <202304170945.28aa41a0-oliver.sang@intel.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <202304170945.28aa41a0-oliver.sang@intel.com>
Precedence: bulk

On Mon, Apr 17, 2023 at 10:18:37AM +0800, kernel test robot wrote:
> commit: 3f1c33c25c31221a7a27d302ce6aac8e9b71edbb ("[PATCH] mm/filemap: allocate folios according to the blocksize")

> [ 96.797305][ T3511] WARNING: CPU: 3 PID: 3511 at include/linux/pagemap.h:344 split_huge_page_to_list (include/linux/pagemap.h:344 mm/huge_memory.c:2767) 

Oh, funny.  That's:

        WARN_ON_ONCE(mapping_large_folio_support(mapping) == 0);

so it's found a large folio in a mapping which doesn't claim to support
large folios.  This is a good warning; thank you, bot!

> [   96.994758][ T3511] Call Trace:
> [   96.997888][ T3511]  <TASK>
> [ 97.016651][ T3511] truncate_inode_partial_folio (mm/truncate.c:243) 
> [ 97.022376][ T3511] truncate_inode_pages_range (mm/truncate.c:380) 
> [ 97.090958][ T3511] truncate_pagecache (mm/truncate.c:744) 
> [ 97.095638][ T3511] smb3_simple_falloc+0xcbf/0x1840 cifs

Ah, it's coming from smb/cifs.  I bet it's set i_blkbits to something
large.  Looks like we need to do something like I suggested to set the
minimum folio size in the mapping, rather than basing it on
mapping->host->i_blkbits.