Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Mon, 7 Aug 2023 22:58:27 +0100 (BST)
From:   "Maciej W. Rozycki" <macro@orcam.me.uk>
To:     Borislav Petkov <bp@alien8.de>
cc:     Arnd Bergmann <arnd@arndb.de>, Arnd Bergmann <arnd@kernel.org>,
        linux-kernel@vger.kernel.org, x86@kernel.org,
        Thomas Gleixner <tglx@linutronix.de>,
        Ingo Molnar <mingo@redhat.com>,
        Dave Hansen <dave.hansen@linux.intel.com>,
        "H. Peter Anvin" <hpa@zytor.com>
Subject: Re: [PATCH 2/5] [RESEND] x86: avoid unneeded __div64_32 function
 definition
In-Reply-To: <20230807211659.GKZNFfS+7PK71yYt0p@fat_crate.local>
Message-ID: <alpine.DEB.2.21.2308072249100.38537@angie.orcam.me.uk>
References: <20230725134837.1534228-1-arnd@kernel.org> <20230725134837.1534228-3-arnd@kernel.org> <20230801170315.GGZMk60zojZOeuUwX7@fat_crate.local> <baf750f4-a42c-453a-91dc-7fd457bc1e80@app.fastmail.com> <20230802172030.GEZMqQXmeb98Tm+Qhg@fat_crate.local>
 <alpine.DEB.2.21.2308072124320.38537@angie.orcam.me.uk> <20230807211659.GKZNFfS+7PK71yYt0p@fat_crate.local>
User-Agent: Alpine 2.21 (DEB 202 2017-01-01)
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Precedence: bulk

On Mon, 7 Aug 2023, Borislav Petkov wrote:

> On Mon, Aug 07, 2023 at 09:37:00PM +0100, Maciej W. Rozycki wrote:
> >  Otherwise you risk `__div64_32(n, base)' getting expanded to `(n, base)', 
> 
> You mean in the very obscure case of a 32-bit kernel where they don't call
> do_div() but call this low level function?
> 
> I'd say if they can't be bothered to even grep the tree for the right usage,
> they deserve both pieces... :)

 Well, people do make mistakes and life is tough enough already, so why 
make it tougher when we can have a free sanity check?  I mean there's 
hardly any cost from the extra characters added and it can save someone 
hassle with debugging, which is always tough by definition.

 I've suffered from silly mistakes myself on many occasions, possibly from
being distracted in the middle of doing something, and while I figured 
things out eventually, it often cost me a day of effort or so wasted in 
chasing them.

  Maciej