From: Richard Weinberger <richard@nod.at>
To: Martin Townsend <mtownsend1973@gmail.com>
Cc: "linux-mtd@lists.infradead.org" <linux-mtd@lists.infradead.org>
Subject: Re: UBIFS assert when rebooting a read only ubifs when it's been remounted r/w
Date: Fri, 20 May 2016 09:51:27 +0200 [thread overview]
Message-ID: <573EC1FF.6070709@nod.at> (raw)
In-Reply-To: <573EC17D.40809@nod.at>
Am 20.05.2016 um 09:49 schrieb Richard Weinberger:
> Am 18.05.2016 um 12:39 schrieb Martin Townsend:
>> I can't do the sync as the error only occurs during boot and I can see
>> the assert whilst systemd is rmounting all the filesystems. Once I've
>> logged in I can't get the assert to trigger again. Just to summarise
>> the conditions for triggering the assert
>> To trigger I login and remount / as rw and then reboot (to a ro
>> rootfs) and the assert first during the boot.
>> The assert will appear on all subsequent boots (Providing I don't do
>> the following)
>> To get rid of the assert I can, once logged in, remount / rw and then
>> ro and then reboot.
>
> Hmm, so there is no reset/power cut involved at all?
> Can you trigger it when you mount a UBIFS by hand ro and remount?
Another thought, can you try this patch (if it is not already applied)?
commit 807612db2f9940b9fa6deaef054eb16d51bd3e00
Author: Andrew Ruder <andrew.ruder@elecsyscorp.com>
Date: Thu Jan 30 09:26:54 2014 -0600
fs/super.c: sync ro remount after blocking writers
Move sync_filesystem() after sb_prepare_remount_readonly(). If writers
sneak in anywhere from sync_filesystem() to sb_prepare_remount_readonly()
it can cause inodes to be dirtied and writeback to occur well after
sys_mount() has completely successfully.
This was spotted by corrupted ubifs filesystems on reboot, but appears
that it can cause issues with any filesystem using writeback.
Cc: Artem Bityutskiy <dedekind1@gmail.com>
Cc: Christoph Hellwig <hch@infradead.org>
Cc: Alexander Viro <viro@zeniv.linux.org.uk>
CC: Richard Weinberger <richard@nod.at>
Co-authored-by: Richard Weinberger <richard@nod.at>
Signed-off-by: Andrew Ruder <andrew.ruder@elecsyscorp.com>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Thanks,
//richard
next prev parent reply other threads:[~2016-05-20 7:51 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-05-17 15:19 UBIFS assert when rebooting a read only ubifs when it's been remounted r/w Martin Townsend
2016-05-17 16:13 ` Richard Weinberger
2016-05-18 9:24 ` Martin Townsend
2016-05-18 9:30 ` Martin Townsend
2016-05-18 10:13 ` Richard Weinberger
2016-05-18 10:39 ` Martin Townsend
2016-05-20 7:49 ` Richard Weinberger
2016-05-20 7:51 ` Richard Weinberger [this message]
2016-05-20 9:47 ` Martin Townsend
2016-05-23 14:16 ` Martin Townsend
2016-05-23 19:57 ` Richard Weinberger
2016-05-24 8:31 ` Martin Townsend
2016-05-24 9:04 ` Richard Weinberger
2016-05-24 10:37 ` Martin Townsend
2016-05-24 12:26 ` Richard Weinberger
2016-05-24 13:23 ` Martin Townsend
2016-05-24 13:42 ` Richard Weinberger
2016-05-24 14:12 ` Martin Townsend
2016-05-24 16:51 ` Richard Weinberger
2016-05-25 8:04 ` Martin Townsend
2016-10-12 13:07 ` Martin Townsend
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=573EC1FF.6070709@nod.at \
--to=richard@nod.at \
--cc=linux-mtd@lists.infradead.org \
--cc=mtownsend1973@gmail.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 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.