diff mbox series

[v6,16/43] hw/cxl/rp: Add a root port

Message ID 20220211120747.3074-17-Jonathan.Cameron@huawei.com
State Superseded
Headers show
Series CXl 2.0 emulation Support | expand

Commit Message

Jonathan Cameron Feb. 11, 2022, 12:07 p.m. UTC
From: Ben Widawsky <ben.widawsky@intel.com>

This adds just enough of a root port implementation to be able to
enumerate root ports (creating the required DVSEC entries). What's not
here yet is the MMIO nor the ability to write some of the DVSEC entries.

This can be added with the qemu commandline by adding a rootport to a
specific CXL host bridge. For example:
  -device cxl-rp,id=rp0,bus="cxl.0",addr=0.0,chassis=4

Like the host bridge patch, the ACPI tables aren't generated at this
point and so system software cannot use it.

Signed-off-by: Ben Widawsky <ben.widawsky@intel.com>
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
---
 hw/pci-bridge/Kconfig          |   5 +
 hw/pci-bridge/cxl_root_port.c  | 231 +++++++++++++++++++++++++++++++++
 hw/pci-bridge/meson.build      |   1 +
 hw/pci-bridge/pcie_root_port.c |   6 +-
 hw/pci/pci.c                   |   4 +-
 5 files changed, 245 insertions(+), 2 deletions(-)

Comments

Alex Bennée March 1, 2022, 6:08 p.m. UTC | #1
Jonathan Cameron <Jonathan.Cameron@huawei.com> writes:

