All of lore.kernel.org
 help / color / mirror / Atom feed
From: Su Yue <l@damenly.su>
To: dsterba@suse.cz
Cc: linux-btrfs@vger.kernel.org
Subject: Re: [PATCH] btrfs-progs: do not BUG_ON if btrfs_add_to_fsid succeeded to write superblock
Date: Fri, 14 May 2021 21:23:12 +0800	[thread overview]
Message-ID: <eee9l9iy.fsf@damenly.su> (raw)
In-Reply-To: <20210514112243.GT7604@suse.cz>


On Fri 14 May 2021 at 19:22, David Sterba <dsterba@suse.cz> wrote:

> On Thu, May 13, 2021 at 08:37:29AM +0800, Su Yue wrote:
>>
>> On Wed 12 May 2021 at 22:01, David Sterba <dsterba@suse.cz> 
>> wrote:
>>
>> > On Tue, May 11, 2021 at 12:25:01PM +0800, Su Yue wrote:
>> >> Commit 8ef9313cf298 ("btrfs-progs: zoned: implement
>> >> log-structured
>> >> superblock") changed to write BTRFS_SUPER_INFO_SIZE bytes to
>> >> device.
>> >> The before num of bytes to be written is sectorsize.
>> >> It causes mkfs.btrfs failed on my 16k pagesize kvm:
>> >
>> > What architecture is that?
>> >
>> The host chip is Apple m1 so it's arm64 but only supporting 16k 
>> and 4k
>> pagesize. Since btrfs subpage work cares 64k pagesize for now, 
>> I
>> usually run xfstests with 16k pagesize and 16k sectorsize. So 
>> far, so
>> good.
>
> Interesting, what's the distro? I haven't found one that would 
> be
> pre-built with 16k pages so I assume it's built from scratch. 
> Among all
>

Right, I initially booted the kvm using Ubuntu kernel built with 
4k
pages then compiled 16k pagesize kernel manully.

--
Su
> the page sizes we've seen so far 4k is almost everywhere, 64k is 
> ppc and
> arm (both native), and sparc has 8k. 16k is a new one, though I 
> don't
> think it would catch something we haven't seen so far it adds a 
> bit to
> the CPU capabilities coverage.
>


  reply	other threads:[~2021-05-14 13:39 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-05-11  4:25 [PATCH] btrfs-progs: do not BUG_ON if btrfs_add_to_fsid succeeded to write superblock Su Yue
2021-05-12 14:01 ` David Sterba
2021-05-13  0:37   ` Su Yue
2021-05-14 11:22     ` David Sterba
2021-05-14 13:23       ` Su Yue [this message]
2021-05-14 13:50         ` Qu Wenruo
2021-05-14 14:28           ` David Sterba

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=eee9l9iy.fsf@damenly.su \
    --to=l@damenly.su \
    --cc=dsterba@suse.cz \
    --cc=linux-btrfs@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.