[lvc-project] [PATCH 6.1] mm/damon: get rid of overlapping areas

Fedor Pchelkin pchelkin at ispras.ru
Tue Jun 23 13:37:57 MSK 2026


On Tue, 23. Jun 13:26, Марков Глеб Игоревич wrote:
> Замена strncmp() на менее (на первый взгляд) безопасный strcmp() гарантирует
> корректность, поскольку его значение явно определено и работа с изолированным токеном,
> имхо, является более безопасным вариантом, чем использовать внешний count при сравнении.

Зачем это делать в текущем патче?  Это как-то относится к решаемой
проблеме?

> Альтернатива в виде strim() тоже отпадает, поскольку обрезается лишь значение после токена,
> а теоретические лидирующие пробелы не отпадают, а значит работа будет вестить с
> некорректным значением.

/**
 * strim - Removes leading and trailing whitespace from @s.
 * @s: The string to be stripped.
 *
 * Note that the first trailing whitespace is replaced with a %NUL-terminator
 * in the given string @s. Returns a pointer to the first non-whitespace
 * character in @s.
 */
char *strim(char *s)
{
	size_t size;
	char *end;

	size = strlen(s);
	if (!size)
		return s;

	end = s + size - 1;
	while (end >= s && isspace(*end))
		end--;
	*(end + 1) = '\0';

	return skip_spaces(s);
}
EXPORT_SYMBOL(strim);



More information about the lvc-project mailing list