> From: Ben Widawsky <ben.widawsky@intel.com>
>
> This adds just enough of a root port implementation to be able to
> enumerate root ports (creating the required DVSEC entries). What's not
> here yet is the MMIO nor the ability to write some of the DVSEC entries.
>
> This can be added with the qemu commandline by adding a rootport to a
> specific CXL host bridge. For example:
>   -device cxl-rp,id=rp0,bus="cxl.0",addr=0.0,chassis=4
>
> Like the host bridge patch, the ACPI tables aren't generated at this
> point and so system software cannot use it.
>
> Signed-off-by: Ben Widawsky <ben.widawsky@intel.com>
> Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
> ---
>  hw/pci-bridge/Kconfig          |   5 +
>  hw/pci-bridge/cxl_root_port.c  | 231 +++++++++++++++++++++++++++++++++
>  hw/pci-bridge/meson.build      |   1 +
>  hw/pci-bridge/pcie_root_port.c |   6 +-
>  hw/pci/pci.c                   |   4 +-
>  5 files changed, 245 insertions(+), 2 deletions(-)
>
> diff --git a/hw/pci-bridge/Kconfig b/hw/pci-bridge/Kconfig
> index f8df4315ba..02614f49aa 100644
> --- a/hw/pci-bridge/Kconfig
> +++ b/hw/pci-bridge/Kconfig
> @@ -27,3 +27,8 @@ config DEC_PCI
>  
>  config SIMBA
>      bool
> +
> +config CXL
> +    bool
> +    default y if PCI_EXPRESS && PXB
> +    depends on PCI_EXPRESS && MSI_NONBROKEN && PXB
> diff --git a/hw/pci-bridge/cxl_root_port.c b/hw/pci-bridge/cxl_root_port.c
> new file mode 100644
> index 0000000000..dd714db836
> --- /dev/null
> +++ b/hw/pci-bridge/cxl_root_port.c
> @@ -0,0 +1,231 @@
> +/*
> + * CXL 2.0 Root Port Implementation
> + *
> + * Copyright(C) 2020 Intel Corporation.
> + *
> + * This library is free software; you can redistribute it and/or
> + * modify it under the terms of the GNU Lesser General Public
> + * License as published by the Free Software Foundation; either
> + * version 2 of the License, or (at your option) any later version.
> + *
> + * This library is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
> + * Lesser General Public License for more details.
> + *
> + * You should have received a copy of the GNU Lesser General Public
> + * License along with this library; if not, see <http://www.gnu.org/licenses/>
> + */
> +
> +#include "qemu/osdep.h"
> +#include "qemu/log.h"
> +#include "qemu/range.h"
> +#include "hw/pci/pci_bridge.h"
> +#include "hw/pci/pcie_port.h"
> +#include "hw/qdev-properties.h"
> +#include "hw/sysbus.h"
> +#include "qapi/error.h"
> +#include "hw/cxl/cxl.h"
> +
> +#define CXL_ROOT_PORT_DID 0x7075
> +
> +/* Copied from the gen root port which we derive */
> +#define GEN_PCIE_ROOT_PORT_AER_OFFSET 0x100
> +#define GEN_PCIE_ROOT_PORT_ACS_OFFSET \
> +    (GEN_PCIE_ROOT_PORT_AER_OFFSET + PCI_ERR_SIZEOF)
> +#define CXL_ROOT_PORT_DVSEC_OFFSET \
> +    (GEN_PCIE_ROOT_PORT_ACS_OFFSET + PCI_ACS_SIZEOF)
> +
> +typedef struct CXLRootPort {
> +    /*< private >*/
> +    PCIESlot parent_obj;
> +
> +    CXLComponentState cxl_cstate;
> +    PCIResReserve res_reserve;
> +} CXLRootPort;
> +
> +#define TYPE_CXL_ROOT_PORT "cxl-rp"
> +DECLARE_INSTANCE_CHECKER(CXLRootPort, CXL_ROOT_PORT, TYPE_CXL_ROOT_PORT)
> +
> +static void latch_registers(CXLRootPort *crp)
> +{
> +    uint32_t *reg_state = crp->cxl_cstate.crb.cache_mem_registers;
> +
> +    cxl_component_register_init_common(reg_state, CXL2_ROOT_PORT);
> +}
> +
> +static void build_dvsecs(CXLComponentState *cxl)
> +{
> +    uint8_t *dvsec;
> +
> +    dvsec = (uint8_t *)&(struct cxl_dvsec_port_extensions){ 0 };
> +    cxl_component_create_dvsec(cxl, EXTENSIONS_PORT_DVSEC_LENGTH,
> +                               EXTENSIONS_PORT_DVSEC,
> +                               EXTENSIONS_PORT_DVSEC_REVID, dvsec);
> +
> +    dvsec = (uint8_t *)&(struct cxl_dvsec_port_gpf){
> +        .rsvd        = 0,
> +        .phase1_ctrl = 1, /* 1μs timeout */
> +        .phase2_ctrl = 1, /* 1μs timeout */
> +    };
> +    cxl_component_create_dvsec(cxl, GPF_PORT_DVSEC_LENGTH, GPF_PORT_DVSEC,
> +                               GPF_PORT_DVSEC_REVID, dvsec);
> +
> +    dvsec = (uint8_t *)&(struct cxl_dvsec_port_flexbus){
> +        .cap                     = 0x26, /* IO, Mem, non-MLD */
> +        .ctrl                    = 0,
> +        .status                  = 0x26, /* same */
> +        .rcvd_mod_ts_data_phase1 = 0xef, /* WTF? */
> +    };
> +    cxl_component_create_dvsec(cxl, PCIE_FLEXBUS_PORT_DVSEC_LENGTH_2_0,
> +                               PCIE_FLEXBUS_PORT_DVSEC,
> +                               PCIE_FLEXBUS_PORT_DVSEC_REVID_2_0, dvsec);
> +
> +    dvsec = (uint8_t *)&(struct cxl_dvsec_register_locator){
> +        .rsvd         = 0,
> +        .reg0_base_lo = RBI_COMPONENT_REG | CXL_COMPONENT_REG_BAR_IDX,
> +        .reg0_base_hi = 0,
> +    };
> +    cxl_component_create_dvsec(cxl, REG_LOC_DVSEC_LENGTH, REG_LOC_DVSEC,
> +                               REG_LOC_DVSEC_REVID, dvsec);
> +}
> +
> +static void cxl_rp_realize(DeviceState *dev, Error **errp)
> +{
> +    PCIDevice *pci_dev     = PCI_DEVICE(dev);
> +    PCIERootPortClass *rpc = PCIE_ROOT_PORT_GET_CLASS(dev);
> +    CXLRootPort *crp       = CXL_ROOT_PORT(dev);
> +    CXLComponentState *cxl_cstate = &crp->cxl_cstate;
> +    ComponentRegisters *cregs = &cxl_cstate->crb;
> +    MemoryRegion *component_bar = &cregs->component_registers;
> +    Error *local_err = NULL;
> +
> +    rpc->parent_realize(dev, &local_err);
> +    if (local_err) {
> +        error_propagate(errp, local_err);
> +        return;
> +    }
> +
> +    int rc =
> +        pci_bridge_qemu_reserve_cap_init(pci_dev, 0, crp->res_reserve, errp);
> +    if (rc < 0) {
> +        rpc->parent_class.exit(pci_dev);
> +        return;
> +    }
> +
> +    if (!crp->res_reserve.io || crp->res_reserve.io == -1) {
> +        pci_word_test_and_clear_mask(pci_dev->wmask + PCI_COMMAND,
> +                                     PCI_COMMAND_IO);
> +        pci_dev->wmask[PCI_IO_BASE]  = 0;
> +        pci_dev->wmask[PCI_IO_LIMIT] = 0;
> +    }
> +
> +    cxl_cstate->dvsec_offset = CXL_ROOT_PORT_DVSEC_OFFSET;
> +    cxl_cstate->pdev = pci_dev;
> +    build_dvsecs(&crp->cxl_cstate);
> +
> +    cxl_component_register_block_init(OBJECT(pci_dev), cxl_cstate,
> +                                      TYPE_CXL_ROOT_PORT);
> +
> +    pci_register_bar(pci_dev, CXL_COMPONENT_REG_BAR_IDX,
> +                     PCI_BASE_ADDRESS_SPACE_MEMORY |
> +                         PCI_BASE_ADDRESS_MEM_TYPE_64,
> +                     component_bar);
> +}
> +
> +static void cxl_rp_reset(DeviceState *dev)
> +{
> +    PCIERootPortClass *rpc = PCIE_ROOT_PORT_GET_CLASS(dev);
> +    CXLRootPort *crp = CXL_ROOT_PORT(dev);
> +
> +    rpc->parent_reset(dev);
> +
> +    latch_registers(crp);
> +}
> +
> +static Property gen_rp_props[] = {
> +    DEFINE_PROP_UINT32("bus-reserve", CXLRootPort, res_reserve.bus, -1),
> +    DEFINE_PROP_SIZE("io-reserve", CXLRootPort, res_reserve.io, -1),
> +    DEFINE_PROP_SIZE("mem-reserve", CXLRootPort, res_reserve.mem_non_pref, -1),
> +    DEFINE_PROP_SIZE("pref32-reserve", CXLRootPort, res_reserve.mem_pref_32,
> +                     -1),
> +    DEFINE_PROP_SIZE("pref64-reserve", CXLRootPort, res_reserve.mem_pref_64,
> +                     -1),
> +    DEFINE_PROP_END_OF_LIST()
> +};
> +
> +static void cxl_rp_dvsec_write_config(PCIDevice *dev, uint32_t addr,
> +                                      uint32_t val, int len)
> +{
> +    CXLRootPort *crp = CXL_ROOT_PORT(dev);
> +
> +    if (range_contains(&crp->cxl_cstate.dvsecs[EXTENSIONS_PORT_DVSEC], addr)) {
> +        uint8_t *reg = &dev->config[addr];
> +        addr -= crp->cxl_cstate.dvsecs[EXTENSIONS_PORT_DVSEC].lob;
> +        if (addr == PORT_CONTROL_OFFSET) {
> +            if (pci_get_word(reg) & PORT_CONTROL_UNMASK_SBR) {
> +                /* unmask SBR */
> +            }
> +            if (pci_get_word(reg) & PORT_CONTROL_ALT_MEMID_EN) {
> +                /* Alt Memory & ID Space Enable */
> +            }

Can we have LOG_UNIMPs for these null implementations please.

> +        }
> +    }
> +}
> +
> +static void cxl_rp_write_config(PCIDevice *d, uint32_t address, uint32_t val,
> +                                int len)
> +{
> +    uint16_t slt_ctl, slt_sta;
> +
> +    pcie_cap_slot_get(d, &slt_ctl, &slt_sta);
> +    pci_bridge_write_config(d, address, val, len);
> +    pcie_cap_flr_write_config(d, address, val, len);
> +    pcie_cap_slot_write_config(d, slt_ctl, slt_sta, address, val, len);
> +    pcie_aer_write_config(d, address, val, len);
> +
> +    cxl_rp_dvsec_write_config(d, address, val, len);
> +}
> +
> +static void cxl_root_port_class_init(ObjectClass *oc, void *data)
> +{
> +    DeviceClass *dc        = DEVICE_CLASS(oc);
> +    PCIDeviceClass *k      = PCI_DEVICE_CLASS(oc);
> +    PCIERootPortClass *rpc = PCIE_ROOT_PORT_CLASS(oc);
> +
> +    k->vendor_id = PCI_VENDOR_ID_INTEL;
> +    k->device_id = CXL_ROOT_PORT_DID;
> +    dc->desc     = "CXL Root Port";
> +    k->revision  = 0;
> +    device_class_set_props(dc, gen_rp_props);
> +    k->config_write = cxl_rp_write_config;
> +
> +    device_class_set_parent_realize(dc, cxl_rp_realize, &rpc->parent_realize);
> +    device_class_set_parent_reset(dc, cxl_rp_reset, &rpc->parent_reset);
> +
> +    rpc->aer_offset = GEN_PCIE_ROOT_PORT_AER_OFFSET;
> +    rpc->acs_offset = GEN_PCIE_ROOT_PORT_ACS_OFFSET;
> +
> +    /*
> +     * Explain
> +     */

You might as well either expand the comment or drop it, the code makes
it clear it is not hot pluggable (although an explanation would be
better - given it's PCIe like roots isn't hotplug a thing you see on CXL?)

> +    dc->hotpluggable = false;
> +}
> +
> +static const TypeInfo cxl_root_port_info = {
> +    .name = TYPE_CXL_ROOT_PORT,
> +    .parent = TYPE_PCIE_ROOT_PORT,
> +    .instance_size = sizeof(CXLRootPort),
> +    .class_init = cxl_root_port_class_init,
> +    .interfaces = (InterfaceInfo[]) {
> +        { INTERFACE_CXL_DEVICE },
> +        { }
> +    },
> +};
> +
> +static void cxl_register(void)
> +{
> +    type_register_static(&cxl_root_port_info);
> +}
> +
> +type_init(cxl_register);
> diff --git a/hw/pci-bridge/meson.build b/hw/pci-bridge/meson.build
> index daab8acf2a..b6d26a03d5 100644
> --- a/hw/pci-bridge/meson.build
> +++ b/hw/pci-bridge/meson.build
> @@ -5,6 +5,7 @@ pci_ss.add(when: 'CONFIG_IOH3420', if_true: files('ioh3420.c'))
>  pci_ss.add(when: 'CONFIG_PCIE_PORT', if_true: files('pcie_root_port.c', 'gen_pcie_root_port.c', 'pcie_pci_bridge.c'))
>  pci_ss.add(when: 'CONFIG_PXB', if_true: files('pci_expander_bridge.c'))
>  pci_ss.add(when: 'CONFIG_XIO3130', if_true: files('xio3130_upstream.c', 'xio3130_downstream.c'))
> +pci_ss.add(when: 'CONFIG_CXL', if_true: files('cxl_root_port.c'))
>  
>  # NewWorld PowerMac
>  pci_ss.add(when: 'CONFIG_DEC_PCI', if_true: files('dec.c'))
> diff --git a/hw/pci-bridge/pcie_root_port.c b/hw/pci-bridge/pcie_root_port.c
> index f1cfe9d14a..460e48269d 100644
> --- a/hw/pci-bridge/pcie_root_port.c
> +++ b/hw/pci-bridge/pcie_root_port.c
> @@ -67,7 +67,11 @@ static void rp_realize(PCIDevice *d, Error **errp)
>      int rc;
>  
>      pci_config_set_interrupt_pin(d->config, 1);
> -    pci_bridge_initfn(d, TYPE_PCIE_BUS);
> +    if (d->cap_present & QEMU_PCIE_CAP_CXL) {
> +        pci_bridge_initfn(d, TYPE_CXL_BUS);
> +    } else {
> +        pci_bridge_initfn(d, TYPE_PCIE_BUS);
> +    }
>      pcie_port_init_reg(d);
>  
>      rc = pci_bridge_ssvid_init(d, rpc->ssvid_offset, dc->vendor_id,
> diff --git a/hw/pci/pci.c b/hw/pci/pci.c
> index cafebf6f59..cc4f06937d 100644
> --- a/hw/pci/pci.c
> +++ b/hw/pci/pci.c
> @@ -2708,7 +2708,9 @@ static void pci_device_class_base_init(ObjectClass *klass, void *data)
>              object_class_dynamic_cast(klass, INTERFACE_CONVENTIONAL_PCI_DEVICE);
>          ObjectClass *pcie =
>              object_class_dynamic_cast(klass, INTERFACE_PCIE_DEVICE);
> -        assert(conventional || pcie);
> +        ObjectClass *cxl =
> +            object_class_dynamic_cast(klass, INTERFACE_CXL_DEVICE);
> +        assert(conventional || pcie || cxl);
>      }
>  }

