Re: [PATCH v4 1/9] sched: Add a generic function to return the preemption string.

[Ingo Molnar]

* Sebastian Andrzej Siewior <bigeasy@xxxxxxxxxxxxx> wrote:

> +const char *preempt_modes[] = {
> +	"none", "voluntary", "full", "lazy", NULL,
> +};

> +	/* Count entries in NULL terminated preempt_modes */
> +	for (j = 0; preempt_modes[j]; j++)
> +		;

I'm pretty sure the build-time ARRAY_SIZE() primitive is superior here. ;-)

Thanks,

	Ingo