* [PATCH v3] hw/cxl/cxl-mailbox-utils: CXL CCI Get/Set alert config commands implmented as per CXL Specification 3.2 section 8.2.10.9.3 [not found] <CGME20250218052813epcas5p2834e4e039b4aa130af45b90459e14881@epcas5p2.samsung.com> @ 2025-02-18 5:28 ` Sweta Kumari 2025-02-20 15:28 ` Jonathan Cameron via 0 siblings, 1 reply; 3+ messages in thread From: Sweta Kumari @ 2025-02-18 5:28 UTC (permalink / raw) To: qemu-devel Cc: gost.dev, linux-cxl, nifan.cxl, dave, vishak.g, krish.reddy, a.manzanares, alok.rathore, Sweta Kumari 1)get alert configuration(Opcode 4201h) 2)set alert configuration(Opcode 4202h) The patch is generated against the Johnathan's tree https://gitlab.com/jic23/qemu.git and branch cxl-2024-11-27. Signed-off-by: Sweta Kumari <s5.kumari@samsung.com> --- Changes in V2: - Removed cover letter as it's a single patch - Added latest spec reference - Fixed alignment issues - Updated shorter variable names to be more descriptive - Replaced field-by-field initialization in 'init_alert_config' with structured initialization for improved readability. - Replaced bit fields with 'uint8_t' and added defines for individual bits. Changes in v3: - Addressed the review comments from Jonathan Cameron. - Shortened macro and field names for improved readability. hw/cxl/cxl-mailbox-utils.c | 105 ++++++++++++++++++++++++++++++++++++ hw/mem/cxl_type3.c | 16 ++++++ include/hw/cxl/cxl_device.h | 15 ++++++ 3 files changed, 136 insertions(+) diff --git a/hw/cxl/cxl-mailbox-utils.c b/hw/cxl/cxl-mailbox-utils.c index 9c7ea5bc35..5187f1605e 100644 --- a/hw/cxl/cxl-mailbox-utils.c +++ b/hw/cxl/cxl-mailbox-utils.c @@ -28,6 +28,11 @@ #define CXL_DC_EVENT_LOG_SIZE 8 #define CXL_NUM_EXTENTS_SUPPORTED 512 #define CXL_NUM_TAGS_SUPPORTED 0 +#define CXL_ALERTS_LIFE_USED_WARN_THRESH (1 << 0) +#define CXL_ALERTS_OVER_TEMP_WARN_THRESH (1 << 1) +#define CXL_ALERTS_UNDER_TEMP_WARN_THRESH (1 << 2) +#define CXL_ALERTS_COR_VMEM_ERR_WARN_THRESH (1 << 3) +#define CXL_ALERTS_COR_PMEM_ERR_WARN_THRESH (1 << 4) /* * How to add a new command, example. The command set FOO, with cmd BAR. @@ -86,6 +91,9 @@ enum { #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 @@ -1625,6 +1633,97 @@ static CXLRetCode cmd_ccls_set_lsa(const struct cxl_cmd *cmd, return CXL_MBOX_SUCCESS; } +/* CXL r3.2 Section 8.2.10.9.3.2 Get Alert Configuration (Opcode 4201h) */ +static CXLRetCode cmd_get_alert_config(const struct cxl_cmd *cmd, + uint8_t *payload_in, + size_t len_in, + uint8_t *payload_out, + size_t *len_out, + CXLCCI *cci) +{ + CXLType3Dev *ct3d = CXL_TYPE3(cci->d); + CXLAlertConfig *out = (CXLAlertConfig *)payload_out; + + memcpy(out, &ct3d->alert_config, sizeof(ct3d->alert_config)); + *len_out = sizeof(ct3d->alert_config); + + return CXL_MBOX_SUCCESS; +} + +/* CXL r3.2 Section 8.2.10.9.3.3 Set Alert Configuration (Opcode 4202h) */ +static CXLRetCode cmd_set_alert_config(const struct cxl_cmd *cmd, + uint8_t *payload_in, + size_t len_in, + uint8_t *payload_out, + size_t *len_out, + CXLCCI *cci) +{ + CXLType3Dev *ct3d = CXL_TYPE3(cci->d); + CXLAlertConfig *alert_config = &ct3d->alert_config; + struct { + uint8_t valid_alert_actions; + uint8_t enable_alert_actions; + uint8_t life_used_warn_thresh; + uint8_t rsvd; + uint16_t over_temp_warn_thresh; + uint16_t under_temp_warn_thresh; + uint16_t cor_vmem_err_warn_thresh; + uint16_t cor_pmem_err_warn_thresh; + } QEMU_PACKED *in = (void *)payload_in; + + if (in->valid_alert_actions & CXL_ALERTS_LIFE_USED_WARN_THRESH) { + /* + * CXL 3.2 Table 8-149 The life used warning threshold shall be + * less than the life used critical alert value. + */ + if (in->life_used_warn_thresh >= + alert_config->life_used_crit_alert_thresh) { + return CXL_MBOX_INVALID_INPUT; + } + alert_config->life_used_warn_thresh = in->life_used_warn_thresh; + alert_config->enable_alerts |= CXL_ALERTS_LIFE_USED_WARN_THRESH; + } + + if (in->valid_alert_actions & CXL_ALERTS_OVER_TEMP_WARN_THRESH) { + /* + * CXL 3.2 Table 8-149 The Device Over-Temperature Warning Threshold + * shall be less than the the Device Over-Temperature Critical + * Alert Threshold. + */ + if (in->over_temp_warn_thresh >= + alert_config->over_temp_crit_alert_thresh) { + return CXL_MBOX_INVALID_INPUT; + } + alert_config->over_temp_warn_thresh = in->over_temp_warn_thresh; + alert_config->enable_alerts |= CXL_ALERTS_OVER_TEMP_WARN_THRESH; + } + + if (in->valid_alert_actions & CXL_ALERTS_UNDER_TEMP_WARN_THRESH) { + /* + * CXL 3.2 Table 8-149 The Device Under-Temperature Warning Threshold + * shall be higher than the the Device Under-Temperature Critical + * Alert Threshold. + */ + if (in->under_temp_warn_thresh <= + alert_config->under_temp_crit_alert_thresh) { + return CXL_MBOX_INVALID_INPUT; + } + alert_config->under_temp_warn_thresh = in->under_temp_warn_thresh; + alert_config->enable_alerts |= CXL_ALERTS_UNDER_TEMP_WARN_THRESH; + } + + if (in->valid_alert_actions & CXL_ALERTS_COR_VMEM_ERR_WARN_THRESH) { + alert_config->cor_vmem_err_warn_thresh = in->cor_vmem_err_warn_thresh; + alert_config->enable_alerts |= CXL_ALERTS_COR_VMEM_ERR_WARN_THRESH; + } + + if (in->valid_alert_actions & CXL_ALERTS_COR_PMEM_ERR_WARN_THRESH) { + alert_config->cor_pmem_err_warn_thresh = in->cor_pmem_err_warn_thresh; + alert_config->enable_alerts |= CXL_ALERTS_COR_PMEM_ERR_WARN_THRESH; + } + return CXL_MBOX_SUCCESS; +} + /* Perform the actual device zeroing */ static void __do_sanitization(CXLType3Dev *ct3d) { @@ -2859,6 +2958,12 @@ static const struct cxl_cmd cxl_cmd_set[256][256] = { [CCLS][GET_LSA] = { "CCLS_GET_LSA", cmd_ccls_get_lsa, 8, 0 }, [CCLS][SET_LSA] = { "CCLS_SET_LSA", cmd_ccls_set_lsa, ~0, CXL_MBOX_IMMEDIATE_CONFIG_CHANGE | CXL_MBOX_IMMEDIATE_DATA_CHANGE }, + [HEALTH_INFO_ALERTS][GET_ALERT_CONFIG] = { + "HEALTH_INFO_ALERTS_GET_ALERT_CONFIG", + cmd_get_alert_config, 0, 0 }, + [HEALTH_INFO_ALERTS][SET_ALERT_CONFIG] = { + "HEALTH_INFO_ALERTS_SET_ALERT_CONFIG", + cmd_set_alert_config, 12, CXL_MBOX_IMMEDIATE_POLICY_CHANGE }, [SANITIZE][OVERWRITE] = { "SANITIZE_OVERWRITE", cmd_sanitize_overwrite, 0, (CXL_MBOX_IMMEDIATE_DATA_CHANGE | CXL_MBOX_SECURITY_STATE_CHANGE | diff --git a/hw/mem/cxl_type3.c b/hw/mem/cxl_type3.c index 5f365afb4d..ecbaa60210 100644 --- a/hw/mem/cxl_type3.c +++ b/hw/mem/cxl_type3.c @@ -956,6 +956,21 @@ static DOEProtocol doe_comp_prot[] = { { } }; +/* + * Initialize CXL device alerts with default threshold values. + */ +static void init_alert_config(CXLType3Dev *ct3d) +{ + ct3d->alert_config = (CXLAlertConfig) { + .life_used_crit_alert_thresh = 75, + .life_used_warn_thresh = 40, + .over_temp_crit_alert_thresh = 35, + .under_temp_crit_alert_thresh = 10, + .over_temp_warn_thresh = 25, + .under_temp_warn_thresh = 20 + }; +} + void ct3_realize(PCIDevice *pci_dev, Error **errp) { ERRP_GUARD(); @@ -1030,6 +1045,7 @@ void ct3_realize(PCIDevice *pci_dev, Error **errp) goto err_free_special_ops; } + init_alert_config(ct3d); pcie_cap_deverr_init(pci_dev); /* Leave a bit of room for expansion */ rc = pcie_aer_init(pci_dev, PCI_ERR_VER, 0x200, PCI_ERR_SIZEOF, NULL); diff --git a/include/hw/cxl/cxl_device.h b/include/hw/cxl/cxl_device.h index a64739be25..7f96bd8637 100644 --- a/include/hw/cxl/cxl_device.h +++ b/include/hw/cxl/cxl_device.h @@ -581,6 +581,19 @@ typedef struct CXLSetFeatureInfo { size_t data_size; } CXLSetFeatureInfo; +typedef struct CXLAlertConfig { + uint8_t valid_alerts; + uint8_t enable_alerts; + uint8_t life_used_crit_alert_thresh; + uint8_t life_used_warn_thresh; + uint16_t over_temp_crit_alert_thresh; + uint16_t under_temp_crit_alert_thresh; + uint16_t over_temp_warn_thresh; + uint16_t under_temp_warn_thresh; + uint16_t cor_vmem_err_warn_thresh; + uint16_t cor_pmem_err_warn_thresh; +} QEMU_PACKED CXLAlertConfig; + struct CXLType3Dev { /* Private */ PCIDevice parent_obj; @@ -605,6 +618,8 @@ struct CXLType3Dev { CXLCCI vdm_fm_owned_ld_mctp_cci; CXLCCI ld0_cci; + CXLAlertConfig alert_config; + /* PCIe link characteristics */ PCIExpLinkSpeed speed; PCIExpLinkWidth width; -- 2.34.1 ^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH v3] hw/cxl/cxl-mailbox-utils: CXL CCI Get/Set alert config commands implmented as per CXL Specification 3.2 section 8.2.10.9.3 2025-02-18 5:28 ` [PATCH v3] hw/cxl/cxl-mailbox-utils: CXL CCI Get/Set alert config commands implmented as per CXL Specification 3.2 section 8.2.10.9.3 Sweta Kumari @ 2025-02-20 15:28 ` Jonathan Cameron via 0 siblings, 0 replies; 3+ messages in thread From: Jonathan Cameron @ 2025-02-20 15:28 UTC (permalink / raw) To: Sweta Kumari Cc: qemu-devel, gost.dev, linux-cxl, nifan.cxl, dave, vishak.g, krish.reddy, a.manzanares, alok.rathore On Tue, 18 Feb 2025 10:58:00 +0530 Sweta Kumari <s5.kumari@samsung.com> wrote: Title is a bit long so I cut it after commands. > 1)get alert configuration(Opcode 4201h) > 2)set alert configuration(Opcode 4202h) > > The patch is generated against the Johnathan's tree > https://gitlab.com/jic23/qemu.git and branch cxl-2024-11-27. Useful info for now, but not in the final commit, so should be below the --- > > Signed-off-by: Sweta Kumari <s5.kumari@samsung.com> I made a few tweaks as mentioned inline and rebased it to be much earlier in my staging tree. I'm applying a few other series today but will be looking to roll up all the simple new mailbox commands into a set to post to Michael Tsirkin to hopefully get applied this cycle. I'll hopefully push out a new tree to gitlab later today. Jonathan > --- > Changes in V2: > - Removed cover letter as it's a single patch > - Added latest spec reference > - Fixed alignment issues > - Updated shorter variable names to be more descriptive > - Replaced field-by-field initialization in 'init_alert_config' with structured initialization for improved readability. > - Replaced bit fields with 'uint8_t' and added defines for individual bits. > > Changes in v3: > - Addressed the review comments from Jonathan Cameron. > - Shortened macro and field names for improved readability. > > hw/cxl/cxl-mailbox-utils.c | 105 ++++++++++++++++++++++++++++++++++++ > hw/mem/cxl_type3.c | 16 ++++++ > include/hw/cxl/cxl_device.h | 15 ++++++ > 3 files changed, 136 insertions(+) > > diff --git a/hw/cxl/cxl-mailbox-utils.c b/hw/cxl/cxl-mailbox-utils.c > index 9c7ea5bc35..5187f1605e 100644 > --- a/hw/cxl/cxl-mailbox-utils.c > +++ b/hw/cxl/cxl-mailbox-utils.c > @@ -28,6 +28,11 @@ > #define CXL_DC_EVENT_LOG_SIZE 8 > #define CXL_NUM_EXTENTS_SUPPORTED 512 > #define CXL_NUM_TAGS_SUPPORTED 0 > +#define CXL_ALERTS_LIFE_USED_WARN_THRESH (1 << 0) > +#define CXL_ALERTS_OVER_TEMP_WARN_THRESH (1 << 1) > +#define CXL_ALERTS_UNDER_TEMP_WARN_THRESH (1 << 2) > +#define CXL_ALERTS_COR_VMEM_ERR_WARN_THRESH (1 << 3) > +#define CXL_ALERTS_COR_PMEM_ERR_WARN_THRESH (1 << 4) > > /* > * How to add a new command, example. The command set FOO, with cmd BAR. > @@ -86,6 +91,9 @@ enum { > #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 > @@ -1625,6 +1633,97 @@ static CXLRetCode cmd_ccls_set_lsa(const struct cxl_cmd *cmd, > return CXL_MBOX_SUCCESS; > } > > +/* CXL r3.2 Section 8.2.10.9.3.2 Get Alert Configuration (Opcode 4201h) */ > +static CXLRetCode cmd_get_alert_config(const struct cxl_cmd *cmd, > + uint8_t *payload_in, > + size_t len_in, > + uint8_t *payload_out, > + size_t *len_out, > + CXLCCI *cci) > +{ > + CXLType3Dev *ct3d = CXL_TYPE3(cci->d); > + CXLAlertConfig *out = (CXLAlertConfig *)payload_out; > + > + memcpy(out, &ct3d->alert_config, sizeof(ct3d->alert_config)); > + *len_out = sizeof(ct3d->alert_config); > + > + return CXL_MBOX_SUCCESS; > +} > + > +/* CXL r3.2 Section 8.2.10.9.3.3 Set Alert Configuration (Opcode 4202h) */ > +static CXLRetCode cmd_set_alert_config(const struct cxl_cmd *cmd, > + uint8_t *payload_in, > + size_t len_in, > + uint8_t *payload_out, > + size_t *len_out, > + CXLCCI *cci) > +{ > + CXLType3Dev *ct3d = CXL_TYPE3(cci->d); > + CXLAlertConfig *alert_config = &ct3d->alert_config; > + struct { > + uint8_t valid_alert_actions; > + uint8_t enable_alert_actions; > + uint8_t life_used_warn_thresh; > + uint8_t rsvd; > + uint16_t over_temp_warn_thresh; > + uint16_t under_temp_warn_thresh; > + uint16_t cor_vmem_err_warn_thresh; > + uint16_t cor_pmem_err_warn_thresh; > + } QEMU_PACKED *in = (void *)payload_in; > + > + if (in->valid_alert_actions & CXL_ALERTS_LIFE_USED_WARN_THRESH) { > + /* > + * CXL 3.2 Table 8-149 The life used warning threshold shall be I'll make that r3.2 and other cases the same. I've been in too many silly discussions on revision vs version to leave it out :( > + * less than the life used critical alert value. > + */ > + if (in->life_used_warn_thresh >= > + alert_config->life_used_crit_alert_thresh) { > + return CXL_MBOX_INVALID_INPUT; > + } > + alert_config->life_used_warn_thresh = in->life_used_warn_thresh; > + alert_config->enable_alerts |= CXL_ALERTS_LIFE_USED_WARN_THRESH; > + } > + > + if (in->valid_alert_actions & CXL_ALERTS_OVER_TEMP_WARN_THRESH) { > + /* > + * CXL 3.2 Table 8-149 The Device Over-Temperature Warning Threshold > + * shall be less than the the Device Over-Temperature Critical > + * Alert Threshold. > + */ > + if (in->over_temp_warn_thresh >= > + alert_config->over_temp_crit_alert_thresh) { > + return CXL_MBOX_INVALID_INPUT; > + } > + alert_config->over_temp_warn_thresh = in->over_temp_warn_thresh; > + alert_config->enable_alerts |= CXL_ALERTS_OVER_TEMP_WARN_THRESH; > + } > + > + if (in->valid_alert_actions & CXL_ALERTS_UNDER_TEMP_WARN_THRESH) { > + /* > + * CXL 3.2 Table 8-149 The Device Under-Temperature Warning Threshold > + * shall be higher than the the Device Under-Temperature Critical > + * Alert Threshold. > + */ > + if (in->under_temp_warn_thresh <= > + alert_config->under_temp_crit_alert_thresh) { > + return CXL_MBOX_INVALID_INPUT; > + } > + alert_config->under_temp_warn_thresh = in->under_temp_warn_thresh; > + alert_config->enable_alerts |= CXL_ALERTS_UNDER_TEMP_WARN_THRESH; > + } > + > + if (in->valid_alert_actions & CXL_ALERTS_COR_VMEM_ERR_WARN_THRESH) { > + alert_config->cor_vmem_err_warn_thresh = in->cor_vmem_err_warn_thresh; > + alert_config->enable_alerts |= CXL_ALERTS_COR_VMEM_ERR_WARN_THRESH; > + } > + > + if (in->valid_alert_actions & CXL_ALERTS_COR_PMEM_ERR_WARN_THRESH) { > + alert_config->cor_pmem_err_warn_thresh = in->cor_pmem_err_warn_thresh; > + alert_config->enable_alerts |= CXL_ALERTS_COR_PMEM_ERR_WARN_THRESH; > + } > + return CXL_MBOX_SUCCESS; > +} > + > /* Perform the actual device zeroing */ > static void __do_sanitization(CXLType3Dev *ct3d) > { > @@ -2859,6 +2958,12 @@ static const struct cxl_cmd cxl_cmd_set[256][256] = { > [CCLS][GET_LSA] = { "CCLS_GET_LSA", cmd_ccls_get_lsa, 8, 0 }, > [CCLS][SET_LSA] = { "CCLS_SET_LSA", cmd_ccls_set_lsa, > ~0, CXL_MBOX_IMMEDIATE_CONFIG_CHANGE | CXL_MBOX_IMMEDIATE_DATA_CHANGE }, > + [HEALTH_INFO_ALERTS][GET_ALERT_CONFIG] = { > + "HEALTH_INFO_ALERTS_GET_ALERT_CONFIG", > + cmd_get_alert_config, 0, 0 }, > + [HEALTH_INFO_ALERTS][SET_ALERT_CONFIG] = { > + "HEALTH_INFO_ALERTS_SET_ALERT_CONFIG", > + cmd_set_alert_config, 12, CXL_MBOX_IMMEDIATE_POLICY_CHANGE }, > [SANITIZE][OVERWRITE] = { "SANITIZE_OVERWRITE", cmd_sanitize_overwrite, 0, > (CXL_MBOX_IMMEDIATE_DATA_CHANGE | > CXL_MBOX_SECURITY_STATE_CHANGE | > diff --git a/hw/mem/cxl_type3.c b/hw/mem/cxl_type3.c > index 5f365afb4d..ecbaa60210 100644 > --- a/hw/mem/cxl_type3.c > +++ b/hw/mem/cxl_type3.c > @@ -956,6 +956,21 @@ static DOEProtocol doe_comp_prot[] = { > { } > }; > > +/* > + * Initialize CXL device alerts with default threshold values. Fits in a single line comment. > + */ > +static void init_alert_config(CXLType3Dev *ct3d) > +{ > + ct3d->alert_config = (CXLAlertConfig) { > + .life_used_crit_alert_thresh = 75, > + .life_used_warn_thresh = 40, > + .over_temp_crit_alert_thresh = 35, > + .under_temp_crit_alert_thresh = 10, > + .over_temp_warn_thresh = 25, > + .under_temp_warn_thresh = 20 > + }; > +} > + > void ct3_realize(PCIDevice *pci_dev, Error **errp) > { > ERRP_GUARD(); > @@ -1030,6 +1045,7 @@ void ct3_realize(PCIDevice *pci_dev, Error **errp) > goto err_free_special_ops; > } > > + init_alert_config(ct3d); > pcie_cap_deverr_init(pci_dev); > /* Leave a bit of room for expansion */ > rc = pcie_aer_init(pci_dev, PCI_ERR_VER, 0x200, PCI_ERR_SIZEOF, NULL); > diff --git a/include/hw/cxl/cxl_device.h b/include/hw/cxl/cxl_device.h > index a64739be25..7f96bd8637 100644 > --- a/include/hw/cxl/cxl_device.h > +++ b/include/hw/cxl/cxl_device.h > @@ -581,6 +581,19 @@ typedef struct CXLSetFeatureInfo { > size_t data_size; > } CXLSetFeatureInfo; > > +typedef struct CXLAlertConfig { > + uint8_t valid_alerts; > + uint8_t enable_alerts; > + uint8_t life_used_crit_alert_thresh; > + uint8_t life_used_warn_thresh; > + uint16_t over_temp_crit_alert_thresh; > + uint16_t under_temp_crit_alert_thresh; > + uint16_t over_temp_warn_thresh; > + uint16_t under_temp_warn_thresh; > + uint16_t cor_vmem_err_warn_thresh; > + uint16_t cor_pmem_err_warn_thresh; > +} QEMU_PACKED CXLAlertConfig; > + > struct CXLType3Dev { > /* Private */ > PCIDevice parent_obj; > @@ -605,6 +618,8 @@ struct CXLType3Dev { > CXLCCI vdm_fm_owned_ld_mctp_cci; > CXLCCI ld0_cci; > > + CXLAlertConfig alert_config; > + > /* PCIe link characteristics */ > PCIExpLinkSpeed speed; > PCIExpLinkWidth width; ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v3] hw/cxl/cxl-mailbox-utils: CXL CCI Get/Set alert config commands implmented as per CXL Specification 3.2 section 8.2.10.9.3 @ 2025-02-20 15:28 ` Jonathan Cameron via 0 siblings, 0 replies; 3+ messages in thread From: Jonathan Cameron via @ 2025-02-20 15:28 UTC (permalink / raw) To: Sweta Kumari Cc: qemu-devel, gost.dev, linux-cxl, nifan.cxl, dave, vishak.g, krish.reddy, a.manzanares, alok.rathore On Tue, 18 Feb 2025 10:58:00 +0530 Sweta Kumari <s5.kumari@samsung.com> wrote: Title is a bit long so I cut it after commands. > 1)get alert configuration(Opcode 4201h) > 2)set alert configuration(Opcode 4202h) > > The patch is generated against the Johnathan's tree > https://gitlab.com/jic23/qemu.git and branch cxl-2024-11-27. Useful info for now, but not in the final commit, so should be below the --- > > Signed-off-by: Sweta Kumari <s5.kumari@samsung.com> I made a few tweaks as mentioned inline and rebased it to be much earlier in my staging tree. I'm applying a few other series today but will be looking to roll up all the simple new mailbox commands into a set to post to Michael Tsirkin to hopefully get applied this cycle. I'll hopefully push out a new tree to gitlab later today. Jonathan > --- > Changes in V2: > - Removed cover letter as it's a single patch > - Added latest spec reference > - Fixed alignment issues > - Updated shorter variable names to be more descriptive > - Replaced field-by-field initialization in 'init_alert_config' with structured initialization for improved readability. > - Replaced bit fields with 'uint8_t' and added defines for individual bits. > > Changes in v3: > - Addressed the review comments from Jonathan Cameron. > - Shortened macro and field names for improved readability. > > hw/cxl/cxl-mailbox-utils.c | 105 ++++++++++++++++++++++++++++++++++++ > hw/mem/cxl_type3.c | 16 ++++++ > include/hw/cxl/cxl_device.h | 15 ++++++ > 3 files changed, 136 insertions(+) > > diff --git a/hw/cxl/cxl-mailbox-utils.c b/hw/cxl/cxl-mailbox-utils.c > index 9c7ea5bc35..5187f1605e 100644 > --- a/hw/cxl/cxl-mailbox-utils.c > +++ b/hw/cxl/cxl-mailbox-utils.c > @@ -28,6 +28,11 @@ > #define CXL_DC_EVENT_LOG_SIZE 8 > #define CXL_NUM_EXTENTS_SUPPORTED 512 > #define CXL_NUM_TAGS_SUPPORTED 0 > +#define CXL_ALERTS_LIFE_USED_WARN_THRESH (1 << 0) > +#define CXL_ALERTS_OVER_TEMP_WARN_THRESH (1 << 1) > +#define CXL_ALERTS_UNDER_TEMP_WARN_THRESH (1 << 2) > +#define CXL_ALERTS_COR_VMEM_ERR_WARN_THRESH (1 << 3) > +#define CXL_ALERTS_COR_PMEM_ERR_WARN_THRESH (1 << 4) > > /* > * How to add a new command, example. The command set FOO, with cmd BAR. > @@ -86,6 +91,9 @@ enum { > #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 > @@ -1625,6 +1633,97 @@ static CXLRetCode cmd_ccls_set_lsa(const struct cxl_cmd *cmd, > return CXL_MBOX_SUCCESS; > } > > +/* CXL r3.2 Section 8.2.10.9.3.2 Get Alert Configuration (Opcode 4201h) */ > +static CXLRetCode cmd_get_alert_config(const struct cxl_cmd *cmd, > + uint8_t *payload_in, > + size_t len_in, > + uint8_t *payload_out, > + size_t *len_out, > + CXLCCI *cci) > +{ > + CXLType3Dev *ct3d = CXL_TYPE3(cci->d); > + CXLAlertConfig *out = (CXLAlertConfig *)payload_out; > + > + memcpy(out, &ct3d->alert_config, sizeof(ct3d->alert_config)); > + *len_out = sizeof(ct3d->alert_config); > + > + return CXL_MBOX_SUCCESS; > +} > + > +/* CXL r3.2 Section 8.2.10.9.3.3 Set Alert Configuration (Opcode 4202h) */ > +static CXLRetCode cmd_set_alert_config(const struct cxl_cmd *cmd, > + uint8_t *payload_in, > + size_t len_in, > + uint8_t *payload_out, > + size_t *len_out, > + CXLCCI *cci) > +{ > + CXLType3Dev *ct3d = CXL_TYPE3(cci->d); > + CXLAlertConfig *alert_config = &ct3d->alert_config; > + struct { > + uint8_t valid_alert_actions; > + uint8_t enable_alert_actions; > + uint8_t life_used_warn_thresh; > + uint8_t rsvd; > + uint16_t over_temp_warn_thresh; > + uint16_t under_temp_warn_thresh; > + uint16_t cor_vmem_err_warn_thresh; > + uint16_t cor_pmem_err_warn_thresh; > + } QEMU_PACKED *in = (void *)payload_in; > + > + if (in->valid_alert_actions & CXL_ALERTS_LIFE_USED_WARN_THRESH) { > + /* > + * CXL 3.2 Table 8-149 The life used warning threshold shall be I'll make that r3.2 and other cases the same. I've been in too many silly discussions on revision vs version to leave it out :( > + * less than the life used critical alert value. > + */ > + if (in->life_used_warn_thresh >= > + alert_config->life_used_crit_alert_thresh) { > + return CXL_MBOX_INVALID_INPUT; > + } > + alert_config->life_used_warn_thresh = in->life_used_warn_thresh; > + alert_config->enable_alerts |= CXL_ALERTS_LIFE_USED_WARN_THRESH; > + } > + > + if (in->valid_alert_actions & CXL_ALERTS_OVER_TEMP_WARN_THRESH) { > + /* > + * CXL 3.2 Table 8-149 The Device Over-Temperature Warning Threshold > + * shall be less than the the Device Over-Temperature Critical > + * Alert Threshold. > + */ > + if (in->over_temp_warn_thresh >= > + alert_config->over_temp_crit_alert_thresh) { > + return CXL_MBOX_INVALID_INPUT; > + } > + alert_config->over_temp_warn_thresh = in->over_temp_warn_thresh; > + alert_config->enable_alerts |= CXL_ALERTS_OVER_TEMP_WARN_THRESH; > + } > + > + if (in->valid_alert_actions & CXL_ALERTS_UNDER_TEMP_WARN_THRESH) { > + /* > + * CXL 3.2 Table 8-149 The Device Under-Temperature Warning Threshold > + * shall be higher than the the Device Under-Temperature Critical > + * Alert Threshold. > + */ > + if (in->under_temp_warn_thresh <= > + alert_config->under_temp_crit_alert_thresh) { > + return CXL_MBOX_INVALID_INPUT; > + } > + alert_config->under_temp_warn_thresh = in->under_temp_warn_thresh; > + alert_config->enable_alerts |= CXL_ALERTS_UNDER_TEMP_WARN_THRESH; > + } > + > + if (in->valid_alert_actions & CXL_ALERTS_COR_VMEM_ERR_WARN_THRESH) { > + alert_config->cor_vmem_err_warn_thresh = in->cor_vmem_err_warn_thresh; > + alert_config->enable_alerts |= CXL_ALERTS_COR_VMEM_ERR_WARN_THRESH; > + } > + > + if (in->valid_alert_actions & CXL_ALERTS_COR_PMEM_ERR_WARN_THRESH) { > + alert_config->cor_pmem_err_warn_thresh = in->cor_pmem_err_warn_thresh; > + alert_config->enable_alerts |= CXL_ALERTS_COR_PMEM_ERR_WARN_THRESH; > + } > + return CXL_MBOX_SUCCESS; > +} > + > /* Perform the actual device zeroing */ > static void __do_sanitization(CXLType3Dev *ct3d) > { > @@ -2859,6 +2958,12 @@ static const struct cxl_cmd cxl_cmd_set[256][256] = { > [CCLS][GET_LSA] = { "CCLS_GET_LSA", cmd_ccls_get_lsa, 8, 0 }, > [CCLS][SET_LSA] = { "CCLS_SET_LSA", cmd_ccls_set_lsa, > ~0, CXL_MBOX_IMMEDIATE_CONFIG_CHANGE | CXL_MBOX_IMMEDIATE_DATA_CHANGE }, > + [HEALTH_INFO_ALERTS][GET_ALERT_CONFIG] = { > + "HEALTH_INFO_ALERTS_GET_ALERT_CONFIG", > + cmd_get_alert_config, 0, 0 }, > + [HEALTH_INFO_ALERTS][SET_ALERT_CONFIG] = { > + "HEALTH_INFO_ALERTS_SET_ALERT_CONFIG", > + cmd_set_alert_config, 12, CXL_MBOX_IMMEDIATE_POLICY_CHANGE }, > [SANITIZE][OVERWRITE] = { "SANITIZE_OVERWRITE", cmd_sanitize_overwrite, 0, > (CXL_MBOX_IMMEDIATE_DATA_CHANGE | > CXL_MBOX_SECURITY_STATE_CHANGE | > diff --git a/hw/mem/cxl_type3.c b/hw/mem/cxl_type3.c > index 5f365afb4d..ecbaa60210 100644 > --- a/hw/mem/cxl_type3.c > +++ b/hw/mem/cxl_type3.c > @@ -956,6 +956,21 @@ static DOEProtocol doe_comp_prot[] = { > { } > }; > > +/* > + * Initialize CXL device alerts with default threshold values. Fits in a single line comment. > + */ > +static void init_alert_config(CXLType3Dev *ct3d) > +{ > + ct3d->alert_config = (CXLAlertConfig) { > + .life_used_crit_alert_thresh = 75, > + .life_used_warn_thresh = 40, > + .over_temp_crit_alert_thresh = 35, > + .under_temp_crit_alert_thresh = 10, > + .over_temp_warn_thresh = 25, > + .under_temp_warn_thresh = 20 > + }; > +} > + > void ct3_realize(PCIDevice *pci_dev, Error **errp) > { > ERRP_GUARD(); > @@ -1030,6 +1045,7 @@ void ct3_realize(PCIDevice *pci_dev, Error **errp) > goto err_free_special_ops; > } > > + init_alert_config(ct3d); > pcie_cap_deverr_init(pci_dev); > /* Leave a bit of room for expansion */ > rc = pcie_aer_init(pci_dev, PCI_ERR_VER, 0x200, PCI_ERR_SIZEOF, NULL); > diff --git a/include/hw/cxl/cxl_device.h b/include/hw/cxl/cxl_device.h > index a64739be25..7f96bd8637 100644 > --- a/include/hw/cxl/cxl_device.h > +++ b/include/hw/cxl/cxl_device.h > @@ -581,6 +581,19 @@ typedef struct CXLSetFeatureInfo { > size_t data_size; > } CXLSetFeatureInfo; > > +typedef struct CXLAlertConfig { > + uint8_t valid_alerts; > + uint8_t enable_alerts; > + uint8_t life_used_crit_alert_thresh; > + uint8_t life_used_warn_thresh; > + uint16_t over_temp_crit_alert_thresh; > + uint16_t under_temp_crit_alert_thresh; > + uint16_t over_temp_warn_thresh; > + uint16_t under_temp_warn_thresh; > + uint16_t cor_vmem_err_warn_thresh; > + uint16_t cor_pmem_err_warn_thresh; > +} QEMU_PACKED CXLAlertConfig; > + > struct CXLType3Dev { > /* Private */ > PCIDevice parent_obj; > @@ -605,6 +618,8 @@ struct CXLType3Dev { > CXLCCI vdm_fm_owned_ld_mctp_cci; > CXLCCI ld0_cci; > > + CXLAlertConfig alert_config; > + > /* PCIe link characteristics */ > PCIExpLinkSpeed speed; > PCIExpLinkWidth width; ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2025-02-20 15:28 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <CGME20250218052813epcas5p2834e4e039b4aa130af45b90459e14881@epcas5p2.samsung.com>
2025-02-18 5:28 ` [PATCH v3] hw/cxl/cxl-mailbox-utils: CXL CCI Get/Set alert config commands implmented as per CXL Specification 3.2 section 8.2.10.9.3 Sweta Kumari
2025-02-20 15:28 ` Jonathan Cameron
2025-02-20 15:28 ` Jonathan Cameron via
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.