chore: resolve copilot comments

Author: mayank-microsoft

opentmk/src/arch/aarch64/hypercall.rs

@@ -3,7 +3,7 @@
 
 //! Hypercall interface for AArch64 architecture.
 
-/// Writes a synthehtic register to tell the hypervisor the OS ID.
+/// Writes a synthetic  register to tell the hypervisor the OS ID.
 fn report_os_id(guest_os_id: u64) {
     // On ARM64, to be able to make hypercalls, one needs first to set the Guest OS ID
     // synthetic register using a hypercall. Can't use `Hvcall::set_register` at that will

opentmk/src/arch/x86_64/interrupt.rs

@@ -26,7 +26,7 @@ static MUTEX: Mutex<()> = Mutex::new(());
 fn no_op() {}
 
 fn common_handler(_stack_frame: InterruptStackFrame, interrupt: u8) {
-    // SAFETY: Handlers are initilizaed to no_op and only set via set_handler which is
+    // SAFETY: Handlers are initialized to no_op and only set via set_handler which is
     // protected by a mutex.
     unsafe {
         HANDLERS[interrupt as usize]();

opentmk/src/platform/hyperv/arch/x86_64/hypercall.rs

@@ -12,13 +12,13 @@ use zerocopy::IntoBytes;
 
 use crate::platform::hyperv::arch::hypercall::HvCall;
 
-// avoiding inline for debuggability in release builds
+// avoiding inline for debuggability in release builds.
 #[inline(never)]
 #[cfg(target_arch = "x86_64")] // xtask-fmt allow-target-arch cpu-intrinsic
 /// Invokes a hypercall specifically for switching to a VTL context.
 ///
 ///  # Safety
-///  The caller must ensure that the hypercall is invoked in a context where it is safe to do so
+///  The caller must ensure that the hypercall is invoked in a context where it is safe to do so.
 unsafe fn invoke_hypercall_vtl(control: hvdef::hypercall::Control) {
     // SAFETY: the caller guarantees the safety of this operation.
     unsafe {
@@ -189,7 +189,7 @@ impl HvCall {
         Ok(context)
     }
 
-    // avoiding inline for debuggability in release builds
+    // avoiding inline for debuggability in release builds.
     #[inline(never)]
     /// Invokes the VtlCall hypercall.
     pub(crate) fn vtl_call() {
@@ -200,7 +200,7 @@ impl HvCall {
         unsafe { invoke_hypercall_vtl(control) };
     }
 
-    // avoiding inline for debuggability in release builds
+    // avoiding inline for debuggability in release builds.
     #[inline(never)]
     /// Invokes the VtlReturn hypercall.
     pub(crate) fn vtl_return() {

opentmk/src/tests/hyperv/hv_error_vp_start.rs

@@ -14,7 +14,7 @@ pub fn exec<T>(ctx: &mut T)
 where
     T: VtlPlatformTrait + VirtualProcessorPlatformTrait<T>,
 {
-    // Skiping VTL setup for now to test the negitive case
+    // Skipping VTL setup for now to test the negative case
 
     let vp_count = ctx.get_vp_count();
     tmk_assert!(vp_count.is_ok(), "get_vp_count should succeed");

opentmk/src/tests/hyperv/test_helpers.rs

@@ -6,7 +6,7 @@
 macro_rules! create_function_with_restore {
     ($func_name:ident, $symbol:ident) => {
         #[inline(never)]
-        // avoiding inline for debuggability in release builds
+        // avoiding inline for debuggability in release builds.
         fn $func_name() {
             // SAFETY: we are calling a function pointer and restoring all registers.
             unsafe {

support/nostd_spin_channel/src/lib.rs

@@ -19,7 +19,7 @@ use thiserror::Error;
 
 /// An unbounded channel implementation with priority send capability.
 /// This implementation works in no_std environments using spin-rs.
-/// It uses a VecDeque as the underlying buffer
+/// It uses a VecDeque as the underlying buffer.
 pub struct Channel<T> {
     inner: Arc<ChannelInner<T>>,
 }