Re: [PATCH 3/3] nvme: Enable autonomous power state transitions

From: Keith Busch
Date: Mon Aug 29 2016 - 12:35:23 EST


On Mon, Aug 29, 2016 at 02:25:46AM -0700, Andy Lutomirski wrote:
> + /*
> + * By default, allow up to 25ms of APST-induced latency. This will
> + * have no effect on non-APST supporting controllers (i.e. any
> + * controller with APSTA == 0).
> + */
> + ctrl->apst_max_latency_ns = 25000000;

Any objection to making this a module parameter? 25ms default sounds
reasonable, but I would still like the option to not ever initialize
APST on a capable device.