[Qemu-commits] [qemu/qemu] d38d20: i386/kvm: support guest access CORE cstate

[Peter Maydell]

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: d38d201f0ed092b8c7f7738a5db8428e12cb04b6
      
https://github.com/qemu/qemu/commit/d38d201f0ed092b8c7f7738a5db8428e12cb04b6
  Author: Wanpeng Li <address@hidden>
  Date:   2019-09-16 (Mon, 16 Sep 2019)

  Changed paths:
    M linux-headers/linux/kvm.h
    M target/i386/kvm.c

  Log Message:
  -----------
  i386/kvm: support guest access CORE cstate

Allow guest reads CORE cstate when exposing host CPU power management 
capabilities
to the guest. PKG cstate is restricted to avoid a guest to get the whole package
information in multi-tenant scenario.

Cc: Eduardo Habkost <address@hidden>
Cc: Paolo Bonzini <address@hidden>
Cc: Radim Krčmář <address@hidden>
Signed-off-by: Wanpeng Li <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 56b15076805a29673c1a90ea9c3ebef25bfcc912
      
https://github.com/qemu/qemu/commit/56b15076805a29673c1a90ea9c3ebef25bfcc912
  Author: Wei Yang <address@hidden>
  Date:   2019-09-16 (Mon, 16 Sep 2019)

  Changed paths:
    M exec.c

  Log Message:
  -----------
  exec.c: replace hwaddr with uint64_t for better understanding

Function phys_page_set() and phys_page_set_level() 's argument *nb*
stands for number of pages to set instead of hardware address.

This would be more proper to use uint64_t instead of hwaddr for its
type.

Signed-off-by: Wei Yang <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: c95cfd040078db8017f74fd3a4d6f798385d960c
      
https://github.com/qemu/qemu/commit/c95cfd040078db8017f74fd3a4d6f798385d960c
  Author: Wei Yang <address@hidden>
  Date:   2019-09-16 (Mon, 16 Sep 2019)

  Changed paths:
    M exec.c

  Log Message:
  -----------
  exec.c: get nodes_nb_alloc with one MAX calculation

The purpose of these two MAX here is to get the maximum of these three
variables:

    A: map->nodes_nb + nodes
    B: map->nodes_nb_alloc
    C: alloc_hint

We can write it like MAX(A, B, C). Since the if condition says A > B,
this means MAX(A, B, C) = MAX(A, C).

This patch just simplify the calculation a bit.

Signed-off-by: Wei Yang <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: b797ab1a15ba8d2b2fc4ec3e1f24d755f6855d05
      
https://github.com/qemu/qemu/commit/b797ab1a15ba8d2b2fc4ec3e1f24d755f6855d05
  Author: Wei Yang <address@hidden>
  Date:   2019-09-16 (Mon, 16 Sep 2019)

  Changed paths:
    M exec.c

  Log Message:
  -----------
  exec.c: subpage->sub_section is already initialized to 0

In subpage_init(), we will set subpage->sub_section to
PHYS_SECTION_UNASSIGNED by subpage_register. Since
PHYS_SECTION_UNASSIGNED is defined to be 0, and we allocate subpage with
g_malloc0, this means subpage->sub_section is already initialized to 0.

This patch removes the redundant setup for a new subpage and also fix
the code style.

Signed-off-by: Wei Yang <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 26ca2075babd7775e246b9eb7da75d6de77eb658
      
https://github.com/qemu/qemu/commit/26ca2075babd7775e246b9eb7da75d6de77eb658
  Author: Wei Yang <address@hidden>
  Date:   2019-09-16 (Mon, 16 Sep 2019)

  Changed paths:
    M exec.c

  Log Message:
  -----------
  exec.c: correct the maximum skip value during compact

skip is defined with 6 bits. So the maximum value should be (1 << 6).

Signed-off-by: Wei Yang <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 526ca2360ea1cd947f74c8c6c38b91b9d6fcfdb5
      
https://github.com/qemu/qemu/commit/526ca2360ea1cd947f74c8c6c38b91b9d6fcfdb5
  Author: Wei Yang <address@hidden>
  Date:   2019-09-16 (Mon, 16 Sep 2019)

  Changed paths:
    M exec.c

  Log Message:
  -----------
  exec.c: add a check between constants to see whether we could skip

The maximum level is defined as P_L2_LEVELS and skip is defined with 6
bits, which means if P_L2_LEVELS < (1 << 6), skip never exceeds the
boundary.

