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 BBB90C433F5 for ; Sat, 18 Dec 2021 23:29:01 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 2D2346B0071; Sat, 18 Dec 2021 18:28:51 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 282326B0073; Sat, 18 Dec 2021 18:28:51 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 122856B0074; Sat, 18 Dec 2021 18:28:51 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0165.hostedemail.com [216.40.44.165]) by kanga.kvack.org (Postfix) with ESMTP id 03B286B0071 for ; Sat, 18 Dec 2021 18:28:51 -0500 (EST) Received: from smtpin23.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay04.hostedemail.com (Postfix) with ESMTP id BFDE089B20 for ; Sat, 18 Dec 2021 23:28:40 +0000 (UTC) X-FDA: 78932506800.23.842B5AF Received: from mail-pf1-f176.google.com (mail-pf1-f176.google.com [209.85.210.176]) by imf06.hostedemail.com (Postfix) with ESMTP id 62B04180030 for ; Sat, 18 Dec 2021 23:28:39 +0000 (UTC) Received: by mail-pf1-f176.google.com with SMTP id i12so5378966pfd.6 for ; Sat, 18 Dec 2021 15:28:40 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc:content-transfer-encoding; bh=pIYxkiGisug+VVJYFtmCG20TylNRXB8zO8kw91s3wyc=; b=TIuyQXz6oo8PN+yMbImlC2c2rmmoCdLu5CdlQlReccXapNNMDZyLSkZvnzlgZ9AIdV 6+qzntuQo5gBX86FaZFlCIaH4FCNC7RUVw367F08QPEjoiFWXa2CV6SAzISHS5RZPpY/ QbkHqa0fdrihFEuRX0X/GeMy6P75DvnBsKWfABXcb5D5/2zTgkKaV4VEiJSuyy+zBwcd zNIVzererH+Fx7ZEOWvTBpmd3THvXh1KHtzsLHvjMiIPuGFGIqMdZbyjf9H7vLYP6+7J cQ/P2vAafsI3DEFtzgX9UWRj1ZmCeXy01hG6ih9PN1N9Tz70SEQsg8aJQpCrrbXl1cqP QppQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc:content-transfer-encoding; bh=pIYxkiGisug+VVJYFtmCG20TylNRXB8zO8kw91s3wyc=; b=5z3U1WG7mItDmaF/l3xj6ZtLvsVU+cXRzJSmOl1p9Ob0HwA2RNXbc64I10oy2HEM8K zlk5jjxKXkJBvoblAGv6fWohbzQzcIWg+i/1xXCEHfecSExeOEf0zlQA8V1CS4Vcwonn Jcihkd/ZfmCXnQacPLRH+nMgGcX6zmlgRWtEJhxrfMmsb8SPCXyGjbRcIOSMBaTeagNp umtXLKgNTScqr+7sgRjUb1vN7c+R19odcshSoNuiLxz6R6jkRL3rtqgOG67hBKejyLkX ZA8/iEC2Xa6DDPjONSnYlOAdp1W3YPkWIGfjGCLLDAaRZqn6355mGQBv0Ht+2VvhJoIo kDXQ== X-Gm-Message-State: AOAM531Wxs/U8FSpkv0q1E7//yeFwi/V0YnMSXojZe39aXbrgkJmJ1rx FY+mxdZ76wkRlskOOhEZQg6nOoF8ZAfURWaBiLQ= X-Google-Smtp-Source: ABdhPJxWDXMTWC77oOKCjFiIpES/JejUzoFwUACXrmSW9T47MOn3N7k2dUXoB6NpZZlsfwgvBd+k97C3LkDR7q0l13Y= X-Received: by 2002:a63:2a0d:: with SMTP id q13mr8815341pgq.513.1639870119035; Sat, 18 Dec 2021 15:28:39 -0800 (PST) MIME-Version: 1.0 References: <20211218212014.1315894-1-yury.norov@gmail.com> <20211218212014.1315894-2-yury.norov@gmail.com> In-Reply-To: From: Yury Norov Date: Sat, 18 Dec 2021 15:28:29 -0800 Message-ID: Subject: Re: [PATCH 01/17] all: don't use bitmap_weight() where possible To: =?UTF-8?B?TWljaGHFgiBNaXJvc8WCYXc=?= Cc: linux-kernel@vger.kernel.org, "James E.J. Bottomley" , "Martin K. Petersen" , "Paul E. McKenney" , "Rafael J. Wysocki" , Alexander Shishkin , Alexey Klimov , Amitkumar Karwar , Andi Kleen , Andrew Lunn , Andrew Morton , Andy Gross , Andy Lutomirski , Andy Shevchenko , Anup Patel , Ard Biesheuvel , Arnaldo Carvalho de Melo , Arnd Bergmann , Borislav Petkov , Catalin Marinas , Christoph Hellwig , Christoph Lameter , Daniel Vetter , Dave Hansen , David Airlie , David Laight , Dennis Zhou , Emil Renner Berthing , Geert Uytterhoeven , Geetha sowjanya , Greg Kroah-Hartman , Guo Ren , Hans de Goede , Heiko Carstens , Ian Rogers , Ingo Molnar , Jakub Kicinski , Jason Wessel , Jens Axboe , Jiri Olsa , Joe Perches , Jonathan Cameron , Juri Lelli , Kees Cook , Krzysztof Kozlowski , Lee Jones , Marc Zyngier , Marcin Wojtas , Mark Gross , Mark Rutland , Matti Vaittinen , Mauro Carvalho Chehab , Mel Gorman , Michael Ellerman , Mike Marciniszyn , Nicholas Piggin , Palmer Dabbelt , Peter Zijlstra , Petr Mladek , Randy Dunlap , Rasmus Villemoes , Russell King , Saeed Mahameed , Sagi Grimberg , Sergey Senozhatsky , Solomon Peachy , Stephen Boyd , Stephen Rothwell , Steven Rostedt , Subbaraya Sundeep , Sudeep Holla , Sunil Goutham , Tariq Toukan , Tejun Heo , Thomas Bogendoerfer , Thomas Gleixner , Ulf Hansson , Vincent Guittot , Vineet Gupta , Viresh Kumar , Vivien Didelot , Vlastimil Babka , Will Deacon , bcm-kernel-feedback-list@broadcom.com, kvm@vger.kernel.org, linux-alpha@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-crypto@vger.kernel.org, linux-csky@vger.kernel.org, linux-ia64@vger.kernel.org, linux-mips@vger.kernel.org, linux-mm@kvack.org, linux-perf-users@vger.kernel.org, linux-riscv@lists.infradead.org, linux-s390@vger.kernel.org, linux-snps-arc@lists.infradead.org, linuxppc-dev@lists.ozlabs.org Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Rspamd-Queue-Id: 62B04180030 X-Stat-Signature: a74u4qrzjptnkz5bmmsgq449k8e3jiog Authentication-Results: imf06.hostedemail.com; dkim=pass header.d=gmail.com header.s=20210112 header.b=TIuyQXz6; dmarc=pass (policy=none) header.from=gmail.com; spf=pass (imf06.hostedemail.com: domain of yury.norov@gmail.com designates 209.85.210.176 as permitted sender) smtp.mailfrom=yury.norov@gmail.com X-Rspamd-Server: rspam02 X-HE-Tag: 1639870119-485956 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 Sat, Dec 18, 2021 at 2:16 PM Micha=C5=82 Miros=C5=82aw wrote: > > On Sat, Dec 18, 2021 at 01:19:57PM -0800, Yury Norov wrote: > > Don't call bitmap_weight() if the following code can get by > > without it. > > > > Signed-off-by: Yury Norov > > --- > > drivers/net/dsa/b53/b53_common.c | 6 +----- > > drivers/net/ethernet/broadcom/bcmsysport.c | 6 +----- > > drivers/thermal/intel/intel_powerclamp.c | 9 +++------ > > 3 files changed, 5 insertions(+), 16 deletions(-) > [...] > > Looks good, Does it mean Acked-by, Reviewed-by, or something else? > but I think this needs to be split per subsystem. What you ask breaks rules: Documentation/process/submitting-patches.rst: Separate each **logical change** into a separate patch. For example, if your changes include both bug fixes and performance enhancements for a single driver, separate those changes into two or more patches. If your changes include an API update, and a new driver which uses that new API, separate those into two patches. On the other hand, if you make a single change to numerous files, group those changes into a single patch. Thus a single logical change is contained within a single patch. This is not a dead rule, refer for example the 96d4f267e40f9 ("Remove 'type' argument from access_ok() functioin.") Or this: https://lkml.org/lkml/2021/6/14/1736 Thanks, Yury