> -----Original Message----- > From: Jan Beulich [mailto:jbeul...@suse.com] > Sent: 13 January 2017 11:45 > To: Paul Durrant <paul.durr...@citrix.com> > Cc: Andrew Cooper <andrew.coop...@citrix.com>; George Dunlap > <george.dun...@citrix.com>; Ian Jackson <ian.jack...@citrix.com>; xen- > de...@lists.xenproject.org; Daniel De Graaf <dgde...@tycho.nsa.gov>; Tim > (Xen.org) <t...@xen.org> > Subject: Re: [PATCH v3 3/8] dm_op: convert HVMOP_track_dirty_vram > > >>> On 12.01.17 at 15:58, <paul.durr...@citrix.com> wrote: > > v3: > > - Check d->max_vcpus rather than d->vcpu, as requested by Jan. > > - The handle type changes (from uint8 to void) are still necessary, hence > > omitting Jan's R-b until this is confirmed to be acceptable. > > Hmm, talk really was of the change from HANDLE_64() to > HANDLE_PARAM(), which I see you also left as they were. In the end, > if this builds I'm fine either way, and since you say you need to do > the uint8 -> void change anyway, doing the 64 -> PARAM change at > once is perhaps even a change to the better. So hypervisor parts > Reviewed-by: Jan Beulich <jbeul...@suse.com> >
Ok. Thanks, Paul > Jan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel
