Commit e02bbe19 authored by Peter Maydell's avatar Peter Maydell
Browse files

Merge remote-tracking branch 'remotes/dgibson/tags/ppc-for-2.10-20170606' into staging 



ppc patch queue 2017-06-06

Accumulated patches for ppc targets and the pseries machine type.

The big thing in this batch is a start on a substantial cleanup of the
pseries hotplug mechanisms, which were pretty confusing.  For now
these shouldn't cause substantial behavioural changes, but I am hoping
these lead to clearer code and eventually to fixes for the bugs we
have in hotplug handling, particularly when hotplug and migration are
combined.

The remaining patches are mostly bugfixes.

# gpg: Signature made Tue 06 Jun 2017 03:48:50 BST
# gpg:                using RSA key 0x6C38CACA20D9B392
# gpg: Good signature from "David Gibson <david@gibson.dropbear.id.au>"
# gpg:                 aka "David Gibson (Red Hat) <dgibson@redhat.com>"
# gpg:                 aka "David Gibson (ozlabs.org) <dgibson@ozlabs.org>"
# gpg:                 aka "David Gibson (kernel.org) <dwg@kernel.org>"
# Primary key fingerprint: 75F4 6586 AE61 A66C C44E  87DC 6C38 CACA 20D9 B392

* remotes/dgibson/tags/ppc-for-2.10-20170606:
  spapr: Remove some non-useful properties on DRC objects
  spapr: Eliminate spapr_drc_get_type_str()
  spapr: Move configure-connector state into DRC
  spapr: Clean up spapr_dr_connector_by_*()
  spapr: Introduce DRC subclasses
  spapr/drc: don't migrate DRC of cold-plugged CPUs and LMBs
  spapr: Allow boot from vhost-*-scsi backends
  ppc/pnv: check the return value of fdt_setprop()
  spapr_nvram: Check return value from blk_getlength()
  target/ppc: Fixup set_spr error in h_register_process_table
  target-ppc: Fix openpic timer read register offset
  spapr: Make DRC get_index and get_type methods into plain functions
  spapr: Abolish DRC set_configured method
  spapr: Abolish DRC get_fdt method
  spapr: Move DRC RTAS calls into spapr_drc.c
  migration: Mark CPU states dirty before incoming migration/loadvm
  migration: remove register_savevm()

Signed-off-by: default avatarPeter Maydell <peter.maydell@linaro.org>
parents a65afaae 91dcb1ff

cpus.c

+9 −0
Original line number Diff line number Diff line
@@ -921,6 +921,15 @@ void cpu_synchronize_all_post_init(void) 
    }

}



void cpu_synchronize_all_pre_loadvm(void)

{

    CPUState *cpu;



    CPU_FOREACH(cpu) {

        cpu_synchronize_pre_loadvm(cpu);

    }

}



static int do_vm_stop(RunState state)

{

    int ret = 0;

hw/intc/openpic.c

+10 −12
Original line number Diff line number Diff line
@@ -801,22 +801,19 @@ static void openpic_tmr_write(void *opaque, hwaddr addr, uint64_t val, 
    OpenPICState *opp = opaque;

    int idx;



    addr += 0x10f0;



    DPRINTF("%s: addr %#" HWADDR_PRIx " <= %08" PRIx64 "\n",

            __func__, addr, val);

            __func__, (addr + 0x10f0), val);

    if (addr & 0xF) {

        return;

    }



    if (addr == 0x10f0) {

    if (addr == 0) {

        /* TFRR */

        opp->tfrr = val;

        return;

    }



    addr -= 0x10;  /* correct for TFRR */

    idx = (addr >> 6) & 0x3;

    addr = addr & 0x30;



    switch (addr & 0x30) {

    case 0x00: /* TCCR */
@@ -844,16 +841,17 @@ static uint64_t openpic_tmr_read(void *opaque, hwaddr addr, unsigned len) 
    uint32_t retval = -1;

    int idx;



    DPRINTF("%s: addr %#" HWADDR_PRIx "\n", __func__, addr);

    DPRINTF("%s: addr %#" HWADDR_PRIx "\n", __func__, addr + 0x10f0);

    if (addr & 0xF) {

        goto out;

    }

    idx = (addr >> 6) & 0x3;

    if (addr == 0x0) {

    if (addr == 0) {

        /* TFRR */

        retval = opp->tfrr;

        goto out;

    }

    addr -= 0x10;  /* correct for TFRR */

    idx = (addr >> 6) & 0x3;

    switch (addr & 0x30) {

    case 0x00: /* TCCR */

        retval = opp->timers[idx].tccr;
@@ -861,10 +859,10 @@ static uint64_t openpic_tmr_read(void *opaque, hwaddr addr, unsigned len) 
    case 0x10: /* TBCR */

        retval = opp->timers[idx].tbcr;

        break;

    case 0x20: /* TIPV */

    case 0x20: /* TVPR */

        retval = read_IRQreg_ivpr(opp, opp->irq_tim0 + idx);

        break;

    case 0x30: /* TIDE (TIDR) */

    case 0x30: /* TDR */

        retval = read_IRQreg_idr(opp, opp->irq_tim0 + idx);

        break;

    }

hw/net/vmxnet3.c

+6 −2
Original line number Diff line number Diff line
@@ -2262,6 +2262,11 @@ static const MemoryRegionOps b1_ops = { 
    },

};



static SaveVMHandlers savevm_vmxnet3_msix = {

    .save_state = vmxnet3_msix_save,

    .load_state = vmxnet3_msix_load,

};



static uint64_t vmxnet3_device_serial_num(VMXNET3State *s)

{

    uint64_t dsn_payload;
@@ -2331,8 +2336,7 @@ static void vmxnet3_pci_realize(PCIDevice *pci_dev, Error **errp) 
                              vmxnet3_device_serial_num(s));

    }



    register_savevm(dev, "vmxnet3-msix", -1, 1,

                    vmxnet3_msix_save, vmxnet3_msix_load, s);

    register_savevm_live(dev, "vmxnet3-msix", -1, 1, &savevm_vmxnet3_msix, s);

}



static void vmxnet3_instance_init(Object *obj)

hw/nvram/spapr_nvram.c

+9 −1
Original line number Diff line number Diff line
@@ -144,7 +144,15 @@ static void spapr_nvram_realize(VIOsPAPRDevice *dev, Error **errp) 
    int ret;



    if (nvram->blk) {

        nvram->size = blk_getlength(nvram->blk);

        int64_t len = blk_getlength(nvram->blk);



        if (len < 0) {

            error_setg_errno(errp, -len,

                             "could not get length of backing image");

            return;

        }



        nvram->size = len;



        ret = blk_set_perm(nvram->blk,

                           BLK_PERM_CONSISTENT_READ | BLK_PERM_WRITE,

hw/ppc/pnv.c

+3 −2
Original line number Diff line number Diff line
@@ -378,8 +378,9 @@ static void powernv_populate_ipmi_bt(ISADevice *d, void *fdt, int lpc_off) 
    _FDT(node);

    g_free(name);



    fdt_setprop(fdt, node, "reg", io_regs, sizeof(io_regs));

    fdt_setprop(fdt, node, "compatible", compatible, sizeof(compatible));

    _FDT((fdt_setprop(fdt, node, "reg", io_regs, sizeof(io_regs))));

    _FDT((fdt_setprop(fdt, node, "compatible", compatible,

                      sizeof(compatible))));



    /* Mark it as reserved to avoid Linux trying to claim it */

    _FDT((fdt_setprop_string(fdt, node, "status", "reserved")));