Bug 969788

In the first paragraph, we need to mention coupling of /dev/hwrng and rngd is a manual step -- ie customers need to do it themselves, via (e.g. "rngd -b -r /dev/hwrng -o /dev/random").

This manual step is NOT needed RHEL 7.1 onwards, but needed in RHEL6 and RHEL7.0.

There is a mention of "Figure 10.15. Random number generator device", should that be removed?

Last para has "Examples of such devices are /dev/random".  Just make it singular instead of plural?

Thanks,

'virtio-rng' and 'virtio-RNG' are all used in this chapter, I prefer the 1st one.


"The virtio-RNG device is available for both Windows and KVM guest virtual machines"

%s/KVM guest/Linux/
The virtio-rng device is available for both Windows and Linux virtual machines.

<source mode='connect' host physical machine='1.2.3.4' service='1234'>

%s/host physical machine/host/

typo?

Just a couple of notes.  We can hot-plug the virtio-rng device as well, so a shutdown isn't the only option, as mentioned in procedure 10.11.

The libvirt backend/model description might need some editing.  I'm busy with other stuff right now so can't do a thorough review of it, asking Peter Krempa if he can do it.  Peter?

2nd para:
> followind
typo

2nd para:
> Refer Note for instuctions on installing the Windows requirements. Unless noted, the followind descriptions are for both Linux and Windows guests virtual and host physical machines.

"guests virtual and host physical machines" doesn't make sense.  Suppose it should be "for both Linux (rather RHEL?) and Windows guest virtual machines".

3rd para:
>  in the /dev/hwrng directory.

/dev/hwrng is a file; not a directory.  I had changed this to read "at the location /dev/hwrng" in the etherpad at http://etherpad.corp.redhat.com/g5KZlqQqGv  why wasn't that incorporated?

Section Note:
> viorng ("VirtIO RNG Device")to be installed.

Missing whitespace after closing parenthesis

The libvirt section still has redundant info.

Not yet to my taste.

(In reply to Amit Shah from comment #20)
> 2nd para:
> > Refer Note for instuctions on installing the Windows requirements. Unless noted, the followind descriptions are for both Linux and Windows guests virtual and host physical machines.
> 
> "guests virtual and host physical machines" doesn't make sense.  Suppose it
> should be "for both Linux (rather RHEL?) and Windows guest virtual machines".

This is still the same.

> The libvirt section still has redundant info.

And the libvirt section still looks the same.

Particularly, there are three bullets in the last para, and they all overlap.  There also is this line:
> "The EGD protocol is a simple protocol that was introduced by the entropy gathering daemon. It is a substitute for machines that don't support /dev/random. "

This is of no concern to us why the EGD was introduced -- because RHEL always has /dev/random.

We really should scratch the entire "virtio-rng elements" section and re-write it.  Please; I think this is the 5th time I'm mentioning this.

This change is now available in the Red Hat Enterprise Linux 6.6 documentation at https://access.redhat.com/documentation/en-US/Red_Hat_Enterprise_Linux/.