Re: [PATCH 6/6] mce-inject: use injected mce only during faked handler call

[Hidetoshi Seto <seto.hidetoshi@jp.fujitsu.com>]

Huang Ying wrote:
> On Fri, 2009-10-09 at 16:31 +0800, Hidetoshi Seto wrote: 
>> In short, I believe that my idea is "use two flags" and your idea is
>> "share one flag" ... right?
> 
> No. There are many flags in .inject_flags (MCJ_EXCEPTION, MCJ_LOADED,
> etc), .finished is just another flag.

Maybe I should have said "to indicate 'loaded' and 'ready for consume'",
"use two variables" and "share one variable."

> It is OK to discuss the name and places of the two flags, but you should
> not send out a similar patch and declare it is your idea. 

I have experienced that some person received an alternative patch from
another person saying that "hey, how about this patch to do same thing in
an other way?" and that the original author replied "oh, it looks much
better, use it instead of mine" with his Acked-by:.

Now I have learned you are not one of such open-minded people, so I will
never try to make such patch for you again.  Please throw all of my patches
posted last week in the trash.

>> The only thing what I want to do here is merge your "fix" into upstream.
> 
> You should provide comment, instead of sending out a similar patch and
> declaring it is your idea.

I hope you could flexibly reflect comments from others, without saying like
"I don't think this is necessary because it works fine."

How long do I have to wait your next post, which will fix my urgent issue on
Nehalem?


Thanks,
H.Seto