>>> On 11.08.15 at 19:41, <julien.gr...@citrix.com> wrote:
> --- a/xen/common/memory.c
> +++ b/xen/common/memory.c
> @@ -126,22 +126,28 @@ static void populate_physmap(struct memop_args *a)
> if ( is_domain_direct_mapped(d) )
> {
> mfn = gpfn;
> - if ( !mfn_valid(mfn) )
> +
> + for ( j = 0; j < (1 << a->extent_order); j++, mfn++ )
While benign I think we shouldn't repeat mistakes like this made
elsewhere in the code: At the very least this should be 1U, but
with j (needlessly, just like i) being unsigned long it would be
more consistent for this to be 1UL unless we clean up the variable
types (which I think I'll do).
Since the change is so minor, I'd be fine with making it while
committing, unless you object.
Jan
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
