Re: [Xen-devel] [PATCH RFC 1 6/8] xen/pt: Make xen_pt_unregister_device idempotent

2015-07-14 Thread Stefano Stabellini
On Thu, 2 Jul 2015, Konrad Rzeszutek Wilk wrote: > > > @@ -858,15 +863,20 @@ static void xen_pt_unregister_device(PCIDevice *d) > > > machine_irq, errno); > > > } > > > } > > > +s->machine_irq = 0; > > > } > > > > > > /* delete a

Re: [Xen-devel] [PATCH RFC 1 6/8] xen/pt: Make xen_pt_unregister_device idempotent

2015-07-02 Thread Konrad Rzeszutek Wilk
> > @@ -858,15 +863,20 @@ static void xen_pt_unregister_device(PCIDevice *d) > > machine_irq, errno); > > } > > } > > +s->machine_irq = 0; > > } > > > > /* delete all emulated config registers */ > > xen_pt_config_delete(s);

Re: [Xen-devel] [PATCH RFC 1 6/8] xen/pt: Make xen_pt_unregister_device idempotent

2015-07-01 Thread Stefano Stabellini
On Mon, 29 Jun 2015, Konrad Rzeszutek Wilk wrote: > To deal with xen_host_pci_[set|get]_ functions returning error values > and clearing ourselves in the init function we should make the > .exit (xen_pt_unregister_device) function be idempotent in case > the generic code starts calling .exit (or fo

[Xen-devel] [PATCH RFC 1 6/8] xen/pt: Make xen_pt_unregister_device idempotent

2015-06-29 Thread Konrad Rzeszutek Wilk
To deal with xen_host_pci_[set|get]_ functions returning error values and clearing ourselves in the init function we should make the .exit (xen_pt_unregister_device) function be idempotent in case the generic code starts calling .exit (or for fun does it before calling .init!). Signed-off-by: Konr