<div dir="ltr"><div dir="ltr"><div>>>> A brute force grep in the kernel shows that there are 130493 " const " in there <br></div><div>Hi Hofrat, <br></div><div>1. may i know the command to do this above grep please..</div><div>2. (and the opposite) may i know the command to grep other type of variables/structures</div><div><br></div><div>thanks again.. your points and Greg's points are very clear. i was having this doubt from last Dec(from that tweet) and it got cleared. <br></div><div><br></div><div>Happy Ganesh Chaturthi to All.</div><div>( Lord Ganesh is widely revered as the remover of obstacles,<sup id="gmail-cite_ref-9" class="gmail-reference"><a href="https://en.wikipedia.org/wiki/Ganesha#cite_note-9">[9]</a></sup> the patron of arts and sciences and the <a href="https://en.wikipedia.org/wiki/Deva_(Hinduism)" title="Deva (Hinduism)">deva</a> of intellect and wisdom)<br></div></div></div><br><div class="gmail_quote"><div dir="ltr">On Wed, Sep 12, 2018 at 12:38 PM Nicholas Mc Guire <<a href="mailto:der.herr@hofr.at">der.herr@hofr.at</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Wed, Sep 12, 2018 at 11:50:35AM +0530, inventsekar wrote:<br>
> Hi All...<br>
> One curious question..<br>
> <br>
> Linux Foundation tweeted this -<br>
> Meet Bhumika Goyal, age 22, from India. She has had more 340 patches<br>
> accepted into the Linux kernel, which helped her land one of our two Linux<br>
> Kernel Guru LiFT scholarships:<br>
> <a href="https://twitter.com/linuxfoundation/status/940340927897489408?lang=en" rel="noreferrer" target="_blank">https://twitter.com/linuxfoundation/status/940340927897489408?lang=en</a><br>
> <br>
> and her commits are -<br>
> <a href="https://github.com/torvalds/linux/commits?author=bhumikagoyal" rel="noreferrer" target="_blank">https://github.com/torvalds/linux/commits?author=bhumikagoyal</a><br>
> <br>
> One of my friend told me that, most/many of patches are just converting<br>
> variables to "*constant*"<br>
> <br>
> For example - platform/chrome: chromeos_laptop: make chromeos_laptop const<br>
> -<br>
> <br>
> Declare chromeos_laptop structures as *const *as they are only used during<br>
> a copy operation. As their value is never modified during runtime, they<br>
> can be made const.<br>
> <br>
> <br>
> the question is that, how these many const variable issues are left/missed<br>
> by the previous developers?!?! per my little knowledge, this task looks<br>
> like a simple one.. many other Kernel Developers should have thought and<br>
> made these changes long before, but i am not able to understand why this<br>
> wasnt done until recently?!?!?<br>
><br>
well (almost) all issues are simple once they are known, and in 16M LoC or<br>
what ever the kernel currently is, it is not a big surprise to find a few<br>
hundred issues of almost any bug-class. If you really want to know how bad<br>
the problem is, or rather how much it says about the overall quality, you<br>
would need to answer at least two questions.<br>
 1) how many cases of const are missing<br>
 2) how many cases of const are in use correctly<br>
just counting how many are missing does not tell you much about how good or<br>
bad kernel developers are. If 99% of the cases are handled correctly I would<br>
say thats good - but given the size of the kernel that would probably mean<br>
that there still are quite a few that are not handled correctly. Further you<br>
would probably need to check how old the code was that that was fixed up.<br>
<br>
A brute force grep in the kernel shows that there are 130493 " const " in <br>
ther (as of 4.18.2 in -stable) so if the developers got 340 wrong - I would<br>
say thats not that scary after all the consequence of not having the const in<br>
there are quite benign.<br>
<br>
I think that it is often times not a mater of not seeing issues but it is a<br>
matter of priority and available time - and what this case really does show<br>
is that there are opportunities for people moving into kernel development<br>
to get to work on simple problems to start with where they can get well<br>
acquainted with the procedural issues first - I think that is a good<br>
thing and may actually be important to allow new developers to join in.<br>
<br>
<br>
thx!<br>
hofrat <br>
</blockquote></div><br clear="all"><br>-- <br><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr">Best Regards,<br>Sekar<br></div></div></div></div>