Comment 6 for bug 2043983

Revision history for this message
Lucas Kanashiro (lucaskanashiro) wrote :

Right, I do not see more activity on this. I think we can try to release a SRU to fix this issue, if more comes in the future we fix them in another SRU.

From the upstream discussion (and what Ivar tested), all we need is this single line patch:

diff --git a/netlink/module-eeprom.c b/netlink/module-eeprom.c
index 49833a2a6a38..8b19f8e28c72 100644
--- a/netlink/module-eeprom.c
+++ b/netlink/module-eeprom.c
@@ -216,6 +216,8 @@ static int eeprom_parse(struct cmd_context *ctx)

        switch (request.data[0]) {
 #ifdef ETHTOOL_ENABLE_PRETTY_DUMP
+ case SFF8024_ID_SOLDERED_MODULE:
        case SFF8024_ID_SFP:
                return sff8079_show_all_nl(ctx);
        case SFF8024_ID_QSFP:

I will be preparing an update with this patch and doing the SRU paper work.

Ivar, I'll need you to perform the test with the fixed version when the time comes, ok?