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

Reply via email to