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;
MIME-Version: 1.0
References: <CABXGCsNzVxo4iq-tJSGm_kO1UggHXgq6CdcHDL=z5FL4njYXSQ@mail.gmail.com>
 <f68699c3-ec5e-d8e8-f101-6e9a7020ac81@gmx.com> <CABXGCsNrm3ddn3p_ECSRe+yQeoF3KojTFvy-CpXNzi9ADkbnvQ@mail.gmail.com>
 <18b5fa1e-7d1e-4560-c98b-d7ac5fc87c3a@gmx.com> <CABXGCsM5y6m=pLX7tyE-RB1yuHEgZh3fxDu5ESfTRxEnW+djZA@mail.gmail.com>
 <0036622d-53f7-eaba-f720-6dd9bdca28ad@gmx.com> <CABXGCsMMYmCT_Hz=wfKWDy4OQJ2KyBO3rNAFC1YCAthziWm+Lg@mail.gmail.com>
 <41734bdb-2df0-6596-01b6-76a7dfd05d64@gmx.com> <CABXGCsM6GOzr+wL9BEk7mD5xLNgsKE5KUBADD8r783V6-FwXdg@mail.gmail.com>
 <e9f4d768-d91b-b5cc-dbe6-041a81833cf3@gmx.com>
In-Reply-To: <e9f4d768-d91b-b5cc-dbe6-041a81833cf3@gmx.com>
From:   Mikhail Gavrilov <mikhail.v.gavrilov@gmail.com>
Date:   Tue, 27 Dec 2022 04:52:57 +0500
Message-ID: <CABXGCsPK6ZZ4=5qathvjqSJNde6MYhA0uvh6zAMznT9Yj3xk2g@mail.gmail.com>
Subject: Re: [6.2][regression] after commit 947a629988f191807d2d22ba63ae18259bb645c5
 btrfs volume periodical forced switch to readonly after a lot of disk writes
To:     Qu Wenruo <quwenruo.btrfs@gmx.com>
Cc:     wqu@suse.com, dsterba@suse.com,
        Btrfs BTRFS <linux-btrfs@vger.kernel.org>,
        Linux List Kernel Mailing <linux-kernel@vger.kernel.org>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

On Tue, Dec 27, 2022 at 4:36 AM Qu Wenruo <quwenruo.btrfs@gmx.com> wrote:
>
>
> Please be aware that, this newer version may cause extra kernel
> warnings, if it found any uninitialized btrfs_tree_parent_check
> structure at submit time.
>
> But there are completely valid cases we can do that, thus it may cause
> much larger dmesg output, and you should not rely on the kernel warning
> to determine if the bug is triggered.
>

The kernel failed to built with v2 patch.

Here list of compilatore error messages:
fs/btrfs/disk-io.c: In function =E2=80=98btrfs_submit_metadata_bio=E2=80=99=
:
fs/btrfs/disk-io.c:5413:2: error: unterminated argument list invoking
macro "WARN_ON"
 5413 | }
      |  ^
  CC      net/ipv4/icmp.o
