From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id DEA2EC4332F for ; Fri, 15 Dec 2023 17:16:13 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 346A46B07A2; Fri, 15 Dec 2023 12:16:13 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 2F5026B07A3; Fri, 15 Dec 2023 12:16:13 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 16E746B07A4; Fri, 15 Dec 2023 12:16:13 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0016.hostedemail.com [216.40.44.16]) by kanga.kvack.org (Postfix) with ESMTP id 070846B07A2 for ; Fri, 15 Dec 2023 12:16:13 -0500 (EST) Received: from smtpin04.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay01.hostedemail.com (Postfix) with ESMTP id C49EF1C186D for ; Fri, 15 Dec 2023 17:16:12 +0000 (UTC) X-FDA: 81569705784.04.42DEB17 Received: from mgamail.intel.com (mgamail.intel.com [192.198.163.11]) by imf19.hostedemail.com (Postfix) with ESMTP id 4F55D1A002F for ; Fri, 15 Dec 2023 17:16:08 +0000 (UTC) Authentication-Results: imf19.hostedemail.com; dkim=pass header.d=intel.com header.s=Intel header.b=FVpB0Rcn; spf=pass (imf19.hostedemail.com: domain of dan.j.williams@intel.com designates 192.198.163.11 as permitted sender) smtp.mailfrom=dan.j.williams@intel.com; arc=reject ("signature check failed: fail, {[1] = sig:microsoft.com:reject}"); dmarc=pass (policy=none) header.from=intel.com ARC-Seal: i=2; s=arc-20220608; d=hostedemail.com; t=1702660569; a=rsa-sha256; cv=fail; b=F794pjeRNYDja/LRwSo570bdA/O408jBec95XkH48KL8U5sxOR0HbXfpAiYv67jnXTQ+7P Uk2rKxQfDvv9+/25rpouaDuCakW1p4itcCeppzf4MQehLq3dUVn37IW8tEvnkYL+FbF5A8 JYYObpKrBsC+9KWv6IxxTJcb6PIFF2k= ARC-Authentication-Results: i=2; imf19.hostedemail.com; dkim=pass header.d=intel.com header.s=Intel header.b=FVpB0Rcn; spf=pass (imf19.hostedemail.com: domain of dan.j.williams@intel.com designates 192.198.163.11 as permitted sender) smtp.mailfrom=dan.j.williams@intel.com; arc=reject ("signature check failed: fail, {[1] = sig:microsoft.com:reject}"); dmarc=pass (policy=none) header.from=intel.com ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1702660569; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=fjuCmNsC69+LsHa/VWzYsVQlVTP7f1Y15UkHBYV2Dgg=; b=tk4geys/NK23xIpskORhMJ4Viqgf/MQg7tS/JE2fG4EhtS9b11Y1yBzxDDQ4PG4Ps2c1Ge Zr9vbUxG0WJ5WKqkLkSvvcpA6GvIwdfuBz9BQgV8vv8n7YR6zG1yRLz241mDbHDuBE7uHW 44s2jihWkvr4srXTkA08OXL+0gxdQwE= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1702660568; x=1734196568; h=date:from:to:cc:subject:message-id:references: in-reply-to:mime-version; bh=DEs28JZ56AKLzE7D39b5v8ZJZb+zVrtJ7Xb7BiDjeTc=; b=FVpB0RcnqFxb1/whDAX5s/nppJcm9E0NdMIF6JXSfMkdbMWMpUHLWbtg 3scbEKmcO9oxu3uqK1954CILWs/U1kbH7cQrGPqkrtHRKXshs11agXxpy kWcJZ6nBXf8xY9lxuC9+OU54pDlmg5rgRyXmsBPm0FJxoffInbhkDXeFo Dr1UPyurqaYmaVxBCtJD+xpfsoYwy5k2KhlhYeS7TbAFiz8zeshsJms2Z nYQkTLzEgtdDxVoIrvhmr/m3tOejrWhlIdiFW3B4ZTq4rV6edabNYd5JI CItSVH9O3Q4cihBZXqWLXzx2x3tmq5Bsa6aJWqJ9/gj3CZPiLWVJnoA3k g==; X-IronPort-AV: E=McAfee;i="6600,9927,10925"; a="2125412" X-IronPort-AV: E=Sophos;i="6.04,279,1695711600"; d="scan'208";a="2125412" Received: from fmsmga007.fm.intel.com ([10.253.24.52]) by fmvoesa105.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 15 Dec 2023 09:15:56 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6600,9927,10925"; a="778328429" X-IronPort-AV: E=Sophos;i="6.04,279,1695711600"; d="scan'208";a="778328429" Received: from fmsmsx602.amr.corp.intel.com ([10.18.126.82]) by fmsmga007.fm.intel.com with ESMTP/TLS/AES256-GCM-SHA384; 15 Dec 2023 09:15:55 -0800 Received: from fmsmsx610.amr.corp.intel.com (10.18.126.90) by fmsmsx602.amr.corp.intel.com (10.18.126.82) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.35; Fri, 15 Dec 2023 09:15:55 -0800 Received: from fmsmsx610.amr.corp.intel.com (10.18.126.90) by fmsmsx610.amr.corp.intel.com (10.18.126.90) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.35; Fri, 15 Dec 2023 09:15:55 -0800 Received: from fmsedg601.ED.cps.intel.com (10.1.192.135) by fmsmsx610.amr.corp.intel.com (10.18.126.90) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.35 via Frontend Transport; Fri, 15 Dec 2023 09:15:55 -0800 Received: from NAM11-DM6-obe.outbound.protection.outlook.com (104.47.57.169) by edgegateway.intel.com (192.55.55.70) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2507.35; Fri, 15 Dec 2023 09:15:51 -0800 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=c/OYjHRP2Ryhq1LoUDdQennYdcmJ6HrI60tcpHjyIhh97KLwyM7XYYOtoVvIOXd2A7gxQ/Z/fNTs3xiC7QZ5JK7wXWp1MAgkH1vFZ+bqvnRBMXXVwvhz1dgsklyJ8wXp0RjWNGTlis5+Ispaq7SK4CYqZcRbf6PhWTLlJDJbb28yHKymAtyI5VPZEERZYNaVK8VmBa3gbCa+VwZzoLZjliuXS5LJl0EbIQELLWz+DHqxUqBFDpnZoXo7t0+cXupROWorOIaaHyUY5ccUB5/og325Lq7Jg2hdnxcWhM1Hfq7zKWfj8D58KWAiPSobNwEvxxHb3Pon1CIV7rwp5Oe90Q== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=fjuCmNsC69+LsHa/VWzYsVQlVTP7f1Y15UkHBYV2Dgg=; b=HR3XKYqHI+LHUZDck2TV8ry4CKKsd2y7HYPUlxkn3z7uDP6MbKL9dMyfLFEQ/WYhkHarZFsV8Bf9jmbTYRXDy9xxipZi3YACulRbVzqbBUcH66gWLG+SK2dIsxIH0vPlyhmNE7I6ZKpmDWqbsmLpNS0wym7jji+51itDkSeWmi00gWVnTTwAgFfmQaidvs5wUyb1rt349ud0llc2ZDpMD1xkUWKGJLgnmB1E3SyjV7SC0ypcGW8naffibY/3REPHAqfPTzB+F81N2q19+zlPVISHHuRZeHuTRi11YMejQ/pL2hmsPi3pObMDRrz+dkWUohrXOaMfljIV2BHlJzh8hA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=intel.com; dmarc=pass action=none header.from=intel.com; dkim=pass header.d=intel.com; arc=none Received: from PH8PR11MB8107.namprd11.prod.outlook.com (2603:10b6:510:256::6) by PH8PR11MB6609.namprd11.prod.outlook.com (2603:10b6:510:1cc::16) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7091.28; Fri, 15 Dec 2023 17:15:49 +0000 Received: from PH8PR11MB8107.namprd11.prod.outlook.com ([fe80::6362:763e:f84b:4169]) by PH8PR11MB8107.namprd11.prod.outlook.com ([fe80::6362:763e:f84b:4169%5]) with mapi id 15.20.7091.028; Fri, 15 Dec 2023 17:15:48 +0000 Date: Fri, 15 Dec 2023 09:15:45 -0800 From: Dan Williams To: Greg Kroah-Hartman , Vishal Verma CC: Dan Williams , Dave Jiang , Andrew Morton , "Oscar Salvador" , , , , David Hildenbrand , Dave Hansen , Huang Ying , , Joao Martins Subject: Re: [PATCH v6 2/4] dax/bus: Use guard(device) in sysfs attribute helpers Message-ID: <657c89c16d5f5_269bd294bd@dwillia2-mobl3.amr.corp.intel.com.notmuch> References: <20231214-vv-dax_abi-v6-0-ad900d698438@intel.com> <20231214-vv-dax_abi-v6-2-ad900d698438@intel.com> <2023121515-litigate-finished-76dd@gregkh> Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <2023121515-litigate-finished-76dd@gregkh> X-ClientProxiedBy: MW4PR04CA0317.namprd04.prod.outlook.com (2603:10b6:303:82::22) To PH8PR11MB8107.namprd11.prod.outlook.com (2603:10b6:510:256::6) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: PH8PR11MB8107:EE_|PH8PR11MB6609:EE_ X-MS-Office365-Filtering-Correlation-Id: 0d02d93d-a17b-4279-75ae-08dbfd917b66 X-LD-Processed: 46c98d88-e344-4ed4-8496-4ed7712e255d,ExtAddr X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: 4lKgr4hNix+uv0QftUqHH6a9U2u5rC/0AUu4/lms2+Gek3h9JQtan6mqgYoo+yjCkzU4unE3rXO2kvjldx9XaL0JpKwcRaYE8BEdw3lGxsEPQNB9tbf/Lh9fxgIdIFkg9J/o/G2CHsnqvwzv8WQHplQ2K/2s0/QMeY1NIa94hAq9mqLgSTTGOZJq99+AglzJ5sNqx3zQaTrgE3fXAps51vpnqqvujqsrbgveHJeF+/3VdkfXY6lDJzkgtFoNoekjodCcSBBtPiDwdyvTh/9CUEhGEQ5ZMRBBBk9SSbGP1KfbJPnedaqy8qfOjC+Uvz+P2Es3xjXJPFTNPoEwV9j/J1lDTvsUKJZ7gXXjV+Uq8327wTATT00sODL6ILBhCRFE8ns1jYQwx2svAhIBek4lIkZCOZbGIeG3QTMekn0ac11Ll2p2lNn5OkREuCRRU3S3Rgwl6VNyZxYih4+K88kC1SSBGtRODuP/td0n/XsRN6ruoPmH7NIO6uNQfQEb4ucwtyYO3lVm5XnT4tdXV31FttIr8Dl+JexunY5FhHOz/42fAC4Gn5VkIFNnLE+e/EM5 X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:PH8PR11MB8107.namprd11.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230031)(346002)(376002)(366004)(136003)(396003)(39860400002)(230922051799003)(1800799012)(451199024)(186009)(64100799003)(6636002)(316002)(83380400001)(38100700002)(66946007)(66476007)(110136005)(66556008)(54906003)(82960400001)(8676002)(8936002)(4326008)(41300700001)(2906002)(7416002)(86362001)(5660300002)(6486002)(6512007)(9686003)(6666004)(478600001)(6506007)(26005);DIR:OUT;SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?cjcKusY9INUoNKQSbaGWTGvkfP98yGc1nNiF58oaXqaetg/3vwK5p6Mq8Qsp?= =?us-ascii?Q?HoQ00z/oAePYk9C+UQxHZ0gHYmuV9Xn0H11ZxxW1TxIuIvggvCwJhZmHkehR?= =?us-ascii?Q?8jc34Ax0Zy3BEV/KWiARkGpWrv2hBSJPGmCbA0+gUoXIqBczHCpezfCwDmk7?= =?us-ascii?Q?HL7+ky9TBAj+/elbn9es9hqPgBegXUJ4tkWONjZT62U/b09NPUHrwUW2vlkL?= =?us-ascii?Q?qLQ+X4NuQKIqHTbwPZHCLsMupFjCwXJgn5WXsFb91pCclHxgne9jVBYZcoul?= =?us-ascii?Q?RITyul2XyIyCfMmZs8OtRxXYuxTtyxrHvg82RLT03+/gMd+oFGAnLJXNzsfe?= =?us-ascii?Q?EzS55paeuyAqbo1adypovJO+Wh7AYliFaM8i9LeNIPKd5UtnkwB7ZdFf5VrJ?= =?us-ascii?Q?0bwRMBy03ZCb2D32j/KZuNMScRKMKHo2Vmx+GSDPQYp/30qIWEM+W3QEsAhb?= =?us-ascii?Q?xe2XbLzkN3pEiy1CuKtaAypX3q/BU5iG3g/vh/S/dEo2KYhzgjLWnC2xRW87?= =?us-ascii?Q?LKKA0I9avwPZd5odvh2jBP/f23MD/7V5WjP0DCPOayk6/bN6jusyNqeje/Z2?= =?us-ascii?Q?8Td3w0T8Osom0XWQe0ngGEuv9eNlCntQc1IA6RuEmVwsuL68MOGKTV4rQoZ0?= =?us-ascii?Q?h0H6/r8H4icWT3FBRc0w06mvQt6VR4XAoi0W5tfnVR5Dh57K0mXxMjg7fqhl?= =?us-ascii?Q?y2gztMM7lnmWRHaeGxhtaum4uIumm3rG8NJUOqr2+eMV+kIR5bLPw5LoICCn?= =?us-ascii?Q?eMz6BrtJ/nZOHLT90YwMLhyNXQENnouBFI1y1L/Wyy+Cg7d4pJrqqhyVcpGi?= =?us-ascii?Q?JEX6XGS06EYlpNA/qobwHVNRP9iQfQ/OIKorgbV2nMoy5jNlRJoZuEbIf0s8?= =?us-ascii?Q?xPTWvyU5gzaP9rXi5EByfoUyK8kmj/zeeXPBvygZFDa4l8cd0gBYlaDstmKP?= =?us-ascii?Q?FwmB0T1mOA6OZeQ4osy4qDzDyV0HLHajBTEt+CmTRW5NuOjSZyeURS78RGIp?= =?us-ascii?Q?aw8F6T4psF8TmNe6Ao5LNkoDLcNa+dUXyeVDlrRoEJBWNxwAJ2YewQvNWMft?= =?us-ascii?Q?hNzzLyMnbxXOmJMOR8amapT/XOdKivPNXmOilZO3bODRD140BV8yYkN2y4Gh?= =?us-ascii?Q?g2P1pr6OoiwDfVWfRFyicl/WaaBqo7wodHecrTI8qZz5EP92wO8U7OnCP6bG?= =?us-ascii?Q?BwXiInKecF04idn7Ii+iJJVejKzhLiyenj87OLt2BxLEbgDvC/1jOyn5nd0f?= =?us-ascii?Q?x1w7eQc9NAZ7/mRyh8UXwAx1VsQTmsbzToxuyMTZjPVqA4aMseJbcsJOAluH?= =?us-ascii?Q?JL4KMjCRz5BZwS4xRW8Wpwz3jD7eJxP73eHbK56uVdCFTPGo+2GWrg0NmhUl?= =?us-ascii?Q?KoMM60nUBAgDQk+KounanyC+AJq1RU6P4r8CMgw/2LjrfD5wnbMaah+R0LgK?= =?us-ascii?Q?amMBkoBmpvZanzAzI9QkFZmYmSR3MXXhXRk6xQVkOrc1Glvd+/c3hpJCIglZ?= =?us-ascii?Q?SvgdrXk/6sKa/CVZwVx+Oic5JZvE9UphuwVZ+AZssLHeUA96TqNiKYrR1jNm?= =?us-ascii?Q?dmgUdUPYk74Hr9cnYrmrsYMX7q0Zk3ihaTlqBJ0v+v1EDDRmVCTDxOHc9AnA?= =?us-ascii?Q?gg=3D=3D?= X-MS-Exchange-CrossTenant-Network-Message-Id: 0d02d93d-a17b-4279-75ae-08dbfd917b66 X-MS-Exchange-CrossTenant-AuthSource: PH8PR11MB8107.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 15 Dec 2023 17:15:48.6873 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: 8CGU+kjtSpxfJTBdAORZS3pBs8fsuY6lAwJaRKjUc63iYQtj/N4PL4Oz7LsDv/ZJu9MbrYYmLOFNhUCwTfmzgeBTAihUp9tMocC/oFwcOnQ= X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH8PR11MB6609 X-OriginatorOrg: intel.com X-Rspamd-Server: rspam08 X-Rspamd-Queue-Id: 4F55D1A002F X-Stat-Signature: 96r9q5qkssmppx3agauqa8xbge4kqmfx X-Rspam-User: X-HE-Tag: 1702660568-702226 X-HE-Meta: U2FsdGVkX1/0yQvPScpGS8gQtGIFWnfXrtT+aWswF/Ix/MGebLtKrIVj3X5F1WIwQMkfop8abPPiv3xOHhwiTfE34ejZ/vZHB+e8EyK8L+ewlTt0Bd+FEseESwKtMdCmyxCioLW6s6/s8XSe1Yv26Pho2OAMP2eXUS6DIFSmhcHqFFI5MqPmBJcK9EC8J0KJ4UTGDo3QF15aZ/fGqQIb3hfBCMMCWHwF87ExqowKhLtOL1bS1QP4Y81LJFLm/jHHuGHIUYkvQS3AU3dGhX9H8nV/1RPv9MkaUNlR+M/qLtHM6tqbn1f8N6ISIKRgDevD0AUyg+WMS46VX/8nxbhWH+kxgiZyuZt1NHjqYLRmsjCixzSHFkrqNaDU/iAaHAkhrtHn4uRqejHyuRdf6OQB0kq22il/zloH48N32uAHJM1cIoV5Lr3Fh6/GSuEd79RIvFc7QVhMN/0MH2KmS/SXW7Aba+Tum6diKBcLQZb4Qe643a7pvPJykiqeskrCSSM06L1AYLaRroeGTNbO5paPic4d804n0VnlKwdNnzgNHVF2OJTyDVXJsb4DZMVktA/8Mz4tKGVqLBQaXL+XSptxrwTMsMUZUoSYaqOj6mXU0MLC2eK68/NooZyRXgAXgtysEdfXBeaHe049CwNCZe8b5C1dK7r+0kc4/807jf1s9Ev524YeaU23NmtY04+dh7LTs7mxTaMKtRGHbQ1eTwreNv0HGvCuU9Dw+9CqPjJlcwWPFJhei9jXXH4o2/fT1BViINRZ7NHFLSqd1cLRdgV2tkhTJg9I7kj+BgzjWxUTdsUIMgZAsjxSd2irCKsiYd2M57UYN4cHj2WVyWUhhZfZroSL8xU2oG947z7F61pcE30lSdSQos//PoWCSwTFAS0cZymUURjJJClCAbLI5L4W86YlgIZpbH9UjJ27vregO9XmRCE+yYjMEH6gdmPtutqHiDy+7+1yBvl7+mhSs2I vB3ZIdnJ VV+xw2wbN7TiWZbAcnDFf3AUSXNWwSMZsBAET5DqByU00EjzznvCl6pw3ZuxbL/FZ9NoQFU+CWUdlH9oeC9gBns22miJ8Di0zFrOJrFTe4SpW7tEwg8OW5ugYhoyTVRZrCVssF3xewY0XovlEL9QlCjioizi+UDya83cLf+C+Xj8JM6gRJ6mAQavm/Gdc4iaMcMPXUNDPGx8kQwpF5VD5mA0wUJnBDgsupQI6366bC55dUXM31ZOLak4GM92qsCTPjIZKoSUW/sGS2sTcn2TdvWxUTkpR5Ec+RqQn X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: Greg Kroah-Hartman wrote: > On Thu, Dec 14, 2023 at 10:25:27PM -0700, Vishal Verma wrote: > > Use the guard(device) macro to lock a 'struct device', and unlock it > > automatically when going out of scope using Scope Based Resource > > Management semantics. A lot of the sysfs attribute writes in > > drivers/dax/bus.c benefit from a cleanup using these, so change these > > where applicable. > > Wait, why are you needing to call device_lock() at all here? Why is dax > special in needing this when no other subsystem requires it? > > > > > Cc: Joao Martins > > Cc: Dan Williams > > Signed-off-by: Vishal Verma > > --- > > drivers/dax/bus.c | 143 ++++++++++++++++++++++-------------------------------- > > 1 file changed, 59 insertions(+), 84 deletions(-) > > > > diff --git a/drivers/dax/bus.c b/drivers/dax/bus.c > > index 1ff1ab5fa105..6226de131d17 100644 > > --- a/drivers/dax/bus.c > > +++ b/drivers/dax/bus.c > > @@ -294,13 +294,10 @@ static ssize_t available_size_show(struct device *dev, > > struct device_attribute *attr, char *buf) > > { > > struct dax_region *dax_region = dev_get_drvdata(dev); > > - unsigned long long size; > > > > - device_lock(dev); > > - size = dax_region_avail_size(dax_region); > > - device_unlock(dev); > > + guard(device)(dev); > > You have a valid device here, why are you locking it? How can it go > away? And if it can, shouldn't you have a local lock for it, and not > abuse the driver core lock? Yes, this is a driver-core lock abuse written by someone who should have known better. And yes, a local lock to protect the dax_region resource tree should replace this. A new rwsem to synchronize all list walks seems appropriate.