DM_CACHE_SMQ is experimental and enabled by default? (was: Re: dm cache: add stochastic-multi-queue (smq) policy)

From: Geert Uytterhoeven
Date: Fri Jun 26 2015 - 03:47:57 EST


On Fri, Jun 26, 2015 at 2:13 AM, Linux Kernel Mailing List
<linux-kernel@xxxxxxxxxxxxxxx> wrote:
> Gitweb: http://git.kernel.org/linus/;a=commit;h=66a636356647a9be8885c2ce2948de126577698a
> Commit: 66a636356647a9be8885c2ce2948de126577698a
> Parent: 40775257b97e27305cf5c2425be7acaa6edee4ea
> Refname: refs/heads/master
> Author: Joe Thornber <ejt@xxxxxxxxxx>
> AuthorDate: Fri May 15 15:33:34 2015 +0100
> Committer: Mike Snitzer <snitzer@xxxxxxxxxx>
> CommitDate: Thu Jun 11 17:12:59 2015 -0400
>
> dm cache: add stochastic-multi-queue (smq) policy

> diff --git a/drivers/md/Kconfig b/drivers/md/Kconfig
> index edcf4ab..b597273 100644
> --- a/drivers/md/Kconfig
> +++ b/drivers/md/Kconfig
> @@ -304,6 +304,18 @@ config DM_CACHE_MQ
> This is meant to be a general purpose policy. It prioritises
> reads over writes.
>
> +config DM_CACHE_SMQ
> + tristate "Stochastic MQ Cache Policy (EXPERIMENTAL)"
> + depends on DM_CACHE
> + default y

A feature cannot be "EXPERIMENTAL", and be enabled by default.
Please drop (at least) one of them.

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/