Otherwise:

Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
Jonathan Cameron March 3, 2022, 5:22 p.m. UTC | #2
On Tue, 01 Mar 2022 18:08:31 +0000
Alex Bennée <alex.bennee@linaro.org> wrote:

> Jonathan Cameron <Jonathan.Cameron@huawei.com> writes:
> 
> > From: Ben Widawsky <ben.widawsky@intel.com>
> >
> > This adds just enough of a root port implementation to be able to
> > enumerate root ports (creating the required DVSEC entries). What's not
> > here yet is the MMIO nor the ability to write some of the DVSEC entries.
> >
> > This can be added with the qemu commandline by adding a rootport to a
> > specific CXL host bridge. For example:
> >   -device cxl-rp,id=rp0,bus="cxl.0",addr=0.0,chassis=4
> >
> > Like the host bridge patch, the ACPI tables aren't generated at this
> > point and so system software cannot use it.
> >
> > Signed-off-by: Ben Widawsky <ben.widawsky@intel.com>
> > Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
> > ---
> >  hw/pci-bridge/Kconfig          |   5 +
> >  hw/pci-bridge/cxl_root_port.c  | 231 +++++++++++++++++++++++++++++++++
> >  hw/pci-bridge/meson.build      |   1 +
> >  hw/pci-bridge/pcie_root_port.c |   6 +-
> >  hw/pci/pci.c                   |   4 +-
> >  5 files changed, 245 insertions(+), 2 deletions(-)
> >
> > diff --git a/hw/pci-bridge/Kconfig b/hw/pci-bridge/Kconfig
> > index f8df4315ba..02614f49aa 100644
> > --- a/hw/pci-bridge/Kconfig
> > +++ b/hw/pci-bridge/Kconfig
> > @@ -27,3 +27,8 @@ config DEC_PCI
> >  
> >  config SIMBA
> >      bool
> > +
> > +config CXL
> > +    bool
> > +    default y if PCI_EXPRESS && PXB
> > +    depends on PCI_EXPRESS && MSI_NONBROKEN && PXB
> > diff --git a/hw/pci-bridge/cxl_root_port.c b/hw/pci-bridge/cxl_root_port.c
> > new file mode 100644
> > index 0000000000..dd714db836
> > --- /dev/null
> > +++ b/hw/pci-bridge/cxl_root_port.c
> > @@ -0,0 +1,231 @@
> > +/*
> > + * CXL 2.0 Root Port Implementation
> > + *
> > + * Copyright(C) 2020 Intel Corporation.
> > + *
> > + * This library is free software; you can redistribute it and/or
> > + * modify it under the terms of the GNU Lesser General Public
> > + * License as published by the Free Software Foundation; either
> > + * version 2 of the License, or (at your option) any later version.
> > + *
> > + * This library is distributed in the hope that it will be useful,
> > + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
> > + * Lesser General Public License for more details.
> > + *
> > + * You should have received a copy of the GNU Lesser General Public
> > + * License along with this library; if not, see <http://www.gnu.org/licenses/>
> > + */
> > +
> > +#include "qemu/osdep.h"
> > +#include "qemu/log.h"
> > +#include "qemu/range.h"
> > +#include "hw/pci/pci_bridge.h"
> > +#include "hw/pci/pcie_port.h"
> > +#include "hw/qdev-properties.h"
> > +#include "hw/sysbus.h"
> > +#include "qapi/error.h"
> > +#include "hw/cxl/cxl.h"
> > +
> > +#define CXL_ROOT_PORT_DID 0x7075
> > +
> > +/* Copied from the gen root port which we derive */
> > +#define GEN_PCIE_ROOT_PORT_AER_OFFSET 0x100
> > +#define GEN_PCIE_ROOT_PORT_ACS_OFFSET \
> > +    (GEN_PCIE_ROOT_PORT_AER_OFFSET + PCI_ERR_SIZEOF)
> > +#define CXL_ROOT_PORT_DVSEC_OFFSET \
> > +    (GEN_PCIE_ROOT_PORT_ACS_OFFSET + PCI_ACS_SIZEOF)
> > +
> > +typedef struct CXLRootPort {
> > +    /*< private >*/
> > +    PCIESlot parent_obj;
> > +
> > +    CXLComponentState cxl_cstate;
> > +    PCIResReserve res_reserve;
> > +} CXLRootPort;
> > +
> > +#define TYPE_CXL_ROOT_PORT "cxl-rp"
> > +DECLARE_INSTANCE_CHECKER(CXLRootPort, CXL_ROOT_PORT, TYPE_CXL_ROOT_PORT)
> > +
> > +static void latch_registers(CXLRootPort *crp)
> > +{
> > +    uint32_t *reg_state = crp->cxl_cstate.crb.cache_mem_registers;
> > +
> > +    cxl_component_register_init_common(reg_state, CXL2_ROOT_PORT);
> > +}
> > +
> > +static void build_dvsecs(CXLComponentState *cxl)
> > +{
> > +    uint8_t *dvsec;
> > +
> > +    dvsec = (uint8_t *)&(struct cxl_dvsec_port_extensions){ 0 };
> > +    cxl_component_create_dvsec(cxl, EXTENSIONS_PORT_DVSEC_LENGTH,
> > +                               EXTENSIONS_PORT_DVSEC,
> > +                               EXTENSIONS_PORT_DVSEC_REVID, dvsec);
> > +
> > +    dvsec = (uint8_t *)&(struct cxl_dvsec_port_gpf){
> > +        .rsvd        = 0,
> > +        .phase1_ctrl = 1, /* 1μs timeout */
> > +        .phase2_ctrl = 1, /* 1μs timeout */
> > +    };
> > +    cxl_component_create_dvsec(cxl, GPF_PORT_DVSEC_LENGTH, GPF_PORT_DVSEC,
> > +                               GPF_PORT_DVSEC_REVID, dvsec);
> > +
> > +    dvsec = (uint8_t *)&(struct cxl_dvsec_port_flexbus){
> > +        .cap                     = 0x26, /* IO, Mem, non-MLD */
> > +        .ctrl                    = 0,
> > +        .status                  = 0x26, /* same */
> > +        .rcvd_mod_ts_data_phase1 = 0xef, /* WTF? */
> > +    };
> > +    cxl_component_create_dvsec(cxl, PCIE_FLEXBUS_PORT_DVSEC_LENGTH_2_0,
> > +                               PCIE_FLEXBUS_PORT_DVSEC,
> > +                               PCIE_FLEXBUS_PORT_DVSEC_REVID_2_0, dvsec);
> > +
> > +    dvsec = (uint8_t *)&(struct cxl_dvsec_register_locator){
> > +        .rsvd         = 0,
> > +        .reg0_base_lo = RBI_COMPONENT_REG | CXL_COMPONENT_REG_BAR_IDX,
> > +        .reg0_base_hi = 0,
> > +    };
> > +    cxl_component_create_dvsec(cxl, REG_LOC_DVSEC_LENGTH, REG_LOC_DVSEC,
> > +                               REG_LOC_DVSEC_REVID, dvsec);
> > +}
> > +
> > +static void cxl_rp_realize(DeviceState *dev, Error **errp)
> > +{
> > +    PCIDevice *pci_dev     = PCI_DEVICE(dev);
> > +    PCIERootPortClass *rpc = PCIE_ROOT_PORT_GET_CLASS(dev);
> > +    CXLRootPort *crp       = CXL_ROOT_PORT(dev);
> > +    CXLComponentState *cxl_cstate = &crp->cxl_cstate;
> > +    ComponentRegisters *cregs = &cxl_cstate->crb;
> > +    MemoryRegion *component_bar = &cregs->component_registers;
> > +    Error *local_err = NULL;
> > +
> > +    rpc->parent_realize(dev, &local_err);
> > +    if (local_err) {
> > +        error_propagate(errp, local_err);
> > +        return;
> > +    }
> > +
> > +    int rc =
> > +        pci_bridge_qemu_reserve_cap_init(pci_dev, 0, crp->res_reserve, errp);
> > +    if (rc < 0) {
> > +        rpc->parent_class.exit(pci_dev);
> > +        return;
> > +    }
> > +
> > +    if (!crp->res_reserve.io || crp->res_reserve.io == -1) {
> > +        pci_word_test_and_clear_mask(pci_dev->wmask + PCI_COMMAND,
> > +                                     PCI_COMMAND_IO);
> > +        pci_dev->wmask[PCI_IO_BASE]  = 0;
> > +        pci_dev->wmask[PCI_IO_LIMIT] = 0;
> > +    }
> > +
> > +    cxl_cstate->dvsec_offset = CXL_ROOT_PORT_DVSEC_OFFSET;
> > +    cxl_cstate->pdev = pci_dev;
> > +    build_dvsecs(&crp->cxl_cstate);
> > +
> > +    cxl_component_register_block_init(OBJECT(pci_dev), cxl_cstate,
> > +                                      TYPE_CXL_ROOT_PORT);
> > +
> > +    pci_register_bar(pci_dev, CXL_COMPONENT_REG_BAR_IDX,
> > +                     PCI_BASE_ADDRESS_SPACE_MEMORY |
> > +                         PCI_BASE_ADDRESS_MEM_TYPE_64,
> > +                     component_bar);
> > +}
> > +
> > +static void cxl_rp_reset(DeviceState *dev)
> > +{
> > +    PCIERootPortClass *rpc = PCIE_ROOT_PORT_GET_CLASS(dev);
> > +    CXLRootPort *crp = CXL_ROOT_PORT(dev);
> > +
> > +    rpc->parent_reset(dev);
> > +
> > +    latch_registers(crp);
> > +}
> > +
> > +static Property gen_rp_props[] = {
> > +    DEFINE_PROP_UINT32("bus-reserve", CXLRootPort, res_reserve.bus, -1),
> > +    DEFINE_PROP_SIZE("io-reserve", CXLRootPort, res_reserve.io, -1),
> > +    DEFINE_PROP_SIZE("mem-reserve", CXLRootPort, res_reserve.mem_non_pref, -1),
> > +    DEFINE_PROP_SIZE("pref32-reserve", CXLRootPort, res_reserve.mem_pref_32,
> > +                     -1),
> > +    DEFINE_PROP_SIZE("pref64-reserve", CXLRootPort, res_reserve.mem_pref_64,
> > +                     -1),
> > +    DEFINE_PROP_END_OF_LIST()
> > +};
> > +
> > +static void cxl_rp_dvsec_write_config(PCIDevice *dev, uint32_t addr,
> > +                                      uint32_t val, int len)
> > +{
> > +    CXLRootPort *crp = CXL_ROOT_PORT(dev);
> > +
> > +    if (range_contains(&crp->cxl_cstate.dvsecs[EXTENSIONS_PORT_DVSEC], addr)) {
> > +        uint8_t *reg = &dev->config[addr];
> > +        addr -= crp->cxl_cstate.dvsecs[EXTENSIONS_PORT_DVSEC].lob;
> > +        if (addr == PORT_CONTROL_OFFSET) {
> > +            if (pci_get_word(reg) & PORT_CONTROL_UNMASK_SBR) {
> > +                /* unmask SBR */
> > +            }
> > +            if (pci_get_word(reg) & PORT_CONTROL_ALT_MEMID_EN) {
> > +                /* Alt Memory & ID Space Enable */
> > +            }  
> 
> Can we have LOG_UNIMPs for these null implementations please.

Added.  I have implementation for SBR unmasking, but that will be part of
a future patch set so definitely makes sense to log for now.

The alt memory stuff is about making a CXL 1.1 device that is attached
to a CXL switch work (?).  In that case we have to make it
magically appear as an RCiEP despite being below a switch and
hence have to mess around with routing to make that work - effectively
we are tunneling an extra PCI topology through the existing one.

Given we generally don't care today about emulating CXL 1.1 devices
at all, it may never be implemented in QEMU.
So definitely good to log that...


> 
> > +        }
> > +    }
> > +}
> > +

