Nam Cao <[email protected]> writes: > Gabriele Monaco <[email protected]> writes: >> static int __ikm_find_monitor_name(char *monitor_name, char *out_name) >> { >> - char *available_monitors, container[MAX_DA_NAME_LEN+1], *cursor, *end; >> - int retval = 1; >> + char *available_monitors, *cursor, *line; >> + int len = strlen(monitor_name); >> + int found = 0; >> >> available_monitors = tracefs_instance_file_read(NULL, >> "rv/available_monitors", NULL); >> if (!available_monitors) >> return -1; >> >> - cursor = strstr(available_monitors, monitor_name); >> - if (!cursor) { >> - retval = 0; >> - goto out_free; >> - } >> + config_is_container = 0; > > Isn't config_is_container unused? > > Perhaps it is used in a follow-up patch? Let me keep reading..
Never mind, I'm stupid. Reviewed-by: Nam Cao <[email protected]>
