Re: [PATCH -v4] ext4: don't BUG if kernel subsystems dirty pages without asking ext4 first

To: "Theodore Ts'o" <tytso@xxxxxxx>
Subject: Re: [PATCH -v4] ext4: don't BUG if kernel subsystems dirty pages without asking ext4 first
From: Christoph Hellwig <hch@xxxxxx>
Date: Thu, 3 Mar 2022 09:21:25 +0100
Cc: Eric Biggers <ebiggers@xxxxxxxxxx>, Lee Jones <lee.jones@xxxxxxxxxx>, linux-ext4@xxxxxxxxxxxxxxx, Christoph Hellwig <hch@xxxxxx>, Dave Chinner <dchinner@xxxxxxxxxx>, Goldwyn Rodrigues <rgoldwyn@xxxxxxxx>, "Darrick J . Wong" <darrick.wong@xxxxxxxxxx>, Bob Peterson <rpeterso@xxxxxxxxxx>, Damien Le Moal <damien.lemoal@xxxxxxx>, Andreas Gruenbacher <agruenba@xxxxxxxxxx>, Ritesh Harjani <riteshh@xxxxxxxxxxxxx>, Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>, Johannes Thumshirn <jth@xxxxxxxxxx>, linux-xfs@xxxxxxxxxxxxxxx, linux-fsdevel@xxxxxxxxxxxxxxx, cluster-devel@xxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx
In-reply-to: <YiBDf7XLnTe4Gwis@mit.edu>
References: <Yg0m6IjcNmfaSokM@google.com> <Yhks88tO3Em/G370@mit.edu> <YhlBUCi9O30szf6l@sol.localdomain> <YhlFRoJ3OdYMIh44@mit.edu> <YhlIvw00Y4MkAgxX@mit.edu> <YiBDf7XLnTe4Gwis@mit.edu>
User-agent: Mutt/1.5.17 (2007-11-01)

Looks good to me as a short-term bandaid:

Reviewed-by: Christoph Hellwig <hch@xxxxxx>

References:
- [REPORT] kernel BUG at fs/ext4/inode.c:2620 - page_buffers()
  - From: Lee Jones
- [PATCH -v2] ext4: don't BUG if kernel subsystems dirty pages without asking ext4 first
  - From: Theodore Ts'o
- Re: [PATCH -v2] ext4: don't BUG if kernel subsystems dirty pages without asking ext4 first
  - From: Eric Biggers
- Re: [PATCH -v2] ext4: don't BUG if kernel subsystems dirty pages without asking ext4 first
  - From: Theodore Ts'o
- [PATCH -v3] ext4: don't BUG if kernel subsystems dirty pages without asking ext4 first
  - From: Theodore Ts'o
- Re: [PATCH -v4] ext4: don't BUG if kernel subsystems dirty pages without asking ext4 first
  - From: Theodore Ts'o