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

Re: [Xen-devel] [PATCH 2/3] x86/hvm: Improvements to external users of decode_register()


  • To: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, Xen-devel <xen-devel@xxxxxxxxxxxxx>
  • From: "Tian, Kevin" <kevin.tian@xxxxxxxxx>
  • Date: Mon, 29 Jan 2018 03:11:56 +0000
  • Accept-language: en-US
  • Cc: "Nakajima, Jun" <jun.nakajima@xxxxxxxxx>, Jan Beulich <JBeulich@xxxxxxxx>
  • Delivery-date: Mon, 29 Jan 2018 03:12:20 +0000
  • Dlp-product: dlpe-windows
  • Dlp-reaction: no-action
  • Dlp-version: 11.0.0.116
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
  • Thread-index: AQHTlqgCKTdf6VW9Akig7vVQdBjzu6OKMGsg
  • Thread-topic: [PATCH 2/3] x86/hvm: Improvements to external users of decode_register()

> From: Andrew Cooper [mailto:andrew.cooper3@xxxxxxxxxx]
> Sent: Friday, January 26, 2018 9:17 PM
> 
>  * Rename to decode_gpr() to be more specific as to its purpose
>  * Drop the highbyte encoding handling, as users care, and it unlikely that
>    future users would care.
>  * Change to a static inline, returning an unsigned long pointer.
> 
> Doing so highlights that the "invalid gpr" paths in hvm_mov_{to,from}_cr()
> were actually unreachable.  All callers already passed in-range gprs, which
> would have hit a BUG() previously.
> 
> Signed-off-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>

Reviewed-by: Kevin Tian <kevin.tian@xxxxxxxxx>

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel

 


Rackspace

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