On 11/18/2013 08:20 AM, Daniel P. Berrange wrote:
>> static const char *const defaultDeviceACL[] = {
>> "/dev/null", "/dev/full", "/dev/zero",
>> - "/dev/random", "/dev/urandom",
>> + "/dev/random", "/dev/urandom", "/dev/hwrng",
>> "/dev/ptmx", "/dev/kvm", "/dev/kqemu",
>> "/dev/rtc", "/dev/hpet", "/dev/vfio/vfio",
>> NULL,
>
> NACK, for any device listed in the XML, we should add it in the per-VM
> cgroups setup code.
>
> The existing /dev/random & /dev/urandom devices are there because they
> are used for basic crypto libraries unrelated to the XML config.
/dev/urandom, probably. /dev/random - really? Isn't that a DoS
potential for one guest to starve entropy from other guests, in spite of
sVirt?
> The
> /dev/hwrng device does not fall into this scenario.
Agreed.
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
Attachment:
signature.asc
Description: OpenPGP digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list