forked from cloud-hypervisor/cloud-hypervisor
-
Notifications
You must be signed in to change notification settings - Fork 0
github: Rust toolchain matrix #3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Closed
Closed
Conversation
This file contains hidden or 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
3dadfdf to
c739b33
Compare
Bumps [micro_http](https://github.com/firecracker-microvm/micro-http) from `e712d6a` to `e89ed14`. - [Release notes](https://github.com/firecracker-microvm/micro-http/releases) - [Commits](firecracker-microvm/micro-http@e712d6a...e89ed14) Signed-off-by: dependabot-preview[bot] <[email protected]>
This will allow the syntax to be reused with cloud-hypervsor binary and ch-remote. Signed-off-by: Rob Bradford <[email protected]>
Split it into a method that creates a single device which is called by the multiple device version so this can be used when dynamically adding a device. Signed-off-by: Rob Bradford <[email protected]>
Signed-off-by: Rob Bradford <[email protected]>
The disk will be hotplugged via DeviceManager and saved in the config for later use. Signed-off-by: Rob Bradford <[email protected]>
Add the HTTP and internal API entry points for adding disks at runtime. Signed-off-by: Rob Bradford <[email protected]>
Signed-off-by: Rob Bradford <[email protected]>
Signed-off-by: Rob Bradford <[email protected]>
Call the new HTTP API for hotplugging disks using the same syntax as disk coldplug. Signed-off-by: Rob Bradford <[email protected]>
Signed-off-by: Rob Bradford <[email protected]>
vm_remove_device was copied from vm_add_device but the error message wasn't correctly updated. Signed-off-by: Rob Bradford <[email protected]>
We made sure gpa is in cache range, but not the end addr of request, which is (gpa + len). If the end addr of request is beyond dax cache window, vmm would corrupt guest memory or crash. Fix it by making sure end addr of request is within cache range as well. And while we're at it, return EFAULT if the request is out of range, as write(2)/read(2) returns EFAULT when buffer is outside accessible address space. Signed-off-by: Eryu Guan <[email protected]>
This will allow the syntax to be reused with cloud-hypervisor binary and ch-remote. Signed-off-by: Rob Bradford <[email protected]>
Split it into a method that creates a single device which is called by the multiple device version so this can be used when dynamically adding a device. Signed-off-by: Rob Bradford <[email protected]>
Signed-off-by: Rob Bradford <[email protected]>
The persistent memory will be hotplugged via DeviceManager and saved in the config for later use. Signed-off-by: Rob Bradford <[email protected]>
Add the HTTP and internal API entry points for adding persistent memory at runtime. Signed-off-by: Rob Bradford <[email protected]>
Signed-off-by: Rob Bradford <[email protected]>
Signed-off-by: Rob Bradford <[email protected]>
Call the new HTTP API for hotplugging persistent memory using the same syntax as coldplug. Signed-off-by: Rob Bradford <[email protected]>
Signed-off-by: Rob Bradford <[email protected]>
The basic idea of virtio-mem is to provide a flexible, cross-architecture memory hot plug and hot unplug solution that avoids many limitations imposed by existing technologies, architectures, and interfaces. More details can be found in https://lkml.org/lkml/2019/12/12/681. This commit add virtio-mem device. Signed-off-by: Hui Zhu <[email protected]>
This commit adds new option hotplug_method to memory config. It can set the hotplug method to "acpi" or "virtio-mem". Signed-off-by: Hui Zhu <[email protected]>
Add make_virtio_mem_devices to add virtio-mem to vmm. Signed-off-by: Hui Zhu <[email protected]>
Add test for hotplug_size and hotplug_method. Signed-off-by: Hui Zhu <[email protected]>
Add integration test test_virtio_mem for virtio-mem. Signed-off-by: Hui Zhu <[email protected]>
Add hotplug_method to MemoryConfig in cloud-hypervisor.yaml. Signed-off-by: Hui Zhu <[email protected]>
Signed-off-by: Samuel Ortiz <[email protected]>
Adds DeviceManager method `make_virtio_fs_device` which creates a single device, and modifies `make_virtio_fs_devices` to use this method. Implements the new `vm.add-fs route`. Signed-off-by: Dean Sheather <[email protected]>
The hotplugged virtio devices were not added to the list of virtio devices from the DeviceManager. This patch fixes it, as it was causing hotplugged virtio-fs devices from not supporting memory hotplug, since they were never getting the update as they were not part of the list of virtio devices held by the DeviceManager. Signed-off-by: Sebastien Boeuf <[email protected]>
Adds integration tests with and without dax for the new vm.add-fs route. Signed-off-by: Dean Sheather <[email protected]>
There is some duplication between regular and hotplug virtio-fs tests that can be factorized by adding a simple hotplug flag to choose if each test should run with or without hotplugging the device. Signed-off-by: Sebastien Boeuf <[email protected]>
Bumps [backtrace-sys](https://github.com/alexcrichton/backtrace-rs) from 0.1.35 to 0.1.36. - [Release notes](https://github.com/alexcrichton/backtrace-rs/releases) - [Commits](rust-lang/backtrace-rs@backtrace-sys-0.1.35...backtrace-sys-0.1.36) Signed-off-by: dependabot-preview[bot] <[email protected]>
Introduce new getter function to the VirtioDevice trait, as it will allow the caller to retrieve the list of userspace mappings associated with the device. Signed-off-by: Sebastien Boeuf <[email protected]>
This will help when we will implement the hot-unplug of the virtio-fs device, as we will have to remove correctly the userspace mappings associated with the device. Signed-off-by: Sebastien Boeuf <[email protected]>
When hot-unplugging the virtio-pmem from the VM, we don't remove the associated userspace mapping. This patch will let us fix this in a following patch. For now, it simply adapts the code so that the Pmem device knows about the mapping associated with it. By knowing about it, it can expose it to the caller through the new userspace_mappings() function. Signed-off-by: Sebastien Boeuf <[email protected]>
Since the virtio-fs device is backed by a vhost-user process, it is important to implement the proper shutdown() function from the VirtioDevice trait, as vhost-user-blk and vhost-user-net do. Signed-off-by: Sebastien Boeuf <[email protected]>
The same way we added a helper for creating userspace memory mappings from the MemoryManager, this patch adds a new helper to remove some previously added mappings. Signed-off-by: Sebastien Boeuf <[email protected]>
When a virtio device is dynamically removed from the VM through the hot-unplug mechanism, every mapping associated with it must be properly removed. Based on the previous patches letting a VirtioDevice expose the list of userspace mappings associated with it, this patch can now remove all the KVM userspace memory regions through the MemoryManager. Signed-off-by: Sebastien Boeuf <[email protected]>
First, this modifies the existing helpers on how to get indexes for available and used rings from memory. Instead of updating the queue through each helper, they are now used as simple getters. Based on these new getters, we could create a new helper to determine if the queue has some available descriptors already queued from the driver side. This helper is going to be particularly helpful when trying to determine from a virtio thread if a queue is already loaded with some available buffers that can be used to send information to the guest. Signed-off-by: Sebastien Boeuf <[email protected]>
On the restore path, using the available and used indexes read from memory to fill the Queue structure was a mistake. Indeed, the available index is written from the guest and it reflects the last available index in the descriptor table. But the driver might have queued a lot of buffers which have not yet been used by the device. This leads to a situation where the next_avail from Queue is completely different from the one we can read from memory. Instead, the right way to determine the next_avail index that should be used by the device is by relying on the used index from the memory. This index represents the correct information we're looking for as it has been updated before the snapshot to let the guest know the next index to process. Signed-off-by: Sebastien Boeuf <[email protected]>
The frame buffer must be updated depending on the amount read from it, which depends on the number and depth of descriptors available at the time of the processing. This patch handles this buffer update, and allow for large buffers to be correctly processed in multiple rounds. Signed-off-by: Sebastien Boeuf <[email protected]>
Signed-off-by: Sebastien Boeuf <[email protected]> Signed-off-by: Yi Sun <[email protected]>
Bumps [arc-swap](https://github.com/vorner/arc-swap) from 0.4.5 to 0.4.6. - [Release notes](https://github.com/vorner/arc-swap/releases) - [Changelog](https://github.com/vorner/arc-swap/blob/master/CHANGELOG.md) - [Commits](vorner/arc-swap@v0.4.5...v0.4.6) Signed-off-by: dependabot-preview[bot] <[email protected]>
Replace alignment calculation of start address with functionally equivalent version that does not assume that the block size is a power of two. Signed-off-by: Martin Xu <[email protected]> Signed-off-by: Rob Bradford <[email protected]>
Bumps [parking_lot_core](https://github.com/Amanieu/parking_lot) from 0.7.1 to 0.7.2. - [Release notes](https://github.com/Amanieu/parking_lot/releases) - [Changelog](https://github.com/Amanieu/parking_lot/blob/master/CHANGELOG.md) - [Commits](Amanieu/parking_lot@0.7.1...core-0.7.2) Signed-off-by: dependabot-preview[bot] <[email protected]>
This brings the migration support to virtio-pmem device. Signed-off-by: Sebastien Boeuf <[email protected]> Signed-off-by: Yi Sun <[email protected]>
This patch implements the Snapshottable trait for virtio-console, which enables migration support for it. A VM with a virtio-console device attached can be snapshot and then restored without issues. Signed-off-by: Sebastien Boeuf <[email protected]> Signed-off-by: Yi Sun <[email protected]>
Bumps [cc](https://github.com/alexcrichton/cc-rs) from 1.0.50 to 1.0.51. - [Release notes](https://github.com/alexcrichton/cc-rs/releases) - [Commits](alexcrichton/cc-rs@1.0.50...1.0.51) Signed-off-by: dependabot-preview[bot] <[email protected]>
This doc describe how to use vhost-user-blk in Cloud Hypervisor with SPDK. With this doc, we can setup simple test environment on Ubuntu release. Signed-off-by: Yang Zhong <[email protected]>
Bumps [pin-utils](https://github.com/rust-lang-nursery/pin-utils) from 0.1.0-alpha.4 to 0.1.0. - [Release notes](https://github.com/rust-lang-nursery/pin-utils/releases) - [Commits](rust-lang/pin-utils@0.1.0-alpha.4...0.1.0) Signed-off-by: dependabot-preview[bot] <[email protected]>
Now that we have multiple virtio devices supporting snapshot and restore operations, we can add a new integration test to validate the migration feature works as expected. The important part is virtio-net as it is used to ssh into the VM to verify the VM has been restored in the proper state. This test only works for virtio-mmio for now. Further support for testing virtio-pci will be added once the virtio-pci transport layer will support migration. Signed-off-by: Sebastien Boeuf <[email protected]>
Bumps [cc](https://github.com/alexcrichton/cc-rs) from 1.0.51 to 1.0.52. - [Release notes](https://github.com/alexcrichton/cc-rs/releases) - [Commits](alexcrichton/cc-rs@1.0.51...1.0.52) Signed-off-by: dependabot-preview[bot] <[email protected]>
The newly added integration test "test_snapshot_restore" is very unstable, which causes our CI to fail on most pull requests, which is not acceptable. That's why we ignore this test until we can fix the stability issue. Signed-off-by: Sebastien Boeuf <[email protected]>
Bumps [ryu](https://github.com/dtolnay/ryu) from 1.0.3 to 1.0.4. - [Release notes](https://github.com/dtolnay/ryu/releases) - [Commits](dtolnay/ryu@1.0.3...1.0.4) Signed-off-by: dependabot-preview[bot] <[email protected]>
The release one is always skipped for PRs which can be confusing for e.g. dependabot. Signed-off-by: Samuel Ortiz <[email protected]>
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.
Signed-off-by: Samuel Ortiz [email protected]