fs/btrfs/disk-io.c:851:17: error: =E2=80=98WARN_ON=E2=80=99 undeclared (fir=
st use in
this function)
  851 |                 WARN_ON(!memcmp(&check, &bbio->parent_check,
      |                 ^~~~~~~
fs/btrfs/disk-io.c:851:17: note: each undeclared identifier is
reported only once for each function it appears in
fs/btrfs/disk-io.c:851:24: error: expected =E2=80=98;=E2=80=99 at end of in=
put
  851 |                 WARN_ON(!memcmp(&check, &bbio->parent_check,
      |                        ^
      |                        ;
......
fs/btrfs/disk-io.c:851:17: error: expected declaration or statement at
end of input
  851 |                 WARN_ON(!memcmp(&check, &bbio->parent_check,
      |                 ^~~~~~~
fs/btrfs/disk-io.c:851:17: error: expected declaration or statement at
end of input
fs/btrfs/disk-io.c:845:22: warning: unused variable =E2=80=98ret=E2=80=99 [=
-Wunused-variable]
  845 |         blk_status_t ret;
      |                      ^~~
fs/btrfs/disk-io.c:844:40: warning: unused variable =E2=80=98check=E2=80=99=
 [-Wunused-variable]
  844 |         struct btrfs_tree_parent_check check =3D {0};
      |                                        ^~~~~
fs/btrfs/disk-io.c:842:31: warning: unused variable =E2=80=98fs_info=E2=80=
=99
[-Wunused-variable]
  842 |         struct btrfs_fs_info *fs_info =3D inode->root->fs_info;
      |                               ^~~~~~~
  HDRTEST usr/include/linux/loadpin.h
fs/btrfs/disk-io.c: At top level:
fs/btrfs/disk-io.c:63:13: warning: =E2=80=98btrfs_destroy_ordered_extents=
=E2=80=99
declared =E2=80=98static=E2=80=99 but never defined [-Wunused-function]
   63 | static void btrfs_destroy_ordered_extents(struct btrfs_root *root);
      |             ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
fs/btrfs/disk-io.c:64:12: warning: =E2=80=98btrfs_destroy_delayed_refs=E2=
=80=99
declared =E2=80=98static=E2=80=99 but never defined [-Wunused-function]
   64 | static int btrfs_destroy_delayed_refs(struct btrfs_transaction *tra=
ns,
      |            ^~~~~~~~~~~~~~~~~~~~~~~~~~
fs/btrfs/disk-io.c:66:13: warning: =E2=80=98btrfs_destroy_delalloc_inodes=
=E2=80=99
declared =E2=80=98static=E2=80=99 but never defined [-Wunused-function]
   66 | static void btrfs_destroy_delalloc_inodes(struct btrfs_root *root);
      |             ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
fs/btrfs/disk-io.c:67:12: warning: =E2=80=98btrfs_destroy_marked_extents=E2=
=80=99
declared =E2=80=98static=E2=80=99 but never defined [-Wunused-function]
   67 | static int btrfs_destroy_marked_extents(struct btrfs_fs_info *fs_in=
fo,
      |            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
fs/btrfs/disk-io.c:70:12: warning: =E2=80=98btrfs_destroy_pinned_extent=E2=
=80=99
declared =E2=80=98static=E2=80=99 but never defined [-Wunused-function]
   70 | static int btrfs_destroy_pinned_extent(struct btrfs_fs_info *fs_inf=
o,
      |            ^~~~~~~~~~~~~~~~~~~~~~~~~~~
fs/btrfs/disk-io.c:72:12: warning: =E2=80=98btrfs_cleanup_transaction=E2=80=
=99
declared =E2=80=98static=E2=80=99 but never defined [-Wunused-function]
   72 | static int btrfs_cleanup_transaction(struct btrfs_fs_info *fs_info)=
;
      |            ^~~~~~~~~~~~~~~~~~~~~~~~~
fs/btrfs/disk-io.c:73:13: warning: =E2=80=98btrfs_error_commit_super=E2=80=
=99 declared
=E2=80=98static=E2=80=99 but never defined [-Wunused-function]
   73 | static void btrfs_error_commit_super(struct btrfs_fs_info *fs_info)=
;
      |             ^~~~~~~~~~~~~~~~~~~~~~~~
fs/btrfs/disk-io.c:828:13: warning: =E2=80=98should_async_write=E2=80=99 de=
fined but
not used [-Wunused-function]
  828 | static bool should_async_write(struct btrfs_fs_info *fs_info,
      |             ^~~~~~~~~~~~~~~~~~
fs/btrfs/disk-io.c:161:13: warning: =E2=80=98btrfs_supported_super_csum=E2=
=80=99
defined but not used [-Wunused-function]
  161 | static bool btrfs_supported_super_csum(u16 csum_type)
      |             ^~~~~~~~~~~~~~~~~~~~~~~~~~
fs/btrfs/disk-io.c:130:12: warning: =E2=80=98verify_parent_transid=E2=80=99=
 defined
but not used [-Wunused-function]
  130 | static int verify_parent_transid(struct extent_io_tree *io_tree,
      |            ^~~~~~~~~~~~~~~~~~~~~
fs/btrfs/disk-io.c:75:13: warning: =E2=80=98btrfs_free_csum_hash=E2=80=99 d=
efined but
not used [-Wunused-function]
   75 | static void btrfs_free_csum_hash(struct btrfs_fs_info *fs_info)
      |             ^~~~~~~~~~~~~~~~~~~~
  CC      kernel/notifier.o
make[3]: *** [scripts/Makefile.build:252: fs/btrfs/disk-io.o] Error 1
make[3]: *** Waiting for unfinished jobs....


--=20
Best Regards,
Mike Gavrilov.