Since this check is between two constants, which leverages compiler
to optimize the code based on different configuration.

Signed-off-by: Wei Yang <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: f64f598ad96977d2e511be0355d4ae48ecb4f694
      
https://github.com/qemu/qemu/commit/f64f598ad96977d2e511be0355d4ae48ecb4f694
  Author: Paolo Bonzini <address@hidden>
  Date:   2019-09-16 (Mon, 16 Sep 2019)

  Changed paths:
    M qemu.nsi

  Log Message:
  -----------
  win32: fix README file in NSIS installer

Adjust after the rST conversion and consequent renaming.

Fixes: 336a7451e8803c21a2da6e7d1eca8cfb8e8b219a
Reviewed-by: Daniel P. Berrangé <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
Tested-by: Philippe Mathieu-Daudé <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 0c9956f8213ad1074474f2f938574043e4ee5dfd
      
https://github.com/qemu/qemu/commit/0c9956f8213ad1074474f2f938574043e4ee5dfd
  Author: Paolo Bonzini <address@hidden>
  Date:   2019-09-16 (Mon, 16 Sep 2019)

  Changed paths:
    M tests/test-char.c

  Log Message:
  -----------
  test-char: fix AddressSanitizer failure

The CharSocketServerTestConfig and CharSocketClientTestConfig
objects escape after they are passed to g_test_add_data_func,
but they cease existing after the scope that defines them is
closed.  Make them static to fix this issue.

Fixes: e7b6ba4186f243f149b0d8cddc129fe681ba3912
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 709ebb905491258676776c78ca38a61563e59aac
      
https://github.com/qemu/qemu/commit/709ebb905491258676776c78ca38a61563e59aac
  Author: Cole Robinson <address@hidden>
  Date:   2019-09-16 (Mon, 16 Sep 2019)

  Changed paths:
    M default-configs/i386-softmmu.mak
    M hw/i386/Kconfig

  Log Message:
  -----------
  hw/i386: Move CONFIG_ACPI_PCI to CONFIG_PC

CONFIG_ACPI_PCI is a hard requirement of acpi-build.c, which is built
unconditionally for x86 target. Putting it in default-configs/ suggests
that it can be easily disabled, which isn't true.

Relocate the symbol with the other acpi-build.c requirements, under
'config PC'. This is similar to what is done for the arm 'virt' machine
type and CONFIG_ACPI_PCI

Signed-off-by: Cole Robinson <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 41a2635124117d1fac7e45b3cafa2c1ac68417fd
      
https://github.com/qemu/qemu/commit/41a2635124117d1fac7e45b3cafa2c1ac68417fd
  Author: Stefano Garzarella <address@hidden>
  Date:   2019-09-16 (Mon, 16 Sep 2019)

  Changed paths:
    M hw/core/loader.c
    M include/hw/elf_ops.h
    M include/hw/loader.h

  Log Message:
  -----------
  elf-ops.h: fix int overflow in load_elf()

This patch fixes a possible integer overflow when we calculate
the total size of ELF segments loaded.

Reported-by: Coverity (CID 1405299)
Reviewed-by: Alex Bennée <address@hidden>
Signed-off-by: Stefano Garzarella <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 72d41eb4b8f923de91e8f06dc20aa86b0a9155fb
      
https://github.com/qemu/qemu/commit/72d41eb4b8f923de91e8f06dc20aa86b0a9155fb
  Author: Stefan Hajnoczi <address@hidden>
  Date:   2019-09-16 (Mon, 16 Sep 2019)

  Changed paths:
    M backends/hostmem-file.c
    M exec.c
    M include/qemu/osdep.h
    M util/oslib-posix.c
    M util/oslib-win32.c

  Log Message:
  -----------
  memory: fetch pmem size in get_file_size()

Neither stat(2) nor lseek(2) report the size of Linux devdax pmem
character device nodes.  Commit 314aec4a6e06844937f1677f6cba21981005f389
("hostmem-file: reject invalid pmem file sizes") added code to
hostmem-file.c to fetch the size from sysfs and compare against the
user-provided size=NUM parameter:

  if (backend->size > size) {
      error_setg(errp, "size property %" PRIu64 " is larger than "
                 "pmem file \"%s\" size %" PRIu64, backend->size,
                 fb->mem_path, size);
      return;
  }

