From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pl1-f171.google.com (mail-pl1-f171.google.com [209.85.214.171]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id DE5CD37C for ; Tue, 7 May 2024 00:05:39 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.171 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1715040342; cv=none; b=NHYnoHBUcjY5AsDteCMa/7gveA190YHV7CJxYIts13PqZsLfsyOng72HzMALCZmF0Fixdv1pBvkBBZn/Dc4jShQ91rVpgcRmk6Y+YtRXFs/n6UEC6ACnCkQo6mvMtf+wp61Pb4mMHhnUWysGXrQgUoEbmOMgFLGqY1aV67JMCuc= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1715040342; c=relaxed/simple; bh=+r96OpFsdE3OiCouE0MboKQhCrMK+ry6ELfJeJnog1c=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=pBk3Ny9mMqaF+BGTUDeAoJsursi2pUFk8JLgoVpDvK29I1Nop8myr81noL4ekUwlO4X8c40VWdz/4AQ0gQmrGOXy6A6geSewLN23fAtuJ8pN/FXuX8WrC8MU8T93njOG+VkMgw61iFhBb0az1n8fSvhnm3K+skBsZ3oTuvQF0lc= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=fromorbit.com; spf=pass smtp.mailfrom=fromorbit.com; dkim=pass (2048-bit key) header.d=fromorbit-com.20230601.gappssmtp.com header.i=@fromorbit-com.20230601.gappssmtp.com header.b=IUyxVQE7; arc=none smtp.client-ip=209.85.214.171 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=fromorbit.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=fromorbit.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=fromorbit-com.20230601.gappssmtp.com header.i=@fromorbit-com.20230601.gappssmtp.com header.b="IUyxVQE7" Received: by mail-pl1-f171.google.com with SMTP id d9443c01a7336-1ee7963db64so517785ad.1 for ; Mon, 06 May 2024 17:05:39 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=fromorbit-com.20230601.gappssmtp.com; s=20230601; t=1715040339; x=1715645139; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=/mO8RAIpI3ciXCmAN8uZAWkBK2a+XdI/Wl85E3eh0Xg=; b=IUyxVQE7r4mBofKiCzu+Q0FePdp2ilg3FpDTBZak3xlnXkJ01fqpceRZRzYFnWokOX e63nJ+KFJvdDjwEPV+fL0PpXQRIsFrH0Nn9eHNNBQ/55jGVmU3yikpReTmY2B8HD+up/ vkFLxvBw68l2/BL5/pMTYyFdC9WFMkntDvc1a8GHgAUo5NgDiQVnAqzBtL2bkW2twxIH KEm3Hb4i5JOIkYKqoWNLBqSQWRtEYe9HpnF6EPcoPs3aPty3ryTXGoJC7azoUIwtMpY+ P2ZxepmzKX9sFZ/p72kOKOpYuO4u+MwNf//NpHJR/0opNGgtRDGSEVs6Mw4Id4oP3RAN yt4Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1715040339; x=1715645139; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=/mO8RAIpI3ciXCmAN8uZAWkBK2a+XdI/Wl85E3eh0Xg=; b=a+t/c+0Pi4Qosf3JBkEQAOgQZTpBuFX+S8weyW3cRvi5NT7jxl7zzhB0uuSbP6OmQV hDpnZ9kPLok/N0Gsp/2m0m6KYT3Pt3wGXX5U7NwUYpowiAxkguzmdkAx4matXBO7K/Ei QAUcoRr0R/YtfDlGp77dcLM9lavadBNsDrSljtSriEqKqYEHUW3yUlhA38CG7lyPuM4h WX7OnyxMCvspkTB32NJOtKwd53R+DtvvI6PFYX79WOdiGHhFtIapwFKph3VY5jw+Fwpb THuQ2OLMfvydQs+BRvoapnm0W8DrIZxJ8lkg8fu7uly1w+3+LfoRkXdv3UpMDmfGPOgJ 2QnQ== X-Forwarded-Encrypted: i=1; AJvYcCX2SILGDsVyqXCGruiDsX56N8cX1jLFMKEHgNqajqQfucPcmKEzyj9ezWFJetTOsLJehvnHVVMlU0nfH0Ww3A6HETF9OV9Vuqe6/NI= X-Gm-Message-State: AOJu0Yz5XeBZYU+WuqhOyuIoBJIbquScnPboVr33l9O7qpeAgKgH+eVy 82itdnuzC1kCrwe3CbejT4Dla/CKGvSZIsSTUuHmmaOk8FHsa3FvcuH6PX3Pn/Q= X-Google-Smtp-Source: AGHT+IFMS+uzDdaAPRHwq/U9jqHYjfdEca1j76tOxwxbkz7wqlqDtFC+MYf5HxlMEVl3x/q1hzXhRw== X-Received: by 2002:a17:903:2306:b0:1eb:1af8:309f with SMTP id d6-20020a170903230600b001eb1af8309fmr18304829plh.4.1715040338808; Mon, 06 May 2024 17:05:38 -0700 (PDT) Received: from dread.disaster.area (pa49-179-32-121.pa.nsw.optusnet.com.au. [49.179.32.121]) by smtp.gmail.com with ESMTPSA id e2-20020a170902f1c200b001e088a9e2bcsm8829552plc.292.2024.05.06.17.05.38 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 06 May 2024 17:05:38 -0700 (PDT) Received: from dave by dread.disaster.area with local (Exim 4.96) (envelope-from ) id 1s48Kt-006745-2E; Tue, 07 May 2024 10:05:35 +1000 Date: Tue, 7 May 2024 10:05:35 +1000 From: Dave Chinner To: Luis Chamberlain Cc: akpm@linux-foundation.org, willy@infradead.org, djwong@kernel.org, brauner@kernel.org, chandan.babu@oracle.com, hare@suse.de, ritesh.list@gmail.com, john.g.garry@oracle.com, ziy@nvidia.com, linux-fsdevel@vger.kernel.org, linux-xfs@vger.kernel.org, linux-mm@kvack.org, linux-block@vger.kernel.org, gost.dev@samsung.com, p.raghav@samsung.com, kernel@pankajraghav.com Subject: Re: [PATCH v5 11/11] xfs: enable block size larger than page size support Message-ID: References: <20240503095353.3798063-1-mcgrof@kernel.org> <20240503095353.3798063-12-mcgrof@kernel.org> Precedence: bulk X-Mailing-List: linux-block@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20240503095353.3798063-12-mcgrof@kernel.org> On Fri, May 03, 2024 at 02:53:53AM -0700, Luis Chamberlain wrote: > From: Pankaj Raghav > > Page cache now has the ability to have a minimum order when allocating > a folio which is a prerequisite to add support for block size > page > size. > > Signed-off-by: Pankaj Raghav > Reviewed-by: Darrick J. Wong > Signed-off-by: Luis Chamberlain ..... > diff --git a/fs/xfs/xfs_super.c b/fs/xfs/xfs_super.c > index bce020374c5e..db3b82c2c381 100644 > --- a/fs/xfs/xfs_super.c > +++ b/fs/xfs/xfs_super.c > @@ -1623,16 +1623,10 @@ xfs_fs_fill_super( > goto out_free_sb; > } > > - /* > - * Until this is fixed only page-sized or smaller data blocks work. > - */ > if (mp->m_sb.sb_blocksize > PAGE_SIZE) { > xfs_warn(mp, > - "File system with blocksize %d bytes. " > - "Only pagesize (%ld) or less will currently work.", > - mp->m_sb.sb_blocksize, PAGE_SIZE); > - error = -ENOSYS; > - goto out_free_sb; > +"EXPERIMENTAL: Filesystem with Large Block Size (%d bytes) enabled.", > + mp->m_sb.sb_blocksize); > } We really don't want to have to test and support this on V4 filesystems as tehy are deprecated, so can you make this conditional on being a V5 filesystem? i.e: if (mp->m_sb.sb_blocksize > PAGE_SIZE) { if (!xfs_has_crc(mp)) { xfs_warn(mp, "V4 File system with blocksize %d bytes. Only pagesize (%ld) is supported.", mp->m_sb.sb_blocksize, PAGE_SIZE); error = -ENOSYS; goto out_free_sb; } xfs_warn(mp, "EXPERIMENTAL: V5 Filesystem with Large Block Size (%d bytes) enabled.", mp->m_sb.sb_blocksize); } -Dave. -- Dave Chinner david@fromorbit.com