On Fri, Nov 21, 2014 at 05:03:09PM +0000, Jan Beulich wrote:
> >>> On 21.11.14 at 16:06, <wei.l...@citrix.com> wrote:
> > @@ -747,6 +786,17 @@ long do_memory_op(unsigned long cmd,
> > XEN_GUEST_HANDLE_PARAM(void) arg)
> > return start_extent;
> > args.domain = d;
> >
> > + args.memflags |=
> > MEMF_node(XENMEMF_get_node(reservation.mem_flags));
> > + if ( reservation.mem_flags & XENMEMF_exact_node_request )
> > + args.memflags |= MEMF_exact_node;
> > +
> > + rc = translate_vnode_to_pnode(d, &reservation, &args);
> > + if ( rc )
> > + {
> > + rcu_unlock_domain(d);
> > + return rc;
>
> I'm afraid you got misguided here by the (buggy) adjacent XSM error
> path: You shouldn't return a negative error code but "start_extent"
> here. And I'll try to remember to fix the XSM path post-4.5.
>
Fixed.
> > +/* Guset can use XENMEMF_vnode to specify virtual node for memory op. */
> > +#define XENFEAT_memory_op_vnode_supported 13
>
> You introduce this flag but then don't use it? Also there's a typo in
> the comment.
>
Oops, a hunk is missing. Fixed.
Wei.
> Jan
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
