Re: [Xen-devel] [PATCH] x86/mm: Swap mfn_valid() to use mfn_t

2017-02-27 Thread George Dunlap
On 16/02/17 20:07, Andrew Cooper wrote: > Replace one opencoded mfn_eq() and some coding style issues on altered lines. > Swap __mfn_valid() to being bool, although it can't be updated to take mfn_t > because of include dependencies. > > No functional change. > > Signed-off-by: Andrew Cooper Ac

Re: [Xen-devel] [PATCH] x86/mm: Swap mfn_valid() to use mfn_t

2017-02-22 Thread Julien Grall
Hi Andre, On 16/02/17 20:07, Andrew Cooper wrote: Replace one opencoded mfn_eq() and some coding style issues on altered lines. Swap __mfn_valid() to being bool, although it can't be updated to take mfn_t because of include dependencies. No functional change. Signed-off-by: Andrew Cooper A

Re: [Xen-devel] [PATCH] x86/mm: Swap mfn_valid() to use mfn_t

2017-02-20 Thread Tim Deegan
At 20:07 + on 16 Feb (1487275634), Andrew Cooper wrote: > Replace one opencoded mfn_eq() and some coding style issues on altered lines. > Swap __mfn_valid() to being bool, although it can't be updated to take mfn_t > because of include dependencies. > > No functional change. > > Signed-off-by

Re: [Xen-devel] [PATCH] x86/mm: Swap mfn_valid() to use mfn_t

2017-02-17 Thread Jan Beulich
>>> On 16.02.17 at 21:07, wrote: > Replace one opencoded mfn_eq() and some coding style issues on altered lines. > Swap __mfn_valid() to being bool, although it can't be updated to take mfn_t > because of include dependencies. > > No functional change. > > Signed-off-by: Andrew Cooper Acked-by

[Xen-devel] [PATCH] x86/mm: Swap mfn_valid() to use mfn_t

2017-02-16 Thread Andrew Cooper
Replace one opencoded mfn_eq() and some coding style issues on altered lines. Swap __mfn_valid() to being bool, although it can't be updated to take mfn_t because of include dependencies. No functional change. Signed-off-by: Andrew Cooper --- CC: Jan Beulich CC: Stefano Stabellini CC: Julien G