Re: Requesting for freeze exception for RMRR

["Jan Beulich" <JBeulich@suse.com>]

>>> On 14.07.15 at 11:25, <ian.campbell@citrix.com> wrote:
> On Tue, 2015-07-14 at 10:18 +0100, Jan Beulich wrote:
> 
>> >>> Y		Y	[v7][PATCH 14/16] xen/vtd: enable USB device assignment
> 
> diffstat:
>  xen/drivers/passthrough/vtd/dmar.h  |  1 -
>  xen/drivers/passthrough/vtd/iommu.c | 11 ++---------
>  xen/drivers/passthrough/vtd/utils.c |  7 -------
>  3 files changed, 2 insertions(+), 17 deletions(-)
> 
>> >>> Y		Y	[v7][PATCH 15/16] xen/vtd: prevent from assign the device with
>> >>> shared rmrr
> 
> xen/drivers/passthrough/vtd/iommu.c | 32 +++++++++++++++++++++++++++++---
>  1 file changed, 29 insertions(+), 3 deletions(-)
> 
>> >>
>> >> And yet again for these two. Please avoid giving a false impression
>> > 
>> > But these two patches really won Kevin's Ack, and also I wrote this line
>> > 
>> > Acked-by: Kevin Tian <kevin.tian@intel.com>
>> > 
>> > both in these two patches.
>> 
>> But talk here is about their review status, not who ack-ed them (and
>> an ack by other than  a maintainer of the affected code is not very
>> meaningful anyway).
> 
> Kevin is maintainer of that code though, isn't he?

Oh, right, I got mislead by the "hv" in the middle of Tiejun's
earlier reply, assuming (base on remembering the ordering of the
series) what follows it are the hvmloader patches (without looking
at the actual titles again). Indeed, the part of my reply above in
parentheses was misguided. The patches not having been
reviewed by anyone still is a fact (as an ack is not a review iirc).

Jan