Messages in this thread | | | Date | Thu, 21 Jun 2018 16:05:02 -0700 | From | Fenghua Yu <> | Subject | Re: [RFC PATCH 00/16] x86/split_lock: Enable #AC exception for split locked accesses |
| |
On Fri, Jun 22, 2018 at 12:10:06AM +0200, Peter Zijlstra wrote: > On Thu, Jun 21, 2018 at 03:00:03PM -0700, Fenghua Yu wrote: > > The control knob for firmware is to choose continuing firmware execution > > by disabling #AC split lock (default) or stopping firmware execution > > by enabling #AC for split lock. Stopping firmware execution may be useful > > in hard real time system to identify any split lock issue on the platform. > > Having the option only allows broken firmware to continue to exist. > Limiting people in how they can use their machines.
But in a real case, when I enable #AC for split lock in kernel, reboot hits #AC because of split lock in firmware code and firmware handles #AC as fatal error and stops continuing to run.
It will take long time/forever for firmware to fix the split lock issue. Before the firmware issue is fixed, reboot or S4 cannot run if the feature is enabled by kernel.
And if unlucky, I'm afraid the patch set even has no chance to be merged to upstream if maintainer's test machine has firmware split lock issue and the machine simply cannot reboot or go to S4 if the feature is enabled.
For those reasons, the current patches just don't trust firmware and disable #AC for split lock for firmware by default and allow sysadmin to enable it for firmware via the control knob.
So is it ok to still keep the control knob and disable #AC for split lock for firmware by default?
Thanks.
-Fenghua
| |