[PATCH ethtool-next v3 09/16] cmis: Enable JSON output support in CMIS modules
From: Danielle Ratson <hidden>
Date: 2025-02-04 13:41:01
Subsystem:
the rest · Maintainer:
Linus Torvalds
A sample output:
$ ethtool --json -m swp23
[ {
"identifier": 24,
"identifier_description": "QSFP-DD Double Density 8X Pluggable
Transceiver (INF-8628)",
"power_class": 5,
"max_power": 10,
"connector": 40,
"connector_description": "MPO 1x16",
"cable_assembly_length": 0,
"tx_cdr_bypass_control": false,
"rx_cdr_bypass_control": false,
"tx_cdr": true,
"rx_cdr": true,
"transmitter_technology": 0,
"transmitter_technology_description": "850 nm VCSEL",
"laser_wavelength": 850,
"laser_wavelength_tolerance": 94.8,
"length_(smf)": 0,
"length_(om5)": 0,
"length_(om4)": 100,
"length_(om3)": 70,
"length_(om2)": 0,
"vendor_name": "FINISAR CORP.",
"vendor_oui": [ 0,144,101 ],
"vendor_pn": "FTCD8613E1PCM",
"vendor_rev": "A0",
"vendor_sn": "X6LBE6H",
"date_code": "211229__",
"revision_compliance": {
"major": 4,
"minor": 0
},
"rx_loss_of_signal": [
"Yes","Yes","Yes","Yes","Yes","Yes","Yes","Yes" ],
"tx_loss_of_signal": false,
"rx_loss_of_lock": false,
"tx_loss_of_lock": false,
"tx_fault": false,
"module_state": 3,
"module_state_description": "ModuleReady",
"low_pwr_allow_request_hw": false,
"low_pwr_request_sw": false,
"module_temperature": 37.5273,
"module_voltage": 3.3358,
"laser_tx_bias_current": [ 0,0,0,0,0,0,0,0 ],
"transmit_avg_optical_power": [
0.0001,0.0001,0.0001,0.0001,0.0001,0.0001,0.0001,0.0001 ],
"rx_power": {
"values": [
0.0001,0.0001,0.0001,0.0001,0.0001,0.0001,0.0001,0.0001 ],
"type": "Rcvr signal avg optical power"
},
"module_temperature_high_alarm": false,
"module_temperature_low_alarm": false,
"module_temperature_high_warning": false,
"module_temperature_low_warning": false,
"module_voltage_high_alarm": false,
"module_voltage_low_alarm": false,
"module_voltage_high_warning": false,
"module_voltage_low_warning": false,
"laser_bias_current_high_alarm": [
false,false,false,false,false,false,false,false ],
"laser_bias_current_low_alarm": [
false,false,false,false,false,false,false,false ],
"laser_bias_current_high_warning": [
false,false,false,false,false,false,false,false ],
"laser_bias_current_low_warning": [
false,false,false,false,false,false,false,false ],
"laser_tx_power_high_alarm": [
false,false,false,false,false,false,false,false ],
"laser_tx_power_low_alarm": [
false,false,false,false,false,false,false,false ],
"laser_tx_power_high_warning": [
false,false,false,false,false,false,false,false ],
"laser_tx_power_low_warning": [
false,false,false,false,false,false,false,false ],
"laser_rx_power_high_alarm": [
false,false,false,false,false,false,false,false ],
"laser_rx_power_low_alarm": [
false,false,false,false,false,false,false,false ],
"laser_rx_power_high_warning": [
false,false,false,false,false,false,false,false ],
"laser_rx_power_low_warning": [
false,false,false,false,false,false,false,false ],
"laser_bias_current": {
"high_alarm_threshold": 13,
"low_alarm_threshold": 3,
"high_warning_threshold": 11,
"low_warning_threshold": 5
},
"laser_output_power": {
"high_alarm_threshold": 3.1623,
"low_alarm_threshold": 0.1,
"high_warning_threshold": 1.9953,
"low_warning_threshold": 0.1585
},
"module_temperature": {
"high_alarm_threshold": 75,
"low_alarm_threshold": -5,
"high_warning_threshold": 70,
"low_warning_threshold": 0
},
"module_voltage": {
"high_alarm_threshold": 3.465,
"low_alarm_threshold": 3.135,
"high_warning_threshold": 3.45,
"low_warning_threshold": 3.15
},
"laser_rx_power": {
"high_alarm_threshold": 3.1623,
"low_alarm_threshold": 0.0398,
"high_warning_threshold": 2.5119,
"low_warning_threshold": 0.0794
},
"active_firmware_version": {
"major": 2,
"minor": 7
},
"inactive_firmware_version": {
"major": 2,
"minor": 7
},
"cdb_instances": 1,
"cdb_background_mode": "Supported",
"cdb_epl_pages": 0,
"cdb_maximum_epl_rw_length": 128,
"cdb_maximum_lpl_rw_length": 128,
"cdb_trigger_method": "Single write"
} ]
Signed-off-by: Danielle Ratson <redacted>
Reviewed-by: Amit Cohen <redacted>
---
Notes:
v3:
* Reword commit message.
cmis.c | 6 ++++++
ethtool.c | 1 +
2 files changed, 7 insertions(+)
diff --git a/cmis.c b/cmis.c
index 267d088..305814c 100644
--- a/cmis.c
+++ b/cmis.c@@ -1122,10 +1122,16 @@ int cmis_show_all_nl(struct cmd_context *ctx) struct cmis_memory_map map = {}; int ret; + new_json_obj(ctx->json); + open_json_object(NULL); + ret = cmis_memory_map_init_pages(ctx, &map); if (ret < 0) return ret; cmis_show_all_common(&map); + close_json_object(); + delete_json_obj(); + return 0; }
diff --git a/ethtool.c b/ethtool.c
index a1393bc..0b876e8 100644
--- a/ethtool.c
+++ b/ethtool.c@@ -6050,6 +6050,7 @@ static const struct option args[] = { }, { .opts = "-m|--dump-module-eeprom|--module-info", + .json = true, .func = do_getmodule, .nlfunc = nl_getmodule, .help = "Query/Decode Module EEPROM information and optical diagnostics if available",
--
2.47.0