linux-ext4.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] mke2fs.8: Improve valid block size documentation
@ 2020-10-13 13:38 Jan Kara
  2020-10-15  0:56 ` Andreas Dilger
  0 siblings, 1 reply; 7+ messages in thread
From: Jan Kara @ 2020-10-13 13:38 UTC (permalink / raw)
  To: Ted Tso; +Cc: linux-ext4, Jan Kara

Explain which valid block sizes mke2fs supports in more detail.

Signed-off-by: Jan Kara <jack@suse.cz>
---
 misc/mke2fs.8.in | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/misc/mke2fs.8.in b/misc/mke2fs.8.in
index e6bfc6d6fd2d..0814d216f3a4 100644
--- a/misc/mke2fs.8.in
+++ b/misc/mke2fs.8.in
@@ -207,10 +207,11 @@ manual page for more details.
 .SH OPTIONS
 .TP
 .BI \-b " block-size"
-Specify the size of blocks in bytes.  Valid block-size values are 1024,
-2048 and 4096 bytes per block.  If omitted,
-block-size is heuristically determined by the filesystem size and
-the expected usage of the filesystem (see the
+Specify the size of blocks in bytes.  Valid block-size values are powers of two
+from 1024 up to 65536 (however note that the kernel is able to mount only
+filesystems with block-size smaller or equal to the system page size - 4k on
+x86 systems). If omitted, block-size is heuristically determined by the
+filesystem size and the expected usage of the filesystem (see the
 .B \-T
 option).  If
 .I block-size
-- 
2.16.4


^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH] mke2fs.8: Improve valid block size documentation
  2020-10-13 13:38 [PATCH] mke2fs.8: Improve valid block size documentation Jan Kara
@ 2020-10-15  0:56 ` Andreas Dilger
  2020-10-15  9:27   ` Jan Kara
  0 siblings, 1 reply; 7+ messages in thread
From: Andreas Dilger @ 2020-10-15  0:56 UTC (permalink / raw)
  To: Jan Kara; +Cc: Ted Tso, linux-ext4

[-- Attachment #1: Type: text/plain, Size: 1505 bytes --]

On Oct 13, 2020, at 7:38 AM, Jan Kara <jack@suse.cz> wrote:
> 
> Explain which valid block sizes mke2fs supports in more detail.
> 
> Signed-off-by: Jan Kara <jack@suse.cz>

Should this mention that the default block size is 4096 bytes for most
filesystems?

It might mention e.g. ppc64 or aarch64 can use 64KB page size, but this
is definitely an improvement already.

Reviewed-by: Andreas Dilger <adilger@dilger.ca>

> ---
> misc/mke2fs.8.in | 9 +++++----
> 1 file changed, 5 insertions(+), 4 deletions(-)
> 
> diff --git a/misc/mke2fs.8.in b/misc/mke2fs.8.in
> index e6bfc6d6fd2d..0814d216f3a4 100644
> --- a/misc/mke2fs.8.in
> +++ b/misc/mke2fs.8.in
> @@ -207,10 +207,11 @@ manual page for more details.
> .SH OPTIONS
> .TP
> .BI \-b " block-size"
> -Specify the size of blocks in bytes.  Valid block-size values are 1024,
> -2048 and 4096 bytes per block.  If omitted,
> -block-size is heuristically determined by the filesystem size and
> -the expected usage of the filesystem (see the
> +Specify the size of blocks in bytes.  Valid block-size values are powers of two
> +from 1024 up to 65536 (however note that the kernel is able to mount only
> +filesystems with block-size smaller or equal to the system page size - 4k on
> +x86 systems). If omitted, block-size is heuristically determined by the
> +filesystem size and the expected usage of the filesystem (see the
> .B \-T
> option).  If
> .I block-size
> --
> 2.16.4
> 


Cheers, Andreas






[-- Attachment #2: Message signed with OpenPGP --]
[-- Type: application/pgp-signature, Size: 873 bytes --]

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] mke2fs.8: Improve valid block size documentation
  2020-10-15  0:56 ` Andreas Dilger
@ 2020-10-15  9:27   ` Jan Kara
  2020-12-15 18:50     ` Theodore Y. Ts'o
  0 siblings, 1 reply; 7+ messages in thread
From: Jan Kara @ 2020-10-15  9:27 UTC (permalink / raw)
  To: Andreas Dilger; +Cc: Jan Kara, Ted Tso, linux-ext4

