Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Feedback-ID: ifd894703:Fastmail
Message-ID: <500758f7-f749-469b-1419-0c08b01c9eab@flygoat.com>
Date:   Thu, 31 Aug 2023 18:38:07 +0800
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Thunderbird/102.14.0
Content-Language: en-GB
To:     "Maciej W. Rozycki" <macro@orcam.me.uk>,
        Tiezhu Yang <yangtiezhu@loongson.cn>
Cc:     linux-mips@vger.kernel.org,
        Thomas Bogendoerfer <tsbogend@alpha.franken.de>,
        linux-kernel@vger.kernel.org, loongson-kernel@lists.loongnix.cn
References: <1692434183-2054-1-git-send-email-yangtiezhu@loongson.cn>
 <ZOxkt/6EkQIy+Jkt@alpha.franken.de> <ZO4Mdr9/XUkXDK9j@alpha.franken.de>
From:   Jiaxun Yang <jiaxun.yang@flygoat.com>
Subject: Re: [PATCH v4 0/4] Modify die() for MIPS
In-Reply-To: <ZO4Mdr9/XUkXDK9j@alpha.franken.de>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit
Precedence: bulk


在 2023/8/29 23:19, Thomas Bogendoerfer 写道:
> On Mon, Aug 28, 2023 at 11:11:19AM +0200, Thomas Bogendoerfer wrote:
>> On Sat, Aug 19, 2023 at 04:36:19PM +0800, Tiezhu Yang wrote:
>>> v4:
>>>    -- Add BUG() at the end of nmi_exception_handler()
>>>    -- Return earlier in die() if notify_die() returns NOTIFY_STOP
>>>    -- Update the patch titles and commit messages
>>>
>>> v3:
>>>    -- Make each patch can be built without errors and warnings.
>>>
>>> v2:
>>>    -- Update the commit message to give more detailed info, split into
>>>       three individual patches, suggested by Maciej, thank you.
>>>
>>> Tiezhu Yang (4):
>>>    MIPS: Add BUG() at the end of nmi_exception_handler()
>>>    MIPS: Do not kill the task in die() if notify_die() returns
>>>      NOTIFY_STOP
>>>    MIPS: Return earlier in die() if notify_die() returns NOTIFY_STOP
>>>    MIPS: Add identifier names to arguments of die() declaration
>>>
>>>   arch/mips/include/asm/ptrace.h |  2 +-
>>>   arch/mips/kernel/traps.c       | 15 +++++++++------
>>>   2 files changed, 10 insertions(+), 7 deletions(-)
>> series applied to mips-next.
> I've dropped the series again after feedback from Maciej, that this
> still needs more changes.
Was the feedback made off-list?

I think it's important to make submitter aware of the problem before
action being taken.

 From my perspective Teizhu was working hard to address review comments
all the way so it looks unfair to neglect his work without transparent 
communication.

Thanks
- Jiaxun
> Thomas.