[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [Xen-devel] RFC/PATCH: xen: race during domain destruction [Re: [xen-4.7-testing test] 105948: regressions - FAIL]
On Sun, 2017-02-26 at 16:53 +0100, Dario Faggioli wrote: > On Fri, 2017-02-24 at 17:14 +0100, Dario Faggioli wrote: > > On Wed, 2017-02-22 at 01:46 -0700, Jan Beulich wrote: > > > > > > However, comparing with the staging version of the file > > > (which is heavily different), the immediate code involved here > > > isn't > > > all that different, so I wonder whether (a) this is a problem on > > > staging too or (b) we're missing another backport. Dario? > > > > > So, according to my investigation, this is a genuine race. It > > affects > > this branch as well as staging, but it manifests less frequently > > (or, > > I > > should say, very rarely) in the latter. > > > Actually, this is probably wrong. It looks like the following commit: > > f3d47501db2b7bb8dfd6a3c9710b7aff4b1fc55b > xen: fix a (latent) cpupool-related race during domain destroy > > is not in staging-4.7. > And my testing confirms that backporting the changeset above (which just applies cleanly on staging-4.7, AFAICT) make the problem go away. As the changelog of that commit says, I've even seen something similar happening already during my development... Sorry I did not recognise it sooner, and for failing to request backport of that change in the first place. I'm therefore doing that now: I ask for backport of: f3d47501db2b7bb8dfd6a3c9710b7aff4b1fc55b xen: fix a (latent) cpupool-related race during domain destroy to 4.7. 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) Attachment:
signature.asc _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx https://lists.xen.org/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |