nice fine. Might be worth adding the text of this comment (modified as needed) to the CL so that in years to come people understand the reasons.
On Tue, Oct 5, 2021 at 12:51 PM Matt B <[email protected]> wrote: > > A quick google turned this up: > https://chromium.googlesource.com/chromiumos/third_party/kernel/+/chromeos-3.0/arch/x86/kernel/cpu/intel.c#253 > > On Tue, Oct 5, 2021 at 4:06 AM Julian Stecklina > <[email protected]> wrote: >> >> On Tue, 2021-10-05 at 09:29 +0300, Kyösti Mälkki wrote: >> > On Mon, Oct 4, 2021 at 8:12 PM Julian Stecklina >> > <[email protected]> wrote: >> > > >> > > But it looks like the workaround was just carried forward with no >> > > discussion >> > > of >> > > whether it's still necessary or what it actually works around. >> > > >> > >> > Hi >> > >> > Removal has been suggested with the X2APIC work: >> > >> > https://review.coreboot.org/c/coreboot/+/55199 >> > >> >> I've been looking at 4.13 instead of master. My bad. In master, indeed most >> atomic accesses are gone and the ones writing to ICR are left. This mostly >> makes >> sense and is much clearer now. :) >> >> Thanks, >> Julian >> >> _______________________________________________ >> coreboot mailing list -- [email protected] >> To unsubscribe send an email to [email protected] > > _______________________________________________ > coreboot mailing list -- [email protected] > To unsubscribe send an email to [email protected] _______________________________________________ coreboot mailing list -- [email protected] To unsubscribe send an email to [email protected]
