于 2014年09月04日 18:30, [email protected] 写道:
> Hi!
>> parent use signal handler to deal with SIGTERM, make sure children processes
>> will be over before parent.
>> command "killall fsstress; wait;" can be used with this patch.
> This patch seems to be verbatim copy of the fix included in the xfstests
> repository. You should at least mention the original authors in the
> commit message here.
>
> And the commit message for the original patch is better as well...
>
> I'm all for getting back the fixes from xfstests but taking the patch
> and not giving credit to the original authors is plain wrong.
>
Sorry about that. That's a mistake and not intended.
I said it took from xfstests in my first email about this patch. I just 
forgot to do that after I ping you and re-created this patch several 
times. I will notice that in the future. Sorry again.

BTW, I am 'not' do nothing and just verbatim copy from xfstests. LTP 
fsstress.c is not all same with xfstests fsstress.c, I have tried to 
make the patch suitable for LTP, and did many test about the patch, to 
make sure it's good.

You can write the original authors and without my name, I just hope this 
feature can be in fsstress. That will useful for our test.

Thank you,
Zorro Lang

------------------------------------------------------------------------------
Slashdot TV.  
Video for Nerds.  Stuff that matters.
http://tv.slashdot.org/
_______________________________________________
Ltp-list mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ltp-list

Reply via email to