On Fri, 2016-04-15 at 16:19 +0100, Ian Jackson wrote: > Juergen Gross writes ("[PATCH 2/3] libxc: adjust retry loop in > xc_cpupool_removecpu()"): > > > > Commit 1ef6beea187b ("libxc: do some retries in > > xc_cpupool_removecpu() > > for EBUSY case") added a retry loop in xc_cpupool_removecpu() for > > the > > EBUSY case. As EBUSY was returned in multiple error situations the > > loop would have been executed in situations where a retry would not > > be successful. Additionally calling sleep(1) between the rerires is > > a > > bad idea when being called in a daemon. > > > > The hypervisor has been changed to return different error values > > now. > > The retry added in above mentioned commit should be done in the > > EADDRINUSE case now. As the error condition should last only for a > > very short time, the sleep(1) call can be removed. > > > > Requested-by: Ian Jackson <ian.jack...@eu.citrix.com> > > Signed-off-by: Juergen Gross <jgr...@suse.com> > Thanks. > > Acked-by: Ian Jackson <ian.jack...@eu.citrix.com> > Reviewed-by: Dario Faggioli <dario.faggi...@citrix.com>
Regards, Dario -- <<This happens because I choose it to happen!>> (Raistlin Majere) ----------------------------------------------------------------- Dario Faggioli, Ph.D, http://about.me/dario.faggioli Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK)
signature.asc
Description: This is a digitally signed message part
_______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel