On Fri, Jan 15, 2021 at 05:26:22PM +0800, Aili Yao wrote: > On Fri, 15 Jan 2021 09:49:24 +0100 > Oscar Salvador <osalvador@xxxxxxx> wrote: > > > I am having a hard time trying to grasp what are you trying to achieve here. > > Could you elaborate some more? Ideally stating what is the problem you are > > fixing here. > > > Sorry for confusion, example: there are four process A,B,C,D,which map the same file into > there process space, which set there PF_MCE_KILL_EARLY flag to TRUE, if process A trigger one > UE with MF_ACTION_REQUIRED set, in current code, only process A will be killed, B,C,D remain > alive, but for the PF_MCE_KILL_EARLY we set, we want B,C,D also be killed. > this is what I want to fix. This should really be part of the changelog. (what is it fixing? how? why?) All this information should be there. And the title should be renamed as well as it is bit confusing as is. "Do not bail out in task_early_kill when force_early is set" or something along those lines. -- Oscar Salvador SUSE L3