[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3] pci : Add pba_offset PCI quirk for Chelsio T
From: |
Bandan Das |
Subject: |
Re: [Qemu-devel] [PATCH v3] pci : Add pba_offset PCI quirk for Chelsio T5 devices |
Date: |
Wed, 01 Jul 2015 14:27:07 -0400 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Alex Williamson <address@hidden> writes:
...
>> */
>> if (vendor == PCI_VENDOR_ID_CHELSIO && (device & 0xff00) == 0x5800) {
>> vdev->msix->pba_offset = 0x1000;
>> } else {
>> error_report("vfio: Hardware reports invalid configuration, "
>> "MSIX data outside of specified BAR");
>> return -EINVAL;
>> }
>> }
>> ...
>>
>> As the hardware problem is only related with the pba_offset and the purpose
>> of the quirk is to correct the known hardware error. The table_offset has
>> never been seen as wrong. Therefore the msix_init() sanity check should take
>> care of a "rare" potential error as you mentioned.
>>
>> This time I'll wait for ACKs from your side before submitting a new version
>> :)
>
> I would s/data/PBA/ in the error_report text for this version. Thanks,
Looks good to me, Gabriel + what Alex mentioned above.
Thanks,
Bandan