It turns out that exec.c:qemu_ram_alloc_from_fd() already has an
equivalent size check but it skips devdax pmem character devices because
lseek(2) returns 0:

  if (file_size > 0 && file_size < size) {
      error_setg(errp, "backing store %s size 0x%" PRIx64
                 " does not match 'size' option 0x" RAM_ADDR_FMT,
                 mem_path, file_size, size);
      return NULL;
  }

This patch moves the devdax pmem file size code into get_file_size() so
that we check the memory size in a single place:
qemu_ram_alloc_from_fd().  This simplifies the code and makes it more
general.

This also fixes the problem that hostmem-file only checks the devdax
pmem file size when the pmem=on parameter is given.  An unchecked
size=NUM parameter can lead to SIGBUS in QEMU so we must always fetch
the file size for Linux devdax pmem character device nodes.

Signed-off-by: Stefan Hajnoczi <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: Eduardo Habkost <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 7a3df11c2a647cf889f6ede8b7d5f81438bb5cc9
      
https://github.com/qemu/qemu/commit/7a3df11c2a647cf889f6ede8b7d5f81438bb5cc9
  Author: Paolo Bonzini <address@hidden>
  Date:   2019-09-16 (Mon, 16 Sep 2019)

  Changed paths:
    M include/exec/memory.h
    M memory.c

  Log Message:
  -----------
  memory: inline and optimize devend_memop

devend_memop can rely on the fact that the result is always either
0 or MO_BSWAP, corresponding respectively to host endianness and
the opposite.  Native (target) endianness in turn can be either
the host endianness, in which case MO_BSWAP is only returned for
host-opposite endianness, or the opposite, in which case 0 is only
returned for host endianness.

With this in mind, devend_memop can be compiled as a setcond+shift
for every target.  Do this and, while at it, move it to
include/exec/memory.h since !NEED_CPU_H files do not (and should not)
need it.

Reviewed-by: Richard Henderson <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 3dcc9c6ec4ea60fd1464b35aa82199483f24ba75
      
https://github.com/qemu/qemu/commit/3dcc9c6ec4ea60fd1464b35aa82199483f24ba75
  Author: Yury Kotov <address@hidden>
  Date:   2019-09-16 (Mon, 16 Sep 2019)

  Changed paths:
    M include/qemu/thread.h
    M util/qemu-thread-posix.c
    M util/qemu-thread-win32.c
    M util/qsp.c

  Log Message:
  -----------
  qemu-thread: Add qemu_cond_timedwait

The new function is needed to implement conditional sleep for CPU
throttling. It's possible to reuse qemu_sem_timedwait, but it's more
difficult than just add qemu_cond_timedwait.

Also moved compute_abs_deadline function up the code to reuse it in
qemu_cond_timedwait_impl win32.

Signed-off-by: Yury Kotov <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: bd1f7ff4b27444740786274c41caf3e9b9c35bba
      
https://github.com/qemu/qemu/commit/bd1f7ff4b27444740786274c41caf3e9b9c35bba
  Author: Yury Kotov <address@hidden>
  Date:   2019-09-16 (Mon, 16 Sep 2019)

  Changed paths:
    M cpus.c

  Log Message:
  -----------
  cpus: Fix throttling during vm_stop

Throttling thread sleeps in VCPU thread. For high throttle percentage
this sleep is more than 10ms. E.g. for 60% - 15ms, for 99% - 990ms.
vm_stop() kicks all VCPUs and waits for them. It's called at the end of
migration and because of the long sleep the migration downtime might be
more than 100ms even for downtime-limit 1ms.
Use qemu_cond_timedwait for high percentage to wake up during vm_stop.

Signed-off-by: Yury Kotov <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 42d400acfc2dabc56653ee5396df14a8d8cb46c7
      
https://github.com/qemu/qemu/commit/42d400acfc2dabc56653ee5396df14a8d8cb46c7
  Author: Philippe Mathieu-Daudé <address@hidden>
  Date:   2019-09-16 (Mon, 16 Sep 2019)

  Changed paths:
    M hw/i386/pc.c

  Log Message:
  -----------
  hw/i386/pc: Use e820_get_num_entries() to access e820_entries

To be able to extract the e820* code out of this file (in the next
patch), access e820_entries with its correct helper.

Reviewed-by: Li Qiang <address@hidden>
Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: d6d059ca07ae907b8945f88c382fb54d43f9f03a
      