...

> > +static void cxl_root_port_class_init(ObjectClass *oc, void *data)
> > +{
> > +    DeviceClass *dc        = DEVICE_CLASS(oc);
> > +    PCIDeviceClass *k      = PCI_DEVICE_CLASS(oc);
> > +    PCIERootPortClass *rpc = PCIE_ROOT_PORT_CLASS(oc);
> > +
> > +    k->vendor_id = PCI_VENDOR_ID_INTEL;
> > +    k->device_id = CXL_ROOT_PORT_DID;
> > +    dc->desc     = "CXL Root Port";
> > +    k->revision  = 0;
> > +    device_class_set_props(dc, gen_rp_props);
> > +    k->config_write = cxl_rp_write_config;
> > +
> > +    device_class_set_parent_realize(dc, cxl_rp_realize, &rpc->parent_realize);
> > +    device_class_set_parent_reset(dc, cxl_rp_reset, &rpc->parent_reset);
> > +
> > +    rpc->aer_offset = GEN_PCIE_ROOT_PORT_AER_OFFSET;
> > +    rpc->acs_offset = GEN_PCIE_ROOT_PORT_ACS_OFFSET;
> > +
> > +    /*
> > +     * Explain
> > +     */  
> 
> You might as well either expand the comment or drop it, the code makes
> it clear it is not hot pluggable (although an explanation would be
> better - given it's PCIe like roots isn't hotplug a thing you see on CXL?)

Hotplug of devices is indeed something we care about in CXL
though for now it's untested. Hotplug of the root ports themselves is going
to be 'interesting' due to the memory decoding targeting the ports.
For now I've just dropped the pointless comment.

Ben: If you had a planned explanation to put here, can you remember what
it was?  We can add it later though - for now just dropping the comment
seems best way forward to me.

Thanks,

Jonathan
diff mbox series

Patch

diff --git a/hw/pci-bridge/Kconfig b/hw/pci-bridge/Kconfig
index f8df4315ba..02614f49aa 100644
--- a/hw/pci-bridge/Kconfig
+++ b/hw/pci-bridge/Kconfig
@@ -27,3 +27,8 @@  config DEC_PCI
 
 config SIMBA
     bool
+
+config CXL
+    bool
+    default y if PCI_EXPRESS && PXB
+    depends on PCI_EXPRESS && MSI_NONBROKEN && PXB
diff --git a/hw/pci-bridge/cxl_root_port.c b/hw/pci-bridge/cxl_root_port.c
new file mode 100644
index 0000000000..dd714db836
--- /dev/null
+++ b/hw/pci-bridge/cxl_root_port.c
@@ -0,0 +1,231 @@ 
+/*
+ * CXL 2.0 Root Port Implementation
+ *
+ * Copyright(C) 2020 Intel Corporation.
+ *
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Software Foundation; either
+ * version 2 of the License, or (at your option) any later version.
+ *
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with this library; if not, see <http://www.gnu.org/licenses/>
+ */
+
+#include "qemu/osdep.h"
+#include "qemu/log.h"
+#include "qemu/range.h"
+#include "hw/pci/pci_bridge.h"
+#include "hw/pci/pcie_port.h"
+#include "hw/qdev-properties.h"
+#include "hw/sysbus.h"
+#include "qapi/error.h"
+#include "hw/cxl/cxl.h"
+
+#define CXL_ROOT_PORT_DID 0x7075
+
+/* Copied from the gen root port which we derive */
+#define GEN_PCIE_ROOT_PORT_AER_OFFSET 0x100
+#define GEN_PCIE_ROOT_PORT_ACS_OFFSET \
+    (GEN_PCIE_ROOT_PORT_AER_OFFSET + PCI_ERR_SIZEOF)
+#define CXL_ROOT_PORT_DVSEC_OFFSET \
+    (GEN_PCIE_ROOT_PORT_ACS_OFFSET + PCI_ACS_SIZEOF)
+
+typedef struct CXLRootPort {
+    /*< private >*/
+    PCIESlot parent_obj;
+
+    CXLComponentState cxl_cstate;
+    PCIResReserve res_reserve;
+} CXLRootPort;
+
+#define TYPE_CXL_ROOT_PORT "cxl-rp"
+DECLARE_INSTANCE_CHECKER(CXLRootPort, CXL_ROOT_PORT, TYPE_CXL_ROOT_PORT)
+
+static void latch_registers(CXLRootPort *crp)
+{
+    uint32_t *reg_state = crp->cxl_cstate.crb.cache_mem_registers;
+
+    cxl_component_register_init_common(reg_state, CXL2_ROOT_PORT);
+}
+
+static void build_dvsecs(CXLComponentState *cxl)
+{
+    uint8_t *dvsec;
+
+    dvsec = (uint8_t *)&(struct cxl_dvsec_port_extensions){ 0 };
+    cxl_component_create_dvsec(cxl, EXTENSIONS_PORT_DVSEC_LENGTH,
+                               EXTENSIONS_PORT_DVSEC,
+                               EXTENSIONS_PORT_DVSEC_REVID, dvsec);
+
+    dvsec = (uint8_t *)&(struct cxl_dvsec_port_gpf){
+        .rsvd        = 0,
+        .phase1_ctrl = 1, /* 1μs timeout */
+        .phase2_ctrl = 1, /* 1μs timeout */
+    };
+    cxl_component_create_dvsec(cxl, GPF_PORT_DVSEC_LENGTH, GPF_PORT_DVSEC,
+                               GPF_PORT_DVSEC_REVID, dvsec);
+
+    dvsec = (uint8_t *)&(struct cxl_dvsec_port_flexbus){
+        .cap                     = 0x26, /* IO, Mem, non-MLD */
+        .ctrl                    = 0,
+        .status                  = 0x26, /* same */
+        .rcvd_mod_ts_data_phase1 = 0xef, /* WTF? */
+    };
+    cxl_component_create_dvsec(cxl, PCIE_FLEXBUS_PORT_DVSEC_LENGTH_2_0,
+                               PCIE_FLEXBUS_PORT_DVSEC,
+                               PCIE_FLEXBUS_PORT_DVSEC_REVID_2_0, dvsec);
+
+    dvsec = (uint8_t *)&(struct cxl_dvsec_register_locator){
+        .rsvd         = 0,
+        .reg0_base_lo = RBI_COMPONENT_REG | CXL_COMPONENT_REG_BAR_IDX,
+        .reg0_base_hi = 0,
+    };
+    cxl_component_create_dvsec(cxl, REG_LOC_DVSEC_LENGTH, REG_LOC_DVSEC,
+                               REG_LOC_DVSEC_REVID, dvsec);
+}
+
+static void cxl_rp_realize(DeviceState *dev, Error **errp)
+{
+    PCIDevice *pci_dev     = PCI_DEVICE(dev);
+    PCIERootPortClass *rpc = PCIE_ROOT_PORT_GET_CLASS(dev);
+    CXLRootPort *crp       = CXL_ROOT_PORT(dev);
+    CXLComponentState *cxl_cstate = &crp->cxl_cstate;
+    ComponentRegisters *cregs = &cxl_cstate->crb;
+    MemoryRegion *component_bar = &cregs->component_registers;
+    Error *local_err = NULL;
+
+    rpc->parent_realize(dev, &local_err);
+    if (local_err) {
+        error_propagate(errp, local_err);
+        return;
+    }
+
+    int rc =
+        pci_bridge_qemu_reserve_cap_init(pci_dev, 0, crp->res_reserve, errp);
+    if (rc < 0) {
+        rpc->parent_class.exit(pci_dev);
+        return;
+    }
+
+    if (!crp->res_reserve.io || crp->res_reserve.io == -1) {
+        pci_word_test_and_clear_mask(pci_dev->wmask + PCI_COMMAND,
+                                     PCI_COMMAND_IO);
+        pci_dev->wmask[PCI_IO_BASE]  = 0;
+        pci_dev->wmask[PCI_IO_LIMIT] = 0;
+    }
+
+    cxl_cstate->dvsec_offset = CXL_ROOT_PORT_DVSEC_OFFSET;
+    cxl_cstate->pdev = pci_dev;
+    build_dvsecs(&crp->cxl_cstate);
+
+    cxl_component_register_block_init(OBJECT(pci_dev), cxl_cstate,
+                                      TYPE_CXL_ROOT_PORT);
+
+    pci_register_bar(pci_dev, CXL_COMPONENT_REG_BAR_IDX,
+                     PCI_BASE_ADDRESS_SPACE_MEMORY |
+                         PCI_BASE_ADDRESS_MEM_TYPE_64,
+                     component_bar);
+}
+
+static void cxl_rp_reset(DeviceState *dev)
+{
+    PCIERootPortClass *rpc = PCIE_ROOT_PORT_GET_CLASS(dev);
+    CXLRootPort *crp = CXL_ROOT_PORT(dev);
+
+    rpc->parent_reset(dev);
+
+    latch_registers(crp);
+}
+
+static Property gen_rp_props[] = {
+    DEFINE_PROP_UINT32("bus-reserve", CXLRootPort, res_reserve.bus, -1),
+    DEFINE_PROP_SIZE("io-reserve", CXLRootPort, res_reserve.io, -1),
+    DEFINE_PROP_SIZE("mem-reserve", CXLRootPort, res_reserve.mem_non_pref, -1),
+    DEFINE_PROP_SIZE("pref32-reserve", CXLRootPort, res_reserve.mem_pref_32,
+                     -1),
+    DEFINE_PROP_SIZE("pref64-reserve", CXLRootPort, res_reserve.mem_pref_64,
+                     -1),
+    DEFINE_PROP_END_OF_LIST()
+};
+
+static void cxl_rp_dvsec_write_config(PCIDevice *dev, uint32_t addr,
+                                      uint32_t val, int len)
+{
+    CXLRootPort *crp = CXL_ROOT_PORT(dev);
+
+    if (range_contains(&crp->cxl_cstate.dvsecs[EXTENSIONS_PORT_DVSEC], addr)) {
+        uint8_t *reg = &dev->config[addr];
+        addr -= crp->cxl_cstate.dvsecs[EXTENSIONS_PORT_DVSEC].lob;
+        if (addr == PORT_CONTROL_OFFSET) {
+            if (pci_get_word(reg) & PORT_CONTROL_UNMASK_SBR) {
+                /* unmask SBR */
+            }
+            if (pci_get_word(reg) & PORT_CONTROL_ALT_MEMID_EN) {
+                /* Alt Memory & ID Space Enable */
+            }
+        }
+    }
+}
+
+static void cxl_rp_write_config(PCIDevice *d, uint32_t address, uint32_t val,
+                                int len)
+{
+    uint16_t slt_ctl, slt_sta;
+
+    pcie_cap_slot_get(d, &slt_ctl, &slt_sta);
+    pci_bridge_write_config(d, address, val, len);
+    pcie_cap_flr_write_config(d, address, val, len);
+    pcie_cap_slot_write_config(d, slt_ctl, slt_sta, address, val, len);
+    pcie_aer_write_config(d, address, val, len);
+
+    cxl_rp_dvsec_write_config(d, address, val, len);
+}
+
+static void cxl_root_port_class_init(ObjectClass *oc, void *data)
+{
+    DeviceClass *dc        = DEVICE_CLASS(oc);
+    PCIDeviceClass *k      = PCI_DEVICE_CLASS(oc);
+    PCIERootPortClass *rpc = PCIE_ROOT_PORT_CLASS(oc);
+
+    k->vendor_id = PCI_VENDOR_ID_INTEL;
+    k->device_id = CXL_ROOT_PORT_DID;
+    dc->desc     = "CXL Root Port";
+    k->revision  = 0;
+    device_class_set_props(dc, gen_rp_props);
+    k->config_write = cxl_rp_write_config;
+
+    device_class_set_parent_realize(dc, cxl_rp_realize, &rpc->parent_realize);
+    device_class_set_parent_reset(dc, cxl_rp_reset, &rpc->parent_reset);
+
+    rpc->aer_offset = GEN_PCIE_ROOT_PORT_AER_OFFSET;
+    rpc->acs_offset = GEN_PCIE_ROOT_PORT_ACS_OFFSET;
+
+    /*
+     * Explain
+     */
+    dc->hotpluggable = false;
+}
+
+static const TypeInfo cxl_root_port_info = {
+    .name = TYPE_CXL_ROOT_PORT,
+    .parent = TYPE_PCIE_ROOT_PORT,
+    .instance_size = sizeof(CXLRootPort),
+    .class_init = cxl_root_port_class_init,
+    .interfaces = (InterfaceInfo[]) {
+        { INTERFACE_CXL_DEVICE },
+        { }
+    },
+};
+
+static void cxl_register(void)
+{
+    type_register_static(&cxl_root_port_info);
+}
+
+type_init(cxl_register);
diff --git a/hw/pci-bridge/meson.build b/hw/pci-bridge/meson.build
index daab8acf2a..b6d26a03d5 100644
--- a/hw/pci-bridge/meson.build
+++ b/hw/pci-bridge/meson.build
@@ -5,6 +5,7 @@  pci_ss.add(when: 'CONFIG_IOH3420', if_true: files('ioh3420.c'))
 pci_ss.add(when: 'CONFIG_PCIE_PORT', if_true: files('pcie_root_port.c', 'gen_pcie_root_port.c', 'pcie_pci_bridge.c'))
 pci_ss.add(when: 'CONFIG_PXB', if_true: files('pci_expander_bridge.c'))
 pci_ss.add(when: 'CONFIG_XIO3130', if_true: files('xio3130_upstream.c', 'xio3130_downstream.c'))
+pci_ss.add(when: 'CONFIG_CXL', if_true: files('cxl_root_port.c'))
 
 # NewWorld PowerMac
 pci_ss.add(when: 'CONFIG_DEC_PCI', if_true: files('dec.c'))
diff --git a/hw/pci-bridge/pcie_root_port.c b/hw/pci-bridge/pcie_root_port.c
index f1cfe9d14a..460e48269d 100644
--- a/hw/pci-bridge/pcie_root_port.c
+++ b/hw/pci-bridge/pcie_root_port.c
@@ -67,7 +67,11 @@  static void rp_realize(PCIDevice *d, Error **errp)
     int rc;
 
     pci_config_set_interrupt_pin(d->config, 1);
-    pci_bridge_initfn(d, TYPE_PCIE_BUS);
+    if (d->cap_present & QEMU_PCIE_CAP_CXL) {
+        pci_bridge_initfn(d, TYPE_CXL_BUS);
+    } else {
+        pci_bridge_initfn(d, TYPE_PCIE_BUS);
+    }
     pcie_port_init_reg(d);
 
     rc = pci_bridge_ssvid_init(d, rpc->ssvid_offset, dc->vendor_id,
diff --git a/hw/pci/pci.c b/hw/pci/pci.c
index cafebf6f59..cc4f06937d 100644
--- a/hw/pci/pci.c
+++ b/hw/pci/pci.c
@@ -2708,7 +2708,9 @@  static void pci_device_class_base_init(ObjectClass *klass, void *data)
             object_class_dynamic_cast(klass, INTERFACE_CONVENTIONAL_PCI_DEVICE);
         ObjectClass *pcie =
             object_class_dynamic_cast(klass, INTERFACE_PCIE_DEVICE);
-        assert(conventional || pcie);
+        ObjectClass *cxl =
+            object_class_dynamic_cast(klass, INTERFACE_CXL_DEVICE);
+        assert(conventional || pcie || cxl);
     }
 }