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

Reply via email to