Messages in this thread | | | From | Linus Torvalds <> | Date | Tue, 25 Jul 2017 08:43:39 -0700 | Subject | Re: [PATCH 2/3] lib/int_sqrt: Optimize initial value compute |
| |
On Tue, Jul 25, 2017 at 1:17 AM, Peter Zijlstra <peterz@infradead.org> wrote: > On Mon, Jul 24, 2017 at 10:35:56AM -0700, Linus Torvalds wrote: >> Ack. You have numbers, it's all good. > > Thanks! > >> Except I'd still want you to comment on why you cared and about which >> piece of your upcoming code this is going to matter for, ok? > > I did an RFC here: > > https://lkml.kernel.org/r/20170719133940.uytsixvfgpmo3ane@hirez.programming.kicks-ass.net
Christ Peter.
I meant add the explanation to the commit log, to explain *why* you care about this function that nobody has ever cared about before.
Please. Document your patches. Don't make people ask for explanations.
I've asked now *three* times about this, and still the patch submission had no explanation at all about why this mattered.
How many times do people have to ask before you start saying "oh, maybe I should explain why.." in your patch.
Linus
| |