Only advertise AMX feature bits to guest when the AMX cpu feature is enabled #5834
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
The following two tests are hanging, and I will need to take a closer look. Convert this PR to draft for now. |
up2wing
reviewed
Oct 16, 2023
likebreath
force-pushed
the
1011/fix_amx_cpuid_bits
branch
from
9c2554e
to
0d06757
Compare
rbradford
approved these changes
Oct 18, 2023
rbradford
reviewed
Oct 18, 2023
This struct contains all configuration fields that controls the way how we generate CPUID for the guest on x86_64. This allows cleaner extension when adding new configuration fields. Signed-off-by: Bo Chen <chen.bo@intel.com>
Fixes: cloud-hypervisor#5833 Signed-off-by: Bo Chen <chen.bo@intel.com>
likebreath
force-pushed
the
1011/fix_amx_cpuid_bits
branch
from
0d06757
to
57d527b
Compare
likebreath
added
bug
russell-islam
pushed a commit
to microsoft/cloud-hypervisor
that referenced
this pull request
Apr 12, 2024
v36.0 This release has been tracked in our [roadmap project](https://github.com/orgs/cloud-hypervisor/projects/6) as iteration v36.0. The following user visible changes have been made: Command Line Changes -------------------- We switched back to use the `clap` crate to create our command line, since the `argh` crate is barely maintained. There were several syntax changes: * All `--option value` commands now are `--option=value`. * The `--disk DISK1 --disk DISK2` command now is `--disk DISK1 DISK2`. * The `-v -v -v`command now is `-vvv`. Note: the released binary size increased around 0.3M due to this change. Enabled Features Reported via API Endpoint and CLI -------------------------------------------------- Now the enabled (Cargo) features of the running Cloud Hypervisor instance can be queried via API endpoint (`/vmm.ping`) and CLI (`--version -v`). NUMA Support for PCI segments ----------------------------- The `--numa` command is augmented with a new option `pci_segment=`, so that users can define the relationship between PCI segments and NUMA nodes. Examples can be found from the [memory documentation](docs/memory.md) CPU Topology Support on AMD Platforms ------------------------------------- Now the CPU topology on x86_64 platforms supports multiple vendors. Unix Socket Backend for Serial Port ----------------------------------- The `--serial` command is augmented with a new option `socket=`, allowing users to access the serial port using a Unix socket. AIO Backend for Block Devices ----------------------------- An AIO backend is added for `virtio-block` devices to improve block device performance when the `io_uring` feature is not supported by the host Operating System. Documentation Improvements -------------------------- * New [documentation](docs/coverage.md) for collecting coverage data * Various typo fixes Notable Bug Fixes ----------------- * Fix a deadlock when TDX is enabled (cloud-hypervisor#5845) * Only advertise AMX feature bits to guest when the AMX cpu feature is enabled (cloud-hypervisor#5834) * Correct default value for vCPU topology on AArch64 (cloud-hypervisor#5893) Contributors ------------ Many thanks to everyone who has contributed to our release: * Anatol Belski <anbelski@linux.microsoft.com> * Bo Chen <chen.bo@intel.com> * Dario Nieuwenhuis <dirbaio@dirbaio.net> * Jianyong Wu <jianyong.wu@arm.com> * Jinank Jain <jinankjain@microsoft.com> * Muminul Islam <muislam@microsoft.com> * Praveen K Paladugu <prapal@linux.microsoft.com> * Ravi kumar Veeramally <ravikumar.veeramally@intel.com> * Rob Bradford <rbradford@rivosinc.com> * Thomas Barrett <tbarrett@crusoeenergy.com> * Wei Liu <liuwe@microsoft.com> * Yi Wang <foxywang@tencent.com> * dom.song <dom.song@amperecomputing.com> * tag 'v36.0': (432 commits) build: Release v36.0 main: Add the '--serial socket=' option help information build: Bump zerocopy from 0.7.8 to 0.7.21 build: Bump serde_json from 1.0.107 to 1.0.108 in /fuzz Jenkinsfile: Skip 'test_vfio' and 'test_vfio_user' on AMD workers scripts: Simplify the script for running bare-metal VFIO tests tests: Remove "test_vfio" from the bare-metal worker aarch64: fdt: Use more appropriate default value for topology build: Bump toml_edit from 0.19.8 to 0.19.15 build: Bump MSRV to 1.66 build: Bump winapi-util from 0.1.5 to 0.1.6 scripts: Clear the toolchain environment variables for virtiofsd build: Bump futures-core from 0.3.28 to 0.3.29 in /fuzz virtio-devices, vmm: Update seccomp list build: Bump seccompiler from 0.3.0 to 0.4.0 build: Bump zerocopy from 0.7.11 to 0.7.20 in /fuzz block: add copyright text to raw_async_aio.rs hypervisor: Add support for MMIO write emulation hypervisor: Add support MMIO read VMGEXIT hypervisor: Add support for legacy I/O port emulation ...
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR fixes #5833 with a refactoring for easier generation of common CPUIDs based on guest VM configurations.