From mboxrd@z Thu Jan 1 00:00:00 1970 From: Benjamin Herrenschmidt Subject: Re: [GIT] Networking Date: Tue, 03 Nov 2015 11:56:16 +1100 Message-ID: <1446512176.17404.30.camel@kernel.crashing.org> References: <20151027.233235.1641084823622810663.davem@davemloft.net> <5637C8DF.800@kernel.org> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Cc: Andy Lutomirski , David Miller , Hannes Frederic Sowa , Andrew Morton , Network Development , Linux Kernel Mailing List , Linus Torvalds To: Andy Lutomirski Return-path: In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org On Mon, 2015-11-02 at 13:30 -0800, Andy Lutomirski wrote: > > I'll stop making inane arguments if you stop bashing arguments I > didn't make. :) I said the helpers were useful for multiplication (by > which I meant both signed and unsigned) and, to a lesser extent, for > signed addition and subtraction. > > I don't believe I even tried to justify usub_overflow as anything > other than an extremely minor optimization that probably isn't > worthwhile. Also how much of the problem is simply that the function signature (naming and choice of arguments) just plain sucks ? Cheers, Ben.