Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752305AbdGEP1Z (ORCPT ); Wed, 5 Jul 2017 11:27:25 -0400 Received: from g2t2353.austin.hpe.com ([15.233.44.26]:48868 "EHLO g2t2353.austin.hpe.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751765AbdGEP1T (ORCPT ); Wed, 5 Jul 2017 11:27:19 -0400 Authentication-Results: hpe.com; dkim=none (message not signed) header.d=none;hpe.com; dmarc=none action=none header.from=hpe.com; Message-ID: <595D0537.1080309@hpe.com> Date: Wed, 05 Jul 2017 11:26:47 -0400 From: Linda Knippers User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 MIME-Version: 1.0 To: Dan Williams , Jerry Hoemann CC: "linux-kernel@vger.kernel.org" , "linux-nvdimm@lists.01.org" Subject: Re: [PATCH v4 3/6] libnvdimm, acpi, nfit: Add bus level dsm mask for pass thru. References: <293ee143dcabb386ca06b384a384171c256a2ecc.1498810220.git.jerry.hoemann@hpe.com> <20170701195805.GA13259@anatevka.americas.hpqcorp.net> <20170701203813.GA13574@anatevka.americas.hpqcorp.net> <20170704200849.GA15713@anatevka.americas.hpqcorp.net> In-Reply-To: Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit X-Originating-IP: [66.187.233.206] X-ClientProxiedBy: CO2PR04CA0106.namprd04.prod.outlook.com (10.165.94.160) To DF4PR84MB0076.NAMPRD84.PROD.OUTLOOK.COM (10.162.192.153) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: a59799be-5cc9-4abd-0417-08d4c3ba514a X-MS-Office365-Filtering-HT: Tenant X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:(300000500095)(300135000095)(300000501095)(300135300095)(22001)(300000502095)(300135100095)(48565401081)(300000503095)(300135400095)(201703131423075)(201703031133081)(300000504095)(300135200095)(300000505095)(300135600095)(300000506095)(300135500095);SRVR:DF4PR84MB0076; X-Microsoft-Exchange-Diagnostics: 1;DF4PR84MB0076;3:CxeAq3lCHMQ+awy1UzQU5Ea+t+meFY/YVjImcwPIE9xth+6qdyyQwRs/4dVcAKxocnivbILai/MIatIXoDbuyJUiKPD/83u/xp2pdyJAwhamErZutVhEjR8M4ANAEb3OqNbjMAdWE+t8KaAAnUWVNbD69WBEYl2evz/FM7aqKpjcmnsZxj5Gao/rlq3lRjHQjai44eFwRQJkJUIPjb6EFaVb37tYdR55BpUrTdejAjqA8IR5XY85DBw2CINpQUZtJu2KWtjVtl/G//TQJAVzDQX3lxPHzccQRVBMRhl1r7RJWOY+I00mrpEw5W8FymJmcWIz0lU3nS8uq799+04OZ61Qj1ZekHESC1MqArmgNRh2HFrbva4ZX4nfe+yA913qGAoEjEeXmHmjp9p9jVXJaWnA+u3Oyb1cV9G2zHoztXIbpnlwbCyMn9zZSDTM3RDNIdrfklTjOjykanp8dkbPzzZ9sBLPXJCuMNtHTZxCUjmZnKxsZqtiJBfLg7z0GLrvzCJO/6R7xBYuUlqzrBSilKPjFsJAEsgeSNu8S4rQIVcRMCnD1nvivbncNMctBsXfpelCTutezUPg+rukWhxHPkkLDiQyAuVmiTpbwLHoOlMiWmLJ3M2U1qJn27iPTEjrHCkTmLHheQc7gjCwteFQ6978WV6RHB0o0z1BQibrAvjOVu6lXkAvY/Ffu7E4LXEb0YhbP9kfho7aegE/MlLfZsoqzGaKvyz5do/fLdSgpJQY2Zeqpz2PrGs6vvuuSDTA/j7f/+KCZZwJywEm3IWNoQ== X-MS-TrafficTypeDiagnostic: DF4PR84MB0076: X-Microsoft-Exchange-Diagnostics: 1;DF4PR84MB0076;25:kN26C8V3j5IXvcdvb1hihcV8pVb2DAVp+8C+sDhiXGs23moa1jCnYSy/Ut1NtzeJZEHXoqCjN0vk9pcCNihiLt33c/EoJzRMkV3pgBC6/qdcLsWiQ0XTkBByupiPdj64jUEKDW+9u/bop5OXDundR7PrpTo00zuXDMxDO78skly8xfsSU8aF6MKa0VcSmhGAb+Swit/43WPxZ9wgWDDLwTnTI3WUZPFyXwVXTMeSIyPeXFndxZfpel0jOuEXfkRuu0FSyaHXZLaXFpzJUEIBUJVmdq78RAeNv2v5J7T6YSIMVvM6P0KxvTMmbRvcWdagNzzZowVQ6tzDXGk6XaEiSOMyJIxzz1W0bJfLK05ep1mvK/beYBQsLPYd9J4P3XxGq7G/L5kWvXCGt7AHQQspIe+mPh/2d6yA9gt5l6h1i9vNBK+PwCMB6S/Az0iQ5XNRqqrqN8YIhQXpKs4IiJvPf/Yr1U+Og32K/VuKf5jecJ8NrTkLcoGBT14eRF4CiBoLJVz7tzzHweDCc64yeVD+x6tcqG/gCKaGV9iE4skV62CVjvGRyLPYUVmtJPcL5YVlWsuLH2ttx+Dj1P7FKhrsRfUX6YjLlaVGtmDJiURbHwiyyrCEM5v0hVddHhc4pu2QOcIgpeaPGvGpDxpNIls/rbx1ieI5cimT6pDP8uNqoHfhwIGHr0LaA+y8bu6TKjaWu92z4YTzrAL+LaoIUwEJgcy5VI1yJ264Gduu7VNKfFyvPN8+6YOnwcqJa8Ul/F7dC/AcuWMKHH0UlDRxYW9VVIrycvc4r3nq1n2pgFKj2m51SVqIcfvnRnuPw5MOevuLhbYWj0GaPQx3TwbSeR6yfyyazTYv0sHdW4JUCXMwO8CkTTlj6fcx4AVRdsfjIszz9zPCBp7y6yYBaj4cprw8P7hnFMhBcn8SuzEAKRm7QZU= X-Microsoft-Exchange-Diagnostics: 1;DF4PR84MB0076;31:2IPnSU/+bXhD5+/H0abimCzL8bCAJ7dI+9/lxA+xUuliy7IpZSfqbyvbpGgCCyr7ByeVYV+QIq2MxAXr2/z3yzGnuJa4aEHu95ih5X7CrruhKExK3eS9/1nYfC41KOjVTdm6in+7v0HQ8hwIJ8TUntaXR55o+Yqe+8MF8kUi0bV81GZ0Fq0uiWBx5suv16XL8juUYmFshafrMTtsTSk10c0W2ldls9WUy55HXBFX3NW7jaf18474dw6fue3tyeBYFQbnPAtNFEVMXPehdSdfIMQljnCNuUb3tMHbJRHbiXJadt+iTveRVupQnUYbYzQTplLetYto+jRhMgj5BoXCMWSQOAY856bSL0drUpCRbxy4hthURqmbZkxu6cuQNToTUsRs3p336/zVTZfkqUxhXeM7ysEjujJLvhqEqooiQ2P1M/i8bxipHc3wS45vBz0jPUm7KU7LEUoRYHGSapulYSAA1mu3O3vqV9JFxTVVFeDgdOIi8z9k57DGPKuIL4MvALzhfeyw7gL5T6G6W3N3bHaez8gjyZ57EMDzvKkoaU/AoAC9kPGR/CM8CPjMiRH+JzSunKbyy4nsEKJDfxSmc6rs/9ZkfNxwcTXUn3S9/wvWSsIZgX9kQIx1VupbtEGxBI8cFqOf4JhPXKGi2QPG/UVIyFsniU2uaJ1aM1y6498= X-Microsoft-Exchange-Diagnostics: 1;DF4PR84MB0076;20:wFLpmlw41LCBYBMpuMtNSex9jKYd0UDEYlXDJOfwUeW8BtgNsYkii7vi43WRrYLUudmN3yCmwOiFS/LPX9vXRclqlSHKJETEgVxo98tn9uYBQvt75j1sk4zAj5ves3iE0EfeH4W6eZbHXxh+PXcOfJ+exe2jGXFoQ/l5YqIaOpiDQ/2EBLGsXSmyeftVZn/UMmXsODWWGXM4iEgrc9yxJJ+sV07SEEnvOjFP6utWwtcXpIYvaDdmdy+0JsF4faNJyJEj2lJasLpQaPwnlfApt6dWOCP0VQJEPxGhW9hI+n7CpSYPaFpqki25cxElSc6NzEnfy3T5+pyNVK/ciIh15FF0keKdvV6uDAYuJNchpxVDlUlM30ozgIgxGUqK5yX6GQ8Dj66hMeJ/1ZpB8SOy7TghP5HyEjG0t6M8/hmQMKfaMgGkIDotbHRoMBgrEe81Y+IocmD0xWWN9pRSywV43hT71xMvEznn1sMMxeSO9LwfAntdgvRxYWUAExLGybH+ X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:(227479698468861)(133145235818549)(236129657087228)(48057245064654)(148574349560750)(162533806227266)(228905959029699)(247924648384137); X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(100000700101)(100105000095)(100000701101)(100105300095)(100000702101)(100105100095)(6040450)(601004)(2401047)(8121501046)(5005006)(93006095)(93001095)(10201501046)(100000703101)(100105400095)(3002001)(6055026)(6041248)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123560025)(20161123558100)(20161123564025)(20161123562025)(20161123555025)(6072148)(100000704101)(100105200095)(100000705101)(100105500095);SRVR:DF4PR84MB0076;BCL:0;PCL:0;RULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095);SRVR:DF4PR84MB0076; X-Microsoft-Exchange-Diagnostics: =?Windows-1252?Q?1;DF4PR84MB0076;4:CfV4fHEIp+YQ3k+XZRmdVX5cRLHeQJvjEkNtJq?= =?Windows-1252?Q?0JZ1fsYIatzaiwR1UErhePoFFYCUMNETJeANclmZ/nfu+P2MNZ5orfeH?= =?Windows-1252?Q?asqylJoN1DX1Y/g22ewyvH5C29KMwgWFjkqlkl1y/cDGQJR/Y24fNSnz?= =?Windows-1252?Q?JI4bh0HLwypW8okYF6M+e5hS6OBf3eJaBWNZkimwTltu2tif5uwBvcTC?= =?Windows-1252?Q?rNvtpIlnc0U5ppiPt/PAw9yWFlpEQsCLXPLaPqoiJedABjiAUhXM5+nJ?= =?Windows-1252?Q?4iNJ6YLuldJOJCNVphkD0vr/3FqXzXdZ5FurBrLcKCnvHKjxXYj/Kwze?= =?Windows-1252?Q?a/7wrf/G76HkL7ePIatMmO1zgUxlz6A327oJtvRQ/Bg/N30MCCybAU1q?= =?Windows-1252?Q?LwaSHFD7ohCUHBwW6YkrjKZ1N1NDuYKn9AHKlz7Jbmv1jbbTvyGGQQFd?= =?Windows-1252?Q?F0IG+L3q+pzWJBsDpj4lEYcM2AufhrQlOBs2UEnfbHioRal3qdL9IMCm?= =?Windows-1252?Q?psEKCpCTRP5jmeDEtskOy7MHfbsDwdAX2Fn2gHRDQ5MtLBd83pSc0Nd9?= =?Windows-1252?Q?6JmOuF8HlVI9qnAxQgMqw06qrzx1Ze3s7MU5n4H3gMeUjE2xAAhgC7y3?= =?Windows-1252?Q?Eui+e1Lxss4lWqu+mtsv0tpKpfQfLfGVOSBrKy1Ss8p/U6qD4rfkOy/K?= =?Windows-1252?Q?BHTadYT6B70F4q/96Fo3BdLunvTf0s858bkb1c7td63UGIsLxVWzU9C0?= =?Windows-1252?Q?YKfuamMj6KXu0qm8upfxhjKO3JY+QObiWmJjOPC1bKy6HG4SolOJclAw?= =?Windows-1252?Q?oGoS46x0sXuykusw8mR7GjZI3PGhYh4J0ltyscFznIN0CZJz2u4pHjw0?= =?Windows-1252?Q?GVxJfHA5JjT8oZjCed97w2RSOMtEVcTbIVtB5As0g8v+kXX3WXU+hVbB?= =?Windows-1252?Q?xRTfw3Vj1czT+/ga42hvfpIGtQLBeqVGuhOLjJJD2r9qj4U9L+rS0LVs?= =?Windows-1252?Q?haiEaBsyCeK65MvrYv/PmvaQrRsFKOZ9Id5VhAOwmdU3XY+BsH+M46sI?= =?Windows-1252?Q?mDxq6DlgduX7NeA0YjdGbxtV6yFg6pJfnLYqlnSLZMlCKrXGA+tCcYJv?= =?Windows-1252?Q?2Ze5hjiOPqfDVqDFCeftAgkveBZRhwGWM2w7TQLw9NarNP3VVITSQJ+3?= =?Windows-1252?Q?x2Ln4W+GlYLi0bk2Hfe2q9X6g1KBcK09bAznBQHsusPW+oWfq7tIgiO1?= =?Windows-1252?Q?PfVbPG+lEs1z6zqfZAbVd7u6oSzES0sCCjJYDu5wt2z+VjBFceqs+/iv?= =?Windows-1252?Q?70GRP1zBBe4tDY6iSa/8TmwOLfJ7XKj2oIh+CiWtIvnkxyHLv5hS4iUW?= =?Windows-1252?Q?5pXa6DxdlbxInLFVrQ7DlJF1abepYql4lnIy7dCllgRYLYf7e2JIde5R?= =?Windows-1252?Q?JJ0vZ8bohBbTKuFNYxm14ZVjfwqBB7maosu9Z8Dy/mZfIm1HAnuf55pe?= =?Windows-1252?Q?34Tkhj0G3SyCdp31jYHI6TbjnR9foiEYTc0/NwB6n7bQUMIo5W801y0U?= =?Windows-1252?Q?6x4BK3dpQSfEQ=3D?= X-Forefront-PRVS: 0359162B6D X-Forefront-Antispam-Report: SFV:NSPM;SFS:(10019020)(4630300001)(6009001)(6069001)(39840400002)(39450400003)(39410400002)(39860400002)(39400400002)(39850400002)(24454002)(377454003)(229853002)(38730400002)(6666003)(478600001)(50986999)(6486002)(3846002)(6116002)(93886004)(6506006)(80316001)(47776003)(65956001)(66066001)(76176999)(6636002)(305945005)(83506001)(81166006)(8676002)(7736002)(966005)(5660300001)(86362001)(54356999)(33656002)(53936002)(65816999)(59896002)(6246003)(6306002)(54906002)(2906002)(6512007)(2950100002)(53546010)(4326008)(4001350100001)(25786009)(23746002)(230700001)(36756003)(189998001)(42186005)(50466002);DIR:OUT;SFP:1102;SCL:1;SRVR:DF4PR84MB0076;H:localhost.localdomain;FPR:;SPF:None;MLV:sfv;LANG:en; X-Microsoft-Exchange-Diagnostics: =?Windows-1252?Q?1;DF4PR84MB0076;23:6zngEuQnsyIx+nyVswLU3JZS0WyVlBxmJ0KYI?= =?Windows-1252?Q?UKuxVkyPF6B5QU2AYDBpOt6Gk2aPiKDg3E425VnEHRmFe6qRZqh8q4MV?= =?Windows-1252?Q?ObJBeZp6aCQcnnY7RcaKKOgPoGnk9ZezeWteFvS/ZEj4FqM2Fq2d2MZV?= =?Windows-1252?Q?6+TT9bVAg3Catp0bfXCnU8bSxeMlMs9S/2w42x1DiG19DJtRZqrtLyPE?= =?Windows-1252?Q?/k68k2O6z4/e9SP+/hAVlS5kP6R5acub8yqFtRNPLwQfmQjeGwB9133V?= =?Windows-1252?Q?sXqjJlDBgKQk0+Wcu/35W7yFypMoU8hCoHwzx9eodbUr553O1leax1Ly?= =?Windows-1252?Q?YViJN44hXShKtVA1QIT0lS9Omfaj5Nx0IYlL5yOAB6purhXSWQ6N0n6h?= =?Windows-1252?Q?YENit0VtNFmcoiHHTRPNtwH3U/NjKvTw6axCPaT2D7hcXmzMpFiXA9S8?= =?Windows-1252?Q?GE6N8as+d7fWSOwOnxnDrWD0QYUIegPax8SoFU12mkVJcIuczhktW/BS?= =?Windows-1252?Q?k+q9EuPj5H0e3Lyf6pAeMU0VIxAeL7F8QKGH1dV5w8AYUSn+siSueevU?= =?Windows-1252?Q?IQiq9fX6qar/HCdZwpyesFKs7jJtO5psR0eAI+T/oydjlXxrnbfcgLdq?= =?Windows-1252?Q?v16kyHfXOuL5TTDVGN0FBRFLbgxDEtjg92yNSyqZh3mi2NsCtihdXYzk?= =?Windows-1252?Q?/1/54X75QYfrrdANd9mLOTe55IbckK3uYCmsz9TL9d79iOM70MQaPDmn?= =?Windows-1252?Q?08jyLV66+dN1dm/uTIW/8bmsOriJaTmNBMAcG9OVoDNtGCn0Q2bqkmc/?= =?Windows-1252?Q?7+74zgRzr1NbF4kezapcXZEFBtUQU191rTq3n/9sIBaGlUi5iFLnQtbt?= =?Windows-1252?Q?U9RZbJR6MOMxjxiU29b/2+ImRjTFPgju8g5EtA9pITwtU7EFJbQm+nGy?= =?Windows-1252?Q?n1M9RNwaQcsXdH9StMFknPyuKGNWFPFnRBMMmJIEPP0OxYlR/zv+Itap?= =?Windows-1252?Q?pzvGompQxwQQmRJjBF52WO7cwiE+4GzzLL5D6BAGoyc303chbv8b+BH2?= =?Windows-1252?Q?cQ8W8/S+A9CecBFO0aUz8Rv8zxCYFWf62MrWkYXE8dsdFDsl5kh99OEf?= =?Windows-1252?Q?ONfOIBPCrOiu84TjanaqC4Hx2uygo+TPd/WE4U30jS8IW61BUEix+lg4?= =?Windows-1252?Q?cn7MuaXHx7rbybIO86nFUVqqsO3vSALDcUOyb20G/hmDj755xuhZPm8d?= =?Windows-1252?Q?9mAP0jDLnxnukPfBIn8AEo8LO8GmQGy46I2XzRiODNF17YosKRzH0R5f?= =?Windows-1252?Q?Ho/QOPw31Gm7i9Aagj4sRkL2tLDCzQQyD6TOVrhedyVG1gobT5ra2WTX?= =?Windows-1252?Q?SgLfqD3G9SfTNlRPmOZm9gUZ4ABl0OqX63lwNCRfj9doR3xv1aVas+FN?= =?Windows-1252?Q?OgNDgUJKHrTVyTWFE39uaSldsfSu9fah080R2wRyw=3D=3D?= X-Microsoft-Exchange-Diagnostics: =?Windows-1252?Q?1;DF4PR84MB0076;6:fO0dm5FmVfHctn0TzQAbwOUNqxSiskuuesHLNM?= =?Windows-1252?Q?PjGs9ONEYxzFrorwMKmwM8Y+WqdRqhSOs81rlEBVg/ohOPZnEzUAAnpb?= =?Windows-1252?Q?m+zse1Ux4oANDcNp3I1Ygs9GXQCApzFmxelZK2RS4c5NZ9ouO2UxEq/h?= =?Windows-1252?Q?jsF8DuE4ia9jUrNJidcz0O7v/0539nZfcEbn9kHJR7Ak7a5uRGNCWu7Y?= =?Windows-1252?Q?oeNH9bZ2uY9ZQqXocanA4Ti9DqwcslxpHhCykT+Brpxc6gZKuNTMG56H?= =?Windows-1252?Q?TtQVZguq1QgmjgQT+Tfu+phRtKygVTWJQDokI4iKWCzmwtVHUwtW25fl?= =?Windows-1252?Q?sfA2Xa1hCrAf4oiTJ5YE2EBkVkmUfAGN1mnEntuF0uy5G826MjEFOX79?= =?Windows-1252?Q?87/AV+/jIj99zQpJ9derO43wFwahs9vsLYOcKStwz61dHXcNlWG9zCEB?= =?Windows-1252?Q?/G6fuDjL7I7X+Gy/16JRWZ1CzwOdJ3sFGa4vTlZ10wMAxxNjxwM75RqQ?= =?Windows-1252?Q?JmBMBh89SRWmEiHkQi2y5Z2hwgX5EjKOlUt4EOIiGn0ee/uZ/JFwwiWa?= =?Windows-1252?Q?4Q2q9Dfq4MHJzUAGiwedoDLLtpzMlM5PjlkuCXfaG5CJ5+cNLejDLjJF?= =?Windows-1252?Q?UuIV8zm1aMsSRA3QfL47GRoLgE/Zh14lLTfxB+K1N2AlrhCz1c7R0j+J?= =?Windows-1252?Q?ar2siG60K3kI5iNNTqFbMapMbbFKp2fJazL5m0n1UkvljTuAVBH/9RdE?= =?Windows-1252?Q?B6mTaP2GIOrSkjX2vZ36jCMsk7aYUK0KGrTr8/dMJvJykehXqYzBszrW?= =?Windows-1252?Q?9RZAF4snUUCjiiw3u6VQ2H/7WGD49QCItg/Ol3vnBv40Gef35N0+diqK?= =?Windows-1252?Q?2ZNKhQTLJpCAOvz3ndvEwj5cDCTArr0oCDbaByaLTqpcptybQ+qxsht0?= =?Windows-1252?Q?gmK++yOYnECLvnPmXGWjlEPrTNDhsz0cnJL9qXC1bBF9xxciQOPhUdPr?= =?Windows-1252?Q?kboLJYFx1YSGGQvK2RQU4lvKM80+hnv3GpGx/QL3haj4WJ/QqmiL9I73?= =?Windows-1252?Q?HQjPaxFDVo6Z9MVwwx9dOfXCZdI9sNvsqa?= X-Microsoft-Exchange-Diagnostics: 1;DF4PR84MB0076;5:W0aTCPYMutDDk2N0gL9OnupMNQ05ts2D8WLQRoJAHjdu9ITDNSoZmwIM7LC6p+BwqPbpua5rxLzaHOJIe2sOYL1eiC3jbd8Uz5vmlU8mxXtvjW1LK+9TqaT/JOvGWTRGDiBoaOCQo+bcL0lpg9Hbfxt1XllBp1hniWd7dPtu3++7PmKXCXOHOqypCVSoVWghbCHqqgu+ga+rvTqLqRnLL+Q4q00LxN1vL2Nu0bKIGIr03EloHmCu4pF89cGejRqkHjM5zhpWrR12+jjVPyHD7A1KL6+71pwmGjJxWra9aAUUiNM6so1PNe9tPEKokVDvDE2tBcCPrZYJYG8OA77EjtJJHWleBHEWQgbO3UOcpWA/OtUMF7d0LOKgyMeZDcAiE2eDg+aVd9weiTm9I7HfFHystupPPWtnIIt+NrFr+Pbt2N6+iA2/Yc1BDPm7DXVvSYUnNXW4YXYt1rwJkjFCB0XnCSwQyxnJAVMwpjL9+BiRL1YO6MMsYtoDv5J5OKqq;24:9aa4ec8LvTo78DHjUO9SkSxNbGowQhe4+zpNXY2rL5yj5fjDXqYcpFq52yayuU14PLmj+Yn8HjcnpWFiLmv1+PMuz9yfrtdeU/T63xule24= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1;DF4PR84MB0076;7:sG12fmEhBVE7FBasO/eZ1YI672LKQJBI3dkeptqDYkNNZGXu9Tkq4R70vEBzpPRwZehbmIOPQhJKtI5W1lY8rfEL6zcXEKB1HyqmuaTxpsxiYRpqyPbIBmfdzJvnFGsir43UhzDwuXaDVNPTYqDBDN8SEaVjq12jFbAp2qOXlak1FoNaEOcXxUTd6fxcz9KGPzPAk1q3Vby9QaX+8ts4G6XCYNIYcqEojm6DFW4SeQd373y/ZjHKFc4Z5Kfhy8JRhYh7QF4Azz8KYlWBHGxM0m6CGjG4kLc2FxQ9LCGIH+nSXvhqhIN7myZdg1m+a4fZDlWigUPeoQRNUsun7QFId07PSr7V1uYbCgRb3RJ1yZRsmeQkAMqdTQph/6ddmnC1qiEtz4NrscGbwapQoPh9QSxRwZPOIWXbFW5AF4KOQVgi9US8cSvI0PkuZlH7ZTO8+7VK3MJdIDwI6pF6bT/odllPQzuuZx7xQqpAHFnsWfcbhymlmWv6uj+Fn7ufkRpJHDFboaNMJC89UYc180LVibW0NzhXO/ZfGWofa5GfU7jzyF9a7Jad/7cIr+anMjL0bjgX+//QGAsg/nSjengk9B0VjY0c3SiRK0VIB6EIcUnNFfpRpWnlLGFTCnhF9HdrGwCLxTSgHx9pDINGxKoR1ccnhXSa8VB/W8fvkFwEzhZtYA8Xhw673k62W9K1Ru0tXJSoXkcUFhdpsjhIntIR0RR7U0zLDHhACXqisc7OtuCHWesukHe2nSbO1xRE9cHg56NAD+uZTUzxrL1LIbL0oAcLs400IL3PNxX4f6cfb04= X-MS-Exchange-CrossTenant-OriginalArrivalTime: 05 Jul 2017 15:27:14.6532 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: DF4PR84MB0076 X-OriginatorOrg: hpe.com Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 5655 Lines: 122 On 07/04/2017 04:37 PM, Dan Williams wrote: > On Tue, Jul 4, 2017 at 1:08 PM, Jerry Hoemann wrote: >> On Sat, Jul 01, 2017 at 01:46:03PM -0700, Dan Williams wrote: >>> On Sat, Jul 1, 2017 at 1:38 PM, Jerry Hoemann wrote: >>>> On Sat, Jul 01, 2017 at 01:10:31PM -0700, Dan Williams wrote: >>>>> On Sat, Jul 1, 2017 at 1:08 PM, Dan Williams wrote: >>>>>> On Sat, Jul 1, 2017 at 12:58 PM, Jerry Hoemann wrote: >>>>>>> On Fri, Jun 30, 2017 at 08:55:22PM -0700, Dan Williams wrote: >>>>>>> >>>>>>> ... >>>>>>> >>>>>>>> On Fri, Jun 30, 2017 at 9:09 AM, Jerry Hoemann wrote: >>>>>>>>> + if (cmd == ND_CMD_CALL) >>>>>>>>> + dsm_mask = nd_desc->bus_dsm_mask; >>>>>>>>> desc = nd_cmd_bus_desc(cmd); >>>>>>>>> uuid = to_nfit_uuid(NFIT_DEV_BUS); >>>>>>>>> handle = adev->handle; >>>>>>>>> @@ -1613,6 +1615,7 @@ static void acpi_nfit_init_dsms(struct acpi_nfit_desc *acpi_desc) >>>>>>>>> struct nvdimm_bus_descriptor *nd_desc = &acpi_desc->nd_desc; >>>>>>>>> const u8 *uuid = to_nfit_uuid(NFIT_DEV_BUS); >>>>>>>>> struct acpi_device *adev; >>>>>>>>> + unsigned long dsm_mask; >>>>>>>>> int i; >>>>>>>>> >>>>>>>>> nd_desc->cmd_mask = acpi_desc->bus_cmd_force_en; >>>>>>>>> @@ -1624,6 +1627,11 @@ static void acpi_nfit_init_dsms(struct acpi_nfit_desc *acpi_desc) >>>>>>>>> if (acpi_check_dsm(adev->handle, uuid, 1, 1ULL << i)) >>>>>>>>> set_bit(i, &nd_desc->cmd_mask); >>>>>>>>> set_bit(ND_CMD_CALL, &nd_desc->cmd_mask); >>>>>>>>> + >>>>>>>>> + dsm_mask = 0x3bf; >>>>>>>> >>>>>>>> I went ahead and fixed this up to use dsm_mask defined like this: >>>>>>>> >>>>>>>> + dsm_mask = >>>>>>>> + (1 << ND_CMD_ARS_CAP) | >>>>>>>> + (1 << ND_CMD_ARS_START) | >>>>>>>> + (1 << ND_CMD_ARS_STATUS) | >>>>>>>> + (1 << ND_CMD_CLEAR_ERROR) | >>>>>>>> + (1 << NFIT_CMD_TRANSLATE_SPA) | >>>>>>>> + (1 << NFIT_CMD_ARS_INJECT_SET) | >>>>>>>> + (1 << NFIT_CMD_ARS_INJECT_CLEAR) | >>>>>>>> + (1 << NFIT_CMD_ARS_INJECT_GET); >>>>>>>> >>>>>>>> This drops function number 0 which userspace has no need to call. >>>>>>> >>>>>>> Actually I like to call function 0. Its an excellent test when >>>>>>> modifying the code path as its a no side effects function whose output >>>>>>> is known in advance and instantly recognizable. I also use it when >>>>>>> testing new firmware. >>>>>>> >>>>>>> What is the downside to allowing it? What bad things happen? >>>>>> >>>>>> It allows implementations to bypass the standardization process and >>>>>> ship new root DSMs. It's always possible to patch the kernel locally >>>>>> for development, so I see no reason to ship this capability globally. >>>> >>>> I don't understand this comment, but I think your next comment >>>> essentially says to disregard this comment? >>> >>> Yes, sorry. >>> >>>>> Actually, just the discovery portion does not lead to this leak, but >>>>> it's redundant when we have the 'dsm_mask' sysfs attribute. >>>> >>>> No. The generation of the mask in sysfs is not done by >>>> executing the code in acpi_nfit_ctl. One of the reasons I call >>>> function 0 to test changes I am making to the ioctl path itself. >>>> The sysfs has nothing to do with that path and cannot be used >>>> to serve this purpose. >>>> >>>> And since the content of sysfs has been edited it also can not be >>>> used as a basic test of firmware. >>>> >>>> What is the downside to allowing the calling of function 0? >>> >>> It needlessly expands the kernel ABI. I would suggest, if you want to >> >> No. It is not needless. It is not an ABI extension. >> Same goes for the override feature. I have never understood why allowing function 0 is considered harmful. It is a standard function defined by ACPI in general and specifically for NVDIMM Rood Device _DSMs. It is also defined for each vendor-specific DSM family. It is not an ABI extension. It is a standard. > If the need is testing then we have a tools/testing/nvdimm for that. > Of course it's an ABI extension, it allows userspace to discover DSM > function numbers the kernel didn't know about at compile time. It also allows user space to determine which DSMs are actually supported by the platform, which may be a subset of the defined set, in a standard way. Exposing information only in /sys just makes it harder for people writing software (tools, tests, whatever) that need to support more than just Linux. >> I hope that ACPI doesn't extend the specification in the future because >> we'll just have to redo these patches yet again. > > Hopefully this is the last ACPI spec version where we add new DSMs to > the root device. I wouldn't bet on it. > All future methods should be named methods like what > the specification started doing for NVIDMM leaf devices with _LSI, > _LSR, and _LSW. Those methods started out as DSMs for a specific vendor and then became standardized. It would not surprise me if that's the path that is taken as new NVDIMM technologies evolve and new functions may be required. It's not always clear on the outset what should be standardized. Aggressively preventing extensibility, especially when it's actually part of a standard, baffles me. -- ljk > _______________________________________________ > Linux-nvdimm mailing list > Linux-nvdimm@lists.01.org > https://lists.01.org/mailman/listinfo/linux-nvdimm >