[Top][All Lists]

[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.


reply via email to

[Prev in Thread] Current Thread [Next in Thread]