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 lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 6F9D7C5AD49 for ; Fri, 30 May 2025 13:44:41 +0000 (UTC) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1uL01g-00067T-KM; Fri, 30 May 2025 09:44:01 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1uL01e-00067F-KS for qemu-devel@nongnu.org; Fri, 30 May 2025 09:43:58 -0400 Received: from [185.176.79.56] (helo=frasgout.his.huawei.com) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1uL01Y-000077-7j for qemu-devel@nongnu.org; Fri, 30 May 2025 09:43:58 -0400 Received: from mail.maildlp.com (unknown [172.18.186.231]) by frasgout.his.huawei.com (SkyGuard) with ESMTP id 4b84Fb2Jxcz6GFS4; Fri, 30 May 2025 21:43:31 +0800 (CST) Received: from frapeml500008.china.huawei.com (unknown [7.182.85.71]) by mail.maildlp.com (Postfix) with ESMTPS id EDD0F1402FF; Fri, 30 May 2025 21:43:36 +0800 (CST) Received: from localhost (10.203.177.66) by frapeml500008.china.huawei.com (7.182.85.71) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2507.39; Fri, 30 May 2025 15:43:36 +0200 Date: Fri, 30 May 2025 14:43:35 +0100 To: Anisa Su CC: Fan Ni , , , Subject: Re: [PATCH v2 01/10] cxl-mailbox-utils: Move opcodes enum to new header file Message-ID: <20250530144335.000064f3@huawei.com> In-Reply-To: References: <20250508001754.122180-1-anisa.su887@gmail.com> <20250508001754.122180-2-anisa.su887@gmail.com> X-Mailer: Claws Mail 4.3.0 (GTK 3.24.42; x86_64-w64-mingw32) MIME-Version: 1.0 Content-Type: text/plain; charset="US-ASCII" Content-Transfer-Encoding: 7bit X-Originating-IP: [10.203.177.66] X-ClientProxiedBy: lhrpeml100001.china.huawei.com (7.191.160.183) To frapeml500008.china.huawei.com (7.182.85.71) X-Host-Lookup-Failed: Reverse DNS lookup failed for 185.176.79.56 (deferred) Received-SPF: pass client-ip=185.176.79.56; envelope-from=jonathan.cameron@huawei.com; helo=frasgout.his.huawei.com X-Spam_score_int: -33 X-Spam_score: -3.4 X-Spam_bar: --- X-Spam_report: (-3.4 / 5.0 requ) BAYES_00=-1.9, RCVD_IN_DNSWL_MED=-2.3, RCVD_IN_MSPIKE_H5=0.001, RCVD_IN_MSPIKE_WL=0.001, RCVD_IN_VALIDITY_RPBL_BLOCKED=0.001, RCVD_IN_VALIDITY_SAFE_BLOCKED=0.001, RDNS_NONE=0.793, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-to: Jonathan Cameron From: Jonathan Cameron via Errors-To: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org Sender: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org On Tue, 20 May 2025 17:33:46 +0000 Anisa Su wrote: > On Tue, May 20, 2025 at 08:37:35AM -0700, Fan Ni wrote: > > On Thu, May 08, 2025 at 12:00:57AM +0000, anisa.su887@gmail.com wrote: > > > From: Anisa Su > > > > > > In preparation for the next patch, move opcodes enum to new cxl_opcodes.h file > > > for visibility from mailbox-utils.c and i2c_mctp_cxl.c, which checks that > > > certain command sets are bound with the correct MCTP binding. > > > > > > Signed-off-by: Anisa Su > > > --- > > > hw/cxl/cxl-mailbox-utils.c | 68 ++---------------------------------- > > > include/hw/cxl/cxl_opcodes.h | 64 +++++++++++++++++++++++++++++++++ > > > > Should we put the opcodes into include/hw/cxl/cxl_mailbox.h instead of > > creating a new file. cxl_mailbox.h only has some macros. > > > > Fan > > > I had some discussion with Jonathan in the v1 thread about this. We > agreed it is fine to use mailbox.h because it only has a few macros in it, > but in case more things get added to it later, I made a separate file. > Then no need to re-organize later. > > We can always move them in future if it turns out cxl_mailbox.h is not a good home. Also, note that the include file added in this patch has no descriptive comments or ifndef magic which would have wanted to be there. Jonathan > > > > > 2 files changed, 66 insertions(+), 66 deletions(-) > > > create mode 100644 include/hw/cxl/cxl_opcodes.h > > > > > > diff --git a/hw/cxl/cxl-mailbox-utils.c b/hw/cxl/cxl-mailbox-utils.c > > > index a02d130926..ed3294530f 100644 > > > --- a/hw/cxl/cxl-mailbox-utils.c > > > +++ b/hw/cxl/cxl-mailbox-utils.c > > > @@ -23,6 +23,7 @@ > > > #include "qemu/uuid.h" > > > #include "system/hostmem.h" > > > #include "qemu/range.h" > > > +#include "hw/cxl/cxl_opcodes.h" > > > > > > #define CXL_CAPACITY_MULTIPLIER (256 * MiB) > > > #define CXL_DC_EVENT_LOG_SIZE 8 > > > @@ -36,7 +37,7 @@ > > > > > > /* > > > * How to add a new command, example. The command set FOO, with cmd BAR. > > > - * 1. Add the command set and cmd to the enum. > > > + * 1. Add the command set and cmd to the enum in cxl_opcodes.h. > > > * FOO = 0x7f, > > > * #define BAR 0 > > > * 2. Implement the handler > > > @@ -59,71 +60,6 @@ > > > * a register interface that already deals with it. > > > */ > > > > > > -enum { > > > - INFOSTAT = 0x00, > > > - #define IS_IDENTIFY 0x1 > > > - #define BACKGROUND_OPERATION_STATUS 0x2 > > > - #define GET_RESPONSE_MSG_LIMIT 0x3 > > > - #define SET_RESPONSE_MSG_LIMIT 0x4 > > > - #define BACKGROUND_OPERATION_ABORT 0x5 > > > - EVENTS = 0x01, > > > - #define GET_RECORDS 0x0 > > > - #define CLEAR_RECORDS 0x1 > > > - #define GET_INTERRUPT_POLICY 0x2 > > > - #define SET_INTERRUPT_POLICY 0x3 > > > - FIRMWARE_UPDATE = 0x02, > > > - #define GET_INFO 0x0 > > > - #define TRANSFER 0x1 > > > - #define ACTIVATE 0x2 > > > - TIMESTAMP = 0x03, > > > - #define GET 0x0 > > > - #define SET 0x1 > > > - LOGS = 0x04, > > > - #define GET_SUPPORTED 0x0 > > > - #define GET_LOG 0x1 > > > - #define GET_LOG_CAPABILITIES 0x2 > > > - #define CLEAR_LOG 0x3 > > > - #define POPULATE_LOG 0x4 > > > - FEATURES = 0x05, > > > - #define GET_SUPPORTED 0x0 > > > - #define GET_FEATURE 0x1 > > > - #define SET_FEATURE 0x2 > > > - IDENTIFY = 0x40, > > > - #define MEMORY_DEVICE 0x0 > > > - CCLS = 0x41, > > > - #define GET_PARTITION_INFO 0x0 > > > - #define GET_LSA 0x2 > > > - #define SET_LSA 0x3 > > > - HEALTH_INFO_ALERTS = 0x42, > > > - #define GET_ALERT_CONFIG 0x1 > > > - #define SET_ALERT_CONFIG 0x2 > > > - SANITIZE = 0x44, > > > - #define OVERWRITE 0x0 > > > - #define SECURE_ERASE 0x1 > > > - #define MEDIA_OPERATIONS 0x2 > > > - PERSISTENT_MEM = 0x45, > > > - #define GET_SECURITY_STATE 0x0 > > > - MEDIA_AND_POISON = 0x43, > > > - #define GET_POISON_LIST 0x0 > > > - #define INJECT_POISON 0x1 > > > - #define CLEAR_POISON 0x2 > > > - #define GET_SCAN_MEDIA_CAPABILITIES 0x3 > > > - #define SCAN_MEDIA 0x4 > > > - #define GET_SCAN_MEDIA_RESULTS 0x5 > > > - DCD_CONFIG = 0x48, > > > - #define GET_DC_CONFIG 0x0 > > > - #define GET_DYN_CAP_EXT_LIST 0x1 > > > - #define ADD_DYN_CAP_RSP 0x2 > > > - #define RELEASE_DYN_CAP 0x3 > > > - PHYSICAL_SWITCH = 0x51, > > > - #define IDENTIFY_SWITCH_DEVICE 0x0 > > > - #define GET_PHYSICAL_PORT_STATE 0x1 > > > - TUNNEL = 0x53, > > > - #define MANAGEMENT_COMMAND 0x0 > > > - MHD = 0x55, > > > - #define GET_MHD_INFO 0x0 > > > -}; > > > - > > > /* CCI Message Format CXL r3.1 Figure 7-19 */ > > > typedef struct CXLCCIMessage { > > > uint8_t category; > > > diff --git a/include/hw/cxl/cxl_opcodes.h b/include/hw/cxl/cxl_opcodes.h > > > new file mode 100644 > > > index 0000000000..26d3a99e8a > > > --- /dev/null > > > +++ b/include/hw/cxl/cxl_opcodes.h > > > @@ -0,0 +1,64 @@ > > > +enum { > > > + INFOSTAT = 0x00, > > > + #define IS_IDENTIFY 0x1 > > > + #define BACKGROUND_OPERATION_STATUS 0x2 > > > + #define GET_RESPONSE_MSG_LIMIT 0x3 > > > + #define SET_RESPONSE_MSG_LIMIT 0x4 > > > + #define BACKGROUND_OPERATION_ABORT 0x5 > > > + EVENTS = 0x01, > > > + #define GET_RECORDS 0x0 > > > + #define CLEAR_RECORDS 0x1 > > > + #define GET_INTERRUPT_POLICY 0x2 > > > + #define SET_INTERRUPT_POLICY 0x3 > > > + FIRMWARE_UPDATE = 0x02, > > > + #define GET_INFO 0x0 > > > + #define TRANSFER 0x1 > > > + #define ACTIVATE 0x2 > > > + TIMESTAMP = 0x03, > > > + #define GET 0x0 > > > + #define SET 0x1 > > > + LOGS = 0x04, > > > + #define GET_SUPPORTED 0x0 > > > + #define GET_LOG 0x1 > > > + #define GET_LOG_CAPABILITIES 0x2 > > > + #define CLEAR_LOG 0x3 > > > + #define POPULATE_LOG 0x4 > > > + FEATURES = 0x05, > > > + #define GET_SUPPORTED 0x0 > > > + #define GET_FEATURE 0x1 > > > + #define SET_FEATURE 0x2 > > > + IDENTIFY = 0x40, > > > + #define MEMORY_DEVICE 0x0 > > > + CCLS = 0x41, > > > + #define GET_PARTITION_INFO 0x0 > > > + #define GET_LSA 0x2 > > > + #define SET_LSA 0x3 > > > + HEALTH_INFO_ALERTS = 0x42, > > > + #define GET_ALERT_CONFIG 0x1 > > > + #define SET_ALERT_CONFIG 0x2 > > > + SANITIZE = 0x44, > > > + #define OVERWRITE 0x0 > > > + #define SECURE_ERASE 0x1 > > > + #define MEDIA_OPERATIONS 0x2 > > > + PERSISTENT_MEM = 0x45, > > > + #define GET_SECURITY_STATE 0x0 > > > + MEDIA_AND_POISON = 0x43, > > > + #define GET_POISON_LIST 0x0 > > > + #define INJECT_POISON 0x1 > > > + #define CLEAR_POISON 0x2 > > > + #define GET_SCAN_MEDIA_CAPABILITIES 0x3 > > > + #define SCAN_MEDIA 0x4 > > > + #define GET_SCAN_MEDIA_RESULTS 0x5 > > > + DCD_CONFIG = 0x48, > > > + #define GET_DC_CONFIG 0x0 > > > + #define GET_DYN_CAP_EXT_LIST 0x1 > > > + #define ADD_DYN_CAP_RSP 0x2 > > > + #define RELEASE_DYN_CAP 0x3 > > > + PHYSICAL_SWITCH = 0x51, > > > + #define IDENTIFY_SWITCH_DEVICE 0x0 > > > + #define GET_PHYSICAL_PORT_STATE 0x1 > > > + TUNNEL = 0x53, > > > + #define MANAGEMENT_COMMAND 0x0 > > > + MHD = 0x55, > > > + #define GET_MHD_INFO 0x0 > > > +}; > > > -- > > > 2.47.2 > > > > > > > -- > > Fan Ni