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