On Wed 14-10-20 18:56:13, Andreas Dilger wrote:
> On Oct 13, 2020, at 7:38 AM, Jan Kara <jack@suse.cz> wrote:
> > 
> > Explain which valid block sizes mke2fs supports in more detail.
> > 
> > Signed-off-by: Jan Kara <jack@suse.cz>
> 
> Should this mention that the default block size is 4096 bytes for most
> filesystems?

That would be to the "heuristic" parts. Yes, I agree, I'll add that.

> It might mention e.g. ppc64 or aarch64 can use 64KB page size, but this
> is definitely an improvement already.

Yeah, I can add that to the "page size" part. But with these archs there's
a catch that page size can be configured in the kernel config so the
formulation will need to be a bit more careful. But I'll add something.

> Reviewed-by: Andreas Dilger <adilger@dilger.ca>

Thanks! I'll send v2.

								Honza
> 
> > ---
> > misc/mke2fs.8.in | 9 +++++----
> > 1 file changed, 5 insertions(+), 4 deletions(-)
> > 
> > diff --git a/misc/mke2fs.8.in b/misc/mke2fs.8.in
> > index e6bfc6d6fd2d..0814d216f3a4 100644
> > --- a/misc/mke2fs.8.in
> > +++ b/misc/mke2fs.8.in
> > @@ -207,10 +207,11 @@ manual page for more details.
> > .SH OPTIONS
> > .TP
> > .BI \-b " block-size"
> > -Specify the size of blocks in bytes.  Valid block-size values are 1024,
> > -2048 and 4096 bytes per block.  If omitted,
> > -block-size is heuristically determined by the filesystem size and
> > -the expected usage of the filesystem (see the
> > +Specify the size of blocks in bytes.  Valid block-size values are powers of two
> > +from 1024 up to 65536 (however note that the kernel is able to mount only
> > +filesystems with block-size smaller or equal to the system page size - 4k on
> > +x86 systems). If omitted, block-size is heuristically determined by the
> > +filesystem size and the expected usage of the filesystem (see the
> > .B \-T
> > option).  If
> > .I block-size
> > --
> > 2.16.4
> > 
> 
> 
> Cheers, Andreas
> 
> 
> 
> 
> 


-- 
Jan Kara <jack@suse.com>
SUSE Labs, CR

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] mke2fs.8: Improve valid block size documentation
  2020-10-15  9:27   ` Jan Kara
@ 2020-12-15 18:50     ` Theodore Y. Ts'o
  2020-12-16 10:22       ` Jan Kara
  0 siblings, 1 reply; 7+ messages in thread
From: Theodore Y. Ts'o @ 2020-12-15 18:50 UTC (permalink / raw)
  To: Jan Kara; +Cc: Andreas Dilger, linux-ext4

On Thu, Oct 15, 2020 at 11:27:31AM +0200, Jan Kara wrote:
> On Wed 14-10-20 18:56:13, Andreas Dilger wrote:
> > On Oct 13, 2020, at 7:38 AM, Jan Kara <jack@suse.cz> wrote:
> > > 
> > > Explain which valid block sizes mke2fs supports in more detail.
> > > 
> > > Signed-off-by: Jan Kara <jack@suse.cz>
> > 
> > Should this mention that the default block size is 4096 bytes for most
> > filesystems?
> 
> That would be to the "heuristic" parts. Yes, I agree, I'll add that.
> 
> > It might mention e.g. ppc64 or aarch64 can use 64KB page size, but this
> > is definitely an improvement already.
> 
> Yeah, I can add that to the "page size" part. But with these archs there's
> a catch that page size can be configured in the kernel config so the
> formulation will need to be a bit more careful. But I'll add something.
> 
> > Reviewed-by: Andreas Dilger <adilger@dilger.ca>
> 
> Thanks! I'll send v2.

Did you ever send a v2 of this patch?  I can't seem to find it in my
archives or in patchwork.

Thanks!!

					- Ted

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] mke2fs.8: Improve valid block size documentation
  2020-12-15 18:50     ` Theodore Y. Ts'o
