From: Jaegeuk Kim <jaegeuk@kernel.org>
To: Philippe De Muyter <phdm@macq.eu>
Cc: linux-f2fs-devel@lists.sourceforge.net
Subject: Re: f2fs_bug_on in init_dirty_segmap
Date: Fri, 10 Mar 2017 10:20:34 -0800 [thread overview]
Message-ID: <20170310182034.GA22979@jaegeuk.local> (raw)
In-Reply-To: <20170310092810.GA28970@frolo.macqel>
On 03/10, Philippe De Muyter wrote:
...
> > > It is actually a Freescale (now NXP) provided kernel tagged rel_imx_4.1.15_1.2.0_ga
> > > available at git://git.freescale.com/imx/linux-2.6-imx.git
> > >
> > > I surmise that the f2fs driver is the one from the linux-4.1 version.
> > >
> > > >
> > > > You can get the latest f2fs-tools from
> > > > http://git.kernel.org/cgit/linux/kernel/git/jaegeuk/f2fs-tools.git/log/?h=dev-test
> > > >
> > > > Anyway, if you don't have time, you may have some options like:
> > > >
> > > > 1. disable F2FS_CHECK_FS in Kconfig
> > > > : bypass BUG_ON here, so mount-ro would be preferable.
> > >
> > > Actually F2FS_CHECK_FS was disabled, but kernel panic'ed.
> >
> > So, you got a warning message from there?
> >
> > #ifdef CONFIG_F2FS_CHECK_FS
> > #define f2fs_bug_on(sbi, condition) BUG_ON(condition)
> > #define f2fs_down_write(x, y) down_write_nest_lock(x, y)
> > #else
> > #define f2fs_bug_on(sbi, condition) \
> > do { \
> > if (unlikely(condition)) { \
> > WARN_ON(1); \
> > set_sbi_flag(sbi, SBI_NEED_FSCK); \
> > } \
> > } while (0)
> > #define f2fs_down_write(x, y) down_write(x)
> > #endif
>
> Yes, I got a 'panic'. I'll look deeper into that and let you know.
Okay, thank you. ;)
> > > > 2. run fsck.f2fs
> > > > : fix some inconsistency and let you mount your system again.
> > > >
> > > > So, I think you'd better try #1 to retrieve your important data, and then
> > > > #2 to fix something wrong.
> > >
> > > There wasn't a need to retrieve important data, as it is an unattended device
> > > in the wild. The problem was that boot did not succeed anymore and the device
> > > needed a return to our aftersales department for reflashing. Without the 'panic',
> > > and with a read-only mount we could probably fix it without reflashing
> > > and without return.
> >
> > So, there is no fscking before mounting any filesystem during boot-up, right?
> > Most of distros and android do that and provide a recovery mode tho.
>
> No, there is no fsck before mounting the root filesystem during boot-up.
> But I do my best to avoid writing in the root partition. It's not read-only,
> but all log, lock, run, cache and tmp files are in tmpfs partitions. I have
> just noticed though that the partition is mounted with 'relatime', not 'noatime'
> as I would like, but I do not know where this 'relatime' option comes from.
> Is 'relatime' a default mount option for f2fs ?
Not specific from f2fs. It's default given by vfs, and you need to mount with
noatime explicitly, if you want. BTW, once you got a panic during boot-up, how
can you change mount option?
Thanks,
>
> >
> > I think there is a way to mount with ro without loading in-memory structures
> > which results in potential panics. Will cosider about that. ;)
> >
> > Note, there is a backported f2fs for v4.1 which I've been maintaining.
> >
> > http://git.kernel.org/cgit/linux/kernel/git/jaegeuk/f2fs-stable.git/log/?h=linux-4.1.y
>
> Thank you. I will apply it
>
> Philippe
>
> --
> Philippe De Muyter +32 2 6101532 Macq SA rue de l'Aeronef 2 B-1140 Bruxelles
------------------------------------------------------------------------------
Announcing the Oxford Dictionaries API! The API offers world-renowned
dictionary content that is easy and intuitive to access. Sign up for an
account today to start using our lexical data to power your apps and
projects. Get started today and enter our developer competition.
http://sdm.link/oxford
next prev parent reply other threads:[~2017-03-10 18:20 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-03-08 14:24 f2fs_bug_on in init_dirty_segmap Philippe De Muyter
2017-03-08 21:52 ` Jaegeuk Kim
2017-03-09 8:42 ` Philippe De Muyter
2017-03-09 23:11 ` Jaegeuk Kim
2017-03-10 9:28 ` Philippe De Muyter
2017-03-10 18:20 ` Jaegeuk Kim [this message]
2017-03-10 18:58 ` Philippe De Muyter
2017-03-11 21:01 ` Jaegeuk Kim
2017-03-12 10:32 ` Philippe De Muyter
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=20170310182034.GA22979@jaegeuk.local \
--to=jaegeuk@kernel.org \
--cc=linux-f2fs-devel@lists.sourceforge.net \
--cc=phdm@macq.eu \
/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).