On Thu, Mar 30, 2017 at 06:29:29PM +0100, Anthony PERARD wrote:
>On Fri, Mar 17, 2017 at 07:29:17PM +0800, Lan Tianyu wrote:
>> From: Chao Gao <chao....@intel.com>
>> Subject: msi: taking interrupt format into consideration during
>> judging a pirq is binded with a event channel
>
>This is quite a long title, I think we can make it shorter. Maybe "msi:
>Handle MSI remapping format.
OK.
>
>> data = pci_get_word(dev->config + msi_data_off(dev, msi64bit));
>> - if (xen_is_pirq_msi(data)) {
>> + addr_lo = pci_get_word(dev->config + msi_address_lo_off(dev));
>
>This could be get_long, so addr_lo will actually have the all low bits
>of the addr.
yes.
>
>> +int xen_is_pirq_msi(uint32_t msi_data, uint32_t msi_addr_lo)
>> {
>> + /* If msi address is configurate to remapping format, the msi will not
>> + * remapped into a pirq.
>> + */
>> + if ( msi_addr_lo & 0x10 )
>
>That's a magic number, is 0x10 MSI_ADDR_IM_MASK?
Yes.
Really thanks for your comments.
>
>Thanks,
>
>--
>Anthony PERARD
