Source-Changes-D archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: src/sys/arch/x86/x86



Le 10/05/12 14:35, Christoph Egger a écrit :
+               /* Check for race:
+                * When we booted with -x a cpu might already finished
+                * (why doesn't xc_wait() wait for *all* cpus?)
+                * and sc->sc_blob is already freed.
+                * In this case the calculation below touches
+                * non-allocated memory and results in a crash.
+                */

xc_wait should wait for all CPUs, except low priority xcalls and offline CPUs.

Are you sure that moving to low priority xcalls is the way to go? You can end up with CPUs not being updated because they are offline.

--
Jean-Yves Migeon
jeanyves.migeon%free.fr@localhost


Home | Main Index | Thread Index | Old Index