Hi Ian,
On 28/10/2021 16:54, Ian Jackson wrote:
Julien Grall writes ("Re: [PATCH] xen/arm: fix SBDF calculation for vPCI MMIO
handlers"):
On 28/10/2021 16:27, Oleksandr Andrushchenko wrote:
bridge as private and using info->gpa - GUEST_VPCI_ECAM_BASE...
So, I would stay with simpler
if ( bridge )
{
sbdf.sbdf = VPCI_ECAM_BDF(info->gpa - bridge->cfg->phys_addr);
sbdf.seg = bridge->segment;
}
else
sbdf.sbdf = VPCI_ECAM_BDF(info->gpa - GUEST_VPCI_ECAM_BASE);
I am fine with that so long this is part of an helper (maybe
vpci_sbdf_from_addr()) rather than duplicated.
There are a number of patches that I'm getting CC'd on related to ARM
and vpci (according to the Subject). Are these targeted for 4.16 ?
Most of them don't have 4.16 Subject tags.
Oleksandr wants this patch to be included for 4.16 but forgot to tag it
properly.
Cheers,
--
Julien Grall
