From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934831AbcIVUom (ORCPT ); Thu, 22 Sep 2016 16:44:42 -0400 Received: from mx0a-00082601.pphosted.com ([67.231.145.42]:44809 "EHLO mx0a-00082601.pphosted.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933263AbcIVUog (ORCPT ); Thu, 22 Sep 2016 16:44:36 -0400 Subject: Re: [PATCH v4 0/3] nvme power saving To: Andy Lutomirski References: <13b0fe95-01b8-89d7-df76-efc1aa14b27a@fb.com> CC: Andy Lutomirski , Keith Busch , , Christoph Hellwig , "linux-kernel@vger.kernel.org" , J Freyensee From: Jens Axboe Message-ID: <0a2ffd51-6e0f-7db3-8135-84317d8f77bc@fb.com> Date: Thu, 22 Sep 2016 14:43:43 -0600 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.3.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset="utf-8"; format=flowed Content-Transfer-Encoding: 7bit X-Originating-IP: [66.29.164.166] X-ClientProxiedBy: BY1PR19CA0001.namprd19.prod.outlook.com (10.162.139.139) To MWHPR15MB1200.namprd15.prod.outlook.com (10.175.2.142) X-MS-Office365-Filtering-Correlation-Id: 0c6a8b9f-8393-46ce-416a-08d3e32927d2 X-Microsoft-Exchange-Diagnostics: 1;MWHPR15MB1200;2:jeAx9cP/H2ASn9wTEnH68HuaLUe5KcXnuXmDz+pcrYyvLQmZgWZSRdJiainhuBzW/Fc1+OcUbzDwobDsZcg7h2PWv/aIPgIM/Q4QR7sFRzIzll8lXshzJpCV/zHObWYdgnTU05iUjqhEnPMlugre9QHakdp+UE2myex3LVm7RuraEiCp0mYdissI1OvGA0y6;3:U66DlR717ao28lt8ixZP00scv5TUkd1CciQacSU7JAXiPnhk9fPcnBjEeEXRao/posBcsUZGvpe6/mL94YCF1XiYBONot5QZAEE7eWgxZ7LJmPssyZR/tH8VF3xgp/uc X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:MWHPR15MB1200; X-Microsoft-Exchange-Diagnostics: 1;MWHPR15MB1200;25:7vh52qrgKP4T1TVOGOBRePNpBwfZTzKU3YCTk0k/psydoEO0L/UvKeXHQJZJBmolx87DaRbFOncU9VwIo1Tk1IWkB5L30Az4zJDL/mkmIcWgfd+iDgdpHDlmFQEfpGrt6pFY2aXTpyfXCO6ndMzMFrlVtVORQl+2AX2Giud82DLQFVGFBhUVWl/8OyKedccHWxHUrTNpRv0YWYahxEAoh5TdSxCph/eatc7wlfiE+HMgBQNVCZaYQuOw4daFwFZnkyn6VB1wz2bDgHVlFw8Jyr1xA2Ynh8nh1fshZW9Hq6k4JTan89v88ezdC5a53G6ddSpJdKpsYhsMyrrXMY4rX/dchumXnjE821MjJbYWxVjEXGhwTQdvmJreFNYnC1Mmw/Oguofr6H7/Ts3irMvnMS0onL52MY2bPqSty8DRMcMly4/mgiscgWAWPevPZ9L7VgpApV+vk6qxIjetkcdveVQCLjxGni4ksoU3oeRVgPXyAbOCkTX4t/DKYrGoBYZRN3MEV4IUvOuOo8/WLKavwMcLkJab38qupns32o1h3Emi6utMzcodCFcNLhQh7lx6Y3XMUPzZM8FCrf16L53wNSG6zAGfU4oDkcxGEllhNT5eqvm7XCTLCF7DL4BrQpzRg2eGMsLxLBOXquBfCfAFFNkwpRMEWUhHbsYB5l6ZMuxBbxWO278cl2X1L5Sd7ReJ;31:l6Cbn1AiH017pMbR19Vwhlhco3hHuSVKUqVrq1tcvgXrtLjjuYyoWLZib6Ehielxf90Om7Gux5YouZrRXRejUnt4vMhkGHlm/Zmo9/TkH4SiIkwBt6eMzs/dQ93yV/jbPP5EVjlvJRDa5rMPApVd++lu2aPp8/bZY0j8Cz+AL3NagceU97/BzIA7MA+GKHrvUI0HLKUsVym8Sa2bS3cwyd5FIKkSuhQIi6xljJgeeaM= X-Microsoft-Exchange-Diagnostics: 1;MWHPR15MB1200;20:g7T5pkMjrTnXyNwuLgjeGKv0jQfVC+fV6D1ius91z9v87Cs7hzLaP7NnYgoBpBUyjhdfioAmnin/0VGFEiZZPv3S1M8qSi8HtszM+M7AqJiYbb66euA0lfZc5IujCj+TvapMUpG1azoqo8iXX+HvfFy3tosHvVotamOu0HffBlA=;4:hBK+RZ10VaiNkUNdFexwsf8Sqhn1fgLKOqcmRHmAOb7HrGqZoe8M6flsUe67/9p53zDSqhyWiKT1MwS/5jbr4BS+SKESZhD58UUx4yqlvOgyUsB/XXT01nka4Na/zmr8biRe2MTrKfOcFe9lPrQTiPnOgaPcawzyi2qP1PSyLvfCskdPc7SsTJGtviUA/bMdv7d6wRniE8o1kJ+JLPUnprwAWb7/pycjYhs8vJ+U0xjfALB0+uW83H3QywJM8KxKBj0p/8WKZpYB8B/86gxXpqNodNbMI9khdjs3rCvy9TCtaPoig9Rp22/2ROPHiu0HGjh/mLiccXeNpiKrQttV45Mh8xQCWbLL9icYCNbRAFW7fxGQLkn3TaWzouyS7JeCrp1wjDWDeqhq8nPGatiL8yPwOXz/PEPl+i+lkx/rZ/z80emcN330kknKeiY2Y8kMYd60mOM63QoGeIgYyj1sTg== X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:(158342451672863)(67672495146484); X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(6040176)(601004)(2401047)(8121501046)(5005006)(10201501046)(3002001);SRVR:MWHPR15MB1200;BCL:0;PCL:0;RULEID:;SRVR:MWHPR15MB1200; X-Forefront-PRVS: 0073BFEF03 X-Forefront-Antispam-Report: SFV:NSPM;SFS:(10019020)(4630300001)(6049001)(6009001)(7916002)(377454003)(199003)(189002)(53754006)(24454002)(230700001)(110136003)(92566002)(36756003)(7736002)(42186005)(106356001)(31686004)(5660300001)(33646002)(65826007)(23676002)(86362001)(31696002)(105586002)(305945005)(6116002)(586003)(3846002)(83506001)(7846002)(2906002)(4001350100001)(81166006)(19580405001)(19580395003)(65806001)(50466002)(81156014)(68736007)(189998001)(50986999)(54356999)(47776003)(65956001)(117156001)(2950100001)(66066001)(77096005)(4326007)(8676002)(97736004)(64126003)(76176999)(101416001);DIR:OUT;SFP:1102;SCL:1;SRVR:MWHPR15MB1200;H:[192.168.1.176];FPR:;SPF:None;PTR:InfoNoRecords;A:1;MX:1;LANG:en; X-Microsoft-Exchange-Diagnostics: =?utf-8?B?MTtNV0hQUjE1TUIxMjAwOzIzOk1TRHp0aHlEWC9TbG5HYVZKVUFhRnUyRTNh?= =?utf-8?B?MCt6c2NOZ1FwTXNrZDdxMmJOdXhKNzVsTnhDZS9XNFQ4SEV2WVFGeXJXVTA2?= =?utf-8?B?UktOblk2eXVHYkRTMlNqaUtXa2RKM0tmSzdyRUNVdUlCNjhSMGVENWJnZWhV?= =?utf-8?B?UXErZDVZRWN6VTQzTTlkZEVXM0JkaEVXSnpoM1VQMFMvVGR5ZzhxMTJEKzBh?= =?utf-8?B?T3dkdWhjQ0lSaHBWeDZSUWYxUmE5ODlGUEdoVjlkc1RSOWMrbTk0aTJsTkth?= =?utf-8?B?TTFvNmp0NDVTVG9KUzhNc3BWY0cyZHMydGRSZlRhNDk3dE9DU2JHSWt6dXBM?= =?utf-8?B?Y1lOTmhXbWN4UWMvMlZHT2ZtNjZNNnJVTWhZSTZoU1A3QXJVZHB3clJCK1p4?= =?utf-8?B?bXgvV09rMWJxaVlaR1FlMTdPRzJtQ2ViWllwOUdHSzNtS3A0RTlYbnhub1hB?= =?utf-8?B?VTBBbm9kWlFRb3JicUFnSmh3bEp2dGkyRGhCMExjY3ZveDZiL2R4amYvVXpE?= =?utf-8?B?eGVUMGpmcXRraVJtcDc3SndTWG5EMTJsK1dHKzFjTklobU5ZNVhoU01tUzdz?= =?utf-8?B?WjE4M1lwRmwybDdHQzBNdmU0YlpWRWtzM21aRkVyV0VwTlhySHJNWXlFNlUz?= =?utf-8?B?dGplR0ZiKzc0Nk1QMHIzQk9aVzEvb1FuWUVxQWlEcUxYbkptWmd0T2kwZHo4?= =?utf-8?B?K2lFbWFVMnE5OGFMVkdRSkl6TWd2Y1FwSTB6T1NhcW40cTRyV3VPaHNFNitW?= =?utf-8?B?akR2SWV3OGtSRm5KTGV3Q1lBMzhmNEVyem4yWWk5SU5xNlA2ZGdIQW9HZGJs?= =?utf-8?B?SnJlang0VHoxL1hidXYvMUt0TGlVdVR6SmZNS1VkRWJjbGhaSS9Wa2FESTU2?= =?utf-8?B?WHQ2ZHFSV1RHbVJzT2E5dUxMdEpNV2lHOUs0TEJCSDE3YkFCQ3dic0toTXpS?= =?utf-8?B?RS9iVWlYZ2FrQ1puVlpGNkRoZUlRNXZiZWszT1NLUDhRa3c4ZUo3elgrTEZY?= =?utf-8?B?S2dHeDRwcHVQWFFuWk90dUI0Z3ZFSktoUTQ4d1h0dUJMd082NnM3L0hGRU02?= =?utf-8?B?WHRIdWh4ajI1dkNaNm9hbFA2c1M2MzNuaCtrWWxiRFdJT0hyNVRJQlpOemlY?= =?utf-8?B?QlJTMWlIZ2tRRU9FRFhNWS9TNGpUTFBzZ0VJejlHazgwYTJHNWdDY29udFg0?= =?utf-8?B?c0NpSUZyN25TTjFLUWI5RkY3NXhVZVNKc3BvdEFXTU1pcUZ0cERkVEllNVpt?= =?utf-8?B?UGlzaGNyZVE1S1p1NWdYaU9mdDcyZitCLytmZDY3YUhqR0tjR2FSMitqb3hL?= =?utf-8?B?a0JiYTArVzhSa284MlVGZnJwNVNHVmFkdWdOc2ZESVFNZXlTc3pKazJkL3ox?= =?utf-8?B?TEZVb0x2QkxSTEc5V0FlSm9WaXVGVkJWMXg3TXpPVUN3czJ5bzBPV1VkUWZq?= =?utf-8?B?RHFNWUFUSTJjamlFZVFzV2FzZ1phamFsVnU2ZzYxU2xYV0RJa2h4NUcxbmNE?= =?utf-8?B?UFNLb0E5blNQcGVGcHdKZW5TcWUyNm1RUldML3pMZ29wUFpEWS9zU01FSjha?= =?utf-8?B?YW5XWVJJb082ZzBBUktXNENPMjJtNkFIZENMQkxaMHpMaXlJU0lUZlA2Sm9u?= =?utf-8?B?OUErcG5xaDJ6VEF3QkZxSi9GUDdLc2NHUitrWWJ4d3hPWjBwR3NheWZKRHk5?= =?utf-8?B?RVZNQ2svclgwNlNqc1BaZ3dXTEpyMjZhb3FyMzVuZVNCTGdYMVFId3JvellU?= =?utf-8?B?QmJoOTVSRGZMRXNpRWsyVmpCVEhBR1h0MTVPdTlSOUsrY2YvL0tYbG5TVlA1?= =?utf-8?Q?AIsEVTuLOuXg0?= X-Microsoft-Exchange-Diagnostics: 1;MWHPR15MB1200;6:94UQM6AHn0ZpCPlXtXBfC9ik7AHFIsIcXGju2nwpipjuAluUDWnxvcEZbG86/EugCKss9jZ3bh9gRAxVDBdcnfKlzaj5T6lfnGgApFyYd1m+sM99UMdBGU62TkFL6cyMFAbrnmb9EaNZWIOhoSiDqHSx7eddru+OCkl2yz/BlbCJoUabT8ZtCqK1TkrI6ufBVRhLNWKAhFNAsvtTOWb7RcELwMqu3d5UZx4gt16jRnTgekV31EE2pacN+82vpUe1S9g+1QmWgyHVptm6ewL1E2Kwmiegf5kuIfBIwedGplM=;5:zRo48eIup5YksDz6GxUmsNt5qhzhohXZ96W9cOfwYvodszawEFZG2Ao2O9NRBk6PQX2Z0EPl4D6OQr8yiVxydBqnKz8+h+p0p0dYVR/WaTtcr9sGGF5NLspPtBoiyU9QOsLzSA3QQOuhmMF9UuLl2Q==;24:eyhh6I7cQRiBvHxO5aQbDhzDnE96Zz1nNJXiABhJqzhZrwyO8ryQ8YfDBxGdsOUNg0MXEVUhs8uJRgZXPDPQW430lgO/W/yGX++GNPcUz1Y=;7:xPxD65qk6pMibOSIZDCtWD5i9LO+vxdfB06r2PEho2aG8i2i7cTA27eFlQ/2I4YU01EYTENjNfxDvrqkB8NX+nUPqNn0z9FgU3WMXrLcreI4ha0KkE0Kdjqwek/nUieCdmhiR/HrM9fVhudL2UeLVZOT/U0P5sgv9NiRTKIjkRImcIlcoZDc1kzJJz+7lnlExRFBSVzPIuRngpYTprqkmFMoo0E+K+8XVrTSe/HR66h2+CJn5iuQzYHvHOHT4qflBLjeZw4oAQKef10/O+OtsR42GP5qxIF10frhhjLAIPWYF92bNUN73aUZXtQuTMTi SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1;MWHPR15MB1200;20:4LSgsPFxyfjZ4fAbPqR5p0O81NKUrhNM+i+3RiVY9OsDlk918dKE+nmbiTlPpNPafbe1iDG3v/kRwiDOLng2iiTGR9wlRh3G1h/oB4TRVJrtE92jSxHaCHVYNRAYGQyx19uoZl2vu3nDmsAkPQu+A3x74RoQa3UhDGNM43nmyN4= X-MS-Exchange-CrossTenant-OriginalArrivalTime: 22 Sep 2016 20:43:48.5963 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: MWHPR15MB1200 X-OriginatorOrg: fb.com X-Proofpoint-Spam-Reason: safe X-FB-Internal: Safe X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:,, definitions=2016-09-22_09:,, signatures=0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 09/22/2016 02:11 PM, Andy Lutomirski wrote: > On Thu, Sep 22, 2016 at 7:23 AM, Jens Axboe wrote: >> >> On 09/16/2016 12:16 PM, Andy Lutomirski wrote: >>> >>> Hi all- >>> >>> Here's v4 of the APST patch set. The biggest bikesheddable thing (I >>> think) is the scaling factor. I currently have it hardcoded so that >>> we wait 50x the total latency before entering a power saving state. >>> On my Samsung 950, this means we enter state 3 (70mW, 0.5ms entry >>> latency, 5ms exit latency) after 275ms and state 4 (5mW, 2ms entry >>> latency, 22ms exit latency) after 1200ms. I have the default max >>> latency set to 25ms. >>> >>> FWIW, in practice, the latency this introduces seems to be well >>> under 22ms, but my benchmark is a bit silly and I might have >>> measured it wrong. I certainly haven't observed a slowdown just >>> using my laptop. >>> >>> This time around, I changed the names of parameters after Jay >>> Frayensee got confused by the first try. Now they are: >>> >>> - ps_max_latency_us in sysfs: actually controls it. >>> - nvme_core.default_ps_max_latency_us: sets the default. >>> >>> Yeah, they're mouthfuls, but they should be clearer now. >> >> >> The only thing I don't like about this is the fact that's it's a driver private thing. Similar to ALPM on SATA, it's yet another knob that needs to be set. It we put it somewhere generic, then at least we could potentially use it in a generic fashion. > > Agreed. I'm hoping to hear back from Rafael soon about the dev_pm_qos > thing. > >> >> Additionally, it should not be on by default. > > I think I disagree with this. Since we don't have anything like > laptop-mode AFAIK, I think we do want it on by default. For the > server workloads that want to consume more idle power for faster > response when idle, I think the servers should be willing to make this > change, just like they need to disable overly deep C states, etc. > (Admittedly, unifying the configuration would be nice.) I can see two reasons why we don't want it the default: 1) Changes like this has a tendency to cause issues on various types of hardware. How many NVMe devices have you tested this on? ALPM on SATA had a lot of initial problems, where slowed down some SSDs unberably. 2) Rolling out a new kernel and seeing a weird slowdown on some workloads usually costs a LOT of time to investigate and finally get to the bottom of. It's not that server setups don't want to make this change, it's usually that they don't know about it until it's caused some issue in production (eg slowdown, or otherwise). Either one of those is enough, in my book, to default it to off. I ran it on my laptop and saw no power saving wins, unfortunately, for what it's worth. -- Jens Axboe