<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small">Hi Xu, Petr,</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Petr Vorel <<a href="mailto:pvorel@suse.cz">pvorel@suse.cz</a>> wrote:<br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> I remember Amir also added a similar feature[1] to xfstests when he <br>
> become xfs 5.10 stable kernel maintainer.<br>
<br>
> So maybe we can add a third colume in tst_test struct's tag field  to <br>
> cover kernel commit title? Then this kernel comment can be removed.<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">We can achieve that but the complexity outweighs the benefits (for LTP).<br></div></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Amir got inspiration from LTP, but I'm not a big fan of adding manually git<br>
subject which is now added by script.  Also I'd like to address the change we<br>
agree also in fanotify sources. </blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
> Also, I don't have strong obejection to this patch because it seems <br>
> duplicate, but I just want to keep kernel commit title by using simple way.<br>
<br>
I also wonder why people add commits in this verbose form:<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">That's legacy way in writing comments, but obviously it need unify in format</div><div class="gmail_default" style="font-size:small">for better management in future.</div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
 *  commit 93bc420ed41df63a18ae794101f7cbf45226a6ef<br>
 *  Author: yangerkun <<a href="mailto:yangerkun@huawei.com" target="_blank">yangerkun@huawei.com</a>><br>
 *  Date:   Mon Feb 18 09:07:02 2019 +0800<br>
 *<br>
 *  ext2: support statx syscall<br>
<br>
If commits are really necessary I'd add them inline, i.e.:<br>
<br>
$ git log --pretty=format:'%h ("%s")' -1 93bc420ed41d<br>
93bc420ed41d ("ext2: support statx syscall")<br></blockquote><div><br></div><div class="gmail_default" style="font-size:small">+1, at most we add this format in normal C comments.</div><div class="gmail_default" style="font-size:small"><a href="https://github.com/linux-test-project/ltp/blob/master/testcases/kernel/syscalls/fanotify/fanotify14.c#L17">https://github.com/linux-test-project/ltp/blob/master/testcases/kernel/syscalls/fanotify/fanotify14.c#L17</a></div><div class="gmail_default" style="font-size:small"><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
But again, having git hash at the bottom and then full commit on the top looks<br>
to me just a redundancy (I often want to see changes in kernel thus run git show<br>
hash anyway + there is that link in docparse.<br></blockquote><div> </div><div><div class="gmail_default" style="font-size:small">me as well.</div></div></div><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>