[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