Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1762587AbcLSQhz (ORCPT ); Mon, 19 Dec 2016 11:37:55 -0500 Received: from mail-bn3nam01on0125.outbound.protection.outlook.com ([104.47.33.125]:48920 "EHLO NAM01-BN3-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1756209AbcLSQhy (ORCPT ); Mon, 19 Dec 2016 11:37:54 -0500 From: Matthew Wilcox To: Andy Shevchenko CC: Heiko Stuebner , Andrew Morton , Matthew Wilcox , Zhang Rui , Eduardo Valentin , "linux-kernel@vger.kernel.org" , "Mika Westerberg" , Vinod Koul Subject: RE: thermal zones break with patch "Reimplement IDR and IDA using the radix tree" (mainline+next) Thread-Topic: thermal zones break with patch "Reimplement IDR and IDA using the radix tree" (mainline+next) Thread-Index: AQHSV+EusKhq+NTzpU2L+9hvpr85sqELExFQgAMLI4CAAV1l0A== Date: Mon, 19 Dec 2016 16:37:52 +0000 Message-ID: References: <1758112.VgMJOyEgSu@phil> In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: spf=none (sender IP is ) smtp.mailfrom=mawilcox@microsoft.com; x-originating-ip: [76.10.168.227] x-ms-office365-filtering-correlation-id: 91f256dc-aac1-4ace-2037-08d4282d6089 x-microsoft-antispam: UriScan:;BCL:0;PCL:0;RULEID:(22001);SRVR:BY2PR21MB0035; x-microsoft-exchange-diagnostics: 1;BY2PR21MB0035;7:BV+kvKbQiE8B2dKjrvg9xQbGcXuB/I3PIk2i8o6MddHbIvIBSyLI2eeyzoTjvMpv2AQWA/jK53lORjO2bUbqLwC8B65z6pHfwo3ry2FF1ENGRbF639BtLyadJQ3bVCEQb0upnrV8gKz/H9OVDn58UIZyiDYpFYnO2CAZ+u+WfsKqj24v4kjXtt4CxTPHZjkQxFtaP7tgjYiern60LHrPYgViNp6ZgY23ilK0f9stKEfd+glkA+cUeuGJpHhZdHAyNkDn1AgQvWLWjjV9b7ieji7r2HbAu0hhHcC+00nrI+VPTCBdqv7dVdvbw2kz71oh4gnwspbCrLxircg33BuPiQXYjN9wlQEFd2gsbcXCAVXrMpE7uk3lfHNWiDkOOVABaBI93+Fw0vapse7kcyH4cSVkSt/LluxVPuVzr9h8ZnXLAOsOtHnG6iF3hLrXH6nrX2yF0sSduLIWolBmaYAoYqrmfyHsyG6W6C8hqB2O7uE= x-microsoft-antispam-prvs: x-exchange-antispam-report-test: UriScan:; x-exchange-antispam-report-cfa-test: BCL:0;PCL:0;RULEID:(61425038)(6040375)(601004)(2401047)(8121501046)(5005006)(3002001)(10201501046)(6055026)(61426038)(61427038)(6041248)(20161123555025)(20161123560025)(20161123558021)(20161123564025)(20161123562025)(6072148);SRVR:BY2PR21MB0035;BCL:0;PCL:0;RULEID:;SRVR:BY2PR21MB0035; x-forefront-prvs: 01613DFDC8 x-forefront-antispam-report: SFV:NSPM;SFS:(10019020)(6009001)(7916002)(39860400002)(39450400003)(39840400002)(39410400002)(39850400002)(199003)(24454002)(189002)(377454003)(54356999)(3280700002)(3660700001)(99286002)(122556002)(106116001)(106356001)(2900100001)(10290500002)(8990500004)(76576001)(7696004)(8676002)(4001150100001)(81156014)(81166006)(8936002)(110136003)(97736004)(9686002)(66066001)(68736007)(2950100002)(105586002)(6116002)(102836003)(6916009)(3846002)(189998001)(33656002)(2906002)(4326007)(6436002)(5660300001)(229853002)(7736002)(50986999)(76176999)(74316002)(101416001)(86362001)(575784001)(305945005)(10090500001)(25786008)(38730400001)(6506006)(39060400001)(86612001)(77096006)(5005710100001)(92566002);DIR:OUT;SFP:1102;SCL:1;SRVR:BY2PR21MB0035;H:BY2PR21MB0036.namprd21.prod.outlook.com;FPR:;SPF:None;PTR:InfoNoRecords;A:1;MX:1;LANG:en; spamdiagnosticoutput: 1:99 spamdiagnosticmetadata: NSPM Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 X-OriginatorOrg: microsoft.com X-MS-Exchange-CrossTenant-originalarrivaltime: 19 Dec 2016 16:37:52.2279 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 72f988bf-86f1-41af-91ab-2d7cd011db47 X-MS-Exchange-Transport-CrossTenantHeadersStamped: BY2PR21MB0035 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from base64 to 8bit by mail.home.local id uBJGc0ob022111 Content-Length: 877 Lines: 20 From: Andy Shevchenko [mailto:andy.shevchenko@gmail.com] > On Fri, Dec 16, 2016 at 11:19 PM, Matthew Wilcox > wrote: > > Now ... thermal is actually using an IDR when it could save memory by using > an IDA. Are you interested in doing that conversion? > > +Cc: Mika, Vinod > > Same here for at least DMA Engine (and perhaps another place in > thermal subsystem). Hey Andy, thanks for testing! Could you test another patch for me as well, to switch DMA Engine over to using an IDA instead of an IDR? This should save 576 bytes on a normal setup. The patch itself is here: http://git.infradead.org/users/willy/linux-dax.git/commitdiff/a4f4e8554639b3fba9a62c07ad0e2423ea1f5f31 but it may be better just to test the head of this tree: http://git.infradead.org/users/willy/linux-dax.git/shortlog/refs/heads/idr-2016-12-16 as I keep fixing bugs :-)