Re: [Linux-kernel-mentees] Linux Kernel: Evaluate and Improve checkpatch.pl Mentorship Program

quoted

On Sat, Feb 13, 2021 at 1:02 PM Lukas Bulwahn [off-list ref]
wrote:

On Sa., 13. Feb. 2021 at 00:34, Divya Goswami [off-list ref]
wrote:

Hello Lukas,
                   I have been studying the checkpatch.pl script for a
while now. And as the first task was assigned, I have successfully run a
lot of patches through it. Most of them gave no errors/warnings. Those who
did so far, have been collected (and more to be collected) on my github
repo: checkpatch.pl-results
<https://github.com/rachejazz/checkpatch.pl-results>
Before I proceed further, I wanted to make sure whether I am uploading
the files/outputs in the desired way you wanted. If the file naming or the
hierarchy or anything that should be worked upon, do let me know.
Till then, I continue to work on the given tasks.
All good. It could be that you observe different results when you run
checkpatch in the kernel git root directory compared to the setup you have
now by just copying it into a different directory.

But the task for now is to just run it over the last release, so e.g.,
from v5.10 to v5.11 (which probably comes out tomorrow afternoon Pacific
Time) and create statistics on these results. If you got that, we can start
finetuning.

Please continue.

Lukas

 Hello Lukas,
I have analyzed commits and got the lists alright. Also I have analyzed the
errors and warnings generated by them.
Please do check my repo: checkpatch.pl-results

<https://github.com/rachejazz/checkpatch.pl-results>
Regards,
Divya
On Sun, Feb 14, 2021 at 2:03 PM Divya Goswami [off-list ref] wrote:

On Sat, Feb 13, 2021 at 1:02 PM Lukas Bulwahn [off-list ref] wrote:

On Sa., 13. Feb. 2021 at 00:34, Divya Goswami [off-list ref] wrote:
Hello Lukas,
                   I have been studying the checkpatch.pl script for a while now. And as the first task was assigned, I have successfully run a lot of patches through it. Most of them gave no errors/warnings. Those who did so far, have been collected (and more to be collected) on my github repo: checkpatch.pl-results
Before I proceed further, I wanted to make sure whether I am uploading the files/outputs in the desired way you wanted. If the file naming or the hierarchy or anything that should be worked upon, do let me know.
Till then, I continue to work on the given tasks.

All good. It could be that you observe different results when you run checkpatch in the kernel git root directory compared to the setup you have now by just copying it into a different directory.

But the task for now is to just run it over the last release, so e.g., from v5.10 to v5.11 (which probably comes out tomorrow afternoon Pacific Time) and create statistics on these results. If you got that, we can start finetuning.

Please continue.

Lukas
 Hello Lukas,
I have analyzed commits and got the lists alright. Also I have analyzed the errors and warnings generated by them.
Please do check my repo: checkpatch.pl-results
Looks good. Too bad that you cannot run on a large dataset.

The next task is to look at three types of warnings you encounter and
describe the purpose of the check.

Lukas
_______________________________________________
Linux-kernel-mentees mailing list
Linux-kernel-mentees@lists.linuxfoundation.org
https://lists.linuxfoundation.org/mailman/listinfo/linux-kernel-mentees
On Sun, Feb 14, 2021 at 10:20 PM Lukas Bulwahn [off-list ref]
wrote:

On Sun, Feb 14, 2021 at 2:03 PM Divya Goswami [off-list ref]
wrote:

On Sat, Feb 13, 2021 at 1:02 PM Lukas Bulwahn [off-list ref]
wrote:

On Sa., 13. Feb. 2021 at 00:34, Divya Goswami <
divyadeepti2000@gmail.com> wrote:
Hello Lukas,
                   I have been studying the checkpatch.pl script for
a while now. And as the first task was assigned, I have successfully run a
lot of patches through it. Most of them gave no errors/warnings. Those who
did so far, have been collected (and more to be collected) on my github
repo: checkpatch.pl-results
Before I proceed further, I wanted to make sure whether I am uploading
the files/outputs in the desired way you wanted. If the file naming or the
hierarchy or anything that should be worked upon, do let me know.
Till then, I continue to work on the given tasks.

All good. It could be that you observe different results when you run
checkpatch in the kernel git root directory compared to the setup you have
now by just copying it into a different directory.
But the task for now is to just run it over the last release, so e.g.,
from v5.10 to v5.11 (which probably comes out tomorrow afternoon Pacific
Time) and create statistics on these results. If you got that, we can start
finetuning.
Please continue.

Lukas
 Hello Lukas,
I have analyzed commits and got the lists alright. Also I have analyzed
the errors and warnings generated by them.
Please do check my repo: checkpatch.pl-results
Looks good. Too bad that you cannot run on a large dataset.

The next task is to look at three types of warnings you encounter and
describe the purpose of the check.

Lukas
Hello Lukas,
I have added step3 in my analysis.
I have tried it a bit differently to get ALL the non merge commits between
5.10 to 5.11. Now I can work on a large dataset :) Do check:
checkpatch.pl-results
<https://github.com/rachejazz/checkpatch.pl-results/tree/master/step3>

Regards,
Divya

`h`	back out one level
`j`	next message in thread
`k`	previous message in thread
`l`	drill in
`Esc`	close help / fold thread tree
`?`	toggle this help