plugins/ocp: Align ocp_smart_extended_log struct.

If the dssd point, minor version are declared as __le16, the alignment
will be broken. Remove __packed keyword.
Change parameter void to ocp_smart_extended_log.

Reported-by: Steven Seungcheol Lee <[email protected]>
Signed-off-by: Minsik Jeon <[email protected]>

Author: hmi-jeon

plugins/ocp/ocp-print-json.c

@@ -232,9 +232,9 @@ static void json_smart_extended_log_v1(struct ocp_smart_extended_log *log)
 		json_object_add_value_uint(root, "Errata Version Field",
 						log->dssd_errata_version);
 		json_object_add_value_uint(root, "Point Version Field",
-						le16_to_cpu(log->dssd_point_version));
+						le16_to_cpu(*(uint16_t *)&log->dssd_point_version));
 		json_object_add_value_uint(root, "Minor Version Field",
-						le16_to_cpu(log->dssd_minor_version));
+						le16_to_cpu(*(uint16_t *)&log->dssd_minor_version));
 		json_object_add_value_uint(root, "Major Version Field",
 						log->dssd_major_version);
 		json_object_add_value_uint(root, "NVMe Base Errata Version",
@@ -343,9 +343,9 @@ static void json_smart_extended_log_v2(struct ocp_smart_extended_log *log)
 		json_object_add_value_uint(root, "errata_version_field",
 						log->dssd_errata_version);
 		json_object_add_value_uint(root, "point_version_field",
-						le16_to_cpu(log->dssd_point_version));
+						le16_to_cpu(*(uint16_t *)&log->dssd_point_version));
 		json_object_add_value_uint(root, "minor_version_field",
-						le16_to_cpu(log->dssd_minor_version));
+						le16_to_cpu(*(uint16_t *)&log->dssd_minor_version));
 		json_object_add_value_uint(root, "major_version_field",
 						log->dssd_major_version);
 		json_object_add_value_uint(root, "nvme_base_errata_version",

plugins/ocp/ocp-print-stdout.c

@@ -176,9 +176,9 @@ static void stdout_smart_extended_log(struct ocp_smart_extended_log *log, unsign
 		printf("  Errata Version Field                          %d\n",
 			log->dssd_errata_version);
 		printf("  Point Version Field                           %"PRIu16"\n",
-			le16_to_cpu(log->dssd_point_version));
+			le16_to_cpu(*(uint16_t *)&log->dssd_point_version));
 		printf("  Minor Version Field                           %"PRIu16"\n",
-			le16_to_cpu(log->dssd_minor_version));
+			le16_to_cpu(*(uint16_t *)&log->dssd_minor_version));
 		printf("  Major Version Field                           %d\n",
 			log->dssd_major_version);
 		printf("  NVMe Base Errata Version                      %d\n",

plugins/ocp/ocp-print.c

@@ -36,9 +36,10 @@ void ocp_fw_act_history(const struct fw_activation_history *fw_history, nvme_pri
 	ocp_print(fw_act_history, flags, fw_history);
 }
 
-void ocp_smart_extended_log(void *data, unsigned int version, nvme_print_flags_t flags)
+void ocp_smart_extended_log(struct ocp_smart_extended_log *log, unsigned int version,
+		nvme_print_flags_t flags)
 {
-	ocp_print(smart_extended_log, flags, data, version);
+	ocp_print(smart_extended_log, flags, log, version);
 }
 
 void ocp_show_telemetry_log(struct ocp_telemetry_parse_options *options, nvme_print_flags_t flags)

plugins/ocp/ocp-print.h

@@ -37,7 +37,8 @@ static inline struct ocp_print_ops *ocp_get_json_print_ops(nvme_print_flags_t fl
 
 void ocp_show_hwcomp_log(struct hwcomp_log *log, __u32 id, bool list, nvme_print_flags_t flags);
 void ocp_fw_act_history(const struct fw_activation_history *fw_history, nvme_print_flags_t flags);
-void ocp_smart_extended_log(void *data, unsigned int version, nvme_print_flags_t flags);
+void ocp_smart_extended_log(struct ocp_smart_extended_log *log, unsigned int version,
+		nvme_print_flags_t flags);
 void ocp_show_telemetry_log(struct ocp_telemetry_parse_options *options, nvme_print_flags_t flags);
 void ocp_c3_log(struct nvme_dev *dev, struct ssd_latency_monitor_log *log_data,
 		nvme_print_flags_t flags);

plugins/ocp/ocp-smart-extended-log.h

@@ -58,7 +58,7 @@ struct plugin;
  * @log_page_version:			Log page version
  * @log_page_guid:			Log page GUID
  */
-struct __packed ocp_smart_extended_log {
+struct ocp_smart_extended_log {
 	__u8   physical_media_units_written[16];	/* [15:0] */
 	__u8   physical_media_units_read[16];		/* [31:16] */
 	__u8   bad_user_nand_blocks_raw[6];		/* [37:32] */
@@ -77,8 +77,8 @@ struct __packed ocp_smart_extended_log {
 	__u8   thermal_throttling_event_count;		/* [96] */
 	__u8   thermal_throttling_current_status;	/* [97] */
 	__u8   dssd_errata_version;			/* [98] */
-	__le16 dssd_point_version;			/* [100:99] */
-	__le16 dssd_minor_version;			/* [102:101] */
+	__u8   dssd_point_version[2];			/* [100:99] */
+	__u8   dssd_minor_version[2];			/* [102:101] */
 	__u8   dssd_major_version;			/* [103] */
 	__le64 pcie_correctable_err_count;		/* [111:104] */
 	__le32 incomplete_shoutdowns;			/* [115:112] */