Re: [Qemu-devel] [RFC] Virtio RNG: Consider changing the default entropy source to /dev/urandom?

2019-05-07 Thread Daniel P . Berrangé
On Tue, May 07, 2019 at 10:14:25AM -0700, Richard Henderson wrote: > On 5/7/19 8:22 AM, Daniel P. Berrangé wrote: > > On Tue, May 07, 2019 at 11:59:05AM +0200, Nikos Mavrogiannopoulos wrote: > >> In terms of RHEL what is preferred is (1) use a crypto lib, and (2) if > >> that's not possible use

Re: [Qemu-devel] [RFC] Virtio RNG: Consider changing the default entropy source to /dev/urandom?

2019-05-07 Thread Richard Henderson
On 5/7/19 8:22 AM, Daniel P. Berrangé wrote: > On Tue, May 07, 2019 at 11:59:05AM +0200, Nikos Mavrogiannopoulos wrote: >> In terms of RHEL what is preferred is (1) use a crypto lib, and (2) if >> that's not possible use getrandom(). That is summarized in this >> article: >> >>

Re: [Qemu-devel] [RFC] Virtio RNG: Consider changing the default entropy source to /dev/urandom?

2019-05-07 Thread Eric Blake
On 5/7/19 10:22 AM, Daniel P. Berrangé wrote: > On Tue, May 07, 2019 at 11:59:05AM +0200, Nikos Mavrogiannopoulos wrote: >> In terms of RHEL what is preferred is (1) use a crypto lib, and (2) if >> that's not possible use getrandom(). That is summarized in this >> article: >> >>

Re: [Qemu-devel] [RFC] Virtio RNG: Consider changing the default entropy source to /dev/urandom?

2019-05-07 Thread Daniel P . Berrangé
On Tue, May 07, 2019 at 11:59:05AM +0200, Nikos Mavrogiannopoulos wrote: > In terms of RHEL what is preferred is (1) use a crypto lib, and (2) if > that's not possible use getrandom(). That is summarized in this > article: > >

Re: [Qemu-devel] [RFC] Virtio RNG: Consider changing the default entropy source to /dev/urandom?

2019-05-07 Thread Nikos Mavrogiannopoulos
In terms of RHEL what is preferred is (1) use a crypto lib, and (2) if that's not possible use getrandom(). That is summarized in this article: https://www.redhat.com/en/blog/understanding-red-hat-enterprise-linux-random-number-generator-interface On Thu, May 2, 2019 at 8:02 PM Kashyap Chamarthy

Re: [Qemu-devel] [RFC] Virtio RNG: Consider changing the default entropy source to /dev/urandom?

2019-05-03 Thread Richard W.M. Jones
On Thu, May 02, 2019 at 08:02:01PM +0200, Kashyap Chamarthy wrote: > [Reviving this old thread as I don't think we came to a conclusion on > this.] I guess the best thing is to submit the obvious 1 line patch and see what people think about it? (I agree the default ought to be changed.) Rich.

Re: [Qemu-devel] [RFC] Virtio RNG: Consider changing the default entropy source to /dev/urandom?

2019-05-02 Thread Kashyap Chamarthy
[Reviving this old thread as I don't think we came to a conclusion on this.] On Fri, Sep 21, 2018 at 05:43:23PM +0200, Kashyap Chamarthy wrote: > Hi folks, > > As Markus pointed out in this 'qemu-devel' thread[1], > backends/rng-random.c uses '/dev/random' in TYPE_RNG_RANDOM's > instance_init()

Re: [Qemu-devel] [RFC] Virtio RNG: Consider changing the default entropy source to /dev/urandom?

2018-09-24 Thread Kashyap Chamarthy
On Fri, Sep 21, 2018 at 08:06:24PM +0200, Laszlo Ersek wrote: > On 09/21/18 17:43, Kashyap Chamarthy wrote: [...] > > (b) When QEMU exposes a Virtio RNG device to the guest, that device > > needs a source of entropy, and IIUC, that source needs to be > > "non-blocking" (i.e.

Re: [Qemu-devel] [RFC] Virtio RNG: Consider changing the default entropy source to /dev/urandom?

2018-09-21 Thread Laszlo Ersek
On 09/21/18 17:43, Kashyap Chamarthy wrote: > Hi folks, > > As Markus pointed out in this 'qemu-devel' thread[1], > backends/rng-random.c uses '/dev/random' in TYPE_RNG_RANDOM's > instance_init() method: > > [...] > static void rng_random_init(Object *obj) > { > RngRandom *s

[Qemu-devel] [RFC] Virtio RNG: Consider changing the default entropy source to /dev/urandom?

2018-09-21 Thread Kashyap Chamarthy
Hi folks, As Markus pointed out in this 'qemu-devel' thread[1], backends/rng-random.c uses '/dev/random' in TYPE_RNG_RANDOM's instance_init() method: [...] static void rng_random_init(Object *obj) { RngRandom *s = RNG_RANDOM(obj); object_property_add_str(obj,