Re: [PATCH] lib: add statsort(), a distribution/bucket sort

[Jonathan Corbet]

Peter Taraba <peter.schuster.taraba@xxxxxxxxx> writes:

> Hi Jon and anyone who is reading it,
>
> here are your answers:
>
>>> We don't add new code to the kernel for users that are "secrets".  If
>>> you have an intended user for this code, include it with the series,
>>> please.
>
> I guess I will just go ahead and do it through my work around and keep my
> secrets (dont need to get my ideas to Microsoft, Google or even worse -
> to Facebook, etc. for free). If you change your mind about it in future,
> please do let me know.

OK, good luck with that.  The kernel does not add code to support
proprietary projects.

>>> There is a recursive function in there, which we tend to avoid for the
>>> kernel.  At a minimum, you would need to provide a convincing
>>> explanation of how you prevent it from overrunning the kernel stack.
>
> I am fully aware. Wanted more feedback (not only feedback which I get from AI).
> Thank you.

Feedback is what you are getting, but you seem remarkably uninterested
in it.

>>> Until you showed up with this patch, you have never been seen on the
>>> kernel lists; what inspired you to make this change now?  Was this code
>>> created with assistance from large language models?
>
> I wanted to contribute to linux kernel as I am linux user and it would
> be nice to contribute to Linux community (kind of a proud moment for me).
> Oh well.

If you want to contribute to a community, you need to spend some time
understanding how that community works.  It is not just by chance that
this project remains vital after all these years.  You are asking us to
take on a chunk of code that we will have to maintain indefinitely; we
have learned not to do that without good reason.

> Also, you should do code review, not ask useless questions if LLM wrote the
> code. Even if I answered your questions, you wouldn't have to believe me -
> so what's the point of asking. You should make decision based on the code
> and if it is good for Linux kernel purely.

You don't get to tell me what I should do, sorry.

The community's rules require disclosure of LLM use.  Many first-time
contributors fail to do that, and the reason is often ignorance of the
rules; once informed, they play along.  The question is anything but
useless.

Plus, it seems clear that you have answered it.

Thanks,

jon