[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH 2/6] x86/HVM: remove unnecessary indirection from hvm_get_mem_pinned_cacheattr()

To: Jan Beulich <JBeulich@xxxxxxxx>, xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxxx>
From: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
Date: Thu, 3 Mar 2016 17:35:09 +0000
Cc: Keir Fraser <keir@xxxxxxx>, Wei Liu <wei.liu2@xxxxxxxxxx>, Tim Deegan <tim@xxxxxxx>
Delivery-date: Thu, 03 Mar 2016 17:35:32 +0000
List-id: Xen developer discussion <xen-devel.lists.xen.org>

On 03/03/16 16:36, Jan Beulich wrote:
> Its return value can easily serve the purpose. We cannot, however,
> return unspecific "success" anymore for a domain of the wrong type -
> since no caller exists that would call this for PV domains, simply add
> an ASSERT().
>
> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>

Reviewed-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel

References:
- [Xen-devel] [PATCH 0/6] x86/HVM: cache attribute pinning adjustments
  - From: Jan Beulich
- [Xen-devel] [PATCH 2/6] x86/HVM: remove unnecessary indirection from hvm_get_mem_pinned_cacheattr()
  - From: Jan Beulich

Prev by Date: Re: [Xen-devel] [BUG] xs.watch and xs.unwatch are unreliable
Next by Date: Re: [Xen-devel] [PATCH 1/6] x86/HVM: honor cache attribute pinning for RAM only
Previous by thread: Re: [Xen-devel] [PATCH 2/6] x86/HVM: remove unnecessary indirection from hvm_get_mem_pinned_cacheattr()
Next by thread: [Xen-devel] [PATCH 3/6] x86/HVM: adjust hvm_set_mem_pinned_cacheattr() error indications
Index(es):
- Date
- Thread

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.