[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CALCETrWDRV_TZRpsya=kmG+zDGwRqBpSGh6NGqCgDw3SdMfT9w@mail.gmail.com>
Date: Mon, 29 Aug 2016 16:16:33 -0700
From: Andy Lutomirski <luto@...capital.net>
To: Keith Busch <keith.busch@...el.com>
Cc: Christoph Hellwig <hch@....de>, linux-nvme@...ts.infradead.org,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
Jens Axboe <axboe@...com>
Subject: Re: [PATCH 3/3] nvme: Enable autonomous power state transitions
On Aug 29, 2016 9:35 AM, "Keith Busch" <keith.busch@...el.com> wrote:
>
> 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.
Will do.
Powered by blists - more mailing lists