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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 5B888C43458 for ; Mon, 29 Jun 2026 11:20:13 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=xzQ54dBajdrLtlzQjCgh7CevJT1LrEISm+tiLxZDmTY=; b=CIKUS4a42c+XLx 4hjEZxmzpNZE2OKJrfvAQVo4r2JFnCGRzHyrUh6P8UbYnY/SSpoyIb2oZZSJyCiFQBURWkb/I5z58 hKh7d3lkz7BrfvzUqSIbDO/4sVJhuSZcmNmcTm9yrekTQiq7D+ZlbrB3ea2QsCccQ6shdUZ1NLKvB hZL1bDkqErb4QZxWQj/iM+g599BTLN+vRTXQeHLKw0gzfNB6p6aMu/fucNVsmpsGMg3wF3mfMAreS vJBaE2+SAMBc7+YBqlSGNhmrF44vEx6VTk3z0TjZC97U7/p4Rzn9YIYLIrv8d13RTWx7MtyTDfs7L H54Oa0j69k0miGe1MyBQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1weA24-0000000EQbX-2Aox; Mon, 29 Jun 2026 11:20:08 +0000 Received: from tor.source.kernel.org ([2600:3c04:e001:324:0:1991:8:25]) by bombadil.infradead.org with esmtps (Exim 4.99.1 #2 (Red Hat Linux)) id 1weA22-0000000EQaI-3dJf; Mon, 29 Jun 2026 11:20:06 +0000 Received: from smtp.kernel.org (quasi.space.kernel.org [100.103.45.18]) by tor.source.kernel.org (Postfix) with ESMTP id CA44F60008; Mon, 29 Jun 2026 11:20:05 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 785D81F00A3E; Mon, 29 Jun 2026 11:19:46 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1782732005; bh=B6cSgDNUNCaJrq5YwzbaI3oA+4ARO04BNnR9PnfFHCo=; h=Date:From:To:Cc:Subject:References:In-Reply-To; b=isRnHms082rD0YpFSEFNAirC9nvfh4Nn4Wukfgndq3I2Za6wZ3cZIX3OAtKqyJ7Jk QbxmpIhxp4Zme+ZJuosxaLE4+d9zsfz/skW1tPR2ZZfSPRqJmpRppDTWUgZyVS1CHB 5QsW3n3TKw6RWILA0r+sB1OCe6Mi/a2ArdZ+CsZawVZxFaC5s8Q66AfZ53Z96KwYuW uZGsBXS1+y0VFETJSXBfuVdng+45W1oD9ZySufSdDejxCtS40h2780fAI1q4GM1Tvr c3+dPOqi24Z4+uI+6dYF9faK4Re6IYH+Fd0D81x/uqCCnIeztFgIByQ88ta8CCLLl0 zIbUQJ18s/EdQ== Date: Mon, 29 Jun 2026 13:19:43 +0200 From: Ingo Molnar To: Arnd Bergmann Cc: Juergen Gross , linux-kernel@vger.kernel.org, linux-pm@vger.kernel.org, "linux-edac@vger.kernel.org" , x86@kernel.org, linux-acpi@vger.kernel.org, kvm@vger.kernel.org, linux-coco@lists.linux.dev, linux-pci@vger.kernel.org, virtualization@lists.linux.dev, linux-ide@vger.kernel.org, dri-devel@lists.freedesktop.org, linux-fbdev@vger.kernel.org, linux-crypto@vger.kernel.org, "open list:GPIO SUBSYSTEM" , linux-hyperv@vger.kernel.org, linux-hwmon@vger.kernel.org, linux-perf-users@vger.kernel.org, linux-mtd@lists.infradead.org, platform-driver-x86@vger.kernel.org, "Rafael J . Wysocki" , Daniel Lezcano , Zhang Rui , "lukasz.luba@arm.com" , Jason Baron , Borislav Petkov , Tony Luck , Yazen Ghannam , Len Brown , Pavel Machek , Thomas Gleixner , Ingo Molnar , Dave Hansen , "H. Peter Anvin" , Sean Christopherson , Paolo Bonzini , "Kirill A. Shutemov" , Rick Edgecombe , Pu Wen , Bjorn Helgaas , Ajay Kaher , Alexey Makhalov , Broadcom internal kernel review list , Viresh Kumar , Reinette Chatre , Dave Martin , James Morse , Babu Moger , Tony W Wang-oc , Damien Le Moal , Niklas Cassel , Dave Airlie , Helge Deller , linux-geode@lists.infradead.org, Olivia Mackall , Herbert Xu , Linus Walleij , Bartosz Golaszewski , Greg Kroah-Hartman , "K. Y. Srinivasan" , Haiyang Zhang , Wei Liu , Dexuan Cui , Long Li , Guenter Roeck , Peter Zijlstra , Arnaldo Carvalho de Melo , Namhyung Kim , Mark Rutland , Alexander Shishkin , Jiri Olsa , Ian Rogers , Adrian Hunter , James Clark , Josh Poimboeuf , Pawan Gupta , Vitaly Kuznetsov , Andy Lutomirski , Boris Ostrovsky , Huang Rui , Mario Limonciello , Perry Yuan , K Prateek Nayak , "srinivas.pandruvada@linux.intel.com" , Artem Bityutskiy , Artem Bityutskiy , Miquel Raynal , Richard Weinberger , Vignesh Raghavendra , Ashok Raj , Hans de Goede , Ilpo =?iso-8859-1?Q?J=E4rvinen?= , Rajneesh Bhardwaj , David E Box , xen-devel@lists.xenproject.org Subject: Re: [PATCH 00/32] x86/msr: Drop 32-bit MSR interfaces Message-ID: References: <20260629060526.3638272-1-jgross@suse.com> <7332feff-2649-496c-8e49-b0a19eb54a32@app.fastmail.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <7332feff-2649-496c-8e49-b0a19eb54a32@app.fastmail.com> X-BeenThere: linux-mtd@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-mtd" Errors-To: linux-mtd-bounces+linux-mtd=archiver.kernel.org@lists.infradead.org * Arnd Bergmann wrote: > >>> Note that most patches of this series are independent from each other. > >>> Only the patches removing a specific interface (patches 7, 15, 26 and > >>> 30) and the last two patches of the series depend on all previous > >>> patches. > >> > >> It looks like you are touching most files twice or more here, to > >> first convert from rdmsr to rdmsrq and then to change the > >> two-argument rdmsrq() macro to a single-argument inline. If you > >> introduce the inline version of rdmsrq() first, you should be > >> able to skip the second step (patch 31) as they could be able > >> to coexist. > > > > I've discussed how to structure the series with Ingo Molnar before [1]. The > > current approach was his preference. > > Ok. Note that the individual patches are IMO significantly easier to review through the actual 32-bit => 64-bit variable assignment changes done in isolation (which sometimes include minor cleanups), while the Coccinelle semantic patch: { a(b,c) => c = a(b) } which changes both the function signature and the order of terms as well, is just a single add-on treewide patch. Thanks, Ingo ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/