Re: [Xen-devel] [PATCH 00/15] xen/x86: Clean-up the PoD code

[Andrew Cooper <andrew.cooper3@xxxxxxxxxx>]

On 13/09/2017 18:59, Julien Grall wrote:
> Hi all
>
> I have been attempting to use the PoD code on ARM (it will be sent in a
> separate series) and spent sometimes to clean-up and switch to typesafe gfn
> the current code.
>
> The PoD code has been tested on ARM (the version is slightly different, mostly
> renaming) and the x86 part as only been built test it.
>
> Cheers,
>
> Cc: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
> Cc: George Dunlap <george.dunlap@xxxxxxxxxxxxx>
> Cc: Jan Beulich <jbeulich@xxxxxxxx>
> Cc: Jun Nakajima <jun.nakajima@xxxxxxxxx>
> Cc: Kevin Tian <kevin.tian@xxxxxxxxx>
> Cc: Razvan Cojocaru <rcojocaru@xxxxxxxxxxxxxxx>
> Cc: Tamas K Lengyel <tamas@xxxxxxxxxxxxx>
>
> Julien Grall (15):
>   xen/x86: p2m-pod: Clean-up includes
>   xen/x86: p2m-pod: Remove trailing whitespaces
>   xen/x86: p2m-pod: Fix coding style for comments
>   xen/x86: p2m-pod: Fix coding style
>   xen/x86: p2m-pod: Avoid redundant assignments in
>     p2m_pod_demand_populate
>   xen/x86: p2m-pod: Clean-up use of typesafe MFN
>   xen/x86: p2m-pod: Use typesafe gfn in p2m_pod_decrease_reservation
>   xen/x86: p2m: Use typesafe gfn for the P2M callbacks get_entry and
>     set_entry
>   xen/x86: p2m: Use typesafe GFN in p2m_set_entry
>   xen/x86: p2m-pod: Use typesafe GFN in pod_eager_record
>   xen/x86: p2m-pod: Clean-up p2m_pod_zero_check
>   xen/x86: p2m-pod: Use typesafe gfn in p2m_pod_zero_check
>   xen/x86: p2m-pod: Use typesafe gfn in p2m_pod_demand_populate
>   xen/x86: p2m-pod: Use typesafe gfn for the fields reclaim_single and
>     max_guest
>   xen/x86: p2m-pod: Rework prototype of p2m_pod_demand_populate

Only the one minor comment on patch 8, and its knock-on effect in later
patches.

Acked-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx> (with either my
suggestion, or a suitable alternative)

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