From: Rickard x Andersson <rickaran@axis.com>
To: Richard Weinberger <richard@nod.at>
Cc: Rickard X Andersson <rickard.andersson@axis.com>,
chengzhihao1 <chengzhihao1@huawei.com>,
linux-mtd <linux-mtd@lists.infradead.org>,
rickard314 andersson <rickard314.andersson@gmail.com>,
kernel <kernel@axis.com>
Subject: Re: [PATCH v3 02/10] ubi: Expose mean erase counter for fastmap in sysfs
Date: Thu, 7 Nov 2024 13:12:35 +0100 [thread overview]
Message-ID: <33dc0246-b308-8b4c-addb-7f6791821461@axis.com> (raw)
In-Reply-To: <1446825408.208073.1730979115998.JavaMail.zimbra@nod.at>
On 11/7/24 12:31, Richard Weinberger wrote:
> ----- Ursprüngliche Mail -----
>> Von: "Rickard x Andersson" <rickaran@axis.com>
>> An: "richard" <richard@nod.at>, "Rickard X Andersson" <rickard.andersson@axis.com>
>> One of my devices have the following status:
>>
>> /sys/class/ubi/ubi1 # cat max_ec_data
>> 4672
>> /sys/class/ubi/ubi1 # cat mean_ec_fastmap
>> 8869
>>
>> If you combine the mean EC values you will not be able to tell that the
>> fastmap area is much more worn down. For example the wear of your
>> fastmap area could be on the verge of breaking the flash but that will
>> not be seen on a mean value that includes both the fastmap and data area.
>>
>> If fastmap is not enabled on your system then mean_ec_fastmap will not
>> be visible in sysfs.
>
> But does the user application care?
> Let me ask differently, what is the purpose of exposing these numbers?
> If you just want to know how much your flash is aged, combined max/mean
> ec counters will do it.
>
> For debugging fastmap specific issues, the detailed info via debugfs will do it.
>
> I'm totally fine with exposing more counters, we just need to be clear
> about their meaning and use case. Especially in sysfs we must not make a mess.
>
> Thanks,
> //richard
It is for diagnostics. For example, if you have 100 devices running you
might want to know if some of them are about to be worn down. If you
know that you can take action before they actually break down. One
action could be to for example replace the device in that case.
Since the devices are operate in the "field" it is recommended avoid
using debugfs due to security concerns.
A device might break down due to heavy wear of just the fastmap area.
Thanks,
Rickard A.
______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/
next prev parent reply other threads:[~2024-11-07 12:21 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-10-11 12:58 [PATCH v3 01/10] ubi: Expose mean erase counter in sysfs Rickard Andersson
2024-10-11 12:58 ` [PATCH v3 02/10] ubi: Expose mean erase counter for fastmap " Rickard Andersson
2024-11-06 20:20 ` Richard Weinberger
2024-11-07 10:28 ` Rickard x Andersson
2024-11-07 11:31 ` Richard Weinberger
2024-11-07 12:12 ` Rickard x Andersson [this message]
2024-10-11 12:58 ` [PATCH v3 03/10] ubi: Expose max " Rickard Andersson
2024-10-11 12:58 ` [PATCH v3 04/10] ubi: Expose mean erase counter for data " Rickard Andersson
2024-10-11 12:58 ` [PATCH v3 05/10] ubi: Expose max " Rickard Andersson
2024-10-11 12:59 ` [PATCH v3 06/10] ubi: Add mean erase counter sysfs attribute Rickard Andersson
2024-10-11 12:59 ` [PATCH v3 07/10] ubi: Add mean erase counter sysfs attribute for fastmap area Rickard Andersson
2024-10-11 12:59 ` [PATCH v3 08/10] ubi: Add max " Rickard Andersson
2024-10-11 12:59 ` [PATCH v3 09/10] ubi: Add mean erase counter sysfs attribute for data area Rickard Andersson
2024-10-11 12:59 ` [PATCH v3 10/10] ubi: Add max " Rickard Andersson
2024-11-06 20:34 ` [PATCH v3 01/10] ubi: Expose mean erase counter in sysfs Richard Weinberger
2024-11-07 1:39 ` Zhihao Cheng
2024-11-14 18:50 ` Richard Weinberger
2024-11-15 9:00 ` Rickard x Andersson
2024-11-16 2:55 ` Zhihao Cheng
2024-11-18 17:00 ` Rickard X Andersson
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=33dc0246-b308-8b4c-addb-7f6791821461@axis.com \
--to=rickaran@axis.com \
--cc=chengzhihao1@huawei.com \
--cc=kernel@axis.com \
--cc=linux-mtd@lists.infradead.org \
--cc=richard@nod.at \
--cc=rickard.andersson@axis.com \
--cc=rickard314.andersson@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox