Re: 答复: 答复: 答复: [PATCH V10] kdb: Fix the deadlock issue in KDB debugging.

From: Greg KH
Date: Wed Apr 10 2024 - 04:24:50 EST


On Wed, Apr 10, 2024 at 06:30:59AM +0000, Liuye wrote:
> >On Wed, Apr 10, 2024 at 06:10:10AM +0000, Liuye wrote:
> >> >On Wed, Apr 10, 2024 at 05:54:08AM +0000, Liuye wrote:
> >> >> >> Signed-off-by: Greg KH <gregkh@xxxxxxxxxxxxxxxxxxx>
> >> >> >
> >> >> >I have NOT signed off on this commit. You just said that I made a legal statement about this commit without that actually being true???
> >> >> >
> >> >> >Sorry, but that is flat out not acceptable at all. Please go work with your company lawyers to figure out what you did and come back with an explaination of exactly what this is and how it will not happen again.
> >> >> >
> >> >>
> >> >> >> Signed-off-by: Andy Shevchenko <andy.shevchenko@xxxxxxxxx>
> >> >> >
> >> >> >> V9 -> V10 : Add Signed-off-by of Greg KH and Andy Shevchenko,
> >> >> >> Acked-by of Daniel Thompson
> >> >> >
> >> >> >Huh?!
> >> >>
> >> >> @greg k-h :
> >> >> @Andy Shevchenko :
> >> >>
> >> >> Sorry, it was my mistake. I misunderstood the meaning of "signed-off-by", which led to usage issues.
> >> >>
> >> >> I want to express my gratitude for the suggestions on the patch from the two of you.
> >> >>
> >> >> What do I need to do now? Release PATCH V11 and delete these two signatures in it ?
> >> >
> >> >As I said, go work with your corporate lawyers on this to understand what just happened and have them let us know how it will not happen again.
> >>
> >> I'm very sorry, this is my first time submitting a patch and I made a significant mistake in using "Signed-off-by". I now understand the meaning of this field and will not make the same mistake again in the future.
> >
> >Understood, but you still need to go work with your corporate legal group so that you all ensure this does not happen again for any other developer in your company, as I am sure they will want to know about this.
>
> Ok, I will report this to the company. At the same time, I will add an audit mechanism to the patch submission process. Thanks again for your reminder.
>
> I will remove this part in PATCH V11.

No, you will need to do this before we can accept your change. And some
sort of verification that this is now in place properly for obvious
reasons.

thanks,

greg k-h