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 EBFC8C636D4 for ; Mon, 6 Feb 2023 18:48:31 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 339076B0072; Mon, 6 Feb 2023 13:48:31 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 2E8F16B0073; Mon, 6 Feb 2023 13:48:31 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 13BE66B0074; Mon, 6 Feb 2023 13:48:31 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0014.hostedemail.com [216.40.44.14]) by kanga.kvack.org (Postfix) with ESMTP id 018D06B0072 for ; Mon, 6 Feb 2023 13:48:30 -0500 (EST) Received: from smtpin28.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay03.hostedemail.com (Postfix) with ESMTP id C85EFA09FA for ; Mon, 6 Feb 2023 18:48:30 +0000 (UTC) X-FDA: 80437752780.28.DB98C0E Received: from mga18.intel.com (mga18.intel.com [134.134.136.126]) by imf02.hostedemail.com (Postfix) with ESMTP id F35568000D for ; Mon, 6 Feb 2023 18:48:26 +0000 (UTC) Authentication-Results: imf02.hostedemail.com; dkim=pass header.d=intel.com header.s=Intel header.b=CHTLXNw2; dmarc=pass (policy=none) header.from=intel.com; spf=pass (imf02.hostedemail.com: domain of dan.j.williams@intel.com designates 134.134.136.126 as permitted sender) smtp.mailfrom=dan.j.williams@intel.com; arc=reject ("signature check failed: fail, {[1] = sig:microsoft.com:reject}") ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1675709307; 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=7aBHiXpsv7KXLt5yKDCSHdiTJ7XwjnXFs7KZmRIJt5w=; b=Hh48FUOcgYX2Gc0WblCzM2hoC8R4fT4M/Nk3Bwf3uI9I+vEkhDD5H9DzYtOJ6NN8/ARDfX hx1WuzM9lJ+WQOZgsytEMBkoyaP2Rl6o+X1KEpVKl/RQBpq+djzAQBHVvE2gSlkhk2vr8f nCyLW66Hh2bSNrZLuHRKe4WBsaLR/DU= ARC-Authentication-Results: i=2; imf02.hostedemail.com; dkim=pass header.d=intel.com header.s=Intel header.b=CHTLXNw2; dmarc=pass (policy=none) header.from=intel.com; spf=pass (imf02.hostedemail.com: domain of dan.j.williams@intel.com designates 134.134.136.126 as permitted sender) smtp.mailfrom=dan.j.williams@intel.com; arc=reject ("signature check failed: fail, {[1] = sig:microsoft.com:reject}") ARC-Seal: i=2; s=arc-20220608; d=hostedemail.com; t=1675709307; a=rsa-sha256; cv=fail; b=c1uM1UuZZXAi6UEa3QRr8hDHFu5woe4cIp6GpOVdo/sVciWMwP+8mSmKi/TDfZAEvxX/Qa xHaZUhNipqLWkoQAml27oGzF39UbpYnkg9PtmktYxyOmJw53sZugjKSEuXu3qAH1O1u4Sw lIOBkHFv7ede1jQ5W4OZtIouFko7Ulo= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1675709307; x=1707245307; h=date:from:to:cc:subject:message-id:references: in-reply-to:mime-version; bh=Ewk0ashiigVyIDAVyNytS5Z7BtmLoaiJelhxT8btPY0=; b=CHTLXNw20YVNUPkGMFzp5XGUIcEIzy+rRSUS7FW59hvkcjvCiv9QT9UE DxvcY7iA3JS+WKP24fdLPQlmiSiIE8g9sjnymgfb9qhRuz0lSstOFGwkK Dsc75puVXU+geSCXKbO2pszGT9Df1w7KnpdcdOyQ37/AGIYaJFbWoe6mZ OmTnSyaGrKmTggulsA1iPhgibR/+/Z4Q+tY1zvolO9yPKyJmkzSaIZgtz C1oR5Mm8SBCrJCmtrOvm+hlnkZHxfBUkUtK/eGy4mati+OLcproLKpf1T XhE84bDHU0FMw0Dx+NZvOlIciorlytH3yscM33k9dOS/U72hu7OYcH4iV Q==; X-IronPort-AV: E=McAfee;i="6500,9779,10613"; a="312938892" X-IronPort-AV: E=Sophos;i="5.97,276,1669104000"; d="scan'208";a="312938892" Received: from orsmga001.jf.intel.com ([10.7.209.18]) by orsmga106.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 06 Feb 2023 10:48:25 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6500,9779,10613"; a="698942722" X-IronPort-AV: E=Sophos;i="5.97,276,1669104000"; d="scan'208";a="698942722" Received: from orsmsx601.amr.corp.intel.com ([10.22.229.14]) by orsmga001.jf.intel.com with ESMTP; 06 Feb 2023 10:48:25 -0800 Received: from orsmsx612.amr.corp.intel.com (10.22.229.25) by ORSMSX601.amr.corp.intel.com (10.22.229.14) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.16; Mon, 6 Feb 2023 10:48:24 -0800 Received: from orsmsx601.amr.corp.intel.com (10.22.229.14) by ORSMSX612.amr.corp.intel.com (10.22.229.25) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.16; Mon, 6 Feb 2023 10:48:24 -0800 Received: from orsedg603.ED.cps.intel.com (10.7.248.4) by orsmsx601.amr.corp.intel.com (10.22.229.14) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.16 via Frontend Transport; Mon, 6 Feb 2023 10:48:24 -0800 Received: from NAM10-BN7-obe.outbound.protection.outlook.com (104.47.70.103) by edgegateway.intel.com (134.134.137.100) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2507.16; Mon, 6 Feb 2023 10:48:23 -0800 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=C8bcvStG0lgAYhPzDNXtwUqRYUACM4PY/smiuk942fc4AD4wnGe3PlCwp1WBMLSuBvHEHO+KuMraMVutC16/TXqOPYZzYxoGgZMDdC+hHe6InwMxz8zxW3TW6w4HkcpNvn/mAnSmeWbAgQAEZgMqFEsD3+Z/BXwxDygiS7AddK4N1LjOZTi7rOxDpopqKF6xNzYgx914dqo49Qvt4n76VIncF196HHgAjTKmJTduTexrtXBHcVnW17CnE6d582SAAKJVOsFcE2eb7XWXqtwGsrvJZTls4hjxyQRynNTVNbfstErD7q4fLVCPCykZAq+ska58/EfRqmpN56dFndscnw== 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=7aBHiXpsv7KXLt5yKDCSHdiTJ7XwjnXFs7KZmRIJt5w=; b=I7nAlXxdX6SMzJ9LN7kwO0LXlk6LmPBOP8RPs1COSBDxYzqFzWq+Rs5xgiKE24uCPRVXCFWjvqKX2uRuQsSaN86noNQpXKn2X5V8xF5yywf/ct+40pWsbdIKKJQaQwcuDH+aIQLor8MkDtqP0c+7GXvwdtEYubjLrcISSFZI9S1/gwbbOfep0yYnmKBcQFJ1GbWQLYfQXfFgv5xDc8CvT3KbJCjKCJkUp3WveQyEZibpkRpNtM6sVJMLEhgk0rcVuw9q5zlgyapnXc5fEYqY4HrGXbHOioTEkVJGBgW/mMlGDQovkcPcdSJnDbVjmALzVQOxtKPUD5tstoQS7xvclw== 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 SA2PR11MB5163.namprd11.prod.outlook.com (2603:10b6:806:113::20) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6064.32; Mon, 6 Feb 2023 18:48:22 +0000 Received: from PH8PR11MB8107.namprd11.prod.outlook.com ([fe80::421b:865b:f356:7dfc]) by PH8PR11MB8107.namprd11.prod.outlook.com ([fe80::421b:865b:f356:7dfc%6]) with mapi id 15.20.6064.031; Mon, 6 Feb 2023 18:48:22 +0000 Date: Mon, 6 Feb 2023 10:48:18 -0800 From: Dan Williams To: Jonathan Cameron , Dan Williams CC: , , , Subject: Re: [PATCH 06/18] cxl/region: Refactor attach_target() for autodiscovery Message-ID: <63e14b723009c_e3dae2947b@dwillia2-xfh.jf.intel.com.notmuch> References: <167564534874.847146.5222419648551436750.stgit@dwillia2-xfh.jf.intel.com> <167564538227.847146.16305045998592488364.stgit@dwillia2-xfh.jf.intel.com> <20230206170636.0000739e@Huawei.com> Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <20230206170636.0000739e@Huawei.com> X-ClientProxiedBy: BY3PR04CA0023.namprd04.prod.outlook.com (2603:10b6:a03:217::28) To PH8PR11MB8107.namprd11.prod.outlook.com (2603:10b6:510:256::6) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: PH8PR11MB8107:EE_|SA2PR11MB5163:EE_ X-MS-Office365-Filtering-Correlation-Id: b984d323-4ba2-4ea7-694e-08db0872b851 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: KMpjCdkOGcobtCqLjPET7LrCHSpj72XZXfMsjMn89EPFQAZl0vvNCnNRXIMI75JPkSRLVD3iyY+ALRO1OqiYxKen3NYuXJH2tj6tJoujxN3dqCo+kgQFOjEAGDy1Rhe4W8USGSaVn5EC+370MtATT63gHBNSSHMmfTPclfRnTTX+ba791nNenRKQ/ygzzvx0UyDmDWlhpJ7hr3N420i15CZEip/uHo7JksX8VNKyWyTvHp6MaIFtSBGrNWel0IVyhbWBAs++Rbc3xLoyttyRe39O7KEugJ0WRJ2FkebGOniuFZOJxr8VtdBaGZSv0gt3Slmj+Oef0Rrk5fySJA9IqFsvuOUL9hm6f6kqZVrNDQK3MF32OTC3ZHGr1JcccJB2bEAOw1UP/eT4DRF0qfRbvNTlA6ysN81IGVJmU8ry1sksEGB86CkVyn3GSuFwzD6MHUfPlWUN1o3m/FqUtdCzPM5rTP215JkiQaShRPWLWC5kMOh/73bPGWPaw7tf5V6Hj5+OOxPBL0yl0ZPxpyn6jRv+L7xnjc6aE0Rqe82ZFO0uUVJdgsQx2ahONp5I9aA3k9lz0Hw0Z+BMVuqIPNsfif5QkWnvBAaeDoj9r2ZT9+DZU+FX+43FJmXy0cir+wY0lsVXtQBfZG5oLlhZBcKFgw== 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:(13230025)(39860400002)(366004)(376002)(346002)(396003)(136003)(451199018)(478600001)(66476007)(6486002)(26005)(2906002)(9686003)(6512007)(186003)(6666004)(66946007)(8676002)(4326008)(41300700001)(6506007)(66556008)(5660300002)(316002)(38100700002)(110136005)(82960400001)(86362001)(8936002);DIR:OUT;SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?jEfCarI/jvakWGrtHbO5Oid0e5PCRE8uJVnFxsxUOJu7cacu+K9nsRdcBgnC?= =?us-ascii?Q?wCyNV7wpcZXCT3VmVCeP3YqfdlUwAwVgTdWDvzicVHhZfCVOt4DwHma7nd8b?= =?us-ascii?Q?vI/zVJjyE03sn53AW1ddsddl3zRf1HjR9vhqhR6lyDOdJeWu2UK+t22b4QJu?= =?us-ascii?Q?YmawL0vfxjXqBzNWrmCOvI5jDrWXa883fMKuSl2FIp5Jw+yX5Esn/XKsElxW?= =?us-ascii?Q?p3PVJdN2CDalNeHcebTjxOSFlXmqEF9lkkXF7EvdXFqxkCHKWhWlMxnrEonz?= =?us-ascii?Q?tesEZyvbpSM1nnrIrTspK7bzhSYeqIkVhw1zvHSaPNypa8xTGYwppIuCEJ0Z?= =?us-ascii?Q?d5Mw8erBsRj+HfQ3ViZP1YSFej8IXaRQ8Z9ZG4sGiu0ynHG2PDL7QEEZFYBi?= =?us-ascii?Q?uHd7iv7dLpWkAucE0eSZJEmNq5rICZyBqCUByGBqp9gtaJeWaE7bDhWYvVKZ?= =?us-ascii?Q?Vm8enGDXOIk29aElVJj12q1VBv1Xewq0WV0FhSgUgOFTVPTa9fhz8fWxQ4xb?= =?us-ascii?Q?/lXD5mfVZ5CZnYXTVk5UDK7OGl6uy7FpeKbAhmng0CIflcEQvFduT0jzrOlO?= =?us-ascii?Q?RBWEu8QsH2E+h04bowKu9Tfp3qu2ArRwLXXfHH/+G3oraG7l+1DzkblNV5Om?= =?us-ascii?Q?sfNPxXPUiMaqQBJ2k8b+aTTwdcIhlA1jJwcJ8+axjGW4CV7MXl+GkbTYGbhE?= =?us-ascii?Q?/jUZJnzBa61pzYZwgfCNBJY0CBoxHVM9TIR44+PL87OpmdGCML0SUu4z6UPT?= =?us-ascii?Q?ab91eFm4ZnbsgJrLlA0WV66KGxagUkYYLznKHGK6LEgnwmsEztbwyTqYOPVg?= =?us-ascii?Q?8VRy2VyxmeUhviOt9LDHqYFXkXeGjUmdIjNDQRNLUCLOCh6JUp/Ef1b4ZrX/?= =?us-ascii?Q?SxbSnhDVWt1ZRTGdQ9aAmMoAv5JWvvbQANw6gOdIYkFfsMUIUfCQadSK2BJR?= =?us-ascii?Q?hG92+r0tLTmRfilkyjXTXyGCYfXFMWF9+saALCjh4mB1U82zt3yaATOeKCHO?= =?us-ascii?Q?VZmjCUATnJPiD0DXFuL7Bu2mitv03YaPGCn+KT2rQQi8l42yI17gJxe/cpZY?= =?us-ascii?Q?8oY6o2R6d4HsLG/HHxRYBS4lWH2TQpiOiejy13T1VNHJXOYzw9y3BmwWlCde?= =?us-ascii?Q?95roa7rm2+vW2nU3Ja1W5x8VNHhbp2h9fu3F3S/m7os1kFUHbYIIfcoWf9yU?= =?us-ascii?Q?D//eULybqXotq4dr3M28JVrYJBUMVM+7K+QLxbPsMnJIazDRMlftFXv7NXWD?= =?us-ascii?Q?Be/Cejn+5gNRHUegJVWfk6CoQrAFD6gXeK3ynxyrx+/1Btf1OYnlBC+bqW+F?= =?us-ascii?Q?kvLiJ5SxHReuxCia7bAey4ua5prrgvDZZSxUodhvHpXuQ3oC9A2ZVt08q67N?= =?us-ascii?Q?EoKooSUHUP8hxqvgJOBFpr9sQkbq9TuvUm7cQmqvuzr61NqDySZ2IxLKaEZ1?= =?us-ascii?Q?o6i6BaeLaQZQXUkGsc4VCft267o6O+W+DseAE791mIwL4hmEU3DtdDzuRZqi?= =?us-ascii?Q?za3WCY1qGvdt8LLpI7svSfrEv6zpGWoMco++x1m36pRiupFNuF2P9TIdvjgf?= =?us-ascii?Q?7k+D8DuGlkt7ZFrZlEQDY4iNx6UaqlGWT0B5w1QD4TRDmMf5u0WshzkH3Ea4?= =?us-ascii?Q?Qw=3D=3D?= X-MS-Exchange-CrossTenant-Network-Message-Id: b984d323-4ba2-4ea7-694e-08db0872b851 X-MS-Exchange-CrossTenant-AuthSource: PH8PR11MB8107.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 06 Feb 2023 18:48:21.6725 (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: K+ofq6xAYGvVsorZl22ifahNsY0Fo9iQTBMbljS2bkqDp+nHIosJjP2r0pSypQct1Ruu0TOcAngVPtknxFp/83+xBthkI/kXF+6GvdrOms8= X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA2PR11MB5163 X-OriginatorOrg: intel.com X-Rspamd-Queue-Id: F35568000D X-Rspamd-Server: rspam09 X-Rspam-User: X-Stat-Signature: 5qryorpni1i86oargx9au3hpnkatzt6o X-HE-Tag: 1675709306-932747 X-HE-Meta: U2FsdGVkX1+uu3uMgMjfI0AXURkjeiZG/HHZyl12OWXe1Ltvam5T/qiu6PuS4D7DN5DUeubLMz3Wpl4O8ARAC5i9o46gHoKS88U2RC5wGUZeVpX9/0rSz5h/TuSfhO/u8EEWewX1VJn7yDC9XD32ISyi5Om+gB0nugW2nxp2kLXUKSSBFNeZNyFR+zJ/oCCmdxXULHA5S6GmtiKWOFXHtUYXJrlKmCsxjFnQgBwX2tsNaTypLoJFlbID7/CUjDp6OD7vLGcHUlZ5nh7ffb+I7dvxpWo8eSZhk0Yh0JZF22s9rTYs+99yXUYZPReBhRG1gY6vK7GoKY/DTXHY7IaArm5WhgzQIA8AAgIWJw2GD7YsLJ9anfi7HpOBHO4AxJpI8jz0y5YmMXq6x3i3ZNvYKeHn6+BXvPuaMQSyIa79oeAJ5Cn+11LUWI+F+vpglLMbjq+M4X3C9yfFe5F5k6ve9DPM3VaFuCjnQg8gXK3QWZJwnxQ0IDSJwN1uXCKmCFApEWR5sc3iAudvfWXzxd0jZNRH0WIxKGXM9KVFlH6ZsB49q+OmfTu4iME9pSzckBvJv13MXSUJZtgSGqI+RXZjctRyDvmo1Cuxlz2ArQmnXKa+LA5soxk1xCHQmi6A8HevCJe8xiP0/TNPNMZWVIKw8YqPPjwBr5P1cne5EkWylSsFWQJFDxovkSaxrTlP+yqSy8KNAt9GHUDy/pHr2RX4ek9JEe+SlvMnXxn4uQa2J5xEpEMC2HUPCzxS4tzxNwOaei84PTj6ptcPAvVXkXJJpnbGCbDzPc+5XyNq56/SJkiJ7Uy60mZmm65ZbeF4TIYAScagVwzFYwODDcV4dsnQHbH9cmU7JTtKetdUgRiMzfXsEQkjzBG12rNd+j08XK9gJsI0ALzJaXbyz3aEXXI9/pwbth3yvB4pIQ6MIS+9eUe3xbb2iHN4y9gamDVraEdlCIHOY4zo8Zx0lmlUkMS llZuhADk l4c2qbhE+lh2yNnOhvJ0KDwk960wypTHmyPAvMHn5K1d6f6lFbkJbvbd9wuNHtqIOEuJZhE+L6Sy2QEneTBYT42V+7FZrCmjqk8510mWeiPJQUaMJyX5qi7CMrbBQwd3KPs63bBCDcuPivBxugUZfgdpKq4juKZJhms0eJX5MqLkzTLt/2IuZnk/4kUtBfnmCXfBEeEBgIkOQxPG+tyotJk/QdK51w0Wa+glPiTA24lwQ3xUuFiZUqCnhv/2KnIbZNJC661xc6iRvBSw6nIduSZajNPenavLjvT7nbNUF8X0u1EnikIoOuJDFAhyGsYcOla+r 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: Jonathan Cameron wrote: > On Sun, 05 Feb 2023 17:03:02 -0800 > Dan Williams wrote: > > > Region autodiscovery is the process of kernel creating 'struct > > cxl_region' object to represent active CXL memory ranges it finds > > already active in hardware when the driver loads. Typically this happens > > when platform firmware establishes CXL memory regions and then publishes > > them in the memory map. However, this can also happen in the case of > > kexec-reboot after the kernel has created regions. > > > > In the autodiscovery case the region creation process starts with a > > known endpoint decoder. Refactor attach_target() into a helper that is > > suitable to be called from either sysfs, for runtime region creation, or > > from cxl_port_probe() after it has enumerated all endpoint decoders. > > > > The cxl_port_probe() context is an async device-core probing context, so > > it is not appropriate to allow SIGTERM to interrupt the assembly > > process. Refactor attach_target() to take @cxled and @state as arguments > > where @state indicates whether waiting from the region rwsem is > > interruptible or not. > [..] > > @@ -1418,31 +1418,25 @@ void cxl_decoder_kill_region(struct cxl_endpoint_decoder *cxled) > > up_write(&cxl_region_rwsem); > > } > > > > -static int attach_target(struct cxl_region *cxlr, const char *decoder, int pos) > > +static int attach_target(struct cxl_region *cxlr, > > + struct cxl_endpoint_decoder *cxled, int pos, > > + unsigned int state) > > { > > - struct device *dev; > > - int rc; > > - > > - dev = bus_find_device_by_name(&cxl_bus_type, NULL, decoder); > > - if (!dev) > > - return -ENODEV; > > - > > - if (!is_endpoint_decoder(dev)) { > > - put_device(dev); > > - return -EINVAL; > > - } > > + int rc = 0; > > > > - rc = down_write_killable(&cxl_region_rwsem); > > + if (state == TASK_INTERRUPTIBLE) > > + rc = down_write_killable(&cxl_region_rwsem); > > + else > > + down_write(&cxl_region_rwsem); > > I'd be tempted to do this in two hops for patch readability. First > make the code reorg then follow up with this bit before the use > of it in the next patch. > Sure.