On Fri, Mar 02, 2018 at 04:10:48PM +0100, Paolo Bonzini wrote: > On 01/03/2018 11:33, Liu, Yi L wrote: > > +void pci_setup_sva_ops(PCIDevice *dev, PCISVAOps *ops) > > +{ > > + if (dev) { > > + dev->sva_ops = ops; > > + } > > + return; > > +} > > + > > Better: > > { > assert(ops && !dev->sva_ops); > dev->sva_ops = ops; > }
Thanks, would apply in next version. Regards, Yi Liu