https://github.com/qemu/qemu/commit/d6d059ca07ae907b8945f88c382fb54d43f9f03a
  Author: Philippe Mathieu-Daudé <address@hidden>
  Date:   2019-09-16 (Mon, 16 Sep 2019)

  Changed paths:
    M hw/i386/Makefile.objs
    A hw/i386/e820_memory_layout.c
    A hw/i386/e820_memory_layout.h
    M hw/i386/pc.c
    M include/hw/i386/pc.h
    M target/i386/kvm.c

  Log Message:
  -----------
  hw/i386/pc: Extract e820 memory layout code

Suggested-by: Samuel Ortiz <address@hidden>
Reviewed-by: Li Qiang <address@hidden>
Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 0f432b3064a227bfce34bede1ba40eb7f0cb3262
      
https://github.com/qemu/qemu/commit/0f432b3064a227bfce34bede1ba40eb7f0cb3262
  Author: Philippe Mathieu-Daudé <address@hidden>
  Date:   2019-09-16 (Mon, 16 Sep 2019)

  Changed paths:
    M hw/i386/pc.c

  Log Message:
  -----------
  hw/i386/pc: Use address_space_memory in place

The address_space_memory variable is used once.
Use it in place and remove the argument.

Suggested-by: Samuel Ortiz <address@hidden>
Reviewed-by: Li Qiang <address@hidden>
Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 5888e01cfdf5044d6f43bb5c6be9e3d80bcb6ea7
      
https://github.com/qemu/qemu/commit/5888e01cfdf5044d6f43bb5c6be9e3d80bcb6ea7
  Author: Philippe Mathieu-Daudé <address@hidden>
  Date:   2019-09-16 (Mon, 16 Sep 2019)

  Changed paths:
    M hw/i386/pc.c

  Log Message:
  -----------
  hw/i386/pc: Rename bochs_bios_init as more generic fw_cfg_arch_create

The bochs_bios_init() function is not restricted to the Bochs
BIOS and is useful to other BIOS.
Since it is not specific to the PC machine, and can be reused
by other machines of the X86 architecture, rename it as
fw_cfg_arch_create().

Suggested-by: Samuel Ortiz <address@hidden>
Reviewed-by: Li Qiang <address@hidden>
Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 524acbe97dcaf7e397d75e2d6eee3b0aaabd0b38
      
https://github.com/qemu/qemu/commit/524acbe97dcaf7e397d75e2d6eee3b0aaabd0b38
  Author: Philippe Mathieu-Daudé <address@hidden>
  Date:   2019-09-16 (Mon, 16 Sep 2019)

  Changed paths:
    M hw/i386/pc.c

  Log Message:
  -----------
  hw/i386/pc: Pass the boot_cpus value by argument

The boot_cpus is used once. Pass it by argument, this will
allow us to remove the PCMachineState argument later.

Suggested-by: Samuel Ortiz <address@hidden>
Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 57e23e78372d8b8426b650cde0ab92adc0fe1865
      
https://github.com/qemu/qemu/commit/57e23e78372d8b8426b650cde0ab92adc0fe1865
  Author: Philippe Mathieu-Daudé <address@hidden>
  Date:   2019-09-16 (Mon, 16 Sep 2019)

  Changed paths:
    M hw/i386/pc.c

  Log Message:
  -----------
  hw/i386/pc: Pass the apic_id_limit value by argument

Pass the apic_id_limit value by argument, this will
allow us to remove the PCMachineState argument later.

Suggested-by: Samuel Ortiz <address@hidden>
Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 264b485776062bdc858ea5e6994a61bb68a3eec0
      
https://github.com/qemu/qemu/commit/264b485776062bdc858ea5e6994a61bb68a3eec0
  Author: Philippe Mathieu-Daudé <address@hidden>
  Date:   2019-09-16 (Mon, 16 Sep 2019)

  Changed paths:
    M hw/i386/pc.c

  Log Message:
  -----------
  hw/i386/pc: Pass the CPUArchIdList array by argument

Pass the CPUArchIdList array by argument, this will
allow us to remove the PCMachineState argument later.

Suggested-by: Samuel Ortiz <address@hidden>
Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: bd802bd98148e7022491fedbf6583e57f8ef1a78
      
https://github.com/qemu/qemu/commit/bd802bd98148e7022491fedbf6583e57f8ef1a78
  Author: Paolo Bonzini <address@hidden>
  Date:   2019-09-16 (Mon, 16 Sep 2019)

  Changed paths:
    M hw/i386/pc.c

  Log Message:
  -----------
  hw/i386/pc: Replace PCMachineState argument with MachineState in 
