Linux-mtd Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Richard Weinberger <richard@nod.at>
To: Alexander Dahl <ada@thorsis.com>
Cc: linux-mtd <linux-mtd@lists.infradead.org>,
	david oberhollenzer <david.oberhollenzer@sigma-star.at>
Subject: Re: [PATCH] ubi-utils: Implement a ubihealthd
Date: Thu, 26 Sep 2019 13:44:14 +0200 (CEST)	[thread overview]
Message-ID: <242144136.6776.1569498254158.JavaMail.zimbra@nod.at> (raw)
In-Reply-To: <1846895.TZtMPCjSJF@ada>

Alex,

----- Ursprüngliche Mail -----
> Von: "Alexander Dahl" <ada@thorsis.com>
> What is it exactly, which requires such a new Linux Kernel? The latest LTS is
> still v4.19 … ;-)

The new UBI machinery which allows userspace to control scrubbing was added in 5.2.

[..]

>> +	srand(seed);
>> +}
> 
> Is it really necessary to seed with getrandom() for just randomizing a list of
> PEBs? I know this is embedded, and seeding with the same time on each boot
> (those devices always start 1970 ;-) ) might be a bad idea for crypto, but for
> just shuffling a PEB list to spread those tests randomly?

Yes. The list really needs to be shuffled.

Before getrandom() we used time() and faced exactly the case where customer boards
always started with the same permutation because they had no RTC.

If you don't shuffle the list you cannot have the daemon stateless.

For example, if the board reboots once a day and completing the PEB list
needs a week you will always check the same PEBs and miss the same ones.
 
> I'd really like to test this daemon, but also for older systems not running
> bleeding edge BSPs. Would you accept patches to make it work with let's say
> kernel v4.9 and glibc v2.20 … O:-)

Well, you need to backport the UBI changes anyway.
But patches to make it work on other/older libcs are happily accepted
if they are sane. :-)

Thanks,
//richard

______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/

  reply	other threads:[~2019-09-26 11:44 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-07-25 20:34 [PATCH] ubi-utils: Implement a ubihealthd Richard Weinberger
2019-08-25 11:58 ` David Oberhollenzer
2019-09-26  8:30 ` Alexander Dahl
2019-09-26 11:44   ` Richard Weinberger [this message]
2019-09-26 14:00     ` Richard Weinberger

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=242144136.6776.1569498254158.JavaMail.zimbra@nod.at \
    --to=richard@nod.at \
    --cc=ada@thorsis.com \
    --cc=david.oberhollenzer@sigma-star.at \
    --cc=linux-mtd@lists.infradead.org \
    /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