[Skiboot] [PATCH 3/8] core/pci: Increase the max slot string size
Oliver O'Halloran
oohall at gmail.com
Mon Mar 18 17:10:53 AEDT 2019
The maximum string length for the slot label / device location code in
the PCI summary is currently 32 characters. This results in some IBM
location codes being truncated due to their length, e.g.
PHB#0001:02:11.0 [SWDN] SLOT=C11 x8
PHB#0001:13:00.0 [EP ] *snip* LOC_CODE=U78D3.ND1.WZS004A-P1-C
PHB#0001:13:00.1 [EP ] *snip* LOC_CODE=U78D3.ND1.WZS004A-P1-C
PHB#0001:13:00.2 [EP ] *snip* LOC_CODE=U78D3.ND1.WZS004A-P1-C
PHB#0001:13:00.3 [EP ] *snip* LOC_CODE=U78D3.ND1.WZS004A-P1-C
Which obscure the actual location of the card, and it looks bad. This
patch increases the maximum length of the label string to 80 characters
since that's the maximum length for a location code.
Signed-off-by: Oliver O'Halloran <oohall at gmail.com>
---
core/pci.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/core/pci.c b/core/pci.c
index 454b50102e59..7dbe6aace69e 100644
--- a/core/pci.c
+++ b/core/pci.c
@@ -1471,7 +1471,7 @@ static void pci_print_summary_line(struct phb *phb, struct pci_device *pd,
{
const char *label, *dtype, *s;
u32 vdid;
-#define MAX_SLOTSTR 32
+#define MAX_SLOTSTR 80
char slotstr[MAX_SLOTSTR + 1] = { 0, };
pci_cfg_read32(phb, pd->bdfn, 0, &vdid);
--
2.20.1
More information about the Skiboot
mailing list