[dpdk-ci] [PATCH v4 7/7] tests: add checkpatch

Wei, FangfangX fangfangx.wei at intel.com
Tue Dec 13 10:24:32 CET 2016


>> > About the title:
>> >	[dpdk-test-report] |SUCCESS| pw17860 [PATCH, v2] vmxnet3: fix Rx deadlock
>> >	Please remove the patchwork id pw17860, and remove the comma in [PATCH, v2].
>> 
>> I have remove the comma. 
>> For the patchwork id "pw17860", I think it would be better to be remained. Because if it's removed from the subject, receivers can't be directly told which patch has been complicated, they must open the email to check it. 

>I don't understand.
>With the title "|SUCCESS| [PATCH v2] vmxnet3: fix Rx deadlock]", the receiver see which patch was tested, isn't it?

I mean if removed patchwork id, the receiver know one patch was tested, but they don't see the patch id unless open the email.
But we need the patchwork id for two reasons:
1. For debug, I want to see if the patches were tested in case of leakage tested.
2. We summary the tested patches status weekly. It is intuitive to see the patchwork id in subject.

Best Regards
Fangfang Wei


More information about the ci mailing list