fw_cfg_arch_create

In the previous commit we removed the last access to PCMachineState.
Replace it with a generic MachineState argument and use it to retrieve
the CPUArchIdList.

Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 14fe28075772b882b7be4ec27997b3375ba5dfb6
      
https://github.com/qemu/qemu/commit/14fe28075772b882b7be4ec27997b3375ba5dfb6
  Author: Philippe Mathieu-Daudé <address@hidden>
  Date:   2019-09-16 (Mon, 16 Sep 2019)

  Changed paths:
    M hw/i386/pc.c

  Log Message:
  -----------
  hw/i386/pc: Let pc_build_smbios() take a FWCfgState argument

Pass the FWCfgState object by argument, this will
allow us to remove the PCMachineState argument later.

Suggested-by: Samuel Ortiz <address@hidden>
Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: f16f6f05a0c5a59c27a9427b38f615b39bc89f8a
      
https://github.com/qemu/qemu/commit/f16f6f05a0c5a59c27a9427b38f615b39bc89f8a
  Author: Philippe Mathieu-Daudé <address@hidden>
  Date:   2019-09-16 (Mon, 16 Sep 2019)

  Changed paths:
    M hw/i386/pc.c

  Log Message:
  -----------
  hw/i386/pc: Let pc_build_smbios() take a generic MachineState argument

Let the pc_build_smbios() function take a generic MachineState
argument.

Suggested-by: Samuel Ortiz <address@hidden>
Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: e0e949d7d86b4498cf7e1c8dad15d0c055c60697
      
https://github.com/qemu/qemu/commit/e0e949d7d86b4498cf7e1c8dad15d0c055c60697
  Author: Philippe Mathieu-Daudé <address@hidden>
  Date:   2019-09-16 (Mon, 16 Sep 2019)

  Changed paths:
    M hw/i386/pc.c

  Log Message:
  -----------
  hw/i386/pc: Rename pc_build_smbios() as generic fw_cfg_build_smbios()

Now that the pc_build_smbios() function has been refactored to not
depend of PC specific types, rename it to a more generic name.

Suggested-by: Samuel Ortiz <address@hidden>
Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: eb6449fbd83278ae1376f1c6343b858af479cedc
      
https://github.com/qemu/qemu/commit/eb6449fbd83278ae1376f1c6343b858af479cedc
  Author: Philippe Mathieu-Daudé <address@hidden>
  Date:   2019-09-16 (Mon, 16 Sep 2019)

  Changed paths:
    M hw/i386/pc.c

  Log Message:
  -----------
  hw/i386/pc: Let pc_build_feature_control() take a FWCfgState argument

Pass the FWCfgState object by argument, this will
allow us to remove the PCMachineState argument later.

Suggested-by: Samuel Ortiz <address@hidden>
Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 4e91c7f32e11ac532501d9fbd7c5de5d4a2e3e6b
      
https://github.com/qemu/qemu/commit/4e91c7f32e11ac532501d9fbd7c5de5d4a2e3e6b
  Author: Philippe Mathieu-Daudé <address@hidden>
  Date:   2019-09-16 (Mon, 16 Sep 2019)

  Changed paths:
    M hw/i386/pc.c

  Log Message:
  -----------
  hw/i386/pc: Let pc_build_feature_control() take a MachineState argument

Let the pc_build_feature_control_file() function take a generic MachineState
argument.

Suggested-by: Samuel Ortiz <address@hidden>
Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 367607f9132e97614e7da0e6a45576c51e9db68c
      
https://github.com/qemu/qemu/commit/367607f9132e97614e7da0e6a45576c51e9db68c
  Author: Philippe Mathieu-Daudé <address@hidden>
  Date:   2019-09-16 (Mon, 16 Sep 2019)

  Changed paths:
    M hw/i386/pc.c

  Log Message:
  -----------
  hw/i386/pc: Rename pc_build_feature_control() as generic fw_cfg_build_*

Now that the pc_build_feature_control_file() function has been
refactored to not depend of PC specific types, rename it to a
more generic name.

Suggested-by: Samuel Ortiz <address@hidden>
Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 149c50cabcc4ea46a460d35fc876346ed441304c
      
