External/cloud-hypervisor
1
0
Fork 0
mirror of https://github.com/cloud-hypervisor/cloud-hypervisor.git synced 2024-10-02 11:35:46 +00:00
Code Issues Packages Projects Releases Wiki Activity

arch: x86_64: Disable dead code detection for embedded struct

Browse Source 
These structs directly embed another struct and then implement
ByteValued on that struct to implement ByteValued for the inner struct.
As such the inner struct is never directly accessed so to avoid the dead
code analysis mark this as allowed.

Beta clippy fix:

warning: field `0` is never read
   --> arch/src/x86_64/mod.rs:129:32
    |
129 | struct MemmapTableEntryWrapper(hvm_memmap_table_entry);
    |        ----------------------- ^^^^^^^^^^^^^^^^^^^^^^
    |        |
    |        field in this struct
    |
    = note: `MemmapTableEntryWrapper` has a derived impl for the trait `Clone`, but this is intentionally ignored during dead code analysis
    = note: `#[warn(dead_code)]` on by default
help: consider changing the field to be of unit type to suppress this warning while preserving the field numbering, or remove the field
    |
129 | struct MemmapTableEntryWrapper(());
    |

Signed-off-by: Rob Bradford <rbradford@rivosinc.com>
This commit is contained in:
Rob Bradford 2024-02-06 21:08:32 +00:00
parent 61afd93a50
commit 4e0dc5203a
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
arch/src/x86_64/mod.rs
View File

@ -124,11 +124,11 @@ impl SgxEpcRegion {
#[derive(Copy, Clone, Default)]
struct StartInfoWrapper(hvm_start_info);
#[cfg_attr(fuzzing, allow(dead_code))]
#[allow(dead_code)]
#[derive(Copy, Clone, Default)]
struct MemmapTableEntryWrapper(hvm_memmap_table_entry);
#[cfg_attr(fuzzing, allow(dead_code))]
#[allow(dead_code)]
#[derive(Copy, Clone, Default)]
struct ModlistEntryWrapper(hvm_modlist_entry);
@ -144,7 +144,7 @@ unsafe impl ByteValued for ModlistEntryWrapper {}
// * the type that is implementing the trait is foreign or
// * all of the parameters being passed to the trait (if there are any) are also foreign
// is prohibited.
#[cfg_attr(fuzzing, allow(dead_code))]
#[allow(dead_code)]
#[derive(Copy, Clone, Default)]
struct BootParamsWrapper(boot_params);