[prev in list] [next in list] [prev in thread] [next in thread] 

List:       linux-pci
Subject:    Re: [PATCH v4 00/11] PCI VPD access fixes
From:       Babu Moger <babu.moger () oracle ! com>
Date:       2016-02-29 22:36:41
Message-ID: 56D4C7F9.1060609 () oracle ! com
[Download RAW message or body]

On 2/23/2016 10:52 PM, Seymour, Shane M wrote:
> Retested v4. For the series tested two PCIe cards that support VPD:
> 
> 0d:00.0 Serial Attached SCSI controller: LSI Logic / Symbios Logic SAS2308 \
> PCI-Express Fusion-MPT SAS-2 (rev 05) 1e:00.0 Fibre Channel: QLogic Corp. \
> ISP2432-based 4Gb Fibre Channel to PCI Express HBA (rev 03) 
> The first card exposes 32KiB of NUL bytes as VPD and the second 154 bytes of \
> non-NUL data repeating every 4KiB for 32KiB before the change. After the change the \
> first  card returns no data for VPD (since it contains only NUL bytes) and the \
> second the expected 154 bytes of data. 
> The following warning message comes out as expected for the LSI card (changed from \
> dyn debug controlled message): 
> [  614.475984] mpt3sas 0000:0d:00.0: invalid short VPD tag 00 at offset 1
> 
> lspci works correctly with the patch in the kernel. The only change on the system \
> (apart from the size of the vpd file) was that with the LSI card lspci with -vvvv \
> as root it now says "Not readable"  instead of "Unknown small resource type 00, \
>                 will not decode more." in the VPD section.
> ---
> Tested-by: Shane Seymour <shane.seymour@hpe.com>
> 

I have also retested the V4 series. Did not see any side effects.
Once again here is the bug I opened for this issue.
https://bugzilla.kernel.org/show_bug.cgi?id=110681
---
Tested-by: Babu Moger <babu.moger@oracle.com>
--
To unsubscribe from this list: send the line "unsubscribe linux-pci" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic