aarch64: Remove versionize & versionize_derive

Remove the not-mandatory dependencies in gic source code.

Signed-off-by: Michael Zhao <michael.zhao@arm.com>
This commit is contained in:
Michael Zhao 2022-05-24 09:37:47 +08:00 committed by Xin Wang
parent ad8295e11d
commit 1e732bf532
2 changed files with 8 additions and 16 deletions

View File

@ -16,16 +16,14 @@ pub mod kvm {
use anyhow::anyhow; use anyhow::anyhow;
use hypervisor::kvm::kvm_bindings; use hypervisor::kvm::kvm_bindings;
use hypervisor::CpuState; use hypervisor::CpuState;
use serde::{Deserialize, Serialize};
use std::any::Any; use std::any::Any;
use std::convert::TryInto; use std::convert::TryInto;
use std::sync::Arc; use std::sync::Arc;
use std::{boxed::Box, result}; use std::{boxed::Box, result};
use versionize::{VersionMap, Versionize, VersionizeResult};
use versionize_derive::Versionize;
use vm_memory::Address; use vm_memory::Address;
use vm_migration::{ use vm_migration::{
Migratable, MigratableError, Pausable, Snapshot, Snapshottable, Transportable, Migratable, MigratableError, Pausable, Snapshot, Snapshottable, Transportable,
VersionMapped,
}; };
/// Errors thrown while saving/restoring the GICv3. /// Errors thrown while saving/restoring the GICv3.
@ -67,7 +65,7 @@ pub mod kvm {
vcpu_count: u64, vcpu_count: u64,
} }
#[derive(Versionize)] #[derive(Clone, Default, Serialize, Deserialize)]
pub struct Gicv3State { pub struct Gicv3State {
dist: Vec<u32>, dist: Vec<u32>,
rdist: Vec<u32>, rdist: Vec<u32>,
@ -76,8 +74,6 @@ pub mod kvm {
gicd_ctlr: u32, gicd_ctlr: u32,
} }
impl VersionMapped for Gicv3State {}
impl KvmGicV3 { impl KvmGicV3 {
// Device trees specific constants // Device trees specific constants
pub const ARCH_GIC_V3_MAINT_IRQ: u32 = 9; pub const ARCH_GIC_V3_MAINT_IRQ: u32 = 9;
@ -235,12 +231,12 @@ pub mod kvm {
fn snapshot(&mut self) -> std::result::Result<Snapshot, MigratableError> { fn snapshot(&mut self) -> std::result::Result<Snapshot, MigratableError> {
let gicr_typers = self.gicr_typers.clone(); let gicr_typers = self.gicr_typers.clone();
Snapshot::new_from_versioned_state(&self.id(), &self.state(&gicr_typers).unwrap()) Snapshot::new_from_state(&self.id(), &self.state(&gicr_typers).unwrap())
} }
fn restore(&mut self, snapshot: Snapshot) -> std::result::Result<(), MigratableError> { fn restore(&mut self, snapshot: Snapshot) -> std::result::Result<(), MigratableError> {
let gicr_typers = self.gicr_typers.clone(); let gicr_typers = self.gicr_typers.clone();
self.set_state(&gicr_typers, &snapshot.to_versioned_state(&self.id())?) self.set_state(&gicr_typers, &snapshot.to_state(&self.id())?)
.map_err(|e| { .map_err(|e| {
MigratableError::Restore(anyhow!("Could not restore GICv3 state {:?}", e)) MigratableError::Restore(anyhow!("Could not restore GICv3 state {:?}", e))
}) })

View File

@ -17,15 +17,13 @@ pub mod kvm {
use anyhow::anyhow; use anyhow::anyhow;
use hypervisor::kvm::kvm_bindings; use hypervisor::kvm::kvm_bindings;
use hypervisor::CpuState; use hypervisor::CpuState;
use serde::{Deserialize, Serialize};
use std::any::Any; use std::any::Any;
use std::convert::TryInto; use std::convert::TryInto;
use std::sync::Arc; use std::sync::Arc;
use std::{boxed::Box, result}; use std::{boxed::Box, result};
use versionize::{VersionMap, Versionize, VersionizeResult};
use versionize_derive::Versionize;
use vm_migration::{ use vm_migration::{
Migratable, MigratableError, Pausable, Snapshot, Snapshottable, Transportable, Migratable, MigratableError, Pausable, Snapshot, Snapshottable, Transportable,
VersionMapped,
}; };
const GITS_CTLR: u32 = 0x0000; const GITS_CTLR: u32 = 0x0000;
@ -161,7 +159,7 @@ pub mod kvm {
vcpu_count: u64, vcpu_count: u64,
} }
#[derive(Versionize)] #[derive(Clone, Default, Serialize, Deserialize)]
pub struct Gicv3ItsState { pub struct Gicv3ItsState {
dist: Vec<u32>, dist: Vec<u32>,
rdist: Vec<u32>, rdist: Vec<u32>,
@ -176,8 +174,6 @@ pub mod kvm {
its_baser: [u64; 8], its_baser: [u64; 8],
} }
impl VersionMapped for Gicv3ItsState {}
impl KvmGicV3Its { impl KvmGicV3Its {
fn get_msi_size() -> u64 { fn get_msi_size() -> u64 {
layout::GIC_V3_ITS_SIZE layout::GIC_V3_ITS_SIZE
@ -480,12 +476,12 @@ pub mod kvm {
fn snapshot(&mut self) -> std::result::Result<Snapshot, MigratableError> { fn snapshot(&mut self) -> std::result::Result<Snapshot, MigratableError> {
let gicr_typers = self.gicr_typers.clone(); let gicr_typers = self.gicr_typers.clone();
Snapshot::new_from_versioned_state(&self.id(), &self.state(&gicr_typers).unwrap()) Snapshot::new_from_state(&self.id(), &self.state(&gicr_typers).unwrap())
} }
fn restore(&mut self, snapshot: Snapshot) -> std::result::Result<(), MigratableError> { fn restore(&mut self, snapshot: Snapshot) -> std::result::Result<(), MigratableError> {
let gicr_typers = self.gicr_typers.clone(); let gicr_typers = self.gicr_typers.clone();
self.set_state(&gicr_typers, &snapshot.to_versioned_state(&self.id())?) self.set_state(&gicr_typers, &snapshot.to_state(&self.id())?)
.map_err(|e| { .map_err(|e| {
MigratableError::Restore(anyhow!("Could not restore GICv3ITS state {:?}", e)) MigratableError::Restore(anyhow!("Could not restore GICv3ITS state {:?}", e))
}) })