From: Benjamin Marzinski <bmarzins@redhat.com>
To: cluster-devel.redhat.com
Subject: [Cluster-devel] [PATCH 0/2] fix gfs2 truncate race
Date: Tue, 14 Jun 2016 17:02:47 -0500 [thread overview]
Message-ID: <1465941769-25596-1-git-send-email-bmarzins@redhat.com> (raw)
If gfs2 tries to write out a page of a file with data journaling
enabled, while that file is being truncated, it can cause a kernel
panic. The problem is that the page it is writing out may point to past
the end of the file. If this happens, gfs2 will try to invalidate the
page. However, it can't invalidate a journaled page with buffers in the
in-memory log, without revoking those buffers, so that there is no
chance of corruption if the machine crashes and later has its journal
replayed. If the page is being written out by the log flush code, there
is no way that it can add a revoke entry onto the log during the flush.
To solve this, gfs2 simply writes out journalled data pages that point
past the end of the file, since the truncate is still in progress, and
everything will be cleaned up before the file is unlocked, or the blocks
are marked free. Doing this involves both a gfs2 change and exporting an
additional symbol from the vfs.
Benjamin Marzinski (2):
fs: export __block_write_full_page
gfs2: writeout truncated pages
fs/buffer.c | 3 ++-
fs/gfs2/aops.c | 37 +++++++++++++++++++++++++++----------
include/linux/buffer_head.h | 3 +++
3 files changed, 32 insertions(+), 11 deletions(-)
--
1.8.3.1
next reply other threads:[~2016-06-14 22:02 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-06-14 22:02 Benjamin Marzinski [this message]
2016-06-14 22:02 ` [Cluster-devel] [PATCH 1/2] fs: export __block_write_full_page Benjamin Marzinski
2016-06-14 22:02 ` [Cluster-devel] [PATCH 2/2] gfs2: writeout truncated pages Benjamin Marzinski
2016-06-15 17:08 ` Bob Peterson
2016-06-15 18:45 ` Benjamin Marzinski
2016-06-16 15:56 ` Steven Whitehouse
2016-06-16 17:49 ` Benjamin Marzinski
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=1465941769-25596-1-git-send-email-bmarzins@redhat.com \
--to=bmarzins@redhat.com \
/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 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).