From: Jason Cooper <jason@lakedaemon.net>
To: "H. Peter Anvin" <hpa@zytor.com>
Cc: "Theodore Ts'o" <tytso@mit.edu>,
Andy Lutomirski <luto@amacapital.net>,
Florian Weimer <fweimer@redhat.com>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
Kees Cook <kees@outflux.net>
Subject: Re: [PATCH] random: Add "initialized" variable to proc
Date: Thu, 1 May 2014 10:33:11 -0400 [thread overview]
Message-ID: <20140501143311.GR28159@titan.lakedaemon.net> (raw)
In-Reply-To: <FDF79354-1CD5-4DFF-AC5A-32D8C00CFCB7@zytor.com>
On Wed, Apr 30, 2014 at 10:37:00PM -0700, H. Peter Anvin wrote:
>
> > On Apr 30, 2014, at 19:06, "Theodore Ts'o" <tytso@mit.edu> wrote:
> >
> >> On Wed, Apr 30, 2014 at 01:52:35PM -0700, Andy Lutomirski wrote:
> >>
> >> 1. It simply doesn't work on my system. In particular, it never returns
> >> entropy. It just blocks forever.
> >
> > Why? Is this a bug in qemu? The host OS? The guest OS? It is qemu
> > trying to use /dev/random instead of /dev/urandom? Any thing else?
> >
> >> 3. There should be a way to provide some entropy-free cryptographically
> >> secure data, too. Regardless of the speed of the hosts's /dev/random,
> >> the guest should start with at least 256 bits of cryptographically
> >> secure seed material IMO.
> >
> > Well, the simplest way to do this is to pass it in via the command
> > line, and then have the the kernel make sure it gets obscured so it's
> > not exposed via /proc/cmdline.
> >
> > Otherwise we would have to define an extension where we pass 32 bytes
> > or so after the boot command line. But the downside of doing that is
> > we would have to modify every single architecture to define where
> > those 32 bytes could be found.
> >
> > Aside from passing it on the command line as being a bit grotty, the
> > other big problem this is that some architectures only have 256 bytes
> > of command line, and if we use a base 64 encoding, 256 bits will take
> > 43 characters. Not a problem on x86, and it seems rather unlikely
> > that people would want to virtualize a m68k or avr32 CPU. It just
> > feels really unclean.
> >
> > I've cc'ed Peter Anvin for his opinion about extending Linux boot
> > parameter protocol. I agree it would be a lot simpler and easier to
> > enable things like Kernel ASLR with real randomness on guest OS's if
> > we didn't have to erect the whole virtio-pci infrastructure during
> > early boot. :-)
>
> To do something cross-arch putting it in memory and having something
> point to it is probably easiest, but again, with an in-VM boot loader
> the command line rather sucks. This then becomes a matter for device
> tree/ACPI with all that entails.
I drafted an idea for how to improve early-boot randomness on ARM [1] a
ways back. Requoting:
"""
On Wednesday 12 February 2014 13:45:21 Jason Cooper wrote:
> On Wed, Feb 12, 2014 at 07:17:41PM +0100, Arnd Bergmann wrote:
> > On Wednesday 12 February 2014 12:45:54 Jason Cooper wrote:
> > > I brought this up at last weeks devicetree irc meeting. My goal
> > > is to provide early randomness for kaslr on ARM. Currently, my
> > > idea is modify the init script to save an additional random seed
> > > from /dev/urandom to /boot/random-seed.
> > >
> > > The bootloader would then load this file into ram, and pass the
> > > address/size to the kernel either via dt, or commandline. kaslr
> > > (run in the decompressor) would consume some of this randomness,
> > > and then random.c would consume the rest in a non-crediting
> > > initialization.
> >
> > I like the idea, but wouldn't it be easier to pass actual random
> > data using DT, rather than the address/size?
>
> I thought about that at first, but that requires either that the
> bootloader be upgraded to insert the data, or that userspace is
> modifying the dtb at least twice per boot.
>
> I chose address/size to facilitate modifying existing/fielded devices.
> The user could modify the dtb once, and modify the bootloader
> environment to load X amount to Y address. As a fallback, it could be
> expressed on the commandline for non-DT bootloaders.
Ah, so you are interested in boot loaders that can be scripted to do
what you had in mind but cannot be scripted to add or modify a DT
property. I hadn't considered that, but you are probably right that
this is at least 90% of the systems you'd find in the wild today.
[...]
Arnd
"""
I'm not sure how it would play out on other arch's, or with ACPI. But
if there's interest, I could try to spend some cycles in the next few
weeks to create an RFC.
thx,
Jason.
[1] http://marc.info/?l=linux-kernel&m=139223237824952&w=2
prev parent reply other threads:[~2014-05-01 14:33 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-04-28 19:52 Florian Weimer
2014-04-28 21:41 ` Theodore Ts'o
2014-04-29 17:51 ` Florian Weimer
2014-04-29 18:26 ` Theodore Ts'o
2014-04-30 20:52 ` Andy Lutomirski
2014-05-01 2:06 ` Theodore Ts'o
2014-05-01 4:05 ` H. Peter Anvin
2014-05-01 15:05 ` tytso
2014-05-01 15:35 ` Andy Lutomirski
2014-05-01 18:53 ` Andy Lutomirski
2014-05-01 18:59 ` random: Providing a seed value to VM guests H. Peter Anvin
2014-05-01 5:37 ` [PATCH] random: Add "initialized" variable to proc H. Peter Anvin
2014-05-01 14:33 ` Jason Cooper [this message]
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=20140501143311.GR28159@titan.lakedaemon.net \
--to=jason@lakedaemon.net \
--cc=fweimer@redhat.com \
--cc=hpa@zytor.com \
--cc=kees@outflux.net \
--cc=linux-kernel@vger.kernel.org \
--cc=luto@amacapital.net \
--cc=tytso@mit.edu \
/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
all inboxes | Powered by JetHome®