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 28C29C7EE29 for ; Mon, 29 May 2023 08:58:15 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 801DF900003; Mon, 29 May 2023 04:58:14 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 7B210900002; Mon, 29 May 2023 04:58:14 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 652C5900003; Mon, 29 May 2023 04:58:14 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0017.hostedemail.com [216.40.44.17]) by kanga.kvack.org (Postfix) with ESMTP id 52F8E900002 for ; Mon, 29 May 2023 04:58:14 -0400 (EDT) Received: from smtpin19.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay05.hostedemail.com (Postfix) with ESMTP id 0CEEA40290 for ; Mon, 29 May 2023 08:58:14 +0000 (UTC) X-FDA: 80842690908.19.9EF85B4 Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by imf14.hostedemail.com (Postfix) with ESMTP id 4B17310000E for ; Mon, 29 May 2023 08:58:12 +0000 (UTC) Authentication-Results: imf14.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=R5FDQfcm; spf=pass (imf14.hostedemail.com: domain of johan@kernel.org designates 139.178.84.217 as permitted sender) smtp.mailfrom=johan@kernel.org; dmarc=pass (policy=none) header.from=kernel.org ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1685350692; 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=067b7Mw9xAB5xeFM89mqjalcyQKxOFzYzF0Ba4W8yKA=; b=YbeS72Rc1nvwYuzxJzggG0+gHMrK/bVQ7nMgjDG69JIvHfxtSCBMmmFJMnAMD+iyx74bS+ uG4QyYGmWkzpZUxTtRxfYRzGwNQXnuwZDGvNkkaNPjCfxG1QXsm8jWcEd6aSAfsVPWXeoD O5PTWtV56dJ98ZhnIU1aBZZARU0/Ojk= ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1685350692; a=rsa-sha256; cv=none; b=MMkrVROYzecCvSBgGdJpFpqaoOksSxZAprkwGqzObzBRIkRqq9FsZwGtU2N41dAkqLbaD5 5NO+mQcF+ez+BJ8GA1jjO2KVdFvDF8FRtrmW9wCOlT29OULxmlrWATEREoCAC2PxyRUuuX G4dQnsyGaDuzE56AGFYyqRGGfIfqkFA= ARC-Authentication-Results: i=1; imf14.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=R5FDQfcm; spf=pass (imf14.hostedemail.com: domain of johan@kernel.org designates 139.178.84.217 as permitted sender) smtp.mailfrom=johan@kernel.org; dmarc=pass (policy=none) header.from=kernel.org Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 2D388612FB; Mon, 29 May 2023 08:58:11 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 8A396C433EF; Mon, 29 May 2023 08:58:10 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1685350690; bh=t2lj466261TGVeHMuyfw6d01BSLRoZETH0n0ShM8crM=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=R5FDQfcm8SUNyzpKDat8ojVSi4y8fmhImkdGup3zXnGIsJRsK3eJUlNeQZESjT9Gx S2dvoJ/khyHlknve2mX2V1Pd+pMy1J60DyeL4+/TV+s8kQQcAQmKeKNiqrSN7gDQaS hgO++m+zR2PlpJB45PJF+VWuRIIB2b+rn/ywHnY/ihuutRVOVGCkGtqkRP+ipK1xfF u0TczxO2dm1/bUG6dvdW9R8FcabA/r+zU2FNkhfM/6Q+JY4d8Jh7nG3BkTu628dAwS TP0hw3M2sEqF9b1WVPkJTr4rsf4MFs944By1iyFDCuUPB8MBgBE34BDs8sGeoWwIcT wQkhvy2HfChBg== Received: from johan by xi.lan with local (Exim 4.94.2) (envelope-from ) id 1q3Yhb-0004Ym-74; Mon, 29 May 2023 10:58:07 +0200 Date: Mon, 29 May 2023 10:58:07 +0200 From: Johan Hovold To: Linus Torvalds Cc: Luis Chamberlain , Lucas De Marchi , Petr Pavlu , gregkh@linuxfoundation.org, rafael@kernel.org, song@kernel.org, lucas.de.marchi@gmail.com, christophe.leroy@csgroup.eu, peterz@infradead.org, rppt@kernel.org, dave@stgolabs.net, willy@infradead.org, vbabka@suse.cz, mhocko@suse.com, dave.hansen@linux.intel.com, colin.i.king@gmail.com, jim.cromie@gmail.com, catalin.marinas@arm.com, jbaron@akamai.com, rick.p.edgecombe@intel.com, yujie.liu@intel.com, david@redhat.com, tglx@linutronix.de, hch@lst.de, patches@lists.linux.dev, linux-modules@vger.kernel.org, linux-mm@kvack.org, linux-kernel@vger.kernel.org, pmladek@suse.com, prarit@redhat.com, lennart@poettering.net Subject: Re: [PATCH 2/2] module: add support to avoid duplicates early on load Message-ID: References: <20230524213620.3509138-1-mcgrof@kernel.org> <20230524213620.3509138-3-mcgrof@kernel.org> <8fc5b26b-d2f6-0c8f-34a1-af085dbef155@suse.com> <6gwjomw6sxxmlglxfoilelswv4hgygqelomevb4k4wrlrk3gtm@wrakbmwztgeu> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-Rspamd-Queue-Id: 4B17310000E X-Rspam-User: X-Stat-Signature: ap4xk3zctnyx3muegg7r1ewognxfpg3u X-Rspamd-Server: rspam03 X-HE-Tag: 1685350692-475550 X-HE-Meta: U2FsdGVkX19xkdFFWrQNX5+5DKGNiCG6sE7j+8/FjknE5aQv/R6rMsreAaHJf1puclQzXUunl4xGnS3TjqbSfkzlW8YCApNBgWxlmd9pytpdTd8UnCilfndUTXODkO2FxaS9/RQ5i6Ctqseu599mVXP6JHFYn2P1FnDiBhx4GP4HlhKwjFMBmeblfFKIwgOiqhe1QwhBzWIEIupHa0VLyeOuaDa+2TU9Z8swPs4bKHC1zpWY9QuEIxa5d5ur7bJIAVmy0FGMsdP6Ru5YmUzO+znD8cwcTjqmh8FE+6RQl4RiaEnD6xeuUJz5nFzgQHH0qp7sY9P4MCnfdXEimclJgfER6HdL5EMt6upjPHiCwBakOK2Wfr3W6C13iDQjHqLqRmxXL3MUx6P2q5EHXXvWjkibVXy7Zv/NkNi3bnKvofo/FeIJmBXSPwV/kRiuT1Qbj+mErZGSzvnLnZMvBwCVrKCaOe3jAc6+1rfKNFMlGI+rAD84aWVi0RI3yolpA3t0D56G8CTF162kIHjex+yC5BbV2XRZAh7Ch2t+Kar3gHD67atRspSQyXQxU2eEfO8lJDTMA9C/eDikpqqBagwSm6HpOxjNHTAuHP9XuItfEL1x8lPYTqPTx0OTOMFRFBmWFb0EqD47aqdlLD89W910RTrLZHzsaHJop8kVr0Giw0ul9Gr9+40taKuzZbx7xxWZ59/BG5ipoqjshTms/4gDWYHsHql7h54Yv0AEEX84yP4B6lE7nV5f513SIgTh8z2HEqMAtjAZj0HhDorLMB5IWeN9dH7Z2Hs/N3MFasOdeO5pd7qUqz2ITm/mJ+8c8myngdPFz8svDr+o64ggc7haF2lCDy6n1HxPuaXUg8rdFliwg+0voQqobvXCyQSoIaP150n+bLNeUZym5ivCBQJ1Rbq+o59zTIQgtx5xUu58UAnHx5FBn7hxK0Y0LKlnEzxbp93Es/CPAqi16SMyI0p NAF9opHD rU8yz+8MQxBI8bGTMkkyQ9iQyd7Vje3LXNYGEZ6EWXsS2jljyuSwxOGF36KBCAop+HV78fKvNYeAQUFtB3oVaVY/ZVFs5+SlIoNdyfMZTUDGHNmI/A+4D4ghQbazf9nkZOC1dMCz0E7geseK+qK/YiKF2ceCZzWPnySiZIC1K3YfIT5VSuh+KpP6gfm8JbODG1Zd9avZJHn3bv+Y3/Ez4/qKSZYszpIAVZj/jRgt6Grj6AFaBrc0opORd1WsC09DX5KSO4HXhi1SZnyitwJ84pERgmurRi8n7UW2lYL9LqBqE/eORnUdE6LPC8av4u+/bhEgR 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: On Thu, May 25, 2023 at 06:39:52PM -0700, Linus Torvalds wrote: > Ok, I decided to just move it from my experimental tree to my main tree. > > I think I used about three times the time and effort (and lines of > text) on writing the commit message compared to what I did on the > patch itself. > > I tried to lay out the background and the implications of the change - > it may be pretty darn simple, but it does have some subtle issues. > > Anyway: I've committed it to my tree. This is not necessarily the best > time to do that, but let's get this behind us, and in particular, > let's get it out and into wider testing asap. > > If it causes any problems what-so-ever, I'll just revert it very > aggressively (unless the problem is trivially and obviously fixable). > It is, after all, not a fix for a _kernel_ bug per se, and whil eI > think the patch is very benign, it does change user-visible behavior. > Very intentionally so, but still.. This change breaks module loading during boot on the Lenovo Thinkpad X13s (aarch64). Specifically it results in indefinite probe deferral of the display and USB (ethernet) which makes it a pain to debug. Typing in the dark to acquire some logs reveals that other modules are missing as well. Fortunately commit 9828ed3f695a ("module: error out early on concurrent load of the same module file") stood out when skimming the changes that went into -rc4, and reverting it make all the expected modules be loaded again. I have not tried to figure out exactly why things break, but it does seem like this one should be reverted. Johan