From: Richard Weinberger <richard@nod.at>
To: Ricard Wanderlof <ricard.wanderlof@axis.com>
Cc: Atlant Schmidt <aschmidt@dekaresearch.com>,
"linux-mtd@lists.infradead.org" <linux-mtd@lists.infradead.org>,
Artem Bityutskiy <dedekind1@gmail.com>
Subject: Re: Does UBI LEB-level access interlock happily with UBIfs access?
Date: Mon, 22 Sep 2014 10:42:40 +0200 [thread overview]
Message-ID: <541FE100.5040802@nod.at> (raw)
In-Reply-To: <alpine.DEB.2.02.1409221027560.15431@lnxricardw1.se.axis.com>
Am 22.09.2014 10:34, schrieb Ricard Wanderlof:
>
> On Sat, 20 Sep 2014, Richard Weinberger wrote:
>
>> On Fri, Sep 19, 2014 at 7:24 PM, Artem Bityutskiy <dedekind1@gmail.com> wrote:
>>> On Fri, 2014-09-19 at 13:17 -0400, Atlant Schmidt wrote:
>>>> But as I pointed, this will not force re-read of the volume table LEBs.
>>>> To address this, you'd need to do some additional, not very difficult
>>>> work.
>>>
>>> Oh, and this won't make UBI re-read the EC and VID headers, so they may
>>> bit-rot too.
>>>
>>> So indeed it sounds like UBI needs a separate interface for this kind of
>>> "scrub all bit-flips" issues. I do not think it is hard to do - all the
>>> mechanisms are already implemented, so this would mostly be about
>>> inventing good API.
>>
>> We could implement a trivial knob to trigger such a check in kernel.
>> I.e. you trigger the check via an ioctl() or whatever and UBI schedules
>> such a read-check for every PEB into the UBI background thread.
>
> During the scanning operation that takes place when a partition is attached, doesn't this also trigger a check of all headers, as all the data needs to be read as part of the
> ubiattach process?
Only headers will be read.
And with fastmap enabled only very few of these headers are read.
> Not that that would be a practical solution for many systems where it is not practical to detach and re-attach, for instance the partition where the root volume is located, as that
> would in practice require a reboot.
>
>> I'd volunteer to implement this.
>
> I think it would be good if such a forced re-read could be set to happen automatically at a specified interval, say by default once a day.
cron can trigger that easily.
Thanks,
//richard
next prev parent reply other threads:[~2014-09-22 8:43 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-09-16 14:04 Does UBI LEB-level access interlock happily with UBIfs access? Atlant Schmidt
2014-09-19 15:27 ` Artem Bityutskiy
2014-09-19 16:58 ` Atlant Schmidt
2014-09-19 17:13 ` Artem Bityutskiy
2014-09-19 17:17 ` Atlant Schmidt
2014-09-19 17:24 ` Artem Bityutskiy
2014-09-20 12:54 ` Richard Weinberger
2014-09-22 8:34 ` Ricard Wanderlof
2014-09-22 8:42 ` Richard Weinberger [this message]
2014-09-19 17:06 ` Brian Norris
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=541FE100.5040802@nod.at \
--to=richard@nod.at \
--cc=aschmidt@dekaresearch.com \
--cc=dedekind1@gmail.com \
--cc=linux-mtd@lists.infradead.org \
--cc=ricard.wanderlof@axis.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.