Re: [virtio] [PATCH 1/3] VIRTIO-110: ARM's feedback for MMIO chapter, trivial changes

[Pawel Moll <pawel.moll@arm.com>]

On Tue, 2014-08-05 at 00:07 +0100, Michael S. Tsirkin wrote:
> > @@ -2118,8 +2120,8 @@ synchronization.
> >  
> >  The driver MUST ignore undefined bits in \field{InterruptStatus}.
> >  
> > -The MUST write the events it handled into \field{InterruptACK} when
> > -it finishes handling an interrupt.
> > +The driver MUST write a value with a bit mask describing events it handled into \field{InterruptACK} when
> > +it finishes handling an interrupt and MUST NOT set any of the undefined bits in the value.
> 
> A bit of a tautology here, isn't it?
> How would any undefined bits get set if it only describes the events?

Again, what I heard from the friendly technical writer was along the
lines "it's not a poem, say the same 3 times if you have to".

> Hmm how would you describe this last change? Is this an error correction? An
> editorial change?

Editorial change.

Paweł