virtio, pc: fixes, features
Bugfixes all over the place. CPU hotplug with secureboot. Signed-off-by: Michael S. Tsirkin <mst@redhat.com> -----BEGIN PGP SIGNATURE----- iQFDBAABCAAtFiEEXQn9CHHI+FuUyooNKB8NuNKNVGkFAl4pRnAPHG1zdEByZWRo YXQuY29tAAoJECgfDbjSjVRp+r8H/39Cy1MNMtmGZHKA9BCqJMtasK+x8OhW0R/I 8u9Ye90apNuHRW47r5srqp8mDZ1dlVG171otMRqGFAVbwsDFxLvZxFgrmJYAO9v7 aJHBwJBxzFCLKCYXNhVO0QGJgprWUuKg8QD3ftqtSaw08Ay8CfjsTB8W5bawV63g ofJSTZ4ZvPepD92F6RQLGMJMKmrFfhYDL7aR0aWuDVbpJaFHR8ZtlpnFMR2npPrQ MR0HIMC9M5mFE5WSG7SjtNXGi/3lp2VEGGzy+iQ1Cxo91FJ1fP4iTuLXy4OJIZJi i17y4GDeJSNotXONCQoJtXEqnAuFwTjv/jgSolILCQ6q8A9uvQI= =uDiq -----END PGP SIGNATURE----- Merge remote-tracking branch 'remotes/mst/tags/for_upstream' into staging virtio, pc: fixes, features Bugfixes all over the place. CPU hotplug with secureboot. Signed-off-by: Michael S. Tsirkin <mst@redhat.com> # gpg: Signature made Thu 23 Jan 2020 07:08:32 GMT # gpg: using RSA key 5D09FD0871C8F85B94CA8A0D281F0DB8D28D5469 # gpg: issuer "mst@redhat.com" # gpg: Good signature from "Michael S. Tsirkin <mst@kernel.org>" [full] # gpg: aka "Michael S. Tsirkin <mst@redhat.com>" [full] # Primary key fingerprint: 0270 606B 6F3C DF3D 0B17 0970 C350 3912 AFBE 8E67 # Subkey fingerprint: 5D09 FD08 71C8 F85B 94CA 8A0D 281F 0DB8 D28D 5469 * remotes/mst/tags/for_upstream: vhost: coding style fix i386:acpi: Remove _HID from the SMBus ACPI entry vhost: Only align sections for vhost-user vhost: Add names to section rounded warning vhost-vsock: delete vqs in vhost_vsock_unrealize to avoid memleaks virtio-scsi: convert to new virtio_delete_queue virtio-scsi: delete vqs in unrealize to avoid memleaks virtio-9p-device: convert to new virtio_delete_queue virtio-9p-device: fix memleak in virtio_9p_device_unrealize bios-tables-test: document expected file update acpi: cpuhp: add CPHP_GET_CPU_ID_CMD command acpi: cpuhp: spec: add typical usecases acpi: cpuhp: introduce 'Command data 2' field acpi: cpuhp: spec: clarify store into 'Command data' when 'Command field' == 0 acpi: cpuhp: spec: fix 'Command data' description acpi: cpuhp: spec: clarify 'CPU selector' register usage and endianness tests: q35: MCH: add default SMBASE SMRAM lock test q35: implement 128K SMRAM at default SMBASE address Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
This commit is contained in:
commit
7cea426c1d
|
@ -15,14 +15,14 @@ CPU present bitmap for:
|
|||
PIIX-PM (IO port 0xaf00-0xaf1f, 1-byte access)
|
||||
One bit per CPU. Bit position reflects corresponding CPU APIC ID. Read-only.
|
||||
The first DWORD in bitmap is used in write mode to switch from legacy
|
||||
to new CPU hotplug interface, write 0 into it to do switch.
|
||||
to modern CPU hotplug interface, write 0 into it to do switch.
|
||||
---------------------------------------------------------------
|
||||
QEMU sets corresponding CPU bit on hot-add event and issues SCI
|
||||
with GPE.2 event set. CPU present map is read by ACPI BIOS GPE.2 handler
|
||||
to notify OS about CPU hot-add events. CPU hot-remove isn't supported.
|
||||
|
||||
=====================================
|
||||
ACPI CPU hotplug interface registers:
|
||||
Modern ACPI CPU hotplug interface registers:
|
||||
-------------------------------------
|
||||
Register block base address:
|
||||
ICH9-LPC IO port 0x0cd8
|
||||
|
@ -30,9 +30,25 @@ Register block base address:
|
|||
Register block size:
|
||||
ACPI_CPU_HOTPLUG_REG_LEN = 12
|
||||
|
||||
All accesses to registers described below, imply little-endian byte order.
|
||||
|
||||
Reserved resisters behavior:
|
||||
- write accesses are ignored
|
||||
- read accesses return all bits set to 0.
|
||||
|
||||
The last stored value in 'CPU selector' must refer to a possible CPU, otherwise
|
||||
- reads from any register return 0
|
||||
- writes to any other register are ignored until valid value is stored into it
|
||||
On QEMU start, 'CPU selector' is initialized to a valid value, on reset it
|
||||
keeps the current value.
|
||||
|
||||
read access:
|
||||
offset:
|
||||
[0x0-0x3] reserved
|
||||
[0x0-0x3] Command data 2: (DWORD access)
|
||||
if value last stored in 'Command field':
|
||||
0: reads as 0x0
|
||||
3: upper 32 bits of architecture specific CPU ID value
|
||||
other values: reserved
|
||||
[0x4] CPU device status fields: (1 byte access)
|
||||
bits:
|
||||
0: Device is enabled and may be used by guest
|
||||
|
@ -44,15 +60,17 @@ read access:
|
|||
3-7: reserved and should be ignored by OSPM
|
||||
[0x5-0x7] reserved
|
||||
[0x8] Command data: (DWORD access)
|
||||
in case of error or unsupported command reads is 0xFFFFFFFF
|
||||
current 'Command field' value:
|
||||
0: returns PXM value corresponding to device
|
||||
contains 0 unless value last stored in 'Command field' is one of:
|
||||
0: contains 'CPU selector' value of a CPU with pending event[s]
|
||||
3: lower 32 bits of architecture specific CPU ID value
|
||||
(in x86 case: APIC ID)
|
||||
|
||||
write access:
|
||||
offset:
|
||||
[0x0-0x3] CPU selector: (DWORD access)
|
||||
selects active CPU device. All following accesses to other
|
||||
registers will read/store data from/to selected CPU.
|
||||
Valid values: [0 .. max_cpus)
|
||||
[0x4] CPU device control fields: (1 byte access)
|
||||
bits:
|
||||
0: reserved, OSPM must clear it before writing to register.
|
||||
|
@ -69,9 +87,9 @@ write access:
|
|||
value:
|
||||
0: selects a CPU device with inserting/removing events and
|
||||
following reads from 'Command data' register return
|
||||
selected CPU (CPU selector value). If no CPU with events
|
||||
found, the current CPU selector doesn't change and
|
||||
corresponding insert/remove event flags are not set.
|
||||
selected CPU ('CPU selector' value).
|
||||
If no CPU with events found, the current 'CPU selector' doesn't
|
||||
change and corresponding insert/remove event flags are not modified.
|
||||
1: following writes to 'Command data' register set OST event
|
||||
register in QEMU
|
||||
2: following writes to 'Command data' register set OST status
|
||||
|
@ -79,16 +97,53 @@ write access:
|
|||
other values: reserved
|
||||
[0x6-0x7] reserved
|
||||
[0x8] Command data: (DWORD access)
|
||||
current 'Command field' value:
|
||||
0: OSPM reads value of CPU selector
|
||||
if last stored 'Command field' value:
|
||||
1: stores value into OST event register
|
||||
2: stores value into OST status register, triggers
|
||||
ACPI_DEVICE_OST QMP event from QEMU to external applications
|
||||
with current values of OST event and status registers.
|
||||
other values: reserved
|
||||
other values: reserved
|
||||
|
||||
Selecting CPU device beyond possible range has no effect on platform:
|
||||
- write accesses to CPU hot-plug registers not documented above are
|
||||
ignored
|
||||
- read accesses to CPU hot-plug registers not documented above return
|
||||
all bits set to 0.
|
||||
Typical usecases:
|
||||
- (x86) Detecting and enabling modern CPU hotplug interface.
|
||||
QEMU starts with legacy CPU hotplug interface enabled. Detecting and
|
||||
switching to modern interface is based on the 2 legacy CPU hotplug features:
|
||||
1. Writes into CPU bitmap are ignored.
|
||||
2. CPU bitmap always has bit#0 set, corresponding to boot CPU.
|
||||
|
||||
Use following steps to detect and enable modern CPU hotplug interface:
|
||||
1. Store 0x0 to the 'CPU selector' register,
|
||||
attempting to switch to modern mode
|
||||
2. Store 0x0 to the 'CPU selector' register,
|
||||
to ensure valid selector value
|
||||
3. Store 0x0 to the 'Command field' register,
|
||||
4. Read the 'Command data 2' register.
|
||||
If read value is 0x0, the modern interface is enabled.
|
||||
Otherwise legacy or no CPU hotplug interface available
|
||||
|
||||
- Get a cpu with pending event
|
||||
1. Store 0x0 to the 'CPU selector' register.
|
||||
2. Store 0x0 to the 'Command field' register.
|
||||
3. Read the 'CPU device status fields' register.
|
||||
4. If both bit#1 and bit#2 are clear in the value read, there is no CPU
|
||||
with a pending event and selected CPU remains unchanged.
|
||||
5. Otherwise, read the 'Command data' register. The value read is the
|
||||
selector of the CPU with the pending event (which is already
|
||||
selected).
|
||||
|
||||
- Enumerate CPUs present/non present CPUs
|
||||
01. Set the present CPU count to 0.
|
||||
02. Set the iterator to 0.
|
||||
03. Store 0x0 to the 'CPU selector' register, to ensure that it's in
|
||||
a valid state and that access to other registers won't be ignored.
|
||||
04. Store 0x0 to the 'Command field' register to make 'Command data'
|
||||
register return 'CPU selector' value of selected CPU
|
||||
05. Read the 'CPU device status fields' register.
|
||||
06. If bit#0 is set, increment the present CPU count.
|
||||
07. Increment the iterator.
|
||||
08. Store the iterator to the 'CPU selector' register.
|
||||
09. Read the 'Command data' register.
|
||||
10. If the value read is not zero, goto 05.
|
||||
11. Otherwise store 0x0 to the 'CPU selector' register, to put it
|
||||
into a valid state and exit.
|
||||
The iterator at this point equals "max_cpus".
|
||||
|
|
|
@ -218,6 +218,7 @@ static void virtio_9p_device_unrealize(DeviceState *dev, Error **errp)
|
|||
V9fsVirtioState *v = VIRTIO_9P(dev);
|
||||
V9fsState *s = &v->state;
|
||||
|
||||
virtio_delete_queue(v->vq);
|
||||
virtio_cleanup(vdev);
|
||||
v9fs_device_unrealize_common(s, errp);
|
||||
}
|
||||
|
|
|
@ -12,11 +12,13 @@
|
|||
#define ACPI_CPU_FLAGS_OFFSET_RW 4
|
||||
#define ACPI_CPU_CMD_OFFSET_WR 5
|
||||
#define ACPI_CPU_CMD_DATA_OFFSET_RW 8
|
||||
#define ACPI_CPU_CMD_DATA2_OFFSET_R 0
|
||||
|
||||
enum {
|
||||
CPHP_GET_NEXT_CPU_WITH_EVENT_CMD = 0,
|
||||
CPHP_OST_EVENT_CMD = 1,
|
||||
CPHP_OST_STATUS_CMD = 2,
|
||||
CPHP_GET_CPU_ID_CMD = 3,
|
||||
CPHP_CMD_MAX
|
||||
};
|
||||
|
||||
|
@ -74,11 +76,27 @@ static uint64_t cpu_hotplug_rd(void *opaque, hwaddr addr, unsigned size)
|
|||
case CPHP_GET_NEXT_CPU_WITH_EVENT_CMD:
|
||||
val = cpu_st->selector;
|
||||
break;
|
||||
case CPHP_GET_CPU_ID_CMD:
|
||||
val = cdev->arch_id & 0xFFFFFFFF;
|
||||
break;
|
||||
default:
|
||||
break;
|
||||
}
|
||||
trace_cpuhp_acpi_read_cmd_data(cpu_st->selector, val);
|
||||
break;
|
||||
case ACPI_CPU_CMD_DATA2_OFFSET_R:
|
||||
switch (cpu_st->command) {
|
||||
case CPHP_GET_NEXT_CPU_WITH_EVENT_CMD:
|
||||
val = 0;
|
||||
break;
|
||||
case CPHP_GET_CPU_ID_CMD:
|
||||
val = cdev->arch_id >> 32;
|
||||
break;
|
||||
default:
|
||||
break;
|
||||
}
|
||||
trace_cpuhp_acpi_read_cmd_data2(cpu_st->selector, val);
|
||||
break;
|
||||
default:
|
||||
break;
|
||||
}
|
||||
|
|
|
@ -23,6 +23,7 @@ cpuhp_acpi_read_flags(uint32_t idx, uint8_t flags) "idx[0x%"PRIx32"] flags: 0x%"
|
|||
cpuhp_acpi_write_idx(uint32_t idx) "set active cpu idx: 0x%"PRIx32
|
||||
cpuhp_acpi_write_cmd(uint32_t idx, uint8_t cmd) "idx[0x%"PRIx32"] cmd: 0x%"PRIx8
|
||||
cpuhp_acpi_read_cmd_data(uint32_t idx, uint32_t data) "idx[0x%"PRIx32"] data: 0x%"PRIx32
|
||||
cpuhp_acpi_read_cmd_data2(uint32_t idx, uint32_t data) "idx[0x%"PRIx32"] data: 0x%"PRIx32
|
||||
cpuhp_acpi_cpu_has_events(uint32_t idx, bool ins, bool rm) "idx[0x%"PRIx32"] inserting: %d, removing: %d"
|
||||
cpuhp_acpi_clear_inserting_evt(uint32_t idx) "idx[0x%"PRIx32"]"
|
||||
cpuhp_acpi_clear_remove_evt(uint32_t idx) "idx[0x%"PRIx32"]"
|
||||
|
|
|
@ -1816,7 +1816,6 @@ static void build_smb0(Aml *table, I2CBus *smbus, int devnr, int func)
|
|||
Aml *scope = aml_scope("_SB.PCI0");
|
||||
Aml *dev = aml_device("SMB0");
|
||||
|
||||
aml_append(dev, aml_name_decl("_HID", aml_eisaid("APP0005")));
|
||||
aml_append(dev, aml_name_decl("_ADR", aml_int(devnr << 16 | func)));
|
||||
build_acpi_ipmi_devices(dev, BUS(smbus), "\\_SB.PCI0.SMB0");
|
||||
aml_append(scope, dev);
|
||||
|
|
|
@ -93,7 +93,9 @@
|
|||
#include "fw_cfg.h"
|
||||
#include "trace.h"
|
||||
|
||||
GlobalProperty pc_compat_4_2[] = {};
|
||||
GlobalProperty pc_compat_4_2[] = {
|
||||
{ "mch", "smbase-smram", "off" },
|
||||
};
|
||||
const size_t pc_compat_4_2_len = G_N_ELEMENTS(pc_compat_4_2);
|
||||
|
||||
GlobalProperty pc_compat_4_1[] = {};
|
||||
|
|
|
@ -275,20 +275,20 @@ static const TypeInfo q35_host_info = {
|
|||
* MCH D0:F0
|
||||
*/
|
||||
|
||||
static uint64_t tseg_blackhole_read(void *ptr, hwaddr reg, unsigned size)
|
||||
static uint64_t blackhole_read(void *ptr, hwaddr reg, unsigned size)
|
||||
{
|
||||
return 0xffffffff;
|
||||
}
|
||||
|
||||
static void tseg_blackhole_write(void *opaque, hwaddr addr, uint64_t val,
|
||||
unsigned width)
|
||||
static void blackhole_write(void *opaque, hwaddr addr, uint64_t val,
|
||||
unsigned width)
|
||||
{
|
||||
/* nothing */
|
||||
}
|
||||
|
||||
static const MemoryRegionOps tseg_blackhole_ops = {
|
||||
.read = tseg_blackhole_read,
|
||||
.write = tseg_blackhole_write,
|
||||
static const MemoryRegionOps blackhole_ops = {
|
||||
.read = blackhole_read,
|
||||
.write = blackhole_write,
|
||||
.endianness = DEVICE_NATIVE_ENDIAN,
|
||||
.valid.min_access_size = 1,
|
||||
.valid.max_access_size = 4,
|
||||
|
@ -430,6 +430,46 @@ static void mch_update_ext_tseg_mbytes(MCHPCIState *mch)
|
|||
}
|
||||
}
|
||||
|
||||
static void mch_update_smbase_smram(MCHPCIState *mch)
|
||||
{
|
||||
PCIDevice *pd = PCI_DEVICE(mch);
|
||||
uint8_t *reg = pd->config + MCH_HOST_BRIDGE_F_SMBASE;
|
||||
bool lck;
|
||||
|
||||
if (!mch->has_smram_at_smbase) {
|
||||
return;
|
||||
}
|
||||
|
||||
if (*reg == MCH_HOST_BRIDGE_F_SMBASE_QUERY) {
|
||||
pd->wmask[MCH_HOST_BRIDGE_F_SMBASE] =
|
||||
MCH_HOST_BRIDGE_F_SMBASE_LCK;
|
||||
*reg = MCH_HOST_BRIDGE_F_SMBASE_IN_RAM;
|
||||
return;
|
||||
}
|
||||
|
||||
/*
|
||||
* default/reset state, discard written value
|
||||
* which will disable SMRAM balackhole at SMBASE
|
||||
*/
|
||||
if (pd->wmask[MCH_HOST_BRIDGE_F_SMBASE] == 0xff) {
|
||||
*reg = 0x00;
|
||||
}
|
||||
|
||||
memory_region_transaction_begin();
|
||||
if (*reg & MCH_HOST_BRIDGE_F_SMBASE_LCK) {
|
||||
/* disable all writes */
|
||||
pd->wmask[MCH_HOST_BRIDGE_F_SMBASE] &=
|
||||
~MCH_HOST_BRIDGE_F_SMBASE_LCK;
|
||||
*reg = MCH_HOST_BRIDGE_F_SMBASE_LCK;
|
||||
lck = true;
|
||||
} else {
|
||||
lck = false;
|
||||
}
|
||||
memory_region_set_enabled(&mch->smbase_blackhole, lck);
|
||||
memory_region_set_enabled(&mch->smbase_window, lck);
|
||||
memory_region_transaction_commit();
|
||||
}
|
||||
|
||||
static void mch_write_config(PCIDevice *d,
|
||||
uint32_t address, uint32_t val, int len)
|
||||
{
|
||||
|
@ -456,6 +496,10 @@ static void mch_write_config(PCIDevice *d,
|
|||
MCH_HOST_BRIDGE_EXT_TSEG_MBYTES_SIZE)) {
|
||||
mch_update_ext_tseg_mbytes(mch);
|
||||
}
|
||||
|
||||
if (ranges_overlap(address, len, MCH_HOST_BRIDGE_F_SMBASE, 1)) {
|
||||
mch_update_smbase_smram(mch);
|
||||
}
|
||||
}
|
||||
|
||||
static void mch_update(MCHPCIState *mch)
|
||||
|
@ -464,6 +508,7 @@ static void mch_update(MCHPCIState *mch)
|
|||
mch_update_pam(mch);
|
||||
mch_update_smram(mch);
|
||||
mch_update_ext_tseg_mbytes(mch);
|
||||
mch_update_smbase_smram(mch);
|
||||
|
||||
/*
|
||||
* pci hole goes from end-of-low-ram to io-apic.
|
||||
|
@ -514,6 +559,9 @@ static void mch_reset(DeviceState *qdev)
|
|||
MCH_HOST_BRIDGE_EXT_TSEG_MBYTES_QUERY);
|
||||
}
|
||||
|
||||
d->config[MCH_HOST_BRIDGE_F_SMBASE] = 0;
|
||||
d->wmask[MCH_HOST_BRIDGE_F_SMBASE] = 0xff;
|
||||
|
||||
mch_update(mch);
|
||||
}
|
||||
|
||||
|
@ -563,7 +611,7 @@ static void mch_realize(PCIDevice *d, Error **errp)
|
|||
memory_region_add_subregion(&mch->smram, 0xfeda0000, &mch->high_smram);
|
||||
|
||||
memory_region_init_io(&mch->tseg_blackhole, OBJECT(mch),
|
||||
&tseg_blackhole_ops, NULL,
|
||||
&blackhole_ops, NULL,
|
||||
"tseg-blackhole", 0);
|
||||
memory_region_set_enabled(&mch->tseg_blackhole, false);
|
||||
memory_region_add_subregion_overlap(mch->system_memory,
|
||||
|
@ -575,6 +623,27 @@ static void mch_realize(PCIDevice *d, Error **errp)
|
|||
memory_region_set_enabled(&mch->tseg_window, false);
|
||||
memory_region_add_subregion(&mch->smram, mch->below_4g_mem_size,
|
||||
&mch->tseg_window);
|
||||
|
||||
/*
|
||||
* This is not what hardware does, so it's QEMU specific hack.
|
||||
* See commit message for details.
|
||||
*/
|
||||
memory_region_init_io(&mch->smbase_blackhole, OBJECT(mch), &blackhole_ops,
|
||||
NULL, "smbase-blackhole",
|
||||
MCH_HOST_BRIDGE_SMBASE_SIZE);
|
||||
memory_region_set_enabled(&mch->smbase_blackhole, false);
|
||||
memory_region_add_subregion_overlap(mch->system_memory,
|
||||
MCH_HOST_BRIDGE_SMBASE_ADDR,
|
||||
&mch->smbase_blackhole, 1);
|
||||
|
||||
memory_region_init_alias(&mch->smbase_window, OBJECT(mch),
|
||||
"smbase-window", mch->ram_memory,
|
||||
MCH_HOST_BRIDGE_SMBASE_ADDR,
|
||||
MCH_HOST_BRIDGE_SMBASE_SIZE);
|
||||
memory_region_set_enabled(&mch->smbase_window, false);
|
||||
memory_region_add_subregion(&mch->smram, MCH_HOST_BRIDGE_SMBASE_ADDR,
|
||||
&mch->smbase_window);
|
||||
|
||||
object_property_add_const_link(qdev_get_machine(), "smram",
|
||||
OBJECT(&mch->smram), &error_abort);
|
||||
|
||||
|
@ -601,6 +670,7 @@ uint64_t mch_mcfg_base(void)
|
|||
static Property mch_props[] = {
|
||||
DEFINE_PROP_UINT16("extended-tseg-mbytes", MCHPCIState, ext_tseg_mbytes,
|
||||
16),
|
||||
DEFINE_PROP_BOOL("smbase-smram", MCHPCIState, has_smram_at_smbase, true),
|
||||
DEFINE_PROP_END_OF_LIST(),
|
||||
};
|
||||
|
||||
|
|
|
@ -943,7 +943,13 @@ void virtio_scsi_common_unrealize(DeviceState *dev)
|
|||
{
|
||||
VirtIODevice *vdev = VIRTIO_DEVICE(dev);
|
||||
VirtIOSCSICommon *vs = VIRTIO_SCSI_COMMON(dev);
|
||||
int i;
|
||||
|
||||
virtio_delete_queue(vs->ctrl_vq);
|
||||
virtio_delete_queue(vs->event_vq);
|
||||
for (i = 0; i < vs->conf.num_queues; i++) {
|
||||
virtio_delete_queue(vs->cmd_vqs[i]);
|
||||
}
|
||||
g_free(vs->cmd_vqs);
|
||||
virtio_cleanup(vdev);
|
||||
}
|
||||
|
|
|
@ -335,8 +335,10 @@ static void vhost_vsock_device_realize(DeviceState *dev, Error **errp)
|
|||
sizeof(struct virtio_vsock_config));
|
||||
|
||||
/* Receive and transmit queues belong to vhost */
|
||||
virtio_add_queue(vdev, VHOST_VSOCK_QUEUE_SIZE, vhost_vsock_handle_output);
|
||||
virtio_add_queue(vdev, VHOST_VSOCK_QUEUE_SIZE, vhost_vsock_handle_output);
|
||||
vsock->recv_vq = virtio_add_queue(vdev, VHOST_VSOCK_QUEUE_SIZE,
|
||||
vhost_vsock_handle_output);
|
||||
vsock->trans_vq = virtio_add_queue(vdev, VHOST_VSOCK_QUEUE_SIZE,
|
||||
vhost_vsock_handle_output);
|
||||
|
||||
/* The event queue belongs to QEMU */
|
||||
vsock->event_vq = virtio_add_queue(vdev, VHOST_VSOCK_QUEUE_SIZE,
|
||||
|
@ -363,6 +365,9 @@ static void vhost_vsock_device_realize(DeviceState *dev, Error **errp)
|
|||
err_vhost_dev:
|
||||
vhost_dev_cleanup(&vsock->vhost_dev);
|
||||
err_virtio:
|
||||
virtio_delete_queue(vsock->recv_vq);
|
||||
virtio_delete_queue(vsock->trans_vq);
|
||||
virtio_delete_queue(vsock->event_vq);
|
||||
virtio_cleanup(vdev);
|
||||
close(vhostfd);
|
||||
return;
|
||||
|
@ -379,6 +384,9 @@ static void vhost_vsock_device_unrealize(DeviceState *dev, Error **errp)
|
|||
vhost_vsock_set_status(vdev, 0);
|
||||
|
||||
vhost_dev_cleanup(&vsock->vhost_dev);
|
||||
virtio_delete_queue(vsock->recv_vq);
|
||||
virtio_delete_queue(vsock->trans_vq);
|
||||
virtio_delete_queue(vsock->event_vq);
|
||||
virtio_cleanup(vdev);
|
||||
}
|
||||
|
||||
|
|
|
@ -547,26 +547,28 @@ static void vhost_region_add_section(struct vhost_dev *dev,
|
|||
uintptr_t mrs_host = (uintptr_t)memory_region_get_ram_ptr(section->mr) +
|
||||
section->offset_within_region;
|
||||
RAMBlock *mrs_rb = section->mr->ram_block;
|
||||
size_t mrs_page = qemu_ram_pagesize(mrs_rb);
|
||||
|
||||
trace_vhost_region_add_section(section->mr->name, mrs_gpa, mrs_size,
|
||||
mrs_host);
|
||||
|
||||
/* Round the section to it's page size */
|
||||
/* First align the start down to a page boundary */
|
||||
uint64_t alignage = mrs_host & (mrs_page - 1);
|
||||
if (alignage) {
|
||||
mrs_host -= alignage;
|
||||
mrs_size += alignage;
|
||||
mrs_gpa -= alignage;
|
||||
if (dev->vhost_ops->backend_type == VHOST_BACKEND_TYPE_USER) {
|
||||
/* Round the section to it's page size */
|
||||
/* First align the start down to a page boundary */
|
||||
size_t mrs_page = qemu_ram_pagesize(mrs_rb);
|
||||
uint64_t alignage = mrs_host & (mrs_page - 1);
|
||||
if (alignage) {
|
||||
mrs_host -= alignage;
|
||||
mrs_size += alignage;
|
||||
mrs_gpa -= alignage;
|
||||
}
|
||||
/* Now align the size up to a page boundary */
|
||||
alignage = mrs_size & (mrs_page - 1);
|
||||
if (alignage) {
|
||||
mrs_size += mrs_page - alignage;
|
||||
}
|
||||
trace_vhost_region_add_section_aligned(section->mr->name, mrs_gpa,
|
||||
mrs_size, mrs_host);
|
||||
}
|
||||
/* Now align the size up to a page boundary */
|
||||
alignage = mrs_size & (mrs_page - 1);
|
||||
if (alignage) {
|
||||
mrs_size += mrs_page - alignage;
|
||||
}
|
||||
trace_vhost_region_add_section_aligned(section->mr->name, mrs_gpa, mrs_size,
|
||||
mrs_host);
|
||||
|
||||
if (dev->n_tmp_sections) {
|
||||
/* Since we already have at least one section, lets see if
|
||||
|
@ -590,9 +592,10 @@ static void vhost_region_add_section(struct vhost_dev *dev,
|
|||
* match up in the same RAMBlock if they do.
|
||||
*/
|
||||
if (mrs_gpa < prev_gpa_start) {
|
||||
error_report("%s:Section rounded to %"PRIx64
|
||||
" prior to previous %"PRIx64,
|
||||
__func__, mrs_gpa, prev_gpa_start);
|
||||
error_report("%s:Section '%s' rounded to %"PRIx64
|
||||
" prior to previous '%s' %"PRIx64,
|
||||
__func__, section->mr->name, mrs_gpa,
|
||||
prev_sec->mr->name, prev_gpa_start);
|
||||
/* A way to cleanly fail here would be better */
|
||||
return;
|
||||
}
|
||||
|
|
|
@ -32,6 +32,7 @@
|
|||
#include "hw/acpi/ich9.h"
|
||||
#include "hw/pci-host/pam.h"
|
||||
#include "hw/i386/intel_iommu.h"
|
||||
#include "qemu/units.h"
|
||||
|
||||
#define TYPE_Q35_HOST_DEVICE "q35-pcihost"
|
||||
#define Q35_HOST_DEVICE(obj) \
|
||||
|
@ -54,6 +55,8 @@ typedef struct MCHPCIState {
|
|||
MemoryRegion smram_region, open_high_smram;
|
||||
MemoryRegion smram, low_smram, high_smram;
|
||||
MemoryRegion tseg_blackhole, tseg_window;
|
||||
MemoryRegion smbase_blackhole, smbase_window;
|
||||
bool has_smram_at_smbase;
|
||||
Range pci_hole;
|
||||
uint64_t below_4g_mem_size;
|
||||
uint64_t above_4g_mem_size;
|
||||
|
@ -97,6 +100,13 @@ typedef struct Q35PCIHost {
|
|||
#define MCH_HOST_BRIDGE_EXT_TSEG_MBYTES_QUERY 0xffff
|
||||
#define MCH_HOST_BRIDGE_EXT_TSEG_MBYTES_MAX 0xfff
|
||||
|
||||
#define MCH_HOST_BRIDGE_SMBASE_SIZE (128 * KiB)
|
||||
#define MCH_HOST_BRIDGE_SMBASE_ADDR 0x30000
|
||||
#define MCH_HOST_BRIDGE_F_SMBASE 0x9c
|
||||
#define MCH_HOST_BRIDGE_F_SMBASE_QUERY 0xff
|
||||
#define MCH_HOST_BRIDGE_F_SMBASE_IN_RAM 0x01
|
||||
#define MCH_HOST_BRIDGE_F_SMBASE_LCK 0x02
|
||||
|
||||
#define MCH_HOST_BRIDGE_PCIEXBAR 0x60 /* 64bit register */
|
||||
#define MCH_HOST_BRIDGE_PCIEXBAR_SIZE 8 /* 64bit register */
|
||||
#define MCH_HOST_BRIDGE_PCIEXBAR_DEFAULT 0xb0000000
|
||||
|
|
|
@ -33,6 +33,8 @@ typedef struct {
|
|||
struct vhost_virtqueue vhost_vqs[2];
|
||||
struct vhost_dev vhost_dev;
|
||||
VirtQueue *event_vq;
|
||||
VirtQueue *recv_vq;
|
||||
VirtQueue *trans_vq;
|
||||
QEMUTimer *post_load_timer;
|
||||
|
||||
/*< public >*/
|
||||
|
|
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
|
@ -16,7 +16,10 @@
|
|||
* 1. add empty files for new tables, if any, under tests/data/acpi
|
||||
* 2. list any changed files in tests/bios-tables-test-allowed-diff.h
|
||||
* 3. commit the above *before* making changes that affect the tables
|
||||
* Maintainer:
|
||||
*
|
||||
* Contributor or ACPI Maintainer (steps 4-7 need to be redone to resolve conflicts
|
||||
* in binary commit created in step 6):
|
||||
*
|
||||
* After 1-3 above tests will pass but ignore differences with the expected files.
|
||||
* You will also notice that tests/bios-tables-test-allowed-diff.h lists
|
||||
* a bunch of files. This is your hint that you need to do the below:
|
||||
|
@ -28,13 +31,23 @@
|
|||
* output. If not - disassemble them yourself in any way you like.
|
||||
* Look at the differences - make sure they make sense and match what the
|
||||
* changes you are merging are supposed to do.
|
||||
* Save the changes, preferably in form of ASL diff for the commit log in
|
||||
* step 6.
|
||||
*
|
||||
* 5. From build directory, run:
|
||||
* $(SRC_PATH)/tests/data/acpi/rebuild-expected-aml.sh
|
||||
* 6. Now commit any changes.
|
||||
* 7. Before doing a pull request, make sure tests/bios-tables-test-allowed-diff.h
|
||||
* is empty - this will ensure following changes to ACPI tables will
|
||||
* be noticed.
|
||||
* 6. Now commit any changes to the expected binary, include diff from step 4
|
||||
* in commit log.
|
||||
* 7. Before sending patches to the list (Contributor)
|
||||
* or before doing a pull request (Maintainer), make sure
|
||||
* tests/bios-tables-test-allowed-diff.h is empty - this will ensure
|
||||
* following changes to ACPI tables will be noticed.
|
||||
*
|
||||
* The resulting patchset/pull request then looks like this:
|
||||
* - patch 1: list changed files in tests/bios-tables-test-allowed-diff.h.
|
||||
* - patches 2 - n: real changes, may contain multiple patches.
|
||||
* - patch n + 1: update golden master binaries and empty
|
||||
* tests/bios-tables-test-allowed-diff.h
|
||||
*/
|
||||
|
||||
#include "qemu/osdep.h"
|
||||
|
|
|
@ -186,6 +186,109 @@ static void test_tseg_size(const void *data)
|
|||
qtest_quit(qts);
|
||||
}
|
||||
|
||||
#define SMBASE 0x30000
|
||||
#define SMRAM_TEST_PATTERN 0x32
|
||||
#define SMRAM_TEST_RESET_PATTERN 0x23
|
||||
|
||||
static void test_smram_smbase_lock(void)
|
||||
{
|
||||
QPCIBus *pcibus;
|
||||
QPCIDevice *pcidev;
|
||||
QDict *response;
|
||||
QTestState *qts;
|
||||
int i;
|
||||
|
||||
qts = qtest_init("-M q35");
|
||||
|
||||
pcibus = qpci_new_pc(qts, NULL);
|
||||
g_assert(pcibus != NULL);
|
||||
|
||||
pcidev = qpci_device_find(pcibus, 0);
|
||||
g_assert(pcidev != NULL);
|
||||
|
||||
/* check that SMRAM is not enabled by default */
|
||||
g_assert(qpci_config_readb(pcidev, MCH_HOST_BRIDGE_F_SMBASE) == 0);
|
||||
qtest_writeb(qts, SMBASE, SMRAM_TEST_PATTERN);
|
||||
g_assert_cmpint(qtest_readb(qts, SMBASE), ==, SMRAM_TEST_PATTERN);
|
||||
|
||||
/* check that writing junk to 0x9c before before negotiating is ignored */
|
||||
for (i = 0; i < 0xff; i++) {
|
||||
qpci_config_writeb(pcidev, MCH_HOST_BRIDGE_F_SMBASE, i);
|
||||
g_assert(qpci_config_readb(pcidev, MCH_HOST_BRIDGE_F_SMBASE) == 0);
|
||||
}
|
||||
|
||||
/* enable SMRAM at SMBASE */
|
||||
qpci_config_writeb(pcidev, MCH_HOST_BRIDGE_F_SMBASE, 0xff);
|
||||
g_assert(qpci_config_readb(pcidev, MCH_HOST_BRIDGE_F_SMBASE) == 0x01);
|
||||
/* lock SMRAM at SMBASE */
|
||||
qpci_config_writeb(pcidev, MCH_HOST_BRIDGE_F_SMBASE, 0x02);
|
||||
g_assert(qpci_config_readb(pcidev, MCH_HOST_BRIDGE_F_SMBASE) == 0x02);
|
||||
|
||||
/* check that SMRAM at SMBASE is locked and can't be unlocked */
|
||||
g_assert_cmpint(qtest_readb(qts, SMBASE), ==, 0xff);
|
||||
for (i = 0; i <= 0xff; i++) {
|
||||
/* make sure register is immutable */
|
||||
qpci_config_writeb(pcidev, MCH_HOST_BRIDGE_F_SMBASE, i);
|
||||
g_assert(qpci_config_readb(pcidev, MCH_HOST_BRIDGE_F_SMBASE) == 0x02);
|
||||
|
||||
/* RAM access should go into black hole */
|
||||
qtest_writeb(qts, SMBASE, SMRAM_TEST_PATTERN);
|
||||
g_assert_cmpint(qtest_readb(qts, SMBASE), ==, 0xff);
|
||||
}
|
||||
|
||||
/* reset */
|
||||
response = qtest_qmp(qts, "{'execute': 'system_reset', 'arguments': {} }");
|
||||
g_assert(response);
|
||||
g_assert(!qdict_haskey(response, "error"));
|
||||
qobject_unref(response);
|
||||
|
||||
/* check RAM at SMBASE is available after reset */
|
||||
g_assert_cmpint(qtest_readb(qts, SMBASE), ==, SMRAM_TEST_PATTERN);
|
||||
g_assert(qpci_config_readb(pcidev, MCH_HOST_BRIDGE_F_SMBASE) == 0);
|
||||
qtest_writeb(qts, SMBASE, SMRAM_TEST_RESET_PATTERN);
|
||||
g_assert_cmpint(qtest_readb(qts, SMBASE), ==, SMRAM_TEST_RESET_PATTERN);
|
||||
|
||||
g_free(pcidev);
|
||||
qpci_free_pc(pcibus);
|
||||
|
||||
qtest_quit(qts);
|
||||
}
|
||||
|
||||
static void test_without_smram_base(void)
|
||||
{
|
||||
QPCIBus *pcibus;
|
||||
QPCIDevice *pcidev;
|
||||
QTestState *qts;
|
||||
int i;
|
||||
|
||||
qts = qtest_init("-M pc-q35-4.1");
|
||||
|
||||
pcibus = qpci_new_pc(qts, NULL);
|
||||
g_assert(pcibus != NULL);
|
||||
|
||||
pcidev = qpci_device_find(pcibus, 0);
|
||||
g_assert(pcidev != NULL);
|
||||
|
||||
/* check that RAM is accessible */
|
||||
qtest_writeb(qts, SMBASE, SMRAM_TEST_PATTERN);
|
||||
g_assert_cmpint(qtest_readb(qts, SMBASE), ==, SMRAM_TEST_PATTERN);
|
||||
|
||||
/* check that writing to 0x9c succeeds */
|
||||
for (i = 0; i <= 0xff; i++) {
|
||||
qpci_config_writeb(pcidev, MCH_HOST_BRIDGE_F_SMBASE, i);
|
||||
g_assert(qpci_config_readb(pcidev, MCH_HOST_BRIDGE_F_SMBASE) == i);
|
||||
}
|
||||
|
||||
/* check that RAM is still accessible */
|
||||
qtest_writeb(qts, SMBASE, SMRAM_TEST_PATTERN + 1);
|
||||
g_assert_cmpint(qtest_readb(qts, SMBASE), ==, (SMRAM_TEST_PATTERN + 1));
|
||||
|
||||
g_free(pcidev);
|
||||
qpci_free_pc(pcibus);
|
||||
|
||||
qtest_quit(qts);
|
||||
}
|
||||
|
||||
int main(int argc, char **argv)
|
||||
{
|
||||
g_test_init(&argc, &argv, NULL);
|
||||
|
@ -197,5 +300,7 @@ int main(int argc, char **argv)
|
|||
qtest_add_data_func("/q35/tseg-size/8mb", &tseg_8mb, test_tseg_size);
|
||||
qtest_add_data_func("/q35/tseg-size/ext/16mb", &tseg_ext_16mb,
|
||||
test_tseg_size);
|
||||
qtest_add_func("/q35/smram/smbase_lock", test_smram_smbase_lock);
|
||||
qtest_add_func("/q35/smram/legacy_smbase", test_without_smram_base);
|
||||
return g_test_run();
|
||||
}
|
||||
|
|
Loading…
Reference in a new issue