Skip to content
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

Deprecate GuestMemory stopgap "iterator" methods #133

Closed
Tracked by #141
Daniel-Aaron-Bloom opened this issue Jan 7, 2021 · 0 comments · Fixed by #144
Closed
Tracked by #141

Deprecate GuestMemory stopgap "iterator" methods #133

Daniel-Aaron-Bloom opened this issue Jan 7, 2021 · 0 comments · Fixed by #144
Labels

Comments

@Daniel-Aaron-Bloom
Copy link
Contributor

with_regions, with_regions_mut and map_and_fold were just papering over the lack of iterator-based access to the regions in GuestMemory. After #130 closes out #8, they will no longer be needed and should be deprecated and eventually removed.

@andreeaflorescu andreeaflorescu added the good first issue Good for newcomers label Feb 10, 2021
jiangliu pushed a commit that referenced this issue Mar 11, 2021
* Deprecates `with_regions` `with_regions_mut` and `map_and_fold`.
* Update unit tests

Signed-off-by: defunct <defunct@defunct.io>

Resolves #133
likebreath added a commit to likebreath/cloud-hypervisor that referenced this issue May 27, 2021
Function "GuestMemory::with_regions(_mut)" were mainly temporary methods
to access the regions in `GuestMemory` as the lack of iterator-based
access, and hence they are deprecated in the upstream vm-memory crate [1].

[1] rust-vmm/vm-memory#133

Signed-off-by: Bo Chen <chen.bo@intel.com>
likebreath added a commit to likebreath/cloud-hypervisor that referenced this issue May 27, 2021
Function "GuestMemory::with_regions(_mut)" were mainly temporary methods
to access the regions in `GuestMemory` as the lack of iterator-based
access, and hence they are deprecated in the upstream vm-memory crate [1].

[1] rust-vmm/vm-memory#133

Signed-off-by: Bo Chen <chen.bo@intel.com>
likebreath added a commit to likebreath/cloud-hypervisor that referenced this issue May 27, 2021
Function "GuestMemory::with_regions(_mut)" were mainly temporary methods
to access the regions in `GuestMemory` as the lack of iterator-based
access, and hence they are deprecated in the upstream vm-memory crate [1].

[1] rust-vmm/vm-memory#133

Signed-off-by: Bo Chen <chen.bo@intel.com>
likebreath added a commit to likebreath/cloud-hypervisor that referenced this issue May 27, 2021
Function "GuestMemory::with_regions(_mut)" were mainly temporary methods
to access the regions in `GuestMemory` as the lack of iterator-based
access, and hence they are deprecated in the upstream vm-memory crate [1].

[1] rust-vmm/vm-memory#133

Signed-off-by: Bo Chen <chen.bo@intel.com>
likebreath added a commit to likebreath/cloud-hypervisor that referenced this issue May 31, 2021
Function "GuestMemory::with_regions(_mut)" were mainly temporary methods
to access the regions in `GuestMemory` as the lack of iterator-based
access, and hence they are deprecated in the upstream vm-memory crate [1].

[1] rust-vmm/vm-memory#133

Signed-off-by: Bo Chen <chen.bo@intel.com>
likebreath added a commit to likebreath/cloud-hypervisor that referenced this issue Jun 1, 2021
Function "GuestMemory::with_regions(_mut)" were mainly temporary methods
to access the regions in `GuestMemory` as the lack of iterator-based
access, and hence they are deprecated in the upstream vm-memory crate [1].

[1] rust-vmm/vm-memory#133

Signed-off-by: Bo Chen <chen.bo@intel.com>
likebreath added a commit to likebreath/cloud-hypervisor that referenced this issue Jun 2, 2021
Function "GuestMemory::with_regions(_mut)" were mainly temporary methods
to access the regions in `GuestMemory` as the lack of iterator-based
access, and hence they are deprecated in the upstream vm-memory crate [1].

[1] rust-vmm/vm-memory#133

Signed-off-by: Bo Chen <chen.bo@intel.com>
rbradford pushed a commit to cloud-hypervisor/cloud-hypervisor that referenced this issue Jun 3, 2021
Function "GuestMemory::with_regions(_mut)" were mainly temporary methods
to access the regions in `GuestMemory` as the lack of iterator-based
access, and hence they are deprecated in the upstream vm-memory crate [1].

[1] rust-vmm/vm-memory#133

Signed-off-by: Bo Chen <chen.bo@intel.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants