mirror of
https://github.com/qemu/qemu.git
synced 2024-11-25 20:03:37 +08:00
spapr_pci: Introduce a liobn number generating macros
We are going to have multiple DMA windows per PHB and we want them to migrate so we need a predictable way of assigning LIOBNs. This introduces a macro which makes up a LIOBN from fixed prefix, PHB index (unique PHB id) and window number. This introduces a SPAPR_PCI_DMA_WINDOW_NUM() to know the window number from LIOBN. It is used to distinguish the default 32bit windows from dynamic windows and avoid picking default DMA window properties from a wrong TCE table. Signed-off-by: Alexey Kardashevskiy <aik@ozlabs.ru> Reviewed-by: David Gibson <david@gibson.dropbear.id.au> Signed-off-by: David Gibson <david@gibson.dropbear.id.au> Signed-off-by: Alexander Graf <agraf@suse.de>
This commit is contained in:
parent
f1215ea702
commit
c8545818b3
@ -760,7 +760,7 @@ static void spapr_phb_realize(DeviceState *dev, Error **errp)
|
||||
}
|
||||
|
||||
sphb->buid = SPAPR_PCI_BASE_BUID + sphb->index;
|
||||
sphb->dma_liobn = SPAPR_PCI_BASE_LIOBN + sphb->index;
|
||||
sphb->dma_liobn = SPAPR_PCI_LIOBN(sphb->index, 0);
|
||||
|
||||
windows_base = SPAPR_PCI_WINDOW_BASE
|
||||
+ sphb->index * SPAPR_PCI_WINDOW_SPACING;
|
||||
@ -1101,7 +1101,7 @@ static int spapr_phb_children_dt(Object *child, void *opaque)
|
||||
sPAPRTCETable *tcet;
|
||||
|
||||
tcet = (sPAPRTCETable *) object_dynamic_cast(child, TYPE_SPAPR_TCE_TABLE);
|
||||
if (!tcet) {
|
||||
if (!tcet || SPAPR_PCI_DMA_WINDOW_NUM(tcet->liobn)) {
|
||||
return 0;
|
||||
}
|
||||
|
||||
|
@ -482,7 +482,9 @@ int spapr_rtas_device_tree_setup(void *fdt, hwaddr rtas_addr,
|
||||
#define SPAPR_TCE_PAGE_MASK (SPAPR_TCE_PAGE_SIZE - 1)
|
||||
|
||||
#define SPAPR_VIO_BASE_LIOBN 0x00000000
|
||||
#define SPAPR_PCI_BASE_LIOBN 0x80000000
|
||||
#define SPAPR_PCI_LIOBN(phb_index, window_num) \
|
||||
(0x80000000 | ((phb_index) << 8) | (window_num))
|
||||
#define SPAPR_PCI_DMA_WINDOW_NUM(liobn) ((liobn) & 0xff)
|
||||
|
||||
#define RTAS_ERROR_LOG_MAX 2048
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user