[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAO9xwp37T_pDohXYOpHhb-KhDYUBEMR0qDN0NJvCLRUoG3CK2Q@mail.gmail.com>
Date: Thu, 19 Nov 2020 10:12:52 -0300
From: Mauricio Oliveira <mauricio.oliveira@...onical.com>
To: yangerkun <yangerkun@...wei.com>
Cc: "Theodore Y . Ts'o" <tytso@....edu>, adilger.kernel@...ger.ca,
Jan Kara <jack@...e.cz>, linux-ext4@...r.kernel.org,
"zhangyi (F)" <yi.zhang@...wei.com>, Hou Tao <houtao1@...wei.com>,
zhangxiaoxu5@...wei.com, Ye Bin <yebin10@...wei.com>,
hejie3@...wei.com
Subject: Re: [Bug report] journal data mode trigger panic in jbd2_journal_commit_transaction
On Thu, Nov 19, 2020 at 1:25 AM yangerkun <yangerkun@...wei.com> wrote:
>
>
>
> 在 2020/11/16 21:50, Mauricio Oliveira 写道:
> > Hi Kun,
> >
> > On Sat, Nov 14, 2020 at 5:18 AM yangerkun <yangerkun@...wei.com> wrote:
> >> While using ext4 with data=journal(3.10 kernel), we meet a problem that
> >> we think may never happend...
> > [...]
> >
> > Could you please confirm you mean 5.10-rc* kernel instead of 3.10?
> > (It seems so as you mention a recent commit below.) Thanks!
> >
> >> For now, what I have seen that can dirty buffer directly is
> >> ext4_page_mkwrite(64a9f1449950 ("ext4: data=journal: fixes for
> >> ext4_page_mkwrite()")), and runing ext4_punch_hole with keep_size
> >> /ext4_page_mkwrite parallel can trigger above warning easily.
> > [...]
> >
> >
>
> Hi,
>
> Sorry for the long delay reply... And thanks a lot for your advise! The
> bug trigger with a very low probability. So won't trigger with 5.10 can
> not prove no bug exist in 5.10.
>
No worries, and thanks for following up.
So I understand that the bug report was indeed on 3.10, and 5.10-rcN
is not yet confirmed.
> Google a lot and notice that someone before has report the same bug[1].
> '3b136499e906 ("ext4: fix data corruption in data=journal mode")' seems
> fix the problem. I will try to understand this, and give a analysis
> about how to reproduce it!
Cool, thanks!
> Thanks,
> Kun.
--
Mauricio Faria de Oliveira
Powered by blists - more mailing lists