Pull up the following, requested by msaitoh in ticket #629:

sys/dev/pci/pcireg.h			1.148
	sys/dev/pci/pci_subr.c			1.218-1.219

- Fix a bug that the virtual channel extended configuration's
  arbitration phase register can't be decoded correctly.
- Fix typo.
This commit is contained in:
martin 2020-01-21 15:15:23 +00:00
parent eeb9f0d030
commit 7c9b24a10d
2 changed files with 45 additions and 38 deletions

View File

@ -1,4 +1,4 @@
/* $NetBSD: pci_subr.c,v 1.215.2.1 2019/09/26 18:52:57 martin Exp $ */
/* $NetBSD: pci_subr.c,v 1.215.2.2 2020/01/21 15:15:23 martin Exp $ */
/*
* Copyright (c) 1997 Zubin D. Dittia. All rights reserved.
@ -40,7 +40,7 @@
*/
#include <sys/cdefs.h>
__KERNEL_RCSID(0, "$NetBSD: pci_subr.c,v 1.215.2.1 2019/09/26 18:52:57 martin Exp $");
__KERNEL_RCSID(0, "$NetBSD: pci_subr.c,v 1.215.2.2 2020/01/21 15:15:23 martin Exp $");
#ifdef _KERNEL_OPT
#include "opt_pci.h"
@ -2864,36 +2864,42 @@ pci_conf_print_aer_cap(const pcireg_t *regs, int extcapoff)
}
}
/*
* Helper function to print the arbitration phase register.
*
* phases: Number of phases in the arbitration tables.
* arbsize: Number of bits in each phase.
* indent: Add more two spaces if it's true.
*/
static void
pci_conf_print_vc_cap_arbtab(const pcireg_t *regs, int off, const char *name,
pcireg_t parbsel, int parbsize)
const int phases, int arbsize, bool indent)
{
pcireg_t reg;
int num = 16 << parbsel;
int num_per_reg = sizeof(pcireg_t) / parbsize;
int num_per_reg = 32 / arbsize;
int i, j;
/* First, dump the table */
for (i = 0; i < num; i += num_per_reg) {
reg = regs[o2i(off + i / num_per_reg)];
printf(" %s Arbitration Table: 0x%08x\n", name, reg);
}
/* And then, decode each entry */
for (i = 0; i < num; i += num_per_reg) {
reg = regs[o2i(off + i / num_per_reg)];
for (j = 0; j < num_per_reg; j++)
printf(" Phase[%d]: %d\n", j, reg);
printf("%s %s Arbitration Table:\n", indent ? " " : "", name);
for (i = 0; i < phases; i += num_per_reg) {
reg = regs[o2i(off + (sizeof(uint32_t) * (i / num_per_reg)))];
for (j = 0; j < num_per_reg; j++) {
printf("%s Phase[%d]: 0x%x\n", indent ? " " : "",
i + j,
(uint32_t)(reg & __BITS(arbsize - 1, 0)));
reg >>= arbsize;
}
}
}
/* For VC, bit 4-7 are reserved. For Port, bit 6-7 are reserved */
static const int arb_phases[8] = {0, 32, 64, 128, 128, 256, 0, 0 };
static void
pci_conf_print_vc_cap(const pcireg_t *regs, int extcapoff)
{
pcireg_t reg, n;
int parbtab, parbsize;
pcireg_t parbsel;
int varbtab, varbsize;
pcireg_t varbsel;
int arbtab, parbsize;
pcireg_t arbsel;
int i, count;
printf("\n Virtual Channel Register\n");
@ -2919,19 +2925,23 @@ pci_conf_print_vc_cap(const pcireg_t *regs, int extcapoff)
reg, PCI_VC_CAP2_ARB_CAP_WRR_64);
onoff("WRR arbitration with 128 phases",
reg, PCI_VC_CAP2_ARB_CAP_WRR_128);
varbtab = __SHIFTOUT(reg, PCI_VC_CAP2_ARB_TABLE_OFFSET);
printf(" VC Arbitration Table Offset: 0x%x\n", varbtab);
arbtab = __SHIFTOUT(reg, PCI_VC_CAP2_ARB_TABLE_OFFSET);
printf(" VC Arbitration Table Offset: 0x%x\n", arbtab);
reg = regs[o2i(extcapoff + PCI_VC_CONTROL)] & 0xffff;
printf(" Port VC Control register: 0x%04x\n", reg);
varbsel = __SHIFTOUT(reg, PCI_VC_CONTROL_VC_ARB_SELECT);
printf(" VC Arbitration Select: 0x%x\n", varbsel);
arbsel = __SHIFTOUT(reg, PCI_VC_CONTROL_VC_ARB_SELECT);
printf(" VC Arbitration Select: 0x%x\n", arbsel);
reg = regs[o2i(extcapoff + PCI_VC_STATUS)] >> 16;
printf(" Port VC Status register: 0x%04x\n", reg);
onoff("VC Arbitration Table Status",
reg, PCI_VC_STATUS_LOAD_VC_ARB_TABLE);
if ((arbtab != 0) && (arbsel != 0))
pci_conf_print_vc_cap_arbtab(regs, extcapoff + (arbtab * 16),
"VC", arb_phases[arbsel], 4, false);
for (i = 0; i < count + 1; i++) {
reg = regs[o2i(extcapoff + PCI_VC_RESOURCE_CAP(i))];
printf(" VC number %d\n", i);
@ -2954,9 +2964,10 @@ pci_conf_print_vc_cap(const pcireg_t *regs, int extcapoff)
reg, PCI_VC_RESOURCE_CAP_REJCT_SNOOP_TRANS);
n = __SHIFTOUT(reg, PCI_VC_RESOURCE_CAP_MAX_TIME_SLOTS) + 1;
printf(" Maximum Time Slots: %d\n", n);
parbtab = reg >> PCI_VC_RESOURCE_CAP_PORT_ARB_TABLE_OFFSET_S;
arbtab = __SHIFTOUT(reg,
PCI_VC_RESOURCE_CAP_PORT_ARB_TABLE_OFFSET);
printf(" Port Arbitration Table offset: 0x%02x\n",
parbtab);
arbtab);
reg = regs[o2i(extcapoff + PCI_VC_RESOURCE_CTL(i))];
printf(" VC Resource Control Register: 0x%08x\n", reg);
@ -2967,8 +2978,8 @@ pci_conf_print_vc_cap(const pcireg_t *regs, int extcapoff)
* the Port Arbitration logic and it's always 0 on read, so
* we don't print it.
*/
parbsel = __SHIFTOUT(reg, PCI_VC_RESOURCE_CTL_PORT_ARB_SELECT);
printf(" Port Arbitration Select: 0x%x\n", parbsel);
arbsel = __SHIFTOUT(reg, PCI_VC_RESOURCE_CTL_PORT_ARB_SELECT);
printf(" Port Arbitration Select: 0x%x\n", arbsel);
n = __SHIFTOUT(reg, PCI_VC_RESOURCE_CTL_VC_ID);
printf(" VC ID: %d\n", n);
onoff(" VC Enable", reg, PCI_VC_RESOURCE_CTL_VC_ENABLE);
@ -2980,15 +2991,11 @@ pci_conf_print_vc_cap(const pcireg_t *regs, int extcapoff)
onoff(" VC Negotiation Pending",
reg, PCI_VC_RESOURCE_STA_VC_NEG_PENDING);
if ((parbtab != 0) && (parbsel != 0))
pci_conf_print_vc_cap_arbtab(regs, extcapoff + parbtab,
"Port", parbsel, parbsize);
if ((arbtab != 0) && (arbsel != 0))
pci_conf_print_vc_cap_arbtab(regs,
extcapoff + (arbtab * 16),
"Port", arb_phases[arbsel], parbsize, true);
}
varbsize = 8;
if ((varbtab != 0) && (varbsel != 0))
pci_conf_print_vc_cap_arbtab(regs, extcapoff + varbtab,
" VC", varbsel, varbsize);
}
/*
@ -3531,7 +3538,7 @@ pci_conf_print_page_req_cap(const pcireg_t *regs, int extcapoff)
ctl = reg & 0xffff;
sta = reg >> 16;
printf(" Control Register: 0x%04x\n", ctl);
onoff("Enalbe", reg, PCI_PAGE_REQ_CTL_E);
onoff("Enable", reg, PCI_PAGE_REQ_CTL_E);
onoff("Reset", reg, PCI_PAGE_REQ_CTL_R);
printf(" Status Register: 0x%04x\n", sta);

View File

@ -1,4 +1,4 @@
/* $NetBSD: pcireg.h,v 1.147 2019/03/01 09:26:00 msaitoh Exp $ */
/* $NetBSD: pcireg.h,v 1.147.4.1 2020/01/21 15:15:23 martin Exp $ */
/*
* Copyright (c) 1995, 1996, 1999, 2000
@ -1886,7 +1886,7 @@ struct pci_rom {
* Page Request
*/
#define PCI_PAGE_REQ_CTL 0x04 /* Control Register */
#define PCI_PAGE_REQ_CTL_E __BIT(0) /* Enalbe */
#define PCI_PAGE_REQ_CTL_E __BIT(0) /* Enable */
#define PCI_PAGE_REQ_CTL_R __BIT(1) /* Reset */
#define PCI_PAGE_REQ_STA 0x04 /* Status Register */
#define PCI_PAGE_REQ_STA_RF __BIT(0+16) /* Response Failure */