On 08/06/16 13:54, Jan Beulich wrote: > This allows us to see the full ioreq without having to peek into state > which is supposedly private to the emulation framework. > > Suggested-by: Paul Durrant <paul.durr...@citrix.com> > Signed-off-by: Jan Beulich <jbeul...@suse.com>
Reviewed-by: Andrew Cooper <andrew.coop...@citrix.com>, with two minor style corrections. > @@ -333,23 +332,29 @@ out: > return r; > } > > -static int msixtbl_range(struct vcpu *v, unsigned long addr) > +static int _msixtbl_write(const struct hvm_io_handler *handler, > + uint64_t address, uint32_t len, uint64_t val) Indentation. > @@ -396,10 +401,10 @@ static int msixtbl_range(struct vcpu *v, > return 0; > } > > -static const struct hvm_mmio_ops msixtbl_mmio_ops = { > - .check = msixtbl_range, > +static const struct hvm_io_ops msixtbl_mmio_ops = { > + .accept = msixtbl_range, > .read = msixtbl_read, > - .write = msixtbl_write > + .write = _msixtbl_write Trailing comma. ~Andrew _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel