impl {Clone, Copy} where appropriate

Author: strake

src/addr.rs

@@ -37,7 +37,7 @@ pub struct PhysAddr(u64);
 /// a valid sign extension and are not null either. So automatic sign extension would have
 /// overwritten possibly meaningful bits. This likely indicates a bug, for example an invalid
 /// address calculation.
-#[derive(Debug)]
+#[derive(Debug, Clone, Copy)]
 pub struct VirtAddrNotValid(u64);
 
 impl VirtAddr {
@@ -245,7 +245,7 @@ impl Sub<VirtAddr> for VirtAddr {
 /// A passed `u64` was not a valid physical address.
 ///
 /// This means that bits 52 to 64 are not were not all null.
-#[derive(Debug)]
+#[derive(Debug, Clone, Copy)]
 pub struct PhysAddrNotValid(u64);
 
 impl PhysAddr {

src/instructions/port.rs

@@ -63,7 +63,7 @@ impl PortReadWrite for u32 {
 }
 
 /// An I/O port.
-#[derive(Debug)]
+#[derive(Debug, Clone, PartialEq, Eq)]
 pub struct Port<T: PortReadWrite> {
     port: u16,
     phantom: PhantomData<T>,

src/lib.rs

@@ -10,6 +10,7 @@
 #![cfg_attr(feature = "deny-warnings", deny(missing_docs))]
 #![cfg_attr(not(feature = "deny-warnings"), warn(missing_docs))]
 
+#![deny(missing_copy_implementations)]
 #![deny(missing_debug_implementations)]
 
 #[cfg(test)]

src/registers/control.rs

@@ -6,7 +6,7 @@ use structures::paging::PhysFrame;
 use PhysAddr;
 
 /// Various control flags modifying the basic operation of the CPU.
-#[derive(Debug)]
+#[derive(Debug, Clone, Copy)]
 pub struct Cr0;
 
 impl Cr0 {
@@ -92,7 +92,7 @@ bitflags! {
 }
 
 /// Contains the physical address of the level 4 page table.
-#[derive(Debug)]
+#[derive(Debug, Clone, Copy)]
 pub struct Cr3;
 
 impl Cr3 {

src/registers/model_specific.rs

@@ -1,7 +1,7 @@
 //! Functions to read and write control registers.
 
 /// A model specific register.
-#[derive(Debug)]
+#[derive(Debug, Clone, Copy)]
 pub struct Msr(u32);
 
 impl Msr {
@@ -26,7 +26,7 @@ impl Msr {
 }
 
 /// The Extended Feature Enable Register.
-#[derive(Debug)]
+#[derive(Debug, Clone, Copy)]
 pub struct Efer;
 
 impl Efer {

src/structures/gdt.rs

@@ -50,7 +50,7 @@ impl fmt::Debug for SegmentSelector {
 /// switching between user and kernel mode or for loading a TSS.
 ///
 /// The GDT has a fixed size of 8 entries, trying to add more entries will panic.
-#[derive(Debug)]
+#[derive(Debug, Clone, Copy)]
 pub struct GlobalDescriptorTable {
     table: [u64; 8],
     next_free: usize,
@@ -109,7 +109,7 @@ impl GlobalDescriptorTable {
 ///
 /// Segmentation is no longer supported in 64-bit mode, so most of the descriptor
 /// contents are ignored.
-#[derive(Debug)]
+#[derive(Debug, Clone, Copy)]
 pub enum Descriptor {
     /// Descriptor for a code or data segment.
     ///

src/structures/idt.rs

@@ -31,6 +31,7 @@ use {PrivilegeLevel, VirtAddr};
 /// [AMD64 manual volume 2](https://support.amd.com/TechDocs/24593.pdf)
 /// (with slight modifications).
 #[allow(missing_debug_implementations)]
+#[derive(Clone, Copy)]
 #[repr(C)]
 pub struct InterruptDescriptorTable {
     /// A divide by zero exception (`#DE`) occurs when the denominator of a DIV instruction or
@@ -637,6 +638,7 @@ impl EntryOptions {
 }
 
 /// Represents the exception stack frame pushed by the CPU on exception entry.
+#[derive(Clone, Copy)]
 #[repr(C)]
 pub struct ExceptionStackFrame {
     /// This value points to the instruction that should be executed when the interrupt

src/structures/paging/page_table.rs

@@ -18,6 +18,7 @@ pub enum FrameError {
 }
 
 /// A 64-bit page table entry.
+#[allow(missing_copy_implementations)]
 #[derive(Clone)]
 #[repr(transparent)]
 pub struct PageTableEntry {

src/structures/paging/recursive.rs

@@ -104,11 +104,11 @@ pub struct RecursivePageTable<'a> {
 /// An error indicating that the given page table is not recursively mapped.
 ///
 /// Returned from `RecursivePageTable::new`.
-#[derive(Debug)]
+#[derive(Debug, Clone, Copy)]
 pub struct NotRecursivelyMapped;
 
 /// This error is returned from `map_to` and similar methods.
-#[derive(Debug)]
+#[derive(Debug, Clone, Copy)]
 pub enum MapToError {
     /// An additional frame was needed for the mapping process, but the frame allocator
     /// returned `None`.
@@ -121,7 +121,7 @@ pub enum MapToError {
 }
 
 /// An error indicating that an `unmap` call failed.
-#[derive(Debug)]
+#[derive(Debug, Clone, Copy)]
 pub enum UnmapError {
     /// An upper level page table entry has the `HUGE_PAGE` flag set, which means that the
     /// given page is part of a huge page and can't be freed individually.
@@ -133,7 +133,7 @@ pub enum UnmapError {
 }
 
 /// An error indicating that an `update_flags` call failed.
-#[derive(Debug)]
+#[derive(Debug, Clone, Copy)]
 pub enum FlagUpdateError {
     /// The given page is not mapped to a physical frame.
     PageNotMapped,