Skip to contentSkip to conversation

ipmi: fix seq number of bt msg in debug message

PATCH New Xie XiuQi 1 participant 1 message Apr 17, 2014 IPMI

lore
Share & tools
lore

Xie XiuQi

author
#0 lore raw

PATCH 1/1 ipmi: fix seq number of bt msg in debug message

The seq number of HOST2BMC message in debug message is not the real seq number but the next seq number; we could directly just use bt->write_data[2].

pre:

[99477.760284] BT: +++++++++++++++++ New command
[99477.760286] BT: NetFn/LUN CMD [1 data]: 28 10 0c
[99477.760296] BT: write 5 bytes seq=0x88 04 28 87 10 0c
                                 ^^^^^^^^       ^^
[99477.772078] BT: got 8 bytes seq=0x87 07 2c 87 10 00 f8 07 00
[99477.772091] BT: result 6 bytes: 2c 10 00 f8 07 00

post:

[99477.760284] BT: +++++++++++++++++ New command
[99477.760286] BT: NetFn/LUN CMD [1 data]: 28 10 0c
[99477.760296] BT: write 5 bytes seq=0x87 04 28 87 10 0c
                                 ^^^^^^^^       ^^
[99477.772078] BT: got 8 bytes seq=0x87 07 2c 87 10 00 f8 07 00
[99477.772091] BT: result 6 bytes: 2c 10 00 f8 07 00
---
 drivers/char/ipmi/ipmi_bt_sm.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
drivers/char/ipmi/ipmi_bt_sm.c +1 -1
index f5e4cd7..1a2056f 100644
@@ -341,7 +341,7 @@ static inline void write_all_bytes(struct si_sm_data *bt)
 
 	if (bt_debug & BT_DEBUG_MSG) {
 		printk(KERN_WARNING "BT: write %d bytes seq=0x%02X",
-			bt->write_count, bt->seq);
+			bt->write_count, bt->write_data[2]);
 		for (i = 0; i < bt->write_count; i++)
 			printk(" %02x", bt->write_data[i]);
 		printk("\n");
Signature · 1 line
1.6.0.2