Thread (15 messages) flat view 15 messages, 5 authors, 2016-09-02

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

From: J Freyensee <hidden>
Date: 2016-09-02 18:11:33
Also in: lkml

quoted
quoted
quoted
+?????/*
+??????* 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;
Is it possible to make that a #define please?
I'll make it a module parameter as Keith suggested.
One question, though: should we call this and the sysfs parameter
apst_max_latency or should it be more generically
power_save_max_latency???The idea is that we might want to support
non-automonous transitions some day or even runtime D3.??Or maybe
those should be separately configured if used.
I read the spec and reviewed your latest patchset. ?Personally for me I
like having the field names from the NVMe spec in the names of the
Linux implementation because it makes it easier to find and relate the
two. ?So apst_max_latency makes more sense to me, as this is a
'apst'(e/a) NVMe feature.
--Andy

_______________________________________________
Linux-nvme mailing list
Linux-nvme at lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-nvme
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help