| Date | Mon, 19 Nov 2018 21:21:51 +0100 (CET) | From | Thomas Gleixner <> | Subject | Re: [Patch v5 11/16] x86/speculation: Add Spectre v2 app to app protection modes |
| |
On Fri, 16 Nov 2018, Tim Chen wrote: > +static enum spectre_v2_app2app_mitigation_cmd __init > + spectre_v2_parse_app2app_cmdline(enum spectre_v2_mitigation_cmd v2_cmd) > +{ > + enum spectre_v2_app2app_mitigation_cmd cmd; > + char arg[20]; > + int ret, i; > + > + if (v2_cmd == SPECTRE_V2_CMD_FORCE) { > + cmd = SPECTRE_V2_APP2APP_CMD_FORCE; > + goto show_cmd;
What initializes 'i' in this case? Compiler warnings are overrated.
What exactly handles the SPECTRE_V2_NONE case which you documented in the commandline bits?
Thanks,
tglx
|