Received-SPF: pass (google.com: domain of linux-kernel+bounces-167802-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1;
Feedback-ID: i56a14606:Fastmail
User-Agent: Cyrus-JMAP/3.11.0-alpha0-417-gddc99d37d-fm-hotfix-20240424.001-g2c179674
Precedence: bulk
MIME-Version: 1.0
Message-Id: <d59d9b67-3f0c-423c-a76d-74056a8444cb@app.fastmail.com>
In-Reply-To: <F1D1D3D4-CFB0-47A8-87F0-E1D8A6B5904F@toblux.com>
References: <20240420223836.241472-1-thorsten.blum@toblux.com>
 <42e6a510-9000-44a4-b6bf-2bca9cf74d5e@linux.intel.com>
 <C0856D2E-53FF-45EB-B0F9-D4F836C7222C@toblux.com>
 <e7e4ff27-ebb3-4ed6-a7cc-36c36ab90a36@app.fastmail.com>
 <99B58F85-CC9C-49F6-9A34-B8A59CABE162@toblux.com>
 <F1D1D3D4-CFB0-47A8-87F0-E1D8A6B5904F@toblux.com>
Date: Fri, 03 May 2024 17:05:39 +0200
From: "Arnd Bergmann" <arnd@arndb.de>
To: "Thorsten Blum" <thorsten.blum@toblux.com>
Cc: 
 =?UTF-8?Q?Amadeusz_S=C5=82awi=C5=84ski?= <amadeuszx.slawinski@linux.intel.com>,
 "Xiao W Wang" <xiao.w.wang@intel.com>,
 "Palmer Dabbelt" <palmer@rivosinc.com>,
 "Charlie Jenkins" <charlie@rivosinc.com>,
 "Namhyung Kim" <namhyung@kernel.org>, "Huacai Chen" <chenhuacai@kernel.org>,
 "Youling Tang" <tangyouling@loongson.cn>,
 "Tiezhu Yang" <yangtiezhu@loongson.cn>, "Jinyang He" <hejinyang@loongson.cn>,
 Linux-Arch <linux-arch@vger.kernel.org>, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] bitops: Change function return types from long to int
Content-Type: text/plain

On Fri, May 3, 2024, at 16:49, Thorsten Blum wrote:
> On 22. Apr 2024, at 17:55, Arnd Bergmann <arnd@arndb.de> wrote:
>>> 
>>> I can generally merge such a series with architecture specific
>>> changes through the asm-generic tree, with the appropriate Acks
>>> from the maintainers.
>
> What would it take for this patch (with only generic type changes) to be
> applied?
>
> I did some further investigations and disassembled my test kernel images.
> The patch reduced the number of ARM instructions by 872 with GCC 13 and by
> 2,354 with GCC 14. Other architectures that rely on the generic bitops
> functions will most likely also benefit from this patch.
>
> Tests were done with base commit 9d1ddab261f3e2af7c384dc02238784ce0cf9f98.

Sorry for failing to follow up earlier, I think this is ok, thanks
for your thorough work on this. I've applied it to the asm-generic
tree now.

     Arnd