From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 043FBC77B76 for ; Tue, 18 Apr 2023 02:22:04 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231167AbjDRCWC (ORCPT ); Mon, 17 Apr 2023 22:22:02 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:57992 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229621AbjDRCWA (ORCPT ); Mon, 17 Apr 2023 22:22:00 -0400 Received: from szxga01-in.huawei.com (szxga01-in.huawei.com [45.249.212.187]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id E7D34E7E; Mon, 17 Apr 2023 19:21:57 -0700 (PDT) Received: from kwepemm600004.china.huawei.com (unknown [7.193.23.242]) by szxga01-in.huawei.com (SkyGuard) with ESMTP id 4Q0nhB5NKWzsR94; Tue, 18 Apr 2023 10:20:26 +0800 (CST) Received: from [10.67.103.231] (10.67.103.231) by kwepemm600004.china.huawei.com (7.193.23.242) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.23; Tue, 18 Apr 2023 10:21:55 +0800 Message-ID: <3d8e8817-12b9-62bc-4c04-34d8822d366f@huawei.com> Date: Tue, 18 Apr 2023 10:21:54 +0800 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101 Thunderbird/91.2.0 Subject: Re: [PATCH v2 0/2] mailbox: pcc: Support platform notification for type4 and shared interrupt To: Robbie King , Sudeep Holla CC: "linux-acpi@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "rafael@kernel.org" , "rafael.j.wysocki@intel.com" , "wanghuiqiang@huawei.com" , "zhangzekun11@huawei.com" , "wangxiongfeng2@huawei.com" , "tanxiaofei@huawei.com" , "guohanjun@huawei.com" , "xiexiuqi@huawei.com" , "wangkefeng.wang@huawei.com" , "huangdaode@huawei.com" References: <20221016034043.52227-1-lihuisong@huawei.com> <20230314111135.16520-1-lihuisong@huawei.com> <20230327113326.dgrb5ainl6mv7wr5@bogus> <570a6f6d-87cb-48ca-3bbc-cd1221bfa88c@huawei.com> <7ef4eec4-2cfb-6f66-a9b9-9c130a1b71d8@huawei.com> <57c537d8-6728-2ffc-3421-accd79c1eddf@huawei.com> From: "lihuisong (C)" In-Reply-To: Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Transfer-Encoding: 8bit X-Originating-IP: [10.67.103.231] X-ClientProxiedBy: dggems705-chm.china.huawei.com (10.3.19.182) To kwepemm600004.china.huawei.com (7.193.23.242) Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 在 2023/4/14 21:48, Robbie King 写道: > Sorry for the delay. I ran my simple stress test against the patch set and > saw no issues. For the record it is by no means a thorough regression, but it > has illuminated issues in the past. Thanks for your testing. > > The test itself uses a "heartbeat" module in the SCP firmware that generates > notifications at a programmable interval. The stress test is simply generating > these heartbeats (SCP to AP notifications) while also generating protocol version > queries (AP to SCP). The notifications are sequence numbered to verify none are > lost, however SCP to AP notification support does not support SCP generating > notifications faster than the AP can process them, so the heartbeat rate must be > reasonably slow (on the order of 10s of millliseconds). I understand your concern. I think this doesn't get int the way of what we are doing. My stress tests were also run in type3 and type4 concurrent scenarios. There were two drivers using type3 to send command looply on platform. In the firmware terminal window, there were two channels for type4 to generate notifications from platform at the 1ms(even shorter) interval. I didn't find anything issues in this stress after running a couple of hours. @Robbie King and @Sudeep, what do you think of my test? > > -----Original Message----- > From: lihuisong (C) > Sent: Thursday, April 13, 2023 9:05 PM > To: Robbie King ; Sudeep Holla > Cc: linux-acpi@vger.kernel.org; linux-kernel@vger.kernel.org; rafael@kernel.org; rafael.j.wysocki@intel.com; wanghuiqiang@huawei.com; zhangzekun11@huawei.com; wangxiongfeng2@huawei.com; tanxiaofei@huawei.com; guohanjun@huawei.com; xiexiuqi@huawei.com; wangkefeng.wang@huawei.com; huangdaode@huawei.com > Subject: Re: [PATCH v2 0/2] mailbox: pcc: Support platform notification for type4 and shared interrupt > > > 在 2023/4/11 22:47, Robbie King 写道: >> Apologies, missed earlier emails. Will make this a priority for the week. > Thanks. Looking forward to you reply. >> -----Original Message----- >> From: lihuisong (C) >> Sent: Sunday, April 9, 2023 9:27 PM >> To: Sudeep Holla ; Robbie King >> >> Cc: linux-acpi@vger.kernel.org; linux-kernel@vger.kernel.org; >> rafael@kernel.org; rafael.j.wysocki@intel.com; >> wanghuiqiang@huawei.com; zhangzekun11@huawei.com; >> wangxiongfeng2@huawei.com; tanxiaofei@huawei.com; >> guohanjun@huawei.com; xiexiuqi@huawei.com; wangkefeng.wang@huawei.com; >> huangdaode@huawei.com >> Subject: Re: [PATCH v2 0/2] mailbox: pcc: Support platform >> notification for type4 and shared interrupt >> >> >> 在 2023/3/27 20:31, lihuisong (C) 写道: >>> 在 2023/3/27 19:33, Sudeep Holla 写道: >>>> On Tue, Mar 14, 2023 at 07:11:33PM +0800, Huisong Li wrote: >>>>> PCC supports processing platform notification for slave subspaces >>>>> and shared interrupt for multiple subspaces. >>>>> >>>> Other than a super minor nit in the patch 1/2, this looks good to me. >>>> It would be good if we can get tested-by from Robbie King >>>> >>>> >>>> Reviewed-by: Sudeep Holla >>> Thanks for your review.😁 >>> >>> @Robbie King, can you give us some feedback? >>> Looking forward to you reply. >>> >>> >> @Robbie King, kindly ping.