https://github.com/qemu/qemu/commit/149c50cabcc4ea46a460d35fc876346ed441304c
  Author: Philippe Mathieu-Daudé <address@hidden>
  Date:   2019-09-16 (Mon, 16 Sep 2019)

  Changed paths:
    M hw/i386/fw_cfg.c
    M hw/i386/fw_cfg.h
    M hw/i386/pc.c

  Log Message:
  -----------
  hw/i386/pc: Extract the x86 generic fw_cfg code

Extract all the functions that are not PC-machine specific into
the (arch-specific) fw_cfg.c file. This will allow other X86-machine
to reuse these functions.

Suggested-by: Samuel Ortiz <address@hidden>
Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 186c0ab9b98d90642f37a629b1ff520cbfd06394
      
https://github.com/qemu/qemu/commit/186c0ab9b98d90642f37a629b1ff520cbfd06394
  Author: Peter Maydell <address@hidden>
  Date:   2019-09-17 (Tue, 17 Sep 2019)

  Changed paths:
    M backends/hostmem-file.c
    M cpus.c
    M default-configs/i386-softmmu.mak
    M exec.c
    M hw/core/loader.c
    M hw/i386/Kconfig
    M hw/i386/Makefile.objs
    A hw/i386/e820_memory_layout.c
    A hw/i386/e820_memory_layout.h
    M hw/i386/fw_cfg.c
    M hw/i386/fw_cfg.h
    M hw/i386/pc.c
    M include/exec/memory.h
    M include/hw/elf_ops.h
    M include/hw/i386/pc.h
    M include/hw/loader.h
    M include/qemu/osdep.h
    M include/qemu/thread.h
    M linux-headers/linux/kvm.h
    M memory.c
    M qemu.nsi
    M target/i386/kvm.c
    M tests/test-char.c
    M util/oslib-posix.c
    M util/oslib-win32.c
    M util/qemu-thread-posix.c
    M util/qemu-thread-win32.c
    M util/qsp.c

  Log Message:
  -----------
  Merge remote-tracking branch 'remotes/bonzini/tags/for-upstream' into staging

* Fix Patchew CI failures (myself)
* i386 fw_cfg refactoring (Philippe)
* pmem bugfix (Stefan)
* Support for accessing cstate MSRs (Wanpeng)
* exec.c cleanups (Wei Yang)
* Improved throttling (Yury)
* elf-ops.h coverity fix (Stefano)

# gpg: Signature made Mon 16 Sep 2019 16:13:12 BST
# gpg:                using RSA key BFFBD25F78C7AE83
# gpg: Good signature from "Paolo Bonzini <address@hidden>" [full]
# gpg:                 aka "Paolo Bonzini <address@hidden>" [full]
# Primary key fingerprint: 46F5 9FBD 57D6 12E7 BFD4  E2F7 7E15 100C CD36 69B1
#      Subkey fingerprint: F133 3857 4B66 2389 866C  7682 BFFB D25F 78C7 AE83

* remotes/bonzini/tags/for-upstream: (29 commits)
  hw/i386/pc: Extract the x86 generic fw_cfg code
  hw/i386/pc: Rename pc_build_feature_control() as generic fw_cfg_build_*
  hw/i386/pc: Let pc_build_feature_control() take a MachineState argument
  hw/i386/pc: Let pc_build_feature_control() take a FWCfgState argument
  hw/i386/pc: Rename pc_build_smbios() as generic fw_cfg_build_smbios()
  hw/i386/pc: Let pc_build_smbios() take a generic MachineState argument
  hw/i386/pc: Let pc_build_smbios() take a FWCfgState argument
  hw/i386/pc: Replace PCMachineState argument with MachineState in 
fw_cfg_arch_create
  hw/i386/pc: Pass the CPUArchIdList array by argument
  hw/i386/pc: Pass the apic_id_limit value by argument
  hw/i386/pc: Pass the boot_cpus value by argument
  hw/i386/pc: Rename bochs_bios_init as more generic fw_cfg_arch_create
  hw/i386/pc: Use address_space_memory in place
  hw/i386/pc: Extract e820 memory layout code
  hw/i386/pc: Use e820_get_num_entries() to access e820_entries
  cpus: Fix throttling during vm_stop
  qemu-thread: Add qemu_cond_timedwait
  memory: inline and optimize devend_memop
  memory: fetch pmem size in get_file_size()
  elf-ops.h: fix int overflow in load_elf()
  ...

Signed-off-by: Peter Maydell <address@hidden>


Compare: https://github.com/qemu/qemu/compare/521db80318d6...186c0ab9b98d