Re: S390: Fine-tuning for six function implementations

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> It would be different if your patches fixed actual bugs.

I dare to point change possibilities out which correspond to a special error category.
There can be different opinions about their relevance for further software improvements.


> This is just mindless code transformations that MAY in the best case save a few bytes
> of code here and there (I don't know; you didn't say).

Do you know the run time characteristics for the discussed functions good enough?
http://elixir.free-electrons.com/linux/v4.11/source/fs/seq_file.c#L405


> But the potential gains from these incredibly numerous and tiny patches
> that don't fix anything are so small, it's a waste of time, bandwidth,
> and mental capacity for you and for everybody involved.

I suggest a bit of code reduction at various places once more.


> I just searched my inbox for patches from you and you sent literally
> _hundreds_ over the past few days,

I sent update suggestions in this scale since the year 2014.


> all doing this crazy printf/puts/putc transformation.

I agree that the corresponding number could be remarkable.
But there are also other source code search patterns involved besides information
around these logging functions.


> Another bit of searching and I see that I'm not the first one giving you
> this response:

You are right that some agreements and disagreements were expressed already
(depending on the software area).

Regards,
Markus
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Kernel Development]     [Kernel Announce]     [Kernel Newbies]     [Linux Networking Development]     [Share Photos]     [IDE]     [Security]     [Git]     [Netfilter]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Device Mapper]

  Powered by Linux