Re: [Xen-devel] [PATCH 2/2] x86/pv: Code improvements to do_update_descriptor()

[Wei Liu <wei.liu2@xxxxxxxxxx>]

On Thu, Dec 06, 2018 at 06:03:55PM +0000, Andrew Cooper wrote:
>  * Add "uint64_t raw" to seg_desc_t to remove the opencoded uint64_t casting
>    in this function.
>  * Rename the 'pa' parameter to 'gaddr', because it lives in GFN space rather
>    than physical address space.
>  * Use gfn_t and mfn_t rather than unsigned longs.
>  * Check the alignment and proposed new descriptor before taking a page
>    reference.
>  * Reuse the out label for all exit paths.
>  * Use the more flexible ACCESS_ONCE() accessor in preference to
>    write_atomic()
> 
> No expected change in behaviour.
> 
> Signed-off-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>

Reviewed-by: Wei Liu <wei.liu2@xxxxxxxxxx>

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