Thread (8 messages) 8 messages, 2 authors, 2021-11-18

Re: [PATCH 2/5] nvme-cli: Adds event number in persistent event entries

From: Chaitanya Kulkarni <hidden>
Date: 2021-11-18 01:53:10

quoted hunk ↗ jump to hunk
@@ -1552,12 +1553,12 @@ void nvme_show_persistent_event_log(void *pevent_log_info,
  		switch (pevent_entry_head->etype) {
  		case NVME_PEL_SMART_HEALTH_EVENT:
  			smart_event = pevent_log_info + offset;
-			printf("Smart Health Event: \n");
+			printf("Smart Health Event Entry: \n");
original string is fine, why do we need to change it ?
quoted hunk ↗ jump to hunk
  			nvme_show_smart_log(smart_event, NVME_NSID_ALL, devname, flags);
  			break;
  		case NVME_PEL_FW_COMMIT_EVENT:
  			fw_commit_event = pevent_log_info + offset;
-			printf("FW Commit Event: \n");
+			printf("FW Commit Event Entry: \n");
  			printf("Old Firmware Revision: %"PRIu64"\n",
  				le64_to_cpu(fw_commit_event->old_fw_rev));
  			printf("New Firmware Revision: %"PRIu64"\n",
@@ -1574,7 +1575,7 @@ void nvme_show_persistent_event_log(void *pevent_log_info,
  			break;
  		case NVME_PEL_TIMESTAMP_EVENT:
  			ts_change_event = pevent_log_info + offset;
-			printf("Time Stamp Change Event: \n");
+			printf("Time Stamp Change Event Entry: \n");
same comment as above
quoted hunk ↗ jump to hunk
  			printf("Previous Timestamp: %"PRIu64"\n",
  				le64_to_cpu(ts_change_event->previous_timestamp));
  			printf("Milliseconds Since Reset: %"PRIu64"\n",
@@ -1586,7 +1587,7 @@ void nvme_show_persistent_event_log(void *pevent_log_info,
  
  			por_info_list = por_info_len / sizeof(*por_event);
  
-			printf("Power On Reset Event: \n");
+			printf("Power On Reset Event Entry: \n");
same comment as above
quoted hunk ↗ jump to hunk
  			fw_rev = pevent_log_info + offset;
  			printf("Firmware Revision: %"PRIu64"\n", le64_to_cpu(*fw_rev));
  			printf("Reset Information List: \n");
@@ -1609,13 +1610,13 @@ void nvme_show_persistent_event_log(void *pevent_log_info,
  			break;
  		case NVME_PEL_NSS_HW_ERROR_EVENT:
  			nss_hw_err_event = pevent_log_info + offset;
-			printf("NVM Subsystem Hardware Error Event Code: %u, %s\n",
+			printf("NVM Subsystem Hardware Error Event Code Entry: %u, %s\n",
same comment as above
  				le16_to_cpu(nss_hw_err_event->nss_hw_err_event_code),
  				nvme_show_nss_hw_error(nss_hw_err_event->nss_hw_err_event_code));
  			break;
  		case NVME_PEL_CHANGE_NS_EVENT:
  			ns_event = pevent_log_info + offset;
-			printf("Change Namespace Event: \n");
+			printf("Change Namespace Event Entry: \n");
same comment as above
quoted hunk ↗ jump to hunk
  			printf("Namespace Management CDW10: %u\n",
  				le32_to_cpu(ns_event->nsmgt_cdw10));
  			printf("Namespace Size: %"PRIu64"\n",
@@ -1634,7 +1635,7 @@ void nvme_show_persistent_event_log(void *pevent_log_info,
  			break;
  		case NVME_PEL_FORMAT_START_EVENT:
  			format_start_event = pevent_log_info + offset;
-			printf("Format NVM Start Event: \n");
+			printf("Format NVM Start Event Entry: \n");
same comment as above
quoted hunk ↗ jump to hunk
  			printf("Namespace Identifier: %u\n",
  				le32_to_cpu(format_start_event->nsid));
  			printf("Format NVM Attributes: %u\n",
@@ -1644,7 +1645,7 @@ void nvme_show_persistent_event_log(void *pevent_log_info,
  			break;
  		case NVME_PEL_FORMAT_COMPLETION_EVENT:
  			format_cmpln_event = pevent_log_info + offset;
-			printf("Format NVM Completion Event: \n");
+			printf("Format NVM Completion Event Entry: \n");
  			printf("Namespace Identifier: %u\n",
  				le32_to_cpu(format_cmpln_event->nsid));
  			printf("Smallest Format Progress Indicator: %u\n",
@@ -1658,7 +1659,7 @@ void nvme_show_persistent_event_log(void *pevent_log_info,
  			break;
  		case NVME_PEL_SANITIZE_START_EVENT:
  			sanitize_start_event = pevent_log_info + offset;
-			printf("Sanitize Start Event: \n");
+			printf("Sanitize Start Event Entry: \n");
  			printf("SANICAP: %u\n", sanitize_start_event->sani_cap);
  			printf("Sanitize CDW10: %u\n",
  				le32_to_cpu(sanitize_start_event->sani_cdw10));
@@ -1667,7 +1668,7 @@ void nvme_show_persistent_event_log(void *pevent_log_info,
  			break;
  		case NVME_PEL_SANITIZE_COMPLETION_EVENT:
  			sanitize_cmpln_event = pevent_log_info + offset;
-			printf("Sanitize Completion Event: \n");
+			printf("Sanitize Completion Event Entry: \n");
  			printf("Sanitize Progress: %u\n",
  				le16_to_cpu(sanitize_cmpln_event->sani_prog));
  			printf("Sanitize Status: %u\n",
@@ -1677,7 +1678,7 @@ void nvme_show_persistent_event_log(void *pevent_log_info,
  			break;
  		case NVME_PEL_THERMAL_EXCURSION_EVENT:
  			thermal_exc_event = pevent_log_info + offset;
-			printf("Thermal Excursion Event: \n");
+			printf("Thermal Excursion Event Entry: \n");
  			printf("Over Temperature: %u\n", thermal_exc_event->over_temp);
  			printf("Threshold: %u\n", thermal_exc_event->threshold);
  			break;
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help