@ 2020-12-16 10:22       ` Jan Kara
  0 siblings, 0 replies; 7+ messages in thread
From: Jan Kara @ 2020-12-16 10:22 UTC (permalink / raw)
  To: Theodore Y. Ts'o; +Cc: Jan Kara, Andreas Dilger, linux-ext4

On Tue 15-12-20 13:50:27, Theodore Y. Ts'o wrote:
> On Thu, Oct 15, 2020 at 11:27:31AM +0200, Jan Kara wrote:
> > On Wed 14-10-20 18:56:13, Andreas Dilger wrote:
> > > On Oct 13, 2020, at 7:38 AM, Jan Kara <jack@suse.cz> wrote:
> > > > 
> > > > Explain which valid block sizes mke2fs supports in more detail.
> > > > 
> > > > Signed-off-by: Jan Kara <jack@suse.cz>
> > > 
> > > Should this mention that the default block size is 4096 bytes for most
> > > filesystems?
> > 
> > That would be to the "heuristic" parts. Yes, I agree, I'll add that.
> > 
> > > It might mention e.g. ppc64 or aarch64 can use 64KB page size, but this
> > > is definitely an improvement already.
> > 
> > Yeah, I can add that to the "page size" part. But with these archs there's
> > a catch that page size can be configured in the kernel config so the
> > formulation will need to be a bit more careful. But I'll add something.
> > 
> > > Reviewed-by: Andreas Dilger <adilger@dilger.ca>
> > 
> > Thanks! I'll send v2.
> 
> Did you ever send a v2 of this patch?  I can't seem to find it in my
> archives or in patchwork.

I think I did but probably it got lost somewhere. Anyway, I've resent it
now. Thanks for noticing!

								Honza

-- 
Jan Kara <jack@suse.com>
SUSE Labs, CR

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] mke2fs.8: Improve valid block size documentation
  2020-12-16 10:22 Jan Kara
@ 2021-01-21 23:26 ` Theodore Ts'o
  0 siblings, 0 replies; 7+ messages in thread
From: Theodore Ts'o @ 2021-01-21 23:26 UTC (permalink / raw)
  To: Jan Kara; +Cc: linux-ext4

On Wed, Dec 16, 2020 at 11:22:06AM +0100, Jan Kara wrote:
> Explain which valid block sizes mke2fs supports in more detail.
> 
> Reviewed-by: Andreas Dilger <adilger@dilger.ca>
> Signed-off-by: Jan Kara <jack@suse.cz>

Applied, thanks!

					- Ted

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [PATCH] mke2fs.8: Improve valid block size documentation
@ 2020-12-16 10:22 Jan Kara
  2021-01-21 23:26 ` Theodore Ts'o
  0 siblings, 1 reply; 7+ messages in thread
From: Jan Kara @ 2020-12-16 10:22 UTC (permalink / raw)
  To: Ted Tso; +Cc: linux-ext4, Jan Kara

Explain which valid block sizes mke2fs supports in more detail.

Reviewed-by: Andreas Dilger <adilger@dilger.ca>
Signed-off-by: Jan Kara <jack@suse.cz>
---
 misc/mke2fs.8.in | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/misc/mke2fs.8.in b/misc/mke2fs.8.in
index e6bfc6d6fd2d..171315b142bc 100644
--- a/misc/mke2fs.8.in
+++ b/misc/mke2fs.8.in
@@ -207,12 +207,14 @@ manual page for more details.
 .SH OPTIONS
 .TP
 .BI \-b " block-size"
-Specify the size of blocks in bytes.  Valid block-size values are 1024,
-2048 and 4096 bytes per block.  If omitted,
-block-size is heuristically determined by the filesystem size and
+Specify the size of blocks in bytes.  Valid block-size values are powers of two
+from 1024 up to 65536 (however note that the kernel is able to mount only
+filesystems with block-size smaller or equal to the system page size - 4k on
+x86 systems, upto 64k on ppc64 or aarch64 depending on kernel configuration).
+If omitted, block-size is heuristically determined by the filesystem size and
 the expected usage of the filesystem (see the
 .B \-T
-option).  If
+option).  In most common cases, the default block size is 4k. If
 .I block-size
 is preceded by a negative sign ('-'), then
 .B mke2fs
-- 
2.16.4


^ permalink raw reply related	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2021-01-21 23:27 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-10-13 13:38 [PATCH] mke2fs.8: Improve valid block size documentation Jan Kara
2020-10-15  0:56 ` Andreas Dilger
2020-10-15  9:27   ` Jan Kara
2020-12-15 18:50     ` Theodore Y. Ts'o
2020-12-16 10:22       ` Jan Kara
2020-12-16 10:22 Jan Kara
2021-01-21 23:26 ` Theodore Ts'o

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).