On Wed, Sep 09, 2015 at 06:34:00PM +0200, Paolo Bonzini wrote: > Signed-off-by: Paolo Bonzini <pbonz...@redhat.com> > --- > hw/pci/pcie_aer.c | 4 +- > include/standard-headers/linux/pci_regs.h | 381 > ++++++++++++++++++++------- > include/standard-headers/linux/virtio_ring.h | 3 +- > linux-headers/asm-x86/hyperv.h | 2 + > linux-headers/asm-x86/kvm.h | 4 +- > scripts/update-linux-headers.sh | 1 + > 6 files changed, 294 insertions(+), 101 deletions(-) > > diff --git a/hw/pci/pcie_aer.c b/hw/pci/pcie_aer.c > index f1847ac..8e4bd56 100644 > --- a/hw/pci/pcie_aer.c > +++ b/hw/pci/pcie_aer.c > @@ -827,8 +827,8 @@ typedef struct PCIEAERErrorName { > */ > static const struct PCIEAERErrorName pcie_aer_error_list[] = { > { > - .name = "TRAIN", > - .val = PCI_ERR_UNC_TRAIN, > + .name = "UND", > + .val = PCI_ERR_UNC_UND, > .correctable = false, > }, { > .name = "DLP",
I think we should drop this entry. The spec says: Undefined – The value read from this bit is undefined. In previous versions of this specification, this bit was used to indicate a Link Training Error. System software must ignore the value read from this bit. System software is permitted to write any value to this bit. In other words, this bit should be ignored. -- MST