On 08/07/2017 08:19 PM, Sergey Senozhatsky wrote:
> On (08/07/17 11:52), Prarit Bhargava wrote:
<snip>
> [..]
>> + if (strlen(param) == 1) {
>> + /* Preserve legacy boolean settings */
>> + if (!strcmp("0", param) || !strcmp("n", param) ||
>> + !strcmp("N", param))
>> + _printk_time = PRINTK_TIME_DISABLE;
>> + if (!strcmp("1", param) || !strcmp("y", param) ||
>> + !strcmp("Y", param))
>> + _printk_time = PRINTK_TIME_LOCAL;
>> + }
>> + if (_printk_time == -1) {
>> + for (stamp = 0; stamp <= 4; stamp++) {
>> + if (!strncmp(printk_time_str[stamp], param,
>> + strlen(param))) {
>> + _printk_time = stamp;
>> + break;
>> + }
>> + }
>> + }
>
> you can use match_string() here.
match_string doesn't match correctly. Others have made requests for, for
example, printk.time=r printk.time=real, etc. Instead of butchering the code
for a bunch of strings I chose to allow partial matching and simplify the code.
P.
--
To unsubscribe from this list: send the line "unsubscribe linux-doc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
