From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:48017) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XMZXz-000516-Bm for qemu-devel@nongnu.org; Wed, 27 Aug 2014 05:29:49 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1XMZXq-0001cE-Pm for qemu-devel@nongnu.org; Wed, 27 Aug 2014 05:29:43 -0400 Message-ID: <53FDA4FE.20907@suse.de> Date: Wed, 27 Aug 2014 11:29:34 +0200 From: Alexander Graf MIME-Version: 1.0 References: <1408097555-28126-1-git-send-email-aik@ozlabs.ru> <1408097555-28126-6-git-send-email-aik@ozlabs.ru> In-Reply-To: <1408097555-28126-6-git-send-email-aik@ozlabs.ru> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [RFC PATCH v2 05/13] spapr_pci: Introduce a liobn number generating macros List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Alexey Kardashevskiy , qemu-devel@nongnu.org Cc: Alex Williamson , qemu-ppc@nongnu.org, Gavin Shan , David Gibson On 15.08.14 12:12, Alexey Kardashevskiy wrote: > 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, will be used in next patch(es) to distinguish the default > 32bit windows from dynamic windows. > > Signed-off-by: Alexey Kardashevskiy > --- > hw/ppc/spapr_pci.c | 2 +- > include/hw/ppc/spapr.h | 3 ++- > 2 files changed, 3 insertions(+), 2 deletions(-) > > diff --git a/hw/ppc/spapr_pci.c b/hw/ppc/spapr_pci.c > index 5c46c0d..17eb0d8 100644 > --- a/hw/ppc/spapr_pci.c > +++ b/hw/ppc/spapr_pci.c > @@ -529,7 +529,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; > diff --git a/include/hw/ppc/spapr.h b/include/hw/ppc/spapr.h > index c9d6c6c..782519a 100644 > --- a/include/hw/ppc/spapr.h > +++ b/include/hw/ppc/spapr.h > @@ -443,7 +443,8 @@ 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(i, n) (0x80000000 | ((i) << 8) | (n)) > +#define SPAPR_PCI_DMA_WINDOW_NUM(liobn) ((liobn) & 0xff) Good idea. Mind to convert VIO as well to make them consistent? Alex