On 11/03/15 14:45, Paolo Bonzini wrote: > > > On 03/11/2015 14:25, Laszlo Ersek wrote: >> - Agreement between Paolo, Jordan and Mike about implementing >> broadcast SMIs. I am willing to code up whatever design is >> agreed upon. Can everyone involved please prioritize this >> discussion a little? > > Actually, I was *de*prioritizing it because things _more or less_ work > without it, especially if the timeout is temporarily reduced. We > probably agree that timeouts are evil in a virt setting, but even > without this issue you can commit the ~70 patches that bring us to 99% > of the way. > > Once the slate is clean and everybody is focused on the few remaining > problems, we can tackle them---including broadcast SMIs.
Works for me. Keeping your patch "OvmfPkg: use relaxed AP SMM synchronization mode" in the build (and not broadcasting SMIs in either component) certainly results in a functional binary. The perf penalty could be fine-tuned by decimating the timeout. (I don't recall if I *planned* to divide the current timeout by ten, or actually *tested* it. In any case, I recall thinking that 1/10th of the otherwise seen lag would be pretty tolerable.) So, this works for me definitely. Thank you Laszlo > > In fact, the rest of your email lists some more pressing problems than > broadcast SMIs, which I'm quoting below to further remind other readers. :) > > Paolo > >> - Solving the MP-related ExitBootServices() handler bug in CpuDxe. >> Patches have been on the list for almost a week. While I've been >> breaking my back testing and reviewing patches for others (not >> just on edk2-devel, mind you), nobody has batted an eye about >> that series. >> >> [edk2] [PATCH v2 0/4] UefiCpuPkg/CpuDxe: Fix ExitBootServices() >> callback in the presence of SMIs >> http://thread.gmane.org/gmane.comp.bios.edk2.devel/3518 >> >> Please review it, so that I can include it at the front of my >> upcoming v4 SMM series. >> >> - Thirty (30) patches from the SMM series still need reviews. Once >> all of the above is covered, I'll update the OvmfPkg/README >> patch about the status, and post version 4. I wouldn't mind if >> we could commit the series still in 2015, but I'm getting less >> and less hopeful. _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel