On 07/24/14 17:15, Ján Tomko wrote: > On 07/24/2014 04:07 PM, Peter Krempa wrote: >> Qemu and kernel support multiple RNG's, so I revived my old patch to add the >> support to libvirt. >> >> Peter Krempa (3): >> qemu: cgroup: Don't use NULL path on default backed RNGs >> virtio-rng: allow multiple RNG devices >> test: qemu: Add tests for multiple virtio-rng devices >> >> src/conf/domain_audit.c | 4 +- >> src/conf/domain_conf.c | 58 >> +++++++++++----------- >> src/conf/domain_conf.h | 4 +- >> src/qemu/qemu_cgroup.c | 27 ++++++---- >> src/qemu/qemu_command.c | 29 ++++++----- >> .../qemuxml2argv-virtio-rng-multiple.args | 9 ++++ >> .../qemuxml2argv-virtio-rng-multiple.xml | 31 ++++++++++++ >> tests/qemuxml2argvtest.c | 2 + >> 8 files changed, 109 insertions(+), 55 deletions(-) >> create mode 100644 >> tests/qemuxml2argvdata/qemuxml2argv-virtio-rng-multiple.args >> create mode 100644 >> tests/qemuxml2argvdata/qemuxml2argv-virtio-rng-multiple.xml >> > > ACK series, but moving the default setting of '/dev/random' to the XML parser > would be nicer.
We could do that in qemu's device post parse callback so we don't hardcode that for other drivers. That would also shed some of the code that handles the possibility of using NULL there afterwards. I'm contemplating on doing that in a separate patch though. > > Jan >
signature.asc
Description: OpenPGP digital signature
-- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list
